Skip to content

CI: authenticate croc release metadata fetch in GUI build/release workflows - #2

Merged
interfluve-wav merged 2 commits into
mainfrom
copilot/fix-with-copilot
Jul 24, 2026
Merged

interfluve-wav merged 2 commits into
mainfrom
copilot/fix-with-copilot

Conversation

Copilot AI commented Jul 23, 2026 •

Copy link
Copy Markdown
Contributor

The GUI workflows were intermittently failing while staging the croc binary because unauthenticated calls to repos/schollz/croc/releases/latest returned 403, which then caused JSON parsing to fail. This change hardens release metadata retrieval without altering artifact selection logic.

  • Root-cause-aligned fix: metadata fetch auth

    • Switched release metadata lookup from unauthenticated curl to gh api.
    • Injected GH_TOKEN: ${{ github.token }} for the download step in both workflows.
  • Scope

    • Updated:
      • .github/workflows/build-gui.yml
      • .github/workflows/release-gui.yml
    • Removed now-unused META_URL variable in both files.
  • Behavior preserved

    • Existing asset resolution (endsWith("_" + ASSET_SUFFIX)), archive extraction, and binary staging remain unchanged.
DOWNLOAD_URL=$(gh api repos/schollz/croc/releases/latest | node -e '...')

env:
  ASSET_SUFFIX: ${{ matrix.croc_asset }}
  GH_TOKEN: ${{ github.token }}

Copilot AI changed the title [WIP] Fix issue with Copilot integration CI: authenticate croc release metadata fetch in GUI build/release workflows Jul 23, 2026
Copilot AI requested a review from interfluve-wav July 23, 2026 23:32
@interfluve-wav
interfluve-wav marked this pull request as ready for review July 23, 2026 23:44
@interfluve-wav
interfluve-wav merged commit c2b1a2a into main Jul 24, 2026
5 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