⚡ Bolt: Fast parameter check array allocation

[AhmmedSamier]

💡 What: Replaced .map() with a pre-allocated array and manual for loop in RouteMatcher.precompute.
🎯 Why: .map() incurs function call and iterator creation overhead in V8/Bun. Pre-allocating the array using new Array(length) avoids this overhead entirely.
📊 Impact: Faster execution in RouteMatcher.precompute path and overall burstSearch benchmarks.
🔬 Measurement: Ran bun test src/core/route-matcher.test.ts to verify correctness and bun run benchmarks/run-route-matcher.ts to confirm performance improvements.

---

PR created automatically by Jules for task 137113116280351323 started by @AhmmedSamier

Summary by CodeRabbit

• Performance

  • Improved route matching responsiveness through internal optimizations.

• Documentation

  • Added learning material on performance optimization techniques for array operations.

[google-labs-jules]

👋 Jules, reporting for duty! I'm here to lend a hand with this pull request.

When you start a review, I'll add a 👀 emoji to each comment to let you know I've read it. I'll focus on feedback directed at me and will do my best to stay out of conversations between you and other bots or reviewers to keep the noise down.

I'll push a commit with your requested changes shortly after. Please note there might be a delay between these steps, but rest assured I'm on the job!

For more direct control, you can switch me to Reactive Mode. When this mode is on, I will only act on comments where you specifically mention me with @jules. You can find this option in the Pull Request section of your global Jules UI settings. You can always switch back!

New to Jules? Learn more at jules.google/docs.

---

For security, I will only act on instructions from the user who triggered this task.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: d347bcd3-3da6-44dc-8e6e-966025fec644

📥 Commits

Reviewing files that changed from the base of the PR and between d33be42 and f920497.

📒 Files selected for processing (4)

• .jules/bolt.md
• language-server/benchmarks/benchmarks.json
• language-server/benchmarks/run-route-matcher.ts
• language-server/src/core/route-matcher.ts

---

📝 Walkthrough

Walkthrough

This pull request optimizes array allocation in the route-matching hot path by replacing Array.prototype.map() with pre-allocated arrays and manual for loops. Includes documentation of the optimization approach, updated benchmark metrics reflecting the performance changes, and a new benchmark runner file for the route-matcher.

Changes

Cohort / File(s)	Summary
Documentation & Knowledge .jules/bolt.md	Adds new entry documenting the optimization strategy: replacing Array.prototype.map() with pre-allocated arrays and manual loops to reduce allocations in hot paths.
Benchmark Infrastructure language-server/benchmarks/run-route-matcher.ts, language-server/benchmarks/benchmarks.json	Introduces new benchmark runner file and updates benchmark metrics with recalculated timing values across multiple test cases, including new concurrency scenarios and re-measured statistics.
Route Matcher Optimization language-server/src/core/route-matcher.ts	Replaces Array.prototype.map() with pre-allocated boolean array and manual for loop to compute isParameter without function call overhead, maintaining identical observable behavior.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~25 minutes

Possibly related PRs

• PR #254: Modifies parameter-detection logic in language-server/src/core/route-matcher.ts, replacing string checks with alternative implementations as part of route-matching optimization efforts.
• PR #268: Updates segment processing in language-server/src/core/route-matcher.ts to optimize operations within the route-matching hot path.

Poem

🐰 A loop replaces the map with care,
Pre-allocated arrays float through air,
No allocations in the fast lane—
Benchmarks sing a speedier refrain! ✨

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title uses an emoji and refers to 'Bolt' optimization, and while it's related to the actual changes (improving array allocation in route matching), it's somewhat vague and includes noise (the emoji ⚡). The specific reference to 'Fast parameter check array allocation' does match the core change, but the emoji and 'Bolt' framing reduce clarity.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

• 📝 Generate docstrings (stacked PR)
• 📝 Generate docstrings (commit on current branch)

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch bolt-fast-parameter-check-137113116280351323

📝 Coding Plan

• Generate coding plan for human review comments

---

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