Skip to content

Fix release.yml: secrets context not allowed in if: conditions - #1

Merged
brunkstr merged 2 commits into
mainfrom
fix/release-workflow-secrets-context
Aug 5, 2026
Merged

Fix release.yml: secrets context not allowed in if: conditions#1
brunkstr merged 2 commits into
mainfrom
fix/release-workflow-secrets-context

Conversation

@brunkstr

@brunkstr brunkstr commented Aug 5, 2026

Copy link
Copy Markdown
Owner

Summary

  • release.yml has been failing every run since 2026-03-02 ("failed because of a workflow file issue", no jobs even started).
  • Root cause, confirmed by running actionlint locally: two step-level if: conditions reference the secrets context (secrets.APPLE_SIGNING_ENABLED == 'true'), which GitHub Actions does not permit in if: expressions — only in env:/with:. This is a hard validation failure, not a runtime error.
  • Fix: hoist APPLE_SIGNING_ENABLED into a job-level env: var (from the secret) and reference env.APPLE_SIGNING_ENABLED in the two if: conditions instead. No behavior change otherwise.
  • actionlint now passes clean on the blocking errors; only pre-existing shellcheck info/warning-level quoting nits remain (unrelated, not blocking).

Test plan

  • actionlint .github/workflows/release.yml no longer reports the secrets context error
  • Manually trigger workflow_dispatch on this branch/after merge to confirm the build + release jobs complete end-to-end and produce artifacts (hasn't succeeded in ~4 months, so this needs to be verified live, not assumed)

🤖 Generated with Claude Code

GitHub Actions validation rejects the secrets context inside step-level
if: expressions, which was failing the entire workflow file before any
job could run. Hoist APPLE_SIGNING_ENABLED into a job-level env var and
reference that in the two conditionals instead.
Tauri's universal binary (lipo) merge only handles the main app
binary, not the second stegstr-cli [[bin]] target this package
defines, so bundling failed looking for a universal stegstr-cli
that was never produced (known Tauri limitation, see
tauri-apps/tauri#8152, #9422).

Build natively for the runner's arch (Apple Silicon) instead. Also
fixes two other steps that hardcoded the universal-apple-darwin
bundle path regardless of which target was actually built.
@brunkstr
brunkstr merged commit ad2e10e into main Aug 5, 2026
4 of 6 checks passed
@brunkstr
brunkstr deleted the fix/release-workflow-secrets-context branch August 5, 2026 17:12
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