⚡ Bolt: Optimize yEnc decoding

[xbmc4lyfe]

💡 What: Optimized the _decode_yenc_lines function in verify_nzb.py to use bytes.translate() and bytes.find() instead of a manual while loop that iterates over each byte. Added .jules/bolt.md to document the finding.
🎯 Why: Manual byte-by-byte iteration in pure Python is extremely slow and acts as a major bottleneck when decoding large amounts of NNTP body data.
📊 Impact: Reduces decoding time by approximately 15x (from ~2.27s down to ~0.14s for 1000 lines decoding 100 times in an isolated benchmark).
🔬 Measurement: You can verify the correctness of the decoding algorithm by running the test suite (python3 -m unittest -v), which includes tests for validate_yenc_body.

---

PR created automatically by Jules for task 17733910853938855992 started by @xbmc4lyfe

[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: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: ef31c7b3-efb4-4c23-ad81-a43aafd5e22b

📥 Commits

Reviewing files that changed from the base of the PR and between 0de7ede and 2f8780f.

📒 Files selected for processing (1)

• verify_nzb.py

📜 Recent review details

🔇 Additional comments (1)

verify_nzb.py (1)

118-119: LGTM!

Also applies to: 122-147

---

📝 Walkthrough

Summary by CodeRabbit

• Refactor
  • Optimized yEnc decoding performance with improved internal processing.

Walkthrough

The PR optimizes yEnc decoding in verify_nzb.py by replacing per-byte iteration with precomputed lookup tables and bulk translation. Module-level tables map normal and escaped yEnc bytes to their decoded values, while _decode_yenc_lines() now uses bytes.translate() for segments and find() for escape handling, preserving the existing error contract.

Changes

yEnc Decoding Optimization

Layer / File(s)	Summary
yEnc lookup tables and optimized decode function verify_nzb.py	Introduces _YENC_NORMAL and _YENC_ESCAPED lookup tables at module level. Rewrites _decode_yenc_lines() to use bytes.translate() for bulk decoding on escape-free segments and find()-based indexing for escaped = pairs, eliminating the previous byte-by-byte loop while preserving dangling-escape validation.

🎯 2 (Simple) | ⏱️ ~12 minutes

🐰 A swift translation hops in,
Lookup tables skip each byte,
Decoded lines leap faster—
yEnc escapes met with cheer,
Performance springs up here!

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title accurately summarizes the main change: optimizing yEnc decoding performance in the codebase.
Description check	✅ Passed	The description is directly related to the changeset, explaining the optimization, methodology, performance impact, and verification approach.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

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

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch bolt-optimize-yenc-decode-17733910853938855992

✨ Simplify code

• Create PR with simplified code
• Commit simplified code in branch bolt-optimize-yenc-decode-17733910853938855992

---

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