Skip to content

feat(weekly-review): 分析フェーズを cloud routine へ移行し reminder を監査用途へ転換 (WP-17 PR 4) #354

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
aloekun merged 3 commits into from
Aug 4, 2026
Merged

feat(weekly-review): 分析フェーズを cloud routine へ移行し reminder を監査用途へ転換 (WP-17 PR 4) #354

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
f19d831
refactor(hooks-session-start): weekly-review reminder を監査リマインダーへ転換 (W…
aloekun Aug 3, 2026
b3b0523
docs(adr): ADR-070 (weekly-review の cloud routine 移行) 起票 + ADR-031 整合…
aloekun Aug 3, 2026
9f89806
docs(adr): routine の push が jj-push-guard に阻まれる実測を ADR-070 へ追記 (WP-17…
aloekun Aug 4, 2026
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
16 changes: 10 additions & 6 deletions .claude/hooks-config.toml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -29,12 +29,16 @@ default_branch = "master" # trunk-based 前提、feature branch 運用では
stale_check_enabled = true

# [session_start.weekly_review_reminder]
# - ADR-031 Phase C: `/weekly-review` skill 起動の reminder (試験運用、ADR-039 experimental pattern 準拠)。
# - ADR-031 Phase C / ADR-070: weekly-review の**監査リマインダー** (試験運用、ADR-039 準拠)。
# ADR-070 で分析の主経路が cloud routine (週 1 schedule) へ移り、本 reminder は
# 「レビューを実行せよ」から「routine の稼働と結果の取り込みを確認せよ」へ転換した。
# 2 経路で発火:
# 1. `.claude/weekly-review-last-run.json` の mtime が `reminder_threshold_days` 超過
# 2. `.claude/weekly-reviews/*.md.failed` marker 1 件以上残存 (前回失敗 resume promote)
# 1. `.claude/weekly-review-last-run.json` の `last_run_at` が `reminder_threshold_days` 超過
# 2. `.claude/weekly-reviews/*.md.failed` marker 1 件以上残存 (前回ローカル実行の失敗 resume)
# 両方該当する場合は 1 nudge にまとめて出力。
# 3-5 週の dogfood 後に default-ON 昇格 or 却下を判定 (bounded lifetime)。
# **重要**: last_run_at は skill の**ローカル実行時**にのみ更新される。cloud routine は
# 使い捨てクローンで動くため更新しない = 本 reminder は routine の実行を観測できない。
# 発火は「routine が止まっている」の証拠ではなく定期監査の促し。
# Kill-switch: `enabled = false` で完全停止。
[session_start.weekly_review_reminder]
# 試験運用元 (本リポジトリ) では明示的に enable して reminder を実発火させる運用。
Expand All @@ -43,8 +47,8 @@ stale_check_enabled = true
# 派生プロジェクト deploy 時は default OFF (ADR-039 § 1 opt-in 契約) を維持。
# 次 PR (PR-3) で `[features].enabled` allow-list 方式に移行予定 = 本 `enabled = true` は暫定。
enabled = true
reminder_threshold_days = 7 # ADR-031 § トリガー方式: 「前回実行から 7 日経過で promote」と整合
failed_marker_check_enabled = true # 前回失敗 marker 検出 → resume promote。false で staleness のみに限定可
reminder_threshold_days = 30 # ADR-070: 監査サイクル。週次 (7 日) だと routine 正常時も毎週発火しノイズになる
failed_marker_check_enabled = true # 前回ローカル実行の失敗 marker 検出 → resume promote。false で staleness のみに限定可
# ADR-059 (試験運用、判定期限 2026-08-16): reminder 発火時に systemMessage (ユーザー可視 1 行) を
# additionalContext と併せて出し、「発火しているのにユーザーに見えない」silent 化を解消する。
# source default OFF (派生 repo deploy 時は本行を置かない = additionalContext のみの従来挙動)。
Expand Down
1 change: 1 addition & 0 deletions CLAUDE.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -70,6 +70,7 @@
- [ADR-067: Phase B 無人 fix push — agent を push の主体にしない 4 軸ゲート](docs/adr/adr-067-phase-b-unattended-fix-push.md) *(試験運用)*
- [ADR-068: pre-push fix step の権限境界 — 後退検知 backstop と設計級 remedy の human routing](docs/adr/adr-068-fix-step-authority-boundary.md) *(試験運用)*
- [ADR-069: PR chain 宣言規約 — 分割チェーンと missing-consumer 検査の両立](docs/adr/adr-069-pr-chain-declaration.md) *(試験運用)*
- [ADR-070: weekly-review の分析フェーズを cloud routine へ移行 — 常時性の獲得と成果物デリバリの未解決](docs/adr/adr-070-weekly-review-cloud-routine.md) *(試験運用)*

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

Expand Down
2 changes: 2 additions & 0 deletions docs/adr/adr-031-weekly-review-pipeline.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,8 @@

承認済み (2026-06-01、試験運用 2026-04-27 → 本採用に昇格)

> **2026-08-04 更新 (WP-17 PR 4)**: 起動トリガーを [ADR-070](adr-070-weekly-review-cloud-routine.md) が変更した。分析フェーズ (Phase 1-2 = takt workflow 実行) は cloud routine (週 1 schedule) が担い、SessionStart reminder は「レビューを実行せよ」から「**routine の稼働と結果の取り込みを確認せよ**」の監査リマインダー (既定 30 日) へ転換。**Phase 3 (採否判断) / Phase 4 (task list 反映 + last-run 更新) は従来どおりローカルの人間作業**で、routine は置き換えではない。本 ADR のパイプライン設計そのものは変更なし。

@coderabbitai coderabbitai Bot Aug 4, 2026

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

📐 Maintainability & Code Quality | 🟡 Minor | ⚡ Quick win

接続表現を であり に修正してください。

ローカルの人間作業**で、routine は置き換えではない。 は接続が不自然です。ローカルの人間作業であり、routine は置き換えではない。 のように修正してください。

🧰 Tools
🪛 LanguageTool

[uncategorized] ~7-~7: 文法ミスがあります。"のでは"の間違いです。
Context: ...k list 反映 + last-run 更新) は従来どおりローカルの人間作業**で、routine は置き換えではない。本 ADR のパイプライン設計そのものは変更なし...

(DOUSI_DEHA)

🤖 Prompt for AI Agents 
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@docs/adr/adr-031-weekly-review-pipeline.md` at line 7, Update the wording in
the ADR-031 update so the phrase after “ローカルの人間作業” uses “であり、” instead of “で、”,
preserving the surrounding meaning and formatting.

Source: Linters/SAST tools

>
Comment on lines +7 to +8

@coderabbitai coderabbitai Bot Aug 4, 2026

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

📐 Maintainability & Code Quality | 🟠 Major | ⚡ Quick win

ADR-070 の境界を既存セクションにも反映してください。

Line 7–8 は、Phase 1–2 を週次 cloud routine、SessionStart を 30 日の監査通知へ変更します。しかし同じ docs/adr/adr-031-weekly-review-pipeline.md の Line 130–163 と Line 264–270 には、7 日 stale 判定、/weekly-review の手動実行、skill が Phase 1–4 を実行する旧仕様が残っています。

.claude/hooks-config.toml は既に reminder_threshold_days = 30 と、cloud routine が last_run_at を更新しない契約を定義しています。このままでは ADR-031 内の運用手順が矛盾します。L1/L2、全体フロー、トリガー節を ADR-070 の Phase 1–2 / Phase 3–4 境界と監査 reminder の責務に合わせて同期してください。

Based on learnings: 適用対象の ADR の決定を現行文書へ反映し、superseded ADR を現行要件として扱わない方針に従ってください。

🧰 Tools
🪛 LanguageTool

[uncategorized] ~7-~7: 文法ミスがあります。"のでは"の間違いです。
Context: ...k list 反映 + last-run 更新) は従来どおりローカルの人間作業**で、routine は置き換えではない。本 ADR のパイプライン設計そのものは変更なし...

(DOUSI_DEHA)

🤖 Prompt for AI Agents 
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@docs/adr/adr-031-weekly-review-pipeline.md` around lines 7 - 8, Update the
existing L1/L2, overall flow, and trigger sections in ADR-031 to align with
ADR-070: assign Phase 1–2 to the weekly cloud routine, retain Phase 3–4 as local
human work, and describe SessionStart as a 30-day audit reminder verifying
routine execution and result intake. Remove or revise stale 7-day detection,
manual /weekly-review execution, and skill-based Phase 1–4 claims, while
preserving the contract that the routine does not update last_run_at.

Source: Learnings

> 本 ADR の運用パターンは [ADR-039 (試験運用標準パターン)](adr-039-experimental-feature-standard-pattern.md) で標準化された 3 点セット (config opt-in / kill-switch / bounded lifetime) の対象。本採用判定で ADR-039 の retirement workflow に従い、Phase C/D/E 用 ephemeral handoff doc を retire 済 (Phase E land と同 PR、git log で履歴 trackable)。

### 採用判定の根拠 (Phase E dogfood 観測結果)
Expand Down
Loading
Loading