Skip to content

fix(moshscript): skip sleep during dry runs#32

Open
padsbanger wants to merge 1 commit into
moshcoder:mainfrom
padsbanger:fix/dry-run-sleep
Open

fix(moshscript): skip sleep during dry runs#32
padsbanger wants to merge 1 commit into
moshcoder:mainfrom
padsbanger:fix/dry-run-sleep

Conversation

@padsbanger

Copy link
Copy Markdown

Summary

  • make sleep(ms) honor the moshscript dry-run contract
  • narrate the intended delay instead of blocking
  • add a regression test that verifies Atomics.wait is not called

This fixes the PRD 0004 R7 dry-run guarantee: previews now avoid the sleep side effect along with process and network effects.

Verification

  • node --test test/commands.test.mjs (13/13 passed)
  • npm test on Node 22 (135/135 passed)

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