Skip to content

fix(repo): forbid client info from leaking into PR body #844

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
YusukeHirao merged 2 commits into from
Aug 1, 2026
Merged

fix(repo): forbid client info from leaking into PR body #844

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
ae75364
fix(repo): forbid client info from leaking into PR body
YusukeHirao Jul 31, 2026
5712e5a
Merge branch 'dev' into chore/pr-skill-no-client-info
YusukeHirao Aug 1, 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
4 changes: 2 additions & 2 deletions .claude/skills/pr/SKILL.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -12,8 +12,8 @@ disable-model-invocation: true
- 全てがパスしなければならない。失敗があれば続行前に修正する。
- **`yarn lint` が prettier で意図しない再整形を出した場合は要注意**: コミット済みの整形と食い違う差分は prettier のバージョン差異による drift の可能性がある。自分の変更と無関係な再整形は取り込まず、ユーザーに報告する。
4. 適切な `git` コマンドを使って現在のトピックブランチの変更をレビューする。
- **サンプル慣例のダブルチェック**: base branch との全 diff（`git diff origin/dev...HEAD`）に対して、`.claude/skills/git/SKILL.md` の「サンプル値の慣例チェック」を再実行する。コミット単位のチェックをすり抜けた実在ドメイン・未取得ドメイン・案件識別子・実データ断片がないかを PR 全体で確認する。検出したら汎用値へ書き換えて追加コミットする。
5. **PR body を一時ファイルに保存する**: scratchpad ディレクトリに PR body（markdown）を書き出す。タイトルは英語、本文は日本語で書く。
- **機密・案件情報とサンプル慣例のダブルチェック**: base branch との全 diff（`git diff origin/dev...HEAD`）に対して、`.claude/skills/git/SKILL.md` の「機密・案件情報の検出」と「サンプル値の慣例チェック」を再実行する。コミット単位のチェックをすり抜けた企業名・顧客情報・実在ドメイン・未取得ドメイン・案件識別子・実データ断片がないかを PR 全体で確認する。検出したら汎用値へ書き換えて追加コミットする。
5. **PR body を一時ファイルに保存する**: scratchpad ディレクトリに PR body（markdown）を書き出す。タイトルは英語、本文は日本語で書く。**PR body は diff の要約であり会話の要約ではない** — セッション中に出てきた顧客名・案件名・社外秘情報は、diff 自体に含まれていない限り書き込まない。
6. **push と PR 作成はユーザーが実行する**: エージェントは `git push` / `gh pr create` を実行せず（`.claude/settings.json` で deny されている）、ユーザーがそのまま実行できる `!` 付きコマンドを提示する。ファイルパス引数は必ずダブルクオーテーションで囲む。

```
Expand Down
Loading