Uniswap Execute Swap

AdvisoryAudited by Static analysis on Apr 30, 2026.

Overview

No suspicious patterns detected.

Findings (0)

Artifact-based informational review of SKILL.md, metadata, install specs, static scan signals, and capability signals. ClawScan does not execute the skill or run runtime probes.

What this means

A prompt such as “swap X for Y” could lead the agent to submit a real token trade, not just show a quote, which can move funds and may be hard to reverse.

Why it was flagged

The skill explicitly frames financial trade execution as autonomous, but the workflow does not include a required final user confirmation before the swap is executed.

Skill content
Handles quotes, safety checks, simulation, and execution autonomously.
Recommendation

Require a final explicit confirmation after displaying quote, route, gas, slippage, token addresses, chain, wallet/account, and maximum spend before any signing or broadcast.

ConcernMedium Confidence
ASI03: Identity and Privilege Abuse
What this means

Users cannot tell from the artifact which wallet, account, allowances, or signing permissions the skill or MCP server may use to execute trades.

Why it was flagged

The metadata does not declare wallet credentials or privileged account access, while the skill’s stated purpose is to execute swaps and check spending limits, leaving the actual signing/account authority unclear.

Skill content
Required env vars: none; Primary credential: none; Capability signals: No capability tags were derived.
Recommendation

Declare the required wallet/auth model, supported accounts/chains, spending limits, approval requirements, and exactly what authority the MCP server receives.

What this means

The actual code that may quote, sign, route, or execute trades is outside the reviewed artifact set, so users must trust an additional unreviewed component.

Why it was flagged

The high-impact swap execution depends on an external MCP server package, but the reviewed skill contains no install spec, pin, lockfile, or included server implementation.

Skill content
Repo: [`Agentic-Uniswap` MCP server](https://github.com/wpank/Agentic-Uniswap/tree/main/packages/mcp-server) - Package: `@agentic-uniswap/mcp-server`
Recommendation

Pin and document the MCP server version, provide installation/provenance details, and review the server’s wallet handling and transaction execution code before use.

What this means

Trade instructions and execution authority may pass to another agent component whose behavior and safeguards are not visible in this review.

Why it was flagged

The skill delegates high-impact trade execution to a named subagent that is not included in the artifacts, with no described identity, permission boundary, or approval protocol.

Skill content
Delegate to trade-executor: Launch `Task(subagent_type:trade-executor)` with: tokenIn, tokenOut, amount, chain
Recommendation

Include or reference the reviewed trade-executor specification, define its permissions, and require the main agent to verify and confirm any transaction before execution.