Skip to content

fix: fixed the broken release#6887

Merged
lifeizhou-ap merged 1 commit intomainfrom
lifei/fix-broken-release
Feb 2, 2026
Merged

fix: fixed the broken release#6887
lifeizhou-ap merged 1 commit intomainfrom
lifei/fix-broken-release

Conversation

@lifeizhou-ap
Copy link
Collaborator

Summary

Why
User reports the stable download url returns "not found" error

Root cause

  • Since V1.22.0 release, we published the flatpak artifacts from downloading **/*.flatpak

  • In the bundle-desktop-linux workflows, it has two steps Goose-linux-x64-flatpak and Goose-linux-x64, both of them include the same flatpack artifact, but with different paths.

  • Then in the release.yml workflow, when it downloads **/*.flatpak, it download 2 files with same name. This caused the warning message "Warning: Failed to upload artifact io.atom.electron.goose-app_stable_x86_64.flatpak. Validation Failed: {"resource":"ReleaseAsset","code":"already_exists","field":"name"} - https://docs.github.com/rest." in

    • v1.22.0, release version and release stable, and
    • v1.21.1, release version

    They works ok even with the warning message

  • However, the V1.22.1 release stable failed with "Error: Error 404: Not Found - https://docs.github.com/rest/releases/assets#delete-a-release-asset" because it had problem to delete the duplicate assets in the same tag.

What

  1. removed Goose-linux-x64 step to avoid release step downloading the same artifact. It seems this step is not referred by anywhere
  2. make upload and download regex path more specific

Type of Change

  • Feature
  • Bug fix
  • Refactor / Code quality
  • Performance improvement
  • Documentation
  • Tests
  • Security fix
  • Build / Release
  • Other (specify below)

Copilot AI review requested due to automatic review settings February 2, 2026 03:38
@lifeizhou-ap lifeizhou-ap requested review from The-Best-Codes and michaelneale and removed request for The-Best-Codes and Copilot February 2, 2026 03:39
Copy link
Collaborator

@michaelneale michaelneale left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

approving to unblock @lifeizhou-ap - but not sure if this is intended to not include .deb and .rpm?

@The-Best-Codes
Copy link
Collaborator

The-Best-Codes commented Feb 2, 2026

For context (for others reviewing this PR), here is the failing release workflow:
https://github.com/block/goose/actions/runs/21529244002/job/62162335647

I kicked off the workflow again earlier on, and it is even worse. So this PR should fix it

@lifeizhou-ap
Copy link
Collaborator Author

approving to unblock @lifeizhou-ap - but not sure if this is intended to not include .deb and .rpm?

Thanks @michaelneale. We published our assets in .deb and .rpm separately, and their artifacts are uploaded via the previous steps "Upload .deb package" and "Upload .rpm package" in bundle-desktop-linux For example: goose_1.22.0_amd64.deb.

That's why I think the Goose-linux-x64 step is redundant

@The-Best-Codes
Copy link
Collaborator

Ohhh gotcha. Let me take a look at that workflow again.

@lifeizhou-ap lifeizhou-ap merged commit 8c7bb43 into main Feb 2, 2026
26 of 28 checks passed
@lifeizhou-ap lifeizhou-ap deleted the lifei/fix-broken-release branch February 2, 2026 04:08
lifeizhou-ap added a commit that referenced this pull request Feb 2, 2026
lifeizhou-ap added a commit that referenced this pull request Feb 2, 2026
* main:
  fix: fixed the broken release (#6887)
  feat: Streamable HTTP transport for ACP + goose-acp usage (#6741)
  Add Laminar for Observability (#6514)
  Missed a couple of places that hard code J for the newline key (#6853)
  fix(ui): preserve working directory when creating new chat (#6789)
  blog: add 5 tips for building MCP Apps that work (#6855)
  docs: session isolation (#6846)
  upgrade react and electron to latest (#6845)
  Fix: Small update UI settings prompt injection (#6830)
  Remove autogenerated .gooseignore files that don't belong in repo (#6824)
  Fix case-insensitive matching for builtin extension names (#6825)
  docs: cli newline keybinding (#6823)
  Update version to 1.22.0 (#6821)
  Refactor: move persisting extension to session outside of route (#6685)
  acp: load configured extensions and refactor tests (#6803)
stebbins pushed a commit to stebbins/goose that referenced this pull request Feb 4, 2026
Signed-off-by: Harrison <hcstebbins@gmail.com>
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.

3 participants