ci: add Codex CLI plugin manifest#27109
ci: add Codex CLI plugin manifest#27109internet-dot wants to merge 1 commit intoopen-metadata:mainfrom
Conversation
|
Hi there 👋 Thanks for your contribution! The OpenMetadata team will review the PR shortly! Once it has been labeled as Let us know if you need any help! |
| "mcp", | ||
| "codex" | ||
| ], | ||
| "mcpServers": "./.mcp.json" |
There was a problem hiding this comment.
⚠️ Bug: plugin.json references non-existent .mcp.json file
The mcpServers field in plugin.json (line 15) points to ./.mcp.json, but this file does not exist anywhere in the repository. The Codex plugin scanner CI job will either fail or produce misleading results when it tries to resolve this reference.
Either add the .mcp.json file to the repository, or remove the mcpServers field from plugin.json until MCP server configuration is ready.
Was this helpful? React with 👍 / 👎 | Reply gitar fix to apply this suggestion
| { | ||
| "name": "OpenMetadata", | ||
| "version": "0.1.0", | ||
| "description": "OpenMetadata is a unified metadata platform for data discovery, data observability, and data governance powered by a central metadata repository, in-depth column level lineage, and seamless team colla", |
There was a problem hiding this comment.
💡 Quality: plugin.json description is truncated mid-word
The description field (line 4) ends with "...seamless team colla" — it appears to be cut off mid-word (should be "collaboration"). This truncated text will appear in any plugin registry or listing that consumes this manifest.
Suggested fix:
"description": "OpenMetadata is a unified metadata platform for data discovery, data observability, and data governance powered by a central metadata repository, in-depth column level lineage, and seamless team collaboration."
Was this helpful? React with 👍 / 👎 | Reply gitar fix to apply this suggestion
| "codex" | ||
| ], | ||
| "mcpServers": "./.mcp.json" | ||
| } No newline at end of file |
There was a problem hiding this comment.
💡 Quality: plugin.json missing trailing newline
The file ends without a newline character (the diff shows \ No newline at end of file). Most linting tools and POSIX conventions expect a trailing newline.
Was this helpful? React with 👍 / 👎 | Reply gitar fix to apply this suggestion
Code Review
|
| Compact |
|
Was this helpful? React with 👍 / 👎 | Gitar
This adds a small metadata pass so the repo is easier to wire into Codex without touching runtime code.
I targeted this repo because it already exposes:
It adds
.codex-plugin/plugin.json, a starter.mcp.json, and the scanner workflow.Permissions: read-only. No secrets. No publish. No runtime changes.
If you want different command wiring or metadata values, I can adjust the branch.