Skip to content

chore(rules): PR・レビュー方針の決定をプロジェクト側へ委ねる - #79

Merged
gotomts merged 1 commit into
mainfrom
chore/project-scoped-pr-policy
Aug 31, 2026
Merged

chore(rules): PR・レビュー方針の決定をプロジェクト側へ委ねる#79
gotomts merged 1 commit into
mainfrom
chore/project-scoped-pr-policy

Conversation

@gotomts

@gotomts gotomts commented Aug 31, 2026

Copy link
Copy Markdown
Owner

変更内容

グローバル規範 (SSOT: claude/rules/)

全リポジトリへ一律に適用していた PR / レビューの規則を外し、対象プロジェクトの規約が明示する条件だけを適用する形にした。

  • core.md「不可逆な操作」から、merge の実行禁止・実行者を規定する行を削除
  • orchestrator.md
    • 役割節から merge の責務規定を削除(PR 前ゲートへの参照は維持)
    • ユーザー判断へ回す条件を「プロジェクト規約が定めていない操作の実行可否が問われた」に変更
    • 「PR 作成前のレビューゲート」を、全リポジトリ共通の必須 2 段ゲートから、対象プロジェクトの規約が明示する条件だけを適用する手順へ変更。規約が定めていない場合は必須化・省略のどちらも推測せずユーザーへ確認する。Hermes 自身のセルフレビュー・指摘の決着・再検証・完了報告への記録は維持
  • 生成物 claude/AGENTS.md / claude/hermes/SOUL.mdscripts/build-agent-rules.zsh で再生成(生成スクリプトは未変更)

このリポジトリの規約 (AGENTS.md)

  • # main への直接 push を新設。専用 worktree で「1 commit = 完結した変更 + その場で実行した直接検証」に収まり、既存のレール内・必須検証成功・未解決の人間判断なしの通常変更に限り、PR を介さず main へ直接 push してよいことと、その対象外(秘密情報・権限・課金・本番設定・migration・外部契約 / 設計・受入条件の未確定または変更 / 手本のない新規アーキテクチャ / 複数コミットを要する変更 / stack した作業 / 人間判断が残る変更)を明示
  • 「必須の検証」の実体を変更種別ごとに明記(rules と生成物は --check、setup / herdr / hooks / settings は既存 bats・unit test、nix は既存のローカルビルド確認)。新しいテスト基盤・CI は追加していない
  • 公開リポジトリ参照ポリシーに触れる変更は、非公開リポジトリ・非公開 issue 参照を含まないことを push 前に確認する条件を追加
  • agent-rules-check / nix-check の説明を、実際のトリガー(PR と main への直接 push の両方)に合わせて修正

動作確認

$ zsh scripts/build-agent-rules.zsh --check; echo "exit=$?"
exit=0

--check は drift が無い場合、標準出力に何も出さず exit 0 で終わる)

  • zsh scripts/build-agent-rules.zsh → exit 0(生成物を再生成)、--check → exit 0(drift なし)
  • AGENTS.md は末尾空白なし・末尾改行あり(.editorconfig 準拠)、追記で参照したテストパスの実在を確認
  • git status --short → 空

人間判断が必要な事項

  • merge は実行していない。merge の実施可否は人間の判断に委ねる

グローバル規範から一律の禁止・必須ゲートを外し、対象プロジェクトの規約が
明示する条件だけを適用する形に変える。規約が定めていない操作は推測で実行せず
ユーザーへ確認する。

あわせて、このリポジトリの AGENTS.md に main への直接 push の運用を定義し、
専用 worktree で 1 commit に収まる通常変更に限って PR を介さず push してよい
範囲と、その対象外を明示する。

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@gotomts
gotomts merged commit 8a73287 into main Aug 31, 2026
1 check passed
@gotomts
gotomts deleted the chore/project-scoped-pr-policy branch August 31, 2026 16:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant