[codex] Add platform icons and alias resolution#1
Conversation
|
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Pro Run ID: ⛔ Files ignored due to path filters (2)
📒 Files selected for processing (8)
📝 WalkthroughWalkthroughThis PR adds alias-aware platform ID resolution to the icons library. Platform data and types are extended to support aliases, normalization logic resolves aliases and slugs to canonical IDs, icon lookups are updated to use resolved IDs, viewBox extraction gains logging, and comprehensive tests validate the complete flow. ChangesPlatform Alias Resolution and Icon Lookup Enhancement
Sequence Diagram(s)N/A — the changes do not meet the conditions for a sequence diagram (primarily data structure updates, lookup optimization, and supporting logic without new multi-component workflows). Estimated code review effort🎯 3 (Moderate) | ⏱️ ~25 minutes Poem
✨ Finishing Touches📝 Generate docstrings
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 |
122874b to
0d658f0
Compare
Icon Preview
|
| Before | After | |
|---|---|---|
| Icon | (new) | |
| Badge (light) | (new) | |
| Badge (dark) | (new) |
netflix (added)
| Before | After | |
|---|---|---|
| Icon | (new) | |
| Badge (light) | (new) | |
| Badge (dark) | (new) |
Summary
Validation
Summary by CodeRabbit
New Features
resolvePlatformId()function for flexible platform ID lookup with normalization.hasPlatformIcon()function to check icon availability.Bug Fixes
Tests
Documentation