Refresh hero_proc_sdk pin in Cargo.lock for JobCreateInput.inputs #35

Merged

mik-tf merged 2 commits from development_refresh_hero_proc_sdk_lock into development

2026-04-27 12:53:52 +00:00

fatmaebrahim commented

2026-04-27 11:20:14 +00:00

Member

Summary

Refreshes the stale hero_proc_sdk pin in Cargo.lock so the workspace builds against the current hero_proc development HEAD (which contains the JobCreateInput.inputs field). Also fixes a one-line pre-existing test compile error that was masked behind the prior make check failure.

Closes #34

Changes

Cargo.lock: cargo update -p hero_proc_sdk re-resolves to current hero_proc development HEAD 9b134018. Cargo also re-resolves hero_rpc_derive and hero_rpc_openrpc to current hero_rpc development HEAD 38a09290 (both are git deps reachable through hero_proc_sdk and branch-tracking, so this is the desired refresh). No manifest edits, no rev = introduced.
crates/osis/src/rpc/request_context.rs: adds the missing hero_context_name: None field to the RequestContext literal in the test_serde_roundtrip test. The struct field was added in commit 7d14133 but the test was never updated; the compile error was masked behind the make check failure addressed by this PR.

Test Results

make check passes — the original error[E0560]: struct JobCreateInput has no field named inputs at crates/service/src/lifecycle.rs:290 is gone.
make test passes — 274 tests passed, 0 failed, 1 ignored across all workspace crates.

## Summary Refreshes the stale `hero_proc_sdk` pin in `Cargo.lock` so the workspace builds against the current `hero_proc` `development` HEAD (which contains the `JobCreateInput.inputs` field). Also fixes a one-line pre-existing test compile error that was masked behind the prior `make check` failure. ## Related Issue Closes #34 ## Changes - `Cargo.lock`: `cargo update -p hero_proc_sdk` re-resolves to current `hero_proc` `development` HEAD `9b134018`. Cargo also re-resolves `hero_rpc_derive` and `hero_rpc_openrpc` to current `hero_rpc` `development` HEAD `38a09290` (both are git deps reachable through `hero_proc_sdk` and branch-tracking, so this is the desired refresh). No manifest edits, no `rev =` introduced. - `crates/osis/src/rpc/request_context.rs`: adds the missing `hero_context_name: None` field to the `RequestContext` literal in the `test_serde_roundtrip` test. The struct field was added in commit `7d14133` but the test was never updated; the compile error was masked behind the `make check` failure addressed by this PR. ## Test Results - `make check` passes — the original `error[E0560]: struct JobCreateInput has no field named inputs` at `crates/service/src/lifecycle.rs:290` is gone. - `make test` passes — 274 tests passed, 0 failed, 1 ignored across all workspace crates.

fatmaebrahim added 1 commit

2026-04-27 11:20:14 +00:00

chore(deps): refresh hero_proc_sdk pin in Cargo.lock for JobCreateInput.inputs

Test / test (pull_request) Failing after 0s

Details

Test / test (push) Successful in 1m7s

Details

ddfa75f33f

Re-resolves the lock against current hero_proc development HEAD so
JobCreateInput.inputs is available; also adds the missing hero_context_name
field to a RequestContext test literal that was masked behind the prior
make check failure.

#34

fatmaebrahim referenced this pull request

2026-04-27 11:20:22 +00:00

CI red on development: stale Cargo.lock pin of hero_proc_sdk #34

fatmaebrahim added 1 commit

2026-04-27 11:42:01 +00:00

style: cargo fmt --all on workspace

Test / test (push) Successful in 1m6s

Details

Test / test (pull_request) Successful in 1m6s

Details

8c8b65a9bb

Cleans up pre-existing formatting drift in generator, openrpc, server, and
recipe_server example so the CI Format step is clean.

fatmaebrahim changed title from ~~chore(deps): refresh hero_proc_sdk pin in Cargo.lock for JobCreateInput.inputs~~ to Refresh hero_proc_sdk pin in Cargo.lock for JobCreateInput.inputs

2026-04-27 11:44:53 +00:00