Commit 7703924

committed

Pass the path to shellIntegration.ps1 directly to the server

The server can't use `code --locate-shell-integration-path pwsh` because it doesn't know if it's `code` or `code-insiders` etc. Even when given the direct path to the client's host process, it can't reliably use that API either. Mostly this is due to Windows using `Code.cmd` in the background (which is not `Code.exe`, the host process) but it's also slower anyway as it requires another launch of Node.js. We'll have to rely on the path to the script not changing relative to `vscode.env.appRoot`, but at least that part is a public API.

1 parent 1f62851 commit 7703924Copy full SHA for 7703924

File tree

1 file changed

-1

lines changed

src
- session.ts

1 file changed

-1

lines changed

`‎src/session.ts‎`

Lines changed: 6 additions & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -610,6 +610,11 @@ export class SessionManager implements Middleware {`
`610`	`610`	`});`
`611`	`611`	`};`
`612`	`612`
	`613`	`+ // When Terminal Shell Integration is enabled, we pass the path to the script that the server should execute.`
	`614`	`+ // Passing an empty string implies integration is disabled.`
	`615`	`+ const shellIntegrationEnabled = vscode.workspace.getConfiguration("terminal.integrated.shellIntegration").get<boolean>("enabled");`
	`616`	`+ const shellIntegrationScript = path.join(vscode.env.appRoot, "out", "vs", "workbench", "contrib", "terminal", "browser", "media", "shellIntegration.ps1");`
	`617`	`+`
`613`	`618`	`const clientOptions: LanguageClientOptions = {`
`614`	`619`	`documentSelector: this.documentSelector,`
`615`	`620`	`synchronize: {`
`@@ -622,7 +627,7 @@ export class SessionManager implements Middleware {`
`622`	`627`	`initializationOptions: {`
`623`	`628`	`enableProfileLoading: this.sessionSettings.enableProfileLoading,`
`624`	`629`	`initialWorkingDirectory: await validateCwdSetting(this.logger),`
`625`		`- shellIntegrationEnabled: vscode.workspace.getConfiguration("terminal.integrated.shellIntegration").get<boolean>("enabled"),`
	`630`	`+ shellIntegrationScript: shellIntegrationEnabled ? shellIntegrationScript : "",`
`626`	`631`	`},`
`627`	`632`	`errorHandler: {`
`628`	`633`	`// Override the default error handler to prevent it from`

0 commit comments

Comments

(0)

Navigation Menu

Search code, repositories, users, issues, pull requests...

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Commit 7703924

File tree

1 file changed

1 file changed

`‎src/session.ts‎`

0 commit comments