Build failure: hero_proc_sdk API breaking changes (lifecycle, ActionBuilder) #9

New issue

Closed

opened 2026-03-30 09:37:35 +00:00 by timur · 2 comments

timur commented

2026-03-30 09:37:35 +00:00

Owner

Problem

hero_inspector fails to compile with 5 errors due to breaking API changes in hero_proc_sdk:

error[E0432]: unresolved import `hero_proc_sdk::lifecycle`
error[E0599]: no method named `interpreter` found for struct `ActionBuilder`
error[E0599]: no method named `is_process` (cascading from interpreter)
error[E0282]: type annotations needed (cascading from above)

Root Cause

hero_proc_sdk removed the lifecycle module and the ActionBuilder::interpreter() / ActionBuilder::is_process() builder methods. The hero_inspector binary still uses the old API.

Fix Required

Remove use hero_proc_sdk::lifecycle — use HeroProcRPCAPIClient methods directly (service_set, service_start, service_stop)
Remove .interpreter("exec") and .is_process() from ActionBuilder chains — set action.interpreter = Some("exec".into()) on the ActionSpec after .build()
The retry_builder type inference errors are cascading and will resolve once the above are fixed

Affected File

crates/hero_inspector/src/bin/hero_inspector.rs

## Problem `hero_inspector` fails to compile with 5 errors due to breaking API changes in `hero_proc_sdk`: ``` error[E0432]: unresolved import `hero_proc_sdk::lifecycle` error[E0599]: no method named `interpreter` found for struct `ActionBuilder` error[E0599]: no method named `is_process` (cascading from interpreter) error[E0282]: type annotations needed (cascading from above) ``` ## Root Cause `hero_proc_sdk` removed the `lifecycle` module and the `ActionBuilder::interpreter()` / `ActionBuilder::is_process()` builder methods. The `hero_inspector` binary still uses the old API. ## Fix Required 1. Remove `use hero_proc_sdk::lifecycle` — use `HeroProcRPCAPIClient` methods directly (`service_set`, `service_start`, `service_stop`) 2. Remove `.interpreter("exec")` and `.is_process()` from `ActionBuilder` chains — set `action.interpreter = Some("exec".into())` on the `ActionSpec` after `.build()` 3. The `retry_builder` type inference errors are cascading and will resolve once the above are fixed ## Affected File `crates/hero_inspector/src/bin/hero_inspector.rs`

timur commented

2026-03-30 09:41:05 +00:00

Author

Owner

Fixed in commit b949355 on development branch.

Changes:

Removed lifecycle module import — replaced with direct RPC calls (action_set, service_set, service_start, service_stop)
Removed .interpreter("exec") and .is_process() from ActionBuilder chains — set interpreter on ActionSpec after .build()
Build now succeeds with only a minor dead-code warning (pinned field)

Fixed in commit b949355 on development branch. Changes: - Removed lifecycle module import — replaced with direct RPC calls (action_set, service_set, service_start, service_stop) - Removed .interpreter("exec") and .is_process() from ActionBuilder chains — set interpreter on ActionSpec after .build() - Build now succeeds with only a minor dead-code warning (pinned field)

timur closed this issue

2026-03-30 09:41:22 +00:00

timur referenced this issue from a commit

2026-03-30 09:41:30 +00:00

fix: update hero_proc_sdk API usage for breaking changes

timur commented

2026-03-30 10:32:52 +00:00

Author

Owner

Previous fix was incorrect (removed lifecycle module usage). Reverted in commit 514f0e1.

Actual fix: the hero_proc_sdk on the remote development branch already has lifecycle, interpreter(), and is_process() — the local checkout was just behind. After syncing hero_proc to origin/development (commit 8994743), hero_inspector compiles cleanly with the original code.

Changes:

Reverted commit b949355 (which incorrectly removed lifecycle usage)
Updated cargo dependency to pull hero_proc_sdk v0.4.1 which has lifecycle, interpreter, and is_process support

Previous fix was incorrect (removed lifecycle module usage). Reverted in commit 514f0e1. Actual fix: the hero_proc_sdk on the remote development branch already has lifecycle, interpreter(), and is_process() — the local checkout was just behind. After syncing hero_proc to origin/development (commit 8994743), hero_inspector compiles cleanly with the original code. Changes: - Reverted commit b949355 (which incorrectly removed lifecycle usage) - Updated cargo dependency to pull hero_proc_sdk v0.4.1 which has lifecycle, interpreter, and is_process support