Skip to content

chore: align darwin codesign hook to the fail-loud form#186

Merged
rianjs merged 1 commit into
mainfrom
chore/align-codesign-hook-fail-loud
Jun 4, 2026
Merged

chore: align darwin codesign hook to the fail-loud form#186
rianjs merged 1 commit into
mainfrom
chore/align-codesign-hook-fail-loud

Conversation

@rianjs

@rianjs rianjs commented Jun 4, 2026

Copy link
Copy Markdown
Collaborator

slck carried the original simple darwin codesign hook; the other five family CLIs (cfl/jtk/gro/nrq/cr) ship the hardened fail-loud form — where CODESIGN_DARWIN_SCRIPT set-but-missing or non-executable errors instead of silently skipping. This makes slck's hook byte-identical to theirs so the convention (cli-common distribution.md §2A) is uniform across all six repos.

  • YAML-only; .goreleaser.yaml is not a gated release path → no release is cut.
  • goreleaser check -f .goreleaser.yaml passes.
  • Verified byte-identical to the cfl hook.

slck carried the original simple darwin codesign hook; the other five
family CLIs (cfl/jtk/gro/nrq/cr) use the hardened fail-loud form, where
CODESIGN_DARWIN_SCRIPT set-but-missing or non-executable errors instead
of silently skipping. Make slck's hook byte-identical to theirs so the
convention is uniform. YAML-only — not a gated release path.
@rianjs

rianjs commented Jun 4, 2026

Copy link
Copy Markdown
Collaborator Author

Codex architect review (continuity session from the signing initiative)

The hook is now byte-identical to the refined fail-loud convention. Unset env still no-ops for local builds; set-but-missing/non-executable fails before an unsigned intended-to-sign build can proceed. goreleaser check passes.

STATUS: blockers=0 majors=0 minors=0 nits=0

@rianjs rianjs merged commit b2939b9 into main Jun 4, 2026
9 checks passed
@rianjs rianjs deleted the chore/align-codesign-hook-fail-loud branch June 4, 2026 10:26
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