Skip to content

fix: exercise valid Rust release smoke - #1008

Merged
tomdps merged 1 commit into
mainfrom
codex/fix-rust-release-smoke
Aug 12, 2026
Merged

fix: exercise valid Rust release smoke#1008
tomdps merged 1 commit into
mainfrom
codex/fix-rust-release-smoke

Conversation

@tomdps

@tomdps tomdps commented Aug 12, 2026

Copy link
Copy Markdown
Collaborator

Summary: exercise the existing native greeting-validator protocol for both raw and extracted release executables, preserving the native entrypoint contract. Root cause: Release run 31615491026 failed all five targets because the smoke invoked the intentionally no-args-invalid native stdio entrypoint. Validation: real optimized binary and archive passed; full test suite, lint, typecheck, release workflow guard, Opcore and Opcore Zero passed; independent review approved with no P0-P3 findings.

@greptile-apps

greptile-apps Bot commented Aug 12, 2026

Copy link
Copy Markdown

Greptile Summary

The PR updates Rust release smoke checks to invoke the native greeting-validator protocol for raw and archive-extracted executables.

  • Creates an isolated greeting fixture and supplies the same bytes through the file and stdin.
  • Reuses the smoke helper for both release executable paths.
  • Adds byte-exact unit coverage for the validator invocation.

Confidence Score: 5/5

The PR appears safe to merge.

No blocking failure remains.

Important Files Changed

Filename Overview
scripts/rust-distribution.js Replaces invalid no-argument smoke execution with the native greeting-validator protocol for raw and extracted binaries.
tests/unit/rust-distribution.test.js Adds a non-Windows shell fixture that verifies the argument and exact file/stdin greeting bytes using cmp.

Reviews (2): Last reviewed commit: "fix: exercise valid Rust release smoke" | Re-trigger Greptile

Comment thread tests/unit/rust-distribution.test.js Outdated
@tomdps
tomdps enabled auto-merge August 12, 2026 16:37
@tomdps
tomdps force-pushed the codex/fix-rust-release-smoke branch from 0fb0c44 to 74f521a Compare August 12, 2026 16:39
@tomdps
tomdps added this pull request to the merge queue Aug 12, 2026
@tomdps
tomdps removed this pull request from the merge queue due to a manual request Aug 12, 2026
@tomdps
tomdps added this pull request to the merge queue Aug 12, 2026
Merged via the queue into main with commit 1da5c5d Aug 12, 2026
12 checks passed
@tomdps
tomdps deleted the codex/fix-rust-release-smoke branch August 12, 2026 17:13
@github-actions

Copy link
Copy Markdown

🎉 This PR is included in version 6.39.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant