claude: Simplify import map and add external dependencies support

Remove Deno std entries from import map since esbuild cannot bundle
external URL schemes (jsr:, npm:, https:). Add externals support for
marking imports as external (not bundled).

Changes to quarto-cli:
- Simplify import-map.json: only @quarto/types remains
- Simplify updateImportMap(): just path transformation, no version syncing
- Add externals?: string[] to quartoExtension interface
- Implement --external flag handling in bundle function
- Add default externals to deno.json: ["jsr:*", "npm:*", "https:*", "http:*"]

Rationale:
- esbuild binary only bundles relative file paths
- External URL schemes must be resolved at runtime by Deno
- Import map now serves single purpose: @quarto/types aliasing
- Extension authors use full JSR URLs in source code
- Default externals cover all non-bundleable schemes

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>

Author: gordonwoodhull

package/src/common/prepare-dist.ts

@@ -237,32 +237,8 @@ function updateImportMap(config: Configuration) {
   );
   const importMapContent = JSON.parse(Deno.readTextFileSync(importMapPath));
 
-  // Read the source import map to get current Deno std versions
-  const sourceImportMapPath = join(config.directoryInfo.src, "import_map.json");
-  const sourceImportMap = JSON.parse(Deno.readTextFileSync(sourceImportMapPath));
-  const sourceImports = sourceImportMap.imports as Record<string, string>;
-
-  // Update the import map for distribution:
-  // 1. Change @quarto/types path from dev (../../../packages/...) to dist (./quarto-types.d.ts)
-  // 2. Update all other imports (Deno std versions) from source import map
-  const updatedImports: Record<string, string> = {
-    "@quarto/types": "./quarto-types.d.ts",
-  };
-
-  // Copy all other imports from source, updating versions
-  for (const key in importMapContent.imports) {
-    if (key !== "@quarto/types") {
-      const sourceValue = sourceImports[key];
-      if (!sourceValue) {
-        throw new Error(
-          `Import map key "${key}" not found in source import_map.json`,
-        );
-      }
-      updatedImports[key] = sourceValue;
-    }
-  }
-
-  importMapContent.imports = updatedImports;
+  // Update @quarto/types path from dev (../../../packages/...) to dist (./quarto-types.d.ts)
+  importMapContent.imports["@quarto/types"] = "./quarto-types.d.ts";
 
   // Write back the updated import map
   Deno.writeTextFileSync(

src/command/dev-call/build-ts-extension/cmd.ts

@@ -25,6 +25,7 @@ interface DenoConfig {
     minify?: boolean;
     sourcemap?: boolean;
     target?: string;
+    externals?: string[];
   };
 }
 
@@ -281,6 +282,13 @@ async function bundle(
     args.push("--sourcemap");
   }
 
+  // Add external dependencies (not bundled)
+  if (config.quartoExtension?.externals) {
+    for (const external of config.quartoExtension.externals) {
+      args.push(`--external:${external}`);
+    }
+  }
+
   const result = await execProcess({
     cmd: esbuildBinary,
     args,

src/resources/extension-build/deno.json

@@ -3,5 +3,8 @@
     "strict": true,
     "lib": ["deno.ns", "DOM", "ES2021"]
   },
-  "importMap": "./import-map.json"
+  "importMap": "./import-map.json",
+  "quartoExtension": {
+    "externals": ["jsr:*", "npm:*", "https:*", "http:*"]
+  }
 }

src/resources/extension-build/import-map.json

@@ -1,11 +1,5 @@
 {
   "imports": {
-    "@quarto/types": "../../../packages/quarto-types/dist/index.d.ts",
-    "path": "jsr:@std/path@1.0.8",
-    "path/posix": "jsr:@std/path@1.0.8/posix",
-    "log": "jsr:/@std/log@0.224.0",
-    "log/": "jsr:/@std/log@0.224.0/",
-    "fs/": "jsr:/@std/fs@1.0.16/",
-    "encoding/": "jsr:/@std/encoding@1.0.9/"
+    "@quarto/types": "../../../packages/quarto-types/dist/index.d.ts"
   }
 }