refactor: runtime scripts の重複集約 + dead code 除去 (PR-1/4, Refs #189)

[kosako]

Refs #189(全体リファクタリング・挙動不変)の PR-1/4。ultracode 監査(5 レンズ Find → 敵対 Verify、REFUTED 0)の CONFIRMED 所見のうち runtime scripts 系 5 件を実施する。

変更内容(すべて挙動不変)

1. file_mode() を lib-policy.sh へ集約(3 重複解消)
   • private-backup.sh の関数定義 / preflight.sh のインライン再実装 / test-private-backup.sh の mode_of() を単一定義に。preflight は 2>/dev/null || echo unknown を call site に残すので出力 byte 不変。
2. doctor / preflight の byte-identical 複製 2 箇所を lib-policy.sh へ
   • policy ゲート → run_policy_validation(section "policy" と || exit 1 は call site に残す = report-only 契約の唯一の非 zero path を関数に吸わせない)
   • standard project roots ループ → report_standard_project_roots(directory convention(~/src/agent)と実 checkout(~/dev)の乖離を解消する #134 の「missing root は neutral」の正本が 1 箇所に)
3. validate-policy.sh の sort | uniq -d 重複検出ループ ×5 を fail_duplicates に集約
   • メッセージ prefix は逐語維持("duplicate capability in $profile: " 等)。helper || status=1 形で status の単調更新も同値。
4. manager_present() を install-packages.sh → lib-policy.sh へ移動し、report_catalog_drift の have_* 判定と一本化
   • source→manager 対応表の単一 source 化。probe(command -v)の実行コマンド・回数・順序は不変。移動と置換は同一 commit で原子的に実施(test-policy の drift harness は lib 単独 source のため)。
5. dead 関数 catalog_source_preference() を削除
   • git grep を全リビジョンに対して実行し、導入 commit 以来呼び出しゼロを確認済み。repo 現在形でも参照は定義行のみ。packages.yaml の source_preference キーは「advisory only(人間向け)」のコメント付き意図的データなので残置。

挙動不変の検証(実施済み)

• bash -n / shellcheck -S warning: green(validate-policy.sh の SC2034 1 件は main でも出る既存件 = ローカル shellcheck 0.11.0 と CI の版差。本 PR では増減なし)
• 全テスト suite green(test-lib.sh 除く 16 本)
• render byte-identical: personal / work 両 profile の throwaway apply 出力を変更前 baseline と diff -r → 一致(scripts/ は .chezmoiignore 対象なので構造的にも不変)
• 旧新実出力比較: main worktree と本 branch で doctor.sh / preflight.sh × personal / work を実行し stdout+stderr を diff → 環境由来の差分(worktree path 表示・npm probe の実行時刻)以外は byte-identical。exit code も全一致(rc=0)

レビュー観点(お願いしたいこと)

• 集約後の関数が呼び出し文脈(set -e / || status=1 / condition 文脈)で旧コードと等価か
• fail_duplicates の prefix 逐語性(5 箇所)
• manager_present 移動の原子性(lib 側定義 + install-packages 側削除 + drift 置換が同 commit)

🤖 Generated with Claude Code

https://claude.ai/code/session_016qw8BkUDu5WdwhYfiFFX5n

[kosako]

Codex 相互レビュー結果(author=Claude → reviewer=Codex)

codex exec(gpt-5.6-sol, read-only, reasoning=medium)によるレビュー。

判定: ✅ merge 可(must 0 件) — should 0 / nit 0

重点観点の検証結果(Codex 出力より):

• policy ゲート / project-roots 集約: 問題なし。出力文言・順序・stream、report-only 契約を維持
• set -euo pipefail と || exit/status=1 文脈: 問題なし
• fail_duplicates 5 箇所: prefix・全重複行の報告・status 更新とも旧実装と同値
• manager_present 移動: installer と drift harness の参照先が正しく一本化
• mode_of → file_mode: production 実装と同じ OS 分岐をテストが使用
• catalog_source_preference 削除: 参照漏れなし。YAML の advisory data は維持
• ShellCheck: 既知の SC2034 のみ(スコープ外・増減なし)

補足: Codex 環境は read-only sandbox のため実テスト再実行は不可(bash -n と diff/参照解析は実施)。author 側で全テスト green・render byte-identical・旧新実出力 diff 一致を検証済み。

[kosako]

Codex 増分レビュー(SC2034 修正 commit)

CI runner の shellcheck 更新で main 由来の既存 SC2034 が validate job を fail させるようになったため、category → _category(intentionally-unused 慣用形)で解消する commit を追加した。

判定: merge 可 — 「変数名変更のみ。IFS='|' の 3 フィールド分割位置と $type / $path への割り当て・後続参照に影響なし」(Codex)

CI: validate / render とも green。