[META] Migrate all hero_*_ui producers to _admin convention to align with hero_router SSOT #228
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Context
hero_router(the Hero stack's traffic gateway and emerging service-discovery SSOT) ships acrates/hero_router/src/service_manager/services/<service>.rsdefinition for every Hero service. All 22 of those service definitions already expect_adminbinaries andadmin.socksockets — that's the canonical convention.The skill at
~/.claude/skills/hero_service_test/SKILL.mdcodifies the same convention:hero_${name}_adminadmin.sockhero_${name}_webweb.sockhero_${name}_serverrpc.sockhero_${name}_uiis the legacy admin name and should be migrated everywhere.Current state (audit 2026-05-07, session 76)
_admin(matches hero_router)hero_db(PR lhumina_code/hero_db#29)_adminAND_ui)hero_books_uiConsumer
service_*.numodules inhero_skillsmostly track producer reality (most referenceweb.sock, notadmin.sock). One was caught out:lhumina_code/hero_skills@cc56069revertedservice_proc.nufrom_admin/admin.sockback to_ui/ui.sockbecause the producer hadn't migrated yet.Strategic shape
The migration must be producer-first, consumer-follow:
[package].name/[lib].name/[[bin]].nameto_admin, change socket bindingui.sock/web.sock→admin.sock, rename env vars (e.g.HERO_X_UI_BIND→HERO_X_ADMIN_BIND), updatekill_othersocket lists, updatecrates/hero_${name}_appWASM URL bases (if any), update repo docs (PURPOSE.md, in-repo CLAUDE.md, README.md, specs).vX.Y.0-rc1matching workspace version).service_*.nu): bumpSVX_BINARIES, healthcheck/kill_other paths, dispatcher gap.Reference implementation: hero_db PR #29 at lhumina_code/hero_db#29 . 34-file diff with explicit before/after for every layer, builds + clippy + fmt clean.
Migration plan (one repo per session, in dependency order)
service_proc.nuun-revert._adminand_ui); finish the migration.What this issue tracks
Out of scope (separate)
Refs
~/.claude/skills/hero_service_test/SKILL.md§2 (naming) + §3 (sockets)lhumina_code/hero_skills@cc56069(consumer revert because producer wasn't migrated)Signed-off-by: mik-tf
Closure path consolidated into home#230. #230 Phase 1 completes the partial migrations (hero_voice / hero_proxy / hero_whiteboard / hero_osis —
[[bin]]renamed but[package].name+ crate dir still on_ui). #230 Phase 3 audits the outliers (hero_proc_lib_ui,hero_embeddermissing_admin,hero_foundry_uistandalone). When #230 closes, this issue closes.Closing as substantively superseded.
The
_ui→_adminrollout is now tracked under home#230 Phase 1 (4 partial-migration repos: hero_voice / hero_proxy / hero_whiteboard / hero_osis where upstream flipped[[bin]] namebut left[package].name+ crate dir on_ui) and Phase 3 (outliers: hero_proc_lib_ui rename, hero_embedder missing_admin, hero_foundry_ui standalone via home#229 — survives separately).Session 77 completed the bulk sweep (19 producer migrations + 20 tags). What remained — the 4 partial migrations + 3 outliers — is fully covered by home#230's phase 1 + phase 3 with concrete per-repo work items.
Session 80 also locked D-08 (decisions/D-08-hero_builder-as-canonical-build-tool.md): subsequent rename PRs go through
hero_buildersource-build gate before merge.Tracking continues at home#230. Sub-issue home#229 (hero_foundry_ui standalone-repo fate) remains open as its own decision.