⚡ Bolt: [performance improvement] avoid intermediate arrays in parseSearchPeople

[bartholomej]

💡 What: Replace Array.from().find() with .find() on node-html-parser arrays, and map directly over querySelectorAll result.
🎯 Why: Avoids unnecessary allocation of intermediate arrays when searching for creator paragraphs and parsing person links.
📊 Impact: Lowers memory overhead and slightly improves execution speed.
🔬 Measurement: Run bun test to confirm.

---

PR created automatically by Jules for task 3772258637480593546 started by @bartholomej

Summary by CodeRabbit

• Refactor
  • Improved internal code handling for search functionality to enhance robustness.

[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: e6fb3d02-53b3-4713-9662-28c25bb0ce59

📥 Commits

Reviewing files that changed from the base of the PR and between bebd16c and 12d9852.

📒 Files selected for processing (1)

• src/helpers/search.helper.ts

---

📝 Walkthrough

Walkthrough

The parseSearchPeople helper function is refactored to use optional chaining syntax (?.) instead of traditional conditional checks, and removes explicit Array.from conversions when handling DOM node lists.

Changes

Cohort / File(s)	Summary
Search Helper Refactor src/helpers/search.helper.ts	Updated parseSearchPeople to use optional chaining (el?.querySelectorAll(...)?....find(...)) for safer null/undefined handling, and removed Array.from conversion when deriving the people node list from querySelectorAll.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Poem

🐰 Optional chains now grace our code,
No more Array.from down the road,
?. safely hops through DOM with care,
Where falsy values vanish in the air,
Cleaner syntax, simpler to declare! 🌟

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name	Status	Explanation	Resolution
Description check	❓ Inconclusive	The description provides clear what/why/impact context but lacks completion of the required template sections like Type of change, Related Issues, and Checklist items.	Complete the description template by selecting a Type of change (Refactoring appears most appropriate), adding Related Issues if applicable, and checking the Checklist items to confirm self-review and testing.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title accurately describes the main change: avoiding intermediate arrays in parseSearchPeople through optimization, matching the core refactoring in the PR.
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

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch bolt-search-helper-optimization-3772258637480593546

---

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 and usage tips.}

[codecov-commenter]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 98.80%. Comparing base (bebd16c) to head (12d9852).

Additional details and impacted files

@@           Coverage Diff           @@
##           master     #167   +/-   ##
=======================================
  Coverage   98.80%   98.80%           
=======================================
  Files          34       34           
  Lines         755      755           
  Branches      191      190    -1     
=======================================
  Hits          746      746           
  Misses          9        9           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.