qfai 1.5.5 → 1.5.7

package/README.md

@@ -250,7 +250,7 @@ flowchart LR
 - Contracts SSOT: `.qfai/contracts/**`
 - Report outputs (`.qfai/report/**`) are derived artifacts and not SSOT.
 
-## Minimal tutorial (v1.5.5)
+## Minimal tutorial (v1.5.6)
 
 1. `npx qfai init`
 2. Run `/qfai-discussion` to structure scope, open questions, and produce a discussion pack under `.qfai/discussion/discussion-<ts>/`.

package/assets/init/.qfai/assistant/instructions/agent-selection.md

@@ -10,6 +10,11 @@ update_frequency: occasional
 
 Delegate work to specialized roles to reduce blind spots and improve quality.
 
+## Feedback quality rule (all agents)
+
+- Every sub-agent MUST include a concrete alternative or fix proposal when submitting FAIL verdicts or other negative feedback.
+- Negative feedback without a concrete alternative is invalid and triggers re-judgment.
+
 ## Default delegation map
 
 - **Researcher**: collect pre-knowledge (English sources), glossary, risks, and question angles
@@ -34,6 +39,14 @@ Delegate work to specialized roles to reduce blind spots and improve quality.
 - **Runtime Gatekeeper**: runtime evidence and smoke verification
 - **Prototyping Coverage Auditor**: detect missing spec rows and unresolved checks in prototyping coverage evidence
 - **Doc Steward**: doc impact analysis and README/mermaid updates
+- **Devil's Advocate (devils-advocate)**: challenge all assumptions as fundamentally wrong, provide concrete alternatives for every objection
+  - Responsibility: review under the premise that everything is wrong; use nitpicking, reductio ad absurdum, and forced analogy to present a concrete vision of the ideal state
+  - Delegation rule: executes after existing 10 reviewers (11th). FAIL must include alternative proposal. Bare negation FAIL is invalid. 3 consecutive FAILs trigger advisory demotion (current cycle only)
+  - Selection scenario: mandatory in every skill review cycle (can_be_na: false). Purpose: uncover blind spots in design, specs, and requirements
+- **Pattern Doubler (pattern-doubler)**: demand 2x the current ID-bearing pattern count, propose concrete additions with rationale
+  - Responsibility: demand doubling of ID-bearing items (US, AC, BR, EX, TC) by identifying missing perspectives and proposing additions with justification
+  - Delegation rule: executes after devils-advocate (12th). Rationale for each proposed addition is required
+  - Selection scenario: executes in /qfai-sdd review cycles. N/A allowed only when spec pack has no ID-bearing items (can_be_na: true)
 
 ## If subagents are not supported
 

package/assets/init/.qfai/assistant/skills/qfai-atdd/SKILL.md

@@ -131,6 +131,14 @@ Every major artifact in this stage MUST include this fixed table schema:
   - Floors/ratios are signals, not gates.
   - `scenario.feature` and coverage ledgers are optional legacy inputs, not completion gates.
 - Do not declare DONE until Reviewer returns `PASS`.
+- **All reviewers: alternative proposal obligation**:
+  - Every reviewer MUST provide a concrete alternative or fix proposal when returning FAIL. Feedback without a concrete alternative is invalid and triggers re-judgment.
+- **devils-advocate gate**:
+  - devils-advocate FAIL must include a concrete alternative proposal. Bare negation FAIL triggers re-judgment.
+  - 3 consecutive FAILs trigger advisory demotion and allow progression to the next phase.
+- **pattern-doubler gate**:
+  - Each pattern proposed by pattern-doubler must include rationale.
+  - Artifacts with no ID-bearing items (US/AC/BR/EX/TC) are marked N/A.
 
 ### Work order template (copy/paste)
 

package/assets/init/.qfai/assistant/skills/qfai-configure/SKILL.md

@@ -103,6 +103,14 @@ Every major artifact in this stage MUST include a `## Work Orders Summary` secti
     - E2E/API/Integration coverage aligns with `steering/test-layers.md` and the project’s plan.
     - Do not use pyramid ratios as a gate; use floors/ratios only as signals. Coverage obligations are the gate.
 - Do not declare DONE or handoff until Reviewer returns `PASS`.
+- **All reviewers: alternative proposal obligation**:
+  - Every reviewer MUST provide a concrete alternative or fix proposal when returning FAIL. Feedback without a concrete alternative is invalid and triggers re-judgment.
+- **devils-advocate gate**:
+  - devils-advocate FAIL must include a concrete alternative proposal. Bare negation FAIL triggers re-judgment.
+  - 3 consecutive FAILs trigger advisory demotion and allow progression to the next phase.
+- **pattern-doubler gate**:
+  - Each pattern proposed by pattern-doubler must include rationale.
+  - Artifacts with no ID-bearing items (US/AC/BR/EX/TC) are marked N/A.
 
 ### Work order template (copy/paste)
 

package/assets/init/.qfai/assistant/skills/qfai-discussion/SKILL.md

@@ -74,6 +74,14 @@ Every major artifact in this stage MUST include this table schema:
 - Reviewer must check Drift Protocol compliance and alignment with `.qfai/assistant/steering/test-layers.md`.
 - Test volume floors/ratios are not gates; they are risk signals.
 - Do not declare DONE until Reviewer returns `PASS`; otherwise apply `REVISE`.
+- **All reviewers: alternative proposal obligation**:
+  - Every reviewer MUST provide a concrete alternative or fix proposal when returning FAIL. Feedback without a concrete alternative is invalid and triggers re-judgment.
+- **devils-advocate gate**:
+  - devils-advocate FAIL must include a concrete alternative proposal. Bare negation FAIL triggers re-judgment.
+  - 3 consecutive FAILs trigger advisory demotion and allow progression to the next phase.
+- **pattern-doubler gate**:
+  - Each pattern proposed by pattern-doubler must include rationale.
+  - Artifacts with no ID-bearing items (US/AC/BR/EX/TC) are marked N/A.
 
 ## CRITICAL CONSTRAINTS (Read First)
 
@@ -232,6 +240,14 @@ RCP rules:
 - Any `FAIL` requires return/fix/full rerun from the first reviewer.
 - Mark fixed only when all reviewers are `PASS` or valid `N/A`.
 - `summary.json` `target.kind` must be `"discussion"`.
+- Execution order: existing 10 reviewers (1-10) → devils-advocate (11) → pattern-doubler (12).
+- devils-advocate (11th):
+  - `can_be_na: false` — N/A is not allowed.
+  - FAIL must include a concrete alternative. Bare negation FAIL is invalid.
+  - 3 consecutive FAILs trigger advisory demotion (current review cycle only).
+- pattern-doubler (12th):
+  - `can_be_na: true` — N/A is default in discussion phase as ID-bearing items are sparse.
+  - Evaluates Example Seeds count and perspective coverage.
 
 ## RCP Footer Include (MUST)
 

package/assets/init/.qfai/assistant/skills/qfai-discussion/references/rcp_footer.md

@@ -89,3 +89,29 @@
   - 復旧: 1本でよいので、登場人物（ペルソナ）と主要分岐が分かる flowchart を入れる
 - FAIL: pack 名が不正（latest 判定がブレる）
   - 復旧: 不正 pack を `discussion-legacy-*` に退避し、最新 pack を timestamp 命名で作り直す
+
+---
+
+## 拡張レビュアー
+
+### devils-advocate（11番目）
+
+- 役割: 全否定エージェント — 「現状すべてが間違っている」前提でレビューし、あるべき姿を提示する
+- `can_be_na: false` — N/A は許可されない
+- FAIL 判定時は必ず具体的代替案を提示すること。代替案なしの FAIL は無効とし、再判定を要求する
+- 3 回連続 FAIL → アドバイザリー降格（当該レビューサイクル限定）。降格後はブロッキング力が消失し、フィードバックのみ記録する
+- レビュー記録: `R11_devils-advocate.md`
+
+### pattern-doubler（12番目）
+
+- 役割: パターン倍増エージェント — discussion phase では ID 付き項目が少ないため N/A が基本
+- `can_be_na: true` — discussion phase では Example Seeds の数と観点網羅性を評価対象とする
+- N/A でない場合、追加パターンの根拠提示が必須
+- レビュー記録: `R12_pattern-doubler.md`
+
+### 代表的な FAIL と復旧（拡張レビュアー特化）
+
+- FAIL (devils-advocate): 代替案なしの否定のみ
+  - 復旧: 具体的な「あるべき姿」と移行パスを記述させ、再判定する
+- FAIL (devils-advocate, 3回連続): 無限ループ検知
+  - 復旧: アドバイザリー降格を記録し、devils-advocate のフィードバックを advisory として保存。次フェーズに進行する