Add 'mesh-llm stop' + clean up --help - #21
Merged
Conversation
michaelneale
changed the title
ndizazzo
added a commit
that referenced
this pull request
Aug 20, 2026
…ntory doc
Container jobs resolve the default run: shell to sh -e {0}, not bash -e
{0} — bare-metal Linux/macOS runners default to bash, so this only changes
once a job gains a container: block. Two chunk-5b steps hit this:
- ci-web-slice.yml's ui_e2e preflight used a <<< here-string, which dash
rejects with "Syntax error: redirection unexpected".
- website-pages.yml's Stage Pages artifact used set -euo pipefail, which
dash has no -o pipefail for. This job is unreachable pre-merge
(push-to-main, path-filtered) and would have first failed on the next
main push touching website/** or crates/**, taking the Pages deploy down.
Both now declare shell: bash. Added
test_ci_workflow_container_shell_contract.py, a static sweep for bashisms
in container-job run: steps that have not declared shell: bash — same
blind spot as the ternary contract test, since actionlint's shellcheck
integration assumes bash.
Also fixes three defects in current-inventory.md's containerization
section: mesh-llm-runner-images-public-web does not exist as a separate
repo (the public web backend lives on mesh-llm-runner-images main, #20 and
#21); documents the Playwright-pin/@playwright-test coupling's actual
failure mode and the four-step cross-repo bump sequence; fixes five
table rows that were missing their Job(s) cell; and records the sh-vs-bash
property itself as a named convention.
ndizazzo
added a commit
that referenced
this pull request
Aug 20, 2026
…ntory doc
Container jobs resolve the default run: shell to sh -e {0}, not bash -e
{0} — bare-metal Linux/macOS runners default to bash, so this only changes
once a job gains a container: block. Two chunk-5b steps hit this:
- ci-web-slice.yml's ui_e2e preflight used a <<< here-string, which dash
rejects with "Syntax error: redirection unexpected".
- website-pages.yml's Stage Pages artifact used set -euo pipefail, which
dash has no -o pipefail for. This job is unreachable pre-merge
(push-to-main, path-filtered) and would have first failed on the next
main push touching website/** or crates/**, taking the Pages deploy down.
Both now declare shell: bash. Added
test_ci_workflow_container_shell_contract.py, a static sweep for bashisms
in container-job run: steps that have not declared shell: bash — same
blind spot as the ternary contract test, since actionlint's shellcheck
integration assumes bash.
Also fixes three defects in current-inventory.md's containerization
section: mesh-llm-runner-images-public-web does not exist as a separate
repo (the public web backend lives on mesh-llm-runner-images main, #20 and
#21); documents the Playwright-pin/@playwright-test coupling's actual
failure mode and the four-step cross-repo bump sequence; fixes five
table rows that were missing their Job(s) cell; and records the sh-vs-bash
property itself as a named convention.
ndizazzo
added a commit
that referenced
this pull request
Aug 20, 2026
…ntory doc
Container jobs resolve the default run: shell to sh -e {0}, not bash -e
{0} — bare-metal Linux/macOS runners default to bash, so this only changes
once a job gains a container: block. Two chunk-5b steps hit this:
- ci-web-slice.yml's ui_e2e preflight used a <<< here-string, which dash
rejects with "Syntax error: redirection unexpected".
- website-pages.yml's Stage Pages artifact used set -euo pipefail, which
dash has no -o pipefail for. This job is unreachable pre-merge
(push-to-main, path-filtered) and would have first failed on the next
main push touching website/** or crates/**, taking the Pages deploy down.
Both now declare shell: bash. Added
test_ci_workflow_container_shell_contract.py, a static sweep for bashisms
in container-job run: steps that have not declared shell: bash — same
blind spot as the ternary contract test, since actionlint's shellcheck
integration assumes bash.
Also fixes three defects in current-inventory.md's containerization
section: mesh-llm-runner-images-public-web does not exist as a separate
repo (the public web backend lives on mesh-llm-runner-images main, #20 and
#21); documents the Playwright-pin/@playwright-test coupling's actual
failure mode and the four-step cross-repo bump sequence; fixes five
table rows that were missing their Job(s) cell; and records the sh-vs-bash
property itself as a named convention.
ndizazzo
added a commit
that referenced
this pull request
Aug 20, 2026
…ntory doc
Container jobs resolve the default run: shell to sh -e {0}, not bash -e
{0} — bare-metal Linux/macOS runners default to bash, so this only changes
once a job gains a container: block. Two chunk-5b steps hit this:
- ci-web-slice.yml's ui_e2e preflight used a <<< here-string, which dash
rejects with "Syntax error: redirection unexpected".
- website-pages.yml's Stage Pages artifact used set -euo pipefail, which
dash has no -o pipefail for. This job is unreachable pre-merge
(push-to-main, path-filtered) and would have first failed on the next
main push touching website/** or crates/**, taking the Pages deploy down.
Both now declare shell: bash. Added
test_ci_workflow_container_shell_contract.py, a static sweep for bashisms
in container-job run: steps that have not declared shell: bash — same
blind spot as the ternary contract test, since actionlint's shellcheck
integration assumes bash.
Also fixes three defects in current-inventory.md's containerization
section: mesh-llm-runner-images-public-web does not exist as a separate
repo (the public web backend lives on mesh-llm-runner-images main, #20 and
#21); documents the Playwright-pin/@playwright-test coupling's actual
failure mode and the four-step cross-repo bump sequence; fixes five
table rows that were missing their Job(s) cell; and records the sh-vs-bash
property itself as a named convention.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
mesh-llm stopsubcommand (was referenced in goose launcher but didn't exist)--help: hide advanced/niche options, show them via--help-advanceddropandrotate-keycommands from default help