fix(skills): harden AI agent skills against prompt injection, autonomous spending, and insecure transport - #93
Open
mozluk wants to merge 1 commit into
Open
fix(skills): harden AI agent skills against prompt injection, autonomous spending, and insecure transport#93mozluk wants to merge 1 commit into
mozluk wants to merge 1 commit into
Conversation
…ous spending, and insecure transport ## Motivation This PR hardens multiple AI agent skills within the `skills` repository to address high and medium-severity vulnerabilities identified during the workspace security audit[cite: 6]. Previously, several skills allowed agents to execute irreversible financial transactions without user confirmation, ingested untrusted remote content as instructions (prompt injection), utilized plaintext HTTP for shell script execution, and performed autonomous OTP logins by reading user mailboxes without explicit consent[cite: 6]. ## Modifications * **Irreversible Action Guardrails (`moonpay-swap-tokens`, `moonpay-buy-crypto`, `yield-optimization`)**: * Implemented strict pre-execution confirmation gates[cite: 6]. The agent must now explicitly read back the full destination address and amount to the user for approval before signing or broadcasting any fund-moving transactions[cite: 6]. * **Prompt Injection Defenses (`allium-x402`, etc.)**: * Added explicit directives ensuring the agent treats remote Markdown and paid API responses purely as data to report, rather than instructions to obey[cite: 6]. The agent is now strictly forbidden from complying with third-party content that requests calling another service, raising spend limits, or revealing credentials[cite: 6]. * **Autonomous Login Restrictions (`moonpay-auth`)**: * Removed "no human in the loop" autonomous OTP login[cite: 6]. The skill now requires explicit user consent before triggering a login, restricts mailbox searches exclusively to MoonPay verification messages, and prohibits writing the OTP or session credential anywhere except the verification prompt[cite: 6]. * **Transport Security & Documentation (`allium-x402`, `dune-analytics`, `messari-alpha-scout`, `corbits-marketplace`, `myriad-prediction-markets`)**: * Upgraded the `curl | sh` installer instructions in `allium-x402` from plaintext HTTP to HTTPS to prevent network tampering[cite: 6]. * Cleaned up documentation by removing phantom cross-references to non-existent skills (e.g., `moonpay-bridge-tokens` and `moonpay-budget-agent`)[cite: 6]. ## Checklist - [x] Format your code according to the Contributor Guide. - [ ] Add unit tests as outlined in the Contributor Guide. - [x] Update documentation as needed, including docstrings or example tutorials.
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.
Motivation
This PR hardens multiple AI agent skills within the
skillsrepository to address high and medium-severity vulnerabilities identified during the workspace security audit[cite: 6]. Previously, several skills allowed agents to execute irreversible financial transactions without user confirmation, ingested untrusted remote content as instructions (prompt injection), utilized plaintext HTTP for shell script execution, and performed autonomous OTP logins by reading user mailboxes without explicit consent[cite: 6].Modifications
moonpay-swap-tokens,moonpay-buy-crypto,yield-optimization):allium-x402, etc.):moonpay-auth):allium-x402,dune-analytics,messari-alpha-scout,corbits-marketplace,myriad-prediction-markets):curl | shinstaller instructions inallium-x402from plaintext HTTP to HTTPS to prevent network tampering[cite: 6].moonpay-bridge-tokensandmoonpay-budget-agent)[cite: 6].Checklist
New Skill
Skill name:
skills/{partner}-{name}/(e.g.,corbits-marketplace)Description: What does this skill do?
Primary chain: (e.g., Ethereum, Solana, Polygon)
Primary token: (e.g., USDC, ETH, SOL)
Checklist
skills/{name}/SKILL.mdwith YAML frontmatter (name,description).claude-plugin/marketplace.jsonMoonPay Integration
How does this skill use MoonPay wallets/payments? (e.g., micropayments, transaction signing, wallet funding)
Example Usage