Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 3 additions & 3 deletions agentscore_commerce/challenge/pricing.py
Original file line number Diff line number Diff line change
@@ -1,9 +1,9 @@
"""Pricing block builder + canonical type.

Composes cents-denominated price components into the dollar-string shape that 402
challenge bodies advertise. Lifts the inline pattern from martin-estate's
``purchase.ts`` so every merchant — current and future commerce-platform plugins
(Commerce7, WooCommerce, Shopify) — surfaces the same shape to agents.
challenge bodies advertise. Standardizes the pricing block so every merchant —
current and future commerce-platform plugins (Commerce7, WooCommerce, Shopify) —
surfaces the same shape to agents.

Shipping is included by default because most physical-goods merchants carry it; pass
``shipping_cents=0`` (or omit) for digital goods / services. Tax is optional for
Expand Down
2 changes: 1 addition & 1 deletion agentscore_commerce/payment/idempotency.py
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@
``/v1/credentials/wallets`` capture endpoint dedupes correctly and the operator's
``transaction_count`` doesn't inflate.

Convention (matches what martin-estate uses today):
Convention:
1. Prefer the upstream payment-rail's stable identifier (Stripe PaymentIntent id, x402
tx hash) when one exists — those are already idempotent on their side.
2. Fall back to a synthesized ``pi-{order_id}-{amount_cents}`` key when no upstream id
Expand Down
2 changes: 1 addition & 1 deletion tests/test_skill_md.py
Original file line number Diff line number Diff line change
Expand Up @@ -115,7 +115,7 @@ def test_rejects_name_over_64_chars(self) -> None:

def test_rejects_uppercase_name(self) -> None:
cfg = _base()
cfg.name = "Martin-Estate"
cfg.name = "Example-Merchant"
with pytest.raises(ValueError, match=r"lowercase"):
build_skill_md(cfg)

Expand Down