Skip to content

fix(run): honor the end-of-options separator - #159

Merged
ralyodio merged 1 commit into
moshcoder:mainfrom
phucnguyen1707:fix/run-end-of-options
Jul 31, 2026
Merged

fix(run): honor the end-of-options separator#159
ralyodio merged 1 commit into
moshcoder:mainfrom
phucnguyen1707:fix/run-end-of-options

Conversation

@phucnguyen1707

Copy link
Copy Markdown
Contributor

Summary

  • stop parsing runner options after --
  • preserve option-like arguments in the script's argv
  • keep CLI and TUI /run behavior aligned

Tests

  • node --test --test-name-pattern 'run preserves option-like script args after --|TUI /run preserves option-like script args after --|run rejects unknown options|positional args after the script|TUI /run rejects unknown options' test/run-options.test.mjs test/tui.test.mjs

@ralyodio
ralyodio merged commit abfc582 into moshcoder:main Jul 31, 2026
3 checks passed
@ralyodio ralyodio mentioned this pull request Aug 1, 2026
ralyodio added a commit that referenced this pull request Aug 1, 2026
install.sh resolves releases/latest, so the sixteen commits merged since
v0.13.3 have been sitting on main unreachable — including a fix for a page
that locks browsers up.

The headline is the pit. /pit rendered every ending an account held and a form
per name under each, with no bound on either: at 50 endings x 100 names that
was 3.1 MiB of HTML and 36,082 DOM elements, and it managed to jam a browser
with no script on the page at all (#167). It now draws a window and says what
it is not drawing — 173 KiB, 1,926 elements — with a filter box over the top
that takes `eggs` as a substring and `def*` as a glob, debounced against the
API (#168).

The namespace also stopped being the one part of the product a script could
not touch: /api/moshpit/* now accepts the same API key /api/me and
/api/sessions already did (#169), and /pit/dns finally documents the
TronBrowser route for machines whose DNS is not theirs to change (#165).

  moshcode: foreign keys are enforced, and the licence package.json
  claims actually ships (#154)
  cli: help aliases exit 0 (#157), invalid integration commands fail (#160),
  `--` is honoured (#159), a BOM before a shebang no longer breaks (#158)
  skills: engines with no skills primitive are reported, not dropped (#166);
  `--name` requires a value (#156)
  mcp: an unsupported flag is rejected rather than registered as the server
  name (#164)
  pit: the namespace rules are vendored again with a drift test holding them
  to the published package (#161, #162, #163)

Co-authored-by: Claude Opus 5 (1M context) <noreply@anthropic.com>
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