Skip to content

Add sxyazi/yazi feature#208

Merged
sebst merged 3 commits intomainfrom
copilot/sxyazi-yazi-add-feature
Mar 26, 2026
Merged

Add sxyazi/yazi feature#208
sebst merged 3 commits intomainfrom
copilot/sxyazi-yazi-add-feature

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Mar 26, 2026

Adds a new devcontainer feature for yazi, a Rust-based terminal file manager.

New files

  • src/sxyazi-yazi/devcontainer-feature.json — id sxyazi-yazi, v1.0.0, single version option
  • src/sxyazi-yazi/install.sh — gh release install via zip; musl static builds; Rust arch naming (x86_64/aarch64/i686); URL pattern: yazi-{arch}-unknown-linux-musl.zip
  • src/sxyazi-yazi/NOTES.md — project docs
  • test/sxyazi-yazi/test.sh — validates yazi --version

Modified

  • README.md — table entry inserted alphabetically between swc.rs and tailscale.com
Original prompt

This section details on the original issue you should resolve

<issue_title>Add sxyazi/yazi feature (id: sxyazi-yazi)</issue_title>
<issue_description>https://github.com/sxyazi/yazi/releases</issue_description>

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


📱 Kick off Copilot coding agent tasks wherever you are with GitHub Mobile, available on iOS and Android.

@sebst sebst marked this pull request as ready for review March 26, 2026 15:21
Copilot AI changed the title [WIP] Add sxyazi/yazi feature Add sxyazi/yazi feature Mar 26, 2026
Copilot AI requested a review from sebst March 26, 2026 15:28
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.

tests fail

…yPathInArchiveTemplate for zip subdirectory

Agent-Logs-Url: https://github.com/devcontainer-community/devcontainer-features/sessions/59c4cc78-2bc0-4233-8f09-4474d8a61994

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

Copilot AI commented Mar 26, 2026

tests fail

Fixed in f270f4f. Two bugs were causing the failures:

  1. local version="$2" inside github_get_tag_for_version conflicted with the outer readonly version declaration — renamed to local _version="$2" (matching the fzf template pattern)
  2. The yazi zip archive contains the binary inside a subdirectory (yazi-{arch}-unknown-linux-musl/yazi), not at the root — replaced the static binaryPathInArchive='yazi' with a binaryPathInArchiveTemplate='yazi-${architecture}-unknown-linux-musl/yazi' that gets expanded via envsubst inside install()

Copilot AI requested a review from sebst March 26, 2026 15:41
@sebst sebst merged commit c89927d into main Mar 26, 2026
7 checks passed
@sebst sebst deleted the copilot/sxyazi-yazi-add-feature branch March 26, 2026 15:43
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.

Add sxyazi/yazi feature (id: sxyazi-yazi)

2 participants