Simplify bundle handling for uriHandler (#1832)

- Move the bundle building to a postbuild command that always runs
instead of people having to run build:bundle.
- Consolidate the bundle-nodescript.ps1 logic into build.ps1

Author: hillary-mutisya

dotnet/agentLauncher/src/Build.ps1

@@ -12,6 +12,12 @@ $projectFile = Join-Path $scriptDir "AgentLauncher.csproj"
 $configuration = "Debug"
 $platform = "x64"
 
+# Calculate paths for uriHandler bundle
+$uriHandlerPath = Join-Path $scriptDir "..\..\..\ts\packages\uriHandler" | Resolve-Path -ErrorAction SilentlyContinue
+$bundleOutput = if ($uriHandlerPath) { Join-Path $uriHandlerPath "bundle\agent-uri-handler.bundle.js" } else { $null }
+$targetDir = Join-Path $scriptDir "Scripts"
+$targetPath = Join-Path $targetDir "agent-uri-handler.bundle.js"
+
 # Find MSBuild
 $msbuildPath2022 = "C:\Program Files\Microsoft Visual Studio\2022\Enterprise\MSBuild\Current\Bin\MSBuild.exe";
 $msbuildPath2026 = "C:\Program Files\Microsoft Visual Studio\18\Enterprise\MSBuild\Current\Bin\MSBuild.exe";
@@ -34,14 +40,46 @@ Write-Host "  Configuration: $configuration"
 Write-Host "  Platform: $platform"
 Write-Host ""
 
+# Copy bundle from uriHandler
+Write-Host "Step 1: Copying uriHandler bundle..." -ForegroundColor Yellow
+if (-not $bundleOutput -or -not (Test-Path $bundleOutput)) {
+    Write-Host ""
+    Write-Host "ERROR: Bundle file not found at: $bundleOutput" -ForegroundColor Red
+    Write-Host ""
+    Write-Host "You must first build the uriHandler package:" -ForegroundColor Yellow
+    if ($uriHandlerPath) {
+        Write-Host "  1. Navigate to: $uriHandlerPath" -ForegroundColor Cyan
+    } else {
+        Write-Host "  1. Navigate to the uriHandler package directory" -ForegroundColor Cyan
+    }
+    Write-Host "  2. Run: pnpm run build" -ForegroundColor Cyan
+    Write-Host ""
+    exit 1
+}
+
+$bundleSize = (Get-Item $bundleOutput).Length
+$bundleSizeKB = [Math]::Round($bundleSize / 1KB, 2)
+Write-Host "  Bundle found: $bundleSizeKB KB at $bundleOutput" -ForegroundColor Green
+
+New-Item -ItemType Directory -Force -Path $targetDir | Out-Null
+Copy-Item $bundleOutput $targetPath -Force
+
+if (Test-Path $targetPath) {
+    Write-Host "  Bundle copied successfully to: $targetPath" -ForegroundColor Green
+} else {
+    Write-Host "  ERROR: Failed to copy bundle" -ForegroundColor Red
+    exit 1
+}
+Write-Host ""
+
 # Clean previous build
-Write-Host "Step 1: Cleaning previous build..." -ForegroundColor Yellow
+Write-Host "Step 2: Cleaning previous build..." -ForegroundColor Yellow
 Remove-Item -Recurse -Force obj,bin -ErrorAction SilentlyContinue
 Write-Host "  Clean complete" -ForegroundColor Green
 Write-Host ""
 
 # Build
-Write-Host "Step 2: Building project..." -ForegroundColor Yellow
+Write-Host "Step 3: Building project..." -ForegroundColor Yellow
 $buildArgs = @(
     $projectFile,
     "-t:Restore,Build",
@@ -62,7 +100,7 @@ Write-Host "  Build complete" -ForegroundColor Green
 Write-Host ""
 
 # Package
-Write-Host "Step 3: Creating MSIX package..." -ForegroundColor Yellow
+Write-Host "Step 4: Creating MSIX package..." -ForegroundColor Yellow
 $packageArgs = @(
     $projectFile,
     "-t:_GenerateAppxPackage",

dotnet/agentLauncher/src/Bundle-NodeScript.ps1

@@ -12,6 +12,7 @@ pnpm-lock.yaml
 .rollup.cache/**
 .tsc-cache-*.json
 .frontend-build-cache.json
+packages/uriHandler/bundle/agent-uri-handler.bundle.js
 
 .scratch
 

ts/.prettierignore

@@ -7,9 +7,6 @@ import { dirname, join } from "path";
 
 const __dirname = dirname(fileURLToPath(import.meta.url));
 
-console.log("TypeAgent URI Handler - Bundle Configuration");
-console.log("===========================================\n");
-
 const config = {
     entryPoints: [join(__dirname, "dist", "index.js")],
     bundle: true,
@@ -45,14 +42,6 @@ const require = createRequire(import.meta.url);
     ],
 };
 
-console.log("Configuration:");
-console.log(`  Entry: ${config.entryPoints[0]}`);
-console.log(`  Output: ${config.outfile}`);
-console.log(`  Platform: ${config.platform}`);
-console.log(`  Format: ${config.format}`);
-console.log(`  Target: ${config.target}`);
-console.log("");
-
 try {
     await esbuild.build(config);
     console.log("\n✓ Bundle created successfully");

ts/packages/uriHandler/bundle-config.js

@@ -18,8 +18,7 @@
   },
   "scripts": {
     "build": "npm run tsc",
-    "build:bundle": "npm run build && npm run bundle",
-    "bundle": "node bundle-config.js",
+    "postbuild": "node bundle-config.js",
     "clean": "rimraf --glob dist bundle *.tsbuildinfo *.done.build.log",
     "prettier": "prettier --check . --ignore-path ../../.prettierignore",
     "prettier:fix": "prettier --write . --ignore-path ../../.prettierignore",