docs(onboarding): carry the primer's drift fixes into the other chapters - #222
Merged
Merged
Conversation
Scanned every onboarding chapter for the stale facts corrected in the primer (#221) and fixed the real hits: - CONCEPT-MAPS: one sealed context class per engine, PostgreSQL added. - group-07: a stale RowVersion is 412 under [SupportsIfMatch], 409 only on endpoints without it (ADR-035). - group-14: the DataSource engine list includes PostgreSQL. - group-28 and devops-testing: twenty-two packages, five general test packages plus AI.Testing in group-27; FACTS line refs; ci.yml:158. - devops-cicd: four freshness gates, not three. - primer: ADR-035 row states the ETag / If-Match contract (428 / 412). Group chapters are rebuilt from Docs/Onboarding/parts by concat.ps1; a baseline concat run reproduced every chapter unchanged before the edit. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.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.
Summary
Follow-up to #221. Every onboarding chapter was scanned for the stale facts corrected in the primer (package count, decorator chain, HS256 default, three engines / one context class, alias counts, 409 vs 412, OAuth providers, conformance-base count, AppDomain discovery, freshness-gate count, DataProtection adoption, settings facades, export header, container postures, extension-block counts, package versions, Redis health check, moved line citations, g27/g28 labels). Real hits, each checked against source:
RowVersionis412on a[SupportsIfMatch]endpoint and409only on endpoints without it (ADR-035,SupportsIfMatchAttribute.cs:130-138).DataSourceengine list includes PostgreSQL.AI.Testingcovered in group-27), FACTS line refs,ci.yml:158.ETag/If-Matchcontract (428 with no precondition, 412 when stale).Left as is, deliberately: history quotes in
99-coverage-audit.md's change log, and two "Google/GitHub" mentions that faithfully mirrorIExternalAuthBroker's own doc comment and the Identity test coverage.Verification
Docs/Onboarding/partsbyconcat.ps1; a baseline run before editing reproduced every chapter unchanged, and the post-edit run changed only the three edited chapters.npm run build: dead cross-links: none.Merging publishes the live site.
Generated with Claude Code