Skip to content

fix(sell): enable offers after CLI ERC-8004 register#761

Merged
bussyjd merged 1 commit into
integration/v0.14.0-rc0from
fix/erc8004-register-enable-offers
Jul 15, 2026
Merged

fix(sell): enable offers after CLI ERC-8004 register#761
bussyjd merged 1 commit into
integration/v0.14.0-rc0from
fix/erc8004-register-enable-offers

Conversation

@bussyjd

@bussyjd bussyjd commented Jul 15, 2026

Copy link
Copy Markdown
Contributor

Summary

  1. After successful obol sell register, patch every ServiceOffer with registration.enabled=falsetrue so post-hoc registration activates existing --no-register offers.
  2. Controller: when a shared RegistrationRequest carries an agentId (including after CLI mint), set Registered=True even if phase is empty — fixes non-owner offers stuck Pending and status/checkmarks disagreeing.

Test plan

  • go test ./internal/serviceoffercontroller/ -run TestApplyShared
  • go test ./cmd/obol/ -run TestSell

Part of v0.14.0-rc0 hackathon bugfix train.

obol sell register left ServiceOffers created with --no-register stuck
at registration.enabled=false / Registered=Disabled while the on-chain
AgentIdentity already had an agentId. After a successful register, patch
those offers enabled=true and treat a known agentId as Registered=True
even when RegistrationRequest phase is empty.
@bussyjd
bussyjd merged commit 3e7218a into integration/v0.14.0-rc0 Jul 15, 2026
@bussyjd bussyjd mentioned this pull request Jul 15, 2026
6 tasks
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