Skip to content

chore: track running agents app via marker file#314730

Merged
deepak1556 merged 2 commits intomainfrom
robo/add_agents_running_marker
May 6, 2026
Merged

chore: track running agents app via marker file#314730
deepak1556 merged 2 commits intomainfrom
robo/add_agents_running_marker

Conversation

@deepak1556
Copy link
Copy Markdown
Collaborator

Before we deploy #314409 this change will allow us to restore agents window with a deprecation message.

@deepak1556 deepak1556 added this to the 1.120.0 milestone May 6, 2026
@deepak1556 deepak1556 self-assigned this May 6, 2026
Copilot AI review requested due to automatic review settings May 6, 2026 14:06
@deepak1556 deepak1556 enabled auto-merge (squash) May 6, 2026 14:07
sandy081
sandy081 previously approved these changes May 6, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 6, 2026

Base: 3453ed97 Current: a10ccfb4

No screenshot changes.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Tracks when the embedded Agents sub-application is/was running by writing a marker file into the host VS Code’s user-data area, so a future build (after removing sub-app support) can detect this state and restore or migrate the Agents window appropriately.

Changes:

  • Hook marker tracking into Electron main startup when running as the embedded app.
  • Add AgentsLastRunningTracker that writes agentsLastRunning.json and refreshes it on cross-app IPC connect/disconnect.

Reviewed changes

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

File Description
src/vs/code/electron-main/app.ts Instantiates the marker writer when running as the embedded Agents app and the host user-data path is available.
src/vs/code/electron-main/agentsLastRunningTracker.ts New tracker that writes/updates the host-side marker file with basic liveness metadata.

Comment thread src/vs/code/electron-main/app.ts Outdated
Comment thread src/vs/code/electron-main/agentsLastRunningTracker.ts
@deepak1556 deepak1556 merged commit 98a1eb8 into main May 6, 2026
40 of 41 checks passed
@deepak1556 deepak1556 deleted the robo/add_agents_running_marker branch May 6, 2026 14:58
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.

3 participants