Skip to content

feat: wire up consult mode CLI + fix global squad resolution (#196)#198

Merged
bradygaster merged 3 commits intobradygaster:mainfrom
jsturtevant:fix-consult-mode
Mar 5, 2026
Merged

feat: wire up consult mode CLI + fix global squad resolution (#196)#198
bradygaster merged 3 commits intobradygaster:mainfrom
jsturtevant:fix-consult-mode

Conversation

@jsturtevant
Copy link
Contributor

Summary

Wire up the consult and extract CLI commands, add documentation, fix global squad resolution, and clean up stale acceptance tests.

Changes

Consult mode CLI wiring

  • Register squad consult and squad extract commands in cli-entry.ts
  • Add help text for both commands with flags

Documentation

  • Create docs/features/consult-mode.md — full command reference, quick start, learning classification, license handling
  • Update docs/guide/personal-squad.md — section 6 now recommends squad consult for OSS work, remove resolved limitation from section 9, add cross-references

Fix: global squad resolution (Closes #196)

  • runShell() hardcoded teamRoot = process.cwd(), so the shell never found the global (personal) squad
  • Now uses a fallback chain: local .squad/resolveGlobalSquadPath()cwd (init mode)
  • The /init loop reported in squad init --global mode does not work as expected #196 is resolved — the shell finds the global squad from any directory

Tests

  • Add 6 happy-path E2E tests: init global → consult → status → extract flow
  • Add 8 Gherkin acceptance scenarios for consult and extract commands
  • Fix 5 stale acceptance test assertions (old output strings like Getting Started, Here:, Scaffold ready)
  • All 29 acceptance tests and 14 consult tests pass

Test plan

npx vitest run test/cli/consult.test.ts        # 14 pass
npx vitest run test/acceptance/acceptance.test.ts  # 29 pass

jsturtevant and others added 3 commits March 4, 2026 20:03
- Register consult and extract commands in cli-entry.ts with help text
- Add docs/features/consult-mode.md with full command reference
- Update docs/guide/personal-squad.md to reference consult mode
- Add happy-path E2E tests (init global → consult → status → extract)
- Add acceptance test features for consult and extract commands

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
…ster#196)

runShell() hardcoded teamRoot to process.cwd(), ignoring the global
(personal) squad path. Now uses a fallback chain:
1. Walk up from cwd for local .squad/
2. Check global squad path (resolveGlobalSquadPath)
3. Fall back to cwd (triggers init mode)

Closes bradygaster#196

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Feature files referenced old output strings ("Getting Started",
"Here:", "Scaffold ready") that no longer exist in the CLI help,
status, and init output. Updated to match actual output.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
tmcclell pushed a commit to tmcclell/squad that referenced this pull request Mar 5, 2026
Adds npm workspaces config and package scaffolds for @bradygaster/squad-sdk and @bradygaster/squad-cli.

Closes bradygaster#197
Closes bradygaster#198  
Closes bradygaster#200

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@bradygaster bradygaster merged commit 2b91f69 into bradygaster:main Mar 5, 2026
1 of 2 checks passed
bradygaster added a commit that referenced this pull request Mar 5, 2026
feat: wire up consult mode CLI + fix global squad resolution (#196)
bradygaster added a commit that referenced this pull request Mar 5, 2026
Session: 2026-03-05T21:05Z-release-planning
Requested by: Copilot (Scribe role)

Changes:
- Logged orchestration entries for Keaton, McManus, Kobayashi, Fenster, Coordinator
- Created session log documenting v0.8.21 release planning outcomes
- Merged 3 decision inbox files into decisions.md
- Deleted inbox files (contributor page, PR merges, user directives)

Decision Merged:
- Every release MUST include contributor page update
- Workstreams MUST be included in v0.8.21

Outcomes documented:
- 4 PRs merged to dev (#204, #203, #198, #189)
- 2 issues fixed (#210, #195)
- Build passing, 98.8% test coverage
- Release scope: 18 unreleased commits
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.

squad init --global mode does not work as expected

2 participants