Keep background commands out of the Agents tab entirely - #163
Merged
Conversation
The tab briefly listed non-agent background commands in their own section below the agents, but the header's activity chip already gives every background task a richer surface - its command, terminal toggle and stop button - so the section was a duplicate with a worse address. The Agents tab now holds only agents: the run-branch machinery (AgentRunBranch, runBranch, agentInitiatedRuns, the provenance tag helpers) is deleted, the panel and its store stop carrying backgroundRuns and the terminal toggle, and the launcher row reports "No agents yet" instead of counting commands. Promoted codex exec runs are unaffected: they are agents and keep their rows and stop handles.
|
The latest updates on your projects. Learn more about Vercel for GitHub. 1 Skipped Deployment
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Follow-up to #162 from Will's design call: the Agents tab briefly gave background commands their own section below the agents, but the header's activity chip already lists every background task with its command, terminal toggle, and stop button — a better home, and the panel's section was a duplicate.
The Agents tab is now agents-only:
AgentRunBranch,runBranch,agentInitiatedRuns, the provenance-tag helpers) — net −300 lines.backgroundRunsand the terminal toggle; the launcher's Agents row reads "No agents yet" instead of counting commands.codex execruns are unaffected: they are agents, and keep their rows and stop handles.Typecheck, lint, full web unit (167 files) and browser (469) suites green.