Skip to content

fix: validate the test preflight wrapper - #2327

Merged
codeforester merged 1 commit into
mainfrom
enhancement/2325-20260919-enhancement-harden-test-preflight-wrapper-validation-and-pat
Sep 19, 2026
Merged

codeforester merged 1 commit into
mainfrom
enhancement/2325-20260919-enhancement-harden-test-preflight-wrapper-validation-and-pat

Conversation

@codeforester

Copy link
Copy Markdown
Collaborator

Summary

  • centralize the project-command wrapper path and executable guard
  • validate the wrapper immediately before the test preflight invocation
  • add focused coverage for a missing or non-executable wrapper

Issue

Fixes #2325

Follow-up to the merged PR #2322 review findings 5256545518.

Validation

  • bats cli/bash/commands/basectl/tests/project-command-helpers.bats cli/bash/commands/basectl/tests/test.bats cli/bash/commands/basectl/tests/manifest-command-trust.bats (37 passed)
  • git diff --check
  • ShellCheck on changed files

Notes

The change is limited to the shared project-command path; unrelated Base wrapper call sites and manifest trust policy are unchanged.

@codeforester
codeforester requested a review from a team as a code owner September 19, 2026 17:24
@codeforester
codeforester merged commit 03c2c17 into main Sep 19, 2026
22 checks passed
@codeforester
codeforester deleted the enhancement/2325-20260919-enhancement-harden-test-preflight-wrapper-validation-and-pat branch September 19, 2026 17:48
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.

enhancement: harden test preflight wrapper validation and path ownership

1 participant