bms-speckit-plugin 5.2.0 → 5.2.1

package/agents/quality-control.md

@@ -157,3 +157,13 @@ Remaining (needs user review): X
 - Run tests after EVERY fix to prevent regressions
 - Commit fixes with descriptive messages
 - If you cannot fix an issue, explain why and what the user should do
+
+**Completion Signal:**
+
+After your report, you MUST output exactly one of these lines as the **last line** of your response:
+
+- `QC_RESULT: PASS` — build, lint, and tests all pass; no unfixed critical security issues
+- `QC_RESULT: PASS_WITH_NOTES` — all critical checks pass but some items flagged for user review (e.g., major dependency updates)
+- `QC_RESULT: FAIL` — unfixed build errors, test failures, or critical security vulnerabilities remain
+
+Both `PASS` and `PASS_WITH_NOTES` allow the pipeline to proceed to merge. Only `FAIL` blocks progression.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "bms-speckit-plugin",
-  "version": "5.2.0",
+  "version": "5.2.1",
   "description": "Chain-orchestrated development pipeline: /bms-speckit takes requirements and runs brainstorm → constitution → specify → plan → tasks → analyze → implement → verify with per-step error handling",
   "files": [
     ".claude-plugin/",

package/skills/bms-speckit-auto/SKILL.md

@@ -178,7 +178,7 @@ After the subagent completes, update tasks 1-7 as completed using TaskUpdate, th
   - **D. Accessibility** — alt text, form labels, keyboard nav, heading hierarchy
   - **E. Integration check** — verify all components work together end-to-end
 - The agent fixes everything it can. Major dependency updates are flagged for user review.
-- Only proceed to merge when the agent reports all checks pass.
+- **Completion rule:** When the QC agent returns its report, proceed to Step 11 **unless** the report contains unfixed build errors, unfixed test failures, or unfixed critical security vulnerabilities. Informational findings, flagged-for-review items, and already-fixed issues do NOT block progression. If uncertain, proceed — the QC agent already fixed what it could.
 - **Post-action:** Commit all fixes and push. Message: `fix(speckit): final QC — security, deps, UX consistency, accessibility`
 - **Done:** Update task 10 as completed. Output `[Step 10/11] DONE — quality gate passed`