Skip to content

fix: look up actual GitHub release tag instead of assuming v prefix in download URL#83

Merged
sebst merged 3 commits intomainfrom
copilot/fix-atuin-installation-error
Mar 16, 2026
Merged

fix: look up actual GitHub release tag instead of assuming v prefix in download URL#83
sebst merged 3 commits intomainfrom
copilot/fix-atuin-installation-error

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Mar 16, 2026

Some GitHub releases use tags without a v prefix (e.g., 18.3.0 instead of v18.3.0). All feature install scripts hardcoded v${version} in the download URL path, causing 404s when the actual tag had no v.

Root cause

github_list_releases strips v from tags to return clean semver strings. The URL template then unconditionally re-adds it — correct when tags use v, broken when they don't.

Changes

  • New github_get_tag_for_version function — queries the GitHub releases API and returns the actual tag name for a given version, preserving whatever prefix (or lack thereof) the project uses:

    # tag_name "v18.4.0" → returns "v18.4.0"
    # tag_name "18.4.0"  → returns "18.4.0"
  • URL template updatedv${version}${releaseTag} in the download path for all affected features

  • Error guard added — exits with a clear message if no matching tag is found (e.g., API unreachable or version doesn't exist)

Affected features

atuin.sh, btop, charmbracelet-gum, chezmoi.io, jj-vcs.dev, opencode.ai, smallstep.com, starship.rs, zyedidia-eget, zellij.dev

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • https://api.github.com/repos/atuinsh/atuin/releases
    • Triggering command: /usr/bin/curl curl -s REDACTED (http block)
  • https://api.github.com/repos/atuinsh/atuin/releases/latest
    • Triggering command: /usr/bin/curl curl -s --max-time 10 REDACTED (http block)

If you need me to access, download, or install something from one of these locations, you can either:

Original prompt

This section details on the original issue you should resolve

<issue_title>not installed latest atuin</issue_title>
<issue_description>The problem if you set in devcontainer.json:

"features": {
  "ghcr.io/devcontainer-community/devcontainer-features/atuin.sh:1": {}
}

It will lead to an error while create container.

Please check a line: https://github.com/devcontainer-community/devcontainer-features/blob/2793a58e472ff50deadf1d8654f73b004f9d78b3/src/atuin.sh/install.sh#L10C1-L11C1

And last version format: https://github.com/atuinsh/atuin/releases

The "latest" unreleased version comes without prefix "v".

It seems that you need to be more careful when choosing versions.
Or just remove this prefix from the URL, but then it will have to be specified in the option "version".</issue_description>

Comments on the Issue (you are @copilot in this section)


🔒 GitHub Advanced Security automatically protects Copilot coding agent pull requests. You can protect all pull requests by enabling Advanced Security for your repositories. Learn more about Advanced Security.

…ed v prefix

The downloadUrlTemplate had a hardcoded `v` prefix (`v${version}`) in the
release download path. This broke installations when the GitHub repository
used release tags without the `v` prefix (e.g., `18.3.0` instead of
`v18.3.0`), resulting in a 404 error on the constructed download URL.

Fix: Add a `github_get_tag_for_version` function that looks up the actual
release tag (preserving any `v` prefix or lack thereof), and use `${releaseTag}`
in the URL template instead of the hardcoded `v${version}`.

Affected features: atuin.sh (reported), btop, charmbracelet-gum, chezmoi.io,
jj-vcs.dev, opencode.ai, smallstep.com, starship.rs, zyedidia-eget, zellij.dev

Co-authored-by: sebst <592313+sebst@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix atuin installation in devcontainer configuration fix: look up actual GitHub release tag instead of assuming v prefix in download URL Mar 16, 2026
Copilot AI requested a review from sebst March 16, 2026 13:44
@sebst sebst marked this pull request as ready for review March 16, 2026 13:47
Copy link
Copy Markdown
Member

@sebst sebst left a comment

Choose a reason for hiding this comment

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

All of the affected features (atuin.sh, btop, charmbracelet-gum, chezmoi.io, jj-vcs.dev, opencode.ai, smallstep.com, starship.rs, zyedidia-eget, zellij.dev) need a version bump in their respective devcontainer-feature.json

…atures

Co-authored-by: sebst <592313+sebst@users.noreply.github.com>
Copy link
Copy Markdown
Contributor Author

Copilot AI commented Mar 16, 2026

All of the affected features (atuin.sh, btop, charmbracelet-gum, chezmoi.io, jj-vcs.dev, opencode.ai, smallstep.com, starship.rs, zyedidia-eget, zellij.dev) need a version bump in their respective dev...

Done — bumped patch versions in all 10 affected devcontainer-feature.json files in commit 2eb1c59:

  • atuin.sh: 1.0.0 → 1.0.1
  • btop: 1.0.0 → 1.0.1
  • charmbracelet-gum: 1.0.2 → 1.0.3
  • chezmoi.io: 1.0.0 → 1.0.1
  • jj-vcs.dev: 1.0.0 → 1.0.1
  • opencode.ai: 1.0.1 → 1.0.2
  • smallstep.com: 1.0.0 → 1.0.1
  • starship.rs: 1.0.0 → 1.0.1
  • zyedidia-eget: 1.0.0 → 1.0.1
  • zellij.dev: 1.0.0 → 1.0.1

Copilot AI requested a review from sebst March 16, 2026 13:49
@sebst sebst merged commit 7ca6d36 into main Mar 16, 2026
76 of 81 checks passed
@sebst sebst deleted the copilot/fix-atuin-installation-error branch March 16, 2026 13:53
Copilot AI added a commit that referenced this pull request Mar 16, 2026
Co-authored-by: sebst <592313+sebst@users.noreply.github.com>
sebst added a commit that referenced this pull request Mar 16, 2026
…i, atuin.sh, and btop (#79)

* Initial plan

* fix: remove gettext-base/envsubst dependency from astral.sh-uv and opencode.ai

Co-authored-by: sebst <592313+sebst@users.noreply.github.com>

* fix: remove gettext-base/envsubst dependency from btop and atuin; fix btop URL format

Co-authored-by: sebst <592313+sebst@users.noreply.github.com>

* fix: sync asdf-vm.com with main branch updates (PRs #80-#83)

Co-authored-by: sebst <592313+sebst@users.noreply.github.com>

* fix: resolve merge conflicts - incorporate main's changes and remove envsubst dependency

Co-authored-by: sebst <592313+sebst@users.noreply.github.com>

* Update branch

Co-authored-by: sebst <592313+sebst@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: sebst <592313+sebst@users.noreply.github.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.

not installed latest atuin

2 participants