Repository navigation
Conversation
Two additions: 1. Rule compliance audit now reads project docs/ai/review.md as project-specific deltas on top of the skill's default contracts (not a replacement). 2. Learning Closeout routing guidance: cross-project learnings belong in dotfiles/docs/ai/, not a project-local docs/ai/ that silently diverges. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01MbsaUuqfh83jqTWAWDKHDx
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 1307015e27
ℹ️ 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".
|
|
||
| **Routing:** when a finding is cross-project (the same failure mode or missing | ||
| guardrail would apply across codebases, not just this one), route the learning | ||
| to `dotfiles/docs/ai/<topic>.md` rather than the project's own `docs/ai/`. |
There was a problem hiding this comment.
When /deep-review runs from another project, dotfiles/docs/ai/<topic>.md is interpreted as a nonexistent child path, and it also conflicts with the preceding /learn workflow, which captures into the current repository's docs/learnings/ store. As a result, a cross-project finding may remain local or bypass capture, deduplication, and promotion entirely. Invoke learn --repo <dotfiles-checkout> capture for global evidence and let the learning workflow promote it to the canonical global docs/ai document.
Useful? React with 👍 / 👎.
Target
skill:
.agents/skills/deep-review/SKILL.mdObservations addressed
docs/ai/review.mdas project-specific deltas on top of the skill's default contracts when it exists; (2) Learning Closeout now includes routing guidance — cross-project findings go todotfiles/docs/ai/<topic>.mdrather than a project-local docs/ai/ that silently diverges from other projects.Generated by Claude Code