Skip to content

refactor(runtime): bundle boot inputs into BootEnv - #98

Merged
mfw78 merged 2 commits into
mainfrom
seam/c3-bootenv
Aug 6, 2026
Merged

refactor(runtime): bundle boot inputs into BootEnv#98
mfw78 merged 2 commits into
mainfrom
seam/c3-bootenv

Conversation

@mfw78

@mfw78 mfw78 commented Aug 6, 2026

Copy link
Copy Markdown
Contributor

What

Bundles the loose boot_single boot inputs (limits, configured chains, digest
requirement) into a single BootEnv struct built via BootEnv::from_config.
wire_extensions gains a with_provider_kinds flag so boot_single can share
it with Supervisor::boot instead of duplicating the Shared construction
inline.

Why

boot_single had grown to six positional arguments derived from the same
EngineConfig, which made call sites easy to get wrong and hard to extend.
Bundling them into BootEnv removes the #[allow(clippy::too_many_arguments)]
escape hatch and gives future boot inputs one place to land.
Part of #94

Testing

  • cargo test -p nexum-runtime supervisor: 90 passed, 0 failed
  • cargo test -p nexum-runtime: 419 passed, 0 failed, plus 1 passing doctest

AI Assistance

Implementation by claude-fable-5, red-team review by claude-opus-5, PR by claude-sonnet-5.

@mfw78
mfw78 force-pushed the seam/c2-builder-tail branch from c29999b to b1c30ad Compare August 6, 2026 07:35
@mfw78
mfw78 force-pushed the seam/c3-bootenv branch from 82c7698 to 59029ef Compare August 6, 2026 07:36
@mfw78
mfw78 changed the base branch from seam/c2-builder-tail to main August 6, 2026 07:39
mfw78 added 2 commits August 6, 2026 07:40
…ensions

AI Assistance: Claude (Fable 5) used for implementation and test updates
Mutation runs on the BootEnv car showed the load-bearing
with_provider_kinds=false in boot_single and the env.configured_chains
wiring were covered by no test: flipping either left the suite green.
Add a serviceless-kind fixture pinning both sides of the flag (boot
refuses, boot_single proceeds to the manifest gate) and a single-boot
chain-gate refusal pinning the configured-chains wiring.

AI Assistance: Claude Fable used for the retroactive review.
@mfw78
mfw78 force-pushed the seam/c3-bootenv branch from 59029ef to 086005f Compare August 6, 2026 07:41
@mfw78
mfw78 merged commit cdcf265 into main Aug 6, 2026
4 checks passed
@mfw78
mfw78 deleted the seam/c3-bootenv branch August 6, 2026 07:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant