feat(cli, wizard): support plan mode in wizard#435
Merged
calvinbrewer merged 2 commits intomainfrom May 7, 2026
Merged
Conversation
🦋 Changeset detectedLatest commit: 11f0e8f The changes in this PR will be included in the next version bump. This PR includes changesets to release 3 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (2)
✅ Files skipped from review due to trivial changes (1)
🚧 Files skipped from review as they are similar to previous changes (1)
📝 WalkthroughWalkthroughThis PR introduces plan-mode support to the Stash wizard CLI. The wizard now accepts a ChangesPlan Mode Support
Sequence DiagramsequenceDiagram
participant User
participant CLI as stash CLI
participant WizardBin as wizard binary
participant Gather as gatherContext()
participant Agent as Agent / LLM
participant Gateway as Gateway API
participant Finalize as finalize()
User->>CLI: stash plan (or stash impl)
CLI->>CLI: Determine mode: 'plan' or 'implement'
CLI->>WizardBin: Spawn with --mode <plan|implement>
WizardBin->>WizardBin: parseArgs() → mode
WizardBin->>Gather: gatherContext({..., mode})
alt plan mode
Gather->>Gather: Skip column selection (empty list)
Gather-->>WizardBin: GatheredContext
else implement mode
Gather->>Gather: Prompt & select columns
Gather-->>WizardBin: GatheredContext
end
WizardBin->>Agent: Initialize agent
WizardBin->>Gateway: fetchIntegrationPrompt({..., mode})
Gateway-->>WizardBin: Prompt (plan-specific or impl-specific)
WizardBin->>Agent: Execute agent with prompt
alt plan mode
Agent-->>WizardBin: Agent result
WizardBin->>WizardBin: Write .cipherstash/plan.md
WizardBin->>Finalize: finalize()
else implement mode
WizardBin->>WizardBin: Start call-site scan (background)
Agent-->>WizardBin: Agent result
WizardBin->>WizardBin: Run post-agent steps
WizardBin->>WizardBin: Await scan result
WizardBin->>Finalize: finalize()
end
Finalize->>Finalize: Install Claude skills
Finalize->>Finalize: Record completion telemetry
Finalize->>Finalize: Flush changelog
Finalize-->>WizardBin: Done
WizardBin-->>User: Output outro + next steps
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~25 minutes Possibly related PRs
Suggested reviewers
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
🧹 Nitpick comments (1)
packages/wizard/src/lib/gather.ts (1)
9-16: 💤 Low valueMinor: two separate
importblocks fromnode:fscould be merged.
closeSyncis imported in its own statement (Line 16) separate from the five othernode:fsimports (Lines 9–15). No functional impact, but a single block is conventional.♻️ Proposed merge
import { existsSync, openSync, readFileSync, readSync, readdirSync, + closeSync, } from 'node:fs' -import { closeSync } from 'node:fs'🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@packages/wizard/src/lib/gather.ts` around lines 9 - 16, Merge the two separate node:fs import statements into one by combining existsSync, openSync, readFileSync, readSync, readdirSync and closeSync into a single import declaration; locate the imports at the top of the file where existsSync, openSync, readFileSync, readSync, readdirSync and closeSync are currently imported separately and consolidate them into one import from 'node:fs' to follow convention and reduce redundancy.
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@packages/wizard/src/bin/parse-args.ts`:
- Around line 36-46: The space-separated --mode branch in parse-args.ts
currently skips setting modeError when --mode is the last argument; update the
if (arg === '--mode' && i + 1 < args.length) branch so that when i + 1 >=
args.length you set modeError to a clear message (e.g., "Missing value for
--mode; expected 'plan' or 'implement'") instead of silently falling through;
keep the existing validation for next === 'plan' || 'implement' and increment i
when you consume the value, and ensure mode remains unchanged when modeError is
set.
---
Nitpick comments:
In `@packages/wizard/src/lib/gather.ts`:
- Around line 9-16: Merge the two separate node:fs import statements into one by
combining existsSync, openSync, readFileSync, readSync, readdirSync and
closeSync into a single import declaration; locate the imports at the top of the
file where existsSync, openSync, readFileSync, readSync, readdirSync and
closeSync are currently imported separately and consolidate them into one import
from 'node:fs' to follow convention and reduce redundancy.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro
Run ID: 469af2af-53e6-4dab-b600-716d4dd3bfff
📒 Files selected for processing (18)
.changeset/wizard-plan-mode.mdpackages/cli/src/commands/impl/__tests__/how-to-proceed.test.tspackages/cli/src/commands/impl/steps/handoff-wizard.tspackages/cli/src/commands/impl/steps/how-to-proceed.tspackages/wizard/src/__tests__/parse-args.test.tspackages/wizard/src/agent/__tests__/interface.test.tspackages/wizard/src/agent/fetch-prompt.tspackages/wizard/src/agent/hooks.tspackages/wizard/src/agent/interface.tspackages/wizard/src/bin/parse-args.tspackages/wizard/src/bin/wizard.tspackages/wizard/src/lib/detect.tspackages/wizard/src/lib/format.tspackages/wizard/src/lib/gather.tspackages/wizard/src/lib/prerequisites.tspackages/wizard/src/lib/types.tspackages/wizard/src/run.tsskills/stash-cli/SKILL.md
auxesis
approved these changes
May 7, 2026
Contributor
auxesis
left a comment
auxesis
left a comment
There was a problem hiding this comment.
Nice work @calvinbrewer ❤️
| "stash": minor | ||
| --- | ||
|
|
||
| Add plan-mode support to the wizard so `stash plan` can hand off to the CipherStash Agent. The wizard now accepts `--mode <plan|implement>` (default `implement` for back-compat). In plan mode it skips the column-selection TUI, forwards `mode: 'plan'` to the gateway (which returns a planning prompt whose deliverable is `.cipherstash/plan.md`), and skips the post-agent install/push/migrate and call-site-scan steps. Implement mode is unchanged. |
Contributor
There was a problem hiding this comment.
I'm not convinced backwards compatibility is important here, and would be fine with the default being plan.
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
2 participants
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.
Summary by CodeRabbit
New Features
--plan/--implement/--mode <plan|implement>CLI flags; plan mode produces a draft.cipherstash/plan.mdand skips column selection and post-implementation steps.Improvements
stash planvsstash impl.Tests
Documentation