Webapp: correct the mapping count and record the data repair on the About timeline - #113
Merged
Merged
Conversation
…imeline The landing paragraph still claimed 3,781 control mappings; #110 removed ten rows that were table header lines the parser had mistaken for mappings, so the figure is 3,771. The About timeline gains a row for the repair itself — identifiers now parsed by each framework's grammar rather than by column position, the registries migrated off the ids that produced (1,541 → 991 controls), and the MITRE ATLAS re-transcription. The earlier rows are left as they are: they record what shipped at the time, which is what a timeline is for. Data-fed content only; no structural change. Co-Authored-By: Claude Opus 5 (1M context) <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
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.
Item 1 of the three follow-ups to #35.
Changes
The earlier timeline rows — including v4.0's "1,372 controls" — are left exactly as they are. They record what shipped at the time, which is what a timeline is for; the new row is what tells a reader the number moved and why.
Scope
Data-fed content only, two lines. No structural change, no new markup, no script. C1 respected: the row names no person.
Verification
npm run test:scripts: 85 passed, 0 failed — including the route smoke test from Test the webapp: render every route in CI #104, which walks all 15 routes and checks every timeline row has a date and text.node scripts/validate.js: 0 errors.Still hardcoded
That landing sentence holds
51 entries,26 frameworksand the mapping count as literal text, so it can drift again — this is the second time it has. The page already computes registry totals live from the data bundles. Say the word and I'll render those three figures the same way, which would need a small script change rather than a data edit, so I have not done it under C2.🤖 Generated with Claude Code