Skip to content

chore: add design drift audit skill - #157

Open
sakatai11 wants to merge 1 commit into
mainfrom
chore/issue-156-design-drift-audit
Open

sakatai11 wants to merge 1 commit into
mainfrom
chore/issue-156-design-drift-audit

Conversation

@sakatai11

Copy link
Copy Markdown
Owner

概要

docs/design.md とリポジトリ実装の仕様差分を、週次または任意のタイミングで双方向に監査する Codex/Claude 共通スキルを追加します。

主な変更点

  • design-drift-audit スキルを追加
    • full / weekly の2モード
    • design → repo / repo → design の双方向検証
    • design-only / implementation-only / conflict / deferred-intentional / aligned の分類
    • 根拠、重要度、推奨対応、品質ゲート結果を含むレポート形式
    • 読み取り専用を既定とし、明示承認なしの修正・Issue作成・commit・push・PR作成を禁止
  • Claude / Codex の探索用シンボリックリンクを追加

関連 Issue

refs #156

スクリーンショット(必要に応じて)

なし(UI変更なし)

変更の種類

  • 🐛 バグ修正 (Bug fix)
  • ✨ 新機能 (New feature)
  • 💄 UI/スタイル変更 (UI/Style changes)
  • ⚡ パフォーマンス改善 (Performance improvement)
  • ♻️ リファクタリング (Refactoring)
  • 📝 ドキュメント更新 (Documentation)
  • 🧪 テスト追加・修正 (Tests)
  • 🔧 設定・ツール変更 (Configuration/Tools)
  • 🚀 デプロイ・インフラ (Deploy/Infrastructure)

テスト

  • Skill Creator quick_validate.py: pass
  • skill-audit: pass
  • hooks / agent contract tests: pass
  • pnpm lint: pass
  • pnpm test: pass(45 files / 226 tests)
  • content check と各 TypeScript tsc: pass
  • pnpm typecheck: Work環境で tsx の UNIX socket listen が EPERM となるためラッパーのみ完走不可(Node 22 / 24で再現)。型チェック本体は個別実行で pass

チェックリスト

  • コードレビューの準備ができている
  • 関連するドキュメントを更新した
  • 破壊的変更がある場合、適切に文書化した
  • セキュリティ上の問題がないことを確認した
  • 既存のテストが通ることを確認した
  • コーディングガイドラインに従っている
  • 動作確認済み(主要ブラウザ・画面サイズも確認)
  • 警告が発生していない
  • console.log や debugger が残っていない
  • 機密情報や API キーが含まれていない
  • 不要な再レンダリングが発生していない(useMemo や useCallback など使用検討)
  • 冗長なコードを避け、関数やコンポーネントを再利用している
  • ドキュメントが更新されている(必要に応じて)
  • PR の説明が分かりやすく書かれている

追加の注意事項

  • 内部ディスカバリーレビュー: APPROVE(指摘なし)
  • 内部最終検証: APPROVE(指摘なし)
  • リスクベース判定: LR-1(非実行ファイルのみ)のため外部CLIレビューは不要
  • ローカル環境固有の pnpm typecheck ラッパー制約はPR CIで最終確認します

@coderabbitai

coderabbitai Bot commented Sep 2, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Team

Run ID: e803a721-db3b-4c95-bc1e-9264400e3c8d

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

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.

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 73d7e528f7

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

## 1. 対象スナップショットを確定する

1. 現在のブランチ、HEAD SHA、作業ツリー状態、`develop` と `origin/develop` のSHAを確認する。
2. GitHubへの読み取りアクセスが利用できる場合は `origin/develop` をfetchし、監査対象を最新化する。利用できなければ取得済みrefの日時と制約を報告する。

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Badge Fetch the requested target ref instead of always develop

When the caller uses the supported ref override for another remote branch, this step still fetches only origin/develop (and the preceding step only checks develop SHAs). The requested ref can therefore remain stale or unavailable even though remote access succeeds, causing the audit to inspect an outdated snapshot or stop unnecessarily. Resolve and refresh the selected target ref rather than hard-coding origin/develop.

Useful? React with 👍 / 👎.

Base automatically changed from develop to main September 13, 2026 10:54
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