Repository navigation
fix(bolt): compile hot splits once without reverting main - #2493
seonghobae wants to merge 3 commits into
Conversation
Rebuild #2205 on current main with only the intended module-level re.compile / native split-join changes; the post-merge tip had deleted queue-health and coalesce control-plane files. Co-authored-by: Cursor <cursoragent@cursor.com>
Keep hash-pin detection, package import names, and receipt slug boundaries while the hot patterns compile once.
The compiled clause boundary must still ignore case, matching the previous inline re.split flag.
|
Warning Review limit reachedNext included review available in 17 minutes. View limit detailsLimit details: You’ve used the included review currently available. You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. Review configuration: ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Advanced Run ID: 📒 Files selected for processing (5)
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 |
|
Exact-head admission correction — Ready is review admission only. Fresh audit against base
This PR is moved to Draft/Proposed until the causal owner repair is present on a successor exact head and re-audited. Queued/pending work is neither an additional blocker nor passing evidence. No Close, force push, destructive rebase, manual rerun, synthetic status/approval, merge, auto-merge, or bypass was performed. |
Summary
main: compile the hot hash, package-name, clause, and sentence splits once, and collapse receipt-tool whitespace withstr.split/str.join.85dc0f3aaon that branch deletes queue-health, Strix evidence binding, CodeQL identity, and other control-plane files that currentmainstill owns. This branch does not contain that commit.anyio==4.14.2and the Noema document hash installs are already onmain, so those later bolt commits are not repeated here.Tests
test_requirement_splits_are_compiled_once_and_keep_pin_behaviorlocks the compiled hash delimiter and package-name pattern, including a mixed-whitespace hash line.test_runtime_boundaries_are_compiled_once_and_slugs_stay_stablelocks native tool slugs, case-insensitive clause splits, and sentence splits.177 passed). The two touched modules' changed lines are covered; a subset run still leaves pre-existing uncovered lines inmaterialize_base_python_requirements.pythat this diff does not touch.Developer experience
Reviewers can see the intended regex change without a 12k-line control-plane deletion in the same pull request.
User experience
Hash-pin detection, VCS import names, and browser-receipt tool slugs stay the same. The hot paths no longer recompile those patterns on each call.