Skip to content

fix: add Call-E marketplace icon#125

Merged
internet-dot merged 1 commit into
hashgraph-online:mainfrom
Ray-56:codex/add-call-e-icon
May 18, 2026
Merged

fix: add Call-E marketplace icon#125
internet-dot merged 1 commit into
hashgraph-online:mainfrom
Ray-56:codex/add-call-e-icon

Conversation

@Ray-56
Copy link
Copy Markdown
Contributor

@Ray-56 Ray-56 commented May 18, 2026

Summary

  • add the Call-E marketplace icon field
  • sync the mirrored Call-E plugin manifest with its composerIcon
  • include the Call-E SVG icon asset under the mirrored plugin bundle

Validation

  • python3 -m json.tool .agents/plugins/marketplace.json
  • python3 -m json.tool plugins/CALLE-AI/call-e-integrations/.codex-plugin/plugin.json
  • xmllint --noout plugins/CALLE-AI/call-e-integrations/assets/CALL-E-Icon-Black.svg
  • git diff --check
  • scoped generator assertion for the Call-E marketplace icon path

Related: CALLE-AI/call-e-integrations#35

@gemini-code-assist
Copy link
Copy Markdown
Contributor

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",
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.

@kilo-code-bot
Copy link
Copy Markdown

kilo-code-bot Bot commented May 18, 2026

Code Review Summary

Status: 1 Issue Found | Recommendation: Address before merge

Overview

Severity Count
WARNING 1
Issue Details (click to expand)

WARNING

File Line Issue
plugins/CALLE-AI/call-e-integrations/.codex-plugin/plugin.json 41 composerIcon path ./assets/CALL-E-Icon-Black.svg is wrong relative to .codex-plugin/plugin.json — should be ../assets/CALL-E-Icon-Black.svg because the SVG lives one directory above .codex-plugin/
Other Observations (not in diff)

No issues found in unchanged code.

Files Reviewed (3 files)
  • .agents/plugins/marketplace.json — no issues
  • plugins/CALLE-AI/call-e-integrations/.codex-plugin/plugin.json — 1 issue (incorrect composerIcon path)
  • plugins/CALLE-AI/call-e-integrations/assets/CALL-E-Icon-Black.svg — no issues

Reviewed by step-3.5-flash · 180,084 tokens

@internet-dot internet-dot merged commit 7821924 into hashgraph-online:main May 18, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants