Skip to content

fix(docs): update 225 broken internal links after building/ restructure#5742

Merged
bokelley merged 1 commit into
adcontextprotocol:mainfrom
garvitkaushik-123:fix/docs-broken-internal-links
Jul 18, 2026
Merged

fix(docs): update 225 broken internal links after building/ restructure#5742
bokelley merged 1 commit into
adcontextprotocol:mainfrom
garvitkaushik-123:fix/docs-broken-internal-links

Conversation

@garvitkaushik-123

Copy link
Copy Markdown
Contributor

Summary

  • Fixes 225 broken internal links across 86 documentation files
  • All links pointed to the old flat building/implementation/, building/integration/, building/understanding/ paths that were reorganized into building/by-layer/L0-L4/, building/concepts/, building/operating/, and building/verification/

What changed

20 stale path prefixes replaced with their current locations:

Old path New path
building/integration/accounts-and-agents building/by-layer/L2/accounts-and-agents
building/implementation/security building/by-layer/L1/security
building/compliance-catalog building/verification/compliance-catalog
building/understanding/security-model building/concepts/security-model
building/implementation/error-handling building/by-layer/L3/error-handling
building/implementation/task-lifecycle building/by-layer/L3/task-lifecycle
building/implementation/webhooks building/by-layer/L3/webhooks
building/integration/account-state building/by-layer/L2/account-state
building/validate-your-agent building/verification/validate-your-agent
building/implementation/orchestrator-design building/operating/orchestrator-design
building/implementation/async-operations building/by-layer/L3/async-operations
building/build-an-agent building/by-layer/L4/build-an-agent
building/integration/authentication building/by-layer/L2/authentication
building/conformance building/verification/conformance
building/understanding/industry-landscape building/concepts/industry-landscape
building/integration/context-sessions building/by-layer/L2/context-sessions
building/implementation/seller-integration building/operating/seller-integration
building/understanding/protocol-comparison building/concepts/protocol-comparison
building/get-test-ready building/verification/get-test-ready
building/implementation/comply-test-controller building/by-layer/L3/comply-test-controller

Every replacement target was verified to exist as an .mdx file in the current tree. No content changes — purely mechanical path updates.

Test plan

  • Pre-commit hooks pass (unit tests, typecheck, image quality)
  • Pre-push Mintlify broken-links check passes ("no broken links found")
  • Spot-checked representative diffs in intro.mdx, faq.mdx, quickstart.mdx, and task reference pages
  • Verify links resolve correctly on the deployed docs site

The docs/building/ tree was reorganized from flat
implementation/integration/understanding directories into
by-layer/L0-L4, concepts, operating, and verification, but
cross-references in 86 files still pointed at the old paths.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

@aao-release-bot aao-release-bot Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Argus review could not complete

The automated review encountered an issue (possibly reached max turns, timed out, or failed to post the final gh pr review). A human reviewer should take this PR.

View workflow run

This is an automated message from the Argus AI review workflow.

@bokelley bokelley left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The link replacements themselves are clean and CI validates them, but the required changeset check is red. This PR changes published protocol documentation, including normative task/spec pages, so please add a descriptive changeset for the broken-link correction, then rerun CI.

@bokelley

Copy link
Copy Markdown
Contributor

Correction to the elided changeset type in my review: add a descriptive adcontextprotocol patch changeset.

@bokelley

Copy link
Copy Markdown
Contributor

@garvitkaushik-123 — to address @bokelley's review, please add the following file to your branch and push:

.changeset/fix-building-docs-broken-internal-links.md

---
"adcontextprotocol": patch
---

Fix 225 broken internal links across 86 documentation files after the `building/` directory restructure; updates 20 stale path prefixes (e.g. `building/implementation/`, `building/integration/`, `building/understanding/`) to their current locations under `building/by-layer/`, `building/concepts/`, `building/operating/`, and `building/verification/`.
# From your branch:
cat > .changeset/fix-building-docs-broken-internal-links.md << 'EOF'
---
"adcontextprotocol": patch
---

Fix 225 broken internal links across 86 documentation files after the `building/` directory restructure; updates 20 stale path prefixes (e.g. `building/implementation/`, `building/integration/`, `building/understanding/`) to their current locations under `building/by-layer/`, `building/concepts/`, `building/operating/`, and `building/verification/`.
EOF
git add .changeset/fix-building-docs-broken-internal-links.md
git commit -m "chore(changeset): add patch changeset for broken-link fix"
git push

This is a patch bump because link corrections in normative docs are patch-eligible per repo policy.


Generated by Claude Code

@bokelley bokelley left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The required adcontextprotocol patch changeset is now present. The link replacements were already reviewed as clean; CI can rerun on the updated head.

@bokelley
bokelley enabled auto-merge (squash) July 18, 2026 19:51
@bokelley
bokelley requested a review from EmmaLouise2018 July 18, 2026 19:51
@bokelley
bokelley merged commit 0c8fe6e into adcontextprotocol:main Jul 18, 2026
17 of 18 checks passed
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.

2 participants