Skip to content

refactor(supervisor): converge boot and restart load pipelines - #100

Merged
mfw78 merged 4 commits into
mainfrom
seam/c5-load-convergence
Aug 6, 2026
Merged

refactor(supervisor): converge boot and restart load pipelines#100
mfw78 merged 4 commits into
mainfrom
seam/c5-load-convergence

Conversation

@mfw78

@mfw78 mfw78 commented Aug 6, 2026

Copy link
Copy Markdown
Contributor

What

Converges the boot and restart load pipelines for modules and providers into shared
helpers instead of duplicated skeletons: a shared store-build helper parameterised by
role, shared instantiate/install helpers reused by both the initial load path and the
lifecycle revive path, and a merged Seed type replacing the separate ModuleSeed and
ProviderSeed literals.

Confines run_init to the instantiate seam, drops the boot-only refuel that the dispatch
path already repeats, and adds a regression test pinning the boot-kills vs
restart-defers init verdict so the converged pipelines keep that distinction visible at
the call sites.

Why

Part of #94

The boot and restart load paths carried twin implementations that had drifted apart in
small, hard-to-audit ways (duplicated store-build logic, duplicated seed literals, a
refuel step only one path needed). Converging them onto shared helpers removes that
duplication while keeping the one deliberate behavioural difference, dead-permanent
vs restart-defers on init, explicit at both call sites instead of buried in copies.

Testing

  • cargo nextest run -p nexum-runtime supervisor: 94 passed, 0 failed, including the_same_init_fault_kills_at_boot_and_only_defers_on_restart pinning the boot-kills vs restart-defers verdict.

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/c5-load-convergence branch from c84f78c to c19a510 Compare August 6, 2026 07:32
@mfw78
mfw78 force-pushed the seam/c5-load-convergence branch from c19a510 to eb1478c Compare August 6, 2026 07:36
@mfw78
mfw78 changed the base branch from seam/c4-role to main August 6, 2026 07:50
mfw78 added 4 commits August 6, 2026 07:50
AI Assistance: Claude (Fable 5) used for implementation and test updates
…eats

AI Assistance: Claude (Fable 5) used for implementation and test updates
AI Assistance: Claude (Opus 5) used for red-team review and test authoring
AI Assistance: Claude (Opus 5) used for red-team review and the visibility fix
@mfw78
mfw78 force-pushed the seam/c5-load-convergence branch from eb1478c to 5c03068 Compare August 6, 2026 07:51
@mfw78
mfw78 merged commit 7fc8168 into main Aug 6, 2026
4 checks passed
@mfw78
mfw78 deleted the seam/c5-load-convergence branch August 6, 2026 07:58
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