Conversation
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Team Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
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. Comment |
There was a problem hiding this comment.
💡 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の日時と制約を報告する。 |
There was a problem hiding this comment.
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 👍 / 👎.
概要
docs/design.mdとリポジトリ実装の仕様差分を、週次または任意のタイミングで双方向に監査する Codex/Claude 共通スキルを追加します。主な変更点
design-drift-auditスキルを追加full/weeklyの2モード関連 Issue
refs #156
スクリーンショット(必要に応じて)
なし(UI変更なし)
変更の種類
テスト
quick_validate.py: passskill-audit: passpnpm lint: passpnpm test: pass(45 files / 226 tests)tsc: passpnpm typecheck: Work環境でtsxの UNIX socket listen がEPERMとなるためラッパーのみ完走不可(Node 22 / 24で再現)。型チェック本体は個別実行で passチェックリスト
追加の注意事項
pnpm typecheckラッパー制約はPR CIで最終確認します