Skip to content

docs: protect synced FAQ path in AGENTS.md and CLAUDE.md#27

Merged
Nathan Schram (nathanschram) merged 1 commit intomainfrom
docs/protect-synced-faq-path
May 6, 2026
Merged

docs: protect synced FAQ path in AGENTS.md and CLAUDE.md#27
Nathan Schram (nathanschram) merged 1 commit intomainfrom
docs/protect-synced-faq-path

Conversation

@nathanschram
Copy link
Copy Markdown
Member

Summary

Follow-up to #25 / #26. Adds explicit "do not delete" + "update only when needed" guardrails for docs/faq/index.md so future Claude Code (and other AI tool) sessions don't sweep it away during cleanup or rewrite it on every release.

  • AGENTS.md (canonical): new "Synced Documentation Paths" section explaining what docs/faq/index.md is, that the marketing-site sync hard-fails without it, and the precise update trigger (an answer goes stale — not every release).
  • CLAUDE.md (Claude Code bridge): one row added to the Key Files table mirroring the same guardrail, so it's immediately visible to Claude Code at session start.

Other bridges (.cursorrules, Copilot instructions, .windsurfrules, .clinerules, GEMINI.md) are not edited because this repo's primary AI consumer is Claude Code itself, and AGENTS.md is the canonical source if those bridges ever get added.

Why this is "where needed"

  • MEMORY.md is for cross-session learned patterns, not project policy → not the right home.
  • .claude/rules/* rules cover context-file hygiene (line budgets, Signal Gate, AGENTS↔bridge consistency). The FAQ is user-facing docs, not a context file → out of scope.
  • Hooks like content-filter-guard.sh only protect against API content-filter blocks on Write — there's no existing pattern for "do not delete this path" that would benefit from extension here.

So: AGENTS.md + CLAUDE.md only. Surgical.

Verification

  • AGENTS.md: 59 lines (budget <120) ✅
  • CLAUDE.md: 56 lines (budget <80) ✅
  • typos clean ✅
  • Banned-phrase scan clean ✅
  • bash tests/check-version-consistency.sh green ✅
  • python3 tests/validate-frontmatter.py green ✅

Test plan

  • Visual: confirm "Synced Documentation Paths" renders correctly in AGENTS.md preview.
  • Visual: confirm new row appears at the bottom of the CLAUDE.md Key Files table.
  • CI: lint / links / consistency / validate-plugin / CodeQL / Socket / CodeRabbit all green.
  • (Already proven) docs/faq/index.md itself is unchanged and remains live on main.

🤖 Generated with Claude Code

Adds an explicit "Synced Documentation Paths" section to AGENTS.md
(canonical) and a Key Files row in CLAUDE.md so future sessions know:

- docs/faq/index.md is consumed by the marketing-site sync pipeline
  (scripts/docs-sync.config.ts under contextdocs). The upstream build
  hard-fails if the directory goes missing.
- Update the FAQ only when an answer goes stale — not on every release.
- Existing constraints (each H2 ends with ?, no placeholders, Australian
  English, banned-phrase rules from doc-standards.md) carry over to
  every revision.

Follow-up to #25 / #26 — keeps the FAQ from being deleted by routine
cleanup sweeps and makes the maintenance contract explicit.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 6, 2026

Warning

Rate limit exceeded

@nathanschram has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 42 minutes and 23 seconds before requesting another review.

To continue reviewing without waiting, purchase usage credits in the billing tab.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

ℹ️ Review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: a06f3b51-b912-481a-9dca-ff01c8556396

📥 Commits

Reviewing files that changed from the base of the PR and between 130e103 and 6eabad4.

📒 Files selected for processing (2)
  • AGENTS.md
  • CLAUDE.md
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch docs/protect-synced-faq-path

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 and usage tips.

@nathanschram Nathan Schram (nathanschram) merged commit d48b09c into main May 6, 2026
10 checks passed
@nathanschram Nathan Schram (nathanschram) deleted the docs/protect-synced-faq-path branch May 6, 2026 07:17
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