takt 0.3.6 → 0.3.8

package/dist/workflow/status-rules.js

@@ -0,0 +1,81 @@
+/**
+ * Status rules prompt generation for workflow steps
+ *
+ * Generates structured prompts that tell agents which numbered tags to output
+ * based on the step's rule configuration.
+ */
+/** Localized strings for rules-based status prompt */
+const RULES_PROMPT_STRINGS = {
+    en: {
+        criteriaHeading: '## Decision Criteria',
+        headerNum: '#',
+        headerCondition: 'Condition',
+        headerTag: 'Tag',
+        outputHeading: '## Output Format',
+        outputInstruction: 'Output the tag corresponding to your decision:',
+        appendixHeading: '### Appendix Template',
+        appendixInstruction: 'When outputting `[{tag}]`, append the following:',
+    },
+    ja: {
+        criteriaHeading: '## 判定基準',
+        headerNum: '#',
+        headerCondition: '状況',
+        headerTag: 'タグ',
+        outputHeading: '## 出力フォーマット',
+        outputInstruction: '判定に対応するタグを出力してください:',
+        appendixHeading: '### 追加出力テンプレート',
+        appendixInstruction: '`[{tag}]` を出力する場合、以下を追記してください:',
+    },
+};
+/**
+ * Generate status rules prompt from rules configuration.
+ * Creates a structured prompt that tells the agent which numbered tags to output.
+ *
+ * Example output for step "plan" with 3 rules:
+ *   ## 判定基準
+ *   | # | 状況 | タグ |
+ *   |---|------|------|
+ *   | 1 | 要件が明確で実装可能 | `[PLAN:1]` |
+ *   | 2 | ユーザーが質問をしている | `[PLAN:2]` |
+ *   | 3 | 要件が不明確、情報不足 | `[PLAN:3]` |
+ */
+export function generateStatusRulesFromRules(stepName, rules, language) {
+    const tag = stepName.toUpperCase();
+    const strings = RULES_PROMPT_STRINGS[language];
+    const lines = [];
+    // Criteria table
+    lines.push(strings.criteriaHeading);
+    lines.push('');
+    lines.push(`| ${strings.headerNum} | ${strings.headerCondition} | ${strings.headerTag} |`);
+    lines.push('|---|------|------|');
+    for (const [i, rule] of rules.entries()) {
+        lines.push(`| ${i + 1} | ${rule.condition} | \`[${tag}:${i + 1}]\` |`);
+    }
+    lines.push('');
+    // Output format
+    lines.push(strings.outputHeading);
+    lines.push('');
+    lines.push(strings.outputInstruction);
+    lines.push('');
+    for (const [i, rule] of rules.entries()) {
+        lines.push(`- \`[${tag}:${i + 1}]\` — ${rule.condition}`);
+    }
+    // Appendix templates (if any rules have appendix)
+    const rulesWithAppendix = rules.filter((r) => r.appendix);
+    if (rulesWithAppendix.length > 0) {
+        lines.push('');
+        lines.push(strings.appendixHeading);
+        for (const [i, rule] of rules.entries()) {
+            if (!rule.appendix)
+                continue;
+            const tagStr = `[${tag}:${i + 1}]`;
+            lines.push('');
+            lines.push(strings.appendixInstruction.replace('{tag}', tagStr));
+            lines.push('```');
+            lines.push(rule.appendix.trimEnd());
+            lines.push('```');
+        }
+    }
+    return lines.join('\n');
+}
+//# sourceMappingURL=status-rules.js.map

package/dist/workflow/status-rules.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"status-rules.js","sourceRoot":"","sources":["../../src/workflow/status-rules.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AAIH,sDAAsD;AACtD,MAAM,oBAAoB,GAAG;IAC3B,EAAE,EAAE;QACF,eAAe,EAAE,sBAAsB;QACvC,SAAS,EAAE,GAAG;QACd,eAAe,EAAE,WAAW;QAC5B,SAAS,EAAE,KAAK;QAChB,aAAa,EAAE,kBAAkB;QACjC,iBAAiB,EAAE,gDAAgD;QACnE,eAAe,EAAE,uBAAuB;QACxC,mBAAmB,EAAE,kDAAkD;KACxE;IACD,EAAE,EAAE;QACF,eAAe,EAAE,SAAS;QAC1B,SAAS,EAAE,GAAG;QACd,eAAe,EAAE,IAAI;QACrB,SAAS,EAAE,IAAI;QACf,aAAa,EAAE,aAAa;QAC5B,iBAAiB,EAAE,qBAAqB;QACxC,eAAe,EAAE,gBAAgB;QACjC,mBAAmB,EAAE,gCAAgC;KACtD;CACO,CAAC;AAEX;;;;;;;;;;;GAWG;AACH,MAAM,UAAU,4BAA4B,CAC1C,QAAgB,EAChB,KAAqB,EACrB,QAAkB;IAElB,MAAM,GAAG,GAAG,QAAQ,CAAC,WAAW,EAAE,CAAC;IACnC,MAAM,OAAO,GAAG,oBAAoB,CAAC,QAAQ,CAAC,CAAC;IAE/C,MAAM,KAAK,GAAa,EAAE,CAAC;IAE3B,iBAAiB;IACjB,KAAK,CAAC,IAAI,CAAC,OAAO,CAAC,eAAe,CAAC,CAAC;IACpC,KAAK,CAAC,IAAI,CAAC,EAAE,CAAC,CAAC;IACf,KAAK,CAAC,IAAI,CAAC,KAAK,OAAO,CAAC,SAAS,MAAM,OAAO,CAAC,eAAe,MAAM,OAAO,CAAC,SAAS,IAAI,CAAC,CAAC;IAC3F,KAAK,CAAC,IAAI,CAAC,qBAAqB,CAAC,CAAC;IAClC,KAAK,MAAM,CAAC,CAAC,EAAE,IAAI,CAAC,IAAI,KAAK,CAAC,OAAO,EAAE,EAAE,CAAC;QACxC,KAAK,CAAC,IAAI,CAAC,KAAK,CAAC,GAAG,CAAC,MAAM,IAAI,CAAC,SAAS,SAAS,GAAG,IAAI,CAAC,GAAG,CAAC,OAAO,CAAC,CAAC;IACzE,CAAC;IACD,KAAK,CAAC,IAAI,CAAC,EAAE,CAAC,CAAC;IAEf,gBAAgB;IAChB,KAAK,CAAC,IAAI,CAAC,OAAO,CAAC,aAAa,CAAC,CAAC;IAClC,KAAK,CAAC,IAAI,CAAC,EAAE,CAAC,CAAC;IACf,KAAK,CAAC,IAAI,CAAC,OAAO,CAAC,iBAAiB,CAAC,CAAC;IACtC,KAAK,CAAC,IAAI,CAAC,EAAE,CAAC,CAAC;IACf,KAAK,MAAM,CAAC,CAAC,EAAE,IAAI,CAAC,IAAI,KAAK,CAAC,OAAO,EAAE,EAAE,CAAC;QACxC,KAAK,CAAC,IAAI,CAAC,QAAQ,GAAG,IAAI,CAAC,GAAG,CAAC,SAAS,IAAI,CAAC,SAAS,EAAE,CAAC,CAAC;IAC5D,CAAC;IAED,kDAAkD;IAClD,MAAM,iBAAiB,GAAG,KAAK,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,QAAQ,CAAC,CAAC;IAC1D,IAAI,iBAAiB,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;QACjC,KAAK,CAAC,IAAI,CAAC,EAAE,CAAC,CAAC;QACf,KAAK,CAAC,IAAI,CAAC,OAAO,CAAC,eAAe,CAAC,CAAC;QACpC,KAAK,MAAM,CAAC,CAAC,EAAE,IAAI,CAAC,IAAI,KAAK,CAAC,OAAO,EAAE,EAAE,CAAC;YACxC,IAAI,CAAC,IAAI,CAAC,QAAQ;gBAAE,SAAS;YAC7B,MAAM,MAAM,GAAG,IAAI,GAAG,IAAI,CAAC,GAAG,CAAC,GAAG,CAAC;YACnC,KAAK,CAAC,IAAI,CAAC,EAAE,CAAC,CAAC;YACf,KAAK,CAAC,IAAI,CAAC,OAAO,CAAC,mBAAmB,CAAC,OAAO,CAAC,OAAO,EAAE,MAAM,CAAC,CAAC,CAAC;YACjE,KAAK,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC;YAClB,KAAK,CAAC,IAAI,CAAC,IAAI,CAAC,QAAQ,CAAC,OAAO,EAAE,CAAC,CAAC;YACpC,KAAK,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC;QACpB,CAAC;IACH,CAAC;IAED,OAAO,KAAK,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;AAC1B,CAAC"}

package/dist/workflow/types.d.ts

@@ -65,8 +65,8 @@ export interface WorkflowEngineOptions {
     onIterationLimit?: IterationLimitCallback;
     /** Bypass all permission checks (sacrifice-my-pc mode) */
     bypassPermissions?: boolean;
-    /** Project root directory (where .takt/ lives). Defaults to cwd if not specified. */
-    projectCwd?: string;
+    /** Project root directory (where .takt/ lives). */
+    projectCwd: string;
     /** Language for instruction metadata. Defaults to 'en'. */
     language?: Language;
     provider?: ProviderType;

package/dist/workflow/types.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"types.d.ts","sourceRoot":"","sources":["../../src/workflow/types.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AAEH,OAAO,KAAK,EAAE,YAAY,EAAE,aAAa,EAAE,aAAa,EAAE,QAAQ,EAAE,MAAM,oBAAoB,CAAC;AAC/F,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,qBAAqB,CAAC;AAC1D,OAAO,KAAK,EAAE,iBAAiB,EAAE,sBAAsB,EAAE,MAAM,sBAAsB,CAAC;AACtF,OAAO,KAAK,EAAE,YAAY,EAAE,MAAM,uBAAuB,CAAC;AAE1D,wCAAwC;AACxC,MAAM,WAAW,cAAc;IAC7B,YAAY,EAAE,CAAC,IAAI,EAAE,YAAY,EAAE,SAAS,EAAE,MAAM,EAAE,WAAW,EAAE,MAAM,KAAK,IAAI,CAAC;IACnF,eAAe,EAAE,CAAC,IAAI,EAAE,YAAY,EAAE,QAAQ,EAAE,aAAa,EAAE,WAAW,EAAE,MAAM,KAAK,IAAI,CAAC;IAC5F,aAAa,EAAE,CAAC,IAAI,EAAE,YAAY,EAAE,QAAQ,EAAE,MAAM,EAAE,QAAQ,EAAE,MAAM,KAAK,IAAI,CAAC;IAChF,cAAc,EAAE,CAAC,IAAI,EAAE,YAAY,EAAE,QAAQ,EAAE,aAAa,KAAK,IAAI,CAAC;IACtE,iBAAiB,EAAE,CAAC,IAAI,EAAE,YAAY,EAAE,SAAS,EAAE,MAAM,KAAK,IAAI,CAAC;IACnE,mBAAmB,EAAE,CAAC,KAAK,EAAE,aAAa,KAAK,IAAI,CAAC;IACpD,gBAAgB,EAAE,CAAC,KAAK,EAAE,aAAa,EAAE,MAAM,EAAE,MAAM,KAAK,IAAI,CAAC;IACjE,iBAAiB,EAAE,CAAC,SAAS,EAAE,MAAM,EAAE,aAAa,EAAE,MAAM,KAAK,IAAI,CAAC;IACtE,oBAAoB,EAAE,CAAC,IAAI,EAAE,YAAY,EAAE,gBAAgB,EAAE,MAAM,KAAK,IAAI,CAAC;CAC9E;AAED,2CAA2C;AAC3C,MAAM,WAAW,gBAAgB;IAC/B,+BAA+B;IAC/B,IAAI,EAAE,YAAY,CAAC;IACnB,0CAA0C;IAC1C,QAAQ,EAAE,aAAa,CAAC;IACxB,2DAA2D;IAC3D,MAAM,EAAE,MAAM,CAAC;CAChB;AAED,8BAA8B;AAC9B,MAAM,WAAW,qBAAqB;IACpC,8BAA8B;IAC9B,gBAAgB,EAAE,MAAM,CAAC;IACzB,6BAA6B;IAC7B,aAAa,EAAE,MAAM,CAAC;IACtB,wBAAwB;IACxB,WAAW,EAAE,MAAM,CAAC;CACrB;AAED,mEAAmE;AACnE,MAAM,MAAM,qBAAqB,GAAG,CAAC,SAAS,EAAE,MAAM,EAAE,SAAS,EAAE,MAAM,KAAK,IAAI,CAAC;AAEnF;;;GAGG;AACH,MAAM,MAAM,sBAAsB,GAAG,CAAC,OAAO,EAAE,qBAAqB,KAAK,OAAO,CAAC,MAAM,GAAG,IAAI,CAAC,CAAC;AAEhG,kCAAkC;AAClC,MAAM,WAAW,qBAAqB;IACpC,8CAA8C;IAC9C,QAAQ,CAAC,EAAE,cAAc,CAAC;IAC1B,kEAAkE;IAClE,WAAW,CAAC,EAAE,CAAC,OAAO,EAAE,gBAAgB,KAAK,OAAO,CAAC,MAAM,GAAG,IAAI,CAAC,CAAC;IACpE,mEAAmE;IACnE,eAAe,CAAC,EAAE,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,CAAC;IACzC,gDAAgD;IAChD,eAAe,CAAC,EAAE,qBAAqB,CAAC;IACxC,mEAAmE;IACnE,mBAAmB,CAAC,EAAE,iBAAiB,CAAC;IACxC,mDAAmD;IACnD,iBAAiB,CAAC,EAAE,MAAM,EAAE,CAAC;IAC7B,8CAA8C;IAC9C,iBAAiB,CAAC,EAAE,sBAAsB,CAAC;IAC3C,+FAA+F;IAC/F,gBAAgB,CAAC,EAAE,sBAAsB,CAAC;IAC1C,0DAA0D;IAC1D,iBAAiB,CAAC,EAAE,OAAO,CAAC;IAC5B,qFAAqF;IACrF,UAAU,CAAC,EAAE,MAAM,CAAC;IACpB,2DAA2D;IAC3D,QAAQ,CAAC,EAAE,QAAQ,CAAC;IACpB,QAAQ,CAAC,EAAE,YAAY,CAAC;IACxB,KAAK,CAAC,EAAE,MAAM,CAAC;CAChB;AAED,4BAA4B;AAC5B,MAAM,WAAW,eAAe;IAC9B,MAAM,EAAE,OAAO,CAAC;IAChB,KAAK,EAAE,MAAM,CAAC;IACd,WAAW,EAAE,OAAO,CAAC;IACrB,UAAU,CAAC,EAAE,OAAO,CAAC;CACtB"}
+{"version":3,"file":"types.d.ts","sourceRoot":"","sources":["../../src/workflow/types.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AAEH,OAAO,KAAK,EAAE,YAAY,EAAE,aAAa,EAAE,aAAa,EAAE,QAAQ,EAAE,MAAM,oBAAoB,CAAC;AAC/F,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,qBAAqB,CAAC;AAC1D,OAAO,KAAK,EAAE,iBAAiB,EAAE,sBAAsB,EAAE,MAAM,sBAAsB,CAAC;AACtF,OAAO,KAAK,EAAE,YAAY,EAAE,MAAM,uBAAuB,CAAC;AAE1D,wCAAwC;AACxC,MAAM,WAAW,cAAc;IAC7B,YAAY,EAAE,CAAC,IAAI,EAAE,YAAY,EAAE,SAAS,EAAE,MAAM,EAAE,WAAW,EAAE,MAAM,KAAK,IAAI,CAAC;IACnF,eAAe,EAAE,CAAC,IAAI,EAAE,YAAY,EAAE,QAAQ,EAAE,aAAa,EAAE,WAAW,EAAE,MAAM,KAAK,IAAI,CAAC;IAC5F,aAAa,EAAE,CAAC,IAAI,EAAE,YAAY,EAAE,QAAQ,EAAE,MAAM,EAAE,QAAQ,EAAE,MAAM,KAAK,IAAI,CAAC;IAChF,cAAc,EAAE,CAAC,IAAI,EAAE,YAAY,EAAE,QAAQ,EAAE,aAAa,KAAK,IAAI,CAAC;IACtE,iBAAiB,EAAE,CAAC,IAAI,EAAE,YAAY,EAAE,SAAS,EAAE,MAAM,KAAK,IAAI,CAAC;IACnE,mBAAmB,EAAE,CAAC,KAAK,EAAE,aAAa,KAAK,IAAI,CAAC;IACpD,gBAAgB,EAAE,CAAC,KAAK,EAAE,aAAa,EAAE,MAAM,EAAE,MAAM,KAAK,IAAI,CAAC;IACjE,iBAAiB,EAAE,CAAC,SAAS,EAAE,MAAM,EAAE,aAAa,EAAE,MAAM,KAAK,IAAI,CAAC;IACtE,oBAAoB,EAAE,CAAC,IAAI,EAAE,YAAY,EAAE,gBAAgB,EAAE,MAAM,KAAK,IAAI,CAAC;CAC9E;AAED,2CAA2C;AAC3C,MAAM,WAAW,gBAAgB;IAC/B,+BAA+B;IAC/B,IAAI,EAAE,YAAY,CAAC;IACnB,0CAA0C;IAC1C,QAAQ,EAAE,aAAa,CAAC;IACxB,2DAA2D;IAC3D,MAAM,EAAE,MAAM,CAAC;CAChB;AAED,8BAA8B;AAC9B,MAAM,WAAW,qBAAqB;IACpC,8BAA8B;IAC9B,gBAAgB,EAAE,MAAM,CAAC;IACzB,6BAA6B;IAC7B,aAAa,EAAE,MAAM,CAAC;IACtB,wBAAwB;IACxB,WAAW,EAAE,MAAM,CAAC;CACrB;AAED,mEAAmE;AACnE,MAAM,MAAM,qBAAqB,GAAG,CAAC,SAAS,EAAE,MAAM,EAAE,SAAS,EAAE,MAAM,KAAK,IAAI,CAAC;AAEnF;;;GAGG;AACH,MAAM,MAAM,sBAAsB,GAAG,CAAC,OAAO,EAAE,qBAAqB,KAAK,OAAO,CAAC,MAAM,GAAG,IAAI,CAAC,CAAC;AAEhG,kCAAkC;AAClC,MAAM,WAAW,qBAAqB;IACpC,8CAA8C;IAC9C,QAAQ,CAAC,EAAE,cAAc,CAAC;IAC1B,kEAAkE;IAClE,WAAW,CAAC,EAAE,CAAC,OAAO,EAAE,gBAAgB,KAAK,OAAO,CAAC,MAAM,GAAG,IAAI,CAAC,CAAC;IACpE,mEAAmE;IACnE,eAAe,CAAC,EAAE,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,CAAC;IACzC,gDAAgD;IAChD,eAAe,CAAC,EAAE,qBAAqB,CAAC;IACxC,mEAAmE;IACnE,mBAAmB,CAAC,EAAE,iBAAiB,CAAC;IACxC,mDAAmD;IACnD,iBAAiB,CAAC,EAAE,MAAM,EAAE,CAAC;IAC7B,8CAA8C;IAC9C,iBAAiB,CAAC,EAAE,sBAAsB,CAAC;IAC3C,+FAA+F;IAC/F,gBAAgB,CAAC,EAAE,sBAAsB,CAAC;IAC1C,0DAA0D;IAC1D,iBAAiB,CAAC,EAAE,OAAO,CAAC;IAC5B,mDAAmD;IACnD,UAAU,EAAE,MAAM,CAAC;IACnB,2DAA2D;IAC3D,QAAQ,CAAC,EAAE,QAAQ,CAAC;IACpB,QAAQ,CAAC,EAAE,YAAY,CAAC;IACxB,KAAK,CAAC,EAAE,MAAM,CAAC;CAChB;AAED,4BAA4B;AAC5B,MAAM,WAAW,eAAe;IAC9B,MAAM,EAAE,OAAO,CAAC;IAChB,KAAK,EAAE,MAAM,CAAC;IACd,WAAW,EAAE,OAAO,CAAC;IACrB,UAAU,CAAC,EAAE,OAAO,CAAC;CACtB"}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "takt",
-  "version": "0.3.6",
+  "version": "0.3.8",
   "description": "TAKT: Task Agent Koordination Tool - AI Agent Workflow Orchestration",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",

package/resources/global/en/agents/default/ai-antipattern-reviewer.md

@@ -105,26 +105,70 @@ AI is confidently wrong—code that looks plausible but doesn't work, solutions
 
 **Principle:** The best code is the minimum code that solves the problem.
 
-### 6. Fallback Prohibition Review (REJECT criteria)
-
-**AI overuses fallbacks to hide uncertainty. This is a REJECT by default.**
-
-| Pattern | Example | Verdict |
-|---------|---------|---------|
-| Swallowing with defaults | `?? 'unknown'`, `\|\| 'default'`, `?? []` | REJECT |
-| try-catch returning empty | `catch { return ''; }` `catch { return 0; }` | REJECT |
-| Silent skip via conditionals | `if (!x) return;` skipping what should be an error | REJECT |
-| Multi-level fallback chains | `a ?? b ?? c ?? d` | REJECT |
+### 6. Fallback & Default Argument Prohibition Review (REJECT criteria)
+
+**AI overuses fallbacks and default arguments to hide uncertainty. Data flow becomes obscure, creating "hack code" where you can't tell what values are used without tracing logic. This is a REJECT by default.**
+
+**Core problem:** You can't understand what values are being used without tracing the entire logic path.
+
+| Pattern | Example | Problem | Verdict |
+|---------|---------|---------|---------|
+| Fallback for required data | `user?.id ?? 'unknown'` | Processing continues in an error state | **REJECT** |
+| Default argument abuse | `function f(x = 'default')` where all callers omit | Data flow obscured | **REJECT** |
+| Nullish coalescing with no upstream path | `options?.cwd ?? process.cwd()` with no way to pass | Always uses fallback (meaningless) | **REJECT** |
+| try-catch returning empty | `catch { return ''; }` | Swallows errors | **REJECT** |
+| Multi-level fallback | `a ?? b ?? c ?? d` | Complex value determination logic | **REJECT** |
+| Silent skip via conditionals | `if (!x) return;` skipping error | Hides bugs | **REJECT** |
+
+**Default argument examples:**
+
+```typescript
+// ❌ Bad example - All callers omit
+function loadWorkflow(name: string, cwd = process.cwd()) { ... }
+// All callers: loadWorkflow('default')  ← not passing cwd
+// Problem: Can't tell where cwd value comes from by reading call sites
+// Fix: Make cwd required, pass explicitly at call sites
+
+// ✅ Good example - Only some callers omit
+function query(sql: string, timeout = 30000) { ... }
+// Caller A: query(sql)  ← Uses default
+// Caller B: query(sql, 60000)  ← Explicit
+// Reason: timeout is explicitly optional configuration
+```
+
+**Nullish coalescing examples:**
+
+```typescript
+// ❌ Bad example - No upstream path to pass value
+class Engine {
+  constructor(config, cwd, task, options?) {
+    this.projectCwd = options?.projectCwd ?? cwd
+    // Problem: If options is passed as { }, projectCwd is always undefined
+    //          Result: always uses cwd (fallback is meaningless)
+  }
+}
+// Fix: Modify upstream function signatures to allow passing options.projectCwd
+
+// ✅ Good example - Upstream path exists
+function execute(task, options?: { projectCwd?: string }) {
+  const cwd = options?.projectCwd ?? process.cwd()
+  // Reason: Caller chooses whether to pass options.projectCwd
+}
+```
 
 **Exceptions (do NOT reject):**
 - Default values when validating external input (user input, API responses)
 - Fallbacks with an explicit comment explaining the reason
-- Defaults for optional values in configuration files
+- Defaults for optional values in configuration files (explicitly designed as optional)
+- Only some callers use default argument (REJECT if all callers omit)
 
 **Verification approach:**
-1. Grep the diff for `??`, `||`, `catch`
-2. Check whether each fallback has a legitimate reason
-3. REJECT if even one unjustified fallback exists
+1. Grep the diff for `??`, `||`, `= defaultValue`, `catch`
+2. For each fallback/default argument:
+   - Is it required data? → REJECT
+   - Do all callers omit it? → REJECT
+   - Is there an upstream path to pass value? → If not, REJECT
+3. REJECT if even one unjustified fallback/default argument exists
 
 ### 7. Unused Code Detection
 

package/resources/global/en/agents/default/architecture-reviewer.md

@@ -8,6 +8,16 @@ Code is read far more often than it is written. Poorly structured code destroys
 
 "If the structure is right, the code naturally follows"—that is the conviction of design review.
 
+## Reviewer Stance
+
+**Never defer even minor issues. If a problem can be fixed now, require it to be fixed now.**
+
+- No compromises for "minor issues". Accumulation of small problems becomes technical debt
+- "Address in next task" never happens. If fixable now, fix now
+- No "conditional approval". If there are issues, reject
+- If you find in-scope fixable issues, flag them without exception
+- Existing issues (unrelated to current change) are non-blocking, but issues introduced or fixable in this change must be flagged
+
 ## Areas of Expertise
 
 ### Structure & Design

package/resources/global/en/agents/default/coder.md

@@ -2,6 +2,28 @@
 
 You are the implementer. **Focus on implementation, not design decisions.**
 
+## Coding Stance
+
+**Thoroughness over speed. Code correctness over implementation ease.**
+
+- Don't hide uncertainty with fallback values (`?? 'unknown'`)
+- Don't obscure data flow with default arguments
+- Prioritize "works correctly" over "works for now"
+- Don't swallow errors; fail fast (Fail Fast)
+- Don't guess; report unclear points
+
+**Reviewer's feedback is absolute. Your understanding is wrong.**
+- If reviewer says "not fixed", first open the file and verify the facts
+- Drop the assumption "I should have fixed it"
+- Fix all flagged issues with Edit tool
+- Don't argue; just comply
+
+**Be aware of AI's bad habits:**
+- Hiding uncertainty with fallbacks → Prohibited (will be flagged in review)
+- Writing unused code "just in case" → Prohibited (will be flagged in review)
+- Making design decisions arbitrarily → Report and ask for guidance
+- Dismissing reviewer feedback → Prohibited (your understanding is wrong)
+
 ## Most Important Rule
 
 **Work only within the specified project directory.**
@@ -107,6 +129,66 @@ Perform self-check after implementation.
 | Boy Scout | Leave touched areas slightly improved |
 | Fail Fast | Detect errors early. Don't swallow them |
 
+## Fallback & Default Argument Prohibition
+
+**Don't write code that obscures data flow. Code where you can't tell values without tracing logic is bad code.**
+
+### Prohibited Patterns
+
+| Pattern | Example | Problem |
+|---------|---------|---------|
+| Fallback for required data | `user?.id ?? 'unknown'` | Processing continues in an error state |
+| Default argument abuse | `function f(x = 'default')` where all callers omit | Can't tell where value comes from |
+| Nullish coalescing with no upstream path | `options?.cwd ?? process.cwd()` with no way to pass | Always uses fallback (meaningless) |
+| try-catch returning empty | `catch { return ''; }` | Swallows errors |
+
+### Correct Implementation
+
+```typescript
+// ❌ Prohibited - Fallback for required data
+const userId = user?.id ?? 'unknown'
+processUser(userId)  // Continues with 'unknown'
+
+// ✅ Correct - Fail Fast
+if (!user?.id) {
+  throw new Error('User ID is required')
+}
+processUser(user.id)
+
+// ❌ Prohibited - Default argument with all callers omitting
+function loadConfig(path = './config.json') { ... }
+// All callers: loadConfig()  ← not passing path
+
+// ✅ Correct - Required argument with explicit passing
+function loadConfig(path: string) { ... }
+// Caller: loadConfig('./config.json')  ← Explicit
+
+// ❌ Prohibited - Nullish coalescing with no upstream path
+class Engine {
+  constructor(config, options?) {
+    this.cwd = options?.cwd ?? process.cwd()
+    // Problem: If no path to pass options.cwd, always uses process.cwd()
+  }
+}
+
+// ✅ Correct - Allow passing from upstream
+function createEngine(config, cwd: string) {
+  return new Engine(config, { cwd })
+}
+```
+
+### Allowed Cases
+
+- Default values when validating external input (user input, API responses)
+- Optional values in configuration files (explicitly designed as optional)
+- Only some callers use default argument (prohibited if all callers omit)
+
+### Decision Criteria
+
+1. **Is it required data?** → Don't fallback, throw error
+2. **Do all callers omit it?** → Remove default argument, make it required
+3. **Is there an upstream path to pass value?** → If not, add argument/field
+
 ## Abstraction Principles
 
 **Before adding conditional branches, consider:**

package/resources/global/en/workflows/default.yaml

@@ -74,9 +74,6 @@ steps:
           - {Question 2}
     pass_previous_response: true
     instruction_template: |
-      ## Previous Response (when returned from implement)
-      {previous_response}
-
       Analyze the task and create an implementation plan.
 
       **Note:** If returned from implement step (Previous Response exists),
@@ -208,19 +205,35 @@ steps:
     permission_mode: acceptEdits
     rules:
       - condition: AI issues fixed
-        next: reviewers
+        next: ai_review
       - condition: Cannot proceed, insufficient info
         next: plan
     instruction_template: |
-      ## AI Review Feedback (This is the latest instruction - prioritize this)
-      {previous_response}
-
-      **Important**: Address the AI Reviewer's feedback.
-      Focus on:
-      - Correcting incorrect assumptions
-      - Fixing plausible-but-wrong implementations
-      - Aligning with existing codebase patterns
-      - Removing scope creep
+      **This is AI Review iteration {step_iteration}.**
+
+      If this is iteration 2 or later, it means your previous fixes were not actually applied.
+      **Your belief that you "already fixed it" is wrong.**
+
+      **First, acknowledge:**
+      - Files you thought were "fixed" are actually not fixed
+      - Your understanding of previous work is incorrect
+      - You need to start from zero
+
+      **Required actions:**
+      1. Open all flagged files with Read tool (drop assumptions, verify facts)
+      2. Search for problem code with grep to confirm it exists
+      3. Fix confirmed problems with Edit tool
+      4. Run tests to verify (`./gradlew :backend:test` etc.)
+      5. Report specifically "what you checked and what you fixed"
+
+      **Report format:**
+      - ❌ "Already fixed"
+      - ✅ "Checked file X at L123, found problem Y, fixed to Z"
+
+      **Absolutely prohibited:**
+      - Reporting "fixed" without opening files
+      - Judging based on assumptions
+      - Leaving problems that AI Reviewer REJECTED
     pass_previous_response: true
 
   - name: reviewers
@@ -234,7 +247,7 @@ steps:
             ```markdown
             # Architecture Review
 
-            ## Result: APPROVE / IMPROVE / REJECT
+            ## Result: APPROVE / REJECT
 
             ## Summary
             {1-2 sentences summarizing result}
@@ -245,17 +258,18 @@ steps:
             - [x] Change Scope
 
             ## Issues (if REJECT)
-            | # | Location | Issue | Fix |
-            |---|----------|-------|-----|
-            | 1 | `src/file.ts:42` | Issue description | Fix method |
+            | # | Scope | Location | Issue | Fix |
+            |---|-------|----------|-------|-----|
+            | 1 | In-scope | `src/file.ts:42` | Issue description | Fix method |
+
+            Scope: "In-scope" (fixable now) / "Out-of-scope" (existing issue, non-blocking)
 
-            ## Improvement Suggestions (optional, non-blocking)
-            - {Future improvement suggestions}
+            ## Existing Issues (informational, non-blocking)
+            - {Record of existing issues unrelated to current change}
             ```
 
             **Cognitive load reduction rules:**
-            - APPROVE + no issues -> Summary only (5 lines or less)
-            - APPROVE + minor suggestions -> Summary + suggestions (15 lines or less)
+            - APPROVE -> Summary only (5 lines or less)
             - REJECT -> Issues in table format (30 lines or less)
         allowed_tools:
           - Read
@@ -347,10 +361,7 @@ steps:
       - condition: Cannot proceed, insufficient info
         next: plan
     instruction_template: |
-      ## Review Feedback (This is the latest instruction - prioritize this)
-      {previous_response}
-
-      **Important**: Address the feedback from the reviewers.
+      Address the feedback from the reviewers.
       The "Original User Request" is reference information, not the latest instruction.
       Review the session conversation history and fix the issues raised by the reviewers.
     pass_previous_response: true

package/resources/global/en/workflows/expert-cqrs.yaml

@@ -64,9 +64,6 @@ steps:
       - WebSearch
       - WebFetch
     instruction_template: |
-      ## Previous Response (when returned from implement)
-      {previous_response}
-
       Analyze the task and create an implementation plan.
 
       **Note:** If returned from implement step (Previous Response exists),
@@ -146,6 +143,7 @@ steps:
   - name: ai_review
     edit: false
     agent: ../agents/default/ai-antipattern-reviewer.md
+    pass_previous_response: true
     report:
       name: 03-ai-review.md
       format: |
@@ -206,15 +204,31 @@ steps:
       - WebSearch
       - WebFetch
     instruction_template: |
-      ## AI Review Feedback (This is the latest instruction - prioritize this)
-      {previous_response}
-
-      **Important**: Address the AI Reviewer's feedback.
-      Focus on:
-      - Correcting incorrect assumptions
-      - Fixing plausible-but-wrong implementations
-      - Aligning with existing codebase patterns
-      - Removing scope creep
+      **This is AI Review iteration {step_iteration}.**
+
+      If this is iteration 2 or later, it means your previous fixes were not actually applied.
+      **Your belief that you "already fixed it" is wrong.**
+
+      **First, acknowledge:**
+      - Files you thought were "fixed" are actually not fixed
+      - Your understanding of previous work is incorrect
+      - You need to start from zero
+
+      **Required actions:**
+      1. Open all flagged files with Read tool (drop assumptions, verify facts)
+      2. Search for problem code with grep to confirm it exists
+      3. Fix confirmed problems with Edit tool
+      4. Run tests to verify (`./gradlew :backend:test` etc.)
+      5. Report specifically "what you checked and what you fixed"
+
+      **Report format:**
+      - ❌ "Already fixed"
+      - ✅ "Checked file X at L123, found problem Y, fixed to Z"
+
+      **Absolutely prohibited:**
+      - Reporting "fixed" without opening files
+      - Judging based on assumptions
+      - Leaving problems that AI Reviewer REJECTED
     pass_previous_response: true
     rules:
       - condition: AI Reviewer's issues have been fixed
@@ -251,9 +265,14 @@ steps:
             | Eventual Consistency | ✅ | - |
 
             ## Issues (if REJECT)
-            | # | Location | Issue | Fix |
-            |---|----------|-------|-----|
-            | 1 | `src/file.ts:42` | Issue description | Fix method |
+            | # | Scope | Location | Issue | Fix |
+            |---|-------|----------|-------|-----|
+            | 1 | In-scope | `src/file.ts:42` | Issue description | Fix method |
+
+            Scope: "In-scope" (fixable now) / "Out-of-scope" (existing issue, non-blocking)
+
+            ## Existing Issues (informational, non-blocking)
+            - {Record of existing issues unrelated to current change}
             ```
         allowed_tools:
           - Read
@@ -455,10 +474,7 @@ steps:
       - condition: Cannot proceed, insufficient info
         next: plan
     instruction_template: |
-      ## Review Feedback (This is the latest instruction - prioritize this)
-      {previous_response}
-
-      **Important**: Address the feedback from the reviewers.
+      Address the feedback from the reviewers.
       The "Original User Request" is reference information, not the latest instruction.
       Review the session conversation history and fix the issues raised by the reviewers.
     pass_previous_response: true
@@ -572,10 +588,7 @@ steps:
       - WebSearch
       - WebFetch
     instruction_template: |
-      ## Supervisor Feedback (This is the latest instruction - prioritize this)
-      {previous_response}
-
-      **Important**: Fix the issues pointed out by the supervisor.
+      Fix the issues pointed out by the supervisor.
 
       The supervisor has identified issues from a big-picture perspective.
       Address items in priority order.

package/resources/global/en/workflows/expert.yaml

@@ -76,9 +76,6 @@ steps:
       - WebSearch
       - WebFetch
     instruction_template: |
-      ## Previous Response (when returned from implement)
-      {previous_response}
-
       Analyze the task and create an implementation plan.
 
       **Note:** If returned from implement step (Previous Response exists),
@@ -158,6 +155,7 @@ steps:
   - name: ai_review
     edit: false
     agent: ../agents/default/ai-antipattern-reviewer.md
+    pass_previous_response: true
     report:
       name: 03-ai-review.md
       format: |
@@ -218,14 +216,31 @@ steps:
       - WebSearch
       - WebFetch
     instruction_template: |
-      ## AI Review Feedback (This is the latest instruction - prioritize this)
-      {previous_response}
-
-      **Important**: Address the AI Reviewer's feedback.
-      Focus on:
-      - Correcting incorrect assumptions
-      - Fixing plausible-but-wrong implementations
-      - Aligning with existing codebase patterns
+      **This is AI Review iteration {step_iteration}.**
+
+      If this is iteration 2 or later, it means your previous fixes were not actually applied.
+      **Your belief that you "already fixed it" is wrong.**
+
+      **First, acknowledge:**
+      - Files you thought were "fixed" are actually not fixed
+      - Your understanding of previous work is incorrect
+      - You need to start from zero
+
+      **Required actions:**
+      1. Open all flagged files with Read tool (drop assumptions, verify facts)
+      2. Search for problem code with grep to confirm it exists
+      3. Fix confirmed problems with Edit tool
+      4. Run tests to verify (`./gradlew :backend:test` etc.)
+      5. Report specifically "what you checked and what you fixed"
+
+      **Report format:**
+      - ❌ "Already fixed"
+      - ✅ "Checked file X at L123, found problem Y, fixed to Z"
+
+      **Absolutely prohibited:**
+      - Reporting "fixed" without opening files
+      - Judging based on assumptions
+      - Leaving problems that AI Reviewer REJECTED
       - Removing scope creep
     pass_previous_response: true
     rules:
@@ -248,7 +263,7 @@ steps:
             ```markdown
             # Architecture Review
 
-            ## Result: APPROVE / IMPROVE / REJECT
+            ## Result: APPROVE / REJECT
 
             ## Summary
             {1-2 sentences summarizing result}
@@ -262,17 +277,18 @@ steps:
             - [x] Call Chain Verification
 
             ## Issues (if REJECT)
-            | # | Location | Issue | Fix |
-            |---|----------|-------|-----|
-            | 1 | `src/file.ts:42` | Issue description | Fix method |
+            | # | Scope | Location | Issue | Fix |
+            |---|-------|----------|-------|-----|
+            | 1 | In-scope | `src/file.ts:42` | Issue description | Fix method |
+
+            Scope: "In-scope" (fixable now) / "Out-of-scope" (existing issue, non-blocking)
 
-            ## Improvement Suggestions (optional - non-blocking)
-            - {Future improvement suggestions}
+            ## Existing Issues (informational, non-blocking)
+            - {Record of existing issues unrelated to current change}
             ```
 
             **Cognitive load reduction rules:**
-            - APPROVE + no issues -> Summary only (5 lines or less)
-            - APPROVE + minor suggestions -> Summary + suggestions (15 lines or less)
+            - APPROVE -> Summary only (5 lines or less)
             - REJECT -> Issues in table format (30 lines or less)
         allowed_tools:
           - Read
@@ -471,10 +487,7 @@ steps:
       - condition: Cannot proceed, insufficient info
         next: plan
     instruction_template: |
-      ## Review Feedback (This is the latest instruction - prioritize this)
-      {previous_response}
-
-      **Important**: Address the feedback from the reviewers.
+      Address the feedback from the reviewers.
       The "Original User Request" is reference information, not the latest instruction.
       Review the session conversation history and fix the issues raised by the reviewers.
     pass_previous_response: true
@@ -588,10 +601,7 @@ steps:
       - WebSearch
       - WebFetch
     instruction_template: |
-      ## Supervisor Feedback (This is the latest instruction - prioritize this)
-      {previous_response}
-
-      **Important**: Fix the issues pointed out by the supervisor.
+      Fix the issues pointed out by the supervisor.
 
       The supervisor has identified issues from a big-picture perspective.
       Address items in priority order.