feat: icon update from designer

[evan-taylor]

Linked Issues

Closes #
Linear: (e.g., POLY-123)

Summary

Briefly explain the change and why.

How to Test

Steps to verify locally:

• npm run lint
• npm run typecheck
• npm test
• Manual flow:
  1. npm run dev:backend (in terminal A)
  2. npm run dev (in terminal B)
  3. Verify the change: <describe expected behavior/screens>

Checklist

• Tests added/updated (if applicable)
• Lint/tests pass locally (npm run lint)
• Docs updated (README/ADR/changelog if needed)
• Follows conventional commit format
• No merge conflicts with dev

Screenshots / Demos

(if UI or visible behavior - attach images, videos, or GIFs)

Summary by CodeRabbit

• Chores
  • Updated icon configuration for improved rendering and platform support compatibility.

[vercel]

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

Project	Deployment	Actions	Updated (UTC)
poly-buys	Ready	Preview, Comment	Apr 23, 2026 6:00pm

[coderabbitai]

📝 Walkthrough

Walkthrough

The icon configuration JSON is updated to modify fill color properties, reassign the image layer to a different SVG asset with adjusted positioning, remove rendering properties, and update the supported platforms declaration from iOS-specific to shared.

Changes

Cohort / File(s)	Summary
Icon Configuration frontend/assets/polybuys-ios.icon/icon.json	Replaced automatic fill/color-space defaults with explicit display-p3 RGBA fill. Renamed and repointed image layer to SVG Image 2.svg with updated scale and translation coordinates. Removed layer rendering properties (blending, glass, visibility, opacity). Changed platform support from iOS array to "shared" value.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

• #112: Modifies the same icon metadata file with overlapping changes to SVG layer references, color-space/fill handling, and platform declarations.

Poem

🐰✨ With whiskers twitching and a hop-skip dance,
I spot the icon, dressed in a new stance!
Colors shifted from auto to display-p3 bright,
SVG layers leap to their rightful height—
Shared across platforms, rendering crisp and right! 🎨

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Description check	⚠️ Warning	The description contains the template structure but lacks concrete details: linked issues, summary, and testing steps are incomplete placeholders, and checklist items remain unchecked.	Fill in actual issue references, provide a detailed summary of icon changes, complete testing verification steps, and check off completed checklist items before merging.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title 'feat: icon update from designer' is specific and clearly identifies the main change—an icon configuration update sourced from a designer.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch evan/icon-update

---

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.}

[coderabbitai]

🧹 Nitpick comments (1)

frontend/assets/polybuys-ios.icon/icon.json (1)

9-10: Consider renaming the generic designer export.

SVG Image 2.svg / SVG Image 2 looks like an un-renamed default export name from the design tool. Renaming to something more descriptive (e.g., polybuys-glyph.svg) would improve maintainability and make future asset swaps clearer in diffs. The asset is present in the bundle; this is a refactoring suggestion for better code clarity.

🤖 Prompt for AI Agents

Verify each finding against the current code and only fix it if needed.

In `@frontend/assets/polybuys-ios.icon/icon.json` around lines 9 - 10, The icon
asset uses a generic designer export name ("image-name": "SVG Image 2.svg",
"name": "SVG Image 2"); rename both fields to a descriptive identifier (e.g.,
"polybuys-glyph.svg" and "polybuys-glyph") and update the actual SVG filename in
the bundle and any code references that load this asset (search for "SVG Image
2" and update imports/usages, including any asset manifests or import paths) so
the JSON, file, and all references stay consistent.

🤖 Prompt for all review comments with AI agents

Verify each finding against the current code and only fix it if needed.

Nitpick comments:
In `@frontend/assets/polybuys-ios.icon/icon.json`:
- Around line 9-10: The icon asset uses a generic designer export name
("image-name": "SVG Image 2.svg", "name": "SVG Image 2"); rename both fields to
a descriptive identifier (e.g., "polybuys-glyph.svg" and "polybuys-glyph") and
update the actual SVG filename in the bundle and any code references that load
this asset (search for "SVG Image 2" and update imports/usages, including any
asset manifests or import paths) so the JSON, file, and all references stay
consistent.

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 5219cd48-49e6-4f04-937b-b16c9c2b99fc

📥 Commits

Reviewing files that changed from the base of the PR and between 74dfab7 and a5077ed.

⛔ Files ignored due to path filters (2)

• frontend/assets/polybuys-ios.icon/Assets/SVG Image 2.svg is excluded by !**/*.svg
• frontend/assets/polybuys-ios.icon/Assets/SVG Image.svg is excluded by !**/*.svg

📒 Files selected for processing (1)

• frontend/assets/polybuys-ios.icon/icon.json