Skip to content

test(cli): cover stdin marker before options - #358

Merged
ralyodio merged 1 commit into
moshcoder:mainfrom
aiirvizionz:fix/run-stdin-marker
Aug 10, 2026
Merged

test(cli): cover stdin marker before options#358
ralyodio merged 1 commit into
moshcoder:mainfrom
aiirvizionz:fix/run-stdin-marker

Conversation

@aiirvizionz

@aiirvizionz aiirvizionz commented Aug 10, 2026

Copy link
Copy Markdown
Contributor

Summary

  • add regression coverage for the documented run - stdin marker when followed by --dry-run
  • preserve the existing stdin coverage and option-validation behavior

The parser fix is already present on current upstream main; this PR contributes the missing regression test rather than duplicating that implementation.

Verification

  • node --test --test-name-pattern="run - reads the script from stdin|run accepts - before --dry-run" test/run-options.test.mjs (2 passing)
  • git diff --check
  • full suite: 164 passing, with five pre-existing Windows/POSIX sandbox failures unrelated to this test

Please merge if this test-only coverage is useful; I will invoice only after merge.

@aiirvizionz
aiirvizionz force-pushed the fix/run-stdin-marker branch from fba0365 to c645415 Compare August 10, 2026 00:54
@aiirvizionz aiirvizionz changed the title fix(cli): accept stdin script marker test(cli): cover stdin marker before options Aug 10, 2026
@aiirvizionz

Copy link
Copy Markdown
Contributor Author

Ready for review. The upstream parser fix is already present; this PR contributes the missing regression coverage for the documented stdin marker before --dry-run. Directed tests pass and the PR records the pre-existing full-suite environment failures. I will invoice only after merge under the accepted task.

@ralyodio
ralyodio merged commit 0eaafa0 into moshcoder:main Aug 10, 2026
4 checks passed
@ralyodio ralyodio mentioned this pull request Aug 10, 2026
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.

2 participants