feat(service_aibroker): add --from-ci install path #196

Merged
mik-tf merged 1 commit from development_mik_from_ci_aibroker into development 2026-05-03 17:59:39 +00:00
Owner

Phase 1 progress on hero_demo#54.

Mirrors the pattern shipped in #195 (router/proxy/db/indexer) and the original pilot #193. Asset suffix linux-amd64-musl matches hero_aibroker's release.yaml matrix.

Why this is a separate PR

The v0.1.0 release (2026-04-02) predates hero_aibroker_services (renamed from hero_broker_server in commit 591e071), so wiring --from-ci against the latest release would 404. Required:

  1. hero_aibroker#57 — relaxed the release.yaml gate to allow tagging on development (mirrors hero_proxy). Merged.
  2. hero_aibroker v0.1.1 — fresh tag at the development tip, published the 4 expected binaries via release.yaml. Live.
  3. This PR — one-line consumer wiring.

Smoke test on heroci

hero_aibroker: fetching release v0.1.1 from lhumina_code/hero_aibroker...
  hero_aibroker-linux-amd64-musl           -> /root/hero/bin/hero_aibroker
  hero_aibroker_server-linux-amd64-musl    -> /root/hero/bin/hero_aibroker_server
  hero_aibroker_ui-linux-amd64-musl        -> /root/hero/bin/hero_aibroker_ui
  hero_aibroker_services-linux-amd64-musl  -> /root/hero/bin/hero_aibroker_services
  hero_aibroker installed from CI artifacts (release v0.1.1)

File check on the largest binary: ELF 64-bit LSB pie executable, x86-64, version 1 (SYSV), static-pie linked, stripped. CLI launches and parses args.

Tangentially observed

hero_aibroker#58test_server_rpc_methods fails on every build.yaml run because the test spawns hero_aibroker_server which needs a live hero_db. Pre-existing, not in this PR's scope; filed with two recommended fixes (ignore-by-default, or start hero_db in the CI job).

Signed-off-by: mik-tf

Phase 1 progress on [hero_demo#54](https://forge.ourworld.tf/lhumina_code/hero_demo/issues/54). Mirrors the pattern shipped in [#195](https://forge.ourworld.tf/lhumina_code/hero_skills/pulls/195) (router/proxy/db/indexer) and the original pilot [#193](https://forge.ourworld.tf/lhumina_code/hero_skills/pulls/193). Asset suffix `linux-amd64-musl` matches `hero_aibroker`'s `release.yaml` matrix. ## Why this is a separate PR The v0.1.0 release (2026-04-02) predates `hero_aibroker_services` (renamed from `hero_broker_server` in commit 591e071), so wiring `--from-ci` against the latest release would 404. Required: 1. [hero_aibroker#57](https://forge.ourworld.tf/lhumina_code/hero_aibroker/pulls/57) — relaxed the release.yaml gate to allow tagging on development (mirrors hero_proxy). **Merged.** 2. [hero_aibroker v0.1.1](https://forge.ourworld.tf/lhumina_code/hero_aibroker/releases/tag/v0.1.1) — fresh tag at the development tip, published the 4 expected binaries via release.yaml. **Live.** 3. This PR — one-line consumer wiring. ## Smoke test on heroci ``` hero_aibroker: fetching release v0.1.1 from lhumina_code/hero_aibroker... hero_aibroker-linux-amd64-musl -> /root/hero/bin/hero_aibroker hero_aibroker_server-linux-amd64-musl -> /root/hero/bin/hero_aibroker_server hero_aibroker_ui-linux-amd64-musl -> /root/hero/bin/hero_aibroker_ui hero_aibroker_services-linux-amd64-musl -> /root/hero/bin/hero_aibroker_services hero_aibroker installed from CI artifacts (release v0.1.1) ``` File check on the largest binary: `ELF 64-bit LSB pie executable, x86-64, version 1 (SYSV), static-pie linked, stripped`. CLI launches and parses args. ## Tangentially observed [hero_aibroker#58](https://forge.ourworld.tf/lhumina_code/hero_aibroker/issues/58) — `test_server_rpc_methods` fails on every `build.yaml` run because the test spawns `hero_aibroker_server` which needs a live `hero_db`. Pre-existing, not in this PR's scope; filed with two recommended fixes (ignore-by-default, or start hero_db in the CI job). Signed-off-by: mik-tf
feat(service_aibroker): add --from-ci install path
All checks were successful
Build and Publish Skills / build-and-publish (pull_request) Successful in 3s
f4d03d36c7
Mirrors the pattern shipped in #195 (PR-1: router/proxy/db/indexer) and
the original pilot (#193). Asset suffix linux-amd64-musl matches
hero_aibroker's release.yaml matrix.

Smoke-tested against the freshly-cut
https://forge.ourworld.tf/lhumina_code/hero_aibroker/releases/tag/v0.1.1
(which uploaded the 4 expected binaries thanks to the gate fix in
lhumina_code/hero_aibroker#57).

Phase 1 progress on lhumina_code/hero_demo#54.

Signed-off-by: mik-tf
mik-tf merged commit 9cad828f33 into development 2026-05-03 17:59:39 +00:00
Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
lhumina_code/hero_skills!196
No description provided.