docs: final-verify cleanup (synthesis pointers, source discipline, anchor fixes)#34
Merged
Conversation
…chor fixes) - patterns/patterns.md: drop the cross-cutting doctrine block (vertical slice vs Clean Architecture, bounded contexts, domain events) — patterns.md is a thin index, not a doctrine owner. Anchor slugs corrected throughout. - docs/decision-trees.md Tree 13 (Caching): OutputCache row now points at ch02 §9 as the OWNER; ch03 §14 stays for the data-cache matrix only. Drop the 'Default per ch03' framing. - coverage-map.md ch06: ServiceDefaults caveat — it ships only /health + /alive in Development; production must explicitly MapHealthChecks for /health/live, /health/ready, /health/startup. Cite ch06 §10 + aspire.dev. Chapter section anchor slugs corrected (no more #chapter-NN--name). - checklist.md §9 AuthN/AuthZ: route to local owners first (docs/02-aspnetcore.md#10-authnauthz, decision-trees Tree 12); the external Entra sample is demoted to a non-normative 'See also'. - docs/01-foundations.md Sources: rebalance to <60% learn.microsoft / ≥40% non-learn primary by adding ECMA-334, ECMA-335, dotnet/csharplang LDM notes (C# 13/14 proposals, NRT spec, Records proposal), dotnet/roslyn source generator cookbook + interceptors, dotnet/runtime GC + Tiered/PGO + DI + LoggerMessage + exceptions design docs, RFC 9110, devblogs posts (Stephen Toub async internals, Mads Torgersen NRT/Records, OptionsValidator announcement), Sigstore. Final ratio: 57.9% learn / 42.1% non-learn. - Remove Medium and YouTube links from chapter Sources blocks (docs/02-aspnetcore.md, docs/03-data.md, docs/04-testing.md, docs/05-performance.md). Replace maoni0.medium.com with devblogs.microsoft.com/dotnet/author/maoni and dotnet/runtime GC docs. - docs/05-performance.md: move 'Community / people to follow' out of the canonical Sources block into a 'Further reading (non-normative)' section at the end of the chapter. - Fix all broken internal anchors. The script-defined slug algorithm collapses runs of whitespace to a single hyphen, so '#chapter-01--foundations' → '#chapter-01-foundations', '#5-di--lifetimes' → '#5-di-lifetimes', etc. '`docs/05-performance.md` line ~410: stale '#chapter-01--foundations' on 01-foundations.md (the slug lives in coverage-map.md) → repointed at ch01 §10 + the coverage-map ownership entry. Anchor verification script now reports 'all anchors resolve'. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
GitHub's slug renderer keeps a hyphen for each removed punctuation char, producing '--' where headings contain '—', '&', '/'. Markdownlint MD051 validates intra-file fragments using the GitHub renderer, not the verification script's collapsed-whitespace algorithm. Revert the collapsed anchors so the same-file links in coverage-map resolve. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
slice vs Clean Architecture, bounded contexts, domain events) — patterns.md
is a thin index, not a doctrine owner. Anchor slugs corrected throughout.
ch02 §9 as the OWNER; ch03 §14 stays for the data-cache matrix only.
Drop the 'Default per ch03' framing.
/health/live, /health/ready, /health/startup. Cite ch06 §10 + aspire.dev.
Chapter section anchor slugs corrected (no more #chapter-NN--name).
(docs/02-aspnetcore.md#10-authnauthz, decision-trees Tree 12); the
external Entra sample is demoted to a non-normative 'See also'.
non-learn primary by adding ECMA-334, ECMA-335, dotnet/csharplang LDM
notes (C# 13/14 proposals, NRT spec, Records proposal), dotnet/roslyn
source generator cookbook + interceptors, dotnet/runtime GC + Tiered/PGO
(Stephen Toub async internals, Mads Torgersen NRT/Records, OptionsValidator
announcement), Sigstore. Final ratio: 57.9% learn / 42.1% non-learn.
(docs/02-aspnetcore.md, docs/03-data.md, docs/04-testing.md,
docs/05-performance.md). Replace maoni0.medium.com with
devblogs.microsoft.com/dotnet/author/maoni and dotnet/runtime GC docs.
canonical Sources block into a 'Further reading (non-normative)' section
at the end of the chapter.
collapses runs of whitespace to a single hyphen, so '#chapter-01--foundations'
→ '#chapter-01-foundations', 'fix(ch02): apply rubber-duck critique #5-di--lifetimes' → 'fix(ch02): apply rubber-duck critique #5-di-lifetimes', etc.
'
docs/05-performance.mdline ~410: stale '#chapter-01--foundations' on01-foundations.md (the slug lives in coverage-map.md) → repointed at
ch01 §10 + the coverage-map ownership entry.
Anchor verification script now reports 'all anchors resolve'.
Co-authored-by: Copilot 223556219+Copilot@users.noreply.github.com