Skip to content

Add Approval Risk Auditor submission#198

Open
tolga-tom-nook wants to merge 2 commits into
daydreamsai:masterfrom
tolga-tom-nook:tom-nook-approval-risk-auditor
Open

Add Approval Risk Auditor submission#198
tolga-tom-nook wants to merge 2 commits into
daydreamsai:masterfrom
tolga-tom-nook:tom-nook-approval-risk-auditor

Conversation

@tolga-tom-nook
Copy link
Copy Markdown

Bounty Submission

Related Issue: #5


Submission File

File Path: submissions/approval-risk-auditor-tom-nook.md


Live Agent


Source / Verification

Verification on source:

npm run build
# TypeScript clean

npm test
# 3 test files passed, 20 tests passed

Features

  • x402-gated protected calls: unpaid requests return HTTP 402 with payment requirements before audit work runs.
  • ERC-20 approval scanning with current allowance validation.
  • Etherscan-compatible explorer fallback hooks for Approval logs when API keys are configured.
  • NFT ApprovalForAll discovery hooks and revoke calldata.
  • Revoke calldata for ERC-20 approve(spender, 0) and NFT setApprovalForAll(operator, false).
  • Daydreams-style manifest/entrypoints plus compatibility alias for hyphenated clients.
  • No private-key handling, no signing, no fund movement; unsigned tx targets/calldata only.

x402 Caveat

The deployment proves x402 reachability/payment-requirement behavior: unpaid protected calls return HTTP 402 with requirements. Full paid verify/settle is implemented through configurable facilitator /verify and /settle, but has not been live-tested because the expected facilitator URL/schema and exact Base USDC asset identifier need to be provided/configured.


Solana Wallet

8sqgL8Srd7QCWJnQRFw1Gsi4spS9rndAbER1HEGDHLNT

@tolga-tom-nook
Copy link
Copy Markdown
Author

Best-submission hardening update pushed.

New source commit: tolga-tom-nook/approval-risk-auditor-agent@f388ab2
Submission branch updated: #198

Additional improvements:

  • Expanded supported chains to Ethereum, Base, Polygon, Arbitrum, Optimism, BSC, Avalanche, Fantom, and Gnosis.
  • Added short agent-kit-style compatibility route: POST /entrypoints/audit/invoke.
  • Existing routes remain: canonical /entrypoints/audit_approvals/invoke, Add Approval Risk Auditor submission #167-style /entrypoints/audit-approvals/invoke, and legacy /invoke.
  • Added spender bytecode / known-spender risk flags for unknown EOA/contract spenders when checks are available.
  • Expanded regression tests: npm test now passes 23 tests across 3 files; npm run build is clean.
  • Redeployed Worker; /entrypoints now advertises aliases audit-approvals and audit, and unpaid /entrypoints/audit/invoke returns HTTP 402 with the canonical x402 resource.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant