Skip to content

CI: attach firmware on any published release - #7

Merged
Mattboxx merged 1 commit into
std-bluetooth-trigger-HID-ESP32from
claude/wizardly-johnson-mziz2n
Nov 1, 2026
Merged

Mattboxx merged 1 commit into
std-bluetooth-trigger-HID-ESP32from
claude/wizardly-johnson-mziz2n

Conversation

@Mattboxx

@Mattboxx Mattboxx commented Nov 1, 2026

Copy link
Copy Markdown
Owner

Problem

The release job only ran on tags starting with v. Release 2.0.0 was published from the GitHub UI with tag 2.0.0, so no workflow started and the release has no files.

Fix

  • The workflow now runs on release: published, whatever the tag name.
  • A manual run (Run workflow → tag) builds that tag and attaches the files to an existing release. I'll use it right after merging to fix 2.0.0.
  • The release gets the committed firmware/ files of the tag: the exact binaries the web installer serves and that were tested on the ESP32-S3. CI rebuilds would not be byte-identical.
    • The files are checked against manifest.json (size and SHA-256) before uploading.
    • They are attached only if the build + tests job passes.
  • Still least-privilege: only the release job has contents: write, and actions stay SHA-pinned.

Generated by Claude Code

The release job only ran on "v*" tags, so a release published from the
GitHub UI with tag "2.0.0" did nothing.

- Triggered by "release: published" (any tag name) instead of tag pushes.
- Manual run with a "tag" input builds that tag and attaches the files to
  the existing release (used to fix 2.0.0).
- Attaches the committed firmware/ files of the tag - the exact binaries the
  web installer serves and that were tested - after checking them against
  manifest.json, and only if the build + tests job passes.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01XDVAmsBNuEdJfETCBknBj6
@Mattboxx
Mattboxx merged commit 9aaf601 into std-bluetooth-trigger-HID-ESP32 Nov 1, 2026
3 checks passed
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.

2 participants