fix: add Call-E marketplace icon#125
Conversation
|
Warning You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again! |
| "Use CALL-E to plan a phone call.", | ||
| "Use CALL-E to check a call run status." | ||
| ], | ||
| "composerIcon": "./assets/CALL-E-Icon-Black.svg", |
There was a problem hiding this comment.
WARNING: composerIcon path is incorrect relative to plugin.json location
The file ./assets/CALL-E-Icon-Black.svg resolves to .codex-plugin/assets/CALL-E-Icon-Black.svg, but the actual SVG lives at call-e-integrations/assets/CALL-E-Icon-Black.svg — one directory above .codex-plugin/. This means the icon will not be found by consumers that resolve the path strictly.
The correct relative path is ../assets/CALL-E-Icon-Black.svg (go up one level from .codex-plugin/, which points to call-e-integrations/).
Note: the marketplace icon path (./plugins/CALLE-AI/call-e-integrations/assets/CALL-E-Icon-Black.svg) is consistent with where the SVG actually lives, so that entry is fine.
Code Review SummaryStatus: 1 Issue Found | Recommendation: Address before merge Overview
Issue Details (click to expand)WARNING
Other Observations (not in diff)No issues found in unchanged code. Files Reviewed (3 files)
Reviewed by step-3.5-flash · 180,084 tokens |
Summary
Validation
Related: CALLE-AI/call-e-integrations#35