[codex] Update skills install instructions - #46
Merged
Conversation
mnvsk97
marked this pull request as ready for review
May 19, 2026 19:11
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
There are 2 total unresolved issues (including 1 from previous review).
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 9fe9660. Configure here.
mnvsk97
enabled auto-merge (squash)
May 20, 2026 00:27
rachana-tf
approved these changes
May 20, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.

Summary
npx skills add ... --skill ... --yescommandsonboardandintegrate-gatewaywith the same flag styleMCP Gateway Coverage
mcpServersJSONmcp-server-manager; keep deletes/removals dashboard-onlyValidation
./scripts/validate-skills.sh./scripts/validate-skill-security.shgit diff --check-s,-g,-a,-y, and--allshorthand examplesNotes
tfy applyflows remain gated by summary, YAML/diff, exact command, and explicit user confirmationApply using YAMLpreview remains the source of truth when it differsNote
Low Risk
Low risk documentation-only changes, but they do alter agent guidance around MCP server management (auth/tool filtering/access) and remove the checked-in
.codex/hooks.json, which could affect local hook behavior if anyone relied on it.Overview
Updates installation docs (
README.md,install.md, and per-skill install pages) to standardize on long-formnpx skills add ... --skill ... --yescommands, including examples for global installs and agent linking.Refreshes the
mcp-serversskill to match current MCP Gateway product flows (remote vs official remote, virtual composition, hosted STDIO import, OpenAPI import), with stronger safety guidance (confirmation-gatedtfy apply, no deletes/collaborator removals, secret handling) and new reference docs for UI click paths and YAML/auth variations. Also trims the gatewaymcp-serverschema reference down to a compact index that points to the new detailed references.Removes
.codex/hooks.jsonfrom the repo and stops un-ignoring it in.gitignore.Reviewed by Cursor Bugbot for commit 9fe9660. Bugbot is set up for automated code reviews on this repo. Configure here.