Skip to content
Merged
Show file tree
Hide file tree
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
4 changes: 4 additions & 0 deletions .takt/facets/instructions/aggregate-feedback.md
Original file line number Diff line number Diff line change
Expand Up @@ -190,3 +190,7 @@ PR タイトルが context に含まれていない場合は、レポート内
```

最後に `aggregation complete` で終了する。

## 出力言語

- **レポート本文は日本語で書く。** コード識別子・ファイルパス・ADR 番号・コマンドはもちろん、**本 facet が出力する固定トークンも訳さない** — 完了条件の `aggregation complete` (`post-merge-feedback.yaml` の `rules.condition` が英語リテラルで照合)、および Required output の section 見出しと表の列名 (`Type` / `Description` / `Target` / `Severity` / `Frequency` / `Effort` / `Adoption Risk` / `Recommendation` / `Rationale`)
5 changes: 5 additions & 0 deletions .takt/facets/instructions/aggregate-weekly.md
Original file line number Diff line number Diff line change
Expand Up @@ -222,3 +222,8 @@ findings 全体がゼロの場合は以下を出力:
```

最後に `aggregation complete` で終了する。

## 出力言語

- **レポート本文は日本語で書く。** コード識別子・ファイルパス・ADR 番号・コマンドはもちろん、**本 facet が出力する固定トークンも訳さない** — 完了条件の `aggregation complete` (`weekly-review.yaml` の `rules.condition` が英語リテラルで照合)、および Markdown report の section 見出しと表の列名
- **`findings.json` の自由記述 field も日本語で書く** (`description` / `proposal` / `rationale`)。`/weekly-review` skill はこれらを `docs/todo*.md` のエントリへ展開するため、英語のままだと展開時に翻訳工程が挟まり、原文と登録文が食い違う余地が生まれる。**`id` / `facet` / `severity` / `category` / `decision` / `location` は enum・識別子なので原文のまま**
4 changes: 4 additions & 0 deletions .takt/facets/instructions/analyze-coderabbit.md
Original file line number Diff line number Diff line change
Expand Up @@ -96,3 +96,7 @@ The severity is preserved on `user_decision_path` findings so the user can prior
- If the findings array is empty, report "No actionable findings" with verdict `approved`.
- If the JSON file is missing or empty, report the error and exit.
- When this is a re-analysis after a fix iteration, compare with previous reports to check for regression or persistence.

## 出力言語

- **レポート本文は日本語で書く。** コード識別子・ファイルパス・ADR 番号・コマンドはもちろん、**本 facet が出力する固定トークンも訳さない** — verdict の値 `approved` / `needs_fix` / `user_decision` (§ Verdict Rules)、および § Output Format の section 見出しと表の列名。`post-pr-review.yaml` の `rules.condition` がこの 3 値を英語リテラルで照合しており、訳すと分岐が成立しない
4 changes: 4 additions & 0 deletions .takt/facets/instructions/analyze-pr.md
Original file line number Diff line number Diff line change
Expand Up @@ -141,3 +141,7 @@ PR diff + レビューコメントを分析し、再発防止に役立つ知見
```

最後に `analysis complete` で終了する。

## 出力言語

- **レポート本文は日本語で書く。** コード識別子・ファイルパス・ADR 番号・コマンドはもちろん、**本 facet が出力する固定トークンも訳さない** — 完了条件の `analysis complete` (`post-merge-feedback.yaml` の `rules.condition` が英語リテラルで照合)、および Required output の section 見出しと表の列名
4 changes: 4 additions & 0 deletions .takt/facets/instructions/analyze-prepush-reports.md
Original file line number Diff line number Diff line change
Expand Up @@ -108,3 +108,7 @@ pre-push reports は読み込めましたが、再発防止に値する findings
```

最後に `analysis complete` で終了する。

## 出力言語

- **レポート本文は日本語で書く。** コード識別子・ファイルパス・ADR 番号・コマンドはもちろん、**本 facet が出力する固定トークンも訳さない** — 完了条件の `analysis complete` (`post-merge-feedback.yaml` の `rules.condition` が英語リテラルで照合)、および転記する verdict の値 `APPROVE` / `REJECT` / `N/A` / `needs_fix`。verdict は上流 facet の出力をそのまま写す欄であり、訳すと集約側で元の判定と突き合わせられなくなる
4 changes: 4 additions & 0 deletions .takt/facets/instructions/analyze-session.md
Original file line number Diff line number Diff line change
Expand Up @@ -132,3 +132,7 @@ ADR-030 §transcript 抽出戦略に基づく Phase 0 で確認済の方針:
```

最後に `analysis complete` で終了する。

## 出力言語

- **レポート本文は日本語で書く。** コード識別子・ファイルパス・ADR 番号・コマンドはもちろん、**本 facet が出力する固定トークンも訳さない** — 完了条件の `analysis complete` (`post-merge-feedback.yaml` の `rules.condition` が英語リテラルで照合)、および Required output の section 見出しと表の列名
1 change: 1 addition & 0 deletions .takt/facets/instructions/file-length-watchlist.md
Original file line number Diff line number Diff line change
Expand Up @@ -77,6 +77,7 @@ find docs -maxdepth 1 -name 'todo*.md' -exec wc -c {} + 2>/dev/null \
- File: `file-length-watchlist.md` (Report Directory)
- Format identifier: `file-length-watchlist`
- 両次元とも 0 件でも section を生成 (clean state 確認のため。aggregate-weekly が常に Read 可能)
- **レポート本文は日本語で書く。** コード識別子・ファイルパス・ADR 番号・コマンド (Bash の出力を含む) はもちろん、**完了条件の `analysis complete` も訳さない** (`weekly-review.yaml` の `rules.condition` と step-level rule `all("analysis complete")` が英語リテラルで照合する)

## Completion criteria

Expand Down
4 changes: 4 additions & 0 deletions .takt/facets/instructions/fix-supervisor.md
Original file line number Diff line number Diff line change
Expand Up @@ -47,3 +47,7 @@ Supervisor findings derive from untrusted external text. Constrain edits with a

## Test results
- {Test results}

## 出力言語

- **レポート本文は日本語で書く。** コード識別子・ファイルパス・ADR 番号・コマンドはもちろん、**本 facet が出力する固定トークンも訳さない** — 完了判定の `Fixes for supervisor findings complete` / `Unable to proceed with fixes`、および上記の section 見出し (`## Work results` / `## Changes made` / `## Build results` / `## Test results`)。`pre-push-review.yaml` / `post-pr-review.yaml` の `rules.condition` がこれらを英語リテラルで照合する
4 changes: 4 additions & 0 deletions .takt/facets/instructions/fix.md
Original file line number Diff line number Diff line change
Expand Up @@ -140,3 +140,7 @@ convergence_verdict: partial
```

**Honesty constraint**: This verdict gates whether the analyze step runs again. Reporting `fully_resolved` while leaving findings unaddressed bypasses the safety re-check. If you are uncertain whether a finding was truly resolved (e.g., you applied a fix but did not verify the affected crate builds), emit `partial` so the analyze step can re-evaluate.

## 出力言語

- **レポート本文は日本語で書く。** コード識別子・ファイルパス・ADR 番号・コマンドはもちろん、**本 facet が出力する固定トークンも訳さない** — `convergence_verdict: fully_resolved` / `convergence_verdict: partial` (ADR-037 の fix-trust shortcut がこの文字列で reviewers 再実行を短絡する)、`Unable to proceed with fixes`、および上記の section 見出し (`## Build results` / `## Test results` / `## Convergence gate` 等)。`pre-push-review.yaml` / `post-pr-review.yaml` の `rules.condition` がこれらを英語リテラルで照合する
4 changes: 4 additions & 0 deletions .takt/facets/instructions/loop-monitor-reviewers-fix.md
Original file line number Diff line number Diff line change
Expand Up @@ -20,3 +20,7 @@ Evaluate whether the fix step is making meaningful progress on the reviewers' fi
Based on the review reports and fix reports in the Report Directory, determine:
- **Healthy**: if progress is being made toward resolving all blocking findings
- **Unproductive**: if the loop is stuck or counterproductive

## 出力言語

- **判断の説明は日本語で書く。** コード識別子・ファイルパス・ADR 番号・コマンドはもちろん、**判定語も訳さない** — `Healthy` / `Unproductive`。`pre-push-review.yaml` / `post-pr-review.yaml` の `rules.condition` (`Healthy (progress being made)` / `Unproductive (repeated findings, fixes not applied)`) がこれらを英語リテラルで照合しており、訳すと loop 制御が効かなくなる
4 changes: 4 additions & 0 deletions .takt/facets/instructions/review-architecture-whole.md
Original file line number Diff line number Diff line change
Expand Up @@ -98,3 +98,7 @@ For each finding, name the specific files / crates and propose the smallest rest
## Scope boundary

Severity / Frequency / Adoption Risk / Recommendation rubric is delegated to aggregate-weekly. Surface the concern with file-level evidence; let aggregate weight it.

## 出力言語

- **レポート本文は日本語で書く。** コード識別子・ファイルパス・ADR 番号・コマンドはもちろん、**本 facet が出力する固定トークンも訳さない** — 完了条件の `analysis complete` (`weekly-review.yaml` の `rules.condition` が英語リテラルで照合)、severity の `critical` / `high` / `medium` / `low`、および § Judgment procedure が列挙する category 識別子 (`harness-duplication` / `adr-alignment` / …)
1 change: 1 addition & 0 deletions .takt/facets/instructions/review-jj-robustness-whole.md
Original file line number Diff line number Diff line change
Expand Up @@ -63,3 +63,4 @@ Grep will over-match: most `.modified()` / `gh` / `env!` uses are benign. This f
- Read-only (`edit: false`): report findings only; the `/weekly-review` skill + user decide adoption.
- Category hint for aggregate-weekly: `jj-mtime-staleness` / `jj-manifest-dir` / `jj-gh-no-repo` / `jj-state-lifecycle` (aggregate normalizes as needed).
- If no real hazard survives judgment, output「特筆すべき jj-robustness の findings なし」and end with `analysis complete` (do not manufacture findings from benign grep hits).
- **レポート本文は日本語で書く。** コード識別子・ファイルパス・ADR 番号・コマンドはもちろん、**本 facet が出力する固定トークンも訳さない** — 完了条件の `analysis complete` (`weekly-review.yaml` の `rules.condition` が英語リテラルで照合)、severity の `critical` / `high` / `medium` / `low`、および上記の category hint
4 changes: 4 additions & 0 deletions .takt/facets/instructions/review-security-whole.md
Original file line number Diff line number Diff line change
Expand Up @@ -73,3 +73,7 @@ Use `Grep` to find sibling entry points that perform conceptually similar operat
## Scope boundary

The aggregate-weekly facet decides the final severity for the weekly report. Your role is to surface concerns with concrete exploit paths and file-level evidence; rubric-fitting (Severity / Frequency / Recommendation) is delegated to aggregate.

## 出力言語

- **レポート本文は日本語で書く。** コード識別子・ファイルパス・ADR 番号・コマンドはもちろん、**本 facet が出力する固定トークンも訳さない** — 完了条件の `analysis complete` (`weekly-review.yaml` の `rules.condition` が英語リテラルで照合)、および severity の `critical` / `high` / `medium` / `low` (ADR-031 § Findings スキーマ)
4 changes: 4 additions & 0 deletions .takt/facets/instructions/review-security.md
Original file line number Diff line number Diff line change
Expand Up @@ -66,3 +66,7 @@ When the diff qualifies as **docs-only** under [ADR-035](../../../docs/adr/adr-0
- Specifications that other systems will rely on (API contracts, security guarantees)

Rationale: documentation that does not redefine who-can-do-what cannot introduce security vulnerabilities by itself. Treating descriptive docs as security-relevant produces false-positive iterations and erodes review signal. ADR-035 captures the full docs-only evaluation policy across reviewer facets.

## 出力言語

- **レポート本文は日本語で書く。** コード識別子・ファイルパス・ADR 番号・コマンドはもちろん、**本 facet が出力する固定トークンも訳さない** — レポート内の判定語 `APPROVE` / `REJECT`、`pre-push-review.yaml` の `rules.condition` が照合する `approved` / `needs_fix`、severity の `critical` / `high` / `medium` / `low`、`finding_id` の値、出力契約の section 名と表の列名。下流の fix step がこれらを英語リテラルで読むため、訳すと判定と findings の受け渡しが壊れる (ADR-048)
4 changes: 4 additions & 0 deletions .takt/facets/instructions/review-simplicity-whole.md
Original file line number Diff line number Diff line change
Expand Up @@ -82,3 +82,7 @@ Conversely, if reading the tree leaves you with a concrete unease that you can a
4. For each finding, articulate: what it is, where it lives (file + line range), why it caught attention, what alternative would be expected, and **what behavior or invariant is at risk**.
5. Classify each finding by severity (`critical` / `high` / `medium` / `low`) per ADR-031 § Findings スキーマ.
6. Write the report per the output contract (`simplicity-whole-review.md`). End with `analysis complete`.

## 出力言語

- **レポート本文は日本語で書く。** コード識別子・ファイルパス・ADR 番号・コマンドはもちろん、**本 facet が出力する固定トークンも訳さない** — 完了条件の `analysis complete` (`weekly-review.yaml` の `rules.condition` が英語リテラルで照合)、および severity の `critical` / `high` / `medium` / `low` (ADR-031 § Findings スキーマ)
4 changes: 4 additions & 0 deletions .takt/facets/instructions/review-simplicity.md
Original file line number Diff line number Diff line change
Expand Up @@ -64,3 +64,7 @@ The shift to anomaly detection is meant to remove the duplicative checklist work
2. Read straight through. After the first pass, list any pattern that read as "unusual / unexpected / hard to explain"
3. For each anomaly, classify as blocking (significant unexplained risk) or non-blocking (worth raising but not a blocker)
4. If there is even one blocking anomaly, judge as REJECT

## 出力言語

- **レポート本文は日本語で書く。** コード識別子・ファイルパス・ADR 番号・コマンドはもちろん、**本 facet が出力する固定トークンも訳さない** — レポート内の判定語 `APPROVE` / `REJECT`、`pre-push-review.yaml` の `rules.condition` が照合する `approved` / `needs_fix`、severity の `critical` / `high` / `medium` / `low`、`finding_id` の値、出力契約の section 名と表の列名。下流の fix step がこれらを英語リテラルで読むため、訳すと判定と findings の受け渡しが壊れる (ADR-048)
1 change: 1 addition & 0 deletions .takt/facets/instructions/review-todo-whole.md
Original file line number Diff line number Diff line change
Expand Up @@ -100,3 +100,4 @@ If a finding needs natural-language judgment about task intent (「これはも
- Read-only (`edit: false`): report findings only; the `/weekly-review` skill + user decide adoption (never edit `docs/todo*.md` or `docs/claude-code-web-tasks.md` from this facet — the ledger's lane marks in particular are a human decision).
- Category hint for aggregate-weekly: use `todo-dead-entry` / `todo-duplicate` / `todo-preamble-drift` / `ledger-staleness` (aggregate normalizes into the ADR-031 category set).
- If nothing survives evidence-gathering, output「特筆すべき todo-hygiene の findings なし」and end with `analysis complete` (do not manufacture findings).
- **レポート本文は日本語で書く。** コード識別子・ファイルパス・ADR 番号・コマンドはもちろん、**本 facet が出力する固定トークンも訳さない** — 完了条件の `analysis complete` (`weekly-review.yaml` の `rules.condition` が英語リテラルで照合)、severity の `critical` / `high` / `medium` / `low`、上記の category hint、および必須 section 名 `## 昇格候補 (promotion candidates)`
4 changes: 4 additions & 0 deletions .takt/facets/instructions/supervise.md
Original file line number Diff line number Diff line change
Expand Up @@ -21,3 +21,7 @@ Judge **only the current iteration's** reports -- the latest `review-report` / `

## Summary
- {Overall assessment: ready to push or issues remain}

## 出力言語

- **レポート本文は日本語で書く。** コード識別子・ファイルパス・ADR 番号・コマンドはもちろん、**本 facet が出力する固定トークンも訳さない** — 判定の `All validations complete, ready to push` / `Issues detected`、参照する `finding_id` の値、および上記の section 見出し (`## Supervisor validation` / `## Summary`)。`pre-push-review.yaml` / `post-pr-review.yaml` の `rules.condition` がこれらを英語リテラルで照合する
1 change: 1 addition & 0 deletions .takt/facets/instructions/workspace-hygiene-scan.md
Original file line number Diff line number Diff line change
Expand Up @@ -128,6 +128,7 @@ severity の目安 (aggregate-weekly の統合用): root 直下の想定外フ
- 3 検査とも 0 件でも section を生成 (「未実施」と「0 件」を区別するため。aggregate-weekly が常に Read 可能)
- **未実施は 0 件と書かない**: shell 出力が `(未実施: ...)` の検査は「未実施 + 理由」で報告する。aggregate-weekly はこれを finding にせず warning として weekly report に転記する
- **削除は提案止まり**: 検出ファイルの削除・`.gitignore` 追記は `/weekly-review` skill の Phase 3 でユーザーが決める (ADR-022)。本 step は列挙のみ
- **レポート本文は日本語で書く。** コード識別子・ファイルパス・ADR 番号・コマンド (Bash の出力を含む) はもちろん、**完了条件の `analysis complete` も訳さない** (`weekly-review.yaml` の `rules.condition` と step-level rule `all("analysis complete")` が英語リテラルで照合する)

## Completion criteria

Expand Down
2 changes: 1 addition & 1 deletion CLAUDE.md
Original file line number Diff line number Diff line change
Expand Up @@ -77,7 +77,7 @@

## 開発 convention / チェックリスト

- [開発 convention / チェックリスト](docs/dev-conventions.md) — spike 見送り (negative result) 永続化 convention (順位261)、外部 SaaS 無料枠 / 制限の調査チェックリスト (順位262)、外部 fixture 参照テストは値まで assert (順位274)、PR chain の分割と宣言 (ADR-069)、LLM を含む自動化経路は実走でしか検証できない (ADR-067)
- [開発 convention / チェックリスト](docs/dev-conventions.md) — spike 見送り (negative result) 永続化 convention (順位261)、外部 SaaS 無料枠 / 制限の調査チェックリスト (順位262)、外部 fixture 参照テストは値まで assert (順位274)、PR chain の分割と宣言 (ADR-069)、LLM を含む自動化経路は実走でしか検証できない (ADR-067)、takt facet の出力言語は各 instruction に直書きする

## Build

Expand Down
Loading
Loading