Jun 16, 2025 · Jun 16, 2025 · Jun 16, 2025 · Jun 16, 2025 · Jun 16, 2025 · Jun 16, 2025
diff --git a/.eslintignore b/.eslintignore
 vitest.config.ts
diff --git a/.eslintrc.json b/.eslintrc.json
 "parser": "@typescript-eslint/parser",
 "parserOptions": {
 "ecmaVersion": 6,
 "sourceType": "module"
 "sourceType": "module",
 "project": "./tsconfig.json"
 },
 "plugins": ["@typescript-eslint", "prettier"],
 "extends": [
 "overrides": [
 {
 "files": ["*.md"],
 "parser": "markdown-eslint-parser"
 "parser": "markdown-eslint-parser",
 "rules": {
 "@typescript-eslint/require-await": "off"
 }
 }
 ],
 "rules": {
 "sublings_only": true
 }
 }
 ]
 ],
 "require-await": "off",
    "@typescript-eslint/require-await": "error"
 },
 "ignorePatterns": ["out", "dist", "**/*.d.ts"]
 }
diff --git a/.github/workflows/ci.yaml b/.github/workflows/ci.yaml

      -uses:actions/setup-node@v4
 with:
 node-version:"18"
 node-version:"22"

      -run:yarn


      -uses:actions/setup-node@v4
 with:
 node-version:"18"
 node-version:"22"

      -run:yarn

diff --git a/.github/workflows/release.yaml b/.github/workflows/release.yaml

      - uses: actions/setup-node@v4
        with:
          node-version: "18"
          node-version: "22"

      - run: yarn

diff --git a/.vscode-test.mjs b/.vscode-test.mjs
 import { defineConfig } from "@vscode/test-cli";

 export default defineConfig({
 files: "out/test/**/*.test.js",
 extensionDevelopmentPath: ".",
 extensionTestsPath: "./out/test",
 launchArgs: ["--enable-proposed-api", "coder.coder-remote"],
 mocha: {
 ui: "tdd",
 timeout: 20000,
 },
 });
diff --git a/.vscodeignore b/.vscodeignore
 **/.editorconfig
 **/*.map
 **/*.ts
 *.gif
 *.gif
diff --git a/CLAUDE.md b/CLAUDE.md
 - Run all tests: `yarn test`
 - Run specific test: `vitest ./src/filename.test.ts`
 - CI test mode: `yarn test:ci`
 - Integration tests: `yarn test:integration`

 ## Code Style Guidelines

diff --git a/package.json b/package.json
 "package:prerelease": "npx vsce package --pre-release",
 "lint": "eslint . --ext ts,md",
 "lint:fix": "yarn lint --fix",
 "test": "vitest ./src",
 "test:ci": "CI=true yarn test"
 "test": "vitest",
 "test:ci": "CI=true yarn test",
 "test:integration": "vscode-test",
 "pretest": "yarn run compile-tests && yarn run build && yarn run lint",
 "compile-tests": "tsc -p . --outDir out"
 },
 "devDependencies": {
 "@types/eventsource": "^3.0.0",
 "vitest": "^0.34.6",
 "vscode-test": "^1.5.0",
 "webpack": "^5.99.6",
 "webpack-cli": "^5.1.4"
 "webpack-cli": "^5.1.4",
 "@vscode/test-cli": "^0.0.10"
 },
 "dependencies": {
 "axios": "1.8.4",
 "semver": "^7.7.1",
 "ua-parser-js": "1.0.40",
 "ws": "^8.18.2",
 "zod": "^3.25.1"
 "zod": "^3.25.65"
 },
 "resolutions": {
 "semver": "7.7.1",
diff --git a/src/api.ts b/src/api.ts
 * configuration.  The token may be undefined if some other form of
 * authentication is being used.
 */
 exportasyncfunction makeCoderSdk(
 export function makeCoderSdk(
 baseUrl: string,
 token: string | undefined,
 storage: Storage,
 ):Promise<Api> {
 ): Api {
 const restClient = new Api();
 restClient.setHost(baseUrl);
 if (token) {
diff --git a/src/error.ts b/src/error.ts
 }

 // allowInsecure updates the value of the "coder.insecure" property.
 asyncallowInsecure():Promise<void> {
 allowInsecure(): void {
 vscode.workspace
 .getConfiguration()
 .update("coder.insecure", true, vscode.ConfigurationTarget.Global);
diff --git a/src/extension.ts b/src/extension.ts
 //
 // Cursor and VSCode are covered by ms remote, and the only other is windsurf for now
 // Means that vscodium is not supported by this for now

 const remoteSSHExtension =
 vscode.extensions.getExtension("jeanp413.open-remote-ssh") ||
 vscode.extensions.getExtension("codeium.windsurf-remote-openssh") ||
 vscode.extensions.getExtension("anysphere.remote-ssh") ||
 vscode.extensions.getExtension("ms-vscode-remote.remote-ssh");

 let vscodeProposed: typeof vscode = vscode;

 if (!remoteSSHExtension) {
 vscode.window.showErrorMessage(
 "Remote SSH extension not found, cannot activate Coder extension",
 "Remote SSH extension not found, this may not work as expected.\n" +
 // NB should we link to documentation or marketplace?
 "Please install your choice of Remote SSH extension from the VS Code Marketplace.",
 );
 } else {
 // eslint-disable-next-line @typescript-eslint/no-explicit-any
 vscodeProposed = (module as any)._load(
 "vscode",
 {
 filename: remoteSSHExtension.extensionPath,
 },
 false,
 );
 throw new Error("Remote SSH extension not found");
 }
 // eslint-disable-next-line @typescript-eslint/no-explicit-any
 const vscodeProposed: typeof vscode = (module as any)._load(
 "vscode",
 {
 filename: remoteSSHExtension?.extensionPath,
 },
 false,
 );

 const output = vscode.window.createOutputChannel("Coder");
 const storage = new Storage(
 // Since the "onResolveRemoteAuthority:ssh-remote" activation event exists
 // in package.json we're able to perform actions before the authority is
 // resolved by the remote SSH extension.
 if (vscodeProposed.env.remoteAuthority) {
 //
 // In addition, if we don't have a remote SSH extension, we skip this
 // activation event. This may allow the user to install the extension
 // after the Coder extension is installed, instead of throwing a fatal error
 // (this would require the user to uninstall the Coder extension and
 // reinstall after installing the remote SSH extension, which is annoying)
 if (remoteSSHExtension && vscodeProposed.env.remoteAuthority) {
 const remote = new Remote(
 vscodeProposed,
 storage,
diff --git a/src/remote.ts b/src/remote.ts
 .then((content) => {
 return JSON.parse(content);
 })
 .then((parsed) => {
 .then(async(parsed) => {
 try {
 updateStatus(parsed);
 awaitupdateStatus(parsed);
 } catch (ex) {
 // Ignore
 }
diff --git a/src/test/extension.test.ts b/src/test/extension.test.ts
 import * as assert from "assert";
 import * as vscode from "vscode";

 suite("Extension Test Suite", () => {
 vscode.window.showInformationMessage("Start all tests.");

 test("Extension should be present", () => {
 assert.ok(vscode.extensions.getExtension("coder.coder-remote"));
 });

 test("Extension should activate", async () => {
 const extension = vscode.extensions.getExtension("coder.coder-remote");
 assert.ok(extension);

 if (!extension.isActive) {
 await extension.activate();
 }

 assert.ok(extension.isActive);
 });

 test("Extension should export activate function", async () => {
 const extension = vscode.extensions.getExtension("coder.coder-remote");
 assert.ok(extension);

 await extension.activate();
 // The extension doesn't export anything, which is fine
 // The test was expecting exports.activate but the extension
 // itself is the activate function
 assert.ok(extension.isActive);
 });

 test("Commands should be registered", async () => {
 const extension = vscode.extensions.getExtension("coder.coder-remote");
 assert.ok(extension);

 if (!extension.isActive) {
 await extension.activate();
 }

 // Give a small delay for commands to register
 await new Promise((resolve) => setTimeout(resolve, 100));

 const commands = await vscode.commands.getCommands(true);
 const coderCommands = commands.filter((cmd) => cmd.startsWith("coder."));

 assert.ok(
 coderCommands.length > 0,
 "Should have registered Coder commands",
 );
 assert.ok(
 coderCommands.includes("coder.login"),
 "Should have coder.login command",
 );
 });
 });
diff --git a/src/util.test.ts b/src/util.test.ts
 import { describe, it, expect } from "vitest";
 import { countSubstring, parseRemoteAuthority, toSafeHost } from "./util";

 it("ignore unrelated authorities",async() => {
 it("ignore unrelated authorities", () => {
 const tests = [
 "vscode://ssh-remote+some-unrelated-host.com",
 "vscode://ssh-remote+coder-vscode",
 }
 });

 it("should error on invalid authorities",async() => {
 it("should error on invalid authorities", () => {
 const tests = [
 "vscode://ssh-remote+coder-vscode--foo",
 "vscode://ssh-remote+coder-vscode--",
 }
 });

 it("should parse authority",async() => {
 it("should parse authority", () => {
 expect(
 parseRemoteAuthority("vscode://ssh-remote+coder-vscode--foo--bar"),
 ).toStrictEqual({
 });
 });

 it("escapes url host",async() => {
 it("escapes url host", () => {
 expect(toSafeHost("https://foobar:8080")).toBe("foobar");
 expect(toSafeHost("https://ほげ")).toBe("xn--18j4d");
 expect(toSafeHost("https://test.😉.invalid")).toBe("test.xn--n28h.invalid");
diff --git a/src/util.ts b/src/util.ts
 *
 * Returns null if no port is found.
 */
 exportasyncfunction findPort(text: string):Promise<number | null> {
 export function findPort(text: string): number | null {
 const matches = text.match(RemoteSSHLogPortRegex);
 if (!matches) {
 return null;
diff --git a/src/workspacesProvider.ts b/src/workspacesProvider.ts

 // Create tree items for each workspace
 const workspaceTreeItems = await Promise.all(
 resp.workspaces.map(async(workspace) => {
 resp.workspaces.map((workspace) => {
 const workspaceTreeItem = new WorkspaceTreeItem(
 workspace,
 this.getWorkspacesQuery === WorkspaceQuery.All,
 this._onDidChangeTreeData.fire(item);
 }

 asyncgetTreeItem(element: vscode.TreeItem):Promise<vscode.TreeItem> {
 getTreeItem(element: vscode.TreeItem): vscode.TreeItem {
 return element;
 }

diff --git a/tsconfig.json b/tsconfig.json
 {
 "compilerOptions": {
 "module": "commonjs",
 "target": "es6",
 "target": "ES2022",
 "moduleResolution": "node",
 "outDir": "out",
 // "dom" is required for importing the API from coder/coder.
 "lib": ["es6", "dom"],
 "lib": ["ES2022", "dom"],
 "sourceMap": true,
 "rootDirs": ["node_modules", "src"],
 "strict": true,
 "esModuleInterop": true
 "esModuleInterop": true,
 "skipLibCheck": true,
 "forceConsistentCasingInFileNames": true
 },
 "exclude": ["node_modules", ".vscode-test"]
 "exclude": ["node_modules", "vitest.config.ts"],
 "include": ["src/**/*"]
 }
diff --git a/vitest.config.ts b/vitest.config.ts
 import { defineConfig } from "vitest/config";

 export default defineConfig({
 test: {
 include: ["src/**/*.test.ts"],
 exclude: [
 "**/node_modules/**",
 "**/dist/**",
 "**/build/**",
 "**/out/**",
 "**/src/test/**",
 "src/test/**",
 "./src/test/**",
 ],
 environment: "node",
 },
 });
Original file line number	Diff line number	Diff line change
Expand Up		@@ -3,7 +3,8 @@
		"parser": "@typescript-eslint/parser",
		"parserOptions": {
		"ecmaVersion": 6,
		"sourceType": "module"
		"sourceType": "module",
		"project": "./tsconfig.json"
		},
		"plugins": ["@typescript-eslint", "prettier"],
		"extends": [
Expand All		@@ -17,7 +18,10 @@
		"overrides": [
		{
		"files": ["*.md"],
		"parser": "markdown-eslint-parser"
		"parser": "markdown-eslint-parser",
		"rules": {
		"@typescript-eslint/require-await": "off"
		}
		}
		],
		"rules": {
Expand DownExpand Up		@@ -54,7 +58,9 @@
		"sublings_only": true
		}
		}
		]
		],
		"require-await": "off",
		"@typescript-eslint/require-await": "error"
		},
		"ignorePatterns": ["out", "dist", "*/.d.ts"]
		}
Original file line number	Diff line number	Diff line change
Expand Up		@@ -18,7 +18,7 @@ jobs:

		-uses:actions/setup-node@v4
		with:
		node-version:"18"
		node-version:"22"

		-run:yarn

Expand All		@@ -36,7 +36,7 @@ jobs:

		-uses:actions/setup-node@v4
		with:
		node-version:"18"
		node-version:"22"

		-run:yarn

Expand Down
Original file line number	Diff line number	Diff line change
Expand Up		@@ -18,7 +18,7 @@ jobs:

		- uses: actions/setup-node@v4
		with:
		node-version: "18"
		node-version: "22"

		- run: yarn

Expand Down
Original file line number	Diff line number	Diff line change
		@@ -0,0 +1,12 @@
		import { defineConfig } from "@vscode/test-cli";

		export default defineConfig({
		files: "out/test/*/.test.js",
		extensionDevelopmentPath: ".",
		extensionTestsPath: "./out/test",
		launchArgs: ["--enable-proposed-api", "coder.coder-remote"],
		mocha: {
		ui: "tdd",
		timeout: 20000,
		},
		});
Original file line number	Diff line number	Diff line change
Expand Up		@@ -12,4 +12,4 @@ node_modules/**
		**/.editorconfig
		*/.map
		*/.ts
		*.gif
		*.gif
Original file line number	Diff line number	Diff line change
Expand Up		@@ -10,6 +10,7 @@
		- Run all tests: `yarn test`
		- Run specific test: `vitest ./src/filename.test.ts`
		- CI test mode: `yarn test:ci`
		- Integration tests: `yarn test:integration`

		## Code Style Guidelines

Expand Down
Original file line number	Diff line number	Diff line change
Expand Up		@@ -278,8 +278,11 @@
		"package:prerelease": "npx vsce package --pre-release",
		"lint": "eslint . --ext ts,md",
		"lint:fix": "yarn lint --fix",
		"test": "vitest ./src",
		"test:ci": "CI=true yarn test"
		"test": "vitest",
		"test:ci": "CI=true yarn test",
		"test:integration": "vscode-test",
		"pretest": "yarn run compile-tests && yarn run build && yarn run lint",
		"compile-tests": "tsc -p . --outDir out"
		},
		"devDependencies": {
		"@types/eventsource": "^3.0.0",
Expand DownExpand Up		@@ -311,7 +314,8 @@
		"vitest": "^0.34.6",
		"vscode-test": "^1.5.0",
		"webpack": "^5.99.6",
		"webpack-cli": "^5.1.4"
		"webpack-cli": "^5.1.4",
		"@vscode/test-cli": "^0.0.10"
		},
		"dependencies": {
		"axios": "1.8.4",
Expand All		@@ -326,7 +330,7 @@
		"semver": "^7.7.1",
		"ua-parser-js": "1.0.40",
		"ws": "^8.18.2",
		"zod": "^3.25.1"
		"zod": "^3.25.65"
		},
		"resolutions": {
		"semver": "7.7.1",
Expand Down
Original file line number	Diff line number	Diff line change
Expand Up		@@ -71,11 +71,11 @@ export async function createHttpAgent(): Promise<ProxyAgent> {
		* configuration. The token may be undefined if some other form of
		* authentication is being used.
		*/
		exportasyncfunction makeCoderSdk(
		export function makeCoderSdk(
		baseUrl: string,
		token: string \| undefined,
		storage: Storage,
		):Promise<Api> {
		): Api {
		const restClient = new Api();
		restClient.setHost(baseUrl);
		if (token) {
Expand Down
Original file line number	Diff line number	Diff line change
Expand Up		@@ -126,7 +126,7 @@ export class CertificateError extends Error {
		}

		// allowInsecure updates the value of the "coder.insecure" property.
		asyncallowInsecure():Promise<void> {
		allowInsecure(): void {
		vscode.workspace
		.getConfiguration()
		.update("coder.insecure", true, vscode.ConfigurationTarget.Global);
Expand Down
Original file line number	Diff line number	Diff line change
Expand Up		@@ -21,25 +21,31 @@ export async function activate(ctx: vscode.ExtensionContext): Promise<void> {
		//
		// Cursor and VSCode are covered by ms remote, and the only other is windsurf for now
		// Means that vscodium is not supported by this for now

		const remoteSSHExtension =
		vscode.extensions.getExtension("jeanp413.open-remote-ssh") \|\|
		vscode.extensions.getExtension("codeium.windsurf-remote-openssh") \|\|
		vscode.extensions.getExtension("anysphere.remote-ssh") \|\|
		vscode.extensions.getExtension("ms-vscode-remote.remote-ssh");

		let vscodeProposed: typeof vscode = vscode;

		if (!remoteSSHExtension) {
		vscode.window.showErrorMessage(
		"Remote SSH extension not found, cannot activate Coder extension",
		"Remote SSH extension not found, this may not work as expected.\n" +
		// NB should we link to documentation or marketplace?
jaggederest marked this conversation as resolved. Show resolvedHide resolved
		"Please install your choice of Remote SSH extension from the VS Code Marketplace.",
		);
		} else {
		// eslint-disable-next-line @typescript-eslint/no-explicit-any
		vscodeProposed = (module as any)._load(
		"vscode",
		{
		filename: remoteSSHExtension.extensionPath,
		},
		false,
		);
Comment on lines +40 to 47 Copy link Preview CopilotAIJun 17, 2025 Choose a reason for hiding this comment The reason will be displayed to describe this comment to others.Learn more. Using 'module as any' bypasses type safety; consider a more type-safe approach or adding documentation to justify this workaround. Copilot uses AI. Check for mistakes. Copy link Author jaggederestJun 17, 2025 Choose a reason for hiding this comment The reason will be displayed to describe this comment to others.Learn more. Yeah I get it buddy I don't like it either but we're polymorphic on the SSH extensions. code-asher reacted with laugh emoji
		throw new Error("Remote SSH extension not found");
		}
		// eslint-disable-next-line @typescript-eslint/no-explicit-any
		const vscodeProposed: typeof vscode = (module as any)._load(
		"vscode",
		{
		filename: remoteSSHExtension?.extensionPath,
		},
		false,
		);

		const output = vscode.window.createOutputChannel("Coder");
		const storage = new Storage(
Expand DownExpand Up		@@ -278,7 +284,13 @@ export async function activate(ctx: vscode.ExtensionContext): Promise<void> {
		// Since the "onResolveRemoteAuthority:ssh-remote" activation event exists
		// in package.json we're able to perform actions before the authority is
		// resolved by the remote SSH extension.
		if (vscodeProposed.env.remoteAuthority) {
		//
		// In addition, if we don't have a remote SSH extension, we skip this
		// activation event. This may allow the user to install the extension
		// after the Coder extension is installed, instead of throwing a fatal error
		// (this would require the user to uninstall the Coder extension and
		// reinstall after installing the remote SSH extension, which is annoying)
Comment on lines +291 to +292 Copy link Member code-asherJun 17, 2025 Choose a reason for hiding this comment The reason will be displayed to describe this comment to others.Learn more. 🎉
		if (remoteSSHExtension && vscodeProposed.env.remoteAuthority) {
		const remote = new Remote(
		vscodeProposed,
		storage,
Expand Down