Skip to content

docs(security): fix three stale /docs/reference/security URLs that 404 in 3.0#3332

Draft
bokelley wants to merge 1 commit intomainfrom
claude/issue-2812-security-md-url-fix
Draft

docs(security): fix three stale /docs/reference/security URLs that 404 in 3.0#3332
bokelley wants to merge 1 commit intomainfrom
claude/issue-2812-security-md-url-fix

Conversation

@bokelley
Copy link
Copy Markdown
Contributor

Closes #2812

SECURITY.md at repo root pointed to https://docs.adcontextprotocol.org/docs/reference/security in three places; that path does not exist in 3.0 and 404s. This PR replaces all three occurrences with the canonical 3.0 location (/docs/building/implementation/security) and adds a cross-link to the threat-model page (/docs/building/understanding/security-model) from the Security Best Practices section. The old path was not caught by the Mintlify broken-links pre-push hook because that hook scans docs/** MDX files, not repo-root markdown.

Non-breaking justification: URL-only update in a root markdown file; no schema, task definition, or wire-format change.

Pre-PR review:

  • code-reviewer: approved — all 3 stale URLs replaced correctly; changeset --empty (docs-only, correct); nit: /docs/reference/v2-sunset link in version table is out of scope for this PR but may also need checking.
  • docs-expert: approved — cross-link placement in Security Best Practices section is appropriate; nit: appended sentence is slightly run-on (prose only, not a blocker).

Triage-managed PR. This bot does not currently iterate on
review comments or PR conversation threads (only on the source
issue). To unblock:

  • Push fixup commits directly: gh pr checkout <num>
    fix → push.
  • Or re-trigger: comment /triage execute on the source
    issue.

See #3121
for context.

Session: https://claude.ai/code/session_0156uh5mfdvVz4Y2zP8JwbRw


Generated by Claude Code

…4 in 3.0

Replace all three occurrences of the old `/docs/reference/security` path
in SECURITY.md with `/docs/building/implementation/security` (the canonical
3.0 normative guide), and add a cross-link to the threat-model page. The
old path wasn't scanned by the Mintlify broken-links hook because SECURITY.md
is a repo-root file, not an MDX page in docs/.

Closes #2812

https://claude.ai/code/session_0156uh5mfdvVz4Y2zP8JwbRw
@bokelley bokelley added documentation Improvements or additions to documentation claude-triaged Issue has been triaged by the Claude Code triage routine. Remove to re-triage. labels Apr 26, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

claude-triaged Issue has been triaged by the Claude Code triage routine. Remove to re-triage. documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

SECURITY.md: fix stale /docs/reference/security URL (404s in 3 places)

2 participants