docs(agents): scope external-skill precedence to generic guidance

[BigSimmo]

Follow-up to #1306. CodeRabbit's autofix landed the two inline findings (WCAG target-size citation, third-party ref verification) before that PR merged, but the summary-level nitpick had no inline thread and was skipped.

"Repo docs and committed tests always win" reads as absolute. This scopes it to conflicts with external, generic guidance and states it does not override system, developer, user, security, or compliance requirements.

Docs-only, 3 insertions to AGENTS.md.

RAG impact: no retrieval behaviour change — documentation only, no code touched.

Checks

• prettier --check AGENTS.md — clean
• npm run docs:check-links — 1287 repo path references resolve
• npm run docs:check-index — all 31 top-level modules/routes indexed

🤖 Generated with Claude Code

Summary by CodeRabbit

• Documentation
  • Clarified the precedence of external guidance when it conflicts with repository documentation or tests.
  • Confirmed that system, developer, user, security, and compliance requirements remain higher priority.

[supabase]

This pull request has been ignored for the connected project sjrfecxgysukkwxsowpy because there are no changes detected in supabase directory. You can change this behaviour in Project Integrations Settings ↗︎.

---

Preview Branches by Supabase.
Learn more about Supabase Branching ↗︎.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 0c13d144-1f2c-4e38-a6eb-69ac6e86dc68

📥 Commits

Reviewing files that changed from the base of the PR and between fe09c33 and 352b1f6.

📒 Files selected for processing (1)

• AGENTS.md

---

📝 Walkthrough

Walkthrough

AGENTS.md expands the “External skill precedence” guidance to clarify repository rule and test precedence, while preserving system, developer, user, security, and compliance requirements as higher priority.

Changes

Guidance precedence

Layer / File(s)	Summary
Precedence rule clarification AGENTS.md	The external guidance tie-breaker is explicitly limited to conflicts with repository documentation or committed tests; higher-priority requirements remain authoritative.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Possibly related PRs

• BigSimmo/Database#898: Updates AGENTS.md guidance on external versus repository-local precedence.
• BigSimmo/Database#1306: Clarifies AGENTS.md precedence rules for repository documentation, tests, and external guidance.

Suggested labels:codex

Suggested reviewers:cursoragent

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly matches the docs-only change clarifying external-skill precedence scope.
Description check	✅ Passed	The description includes a clear summary and verification results, though it does not use the template headings exactly.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

---

_{Comment @coderabbitai help to get the list of available commands.}