Skip to content

docs(onboarding): repair dead in-guide anchors and the duplicated rollup table - #219

Merged
ivanball merged 1 commit into
mainfrom
docs/onboarding-anchors-20260926
Sep 26, 2026
Merged

ivanball merged 1 commit into
mainfrom
docs/onboarding-anchors-20260926

Conversation

@ivanball

Copy link
Copy Markdown
Owner

Summary

Follow-up to #218. A new anchor-integrity check in the onboarding verifier (verify.ps1, check 4) found 591 in-guide links pointing at heading ids the site renderer never emits. This PR fixes all but 26 of them and removes a stale duplicate of the per-project rollup table.

  • 426 links to rolled-up test classes (catalogued per project, no heading of their own) now target #per-project-test-rollup.
  • 128 links to one member of a family section now target the family heading.
  • Renamed or re-slugged headings retargeted: IQueryHandler<in TQuery, TResult>, Result, BeginTwoFactorEnrollmentHandlerBase<TCommand>, HandlerTestBase<THandler>, two fixture families, DependencyInjection in Notifications, the primer section 4 link, and the dependency-manifest cycles link (now a stable #cycles id).
  • The testing chapter rendered two per-project rollup tables, the first one stale (for example MMCA.ADC.Conference.Shared.Tests 18 instead of 9). The stale copy is removed; the "Test suites by module" narrative sits under the rollup again with its counts synced to the inventory, and its stale hand-summed totals are dropped.
  • 99-coverage-audit.md: the 2026-09-26 note's verification bullet now carries the real figures.

Verification

  • verify.ps1: 0 missing, 0 rollup contract failures, 0 citation failures, rubric 34/34; anchor integrity 26 remaining (below).
  • test-verify.ps1: every check-specific test passes (31); the two baseline assertions fail only on the 26 open links.
  • npm run build: dead cross-links none.

Known open item

26 links name types that no longer exist in source, mostly the earlier scoring pipeline (ScoreEventSessionsHandler, AnthropicScoringService, SessionScoringProcessor, SessionScoringQueue) plus ValReturn<T>, CacheKeyPrefix, EventValidationRules, DuplicateKeyDetection, AdministrationRequestValidators, FakeRefreshSessionStore, AppHostCompositionSmokeTests. The prose around them is stale and needs re-authoring from source rather than relinking.

🤖 Generated with Claude Code

…lup table

The new anchor-integrity check in verify.ps1 found 591 in-guide links to
heading ids the renderer never emits. Fixed here:

- 426 links to rolled-up test classes now target the per-project rollup table
- 128 links to single members of a family section now target the family heading
- renamed or re-slugged headings retargeted (IQueryHandler, Result,
  BeginTwoFactorEnrollmentHandlerBase<TCommand>, HandlerTestBase<THandler>,
  primer and dependency-manifest links; the cycles link uses a stable #cycles id)
- a stale second copy of the per-project rollup table that rendered above the
  current one removed; the module narrative moved back under the rollup with
  its counts synced and the stale hand-summed totals dropped
- 99-coverage-audit note updated; 26 links naming removed types remain open

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@ivanball
ivanball merged commit 57855c6 into main Sep 26, 2026
4 checks passed
@ivanball
ivanball deleted the docs/onboarding-anchors-20260926 branch September 26, 2026 16:34
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