Skip to content

Gate Wizard canvas generation behind an opt-in open input - #44

Merged
nicolehaugen merged 1 commit into
mainfrom
nicolehaugen-wizard-canvas-feature-flag
Sep 30, 2026
Merged

nicolehaugen merged 1 commit into
mainfrom
nicolehaugen-wizard-canvas-feature-flag

Conversation

@nicolehaugen

Copy link
Copy Markdown
Contributor

Summary

  • Hide Generate canvas by default and expose it only when the Wizard is opened with generateCanvas: true.
  • Load the Designer handoff handler only when the launch endpoint is invoked, so the Wizard can start without Designer files or an enabled Designer provider.
  • Remove unreleased canvas-generation guidance from the Wizard-related READMEs.

Validation

  • Existing Wizard dialog, snapshot, and Designer launch tests pass (28 tests).
  • Session-local Wizard preview opened both without Designer's sibling helper and with the opt-in enabled.

Keep the Wizard independent of Designer until launch, and remove unreleased generation guidance from READMEs.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Copilot AI balanced review requested due to automatic review settings September 30, 2026 19:53

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot review overview

🟡 Changes recommended

The central default-off behavior lacks a regression test.

Review effort: Balanced
Findings: 1 Medium severity

Open (1)
What changed in this PR

Adds an opt-in gate for experimental Wizard canvas generation while decoupling Wizard startup from Designer files.

Changes:

  • Propagates generateCanvas from canvas input to the UI.
  • Lazily imports the Designer launch handler.
  • Removes unreleased generation guidance.
File Description
README.md Removes future handoff messaging.
ui/​phase-runtime.js Conditionally renders Generate canvas.
test/​canvas-designer-dialog.test.mjs Enables the flag in the existing test.
server.mjs Lazily loads the Designer launch handler.
speckit-wizard-canvas/​README.md Removes experimental workflow documentation.
extension.mjs Declares and captures the opt-in input.
e2e/​server.mjs Enables generation in the E2E fixture.
canvas-runtime/​snapshot.mjs Exposes the feature flag to clients.
speckit-canvas-designer/​README.md Simplifies Designer shell documentation.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@nicolehaugen
nicolehaugen merged commit 72032b7 into main Sep 30, 2026
9 checks passed
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.

2 participants