Skip to content

Improve cockpit startup ergonomics#472

Closed
NagyVikt wants to merge 1 commit into
mainfrom
agent/codex/improve-cockpit-startup-ergonomics-2026-04-29-22-11
Closed

Improve cockpit startup ergonomics#472
NagyVikt wants to merge 1 commit into
mainfrom
agent/codex/improve-cockpit-startup-ergonomics-2026-04-29-22-11

Conversation

@NagyVikt
Copy link
Copy Markdown
Collaborator

Summary

  • add gx cockpit --no-tmux for one-shot stdout cockpit previews
  • start tmux cockpit sessions with the real cockpit renderer command
  • target the tmux control pane by returned pane id instead of bare session name

Verification

  • PASS: node --test test/cockpit-command.test.js test/tmux-session.test.js test/cockpit-render.test.js
  • PARTIAL: node --test test/cockpit-command.test.js test/tmux-session.test.js test/cockpit-render.test.js test/agents-launch.test.js test/agents-sessions.test.js test/cli-args-dispatch.test.js (cockpit/tmux tests passed; unrelated agents-launch, agents-sessions, and cli-args-dispatch assertions failed)
  • BLOCKED: npm test remains red on the same unrelated baseline failures above

Risk

  • Narrow cockpit/tmux command surface change; no runtime dependency added.

Cockpit startup now supports a non-interactive render path for quick preview, and tmux startup sends the live cockpit renderer into the pane returned by tmux instead of addressing the session name ambiguously.

Constraint: Users need gx cockpit --no-tmux when tmux is unavailable or unwanted.

Rejected: Keep sending gx agents status to the session target | it does not render the cockpit and can target ambiguously.

Confidence: high

Scope-risk: narrow

Directive: Keep tmux control commands targeted to pane ids, not bare session names.

Tested: node --test test/cockpit-command.test.js test/tmux-session.test.js test/cockpit-render.test.js

Tested: node --test test/cockpit-command.test.js test/tmux-session.test.js test/cockpit-render.test.js test/agents-launch.test.js test/agents-sessions.test.js test/cli-args-dispatch.test.js (cockpit/tmux passed; unrelated agents tests failed)

Not-tested: Full npm test is blocked by unrelated baseline failures in agents-launch, agents-sessions, and cli-args-dispatch.
@NagyVikt NagyVikt closed this Apr 29, 2026
@NagyVikt NagyVikt deleted the agent/codex/improve-cockpit-startup-ergonomics-2026-04-29-22-11 branch April 29, 2026 20:31
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