service.toml is missing [[env]] PATH_ROOT block, panics when supervised by hero_proc #110

New issue

Closed

opened 2026-05-23 03:33:11 +00:00 by mik-tf · 2 comments

mik-tf commented

2026-05-23 03:33:11 +00:00

Owner

When hero_router is launched through hero_proc (rather than directly with the env preloaded), it panics with 'PATH_ROOT is not set' because its service.toml does not include the env blocks that propagate PATH_ROOT / PATH_VAR / PATH_BUILD / PATH_CODE / HERO_SOCKET_DIR from the supervisor. The fix matches the pattern in hero_assistance / hero_proc_admin / hero_cockpit / hero_proc itself: add the canonical env entries in service.toml.

When hero_router is launched through hero_proc (rather than directly with the env preloaded), it panics with 'PATH_ROOT is not set' because its service.toml does not include the [[env]] blocks that propagate PATH_ROOT / PATH_VAR / PATH_BUILD / PATH_CODE / HERO_SOCKET_DIR from the supervisor. The fix matches the pattern in hero_assistance / hero_proc_admin / hero_cockpit / hero_proc itself: add the canonical [[env]] entries in service.toml.

mik-tf referenced this issue from lhumina_code/hero_assistance

2026-05-23 03:39:24 +00:00

[META] Align hero_assistance with the canonical Hero service template #15

mik-tf referenced this issue from a commit

2026-05-23 04:13:56 +00:00

docs(channels/free): s146 verification pass — flip statuses against live local cockpit

mik-tf referenced this issue from a commit

2026-05-23 04:47:09 +00:00

fix(service.toml): add [[env]] PATH_ROOT/HERO_SOCKET_DIR/RUST_LOG block

mik-tf closed this issue

2026-05-23 04:47:10 +00:00

mik-tf commented

2026-05-23 04:48:50 +00:00

Author

Owner

Fixed in 175fff0 on development. Appended PATH_ROOT, HERO_SOCKET_DIR, and RUST_LOG [[env]] blocks to crates/hero_router/service.toml matching the canonical 3-entry pattern from hero_cockpit_server/service.toml. Verified live: lab service hero_router --install --start re-registers the action spec, supervised launch binds 127.0.0.1:9988, /health returns 200, all six lab smoke tests pass.

Fixed in [`175fff0`](https://forge.ourworld.tf/lhumina_code/hero_router/commit/175fff0e12cecdc22ff4e107c724b0a5b5139e8c) on `development`. Appended PATH_ROOT, HERO_SOCKET_DIR, and RUST_LOG `[[env]]` blocks to `crates/hero_router/service.toml` matching the canonical 3-entry pattern from `hero_cockpit_server/service.toml`. Verified live: lab service hero_router --install --start re-registers the action spec, supervised launch binds 127.0.0.1:9988, /health returns 200, all six lab smoke tests pass.

mik-tf referenced this issue from a commit

2026-05-23 05:13:00 +00:00

docs(channels/free): s147 fix pass — flip e2e_checklist rationales after hero_cockpit#6 + hero_router#110 closure

mik-tf commented

2026-05-23 12:16:56 +00:00

Author

Owner

Followup note for the record: this same service.toml [[env]] gap was independently rediscovered yesterday (2026-05-22) from the supervised hero_router launch path, hit the canonical Lesson 17 trap, and was fixed today (2026-05-23) with the same 3-entry [[env]] block (PATH_ROOT, HERO_SOCKET_DIR, RUST_LOG). Same root cause from two different code paths (this issue caught it from hero_assistance smoke through the router on the customer side; the other path caught it from the supervised launch silent panic on the operator side), same fix. Logging the convergence so future readers can find the matched-fix pair.

Followup note for the record: this same `service.toml` `[[env]]` gap was independently rediscovered yesterday (2026-05-22) from the supervised hero_router launch path, hit the canonical Lesson 17 trap, and was fixed today (2026-05-23) with the same 3-entry `[[env]]` block (`PATH_ROOT`, `HERO_SOCKET_DIR`, `RUST_LOG`). Same root cause from two different code paths (this issue caught it from `hero_assistance` smoke through the router on the customer side; the other path caught it from the supervised launch silent panic on the operator side), same fix. Logging the convergence so future readers can find the matched-fix pair.

mik-tf referenced this issue from a commit

2026-05-23 12:20:58 +00:00

docs(e2e_checklist): audit-log entry for first post-D-26 router-fronted walkthrough

mik-tf referenced this issue from a commit

2026-05-23 12:54:23 +00:00

fix(service.toml): add [[env]] PATH_ROOT/HERO_SOCKET_DIR/RUST_LOG block