fix(packages.nu): add service_books + service_office to starts_extra #240

Merged

mik-tf merged 1 commit from development_mik_books_office_starts_extra into development

2026-05-08 15:22:28 +00:00

mik-tf commented

2026-05-08 15:00:26 +00:00

Owner

Summary

service_complete was registering+starting only 16 of the 18 services that have start commands accepting --download/--version. Both service_books and service_office have the install closure (lines 79 and 90) but were missing from the start closure list.

hero_books on the demo VM was install+started by hand (s76 #3 carryover, deferred through s77).
hero_office quietly never appeared in proc service list after service_complete --download despite having a clean v0.1.0-rc2 Forgejo release with 3 musl assets.

Why both in one PR (vs the s78 plan that called for books only)

Same one-line change, same failure mode, same fix. Adding service_office here closes the same class of gap and unblocks the from-nothing 18/18 target for session-78 DO re-validation under home#230 phase 5. If you'd rather split them, happy to revert office to a separate one-line PR.

Test plan

nu -c "use nutools/modules/services/packages.nu *" — parses.
On a fresh VM: service_complete --download --reset produces 18 entries in proc service list (vs 16 today). To be checked during phase 5 re-validation.
Both service_books install --download and service_office install --download already smoke-tested green on heroci in s52 (books) and s68 (office); only the orchestrator wiring was missing.

Refs

home#230 — workspace skill-alignment audit.
s76 carry-forward: hero_books starts_extra was deferred from session 76 item 3 through session 77.
hero_books v0.1.6-rc1
hero_office v0.1.0-rc2

## Summary `service_complete` was registering+starting only 16 of the 18 services that have `start` commands accepting `--download/--version`. Both `service_books` and `service_office` have the install closure (lines 79 and 90) but were missing from the start closure list. - `hero_books` on the demo VM was install+started by hand (s76 #3 carryover, deferred through s77). - `hero_office` quietly never appeared in `proc service list` after `service_complete --download` despite having a clean `v0.1.0-rc2` Forgejo release with 3 musl assets. ## Why both in one PR (vs the s78 plan that called for books only) Same one-line change, same failure mode, same fix. Adding `service_office` here closes the same class of gap and unblocks the from-nothing 18/18 target for session-78 DO re-validation under [home#230](https://forge.ourworld.tf/lhumina_code/home/issues/230) phase 5. If you'd rather split them, happy to revert office to a separate one-line PR. ## Test plan - [x] `nu -c "use nutools/modules/services/packages.nu *"` — parses. - [ ] On a fresh VM: `service_complete --download --reset` produces 18 entries in `proc service list` (vs 16 today). To be checked during phase 5 re-validation. - [ ] Both `service_books install --download` and `service_office install --download` already smoke-tested green on heroci in s52 (books) and s68 (office); only the orchestrator wiring was missing. ## Refs - [home#230](https://forge.ourworld.tf/lhumina_code/home/issues/230) — workspace skill-alignment audit. - s76 carry-forward: hero_books `starts_extra` was deferred from session 76 item 3 through session 77. - [hero_books v0.1.6-rc1](https://forge.ourworld.tf/lhumina_code/hero_books/releases/tag/v0.1.6-rc1) - [hero_office v0.1.0-rc2](https://forge.ourworld.tf/lhumina_code/hero_office/releases/tag/v0.1.0-rc2)

mik-tf added 1 commit

2026-05-08 15:00:26 +00:00

fix(packages.nu): add service_books + service_office to starts_extra

Build and Publish Skills / build-and-publish (pull_request) Failing after 3s

Details

2e1d5cb5ac

`service_complete` was registering+starting only 16 of the 18 services
that have `start` commands accepting `--download/--version`. Both
service_books and service_office have the install closure (lines 79
and 90) but were missing from the start closure list, so:

  - hero_books on the demo VM was install+started by hand (s76 #3
    carryover)
  - hero_office quietly never appeared in `proc service list` after
    `service_complete --download` despite having a clean v0.1.0-rc2
    Forgejo release with 3 musl assets

Both services have CI-built releases with target-triple-named assets
and accept the same `--reset / --update / --download / --version /
--target` shape as service_biz / service_slides / service_voice.

Closes the 16/17 → 18/18 gap blocking session-78 DO from-nothing
re-validation under home#230 phase 5.

Refs:
- lhumina_code/home#230
- https://forge.ourworld.tf/lhumina_code/hero_books/releases/tag/v0.1.6-rc1
- https://forge.ourworld.tf/lhumina_code/hero_office/releases/tag/v0.1.0-rc2

Signed-off-by: mik-tf