Merge pull request #15 from posit-dev/switch-to-upstream-0.37.5

Switch upstream from v0.38 to v0.37.5

Author: sharon-wang

.claude/CLAUDE.md

@@ -0,0 +1,7 @@
+# Claude Instructions
+
+## General Repository Instructions
+
+For all repository instructions, coding standards, and development guidelines, **ALWAYS** read:
+
+**[.github/copilot-instructions.md](../.github/copilot-instructions.md)**

.claude/CLAUDE.md

@@ -6,25 +6,21 @@
 import * as watcher from '@parcel/watcher';
 import * as esbuild from 'esbuild';
 import * as fs from 'fs';
-import { copyFile, mkdir, readdir, rename } from 'fs/promises';
+import { copyFile, mkdir } from 'fs/promises';
 import { glob } from 'glob';
 import * as path from 'path';
 
 const REPO_ROOT = import.meta.dirname;
 const isWatch = process.argv.includes('--watch');
 const isDev = process.argv.includes('--dev');
 const isPreRelease = process.argv.includes('--prerelease');
-const generateSourceMaps = process.argv.includes('--sourcemaps');
-const sourceMapOutDir = './dist-sourcemaps';
 
 const baseBuildOptions = {
 	bundle: true,
 	logLevel: 'info',
 	minify: !isDev,
 	outdir: './dist',
-	// In dev mode, use linked source maps for debugging.
-	// With --sourcemaps flag, generate external source maps (no sourceMappingURL comment in output).
-	sourcemap: isDev ? 'linked' : (generateSourceMaps ? 'external' : false),
+	sourcemap: isDev ? 'linked' : false,
 	sourcesContent: false,
 	treeShaking: true
 } satisfies esbuild.BuildOptions;
@@ -287,42 +283,6 @@ const typeScriptServerPluginBuildOptions = {
 	]
 } satisfies esbuild.BuildOptions;
 
-/**
- * Moves all .map files from the output directories to a separate source maps directory.
- * This keeps source maps out of the packaged extension while making them available for upload.
- */
-async function moveSourceMapsToSeparateDir(): Promise<void> {
-	if (!generateSourceMaps) {
-		return;
-	}
-
-	const outputDirs = [
-		'./dist',
-		'./node_modules/@vscode/copilot-typescript-server-plugin/dist',
-	];
-
-	await mkdir(sourceMapOutDir, { recursive: true });
-
-	for (const dir of outputDirs) {
-		try {
-			const files = await readdir(dir);
-			for (const file of files) {
-				if (file.endsWith('.map')) {
-					const sourcePath = path.join(dir, file);
-					// Prefix with directory name to avoid collisions
-					const prefix = dir === './dist' ? '' : 'ts-plugin-';
-					const destPath = path.join(sourceMapOutDir, prefix + file);
-					await rename(sourcePath, destPath);
-					console.log(`Moved source map: ${sourcePath} -> ${destPath}`);
-				}
-			}
-		} catch (error) {
-			// Directory might not exist in some build configurations
-			console.warn(`Could not process directory ${dir}:`, error);
-		}
-	}
-}
-
 async function main() {
 	if (!isDev) {
 		applyPackageJsonPatch(isPreRelease);
@@ -408,9 +368,6 @@ async function main() {
 			esbuild.build(typeScriptServerPluginBuildOptions),
 			esbuild.build(webviewBuildOptions),
 		]);
-
-		// Move source maps to separate directory so they're not packaged with the extension
-		await moveSourceMapsToSeparateDir();
 	}
 }
 

.esbuild.ts

@@ -337,9 +337,5 @@ The extension uses numerous proposed VS Code APIs for advanced functionality:
 This extension is a complex, multi-layered system that provides comprehensive AI assistance within VS Code. Understanding the service architecture, contribution system, and separation between platform and extension layers is crucial for making effective changes.
 
 ## Best Practices
-- Use services and dependency injection over VS Code extension APIs when possible:
-  - Use `IFileSystemService` instead of Node's `fs` or `vscode.workspace.fs`
-  - Use `ILogService` instead of `console.log`
-  - Look for existing `I*Service` interfaces before reaching for raw APIs
-  - **Why**: Enables unit testing without VS Code host, supports simulation tests, provides cross-platform abstractions (Node vs web), and adds features like caching and size limits
+- Use services and dependency injection whenever possible instead of using node or vscode APIs directly. For example, use `IFileSystemService` instead of node's `fs`.
 - Always use the URI type instead of using string file paths. There are many helpers available for working with URIs.

.github/copilot-instructions.md

@@ -3,7 +3,6 @@
 .vscode-test-web/
 node_modules/
 dist/
-dist-sourcemaps/
 
 # created by simulation
 .simulation

.gitignore

@@ -35,16 +35,11 @@ parameters:
     displayName: 🚀 Publish Pre-Release
     type: boolean
     default: false
-  - name: processLocalization
-    displayName: 🌐 Process Localization
-    type: boolean
-    default: true
 
 extends:
   template: azure-pipelines/extension/pre-release.yml@templates
   parameters:
     l10nSourcePaths: ./src
-    l10nShouldProcess: ${{ parameters.processLocalization }}
     nodeVersion: 22.21.x
     standardizedVersioning: true
 
@@ -131,9 +126,6 @@ extends:
       - script: npm run build -- --prerelease
         displayName: npm run build
 
-    uploadSourceMaps:
-      enabled: true
-
     # testPlatforms:
     #     - name: Linux
     #       nodeVersions: [16.x]

build/pre-release.yml

@@ -117,9 +117,6 @@ extends:
       - script: npm run build
         displayName: npm run build
 
-    uploadSourceMaps:
-      enabled: true
-
     # testPlatforms:
     #     - name: Linux
     #       nodeVersions: [16.x]

build/release.yml

@@ -16,7 +16,7 @@
 	"dependencies": {
 		"@microsoft/tiktokenizer": "^1.0.10",
 		"@sinclair/typebox": "^0.34.41",
-		"@vscode/copilot-api": "^0.2.12",
+		"@vscode/copilot-api": "^0.2.8",
 		"@vscode/l10n": "^0.0.18",
 		"@vscode/prompt-tsx": "^0.4.0-alpha.6",
 		"@vscode/tree-sitter-wasm": "0.0.5-php.2",