Skip to content

fix(runner): bundle src/watch.sh into build so watch subcommand works#736

Merged
Chemaclass merged 2 commits into
mainfrom
fix/735-watch-missing-from-build
Jun 26, 2026
Merged

fix(runner): bundle src/watch.sh into build so watch subcommand works#736
Chemaclass merged 2 commits into
mainfrom
fix/735-watch-missing-from-build

Conversation

@Chemaclass

Copy link
Copy Markdown
Member

🤔 Background

Related #735

The distributable single-file bashunit referenced bashunit::watch::run but never defined it, so bashunit watch failed with command not found in every release since 0.34.0. Same class as the 0.31.0 bench bug: src/watch.sh was sourced by the dev entrypoint but omitted from the build dependency list.

💡 Changes

  • Add src/watch.sh to build::dependencies so the watch functions are bundled into the built binary.
  • Add a guard test asserting every src file sourced by the dev entrypoint is present in the build, preventing this whole class of regression.
  • Add a smoke test confirming the built binary defines bashunit::watch::run.

The distributable single-file binary called bashunit::watch::run but never
defined it: build::dependencies omitted src/watch.sh, and the build strips
all source lines. Add watch.sh to the dependency list and guard the whole
class of regression with a build test asserting every src file sourced by
the dev entrypoint is bundled, plus a smoke test on the built binary.

Closes #735
@Chemaclass Chemaclass added the bug Something isn't working label Jun 26, 2026
@Chemaclass Chemaclass self-assigned this Jun 26, 2026
@Chemaclass Chemaclass enabled auto-merge (squash) June 26, 2026 11:30
@Chemaclass Chemaclass merged commit 6ed8ad6 into main Jun 26, 2026
31 checks passed
@Chemaclass Chemaclass deleted the fix/735-watch-missing-from-build branch June 26, 2026 11:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants