docs: independent validation section and technical write-ups - #184
Merged
Conversation
README section built from real, currently-verified numbers (record count, crosswalk count), not carried forward from an earlier draft. Write-ups sourced entirely from live record data, no reconstructed specifics.
# Conflicts: # dist/ave-records-latest.manifest.json
Resolves conflict in dist/ave-records-latest.manifest.json (generated_at timestamp only; both sides already agreed on record_count: 80). Regenerated the manifest via scripts/build-records.js rather than hand-editing the timestamp, same approach used in e4db9d4. 339 tests pass, all 80 records validate against schema.
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.
New README section on independent crosswalk validation, numbers pulled live rather than assumed. First technical write-up(s) for the highest-severity record(s), every specific claim traced to the real record JSON.
Numbers, verified live (not from memory)
Write-ups (3, not the full priority list, quality over quantity per the task brief)
Every mechanism description, AARF/AIVSS arithmetic, IOC, and remediation step traces to the live
dist/ave-records-latest.jsonpull, not reconstructed from memory. No em-dashes anywhere in the new content.Two things flagged, not fixed here (out of this task's scope)
## Statstable is stale: it reads CRITICAL 1 / HIGH 14 / MEDIUM 56 / LOW 2 = 73, but the live pull (and the file's own hero badge) says 80 records (1/15/62/2). Undercounts by 7 records. Small doc-sync fix, not bundled into this PR.blocking: true, vs. the record's actual external-callback-redirect mechanism). That's on ave-site, out of this repo's scope; flagging for separate follow-up.Base branch note
This repo's own convention is PRs target
develop, notmain(confirmed with the requester), so this targetsdevelopdespite the task brief's literal--base main.🤖 Generated with Claude Code