Skip to content

Allow finishing one known agent session#463

Merged
NagyVikt merged 2 commits into
mainfrom
agent/codex/agent-lane-inspection-cli-2026-04-29-manual
Apr 29, 2026
Merged

Allow finishing one known agent session#463
NagyVikt merged 2 commits into
mainfrom
agent/codex/agent-lane-inspection-cli-2026-04-29-manual

Conversation

@NagyVikt
Copy link
Copy Markdown
Collaborator

Automated by gx branch finish (PR flow).

NagyVikt added 2 commits April 29, 2026 21:49
Add gx agents files, diff, and locks commands so a selected agent branch can be inspected without mutating the lane. The CLI resolves branch base metadata, uses the selected worktree when available, includes dirty worktree paths, and filters lock registry noise from file/diff output.

Constraint: Commands must be read-only and cockpit-compatible.

Rejected: Shelling out to the lock script for locks | JSON output and cockpit reuse need structured data from the registry.

Confidence: high

Scope-risk: narrow

Directive: Keep agent inspection read-only; do not route these commands through finish or cleanup flows.

Tested: node --test test/agents-inspect.test.js test/agents.test.js

Tested: node --check src/agents/inspect.js && node --check src/cli/args.js && node --check src/cli/main.js
The agents CLI already has session metadata and branch finish plumbing, so this adds a small resolver that updates session status around the existing finish command instead of duplicating merge behavior.

Constraint: Must reuse existing gx finish / branch finish logic
Rejected: Reimplement branch finish in agents module | would fork merge semantics
Confidence: medium
Scope-risk: narrow
Tested: node --test test/agents-finish.test.js
Not-tested: live GitHub PR merge flow for gx agents finish
@NagyVikt NagyVikt merged commit dde5db5 into main Apr 29, 2026
2 of 5 checks passed
@NagyVikt NagyVikt deleted the agent/codex/agent-lane-inspection-cli-2026-04-29-manual branch April 29, 2026 19:50
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.

1 participant