Skip to content

Add AgentPay partner skill (x402 specialist agents + AgentWorld economy data) - #231

Closed
shawnhvac wants to merge 2 commits into
BlockRunAI:mainfrom
shawnhvac:add-agentpay-skill
Closed

Add AgentPay partner skill (x402 specialist agents + AgentWorld economy data)#231
shawnhvac wants to merge 2 commits into
BlockRunAI:mainfrom
shawnhvac:add-agentpay-skill

Conversation

@shawnhvac

@shawnhvac shawnhvac commented Aug 2, 2026

Copy link
Copy Markdown

AgentPay partner skill

Following up on #157 the right way — as a partner skill, not an issue. Thanks @VickyXAI for pointing to the pattern; this adds skills/agentpay/SKILL.md in the same shape as the existing predexon / imagegen / phone skills. Self-contained, no src/ changes, no test impact — agents can already call these x402 endpoints directly (as you noted in #157); this just makes the full AgentPay platform discoverable with triggers, endpoints, pricing, and examples.

What agents get — the full AgentPay platform (not just the experts)

1. 19 specialist agents (POST {slug}/api/query, $0.05–$0.20) — WALLY (Wall St), CIPHER (crypto), FEEDS (live crypto data), SCOUT (competitive intel), TREK (travel), sports (DUKE/GRIDIRON/HARDWOOD/BLADES/APEX), skilled trades (FROSTBYTE HVAC/R, VOLTAG, PIPEWELL, WELDCORE, FRAMER, GEARHEAD). Plus per-team deep dives with live odds & prop lines.

2. Sports Betting Hub — live agent betting across NBA/NFL/NHL: lines, open bets, volume, and bettor leaderboards.
GET /api/agentworld/betting/hub-summary · GET /api/agentworld/sports/bets?sport=all

3. Build-Your-Own-Agent — register any HTTP endpoint as an x402-payable agent that earns USDC revenue share, then track balance & claim.
POST /custom/api/register · /api/partner/{register,endpoints,balance,analytics,claim}

4. x402 Facilitator — the payment rail itself: verify EIP-712 grants, settle USDC, and escrow/release/refund for any agent-to-agent transaction across Base, Ethereum, Optimism, Arbitrum, Polygon.
POST /x402/verify · /x402/settle · /escrow · /release · /refund · GET /x402/info · /x402/stats

5. AgentWorld Economy Data API (GET agentworld.me/api/data/*, $0.001–$0.010) — economy snapshot, roster, wealth leaderboard w/ on-chain wallets, 2.5M+ tx firehose, social graph. Catalog free.

All payments settle through the AgentPay facilitator (x402-agent-pay.com, treasury on Base L2) — same wallet model as LLM calls.

Notes

  • Pure docs addition under skills/ — nothing in src/, no test surface.
  • Happy to also add src/partners/registry.ts entries so AgentPay routes through the BlockRun localhost:8402 gateway like Predexon/imagegen — that's a maintainer/routing call, so I left it out. Say the word and I'll open the follow-up.

All endpoints verified live against agentpaystore.com/openapi.json and x402-agent-pay.com/openapi.json on 2026-08-02.

Summary by CodeRabbit

  • New Features
    • Added the AgentPay skill with supported trigger phrases and service configuration.
    • Added access to platform capabilities, specialist-agent routes, pricing information, sports betting APIs, and the AgentWorld economy data catalog.
    • Added support for custom-agent registration, revenue sharing, payment and escrow workflows.
    • Included representative request examples for finance, HVAC, sports, and economy queries.

@coderabbitai

coderabbitai Bot commented Aug 2, 2026

Copy link
Copy Markdown

Review Change Stack

📝 Walkthrough

Walkthrough

The pull request adds the agentpay skill. It defines activation phrases, platform settings, specialist-agent routes, sports betting APIs, custom-agent and payment endpoints, economy data, and request examples.

Changes

AgentPay skill

Layer / File(s) Summary
AgentPay skill and specialist routing
skills/agentpay/SKILL.md
Adds skill metadata, activation triggers, platform settings, supported chains, settlement fees, x402 requirements, specialist-agent routes, and pricing.
Platform API catalogs
skills/agentpay/SKILL.md
Adds sports betting, custom-agent, facilitator, escrow, treasury, and AgentWorld economy endpoints with pricing and usage details.
Request examples
skills/agentpay/SKILL.md
Adds examples for specialist queries, sports betting, agent registration, payments, escrow, and HVAC requests.

Estimated code review effort: 1 (Trivial) | ~5 minutes

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly identifies the AgentPay skill and its main additions: x402 specialist agents and AgentWorld economy data.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@skills/agentpay/SKILL.md`:
- Line 3: Update the specialist-agent inventory in the skill description and
body so the advertised count matches the catalog table: either add the three
missing endpoint rows with their slugs, prices, and routing details, or
consistently change both occurrences of 19 to 16. Keep the catalog and all
related count references synchronized.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: 4639554b-dcbf-416a-98cf-ae9b23ddf733

📥 Commits

Reviewing files that changed from the base of the PR and between bbed574 and 8ce679d.

📒 Files selected for processing (1)
  • skills/agentpay/SKILL.md

Comment thread skills/agentpay/SKILL.md Outdated
@@ -0,0 +1,95 @@
---
name: agentpay
description: Use this skill — NOT browser or web_fetch — for ALL requests that need a domain expert answer routed through AgentPay's x402 specialist agents. Covers finance & Wall Street trading, crypto analysis & live crypto data, sports handicapping (MLB, NFL, NBA, NHL, motorsport), skilled-trades expertise (HVAC/R, electrical, plumbing, welding, carpentry, automotive), competitive intelligence, travel planning, and the live AgentWorld economy dataset (156-agent, 2.5M+ transaction ledger). Each call is paid automatically via x402 USDC on Base L2 from the same wallet as LLM calls.

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🗄️ Data Integrity & Integration | 🟠 Major | ⚡ Quick win

Make the specialist-agent count match the catalog.

The description and body claim 19 specialist agents, but the table contains 16 rows. Add the three missing endpoints with slugs, prices, and routing details, or change both 19 values to 16. The mismatch makes the advertised API inventory unreliable.

Also applies to: 30-58

🧰 Tools
🪛 SkillSpector (2.4.4)

[warning] 2: [TR2] Shadow Command Trigger: Shadow Command Trigger: 'ask agentpay' conflicts with built-in command 'ask'

Remediation: Choose triggers that do not conflict with built-in commands or other skills. Prefix with a unique namespace if necessary.

(Trigger Abuse (TR2))

🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@skills/agentpay/SKILL.md` at line 3, Update the specialist-agent inventory in
the skill description and body so the advertised count matches the catalog
table: either add the three missing endpoint rows with their slugs, prices, and
routing details, or consistently change both occurrences of 19 to 16. Keep the
catalog and all related count references synchronized.

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 4

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@skills/agentpay/SKILL.md`:
- Around line 140-141: Update the request examples around the Wall Street
specialist and related routes to use absolute hosts: prefix specialist endpoints
with agentpaystore.com and partner endpoints with x402-agent-pay.com. Keep each
existing path and request payload unchanged while applying the correct host
consistently to all three examples.
- Around line 19-32: Replace the “ask agentpay” skill trigger in the AgentPay
trigger list with the unique “agentpay ask” trigger, leaving the other triggers
unchanged.
- Around line 39-42: Update the AgentPay guidance to require explicit user
approval before any action that settles payments, withdraws funds, or changes
financial state, including facilitator settlement, escrow, release, refund, and
partner claims. Require checking and enforcing the configured ClawRouter spend
limit, such as perRequest, before proceeding with each payable request.
- Around line 92-98: Align the AgentPay onboarding documentation with the public
enrollment contract by documenting the required POST /api/agentpay/register step
with name and email before provider or endpoint actions, or add a clear
reference proving the alternate credentials model used here. Reconcile the
partner endpoint registration path and ensure the example and API table
consistently identify whether price_usdc and payTo are required or optional.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: 2af2607b-3347-469d-9250-86460b315a81

📥 Commits

Reviewing files that changed from the base of the PR and between 8ce679d and d9c4788.

📒 Files selected for processing (1)
  • skills/agentpay/SKILL.md

Comment thread skills/agentpay/SKILL.md
Comment on lines +19 to +32
- "sports betting hub"
- "betting lines"
- "live bets"
- "bettor leaderboard"
- "build your own agent"
- "create an x402 agent"
- "register my agent"
- "monetize my agent"
- "x402 facilitator"
- "settle x402 payment"
- "verify x402 payment"
- "usdc escrow"
- "agent to agent payment"
- "agentworld economy data"

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🎯 Functional Correctness | 🟠 Major | ⚡ Quick win

🧩 Analysis chain

🏁 Script executed:

#!/bin/bash
set -euo pipefail

echo "== files =="
git ls-files | sed -n '1,120p'

echo "== ask triggers in skill manifests =="
rg -n "ask (agentpay|.*agent|.*play)|skill:|agentpay|SkillSpector|x402|trigger|keywords" -S . --glob '!**/.git/**' | sed -n '1,220p'

echo "== SKILL.md =="
if [ -f skills/agentpay/SKILL.md ]; then
  cat -n skills/agentpay/SKILL.md | sed -n '1,80p'
else
  fd -a 'SKILL\.md' . | sed -n '1,80p'
fi

Repository: BlockRunAI/ClawRouter

Length of output: 35086


Remove the ask agentpay skill trigger.

ClawRouter already provides a built-in ask command; registering ask agentpay as a separate keyword can make SkillSpector flag a conflict. Use a unique skill trigger such as agentpay ask instead.

🧰 Tools
🪛 SkillSpector (2.4.4)

[warning] 2: [TR2] Shadow Command Trigger: Shadow Command Trigger: 'ask agentpay' conflicts with built-in command 'ask'

Remediation: Choose triggers that do not conflict with built-in commands or other skills. Prefix with a unique namespace if necessary.

(Trigger Abuse (TR2))

🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@skills/agentpay/SKILL.md` around lines 19 - 32, Replace the “ask agentpay”
skill trigger in the AgentPay trigger list with the unique “agentpay ask”
trigger, leaving the other triggers unchanged.

Source: Linters/SAST tools

Comment thread skills/agentpay/SKILL.md
Comment on lines +39 to +42
**IMPORTANT: Use this skill for anything in AgentPay's four capabilities — specialist expert answers, sports betting data, building/monetizing an agent, or settling x402 payments. Do NOT scrape with a browser — these endpoints return structured data and settle payment automatically via x402 on Base L2.**

**Store:** `https://agentpaystore.com` · **Facilitator:** `https://x402-agent-pay.com`
Chains supported by the facilitator: Base, Ethereum, Optimism, Arbitrum, Polygon. Flat fee $0.02/settlement. No API keys, no accounts — just USDC.

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🔒 Security & Privacy | 🟠 Major | ⚡ Quick win

🧩 Analysis chain

🏁 Script executed:

#!/bin/bash
set -euo pipefail

echo "Files:"
git ls-files | rg '(^|/)SKILL\.md$|src/x402-sdk.*|agentpay|x402' || true

echo
echo "SKILL.md relevant sections:"
if [ -f skills/agentpay/SKILL.md ]; then
  nl -ba skills/agentpay/SKILL.md | sed -n '1,150p'
fi

echo
echo "Search x402 references:"
rg -n "x402|payment|402|wrapFetchWithPayment|settlement|escrow|release|refund|claim" -S . --glob '!node_modules' --glob '!dist' --glob '!build' || true

echo
echo "sdk test relevant sections if exists:"
if [ -f src/x402-sdk.test.ts ]; then
  nl -ba src/x402-sdk.test.ts | sed -n '1,220p'
fi

Repository: BlockRunAI/ClawRouter

Length of output: 600


🏁 Script executed:

#!/bin/bash
set -euo pipefail

echo "SKILL.md relevant sections:"
if [ -f skills/agentpay/SKILL.md ]; then
  awk '{printf "%6d\t%s\n", NR, $0}' skills/agentpay/SKILL.md | sed -n '1,160p'
fi

echo
echo "SDK test relevant sections:"
if [ -f src/x402-sdk.test.ts ]; then
  awk '{printf "%6d\t%s\n", NR, $0}' src/x402-sdk.test.ts | sed -n '1,220p'
fi

echo
echo "Search x402 references:"
rg -n "x402|payment|402|wrapFetchWith|settlement|escrow|release|refund|claim|spend|confirm|limit|budget" -S . --glob '!node_modules' --glob '!dist' --glob '!build' || true

Repository: BlockRunAI/ClawRouter

Length of output: 50378


🏁 Script executed:

#!/bin/bash
set -euo pipefail

echo "Confining references spend/limit/budget/confirm/key wallet:"
rg -n "spend|limit|budget|confirm|allowlisted|whitelisted|approve|approval|user approval|authorization|confirm|key: 'x402'|PAYMENT_REQUIRED_STATUS_CODES|PAYMENT_REQUIRED|X-PAYMENT|pre-sign|Pre-sign|402" -S src/scripts docs CLAUDE.md README.md --glob '!package-lock.json' --glob '!node_modules' || true

echo
echo "Spend control source:"
if [ -f src/spend-control.ts ]; then
  wc -l src/spend-control.ts
  awk '{printf "%6d\t%s\n", NR, $0}' src/spend-control.ts | sed -n '1,240p'
fi

echo
echo "Payment/preauth relevant source snippets (focused):"
for f in src/payment-preauth.ts src/x402.ts src/x402-sdk.test.ts; do
  if [ -f "$f" ]; then
    echo "--- $f ---"
    wc -l "$f"
    awk '{printf "%6d\t%s\n", NR, $0}' "$f" | sed -n '1,220p'
  fi
done

echo
echo "Search for spendControl calls and provider blocking:"
rg -n "spendControl|SpendControl|should|allow|block|reject|limit|limitExceeded|perRequest|hourly|daily|session|x402ApiKey|apiKey|x402|PAYMENT_REQUIRED|402|startProxy|fetchRequest|v1" src -S --glob '!node_modules' --glob '!dist' --glob '!build' | head -n 300

Repository: BlockRunAI/ClawRouter

Length of output: 50377


Require user approval before AgentPay actions that withdraw funds or change financial state.

skills/agentpay/SKILL.md says the skill may settle x402 payments directly, including facilitator settlement, escrow, release, refund, and partner claim actions. Add a confirmation step for those actions and document/check an enforced spend limit such as perRequest from the ClawRouter spend control path.

🧰 Tools
🪛 SkillSpector (2.4.4)

[warning] 2: [TR2] Shadow Command Trigger: Shadow Command Trigger: 'ask agentpay' conflicts with built-in command 'ask'

Remediation: Choose triggers that do not conflict with built-in commands or other skills. Prefix with a unique namespace if necessary.

(Trigger Abuse (TR2))

🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@skills/agentpay/SKILL.md` around lines 39 - 42, Update the AgentPay guidance
to require explicit user approval before any action that settles payments,
withdraws funds, or changes financial state, including facilitator settlement,
escrow, release, refund, and partner claims. Require checking and enforcing the
configured ClawRouter spend limit, such as perRequest, before proceeding with
each payable request.

Comment thread skills/agentpay/SKILL.md
Comment on lines +92 to +98
- **Register an agent:** POST `https://agentpaystore.com/custom/api/register`
body: `{ "name": "...", "slug": "...", "endpoint_url": "https://...", "description": "...", "price_usdc": "0.10", "payTo": "0x..." }`
(required: `name`, `endpoint_url`).
- **Register as a revenue-share provider:** POST `https://x402-agent-pay.com/api/partner/register`
- **Add / list your endpoints:** POST / GET `https://x402-agent-pay.com/api/partner/endpoints`
- **Check earnings & claim:** GET `https://x402-agent-pay.com/api/partner/balance` · GET `/api/partner/analytics` · POST `/api/partner/claim`
- **Browse the ecosystem:** GET `https://x402-agent-pay.com/api/agentpay/ecosystem`

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🗄️ Data Integrity & Integration | 🟠 Major | ⚡ Quick win

🧩 Analysis chain

🏁 Script executed:

#!/bin/bash
set -euo pipefail

echo "Repo files matching SKILL.md:"
fd -a 'SKILL\.md$' . | sed 's#^\./##'

echo
echo "Target file excerpt if present:"
if [ -f skills/agentpay/SKILL.md ]; then
  nl -ba skills/agentpay/SKILL.md | sed -n '70,170p'
else
  echo "skills/agentpay/SKILL.md not found"
fi

echo
echo "Search registration API references:"
rg -n "agentpaystore\.com|x402-agent-pay\.com|register|partner/endpoints|partner/balance|payTo|price_usdc|slug|endpoint_url" . --glob '!node_modules' --glob '!dist' --glob '!build' | head -200

Repository: BlockRunAI/ClawRouter

Length of output: 619


🏁 Script executed:

#!/bin/bash
set -euo pipefail

echo "skills/agentpay/SKILL.md excerpt:"
sed -n '70,170p' skills/agentpay/SKILL.md | awk '{printf "%5d\t%s\n", NR+69, $0}'

echo
echo "AgentPay API references:"
rg -n "agentpaystore\.com|x402-agent-pay\.com|register|partner/endpoints|partner/balance|payTo|price_usdc|slug|endpoint_url|ecosystem" skills/agentpay/SKILL.md || true

echo
echo "Repository references:"
rg -n "agentpaystore\.com|x402-agent-pay\.com|partner/endpoints|partner/balance|partner/claim|payTo|price_usdc|slug|endpoint_url" . --glob '!node_modules' --glob '!dist' --glob '!build' --glob '!.git' | head -200 || true

Repository: BlockRunAI/ClawRouter

Length of output: 16617


🌐 Web query:

AgentPay x402 custom API register partner register endpoint require slug price_usdc payTo

💡 Result:

The AgentPay x402 ecosystem provides infrastructure for registering AI agents and service endpoints to enable autonomous, per-request payments using the x402 protocol (typically on the Base L2 network) [1][2]. Regarding your specific query about registering an endpoint with a slug, price, and payment address, this functionality is primarily associated with x402 hosting hubs or gateways, such as the one documented at agent-tools.cloud [3]. These services allow you to wrap an existing API into a pay-per-call x402 endpoint by submitting your configuration [3]. Endpoint Registration (Hosting Hub Model) To register a service on such a hub, you typically submit a POST request to an endpoint like /api/v1/submit with the following parameters [3]: slug (e.g., service_slug): A unique identifier for your endpoint [3]. price_usdc: The amount in USDC to charge per request [3]. payTo (e.g., payout_address): Your wallet address (Base L2) where payments will be settled [3]. Other fields often include the upstream URL and any necessary authentication headers for the proxied service [3]. Example Registration Request: curl -X POST https://hub.agent-tools.cloud/api/v1/submit \ -H 'content-type: application/json' \ -d '{ "service_slug": "my-api", "payout_address": "0xYourBaseWallet", "upstream_base_url": "https://api.example.com", "price_usdc": 0.01 }' Key Differences in AgentPay Registration It is important to distinguish between registering an AI agent (the consumer) and registering a service endpoint (the provider) [4][5][2]. - Registering an AI Agent: Use POST /api/agentpay/register with name and email; this returns a partner_id and API key for making payments [4][6]. - Registering a Service Endpoint: This makes your existing API "x402-enabled" so it can accept payments [2]. While the core x402 protocol uses headers like X-402-Payment, the actual registration of a proxied, paid endpoint is often handled via these third-party hub services which manage the settlement logic [3][2]. Always verify the specific documentation for the hub or facilitator you are using, as requirements can vary between providers [1][7][3].

Citations:


🌐 Web query:

site:x402-agent-pay.com/onboard /api/partner/register partner balance register

💡 Result:

I found no indexed page for /api/partner/register or a “partner balance register” endpoint.

The current onboarding page documents:

  • Partner registration: POST https://x402-agent-pay.com/api/agentpay/register
  • Required fields: name, email
  • Authentication header afterward: X-Partner-Token: YOUR_API_KEY
  • Balance: available through the dashboard, linked from the onboarding page. (x402-agent-pay.com)

Example:

curl -X POST https://x402-agent-pay.com/api/agentpay/register \
  -H "Content-Type: application/json" \
  -d '{"name":"YOUR_NAME","email":"YOUR_EMAIL"}'

The site also references a separate endpoint-registration route: POST /api/register-endpoint. (x402-agent-pay.com)

Citations:


Align the enrollment flow with the documented AgentPay contracts.

The public AgentPay onboarding page uses POST https://x402-agent-pay.com/api/agentpay/register with name and email before provider/endpoint actions, but this skill documents https://x402-agent-pay.com/api/partner/register and POST /partners/register-endpoint while the example omits partner registration. Update the steps or add a reference that proves the AgentPay credentials/enrollment model this skill uses. Keep partner-enrollment data required/optional explicit, because price_usdc and payTo are marked as required in the example body but listed as optional in the API table.

🧰 Tools
🪛 SkillSpector (2.4.4)

[warning] 2: [TR2] Shadow Command Trigger: Shadow Command Trigger: 'ask agentpay' conflicts with built-in command 'ask'

Remediation: Choose triggers that do not conflict with built-in commands or other skills. Prefix with a unique namespace if necessary.

(Trigger Abuse (TR2))

🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@skills/agentpay/SKILL.md` around lines 92 - 98, Align the AgentPay onboarding
documentation with the public enrollment contract by documenting the required
POST /api/agentpay/register step with name and email before provider or endpoint
actions, or add a clear reference proving the alternate credentials model used
here. Reconcile the partner endpoint registration path and ensure the example
and API table consistently identify whether price_usdc and payTo are required or
optional.

Comment thread skills/agentpay/SKILL.md
Comment on lines +140 to +141
**User:** Ask a Wall Street expert whether to hold TSLA into earnings.
→ POST `/wally/api/query` `{ "query": "Hold TSLA into earnings — bull vs bear case?" }`.

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🎯 Functional Correctness | 🟡 Minor | ⚡ Quick win

Use absolute hosts in every request example.

Examples at Lines 141, 150, and 159 use relative paths, while the skill uses two different hosts. Specify agentpaystore.com for specialist routes and x402-agent-pay.com for partner routes.

Proposed documentation fix
-→ POST `/wally/api/query` `{ "query": "Hold TSLA into earnings — bull vs bear case?" }`.
+→ POST `https://agentpaystore.com/wally/api/query` `{ "query": "Hold TSLA into earnings — bull vs bear case?" }`.

-→ POST `https://agentpaystore.com/custom/api/register` with `name`, `endpoint_url`, `price_usdc`, `payTo`. Then GET `/api/partner/balance` to track earnings.
+→ POST `https://agentpaystore.com/custom/api/register` with `name`, `endpoint_url`, `price_usdc`, `payTo`. Then GET `https://x402-agent-pay.com/api/partner/balance` to track earnings.

-→ POST `/frostbyte/api/query` `{ "query": "Condenser short-cycling every 3 min on R-410A — causes?" }`.
+→ POST `https://agentpaystore.com/frostbyte/api/query` `{ "query": "Condenser short-cycling every 3 min on R-410A — causes?" }`.

Also applies to: 149-150, 158-159

🧰 Tools
🪛 SkillSpector (2.4.4)

[warning] 2: [TR2] Shadow Command Trigger: Shadow Command Trigger: 'ask agentpay' conflicts with built-in command 'ask'

Remediation: Choose triggers that do not conflict with built-in commands or other skills. Prefix with a unique namespace if necessary.

(Trigger Abuse (TR2))

🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@skills/agentpay/SKILL.md` around lines 140 - 141, Update the request examples
around the Wall Street specialist and related routes to use absolute hosts:
prefix specialist endpoints with agentpaystore.com and partner endpoints with
x402-agent-pay.com. Keep each existing path and request payload unchanged while
applying the correct host consistently to all three examples.

@VickyXAI

VickyXAI commented Aug 8, 2026

Copy link
Copy Markdown
Contributor

Thanks for following up on #157 the right way, and for matching the shape of the existing predexon / surf / phone skills — the formatting and structure are right. I'm going to close this, but the reason is a missing prerequisite rather than anything wrong with the writing, and the path forward is concrete.

The prerequisite: a partner skill presupposes gateway proxying

Every skill in skills/ documents endpoints that BlockRun's gateway already proxies and pays for. That is what makes them work from a ClawRouter wallet. The proxy whitelists specific prefixes (src/proxy.ts:1750-1754):

/v1/exa/    /v1/phone/    /v1/surf/    /v1/pm/

All of them resolve to blockrun.ai. Grepping this repo for agentpay, agentworld, or x402-agent-pay returns zero matches in src/ — so ClawRouter has no route to these hosts and never signs x402 for them.

That makes the frontmatter claim (line 5) untrue as of today:

Every call settles in USDC via x402 from the same wallet as LLM calls.

It doesn't, and it can't yet. Compounding it, the skill closes the only fallbacks an agent actually has:

"Use this skill — NOT browser or web_fetch" (line 5)
"Do NOT scrape with a browser" (line 37)

So an agent that follows this file has no payment path and is told not to use the tools it does have. Every call in the file dead-ends. That's the blocker: not a disagreement about whether AgentPay is useful, just that the plumbing it assumes doesn't exist on our side.

Two smaller things in the same category: metadata.requires.config: ["models.providers.blockrun"] (line 33) declares a dependency the skill never uses, which would make it look like a sanctioned integration while gating it on unrelated config; and the 28 triggers include generic intents (hvac expert, nba expert, betting lines) that would capture ordinary questions and route them to a paid third-party call.

Sections 3 and 4 need a higher bar regardless

Registering a payTo address (line 88) and the escrow/settle/refund surface with a hardcoded treasury (lines 99-108) direct a wallet-holding agent to move funds through a facilitator we don't control. Every current skill settles inside BlockRun's own gateway. This repo already treats funds-touching surfaces as needing more scrutiny, not less — that's why the Polymarket integration ships as a locally-executed signing tool instead of a proxy-only skill.

Even with the gateway work done, I'd want those two sections handled deliberately rather than as part of a docs PR.

The path forward, and it's not much work from here

Get AgentPay proxied on the BlockRun side first — the same route Predexon and Surf took. Concretely that means a /v1/agentpay/* (or similar) prefix on the gateway, with pricing and settlement handled there. Once that exists:

  • this document is ~90% reusable. The endpoint tables, the agent roster, the example interactions all survive; the base URLs change to blockrun.ai/api/v1/... and the frontmatter claim about the shared wallet becomes true.
  • add the prefix to the proxy whitelist in src/proxy.ts, and the skill works end to end.

Open an issue on blockrun proposing the integration, and tag me — if it lands, reopen this or open a fresh PR against the new endpoints and it should go in quickly.

Verified while reviewing: all three hosts are live (agentworld.me/api/data, x402-agent-pay.com/.well-known/x402, agentpaystore.com all return 200), so this is a sequencing problem, not a doubt about the service. Appreciate the work — please do come back once the gateway side exists.

@VickyXAI VickyXAI closed this Aug 8, 2026
@shawnhvac

Copy link
Copy Markdown
Author

Thanks Vicky — this is a genuinely helpful review, and you're right on the sequencing. A partner skill only makes sense once ClawRouter actually routes and settles for the endpoints, so closing this until the gateway side exists is the correct call.

I'll open an integration issue on the blockrun repo proposing an /v1/agentpay/* prefix — verify/settle plus the specialist and economy endpoints — and tag you there. Once that lands I'll come back with a fresh PR against the new base URLs, and I'll handle the two things you flagged deliberately: drop the unused models.providers.blockrun requirement and the over-broad triggers, and treat the escrow/settle surface as a funds-touching integration rather than a docs addition.

Appreciate you taking the time to lay out the path instead of just declining. More soon.

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.

2 participants