Remove interactive flow references from CLI architecture spec#151
Remove interactive flow references from CLI architecture spec#151nick-pape wants to merge 1 commit intoSharePoint:mainfrom
Conversation
All CLI flows are non-interactive — every parameter must be supplied via flags. Remove the three places in the spec that described interactive fallback prompts as planned behavior. Co-Authored-By: Claude Sonnet 4.6 (1M context) <noreply@anthropic.com>
There was a problem hiding this comment.
Pull request overview
Updates the SPFx CLI architecture RFC to remove references to interactive prompting, aligning the document with the current design/behavior that CLI usage is flag-driven.
Changes:
- Removed mentions of “interactive prompts” from the CLI description.
- Removed the planned “interactive fallback” description for missing parameters.
- Removed “Interactive fallback flow” from the Phase 2 plan.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| to `@microsoft/spfx-template-api`. All flows are non-interactive — every parameter | ||
| must be supplied via flags. |
There was a problem hiding this comment.
The statement “every parameter must be supplied via flags” conflicts with the parameter table below, which explicitly marks several parameters as optional with defaults (e.g., --target-dir defaults to CWD, --component-alias defaults to component name). Consider rephrasing to “no interactive prompting; any required parameters must be supplied via flags (missing required params error)” so the doc is internally consistent.
| to `@microsoft/spfx-template-api`. All flows are non-interactive — every parameter | |
| must be supplied via flags. | |
| to `@microsoft/spfx-template-api`. All flows are non-interactive — there is no prompting; | |
| any required parameters must be supplied via flags, and optional parameters use documented | |
| defaults when omitted. |
Description
Removes interactive prompt flow references from the CLI architecture spec. All CLI flows are non-interactive — every parameter must be supplied via flags.
Closes #95.
Changes
How was this tested
Docs-only change.
Type of change
🤖 Generated with Claude Code