⚡ Precompile regex in update-bottles to improve performance

[Serendeep]

💡 What: The optimization implemented
Extracted the regex r'\[bottle\.[^\]]+\][ \t]*\nurl[ \t]*=[ \t]*"[^"]*"[ \t]*\nsha256[ \t]*=[ \t]*"[^"]*"[ \t]*\n' to a module-level constant named BOTTLE_SECTION_PATTERN compiled with re.compile(). We then use BOTTLE_SECTION_PATTERN.finditer(content) instead of re.finditer(..., content).

🎯 Why: The performance problem it solves
In scripts/update-bottles.py, the update_bottle_section function had a static regex inside a call to re.finditer(...). This regex was being repeatedly compiled every time the function was called. Pre-compiling it avoids that overhead.

📊 Measured Improvement:
I created a benchmark.py using timeit to run the loop logic 100,000 times over a dummy formula string.
Results:

• Unoptimized time: 0.582422s
• Optimized time: 0.398350s
• Improvement: 31.60%

---

PR created automatically by Jules for task 3595482695959040259 started by @Serendeep

[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.

[Copilot]

Pull request overview

Optimizes scripts/update-bottles.py by precompiling the static “bottle section” regex so repeated calls avoid re-compilation overhead, and adds a standalone benchmark script used to measure the improvement.

Changes:

• Precompile the bottle-section regex as a module-level BOTTLE_SECTION_PATTERN and reuse it via .finditer(...).
• Replace the inline re.finditer(...) call in update_bottle_section with the precompiled pattern.
• Add a benchmark.py script to measure the optimization impact.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File	Description
scripts/update-bottles.py	Extracts and precompiles the bottle-section regex for reuse during section scanning.
benchmark.py	Adds a local timeit benchmark to compare unoptimized vs optimized regex iteration.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

import re at module top is unused in this benchmark script (the regex module is only imported inside the timeit setup strings). Please remove the unused import to avoid lint failures and reduce confusion about what the benchmark depends on.

Suggested change

import re

[Copilot]

This PR adds benchmark.py at the repository root, but it isn't referenced by docs, CI, or the scripts/ tooling. Consider moving it under a dedicated location (e.g., scripts/bench/ or tools/) or excluding it from the PR to avoid leaving an orphan utility at the top level.