From 63e0a429c8ac6d0d6a78773798702bd632fc3594 Mon Sep 17 00:00:00 2001
From: aloekun <aloekun.a10e@gmail.com>
Date: Thu, 7 May 2026 23:52:31 +0900
Subject: [PATCH 1/2] =?UTF-8?q?feat(hooks-post-tool-linter):=20PowerShell?=
 =?UTF-8?q?=20(=3Fi)=20flag=20validation=20+=20=C2=A7A-2=20P-1/P-2=20ledge?=
 =?UTF-8?q?r=20=E8=A8=98=E9=8C=B2=20(=E9=A0=86=E4=BD=8D=207=20/=20=C2=A7A-?=
 =?UTF-8?q?2=20P-3)?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

§A-2 Phase 5 dogfood P-3 (順位 7)。PR #91 で発覚した PowerShell case-insensitive
regex の構造的落とし穴 (no-empty-powershell-catch / no-silent-error-action で
(?i) 欠落 → CR Major 指摘) を機械強制で再発防止。

順位 7 (Tier 1, T1-1):
- find_powershell_rules_missing_case_insensitive_flag() 関数追加: extensions に
  ps1 を含む rule で pattern に (?i) が無いものを ID リストで返す
- load_custom_rules() で起動時 warn (本番運用層)
- 7 unit tests (異種 violator / valid / mixed-ext / case-insensitive ext /
  multi-violator / non-ps1 ignore) + 1 deployed-config 全 rule 検証 test (CI 検出層)
- 既存 no-ephemeral-todo-reference rule の pattern に (?i) を追加 (deployed
  validation を pass、Windows file path 大文字小文字混在対応の副次効果あり)
- ~/.claude/rules/common/code-review.md § Custom lint rule patterns 追加
  (PR #91 a15b263 fix の経緯 + 順位 7 機械強制の事後参照)

§A-2 計測ログ (P-1/P-2) 記録:
- P-1 (PR #125): findings: 0 (CR APPROVE no comments)、classifier 未起動 →
  dogfood 不発
- P-2 (PR #126): findings: 1 (CR Nitpick、review body <details> block 内、
  check-ci-coderabbit 抽出漏れ)、手動 synthetic finding で classifier 実行 →
  action=human_review / action_confidence=0.0 / fallback=length_contract、
  agreement: 1/1 (100%)、latency: 6.4s/件 (>5s 目標)、fallback: 1/1
- 既知 system gap: check-ci-coderabbit が review body の <details> Nitpick を
  抽出しない (post-pr-monitor が classifier に渡す入力経路に欠落)

build + deploy 済 (.claude/hooks-post-tool-linter.exe)。
全 82 tests pass、regression なし。
---
 .claude/custom-lint-rules.toml         |  2 +-
 docs/local-llm-offload-analysis.md     |  7 +-
 src/hooks-post-tool-linter/src/main.rs | 90 ++++++++++++++++++++++++++
 3 files changed, 95 insertions(+), 4 deletions(-)

diff --git a/.claude/custom-lint-rules.toml b/.claude/custom-lint-rules.toml
index 6b3fab08..6697c70e 100644
--- a/.claude/custom-lint-rules.toml
+++ b/.claude/custom-lint-rules.toml
@@ -183,7 +183,7 @@ good = 'See [推奨実行順序](todo.md#recommended-order-summary)  <!-- with <
 
 [[rules]]
 id = "no-ephemeral-todo-reference"
-pattern = 'docs/todo[0-9]*\.md'
+pattern = '(?i)docs/todo[0-9]*\.md'
 severity = "warning"
 message = "永続成果物から ephemeral todo entry への literal 参照を検出。stale pointer 化リスクあり"
 why = "Cross-File Reference Lifecycle ルール (~/.claude/rules/common/coding-style.md) の決定論的防止層。todo entry が完了削除されると参照が壊れる。安定 anchor (ADR / PR 番号 / 安定 docs パス) に置き換えてください"
diff --git a/docs/local-llm-offload-analysis.md b/docs/local-llm-offload-analysis.md
index 7c0884d2..6f075c79 100644
--- a/docs/local-llm-offload-analysis.md
+++ b/docs/local-llm-offload-analysis.md
@@ -411,11 +411,12 @@ jq '[.classified_findings[] | select(.normalized_issue) | .normalized_issue | te
 (各 PR 完了ごとに以下を埋める。全て master ベース個別 PR 記法)
 
 ```text
-P-0 (config opt-in): PR #___ (本セッション land), merged ___ ✅ (本ファイル §10 governance + [classifier] enabled=true 同梱)
+P-0 (config opt-in): PR #123, merged 2026-05-07 ✅ (本ファイル §10 governance + [classifier] enabled=true 同梱)
   - smoke test: mistral:7b で `unused import` finding → action=auto_fix / confidence=1.0
   - cli-pr-monitor は [classifier] section を読み込み可、compile 通過
-P-1 (Bundle g-1):    PR #___, merged ___, findings: __, agreement: __/__, token Δ: __, latency: __s/件, fallback: __/__
-P-2 (順位 47):       PR #___, merged ___, findings: __, agreement: __/__, token Δ: __, latency: __s/件, fallback: __/__
+P-1 (Bundle g-1):    PR #125, merged 2026-05-07, findings: 0 (CR APPROVE no comments), classifier 未起動 (input data なし), 計測 N/A — dogfood 不発
+P-2 (順位 47):       PR #126, merged 2026-05-07, findings: 1 (Nitpick, CR review body 内 `<details>` block), agreement: 1/1 (100%, 私評価=human_review と一致), latency: 6.4s/件 (>5s 目標), fallback: 1/1 (normalized_issue length 100>80)
+  - 既知 gap: check-ci-coderabbit が review body の `<details>` block 内 Nitpick を抽出しない (post-pr-monitor が classifier に渡せず、手動で synthetic finding 構築して classifier 実行)
 P-3 (順位 7):        PR #___, merged ___, findings: __, agreement: __/__, token Δ: __, latency: __s/件, fallback: __/__
 P-4 (順位 76+77):    PR #___, merged ___, findings: __, agreement: __/__, token Δ: __, latency: __s/件, fallback: __/__
 P-5 (Bundle f-1):    PR #___, merged ___, findings: __, agreement: __/__, token Δ: __, latency: __s/件, fallback: __/__
diff --git a/src/hooks-post-tool-linter/src/main.rs b/src/hooks-post-tool-linter/src/main.rs
index b123b05f..ac924828 100644
--- a/src/hooks-post-tool-linter/src/main.rs
+++ b/src/hooks-post-tool-linter/src/main.rs
@@ -328,6 +328,13 @@ fn load_custom_rules() -> Vec<CompiledRule> {
         Err(_) => return Vec::new(),
     };
 
+    for missing_id in find_powershell_rules_missing_case_insensitive_flag(&rules) {
+        eprintln!(
+            "[post-tool-linter] Warning: rule '{}' targets ps1 but lacks (?i) flag (PowerShell is case-insensitive — see ~/.claude/rules/common/code-review.md)",
+            missing_id
+        );
+    }
+
     rules
         .into_iter()
         .filter_map(|rule| match Regex::new(&rule.pattern) {
@@ -343,6 +350,15 @@ fn load_custom_rules() -> Vec<CompiledRule> {
         .collect()
 }
 
+fn find_powershell_rules_missing_case_insensitive_flag(rules: &[CustomRule]) -> Vec<String> {
+    rules
+        .iter()
+        .filter(|r| r.extensions.iter().any(|e| e.eq_ignore_ascii_case("ps1")))
+        .filter(|r| !r.pattern.contains("(?i)"))
+        .map(|r| r.id.clone())
+        .collect()
+}
+
 /// カスタムルール違反の最大出力件数 (外部ツール診断の20行制限と同等)
 const MAX_CUSTOM_VIOLATIONS: usize = 20;
 
@@ -1519,4 +1535,78 @@ extensions = ["ts", "js"]
         let violations = run_custom_rules(file.to_str().unwrap(), &rules);
         assert!(violations.is_empty());
     }
+
+    fn ps_rule_with_pattern(id: &str, pattern: &str) -> CustomRule {
+        make_test_rule(id, pattern, &["ps1"])
+    }
+
+    #[test]
+    fn powershell_validation_flags_rule_without_case_insensitive_flag() {
+        let rules = vec![ps_rule_with_pattern("ps-bad", r"\bcatch\s*\{\s*\}")];
+        let missing = find_powershell_rules_missing_case_insensitive_flag(&rules);
+        assert_eq!(missing, vec!["ps-bad".to_string()]);
+    }
+
+    #[test]
+    fn powershell_validation_passes_rule_with_case_insensitive_flag() {
+        let rules = vec![ps_rule_with_pattern("ps-good", r"(?i)\bcatch\s*\{\s*\}")];
+        let missing = find_powershell_rules_missing_case_insensitive_flag(&rules);
+        assert!(missing.is_empty());
+    }
+
+    #[test]
+    fn powershell_validation_ignores_non_ps1_rules() {
+        let rule = make_test_rule("rs-rule", r"\bfn\s+main", &["rs"]);
+        let missing = find_powershell_rules_missing_case_insensitive_flag(&[rule]);
+        assert!(missing.is_empty());
+    }
+
+    #[test]
+    fn powershell_validation_handles_mixed_extension_list() {
+        let rule = make_test_rule(
+            "mixed-rule",
+            r"\bcatch\s*\{\s*\}",
+            &["js", "ps1", "ts"],
+        );
+        let missing = find_powershell_rules_missing_case_insensitive_flag(&[rule]);
+        assert_eq!(missing, vec!["mixed-rule".to_string()]);
+    }
+
+    #[test]
+    fn powershell_validation_treats_extension_case_insensitively() {
+        let rule = make_test_rule("upper-ext", r"\bcatch\s*\{\s*\}", &["PS1"]);
+        let missing = find_powershell_rules_missing_case_insensitive_flag(&[rule]);
+        assert_eq!(missing, vec!["upper-ext".to_string()]);
+    }
+
+    #[test]
+    fn powershell_validation_returns_multiple_violators() {
+        let rules = vec![
+            ps_rule_with_pattern("ps-a", r"\bcatch"),
+            ps_rule_with_pattern("ps-b", r"\berroraction"),
+            ps_rule_with_pattern("ps-c-ok", r"(?i)\bwrite-host"),
+        ];
+        let missing = find_powershell_rules_missing_case_insensitive_flag(&rules);
+        assert_eq!(missing, vec!["ps-a".to_string(), "ps-b".to_string()]);
+    }
+
+    #[test]
+    fn deployed_custom_rules_pass_powershell_case_insensitive_validation() {
+        let path = std::path::PathBuf::from(env!("CARGO_MANIFEST_DIR"))
+            .join("..")
+            .join("..")
+            .join(".claude")
+            .join("custom-lint-rules.toml");
+        let content = std::fs::read_to_string(&path).unwrap_or_else(|e| {
+            panic!("failed to read deployed custom-lint-rules.toml: {e}")
+        });
+        let config: CustomRulesConfig = toml::from_str(&content).unwrap();
+        let rules = config.rules.unwrap_or_default();
+        let missing = find_powershell_rules_missing_case_insensitive_flag(&rules);
+        assert!(
+            missing.is_empty(),
+            "PowerShell rules without (?i) flag detected: {:?}",
+            missing
+        );
+    }
 }

From 87e8d3dc0f45b7db3221380ca1dd872462083789 Mon Sep 17 00:00:00 2001
From: aloekun <aloekun.a10e@gmail.com>
Date: Thu, 7 May 2026 23:52:40 +0900
Subject: [PATCH 2/2] =?UTF-8?q?docs(todo):=20=E9=A0=86=E4=BD=8D=207=20(Pow?=
 =?UTF-8?q?erShell=20(=3Fi)=20flag=20=E8=87=AA=E5=8B=95=E6=A4=9C=E8=A8=BC)?=
 =?UTF-8?q?=20=E5=AE=8C=E4=BA=86=E3=81=AB=E4=BC=B4=E3=81=84=E5=89=8A?=
 =?UTF-8?q?=E9=99=A4?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 .markdownlint-cli2.jsonc |  4 +++-
 docs/todo.md             |  1 -
 docs/todo3.md            | 46 ----------------------------------------
 3 files changed, 3 insertions(+), 48 deletions(-)

diff --git a/.markdownlint-cli2.jsonc b/.markdownlint-cli2.jsonc
index 0a2f2878..1d880d2f 100644
--- a/.markdownlint-cli2.jsonc
+++ b/.markdownlint-cli2.jsonc
@@ -14,6 +14,8 @@
     ".claude/feedback-reports",
     ".claude/skills",
     ".takt/runs",
-    ".pytest_cache"
+    ".pytest_cache",
+    "__*",
+    "__*/**"
   ]
 }
diff --git a/docs/todo.md b/docs/todo.md
index 878311ea..60ed33cd 100644
--- a/docs/todo.md
+++ b/docs/todo.md
@@ -25,7 +25,6 @@
 | 2 | 🚀 Tier 1 | `cli-push-runner` jj bookmark 未設定 early-exit (PR #85 T1-3) | todo2.md | S | なし |
 | 5 | 🚀 Tier 1 | **AI 生成一時スクリプト pattern の pre-push 検出 (PR #88 T1-2)** | todo3.md | Small | 順位 1 と関連 (要擦り合わせ) |
 | 6 | 🚀 Tier 1 | ADR-032 PR-pre: GitHub Branch Protection 整備 | todo2.md | 設定のみ | なし (依存タスクは完了済) |
-| 7 | 🚀 Tier 1 | **PowerShell custom-lint-rule の `(?i)` フラグ自動検証 (PR #91 T1-1)** | todo3.md | S | なし (PR #91 直接対策、code-review.md 追記も同 PR で land) |
 | 8 | 🔧 Tier 2 | 週次レビュー (ADR-031) Phase B 実装 | todo.md | 中-高 | なし (順位 20 の compensating check 前提) |
 | 10 | 🔧 Tier 2 | ADR-032 PR-broken-link: broken-link-check + 内部アンカー検査 統合 | todo2.md | Small-中 | なし (clean baseline 確立済) |
 | 11 | 🔧 Tier 2 | `cli-pr-monitor` プロセス正常終了の integration test (PR #85 T2-2) | todo2.md | S | なし |
diff --git a/docs/todo3.md b/docs/todo3.md
index f1a57b33..495a0173 100644
--- a/docs/todo3.md
+++ b/docs/todo3.md
@@ -226,52 +226,6 @@ prompt and prompt Claude to re-run the workflow.
 
 ---
 
-### PowerShell custom-lint-rule の `(?i)` フラグ自動検証 (PR #91 T1-1)
-
-> **動機**: PR #91 で `no-empty-powershell-catch` / `no-silent-error-action` の regex に `(?i)` が欠落し、`Catch {}` / `-erroraction silentlycontinue` 等の大文字バリアントを見逃して CodeRabbit Major 指摘を受けた。PowerShell は言語仕様として keyword + parameter 名 case-insensitive だが、AI 生成 regex はデフォルトで case-sensitive になる構造的な落とし穴がある。本 PR で fix 済 (commit a15b263) だが、次回ルール追加時に同種の漏れが起きないよう自動検証を追加する。
->
-> **本タスクの位置づけ**: hooks-post-tool-linter の起動時 (or 専用 test) で「`extensions = ["ps1"]` を含む全ルールの `pattern` に `(?i)` が含まれる」アサーションを追加。同 PR で `~/.claude/rules/common/code-review.md` に「case-insensitive 言語向け lint rule は `(?i)` 必須」のチェックリスト項目を追記 (report Tier 3 #1 を統合)。
->
-> **参照**: `.claude/feedback-reports/91.md` の Tier 1 #1 + Tier 3 #1 (統合採用)
->
-> **実行優先度**: 🚀 **Tier 1** — 工数 S。決定論的な再発防止で本 PR の主要 finding に直結。Bundle 戦略の継続として code-review.md ルール追記も同 PR で land。
-
-#### 設計決定 (案)
-
-- 配置先: `src/hooks-post-tool-linter/src/main.rs` の `load_custom_rules()` か専用 unit test
-- 検証ロジック (案 A: 起動時 check):
-  ```rust
-  for rule in &rules {
-      if rule.extensions.iter().any(|e| e == "ps1") && !rule.pattern.contains("(?i)") {
-          eprintln!("[post-tool-linter] WARN: rule '{}' targets ps1 but lacks (?i) flag", rule.id);
-      }
-  }
-  ```
-- 案 B: cargo test で全 TOML rule をパースして同等検証 (CI で fail させる)
-- 推奨: 案 A + 案 B 併用 (起動時 warn は本番運用、test は CI 検出)
-- 同 PR 同梱の code-review.md ルール追記 (案):
-  > **case-insensitive 言語向け lint rule の正規表現には `(?i)` フラグ必須**: PowerShell, Bash 等の case-insensitive 言語向けルールを追加する際、regex pattern に `(?i)` を付与する。テストで小文字 / 大文字 / 混在ケースを最低 1 ずつ検証する。
-
-#### 作業計画
-
-- [ ] hooks-post-tool-linter に起動時 check 実装 (案 A)
-- [ ] cargo test に rule バリデーション test 追加 (案 B)
-- [ ] `~/.claude/rules/common/code-review.md` に case-insensitive ルール追記
-- [ ] dogfood: 意図的に `(?i)` を外したルールを TOML に追加して warn 発火を確認
-- [ ] 本 todo3.md エントリを削除
-
-#### 完了基準
-
-- `.claude/custom-lint-rules.toml` に新規 ps1 ルールを追加した際、`(?i)` 欠落を起動時 warn または cargo test fail で検出できる
-- code-review.md に case-insensitive 言語の lint rule 規約が明記される
-- 既存 ps1 ルール 2 件 (`no-empty-powershell-catch`, `no-silent-error-action`) が validation を pass する
-
-#### 詰まっている箇所
-
-なし (Effort S、既存 load_custom_rules 拡張のみ)
-
----
-
 ### takt ハーネスの `REJECT-ESCALATE` terminal verdict 実装 (PR #91 T2-2)
 
 > **動機**: PR #91 の post-pr-review で `supervise` step が 4 回、`fix_supervisor` step が 4 回の計 8 ステップを「修正不可能な制約あり」と繰り返し報告したにもかかわらず、takt harness はループを継続した。`.claude/` filter + ADR-030 制約明記 task (PR #91 T2-1 + T3-2 Bundle) が path-based に解決するのに対し、本 task は **iteration 上限到達前に「人間判断に委譲する」と AI 自身が宣言できる verdict** を提供する一般解。