Commit 3f4ec8a
committed
fix(workflows): drop the bundled speckit workflow's inert scope input
The bundled workflow declared a required-choice scope input that no step
referenced. All four steps pass {{ inputs.spec }}, so every choice behaved
identically.
The value was enum-validated and then discarded, which is worse than being
ignored outright: -i scope=bogus errors while -i scope=backend-only is
accepted, so the rejection of an invalid value is evidence to the user that
the flag does something.
Propagating scope instead would need a contract that does not exist -- the
steps take a single args string, so a scope could only be appended as prose
to the user's own description and left to the agent to interpret. That is a
feature with a design question attached, not a bug fix, so this removes the
selector and leaves the door open for a real one.
workflows/README.md demonstrated multiple inputs using this exact input
against the bundled workflow, so that example moves to a declared one.
Every declared input is now referenced and every referenced input declared.
Fixes #43841 parent 014a537 commit 3f4ec8a
2 files changed
Lines changed: 1 addition & 5 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
73 | 73 | | |
74 | 74 | | |
75 | 75 | | |
76 | | - | |
| 76 | + | |
77 | 77 | | |
78 | 78 | | |
79 | 79 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
35 | 35 | | |
36 | 36 | | |
37 | 37 | | |
38 | | - | |
39 | | - | |
40 | | - | |
41 | | - | |
42 | 38 | | |
43 | 39 | | |
44 | 40 | | |
| |||
0 commit comments