Skip to content
Merged
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
21 changes: 13 additions & 8 deletions .takt/facets/instructions/aggregate-feedback.md
Original file line number Diff line number Diff line change
Expand Up @@ -44,7 +44,7 @@ PR タイトルが context に含まれていない場合は、レポート内
1. **重複検出**: 同じ `Target` + 似た `Description` の提案はマージする
2. **根拠統合**: マージした提案の `Rationale` カラムには複数ソース (PR diff / session / prepush) を併記する
3. **Tier 並び**: 最終リストは Tier 1 → Tier 2 → Tier 3 の順
4. **品質フィルタ**: 以下の提案は除外する (Recommendation 列での `❌ 却下` と区別: ここで除外するのは「最初から表に乗せない」レベル)
4. **品質フィルタ**: 以下の提案は除外する (Recommendation 列での `❌ 却下推奨` と区別: ここで除外するのは「最初から表に乗せない」レベル)
- 一般的なベストプラクティスの押し付け (具体的根拠がない)
- すでに hooks-config.toml / custom-lint-rules.toml に存在するルール (Read で確認可能)
- 対象ファイルが read-only zone (`.takt/`, `docs/adr/`, `templates/`) のみで具体的な編集箇所が示せないもの
Expand All @@ -55,6 +55,8 @@ PR タイトルが context に含まれていない場合は、レポート内

各提案について、以下の rubric に基づいて 4 つの判定列を埋める。**この評価は採用判定をユーザーへ委ねるための材料**であり、AI が判定を独占するわけではない。明確に判定できない場合は中庸な値 (`Medium` / `🤔 様子見`) を選び、`Rationale` で不確実性を明示する。

> **AI agent への明示禁則**: 本 report の生成完了 = ユーザーへの提示完了に過ぎず、Claude / Codex / Opencode 等の agent は `✅ 採用候補` を読んだだけで採用処理 (`docs/todo*.md` への entry 追加 / 実装着手 / ADR 編集 等) に進んではならない。**必ずユーザーの明示承認 (AskUserQuestion 回答 or テキスト承認のいずれか) を待つこと**。本 report の Recommendation 列は analyzer 推奨であり、確定判断ではない。

### Severity rubric

| 値 | 該当する状況 |
Expand Down Expand Up @@ -108,9 +110,9 @@ PR タイトルが context に含まれていない場合は、レポート内

| 値 | 該当する状況 |
|---|---|
| `✅ 採用` | `(Effort ∈ {XS, S, M})` AND `(Severity ∈ {Medium, High, Critical} OR Frequency ∈ {Medium, High})` AND `(Adoption Risk が weak)` |
| `✅ 採用候補` | `(Effort ∈ {XS, S, M})` AND `(Severity ∈ {Medium, High, Critical} OR Frequency ∈ {Medium, High})` AND `(Adoption Risk が weak)`。**判定は analyzer 推奨であり、ユーザー承認後に採用確定**。 |
| `🤔 様子見` | 採用根拠は弱いが将来発生時に再評価したい (一般原則 / 不確実性高 / dogfood トリガ待ち / Severity 高だが Frequency Very Low 等)。✅ にも ❌ にも振り切れない場合の中庸 |
| `❌ 却下` | `(Frequency ∈ {Low, Very Low} AND Effort ∈ {L, XL})` OR `(Adoption Risk が strong)` OR `(実害観測前の preventive over-engineering)` |
| `❌ 却下推奨` | `(Frequency ∈ {Low, Very Low} AND Effort ∈ {L, XL})` OR `(Adoption Risk が strong)` OR `(実害観測前の preventive over-engineering)`。**判定は analyzer 推奨であり、ユーザー承認後に却下確定** (Claude 単独で却下処理しない)。 |

**Adoption Risk の「weak / strong」定義** (上記条件式で参照):

Expand All @@ -131,9 +133,9 @@ PR タイトルが context に含まれていない場合は、レポート内

例:

- 採用 例: `PR diff; Session; collect_all_violations の MAX_VIOLATIONS contract を test 化、将来の lint 追加時の regression 防止網。Effort S かつ Frequency Medium`
- 採用候補 例: `PR diff; Session; collect_all_violations の MAX_VIOLATIONS contract を test 化、将来の lint 追加時の regression 防止網。Effort S かつ Frequency Medium で採用候補と判定`
- 様子見 例: `Session; Honesty constraint で抑制中、実観測 0 件、dogfood で虚偽申告観測後に着手`
- 却下 例: `Session; 1 観測の局所 artifact、汎用 regex は英語固有名詞・略語で誤検出確実、ROI 不見合い`
- 却下推奨 例: `Session; 1 観測の局所 artifact、汎用 regex は英語固有名詞・略語で誤検出確実、ROI 不見合い`

---

Expand All @@ -152,7 +154,7 @@ PR タイトルが context に含まれていない場合は、レポート内

| # | Type | Description | Target | Severity | Frequency | Effort | Adoption Risk | Recommendation | Rationale |
|---|------|-------------|--------|----------|-----------|--------|---------------|----------------|-----------|
| 1 | custom_lint_rule | ... | .claude/custom-lint-rules.toml | Medium | High | S | None | ✅ 採用 | PR diff; Session; ... |
| 1 | custom_lint_rule | ... | .claude/custom-lint-rules.toml | Medium | High | S | None | ✅ 採用候補 | PR diff; Session; ... |

#### Tier 2: テスト/自動化

Expand All @@ -166,8 +168,11 @@ PR タイトルが context に含まれていない場合は、レポート内

### 次のアクション

- ユーザーがレポートを確認後、Recommendation 列を参考に採用判断を下す (✅ 採用は基本採用、🤔 様子見は dogfood トリガ次第、❌ 却下は不要)
- 採用された提案は `docs/todo.md` 系列に登録するか直接実装へ進む
**重要**: 本 report の Recommendation 列はすべて analyzer の推奨であり、ユーザー明示承認なしに採用・却下を確定してはならない。Claude / 他 AI agent は report を読んだだけで `docs/todo*.md` への entry 追加、実装着手、ADR 編集等を実行してはならず、**必ずユーザー承認 (AskUserQuestion 回答 or テキスト承認のいずれか) を待つこと**。

- `✅ 採用候補`: ユーザー承認後に採用、`docs/todo.md` 系列への登録または直接実装へ進む
- `🤔 様子見`: dogfood トリガ次第で再評価、現時点で action なし
- `❌ 却下推奨`: ユーザー承認後に却下確定、`docs/todo*.md` への登録不要 (Claude 単独で却下処理しない)
- このレポートは `.claude/feedback-reports/<pr_number>.md` に保存される (`.gitignore` 除外、内部 artifact)
```

Expand Down