equalFlowControl: mirror the optimalControl dispatch reorder
The priority-control codepath had the same stale dispatch shape that caused the live deadlock in optimalControl: only handling idle and operational states, and chaining flowmovement after execsequence startup. Aligns it with the optimalControl fix so a future mode switch to prioritycontrol doesn't reintroduce the bug. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
This commit is contained in:
@@ -1121,15 +1121,17 @@ class MachineGroup {
|
||||
this.logger.debug(this.machines[machineId].state);
|
||||
const currentState = this.machines[machineId].state.getCurrentState();
|
||||
|
||||
if (flow <= 0 && (currentState === "operational" || currentState === "accelerating" || currentState === "decelerating")) {
|
||||
await machine.handleInput("parent", "execsequence", "shutdown");
|
||||
}
|
||||
else if (currentState === "idle" && flow > 0) {
|
||||
// Same dispatch shape as optimalControl — see the comment
|
||||
// there for the rationale. flowmovement BEFORE startup so
|
||||
// concurrent retargets can update delayedMove without a
|
||||
// stale chained flowmovement overwriting it after startup.
|
||||
if (flow > 0) {
|
||||
await machine.handleInput("parent", "flowmovement", this._canonicalToOutputFlow(flow));
|
||||
if (currentState === "idle") {
|
||||
await machine.handleInput("parent", "execsequence", "startup");
|
||||
await machine.handleInput("parent", "flowmovement", this._canonicalToOutputFlow(flow));
|
||||
}
|
||||
else if (currentState === "operational" && flow > 0) {
|
||||
await machine.handleInput("parent", "flowmovement", this._canonicalToOutputFlow(flow));
|
||||
} else if (currentState === "operational" || currentState === "accelerating" || currentState === "decelerating") {
|
||||
await machine.handleInput("parent", "execsequence", "shutdown");
|
||||
}
|
||||
}));
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user