Skip to content

feat(registry-broker): agent-network owner reads + push subscriptions - #183

Merged
kantorcodes merged 1 commit into
mainfrom
feat/agent-network-subscriptions
Oct 1, 2026
Merged

kantorcodes merged 1 commit into
mainfrom
feat/agent-network-subscriptions

Conversation

@kantorcodes

Copy link
Copy Markdown
Member

Summary

  • Correct agent-network client schema drift against the merged broker contract: providers (api/other, drop generic), connection states (polling_verified/push_verified/needs_reconnect/paused/degraded), receive modes (none/poll/push), hol-agent-message/1 literal schemaVersion, reply outcomes (answered/partial/refused), separate registration-result states
  • Fix listRuntimes returning parsed.data.runtime instead of parsed.data.runtimes
  • Owner-plane reads: listRuntimeInbox, getRuntimeConversation (owner credentials, no bot grant token)
  • Push subscriptions: createAgentSubscription / listAgentSubscriptions / revokeAgentSubscription matching broker routes from hashgraph-online/registry-broker#256; signingSecret is send-only and never parsed from responses
  • Export new schemas/types from the client barrel

Test plan

  • jest __tests__/services/registry-broker-agent-network.test.ts — 19/19 pass (new owner-inbox, conversation, and subscription lifecycle tests assert exact paths, auth plane, and secret non-exposure)
  • tsc --noEmit clean
  • pnpm run build — cjs/es/browser bundles + declarations built; runtime export check passes
  • eslint on changed files — only pre-existing warnings remain

Generated with Devin

- Fix schema drift vs merged broker contract (providers, connection/
  registration states, receive modes, hol-agent-message/1 literal,
  reply outcomes)
- Fix listRuntimes returning parsed.data.runtime instead of runtimes
- Owner-plane inbox + conversation reads (x-api-key/session auth)
- Push subscription create/list/revoke methods; secrets send-only

Signed-off-by: Michael Kantor <6068672+kantorcodes@users.noreply.github.com>

@zerocodefast zerocodefast 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.

Schema parity with merged broker contract verified; owner-plane reads and subscription lifecycle covered by 19 focused tests; secret is send-only.

@kantorcodes
kantorcodes merged commit 3798053 into main Oct 1, 2026
10 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