takt 0.32.0 → 0.32.2

package/builtins/ja/facets/policies/coding.md

@@ -204,13 +204,35 @@ return storage.upload(file, options)
 - 50行超のUI/ロジック → 分離
 - 複数の責務がある → 分離
 
+### 機能追加時の到達経路
+
+新しい機能や画面を追加したら、実装と同じ変更セットで利用者が到達する経路も更新する。フレームワーク固有の配線方法は各ドメイン知識に従う。
+
+| 基準 | 判定 |
+|------|------|
+| 新機能の実装だけ追加し、呼び出し側・導線・到達経路の更新を忘れる | REJECT |
+| 利用者がどこから到達するか未定義のまま公開機能を追加する | REJECT |
+| 実装追加と同じ変更セットで導線と到達経路を更新する | OK |
+| 一時導線を追加した場合、その用途と除去条件を記録する | OK |
+
 ### 依存の方向
 
 - 上位層 → 下位層（逆方向禁止）
 - データ取得はルート（View/Controller）で行い、子に渡す
 - 子は親のことを知らない
 
-### 状態管理
+### 実行条件と依存条件の一致
+
+依存やトリガーは、実際にその処理を再実行したい条件と一致させる。静的ルールや実装都合のためだけに依存を増やし、意図しない再実行を起こさない。
+
+| 基準 | 判定 |
+|------|------|
+| lint や実装都合だけで依存やトリガーを増やし、再実行ループを生む | REJECT |
+| 無関係な state 変化や callback 再生成で初期処理が再実行される | REJECT |
+| 再実行条件が URL・フィルタ・明示的更新操作などの仕様に対応している | OK |
+| 初期化と再取得のトリガーを分けて設計している | OK |
+
+## 状態管理
 
 - 状態は使う場所に閉じ込める
 - 子は状態を直接変更しない（イベントを親に通知）
@@ -305,6 +327,73 @@ function formatDate(date: Date): string { ... }
 function formatPercentage(value: number): string { ... }
 ```
 
+## 同一実装の別名関数（DRY 違反）
+
+AIは同じ処理を異なる関数名で複数定義しがちである。
+
+| パターン | 例 | 判定 |
+|---------|-----|------|
+| 同一実装の別名関数 | `copyFacets()` と `placeFacetFiles()` が同じ処理 | REJECT |
+| 引数シグネチャが同一で本体も同一 | 2つの関数が同じパラメータを受け取り同じ処理を行う | REJECT |
+
+```typescript
+// REJECT - 同じ実装が別名で存在
+function copyFiles(src: string, dest: string): void {
+  for (const f of readdirSync(src)) {
+    copyFileSync(join(src, f), join(dest, f));
+  }
+}
+function placeFiles(src: string, dest: string): void {
+  for (const f of readdirSync(src)) {
+    copyFileSync(join(src, f), join(dest, f));
+  }
+}
+
+// OK - 1つの関数にまとめる
+function copyFiles(src: string, dest: string): void {
+  for (const f of readdirSync(src)) {
+    copyFileSync(join(src, f), join(dest, f));
+  }
+}
+```
+
+検証アプローチ:
+1. 新規追加された関数の本体が、既存関数と同一または酷似していないか確認
+2. 同じファイル内の関数同士、および同じモジュール内の関数同士を比較
+3. 重複があれば1つにまとめ、呼び出し元を統一
+
+## Stateful Regex の危険なパターン
+
+`/g` フラグ付き正規表現はステートフル（`lastIndex` を保持する）。モジュールスコープに定義して `test()` と `replace()` を混用すると予期しない結果になる。
+
+| パターン | 例 | 判定 |
+|---------|-----|------|
+| モジュールスコープの `/g` 正規表現を `test()` で使用 | `const RE = /x/g; if (RE.test(s)) ...` | REJECT |
+| `/g` 正規表現を `test()` と `replace()` で使い回し | `RE.test(s)` の後に `s.replace(RE, ...)` | REJECT |
+
+```typescript
+// REJECT - モジュールスコープの /g 正規表現を test() で使用
+const PATTERN = /\{\{facet:(\w+)\}\}/g;
+function hasFacetRef(text: string): boolean {
+  return PATTERN.test(text);  // lastIndex が進み、次回の呼び出しで結果が変わる
+}
+
+// OK - test() には /g を付けない、または関数内で new RegExp
+const PATTERN_CHECK = /\{\{facet:(\w+)\}\}/;  // /g なし
+const PATTERN_REPLACE = /\{\{facet:(\w+)\}\}/g;  // replace 用は /g
+function hasFacetRef(text: string): boolean {
+  return PATTERN_CHECK.test(text);
+}
+function replaceFacetRefs(text: string): string {
+  return text.replace(PATTERN_REPLACE, ...);
+}
+```
+
+検証アプローチ:
+1. モジュールスコープの正規表現に `/g` フラグがあるか確認
+2. `/g` 付き正規表現が `test()` で使われていないか確認
+3. 同一の正規表現が `test()` と `replace()` の両方で使われていないか確認
+
 ## 禁止事項
 
 - **フォールバックは原則禁止** - `?? 'unknown'`、`|| 'default'`、`try-catch` で握りつぶすフォールバックを書かない。エラーは上位に伝播させる。どうしても必要な場合はコメントで理由を明記する

package/builtins/ja/facets/policies/design-fidelity.md

@@ -0,0 +1,51 @@
+# デザイン忠実再現ポリシー
+
+デザイン参照が指定されている場合、UIの見た目・構造・文言をデザインに一致させる。
+
+## 原則
+
+| 原則 | 基準 |
+|------|------|
+| デザインが正 | デザイン参照が指定されている場合、それがUIの唯一のソース・オブ・トゥルース |
+| 要素の一致 | デザインに存在する要素はすべて実装する。省略しない |
+| 独自追加の禁止 | デザインにない要素を独自判断で追加しない |
+| 文言の一致 | ラベル・ボタン文言・プレースホルダーはデザインの文言をそのまま使う |
+| レイアウトの一致 | 要素の配置順序・並び方向・階層構造をデザインに合わせる |
+| 色・間隔の一致 | 色値・余白・角丸・フォントサイズはデザインの指定に従う |
+
+## 適用条件
+
+このポリシーはタスク指示書またはその参照資料にデザイン参照（UIサンプル、モックアップ、デザインファイル等）が含まれている場合に適用する。デザイン参照がない場合、このポリシーは適用されない。
+
+## 判定基準
+
+| 基準 | 判定 |
+|------|------|
+| デザインにある要素が実装から欠落 | REJECT |
+| デザインにない要素が独自に追加されている | REJECT |
+| 文言がデザインと異なる | REJECT |
+| 要素の配置順序がデザインと異なる | REJECT |
+| 色値がデザインと異なる | REJECT |
+| 余白・間隔がデザインと明らかに異なる | REJECT |
+| デザインにないエッジケースUI（ローディング、エラー、空状態）の追加 | OK |
+| デザインが曖昧な箇所の解釈（判断根拠を記録すること） | OK |
+
+## 実装時の手順
+
+1. デザイン参照の要素を列挙する（画面単位・セクション単位）
+2. 各要素のレイアウト・文言・色・間隔を確認する
+3. 実装がデザインと一致していることを要素単位で照合する
+4. デザインが曖昧な箇所は判断根拠を決定ログに記録する
+
+## レビュー時の手順
+
+1. デザイン参照と実装を要素単位で比較する
+2. 差分がある場合、各差分について「デザインの指定」と「実装の状態」を具体的に記述する
+3. 差分が意図的かどうかを決定ログで確認する
+4. 意図的でない差分はブロッキング問題として報告する
+
+## 許容されるケース
+
+- デザインに含まれないエッジケースのUI（ローディング状態、エラー表示、データが空の場合の表示）
+- プラットフォーム固有の制約による代替実装（判断根拠を記録すること）
+- デザインの解像度が不十分な箇所の合理的な解釈（判断根拠を記録すること）

package/builtins/ja/facets/policies/design-planning.md

@@ -0,0 +1,52 @@
+# デザイン計画ポリシー
+
+デザイン参照が指定されている計画では、要素の棚卸しとスコープ判断を曖昧にしない。
+
+## 原則
+
+| 原則 | 基準 |
+|------|------|
+| 参照優先 | デザイン参照がある場合、計画は参照資料を一次情報として扱う |
+| 要素単位の棚卸し | 画面単位ではなく要素単位で差分を確認する |
+| 変更要否の明示 | 各要素について変更要/不要を根拠付きで示す |
+| スコープ外の説明責任 | デザインにある要素を外す場合は除外理由を明示する |
+| 実装境界の明確化 | 実装対象と非対象の境界を計画段階で固定する |
+
+## 適用条件
+
+このポリシーはタスク指示書または参照資料にデザイン参照が含まれている計画に適用する。デザイン参照がない計画には適用しない。
+
+## 判定基準
+
+| 基準 | 判定 |
+|------|------|
+| デザイン参照の主要要素を列挙せず計画している | REJECT |
+| 要素ごとの変更要/不要がない | REJECT |
+| 「今回は一覧のみ」など、要素名なしにスコープを狭めている | REJECT |
+| スコープ外要素の除外理由がない | REJECT |
+| デザイン参照の解釈が曖昧だが、判断根拠を残している | OK |
+
+## 計画時の判断
+
+計画では、デザイン参照に含まれる要素を落とさず棚卸しする。
+
+棚卸し時の観点:
+- 主要セクションだけでなく、詳細導線、モーダル、操作要素、状態表示まで確認する
+- 各要素ごとに、現行実装との差分と変更要否を記録する
+- 既存実装を維持する場合も、対象ファイルと根拠を示す
+
+## スコープ判断
+
+デザイン参照の要素をスコープ外にする場合は、少なくとも以下を計画に残す。
+
+- 除外する要素名
+- 除外理由
+- 今回代替実装を採らない理由
+
+デザイン参照に含まれる要素を、根拠なく「別タスク」とみなして切り離さない。
+
+## 禁止事項
+
+- **画面単位の雑な要約だけで計画すること** - 要素の抜け漏れが発生する
+- **根拠なしに参照資料の意図を狭めること** - 実装とレビューの前提がずれる
+- **スコープ外の説明を省略すること** - 後続ムーブメントで判断不能になる

package/builtins/ja/facets/policies/qa.md

@@ -26,3 +26,18 @@
 | 理由なしの @ts-ignore, @ts-expect-error | 警告 |
 | 理由なしの eslint-disable | 警告 |
 | 非推奨APIの使用 | 警告 |
+
+## 書き込み後の副作用チェック
+
+ファイルやディレクトリへの書き込み後にスキャン・読み取りを行う処理がある場合、書き込んだファイルが意図せずスキャン対象に含まれていないか確認する。
+
+| パターン | 例 | 判定 |
+|---------|-----|------|
+| 出力先ディレクトリのスキャン | facets をコピーした後に同ディレクトリを走査して構文を検出 | REJECT |
+| 一時ファイルの読み戻し | 一時ディレクトリに書き込み後、同ディレクトリの全ファイルを処理 | REJECT |
+| 自己参照的な処理 | 生成したファイルが次の処理パイプラインの入力になる | 警告 |
+
+検証アプローチ:
+1. ファイル書き込み後にディレクトリスキャン（readdir, glob 等）を行う箇所を確認
+2. スキャン対象に書き込み先が含まれていないか確認
+3. 含まれている場合、除外フィルタが存在するか確認

package/builtins/ja/facets/policies/testing.md

@@ -28,7 +28,9 @@
 |--------|------|
 | 高 | ビジネスロジック、状態遷移 |
 | 中 | エッジケース、エラーハンドリング |
-| 低 | 単純なCRUD、UIの見た目 |
+| 低 | 単純なCRUD |
+
+**注意:** デザイン参照が指定されている場合、UIの見た目の検証は中優先度に格上げする。デザイン忠実再現ポリシーを参照。
 
 ## テスト構造: Given-When-Then
 
@@ -64,6 +66,37 @@ test('ユーザーが存在しない場合、NotFoundエラーを返す', async
 - Arrange-Act-Assert パターン（Given-When-Then と同義）
 - マジックナンバー・マジックストリングを避ける
 
+## 再取得ループのリグレッション
+
+画面の初期取得がある場合、無関係な再レンダ、loading 切替、Context callback の参照変化で API が再実行されないことをテストで担保する。
+
+| 基準 | 判定 |
+|------|------|
+| 初期取得バグ修正に対し、重複 API 呼び出しの回帰テストがない | REJECT |
+| 1回呼ばれたことだけを確認し、再レンダ後の安定性を見ていない | 警告 |
+| rerender や state 更新後も呼び出し回数が増えないことを検証している | OK |
+
+## 到達経路のリグレッション
+
+利用者向け機能や画面を追加・変更した場合、利用者がその機能へ到達できることをテストまたは同等の検証で担保する。
+
+| 基準 | 判定 |
+|------|------|
+| 新規の画面・機能を追加したのに、到達経路や起動条件の検証がない | REJECT |
+| 画面ファイル単体の描画だけを見て、入口からの到達確認をしていない | 警告 |
+| route、メニュー、ボタン、リンク、外部呼び出しなど実際の入口から対象機能へ到達できることを確認している | OK |
+
+## UIライブラリ統合のリグレッション
+
+DataGrid、日付ピッカー、仮想リスト、チャートなど、外部 UI ライブラリの主要コンポーネントを導入・変更した場合は、実コンポーネントをマウントするテストでクラッシュしないことを担保する。
+
+| 基準 | 判定 |
+|------|------|
+| 外部 UI ライブラリの主要コンポーネントを追加・変更したのに、実マウントの回帰テストがない | REJECT |
+| ライブラリの props 整合性を、浅いモックや存在確認だけで済ませている | 警告 |
+| route から対象画面を描画し、主要 UI が例外なくマウントされることを確認している | OK |
+| 主要 UI コンポーネント単体でも、代表的な props で実 render している | OK |
+
 ## テスト戦略
 
 - ロジックにはユニットテスト、境界にはインテグレーションテストを優先
@@ -81,6 +114,26 @@ test('ユーザーが存在しない場合、NotFoundエラーを返す', async
 | 新しいオプションが呼び出しチェーンを通じて末端まで伝搬する | チェーン全体の結合テスト必須 |
 | 各モジュールのユニットテストが全てパスしている | ユニットテストのみで十分（上記に該当しない場合） |
 
+## ユニットテスト基準
+
+| 基準 | 判定 |
+|------|------|
+| テスト対象の内部実装をモックする（振る舞いではなく実装を検証） | REJECT |
+| テスト間でフィクスチャを共有して変更する | REJECT。テスト独立性の喪失 |
+| モックの戻り値が実際の型と乖離している | 警告。型安全なモックを使う |
+| 正常系のみテストして境界値がない | 警告 |
+
+## E2Eテスト基準
+
+| 基準 | 判定 |
+|------|------|
+| 外部API呼び出しをモックせず本番APIを叩く | REJECT。テストの再現性が失われる |
+| テスト対象のコア処理をモックする | REJECT。E2Eの意味がなくなる |
+| 固定 sleep でタイミングを合わせる | REJECT。状態ベースの待機を使う |
+| テスト間で共有状態を持つ | 警告。テストの独立性が損なわれる |
+| 正常フローだけテストして異常フローがない | 警告 |
+| ユニットテストでカバーできるロジック検証をE2Eで書く | 警告 |
+
 ## テスト環境の分離
 
 テストインフラの設定はテストシナリオのパラメータに連動させる。ハードコードされた前提は別シナリオで壊れる。

package/builtins/ja/piece-categories.yaml

@@ -47,10 +47,11 @@ piece_categories:
       - review-fix-backend-cqrs
       - review-takt-default
       - review-fix-takt-default
+      - security-audit
   🧪 テスト:
     pieces:
-      - unit-test
-      - e2e-test
+      - fill-unit
+      - fill-e2e
   🎵 TAKT開発:
     pieces:
       - takt-default

package/builtins/ja/pieces/backend-cqrs.yaml

@@ -70,6 +70,8 @@ movements:
       - backend
       - cqrs-es
       - architecture
+      - unit-testing
+      - e2e-testing
     provider_options:
       claude:
         allowed_tools:
@@ -244,6 +246,9 @@ movements:
         policy:
           - review
           - testing
+        knowledge:
+          - unit-testing
+          - e2e-testing
         provider_options:
           claude:
             allowed_tools:

package/builtins/ja/pieces/backend.yaml

@@ -68,6 +68,8 @@ movements:
     knowledge:
       - backend
       - architecture
+      - unit-testing
+      - e2e-testing
     provider_options:
       claude:
         allowed_tools:
@@ -240,6 +242,9 @@ movements:
         policy:
           - review
           - testing
+        knowledge:
+          - unit-testing
+          - e2e-testing
         provider_options:
           claude:
             allowed_tools:

package/builtins/ja/pieces/default.yaml

@@ -83,6 +83,8 @@ movements:
       - testing
     knowledge:
       - architecture
+      - unit-testing
+      - e2e-testing
     provider_options:
       claude:
         allowed_tools:

package/builtins/ja/pieces/dual-cqrs-mini.yaml

@@ -25,6 +25,8 @@ movements:
   - name: plan
     edit: false
     persona: planner
+    policy:
+      - design-planning
     knowledge:
       - frontend
       - backend
@@ -51,13 +53,14 @@ movements:
     output_contracts:
       report:
         - name: plan.md
-          format: plan
+          format: plan-frontend
   - name: implement
     edit: true
     persona: coder
     policy:
       - coding
       - testing
+      - design-fidelity
     knowledge:
       - frontend
       - backend
@@ -163,6 +166,7 @@ movements:
         policy:
           - coding
           - testing
+          - design-fidelity
         knowledge:
           - frontend
           - backend
@@ -191,6 +195,7 @@ movements:
         policy:
           - coding
           - testing
+          - design-fidelity
         knowledge:
           - frontend
           - backend
@@ -223,6 +228,7 @@ movements:
     policy:
       - coding
       - testing
+      - design-fidelity
     knowledge:
       - frontend
       - backend
@@ -256,6 +262,7 @@ movements:
     policy:
       - coding
       - testing
+      - design-fidelity
     knowledge:
       - frontend
       - backend

package/builtins/ja/pieces/dual-cqrs.yaml

@@ -37,6 +37,8 @@ movements:
   - name: plan
     edit: false
     persona: planner
+    policy:
+      - design-planning
     knowledge:
       - frontend
       - backend
@@ -60,13 +62,14 @@ movements:
     output_contracts:
       report:
         - name: plan.md
-          format: plan
+          format: plan-frontend
   - name: implement
     edit: true
     persona: coder
     policy:
       - coding
       - testing
+      - design-fidelity
     session: refresh
     knowledge:
       - frontend
@@ -133,6 +136,7 @@ movements:
     policy:
       - coding
       - testing
+      - design-fidelity
     session: refresh
     knowledge:
       - frontend
@@ -204,7 +208,9 @@ movements:
       - name: frontend-review
         edit: false
         persona: frontend-reviewer
-        policy: review
+        policy:
+          - review
+          - design-fidelity
         knowledge: frontend
         provider_options:
           claude:
@@ -276,6 +282,7 @@ movements:
     policy:
       - coding
       - testing
+      - design-fidelity
     knowledge:
       - frontend
       - backend
@@ -333,6 +340,7 @@ movements:
     policy:
       - coding
       - testing
+      - design-fidelity
     knowledge:
       - frontend
       - backend

package/builtins/ja/pieces/dual-mini.yaml

@@ -25,6 +25,8 @@ movements:
   - name: plan
     edit: false
     persona: planner
+    policy:
+      - design-planning
     knowledge:
       - frontend
       - backend
@@ -50,13 +52,14 @@ movements:
     output_contracts:
       report:
         - name: plan.md
-          format: plan
+          format: plan-frontend
   - name: implement
     edit: true
     persona: coder
     policy:
       - coding
       - testing
+      - design-fidelity
     knowledge:
       - frontend
       - backend
@@ -160,6 +163,7 @@ movements:
         policy:
           - coding
           - testing
+          - design-fidelity
         knowledge:
           - frontend
           - backend
@@ -187,6 +191,7 @@ movements:
         policy:
           - coding
           - testing
+          - design-fidelity
         knowledge:
           - frontend
           - backend
@@ -218,6 +223,7 @@ movements:
     policy:
       - coding
       - testing
+      - design-fidelity
     knowledge:
       - frontend
       - backend
@@ -250,6 +256,7 @@ movements:
     policy:
       - coding
       - testing
+      - design-fidelity
     knowledge:
       - frontend
       - backend

package/builtins/ja/pieces/dual.yaml

@@ -37,6 +37,8 @@ movements:
   - name: plan
     edit: false
     persona: planner
+    policy:
+      - design-planning
     knowledge:
       - frontend
       - backend
@@ -63,7 +65,7 @@ movements:
     output_contracts:
       report:
         - name: plan.md
-          format: plan
+          format: plan-frontend
   - name: write_tests
     edit: true
     persona: coder
@@ -74,6 +76,8 @@ movements:
       - frontend
       - backend
       - architecture
+      - unit-testing
+      - e2e-testing
     provider_options:
       claude:
         allowed_tools:
@@ -110,6 +114,7 @@ movements:
     policy:
       - coding
       - testing
+      - design-fidelity
     session: refresh
     knowledge:
       - frontend
@@ -191,6 +196,7 @@ movements:
     policy:
       - coding
       - testing
+      - design-fidelity
     session: refresh
     knowledge:
       - frontend
@@ -263,7 +269,9 @@ movements:
       - name: frontend-review
         edit: false
         persona: frontend-reviewer
-        policy: review
+        policy:
+          - review
+          - design-fidelity
         knowledge: frontend
         provider_options:
           claude:
@@ -288,6 +296,9 @@ movements:
         policy:
           - review
           - testing
+        knowledge:
+          - unit-testing
+          - e2e-testing
         provider_options:
           claude:
             allowed_tools:
@@ -389,6 +400,7 @@ movements:
     policy:
       - coding
       - testing
+      - design-fidelity
     knowledge:
       - frontend
       - backend