Skip to content

⚡ Bolt: Optimize file extension checks by preventing string allocation o (salvages #170)#196

Open
abhimehro wants to merge 1 commit into
mainfrom
cursor-agent/salvage-seatek-170-bolt-optimize-file-extensio-20260519-seatek-batch1
Open

⚡ Bolt: Optimize file extension checks by preventing string allocation o (salvages #170)#196
abhimehro wants to merge 1 commit into
mainfrom
cursor-agent/salvage-seatek-170-bolt-optimize-file-extensio-20260519-seatek-batch1

Conversation

@abhimehro
Copy link
Copy Markdown
Owner

Salvage of #170

Rebased on main (Seatek batch 1).

Salvages: #170

…vages #170)

Refs: #170

LiveReview Pre-Commit Check: skipped (iter:1, coverage:0%)
@trunk-io
Copy link
Copy Markdown

trunk-io Bot commented May 19, 2026

Merging to main in this repository is managed by Trunk.

  • To merge this pull request, check the box to the left or comment /trunk merge below.

After your PR is submitted to the merge queue, this comment will be automatically updated with its status. If the PR fails, failure details will also be posted here

@cursor
Copy link
Copy Markdown

cursor Bot commented May 19, 2026

PR Summary

Low Risk
Low risk performance tweak that only affects language detection by file extension; coverage is extended with mixed-case tests to guard behavior.

Overview
Improves get_language to avoid per-call string allocations by checking file extensions via .endswith() with mixed-case tuples (instead of normalizing via .lower()), keeping language detection fast.

Adds a small mixed-case extension test in tests/test_code_health_scanner.py and documents the optimization in .jules/bolt.md.

Reviewed by Cursor Bugbot for commit c91d5c1. Configure here.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@abhimehro abhimehro marked this pull request as ready for review May 19, 2026 12:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants