feat(brand): bundle authentic Meta Muse brand artwork for Mac and iOS - #165
Conversation
Replace the preliminary monogram SVG with the authentic Meta Muse blue script 'm' artwork from the brand repository. Add provider-muse asset catalog imageset for iOS companion (1x, 2x, 3x) and update widget and companion model asset mapping.
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
Code Review Completed! 🔥The code review was successfully completed based on your current configurations.
|
| Options | Enabled |
|---|---|
| Bug | ✅ |
| Performance | ✅ |
| Security | ✅ |
| Business Logic | ✅ |
| if low.contains("openai") || low.contains("codex") { return "provider-openai" } | ||
| if low.contains("cursor") { return "provider-cursor" } | ||
| if low.contains("minimax") { return "provider-minimax" } | ||
| if low.contains("muse") { return "provider-muse" } |
There was a problem hiding this comment.
The visible Muse provider mark added to the iOS widget UI by ios/CodeCapsCompanion/Widgets/WidgetViews.swift lacks the required automated simulator screenshot coverage. Add a repeatable test or CI step that launches the companion in an iOS simulator, displays the Muse mark, and runs xcrun simctl io booted screenshot.
Also found in:
ios/CodeCapsCompanion/App/Models/CompanionQuotaModel.swift:296-296
Kody rule violation: Keep credentials out of public source and verify UI changes with automated screenshots
Prompt for LLM
File ios/CodeCapsCompanion/Widgets/WidgetViews.swift:
Line 142:
The visible Muse provider mark added to the iOS widget UI by `ios/CodeCapsCompanion/Widgets/WidgetViews.swift` lacks the required automated simulator screenshot coverage. Add a repeatable test or CI step that launches the companion in an iOS simulator, displays the Muse mark, and runs `xcrun simctl io booted screenshot`.
**Also found in:**
- `ios/CodeCapsCompanion/App/Models/CompanionQuotaModel.swift:296-296`
Talk to Kody by mentioning @kody
Was this suggestion helpful? React with 👍 or 👎 to help Kody learn from this interaction.
| if low.contains("openai") || low.contains("codex") { return "provider-openai" } | ||
| if low.contains("cursor") { return "provider-cursor" } | ||
| if low.contains("minimax") { return "provider-minimax" } | ||
| if low.contains("muse") { return "provider-muse" } |
There was a problem hiding this comment.
Missing screenshot verification leaves the Muse artwork change from the added SwiftUI mapping in ios/CodeCapsCompanion/Widgets/WidgetViews.swift unvalidated. Add a simulator test or CI step that exercises the Muse provider mark and captures it with xcrun simctl io booted screenshot before treating the UI as verified.
Also found in:
ios/CodeCapsCompanion/App/Models/CompanionQuotaModel.swift:296-296
Kody rule violation: Gate iOS/TestFlight releases on a signing-safe workflow and verify UI changes with automated screenshots
Prompt for LLM
File ios/CodeCapsCompanion/Widgets/WidgetViews.swift:
Line 142:
Missing screenshot verification leaves the Muse artwork change from the added SwiftUI mapping in `ios/CodeCapsCompanion/Widgets/WidgetViews.swift` unvalidated. Add a simulator test or CI step that exercises the Muse provider mark and captures it with `xcrun simctl io booted screenshot` before treating the UI as verified.
**Also found in:**
- `ios/CodeCapsCompanion/App/Models/CompanionQuotaModel.swift:296-296`
Talk to Kody by mentioning @kody
Was this suggestion helpful? React with 👍 or 👎 to help Kody learn from this interaction.
| if low.contains("openai") || low.contains("codex") { return "provider-openai" } | ||
| if low.contains("cursor") { return "provider-cursor" } | ||
| if low.contains("minimax") { return "provider-minimax" } | ||
| if low.contains("muse") { return "provider-muse" } |
There was a problem hiding this comment.
ProviderMarkView.body renders every provider mark with .resizable().scaledToFit() inside a fixed size × size frame (WidgetViews.swift:69-74), but the 512×512 provider-muse PNG contains transparent padding that makes its glyph visibly smaller than the tightly cropped SVG marks in 13pt widget rows. Crop the transparent margins from the source PNGs, or inset the Muse mark with .frame(width: size, height: size).padding(size * 0.18) so its visible glyph matches the others.
Prompt for LLM
File ios/CodeCapsCompanion/Widgets/WidgetViews.swift:
Line 142:
`ProviderMarkView.body` renders every provider mark with `.resizable().scaledToFit()` inside a fixed `size × size` frame (WidgetViews.swift:69-74), but the 512×512 `provider-muse` PNG contains transparent padding that makes its glyph visibly smaller than the tightly cropped SVG marks in 13pt widget rows. Crop the transparent margins from the source PNGs, or inset the Muse mark with `.frame(width: size, height: size).padding(size * 0.18)` so its visible glyph matches the others.
Talk to Kody by mentioning @kody
Was this suggestion helpful? React with 👍 or 👎 to help Kody learn from this interaction.
Summary
Review Follow-ups