Draw the OpenCode mark on the taskbar strip - #290
Conversation
Deploying with
|
| Status | Name | Latest Commit | Preview URL | Updated (UTC) |
|---|---|---|---|---|
| ✅ Deployment successful! View logs |
ceiling | b5d92e8 | Commit Preview URL Branch Preview URL |
Aug 14 2026, 05:37 AM |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughThe taskbar widget adds an OpenCode icon mask and maps both ChangesOpenCode taskbar rendering
Estimated code review effort: 1 (Trivial) | ~2 minutes Mergeability Score: ⚪ Minimal · up to This localized taskbar icon change is merge-ready after normal checks and review; no actionable merge-blocking risk remains. Suggested reviewers: 🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✨ Finishing Touches 💡 1📝 Generate docstrings 💡
🧪 Generate unit tests (beta)
Comment |
Automated reviewFound 1 issue:
For coding agents: fix BLOCK and FIX IF QUICK findings now; everything else is tracked or informational; never exceed one CodeRev fix round per PR. Advisory. Findings generated by |
| "grok" => Some(&GROK), | ||
| "gemini" => Some(&GEMINI), | ||
| "antigravity" | "agy" => Some(&ANTIGRAVITY), | ||
| "opencode" | "opencodego" => Some(&OPENCODE), |
There was a problem hiding this comment.
No failing test covers this change · disposition: fix-if-quick · confidence: high · severity: medium · quick win
Before this diff, provider ids opencode and opencodego fell through draw_provider_icon to the unknown-provider solid disc and through provider_color to rgb(204, 211, 220). After it they take the new OPENCODE 16x16 mask and rgb(59, 130, 246). No test in the diff (and none in this module) asserts either mapping, so deleting the match arms or changing the COLORREF would still pass cargo test.
Prompt for AI agents
Lift the id-to-mask and id-to-color matches out of the GDI paint path (or pub(super) them inside windows_host) and add a table test that opencode and opencodego both resolve to the OPENCODE rows and rgb(59, 130, 246), and that an unknown id does not. Verify against the current code first; if no longer valid, skip with a brief reason. Keep the change minimal.
CodeRev · advisory
Summary: the taskbar overlay strip renders OpenCode and OpenCode Go as a blank light-gray disc because draw_provider_icon had no bitmap for them and fell through to the solid-disc fallback. Adds a 16x16 square-ring OPENCODE mask and a provider_color entry matching the web registry. Tests: cargo test (taskbar_widget, 54 passed), cargo fmt --check, cargo clippy --all-targets -- -D warnings.
Note
Draw the OpenCode mark on the Windows taskbar strip
Adds OpenCode-specific rendering to the Windows taskbar widget in taskbar_widget.rs.
opencodeandopencodegoto this mask and to the colorrgb(59, 130, 246), matching the web registry color.Macroscope summarized b5d92e8.
Summary by CodeRabbit