Restore SKILL.md formatting and disable model invocation - #55
Conversation
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 88f8b01397
ℹ️ About Codex in GitHub
Your team has set up Codex to 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 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| tuning/sizing (temporal-workertuning).' | ||
| description: 'Administer and diagnose running Temporal Cloud or self-hosted Temporal Server environments via CLI (temporal, tcld) — not SDK code. Operations: namespace CRUD, Cloud capacity/APS, API-key rotation, mTLS cert rotation, workflow health, batch cancel/terminate/reset, export, search attributes, Ops API, billing, audit logs, Terraform, SAML/SCIM, migration. Diagnosis: bottom-up triage of stuck workflows, non-determinism, worker-health, task-queue problems, HA failover, payload size limits, performance bottlenecks, missed schedules. Do NOT trigger for generic TLS/gRPC errors unrelated to Temporal, writing application code (temporal-developer), or worker tuning/sizing (temporal-workertuning).' | ||
| version: 0.2.1 | ||
| disable-model-invocation: true |
There was a problem hiding this comment.
Preserve cross-agent compatibility when disabling invocation
When this shared skill is installed for Codex—one of the coding agents the README says is supported—this Claude-specific field does not disable automatic selection: the current Codex skill contract uses only name and description for triggering, and its plugin validator explicitly rejects disable-model-invocation: true. Consequently the safety behavior is absent for direct Codex installs and Codex plugin packaging cannot accept the manifest; implement the restriction in agent-specific packaging rather than adding it to the shared SKILL.md.
Useful? React with 👍 / 👎.
Summary
disable-model-invocation: trueto the frontmatterValidation