Conversation
wacore/whatsapp-rust#865 replaced HashSet with Vec linear scan for index_mac dedup in the appstate patch path, reducing wacore_appstate .text by -1.1 KiB (-3.03%) with no behavior change. https://claude.ai/code/session_015k6rnJKadcTFdVwEYxESt8
|
Preview deployment for your docs. Learn more about Mintlify Previews.
💡 Tip: Enable Workflows to automatically generate PRs for you. |
|
Warning Review limit reached
More reviews will be available in 22 minutes and 41 seconds. Learn how PR review limits work. Your organization has used up its prepaid credits, and credit purchases are no longer available. Enable the review add-on in the billing tab to keep reviews running — you're only billed for reviews past your plan's rate limits ($0.25/file). ⌛ How to resolve this issue?After more reviews become available, 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 include higher PR review limits than trial, open-source, and free plans. In all cases, reviews become available again over time. During sustained high-volume PR review activity, CodeRabbit may temporarily slow when the next review becomes available. Please see our Fair Usage Limits Policy for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus 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 |
![cubic-dev-ai[bot]](https://avatars.githubusercontent.com/in/1082092?s=60&v=4){kind=small}
There was a problem hiding this comment.
1 issue found across 2 files
Reply with feedback, questions, or to request a fix.
Fix all with cubic | Re-trigger cubic
2 participants
Summary
Adds a changelog entry for whatsapp-rust#865, which replaced
HashSet<&[u8]>withVec<&[u8]>linear scan forindex_macdedup in two spots in the appstate patch path.changelog/2026-06-14-appstate-scan-dedup.mdxdocs.jsonto register it at the top of the Changelog tabWhy document it
Although purely internal (no API or behavior changes), the project tracks binary size improvements in the changelog — and this refactor drops
wacore_appstate.textby −1.1 KiB (−3.03%) by eliminating the monomorphizedHashSet/SipHash machinery. It follows the same rationale documented forcollect_unique_index_macsin #856.Test plan
docs.jsonentry matches the new filename exactlyhttps://claude.ai/code/session_015k6rnJKadcTFdVwEYxESt8
Generated by Claude Code
Summary by cubic
Add a changelog entry documenting the appstate
index_macdedup refactor from PR #865, replacingHashSet<&[u8]>withVec<&[u8]>linear scans in two patch-path spots and reducingwacore_appstate.textby −1.1 KiB (−3.03%) with no behavior change. Register the page indocs.jsonand correct a minor capitalization typo in the entry.Written for commit eb46974. Summary will update on new commits.