Show Remote SSH Output panel on workspace start

Author: EhabY

CLAUDE.md

@@ -1,5 +1,43 @@
 # Coder Extension Development Guidelines
 
+## Working Style
+
+You're an experienced, pragmatic engineer. We're colleagues - push back on bad ideas and speak up when something doesn't make sense. Honesty over agreeableness.
+
+- Simple solutions over clever ones. Readability is a primary concern.
+- YAGNI - don't add features we don't need right now
+- Make the smallest reasonable changes to achieve the goal
+- Reduce code duplication, even if it takes extra effort
+- Match the style of surrounding code - consistency within a file matters
+- Fix bugs immediately when you find them
+
+## Naming and Comments
+
+Names should describe what code does, not how it's implemented:
+
+- Good: `Tool`, `RemoteTool`, `execute()`
+- Bad: `MCPToolWrapper`, `NewAPI`, `executeToolWithValidation()`
+
+Comments explain what code does or why it exists:
+
+- Never add comments about what used to be there or how things changed
+- Never use temporal terms like "new", "improved", "refactored", "legacy"
+- Code should be evergreen - describe it as it is
+- Do not add comments when you can instead use proper variable/function naming
+
+## Testing and Debugging
+
+- Tests must comprehensively cover functionality
+- Never mock behavior in end-to-end tests - use real data
+- Mock as little as possible in unit tests - try to use real data
+- Find root causes, not symptoms. Read error messages carefully before attempting fixes.
+
+## Version Control
+
+- Commit frequently throughout development
+- Never skip or disable pre-commit hooks
+- Check `git status` before using `git add`
+
 ## Build and Test Commands
 
 - Build: `yarn build`
@@ -8,20 +46,22 @@
 - Lint: `yarn lint`
 - Lint with auto-fix: `yarn lint:fix`
 - Run all tests: `yarn test`
-- Run specific test: `vitest ./src/filename.test.ts`
-- CI test mode: `yarn test:ci`
+- Unit tests: `yarn test:ci`
 - Integration tests: `yarn test:integration`
+- Run specific unit test: `yarn test:ci ./test/unit/filename.test.ts`
+- Run specific integration test: `yarn test:integration ./test/integration/filename.test.ts`
 
-## Code Style Guidelines
+## Code Style
 
 - TypeScript with strict typing
-- No semicolons (see `.prettierrc`)
+- Use semicolons
 - Trailing commas for all multi-line lists
 - 120 character line width
 - Use ES6 features (arrow functions, destructuring, etc.)
 - Use `const` by default; `let` only when necessary
+- Never use `any`, and use exact types when you can
 - Prefix unused variables with underscore (e.g., `_unused`)
-- Sort imports alphabetically in groups: external → parent → sibling
+- Sort imports alphabetically in groups (see `import/order` in `.eslintrc.json`): external → parent → sibling
 - Error handling: wrap and type errors appropriately
 - Use async/await for promises, avoid explicit Promise construction where possible
-- Test files must be named `*.test.ts` and use Vitest
+- Unit test files must be named `*.test.ts` and use Vitest, they should be placed in `./test/unit/<path in src>`

src/extension.ts

@@ -353,6 +353,7 @@ export async function activate(ctx: vscode.ExtensionContext): Promise<void> {
 		}),
 	);
 
+	let shouldShowSshOutput = false;
 	// 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.
@@ -370,6 +371,7 @@ export async function activate(ctx: vscode.ExtensionContext): Promise<void> {
 			);
 			if (details) {
 				ctx.subscriptions.push(details);
+				shouldShowSshOutput = details.startedWorkspace;
 				// Authenticate the plugin client which is used in the sidebar to display
 				// workspaces belonging to this deployment.
 				client.setHost(details.url);
@@ -460,9 +462,27 @@ export async function activate(ctx: vscode.ExtensionContext): Promise<void> {
 			}
 		}
 	}
+
+	if (shouldShowSshOutput) {
+		showSshOutput();
+	}
 }
 
 async function showTreeViewSearch(id: string): Promise<void> {
 	await vscode.commands.executeCommand(`${id}.focus`);
 	await vscode.commands.executeCommand("list.find");
 }
+
+function showSshOutput(): void {
+	for (const command of [
+		"opensshremotes.showLog",
+		"windsurf-remote-openssh.showLog",
+	]) {
+		/**
+		 * We must not await this command because
+		 * 1) it may not exist
+		 * 2) it might cause the Remote SSH extension to be loaded synchronously
+		 */
+		void vscode.commands.executeCommand(command);
+	}
+}

src/remote/remote.ts

@@ -51,6 +51,7 @@ import { computeSSHProperties, sshSupportsSetEnv } from "./sshSupport";
 export interface RemoteDetails extends vscode.Disposable {
 	url: string;
 	token: string;
+	startedWorkspace: boolean;
 }
 
 export class Remote {
@@ -415,6 +416,7 @@ export class Remote {
 			}
 		}
 
+		let startedWorkspace = false;
 		const disposables: vscode.Disposable[] = [];
 		try {
 			// Register before connection so the label still displays!
@@ -442,6 +444,7 @@ export class Remote {
 					await this.closeRemote();
 					return;
 				}
+				startedWorkspace = true;
 				workspace = updatedWorkspace;
 			}
 			this.commands.workspace = workspace;
@@ -681,6 +684,7 @@ export class Remote {
 		return {
 			url: baseUrlRaw,
 			token,
+			startedWorkspace,
 			dispose: () => {
 				disposables.forEach((d) => d.dispose());
 			},