Repository navigation
fix: use live ProductionUnit status throughout Phase 5 (#638) - #639
Conversation
zen-edit: applied
From |
zen-edit: applied
From |
Producer handoff — exact-head judgement requestedSLOPSTER ( This bounded repair removes all remaining runtime Focused seed/live-divergence regressions cover reactivated ACTIVE production/transition/coverage/SELL equivalence, non-ACTIVE refusal controls, CLOSING OUTPUT liquidation independent of seed status, and fail-closed ACTIVE input/extraction provenance. Exact-head evidence: CI #1497 passed retained .NET restore/build/test; TypeScript install/typecheck/full tests (963/963)/build; policy guard including handoff scope, issue labels, generated coverage and ledger honesty; mergeability is green. PR is ready for review and remains deliberately unmerged at the producer→judge boundary. Please return a current exact-head |
Verdict: ACCEPTExact head judged: Independent source review found no merge-blocking defect against #638. The six Phase-5 runtime lifecycle gates now use live Exact-head evidence is green: |
Outcome
Repair the post-#636 Phase-5 lifecycle-authority regression so current production behavior is governed only by
ProductionUnitState.status. All six runtime Phase-5 seed-status gates are now live-status gates, CLOSING units can liquidate residual OUTPUT through the existing ordinary SELL path while still realizing zero normal production, and retained Phase-5 test fixtures now isolate units through live lifecycle state rather than mutating immutable seed provenance.Closes #638
Issue and tested revision
51b188a0da2f65bc9a43f2a6fe2a6a68f6ddaab1Changed artifacts
src/simulation/productionExecution.tssrc/simulation/productionExecution.test.tssrc/simulation/productionExecutionCoverage.test.tssrc/simulation/productionExecutionLifecycleAuthority.test.tssrc/simulation/productionExecutionPersistence.test.tssrc/simulation/resourceGenesisAuthority.test.tsAcceptance criteria
unit.seed.statuslifecycle gate remains inproductionExecution.ts.build-and-test,typescript,policy-guard, andmergeabilityare green.Verification
Exact head
51b188a0da2f65bc9a43f2a6fe2a6a68f6ddaab1:build-and-test, CI #1497).npm ci,npm run typecheck, fullnpm test(963/963),npm run build— PASS (typescript, CI #1497).policy-guard— PASS, including handoff scope, Issue label axes, generated coverage, and ledger honesty.mergeability— PASS.Assumptions / unknowns
No specification choice was introduced. CLOSING OUTPUT liquidation uses the already-required ordinary OUTPUT SELL-intent path; PLANNED and MOTHBALLED do not acquire ACTIVE selling privileges in this bounded repair. Immutable
seed.statusremains scenario provenance only.Highest-risk review area
The boundary between zero normal production and residual OUTPUT liquidation for live CLOSING units, plus seed/live divergence in transition coverage and physical provenance validation.
Remaining gate
Current exact-head SLOPSTER judgement. This authoring run will not merge the PR.