Skip to content

Wire the ecosystem-alias overlay module and settings [OPL-4463] - #30

Merged
shruggr merged 40 commits into
masterfrom
codex/opl-4463-ecosystemalias-wiring
Sep 5, 2026
Merged

shruggr merged 40 commits into
masterfrom
codex/opl-4463-ecosystemalias-wiring

Conversation

@rohenaz

@rohenaz rohenaz commented Sep 4, 2026

Copy link
Copy Markdown
Contributor

Wire NewLookupService(topicStorage), module routes, capabilities, and runtime settings using the shared ModuleDeps, including Broadcaster and P2PBus. Keep the module disabled by default. Incorporates the event-based lookup, numeric output ordering, and bounded skip/limit fixes.

Validation on the updated wiring: ecosystemalias, overlay/storage, config, cmd/server, admin, and docs/architecture tests pass; server build passes. The event-score regression also ran against real PostgreSQL and SQLite.

@linear

linear Bot commented Sep 4, 2026

Copy link
Copy Markdown

OPL-4463

rohenaz and others added 2 commits September 4, 2026 20:40
…ettings-docs

feat(admin): configure ecosystem alias routes and sync
Drop ea1 cursors and txid sort. Lookup pages with skip/limit; event
order is overlay HeightScore then vout.
@shruggr

shruggr commented Sep 5, 2026

Copy link
Copy Markdown
Collaborator

Rebased onto #27 and pushed wiring that constructs NewLookupService(topicStorage) only. No ClaimStore. Engine still gets the shared ModuleDeps including Broadcaster and P2PBus.

Docs no longer mention ecosystem_alias_claims or ea1. cursors. Lookup is events + skip/limit.

#33 should not nil Broadcaster/P2PBus. Overlay Submit should broadcast like the other modules.

@shruggr
shruggr force-pushed the codex/opl-4463-ecosystemalias-wiring branch from ae9e559 to 557ff88 Compare September 5, 2026 16:08
@shruggr
shruggr force-pushed the codex/opl-4466-ecosystemalias-lookup branch from 64fa72f to 3322a5b Compare September 5, 2026 16:11
@shruggr
shruggr force-pushed the codex/opl-4463-ecosystemalias-wiring branch from 557ff88 to ae9e559 Compare September 5, 2026 16:11
@rohenaz rohenaz changed the title Wire the ecosystem alias module [OPL-4463] Wire the ecosystem-alias overlay module and settings [OPL-4463] Sep 5, 2026
@rohenaz

rohenaz commented Sep 5, 2026

Copy link
Copy Markdown
Contributor Author

Merged the fixes from #23/#27 into your wiring, preserving the rewritten README and shared Broadcaster/P2PBus. Resolved only the documentation conflict; no broadcaster disabling or claims table was restored.

Validation on the updated wiring: ecosystemalias, overlay/storage, config, cmd/server, admin, and docs/architecture tests pass; server build passes. The event-score regression also ran against real PostgreSQL and SQLite.

SDK #41 now uses skip/limit. Sigma follow-up https://github.com/b-open-io/sigma-auth/pull/354 corrects the wallet noSend/overlay-broadcast guarantees.

Topic enumeration is GASP FindUTXOs in ingest order. Lookup is alias and
domain events only. findAll is not in BRC-169.
Every admit also writes event "*". {} or skip/limit only lists live
claims in HeightScore order. Overlay peers still GASP.
@rohenaz

rohenaz commented Sep 5, 2026

Copy link
Copy Markdown
Contributor Author

The new real HTTP lifecycle gate found two integration blockers that the earlier provider tests did not cover:

  1. { "service": "ls_ecosystemalias", "query": {} } returns HTTP 400 before dispatch because go-overlay-services rejects every zero-length map. Upstream fix: fix(server): allow explicit empty lookup query objects bsv-blockchain/go-overlay-services#365 (explicit empty objects accepted; missing/null still rejected).
  2. A populated lookup returns HTTP 500 (FindOutput: topic is required) because engine formula hydration omits the topic. Draft stack Gate alias HTTP lifecycle and fix topic-scoped BEEF hydration [OPL-4473] #34 binds hydration to a single-topic engine's own store and includes a cross-topic regression.

With the pinned dependency plus #365's isolated patch, the full Go suite and server build pass. The new gate exercises real Fiber routes, engine, SQLite, disk BEEF, historical/current modes, conflicts, paging, spend eviction, and restart persistence with synthetic chain dependencies. #34 intentionally remains draft until #365 is included in the dependency pin. My earlier no-new-blockers assessment was limited to the package tests; these are additional public-path blockers.

@shruggr
shruggr changed the base branch from codex/opl-4466-ecosystemalias-lookup to master September 5, 2026 23:51
@shruggr
shruggr merged commit d511f28 into master Sep 5, 2026
1 check 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