Skip to content

test(release): cover default runner path - #276

Merged
steipete merged 1 commit into
mainfrom
steipete/default-release-runner-coverage-193370
Aug 3, 2026
Merged

test(release): cover default runner path#276
steipete merged 1 commit into
mainfrom
steipete/default-release-runner-coverage-193370

Conversation

@steipete

@steipete steipete commented Aug 3, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • exercise the release packager happy path with MCP_RUNNER explicitly unset
  • retain the override-based failure cases for fast, hermetic setup
  • make a stale or missing default runner fail the release contract suite

Proof

  • ./scripts/test-release.sh
  • mutation check: changed the default runner to a nonexistent path and confirmed the release contract test failed with exit 127
  • pnpm check
  • pnpm test
  • autoreview clean with Codex gpt-5.6-sol

@clawsweeper

clawsweeper Bot commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

ClawSweeper status: review started.

I am starting a fresh review of this pull request: test(release): cover default runner path This is item 1/1 in the current shard. Shard 0/1.

This placeholder means the worker is alive and reading the current context. I will edit this same comment with the actual review when the claws are done clicking.

Crustacean status: shell secured, claws on keyboard, evidence pebbles being sorted.

@steipete
steipete merged commit ecd8561 into main Aug 3, 2026
5 checks passed
@steipete
steipete deleted the steipete/default-release-runner-coverage-193370 branch August 3, 2026 19:04
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.

1 participant