Skip to content

feat(simulation): add explicit post-stabilization hook at end of play() #935

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
yashveer-1 wants to merge 2 commits into
base: main
Choose a base branch
from
Open

feat(simulation): add explicit post-stabilization hook at end of play() #935

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
cea94ee
feat(simulation): add explicit post-stabilization hook at end of play()
yashveer-1 Feb 13, 2026
63627c7
fix(simulation): fire stabilization hook only on clean queue drain
yashveer-1 Feb 13, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
23 changes: 23 additions & 0 deletions src/simulator/src/engine.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -440,6 +440,7 @@ export function play(scope = globalScope, resetNodes = false) {
}
}
// Check for Contentions
let hadContention = false;
if (simulationArea.contentionPending.size() > 0) {
for (const [ourNode, theirNode] of simulationArea.contentionPending.nodes()) {
ourNode.highlighted = true;
Expand All @@ -448,13 +449,35 @@ export function play(scope = globalScope, resetNodes = false) {

forceResetNodesSet(true);
showError('Contention Error: One or more bus contentions in the circuit (check highlighted nodes)');
hadContention = true;
}
// Post-stabilization hook: only when queue drained cleanly
if (!hadContention) {
onSimulationStabilized(scope);

}
}

export function resetNodeHighlights(scope) {
for (const node of scope.allNodes) node.highlighted = false;
}

/**
* Internal hook invoked once after the simulation queue has been fully drained
* and the circuit has reached a stabilized state for the current tick.
*
* This provides an explicit post-tick boundary for future debugging,
* deterministic snapshotting, and runtime instrumentation layers.
*
* The function is intentionally a no-op and does not modify simulation behavior.
*
* @param {Scope} scope
*/
function onSimulationStabilized(scope) {
// intentionally empty
}


/**
* Function to check for any UI update, it is throttled by time
* @param {number=} count - this is used to force update
Expand Down