Skip to content

launch: Show agent session in editor window title - #334108

Merged
Megan Rogge (meganrogge) merged 3 commits into
mainfrom
agents/vscode-issue-334097-implementation
Sep 2, 2026
Merged

launch: Show agent session in editor window title#334108
Megan Rogge (meganrogge) merged 3 commits into
mainfrom
agents/vscode-issue-334097-implementation

Conversation

@meganrogge

@meganrogge Megan Rogge (meganrogge) commented Sep 2, 2026

Copy link
Copy Markdown
Collaborator

Summary

Image
  • pass the current Agents session title to the launch skill with --session-title
  • put the session title first in the isolated editor window title, followed by the workspace and application name
  • preserve JSONC settings and replace copied settings symlinks with isolated regular files before writing
  • reject --session-title for standalone Agents windows, where window.title is read-only

This is a stable, session-first alternative to #334104. Unlike that PR, it intentionally omits the active editor title (Welcome or a filename), so the identifying title does not change while navigating editors. It also does not add a fallback launch identifier when no session title is provided.

Fixes #334097

Validation

  • npm run compile
  • npm run eslint -- .agents/skills/launch/scripts/updateSettings.ts
  • Bash syntax validation
  • PowerShell parser validation
  • JSONC, nested-key, symlink-isolation, quoting, Unicode, dollar-sign, and existing-setting cases
  • real isolated Code OSS launch verified over CDP as Show originating session — vscode-issue-334097-implementation — Code - OSS Dev

The repository hygiene runner has no eligible paths under .agents/skills/launch; invoking it with the changed files reports that no hygiene-eligible files matched.

Identify isolated Code OSS editor windows by placing the originating agent session title first in window.title. Preserve JSONC settings and keep copied symlinked settings isolated from the source profile.\n\nFixes #334097\n\nCo-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot AI balanced review requested due to automatic review settings September 2, 2026 20:31

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot review overview

🟡 Changes recommended

Relative settings symlinks and duplicate JSONC properties can prevent the intended settings from taking effect.

Once you've addressed the issues Copilot identified, you can request another Copilot review.

Review tier: Balanced
Findings: 3 Medium severity

New issues introduced by this change (3)
Severity Finding
Medium severity .agents/​skills/​launch/​scripts/​launch.sh — With set -u, invoking launch.sh --session-title without a value expands an unset $2 and exits…
Medium severity .agents/​skills/​launch/​scripts/​updateSettings.ts — A copied relative symlink can be dangling at this point. rsync -a preserves User/settings.json
Medium severity .agents/​skills/​launch/​scripts/​updateSettings.ts — This updates only the first occurrence of a root setting. VS Code's JSON parser assigns properties…
What changed in this PR

Adds stable session-first titles to isolated editor windows launched from agent sessions.

Changes:

  • Adds cross-platform --session-title support.
  • Updates isolated JSONC settings while breaking settings symlinks.
  • Documents usage and Agents-window restrictions.
File Description
.agents/​skills/​launch/​SKILL.md Documents session-title launching.
.agents/​skills/​launch/​scripts/​updateSettings.ts Updates isolated JSONC settings.
.agents/​skills/​launch/​scripts/​launch.sh Adds Bash option handling.
.agents/​skills/​launch/​scripts/​launch.ps1 Adds PowerShell option handling.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread .agents/skills/launch/scripts/launch.sh Outdated
Comment thread .agents/skills/launch/scripts/updateSettings.ts Outdated
Comment thread .agents/skills/launch/scripts/updateSettings.ts Outdated
Validate missing Bash option values, dereference relative settings symlinks from the source profile, and update the last duplicate root setting so the effective JSONC value changes.\n\nCo-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@meganrogge
Megan Rogge (meganrogge) merged commit a0a0a37 into main Sep 2, 2026
40 checks passed
@meganrogge
Megan Rogge (meganrogge) deleted the agents/vscode-issue-334097-implementation branch September 2, 2026 21:36
@vs-code-engineering vs-code-engineering Bot added this to the 1.137.0 milestone Sep 2, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Show the originating agent session title in launched OSS windows

4 participants