Skip to content

fix: wire missing CLI commands into cli-entry.ts#244

Merged
bradygaster merged 2 commits intodevfrom
squad/237-wire-cli-commands
Mar 7, 2026
Merged

fix: wire missing CLI commands into cli-entry.ts#244
bradygaster merged 2 commits intodevfrom
squad/237-wire-cli-commands

Conversation

@bradygaster
Copy link
Copy Markdown
Owner

Summary

Wires 4 CLI commands that existed in \packages/squad-cli/src/cli/commands/\ but had no route in \cli-entry.ts:

Command Route What it does
\
c\ / \
emote-control\ \squad rc [--tunnel] [--port] [--path]\ Starts RemoteBridge (phone/browser → Copilot)
\copilot-bridge\ \squad copilot-bridge\ Checks Copilot ACP stdio compatibility
\init-remote\ \squad init-remote \ Standalone shorthand for \init --mode remote\
\
c-tunnel\ \squad rc-tunnel\ Checks devtunnel CLI availability

The other commands listed in #237 (\�spire, \link, \loop, \hire) were already wired — verified and left in place.

Help text entries added for all 4 new routes. GREEN/YELLOW imports added to output line.

Tests: 3655 passed, 1 pre-existing flaky speed-gate (unrelated).

Working as Fenster (Core Dev).

Closes #237

bradygaster and others added 2 commits March 6, 2026 21:00
…se wave done

Session: 2026-03-07T04-55-00Z-phase4-complete
Phase 4 Summary:
- Merged PR #235 (commit ce418c6): Test stabilization
- Merged PR #234 (commit f88bf4c): Runtime bug fixes
- Merged PR #233 (commit 2f42e32): CLI wiring + CI fix
- Closed 8 issues from PRs #234 and #233
- Test suite now at 100% pass rate (3,656 passing)
- CI fully green and stable
- Release wave v0.8.21 COMPLETE

Changes:
- Merged 4 decision inbox files into decisions.md
- Deleted 4 inbox files (now in shared decisions)
- 1 PR (#235) + 1 PR (#234) = 2 PRs already merged (Kobayashi phase 4)
- 1 PR (#233) rebased and merged by Coordinator
- All team state preserved via merge=union

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Session: 2026-03-07-next-wave-triage
Requested by: Brady

Changes:
- Merged 3 inbox decisions (Keaton triage, Fenster PR review, Hockney test health)
- Updated agent history files with cross-team context propagation
- Deleted TRIAGE_REPORT.md from repo root (triage data belongs in .squad/)
- Orchestration logs written for all 3 agents
- Session log created for triage fan-out results
- Decision deduplication verified (no exact duplicates)

Decisions merged:
- Keaton: Full 22-issue triage, P0-P3 prioritization, v0.8.22 plan (11 issues)
- Fenster: 4 PR review, all retarget to dev, CLI wiring pattern adopted
- Hockney: Test suite health, 8 CLI commands untested, 30+ error-handling tests needed

Cross-agent context:
- Fenster: Notified of test gaps for CLI wiring work
- Edie: Notified of model config priority (#223 > #205)
- Rabin: Notified of migration wave grouping (#197/#231/#126) and ADO config blocker
- Keaton: Updated with triage scope and decisions
- Hockney: Updated with test infrastructure gaps for next-wave

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
bradygaster added a commit that referenced this pull request Mar 7, 2026
Session: 2026-03-07T14-16-00Z-next-wave-fenster-completions
Requested by: Scribe (autonomous)

Changes:
- Logged agent-32 completion: CLI command wiring (#237) — 4 commands wired, 3655 tests pass, PR #244
- Logged agent-31 completion: Model config reliability (#223) — root cause fixed, backwards compatible, PR #245
- Updated Fenster's history.md with cross-agent context about CLI wiring and model config patterns
@bradygaster bradygaster merged commit a488eb8 into dev Mar 7, 2026
1 check passed
@bradygaster bradygaster deleted the squad/237-wire-cli-commands branch March 7, 2026 14:19
bradygaster added a commit that referenced this pull request Mar 7, 2026
Session: 2026-03-07T143000Z-quality-review-cycle
Requested by: Brady

Changes:
- Merged Brady quality cross-review directive from inbox
- All team members must double-and-triple check work (PRs #244-246 inconsistencies)
- Updated decisions.md with new directive (appended)
- Cleaned up inbox file after merge
jongio pushed a commit to jongio/squad that referenced this pull request Mar 9, 2026
…radygaster#242, bradygaster#243, bradygaster#244)

* feat: shell UI components — AgentPanel, MessageStream, InputPrompt (bradygaster#242, bradygaster#243, bradygaster#244)

Add ink-based React components for the interactive shell:
- AgentPanel: displays live agent status with role and status icons
- MessageStream: renders streaming output with message history
- InputPrompt: readline input with history navigation (up/down arrow)
- Barrel export from components/index.ts
- Added jsx: react-jsx to tsconfig.json for TSX compilation
- Removed components/.gitkeep (replaced by actual components)

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

* fix: remove --provenance from publish (private repo limitation)

npm provenance requires a public source repository. Since squad-pr is private,
remove --provenance and id-token: write from both publish workflows.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

* test: shell integration tests — lifecycle, spawn, coordinator, sessions (bradygaster#248)

Add 47 integration tests for the shell module covering:
- SessionRegistry: register, get, getAll, getActive, updateStatus, remove, clear
- Spawn: loadAgentCharter, buildAgentPrompt (with/without systemContext)
- Coordinator: buildCoordinatorPrompt, parseCoordinatorResponse (DIRECT/ROUTE/MULTI/fallback), formatConversationContext
- ShellLifecycle: init, message history, agent discovery, shutdown
- StreamBridge: message_delta, usage, reasoning_delta, flush, buffer accumulation, clear

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

---------

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
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