Skip to content

docs: ドキュメント最新化(ADR 0007追加・hooks README更新) - #801

Merged
keito4 merged 1 commit into
mainfrom
claude/issue-799-20260605-0301
Jun 5, 2026
Merged

docs: ドキュメント最新化(ADR 0007追加・hooks README更新)#801
keito4 merged 1 commit into
mainfrom
claude/issue-799-20260605-0301

Conversation

@keito4

@keito4 keito4 commented Jun 5, 2026

Copy link
Copy Markdown
Owner

Closes #799

変更内容

  • docs/adr/README.md: ADR 0007 をテーブルに追加
  • README.md: block_inline_secrets.py をディレクトリ構造に追加
  • .claude/hooks/README.md: 5本の未ドキュメントhookセクションを追加

Generated with Claude Code

- docs/adr/README.md: ADR 0007(Claude PR作成分離)をテーブルに追加
- README.md: hooks ディレクトリ構造に block_inline_secrets.py を追加
- .claude/hooks/README.md: 未ドキュメントのhook 5本を追加
  - block_config_edit.py(リンター設定保護)
  - block_dangerous_commands.py(破壊的コマンドブロック)
  - block_inline_secrets.py(GitLab/Doppler/Google含むインライン秘密情報検出)
  - post_edit_auto_lint.py(編集後自動リント)
  - stop_test_verification.py(完了前テスト検証)

Closes #799

Co-authored-by: keito4 <keito4@users.noreply.github.com>
@coderabbitai

coderabbitai Bot commented Jun 5, 2026

Copy link
Copy Markdown

Warning

Review limit reached

@keito4, we couldn't start this review because you've reached your PR review rate limit.

More reviews will be available in 42 minutes and 50 seconds. Learn how PR review limits work.

Your organization has run out of usage credits. Purchase more in the billing tab.

⌛ How to resolve this issue?

After more reviews become available, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans include higher PR review limits than trial, open-source, and free plans. In all cases, reviews become available again over time. During sustained high-volume PR review activity, CodeRabbit may temporarily slow when the next review becomes available.

Please see our Fair Usage Limits Policy for further information.

ℹ️ Review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 218d3263-003d-40ed-9ec4-a369bbec971d

📥 Commits

Reviewing files that changed from the base of the PR and between d412d5e and 2d05ede.

📒 Files selected for processing (3)
  • .claude/hooks/README.md
  • README.md
  • docs/adr/README.md
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch claude/issue-799-20260605-0301

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@claude

claude Bot commented Jun 5, 2026

Copy link
Copy Markdown
Contributor

Claude finished @keito4's task in 2m 19s —— View job


コードレビュー完了

  • コンテキスト収集
  • 変更ファイルの読み込み
  • レビュー実施
  • コメント更新

概要

ドキュメント専用のPRです。未文書化だったhook 5本(3〜7番)を追加し、block_config_edit.pyblock_dangerous_commands.pyblock_inline_secrets.pypost_edit_auto_lint.pystop_test_verification.py の新規セクションも追加されています。ADR 0007の追加・README.mdのディレクトリ構造更新はいずれも正確です。


指摘事項

minor: post_commit_adr_reminder.py のアーキテクチャシグナル表が不完全

.claude/hooks/README.md セクション7の検出シグナル表から以下の3行が削除されていますが、実際の Python コードには依然として存在します。

ドキュメントに記載なし:

シグナル 対象ファイル コード行
モジュールエントリポイント src/**/index.*, src/**/main.* L83
IaC terraform/ L86
DBマイグレーション supabase/migrations/ L88

コードと一致させるか、意図的に削除する場合は実際の Python コードからも削除する必要があります。Fix this →

minor: pre_git_quality_gates.py セクションから追加スクリプトの記述が削除されている

旧バージョンに存在し削除された記述ですが、実際のコード(L309, L315)では依然として実行されます:

さらに以下のスクリプトが存在する場合も実行:
- `script/security-credential-scan.sh --strict`
- `script/code-complexity-check.sh --strict`

また、package.jsonscripts を解析するという説明文も削除されていますが、フック動作の理解に有用な情報です。

minor: block_inline_secrets.py のMarkdownテーブルが破損

正規表現パターン内の | 文字がMarkdownの列区切りとして解釈され、テーブルが崩れています。

崩れているヘッダ行:

| ---------------------------------------- | ---------------------------- | ------------------- | --- | ---- | ----------- | ---------------- |

影響を受けているパターン例:

  • (AKIA|ASIA)[0-9A-Z]{16}| が列区切りに
  • [sr]k\_(live|test)\_... → 同上
  • dp.(pt|st|sa|ct|scim|audit).... → 同上

修正方法: パイプ文字を \| にエスケープするか、表形式ではなくリスト形式に変更する。Fix this →


問題なし

観点 判定 備考
Security ドキュメントのみ、問題なし
Performance 対象外
Quality ⚠️ 上記3件のminor指摘あり
Accessibility 対象外
AI Residuals ハードコード値等なし

verdict: APPROVE — 3件はいずれも minor で、ブロッキング理由にはなりません。ドキュメントの正確性向上のため次のコミットで修正することを推奨します。

@keito4 keito4 self-assigned this Jun 5, 2026
@keito4
keito4 merged commit f36e1c9 into main Jun 5, 2026
6 checks passed
@keito4
keito4 deleted the claude/issue-799-20260605-0301 branch June 5, 2026 04:40
@github-actions

github-actions Bot commented Jun 5, 2026

Copy link
Copy Markdown
Contributor

🎉 This PR is included in version 1.115.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

released リリース済み

Projects

None yet

Development

Successfully merging this pull request may close these issues.

ドキュメント最新化

1 participant