Repository navigation
fix(bot): strip Brazilian version qualifiers from track titles for dedup - #584
Conversation
|
Warning Rate limit exceeded
Your organization is not enrolled in usage-based pricing. Contact your admin to enable usage-based pricing to continue reviews beyond the rate limit, or try again in 28 minutes and 26 seconds. ⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (2)
✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|



Problem
Autoplay was playing different versions of the same song — e.g. the original "Halo" followed by "Halo (Versão Forró)" or "Halo - Versão Forró". These have the same song core but the dedup system treated them as distinct tracks.
Root Cause
cleanTitledidn't strip Brazilian/Portuguese version descriptors, so:"Halo (Versão Forró)"→ normalized to"haloversoforró"(≠"halo")"Halo - Versão Forró"→ normalized to"haloversoforró"(≠"halo")"Evidências - Ao Vivo"→ normalized to"evidenciasaovivo"(≠"evidencias")Fix
Two additions to
searchQueryCleaner.ts:NOISE_PATTERNS — strip parenthetical forms:
(Versão Forró),(Versão Acústica),[Versão ...](Ao Vivo),[Ao Vivo]HYPHENATED_VERSION_SUFFIXES — strip hyphenated forms:
Song - Versão Forró→SongSong - Ao Vivo em São Paulo→SongSong - Forró→SongThe
"Beyoncé - Halo - Versão Forró"(artist-prefix) case was already handled byextractSongCore's inner-sep trim — no change needed there.Tests
8 new
cleanTitletests covering each new pattern. All 2149 bot tests pass.Builds on #583 (merged).