Repository navigation
fix: remove sandbox credential names from the Codex and Cursor skills - #8
Merged
Merged
Conversation
The Claude plugin directory scans every folder in plugins/macaly-code. It found $MACALY_API_TOKEN in skills-codex and put the Claude plugin on a policy hold (MCP_FORWARDS_CREDENTIAL_ENV). The scan read the token as a credential on the user's machine. The token exists only in the Macaly cloud sandbox, and the agent never reads it. The skills now say that the project command runs in the isolated cloud sandbox, which already has the Macaly project environment. The agent behavior does not change.
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
The Claude plugin directory put the Claude plugin on a policy hold: "Uses a credential from the user's machine" (
MCP_FORWARDS_CREDENTIAL_ENV).The directory scans all folders in
plugins/macaly-code, not only the Claude files. It found$MACALY_API_TOKENinskills-codex/build-app-on-macaly/SKILL.md. It read the token as a credential on the user's machine that the plugin sends to the MCP server. This is a false positive. The token exists only in the Macaly cloud sandbox, and the agent never reads it.Changes
skills-codex/build-app-on-macaly/SKILL.mdandskills-cursor/build-app-on-macaly/SKILL.md: remove the$MACALY_API_TOKEN,$MACALY_BASE_URL, and$MACALY_CHAT_IDnames.The agent behavior does not change. The
skill_infoguides run verbatim in the sandbox, as before.This change was pushed to #7 after #7 merged, so it is not on
main.Validation
plugins/finds no credential names. The only match is the$ARGUMENTSslash-command placeholder.claude plugin validate .andclaude plugin validate plugins/macaly-codepass with no warnings.scripts/pass.🤖 Generated with Claude Code
Need help on this PR? Tag
@codesmith-botwith what you need.