Skip to content

Fix base registrations, pretty cli sell status#433

Merged
bussyjd merged 1 commit into
mainfrom
oisin/sell-status
May 6, 2026
Merged

Fix base registrations, pretty cli sell status#433
bussyjd merged 1 commit into
mainfrom
oisin/sell-status

Conversation

@OisinKyne
Copy link
Copy Markdown
Contributor

@OisinKyne OisinKyne commented May 6, 2026

Summary

What changed:

Why it matters:

Risk level:

Commit under test:

Base branch:

Scope

  • Code
  • Charts / manifests
  • Flows / QA scripts
  • Docs / skills
  • Images / dependencies
  • Other:

Validation

CI checks:

Check Status Link

Unit tests:

<!-- command, result, commit/SHA -->

Integration tests:

<!-- command, result, commit/SHA -->

Flow tests:

Flow Network QA machine label Worktree Result Artifacts

Release smoke:

<!-- command, flags, result -->

Live Chain Evidence

Do not include private keys, seed phrases, passwords, hostnames, personal paths, or raw bearer tokens.

Network:

RPC/provider:

Facilitator:

Contracts and tokens:

Name Address Version / notes

Wallet roles:

Role Address Source
Alice / seller / register
Bob / buyer / payer
Facilitator / receiver

Balances:

Token Address Before After Expected delta Actual delta

Transaction receipts:

Purpose Tx hash From To Amount / event Status
ERC-8004 registration
Metadata / service offer
Approval / permit
Purchase request
Settlement transfer

Runtime Evidence

QA environment:

Item Value
OS / arch
Backend
Tool versions
QA agent/model

Images:

Component Image Tag / digest Source

Kubernetes / stack:

Item Value
Stack IDs
Namespaces
Pod readiness
Cleanup result

Model and routing:

Item Value
Agent/model used
LiteLLM route
Paid endpoint status
Auth token source

Artifacts and logs:

Artifact Location / link Notes

Demo readiness:

Item Status Notes
Seller visible / registered
Buyer discovery works
Paid route works
Settlement visible on-chain

Review Notes

Known gaps:

Follow-ups:

Reviewer focus:

JD Validation Update (2026-05-06)

Status:

Targeted tests on current main:

go test ./cmd/obol ./internal/erc8004
ok  github.com/ObolNetwork/obol-stack/cmd/obol           1.211s
ok  github.com/ObolNetwork/obol-stack/internal/erc8004   8.607s

Live surface exercised on top of this merged state:

  • obol sell status spark2-reg -n llm rendered the prettier CLI status output correctly, including:
    • endpoint
    • network
    • price / pay-to
    • Agent ID link
    • registration tx link
    • per-condition readiness output
  • Registration on Base Sepolia completed successfully:
    • Agent ID: 5621
    • Tx: 0xbb59a4dfd1a78cfb3194b07dc557c0a76d0395108b1195fb7487f7c7637fab66

Shared blocker observed while validating rebased #434 on top of this merge:

  • purchase creation/publishing succeeded, but actual paid inference execution still returned 503 Payment verification failed
  • verifier logs showed:
x402: facilitator verify error: facilitator verify failed (400): invalid_format

Because #434 was rebased directly onto 620a267, I am treating that paid-route failure as a current-main/shared x402 path issue rather than something isolated to the host-CLI change in #434.

@bussyjd bussyjd merged commit 620a267 into main May 6, 2026
6 checks passed
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