docs: add AGENTS.md — Codex review entry point (pointer-only)#281
Merged
Conversation
…ean) Codex reads AGENTS.md at repo root for instructions. Give it elidex's project-specific review lenses so its PR reviews add diversity value (elidex-specific design issues) beyond generic correctness the local Claude gate already covers. Pointer-only by design: delegates rules to CLAUDE.md and the 5-axis review lenses to .claude/skills/elidex-review/axes.md (both SSoT). No axis/rule content is duplicated here — duplication would drift (CLAUDE.md "One issue, one way"). Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
There was a problem hiding this comment.
Pull request overview
Adds a root-level AGENTS.md to serve as a pointer-only entry point for Codex/agent-based PR review guidance, keeping CLAUDE.md and the existing .claude/skills/elidex-review/axes.md as the single sources of truth.
Changes:
- Introduces
AGENTS.mdwith instructions to defer toCLAUDE.mdfor engineering/design philosophy. - Directs PR reviewers to apply the 5-axis elidex review lenses from
.claude/skills/elidex-review/axes.md. - Emphasizes prioritizing design/correctness and citing specific spec sections for spec-related claims.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Comment on lines
+3
to
+5
| This repo's engineering rules, architecture, and design philosophy are the | ||
| **`CLAUDE.md`** at the repo root — read it first; it is the single source of | ||
| truth. Do not restate or fork its rules here. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Why
Codex (ChatGPT Plus, non-metered) is being trialed as the post-push non-Claude diversity reviewer (replacing per-credit Copilot). A generic Codex review adds little — the local Claude pre-push gate already covers generic correctness/style. To get diversity value, Codex needs elidex's project-specific review lenses (Layering mandate / ECS-native / spec citation / core-compat split). Codex reads
AGENTS.mdat the repo root for instructions, so that's where the steering goes.What
A pointer-only
AGENTS.md:CLAUDE.md(SSoT)..claude/skills/elidex-review/axes.md(SSoT) + prioritize design over nits + cite spec sections.Doc-only (root markdown, no Rust) → Rust CI jobs skip via path filter.
🤖 Generated with Claude Code