feat: add secure-payments reference for hardening agent payments#41
Open
theyavuzarslan wants to merge 1 commit into
Open
feat: add secure-payments reference for hardening agent payments#41theyavuzarslan wants to merge 1 commit into
theyavuzarslan wants to merge 1 commit into
Conversation
🟡 Heimdall Review Status
|
Adds a secure-payments.md reference and a routing row pointing to it. The existing references cover how to move money (send, trade, x402 pay); none cover the checks that should run before money moves when the amount, address, or endpoint came from a source the agent does not control. Uses only commands already in the skill: x402 details to inspect an endpoint before paying, and --max-amount as the per-payment cap. The provenance check, session/rate budget, kill switch, and audit log are described as agent-side logic, so nothing falls outside the skill's existing allowed-tools. Composes with the current references without changing their behavior.
theyavuzarslan
force-pushed
the
add-secure-payments-reference
branch
from
July 10, 2026 00:40
ac03649 to
303e767
Compare
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.
What
Adds a
secure-payments.mdreference and one routing row inSKILL.mdthat points to it.Why
The existing references cover how to move money (
send,trade,x402 pay). None cover the checks that should run before money moves when the amount, address, or endpoint came from somewhere the agent does not control (a web page, a tool result, a bazaar listing, another agent). The risk is concrete: if a recipient address or amount arrives inside untrusted text, nothing downstream re-checks it before the wallet spends.What it adds
The reference sequences existing skill commands into a safe order and adds agent-side guardrails:
x402 detailsbefore paying, and check price, asset, chain, and pay-to against expectations.--max-amountper-payment cap.Scope
skills/agentic-wallet/references/secure-payments.mdSKILL.mdrouting tableThe only literal command blocks are
awal status,x402 details, andx402 pay --max-amount, all already permitted by the skill'sallowed-tools. Budget, kill switch, and audit log are written as agent-side logic, not shell the skill runs, so nothing falls outside the existing tool policy. No change to any current reference.Disclosure
I work on agent security at GoPlus Security, and I proposed a version of this pattern for Circle's agent wallet skill (circlefin/skills#35). This reference is rewritten for
awaland the x402 bazaar, is vendor-neutral, and references no external product.