Skip to content

perf(onboarding): debounce la recherche de technologies (300ms)#161

Merged
KxlSys merged 1 commit into
mainfrom
feat/onboarding-debounce
Jun 15, 2026
Merged

perf(onboarding): debounce la recherche de technologies (300ms)#161
KxlSys merged 1 commit into
mainfrom
feat/onboarding-debounce

Conversation

@KxlSys

@KxlSys KxlSys commented Jun 15, 2026

Copy link
Copy Markdown
Owner

Reprend proprement la PR #118 (Bolt/Jules) sur main à jour, sans réintroduire le doublon de casse .jules/.Jules.

Découple l'état du champ (localTechSearch, instantané) de l'état qui déclenche le filtrage coûteux de TECH_OPTIONS (techSearch), synchronisés via un useEffect debounce de 300ms. Supprime le lag de saisie.

Note de doc ajoutée dans .Jules/bolt.md (casse canonique). Le bump react-router 6.30.4 de la PR est déjà présent sur main.

Summary by CodeRabbit

Release Notes

  • Performance
    • Enhanced technology search responsiveness during the onboarding process.

Reprend proprement la PR #118 (Bolt/Jules) sur main à jour, sans
réintroduire le doublon de casse .jules/.Jules.

Découple l'état du champ (localTechSearch, instantané) de l'état qui
déclenche le filtrage coûteux de TECH_OPTIONS (techSearch), synchronisés
via un useEffect debounce de 300ms. Supprime le lag de saisie.

Note de doc ajoutée dans .Jules/bolt.md (casse canonique). Le bump
react-router 6.30.4 de la PR est déjà présent sur main.

Co-authored-by: google-labs-jules[bot] <161369871+google-labs-jules[bot]@users.noreply.github.com>
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, you can upgrade your account or add credits to your account and enable them for code reviews in your settings.

@vercel

vercel Bot commented Jun 15, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
bisomaptech Ready Ready Preview, Comment Jun 15, 2026 1:02pm

@coderabbitai

coderabbitai Bot commented Jun 15, 2026

Copy link
Copy Markdown

Review Change Stack

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 9857ef19-5af1-4a9e-9eee-78d2b801f395

📥 Commits

Reviewing files that changed from the base of the PR and between 945845a and b8d969f.

📒 Files selected for processing (2)
  • .Jules/bolt.md
  • src/components/onboarding/onboarding-stepper.tsx

📝 Walkthrough

Walkthrough

OnboardingStepper gains a localTechSearch state that captures every keystroke immediately, while a 300ms debounced useEffect propagates the value into the existing techSearch state used for filtering. The technologies search <Input> is rebound to localTechSearch. The .Jules/bolt.md journal receives a matching entry.

Changes

Debounced Tech Search in OnboardingStepper

Layer / File(s) Summary
Debounce state, effect, and Input rewiring
src/components/onboarding/onboarding-stepper.tsx, .Jules/bolt.md
Adds localTechSearch state and a useEffect that syncs it into techSearch after 300ms (with timeout cleanup on each edit and unmount). The technologies search <Input> value and onChange are rebound to localTechSearch. Journal entry documents the pattern.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

  • KxlSys/BisoMapTech#108: Applies the same localSearch + 300ms useEffect debounce pattern to AdminPage/ContributorsPage search inputs.
  • KxlSys/BisoMapTech#156: Introduces the same debounced input isolation pattern to prevent per-keystroke filter updates across other components.
  • KxlSys/BisoMapTech#104: Implements the identical localSearchsearch 300ms debounce pattern in PlacesPage, also with a .Jules/bolt.md journal entry.

Poem

🐇 A rabbit types fast, but the list shouldn't chase,
So I split the two states and gave filtering space.
Three hundred milliseconds, a gentle delay,
localTechSearch leads, techSearch follows away.
Hop hop, fewer renders — a tidier day! 🌿

✨ 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 feat/onboarding-debounce

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

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

@KxlSys KxlSys merged commit a2a7cb8 into main Jun 15, 2026
5 of 7 checks passed
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.

1 participant