feat: add MuAPI media generation skill - #1257
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 20ecba8966
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
| name: muapi-media | ||
| description: "Generate images and videos with MuAPI's schema-driven asynchronous media API while protecting keys, polling, and output downloads." | ||
| category: media | ||
| risk: safe |
There was a problem hiding this comment.
Classify billable state-changing generation as critical
Users installing only --risk safe,none would receive this skill even though it creates billable remote generation jobs and writes downloaded media. The repository's classification in docs/contributors/quality-bar.md reserves safe for read-only/non-destructive commands and assigns state-modifying operations to critical; explicit confirmation reduces execution risk but does not make the capability read-only, so this should be labeled risk: critical.
AGENTS.md reference: AGENTS.md:L52-L54
Useful? React with 👍 / 👎.
| curl --fail --silent --show-error \ | ||
| --request POST \ | ||
| "https://api.muapi.ai${model_endpoint}" \ | ||
| --header "x-api-key: $MUAPI_API_KEY" \ |
There was a problem hiding this comment.
Keep the API key out of curl's argument vector
On shared hosts or systems that collect process command lines, expanding $MUAPI_API_KEY directly into --header exposes the billable credential through curl's argv for every POST and polling request (the same pattern recurs at line 171). curl --help all documents --header <header/@file> and --config <file>; pass a protected header source or a config through stdin instead so the key remains out of process listings.
AGENTS.md reference: AGENTS.md:L52-L54
Useful? React with 👍 / 👎.
Summary
Adds
skills/muapi-media/SKILL.md, an instruction-only skill for safe MuAPI image and video generation workflows.The skill covers:
Change Classification
Quality Bar Checklist ✅
SKILL.mdfrontmatter passesnpm run validate.risk: safe.When to Usesection is explicit and task-oriented.## Limitationssection.npm run security:docs.skill-reviewcheck and address actionable feedback.npm run validate:referencespassed.skills/muapi-media/SKILL.md; generated registry artifacts are omitted.Validation
npm run validate— passed.npm run validate:references— passed.npm run security:docs— passed with no findings.npm run pr:preflight -- --no-run --json— source-only, no derived changes.npm run pr:evidence -- --base origin/main --head HEAD— one added skill, zero audit warnings/errors.The repository's CI refreshes generated indexes before its full test contract; this contribution intentionally contains no generated files.
Additional Resources