Merge train 183: #10207

[proggeramlug]

Merge train 183: lands #10207 (stop capping replace and split output lists at the scratch limit; part of #10164) at head 7b10d913b6, plus the workspace version bump to 0.5.1556.

Both PR commits were cherry-picked onto 0956673b5 without conflicts. main has not moved since the PR's base, so the tree before the bump is byte-identical to the PR head.

Review: List::push no longer refuses at SCRATCH_BYTES / 8 entries, which capped a callback replace over 'ab12 cd345;' at about 700,000 records with "Regular expression memory limit exceeded". The list is a GC array. List::get indexes with as u32, but js_array_push_f64 at length u32::MAX takes the spec path and throws a RangeError from ArraySetLength, so the list can never silently wrap. Reaching that length would need an array of about 34 GB.

Local validation of this branch (perrymaster, Linux x86_64, --locked):

• cargo metadata --locked, cargo fmt --all -- --check
• cargo check -p perry-runtime --no-default-features --features full --lib (regex feature off): 0 warnings
• cargo check -p perry-runtime --lib --tests: no warnings outside the known global_this_webassembly.rs dead code
• cargo test -p perry-runtime --lib -- --test-threads=1: 3725 passed, 1 failed. The failure is native_stack::tests::stack_top_respects_custom_thread_stack_sizes, red on main. The suite includes the new perex_replace_output_is_not_capped_by_the_scratch_limit.
• cargo clippy -p perry-runtime --lib --tests: the same 12 approx_constant errors, and the warning diff is identical to train 181's
• scripts/run_lint_gates.sh script tier: 76 of 77 pass. The failure is public benchmark evidence freshness, identical on main.
• gc_runtime_root_holders.py passes; check_changeset_fragment.sh PerryTS/perry 10207: pass

https://claude.ai/code/session_01Da12JXeG5XuVBma5yWp5C9

Summary by CodeRabbit

• New Features

  • Increased the maximum output size for regular-expression replace, replaceAll, and split operations by removing the previous fixed entry limit.
  • Large operations are now constrained by available memory and maximum string length rather than the former list cap.

• Bug Fixes

  • Fixed global replacements failing with a memory-limit error when processing very large outputs.

• Chores

  • Updated the application version to 0.5.1556.

[coderabbitai]

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Advanced

Run ID: 7e5590f4-f504-483c-b635-3d29b2845cbc

📥 Commits

Reviewing files that changed from the base of the PR and between 0956673 and b8bd6ce.

⛔ Files ignored due to path filters (1)

• Cargo.lock is excluded by !**/*.lock

📒 Files selected for processing (5)

• CLAUDE.md
• Cargo.toml
• changelog.d/10207-regex-list-cap.md
• crates/perry-runtime/src/gc/tests/runtime_roots/perex_replace.rs
• crates/perry-runtime/src/regex/perex_replace_storage.rs

---

📝 Walkthrough

Walkthrough

The regex runtime no longer applies the fixed scratch-buffer entry cap to operation lists. A regression test verifies large global replacement output. Workspace version documentation updates to 0.5.1556.

Changes

Regex list capacity

Layer / File(s)	Summary
Remove regex list entry cap crates/perry-runtime/src/regex/perex_replace_storage.rs, changelog.d/10207-regex-list-cap.md	List::push no longer rejects entries at the scratch-buffer limit. The changelog documents the updated behavior for replace, replaceAll, and split.
Validate behavior and update version crates/perry-runtime/src/gc/tests/runtime_roots/perex_replace.rs, Cargo.toml, CLAUDE.md	A regression test verifies global replacement beyond the former limit. The workspace and documented versions update to 0.5.1556.

Estimated code review effort: 2 (Simple) | ~10 minutes

Change: Bug fix

✨ Finishing Touches 💡 1

🛠️ Fix failing CI checks 💡

• Create stacked PR
• Commit on current branch

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch train183

---

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.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}