Terminal tab title: Show "~" instead of "$HOME"

[Dan Brown (jdanbrown)]

Fixes #274200

[Dan Brown (jdanbrown)]

@microsoft-github-policy-service agree

[Dan Brown (jdanbrown)]

👋 Megan Rogge (@meganrogge) I made this PR for #274200, which you got assigned to.

[Copilot]

Pull Request Overview

This PR adds support for tildifying (replacing user home paths with ~) in terminal instance labels. When displaying terminal working directories, paths under the user's home directory now show as ~/path instead of /home/user/path.

Key changes:

• Modified terminal label computation to use the tildify utility function for CWD paths
• Added special handling for the exact user home path to display as ~
• Updated test fixtures to include a default userHome value and added test cases for tildification scenarios

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File	Description
src/vs/workbench/contrib/terminal/browser/terminalInstance.ts	Imports tildify function, adds os to type signatures, implements tildification logic for CWD paths in label computation
src/vs/workbench/contrib/terminal/test/browser/terminalInstance.test.ts	Updates default userHome in test fixture from undefined to /home/user, adds three new test cases for tildification behavior

Comments suppressed due to low confidence (1)

src/vs/workbench/contrib/terminal/test/browser/terminalInstance.test.ts:305

• The return type of createInstance is missing the os property that was added to the refreshLabel and computeLabel method signatures in the production code. This type mismatch could cause the tests to not properly validate the OS-specific behavior of tildification.

		function createInstance(partial?: Partial<ITerminalInstance>): Pick<ITerminalInstance, 'shellLaunchConfig' | 'shellType' | 'userHome' | 'cwd' | 'initialCwd' | 'processName' | 'sequence' | 'workspaceFolder' | 'staticTitle' | 'capabilities' | 'title' | 'description'> {

[Dan Brown (jdanbrown)]

Hi Megan Rogge (@meganrogge), will you or someone else be able to review this to get it merged? It's a pretty trivial code change for a small but meaningful UI improvement.

[Megan Rogge (meganrogge)]

LGTM, thanks!

[Daniel Imms (Tyriar)]

Failing tests on Windows

[vs-code-engineering]

📬 CODENOTIFY

The following users are being notified based on files changed in this PR:

Anthony Kim (@anthonykim1)

Matched files:

• src/vs/workbench/contrib/terminal/browser/terminalInstance.ts
• src/vs/workbench/contrib/terminal/test/browser/terminalInstance.test.ts