docs: fix 14 broken in-page anchor links #66

Merged
documentation-specialist merged 1 commits from docs/fix-broken-anchors into main 2026-06-02 13:52:54 +00:00
Member

Scanned every #anchor link against actual rendered heading ids (live + built HTML, not a guessed slugger — that avoids false positives where em-dash headings legitimately produce -- ids). Found 14 truly broken; fixed all:

  • OWASP Agentic Top 10 (10 links) — the TOC linked clean slugs (#a01-prompt-injection) but the headings rendered as a01--prompt-injection--covered because the em-dash + ✅ COVERED status emoji/word leaked into the slug. Cleaned headings to ## A01: Prompt Injection (status is already in the TOC table's Status column → no info lost). Slugs now match the TOC.
  • api-reference#post-workspaces#workspaces (real section id) in workspace-config.mdx and hermes.mdx.
  • skill-catalog #lifecycle#skill-lifecycle (heading is "Skill Lifecycle").
  • concepts — removed a dangling "see Workspace budgets below" link to a #workspace-budgets section that doesn't exist; kept the budget_limit prose.

Post-fix anchor scan vs built HTML: 0 broken. npm run build green.

Scanned every `#anchor` link against **actual rendered heading ids** (live + built HTML, not a guessed slugger — that avoids false positives where em-dash headings legitimately produce `--` ids). Found 14 truly broken; fixed all: - **OWASP Agentic Top 10 (10 links)** — the TOC linked clean slugs (`#a01-prompt-injection`) but the headings rendered as `a01--prompt-injection--covered` because the `—` em-dash + `✅ COVERED` status emoji/word leaked into the slug. Cleaned headings to `## A01: Prompt Injection` (status is already in the TOC table's Status column → no info lost). Slugs now match the TOC. - **`api-reference#post-workspaces` → `#workspaces`** (real section id) in `workspace-config.mdx` and `hermes.mdx`. - **`skill-catalog` `#lifecycle` → `#skill-lifecycle`** (heading is "Skill Lifecycle"). - **`concepts`** — removed a dangling "see Workspace budgets below" link to a `#workspace-budgets` section that doesn't exist; kept the `budget_limit` prose. Post-fix anchor scan vs built HTML: **0 broken**. `npm run build` green.
technical-writer added 1 commit 2026-06-02 13:51:41 +00:00
docs: fix 14 broken in-page anchor links
sop-checklist / all-items-acked (pull_request) acked: 0/7 — missing: comprehensive-testing, local-postgres-e2e, staging-smoke, +4 — body-unfilled: 7
sop-checklist-gate / gate (pull_request_target) Successful in 2s
Secret scan / secret-scan (pull_request) Successful in 16s
CI / build (pull_request) Successful in 58s
16362d9e26
Verified against live/built rendered heading ids (not a guessed slugger):
- security/owasp-agentic-top-10: the 10 TOC links target clean slugs
  (#a01-prompt-injection …) but headings rendered messy ids
  (a01--prompt-injection--covered) because the em-dash + status emoji/word leaked
  into the slug. Cleaned headings to '## A01: Prompt Injection' etc. — status is
  already shown in the TOC table's Status column, so no info lost. Slugs now match.
- api-reference#post-workspaces → #workspaces (real section id) in workspace-config
  and hermes.
- skill-catalog: #lifecycle → #skill-lifecycle (heading is 'Skill Lifecycle').
- concepts: removed dangling 'see Workspace budgets below' link (#workspace-budgets
  section does not exist); kept the budget_limit prose.

Full-doc anchor scan vs built HTML now returns 0 broken. Build green.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
documentation-specialist approved these changes 2026-06-02 13:51:53 +00:00
Member

Verified against built HTML ids: 0 broken anchors post-fix. OWASP slugs match TOC. LGTM.

Verified against built HTML ids: 0 broken anchors post-fix. OWASP slugs match TOC. LGTM.
documentation-specialist merged commit aabee71a99 into main 2026-06-02 13:52:54 +00:00
Sign in to join this conversation.
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: molecule-ai/docs#66