Repository navigation
perf: store census data as single JSON arrays - #2
Conversation
The proven-suite bench gate failed: ijson_jcs ran 63x its baseline. The checker spawns one runtime process per JSON Lines line (~15 ms each), and this PR added ~750 lines. repos, unreachable and catalogue-rows are now one JCS array per file (.json), so the checker spawns three times: check-ijson-jcs --enforce over the whole repo drops from ~11.5 s to ~0.3 s locally, all 15 files canonical. The per-line spawning itself is a template checker issue and is left for a separate fix upstream. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_019a95kk88TZ1iegsBK3kox3
|
Caution Review failedThe pull request is closed. Finishing Touches
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
|
Two checks are red on be9978b: Canon lockstep and openssf-compliance. Neither is caused by this PR's change, which only switches the census data from JSONL to single JSON arrays.
Both are red on Generated by Claude Code |
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |



Summary
#1 was merged at 55aa899, just before this follow-up commit was pushed to the same branch, so it never reached
main. This PR carries that one commit.The proven-suite bench gate failed on #1: the
ijson_jcsbenchmark ran at 63× its baseline.scripts/check-ijson-jcs.shstarts a separate runtime process for every JSON Lines line (about 15 ms each), and the census added about 750 lines.repos,unreachableandcatalogue-rowsare now each one I-JSON/JCS array in a.jsonfile, so the checker starts three processes instead of about 750. The census scripts, Justfile and README are updated to match.Checks (local)
check-ijson-jcs.sh --enforce .drops from about 11.5 s to about 0.3 s, with all 16 JSON files canonical.just catalogue-checkpasses; the catalogue renders byte-identically from the arrays.check-root-shape,check-no-md-in-docsandcheck-no-vlangall pass.Not in this PR
openssf-complianceandCanon lockstepstay red until the repo is minted withjust repo-init. That's being handled separately.Generated by Claude Code