fix(docs): update 225 broken internal links after building/ restructure#5742
Conversation
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>
There was a problem hiding this comment.
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.
This is an automated message from the Argus AI review workflow.
bokelley
left a comment
There was a problem hiding this comment.
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.
|
Correction to the elided changeset type in my review: add a descriptive adcontextprotocol patch changeset. |
|
@garvitkaushik-123 — to address @bokelley's review, please add the following file to your branch and push:
---
"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 pushThis is a Generated by Claude Code |
bokelley
left a comment
There was a problem hiding this comment.
The required adcontextprotocol patch changeset is now present. The link replacements were already reviewed as clean; CI can rerun on the updated head.
Summary
building/implementation/,building/integration/,building/understanding/paths that were reorganized intobuilding/by-layer/L0-L4/,building/concepts/,building/operating/, andbuilding/verification/What changed
20 stale path prefixes replaced with their current locations:
building/integration/accounts-and-agentsbuilding/by-layer/L2/accounts-and-agentsbuilding/implementation/securitybuilding/by-layer/L1/securitybuilding/compliance-catalogbuilding/verification/compliance-catalogbuilding/understanding/security-modelbuilding/concepts/security-modelbuilding/implementation/error-handlingbuilding/by-layer/L3/error-handlingbuilding/implementation/task-lifecyclebuilding/by-layer/L3/task-lifecyclebuilding/implementation/webhooksbuilding/by-layer/L3/webhooksbuilding/integration/account-statebuilding/by-layer/L2/account-statebuilding/validate-your-agentbuilding/verification/validate-your-agentbuilding/implementation/orchestrator-designbuilding/operating/orchestrator-designbuilding/implementation/async-operationsbuilding/by-layer/L3/async-operationsbuilding/build-an-agentbuilding/by-layer/L4/build-an-agentbuilding/integration/authenticationbuilding/by-layer/L2/authenticationbuilding/conformancebuilding/verification/conformancebuilding/understanding/industry-landscapebuilding/concepts/industry-landscapebuilding/integration/context-sessionsbuilding/by-layer/L2/context-sessionsbuilding/implementation/seller-integrationbuilding/operating/seller-integrationbuilding/understanding/protocol-comparisonbuilding/concepts/protocol-comparisonbuilding/get-test-readybuilding/verification/get-test-readybuilding/implementation/comply-test-controllerbuilding/by-layer/L3/comply-test-controllerEvery replacement target was verified to exist as an
.mdxfile in the current tree. No content changes — purely mechanical path updates.Test plan