Skip to content

Add helix-editor.com devcontainer feature - #99

Merged
sebst merged 3 commits into
mainfrom
copilot/add-helix-editor-feature
Mar 18, 2026
Merged

sebst merged 3 commits into
mainfrom
copilot/add-helix-editor-feature

Conversation

Copilot AI commented Mar 16, 2026 •

Copy link
Copy Markdown
Contributor

Adds a new devcontainer feature that installs the Helix editor (hx binary) from GitHub releases.

Key implementation notes

  • Non-standard versioning: Helix uses YY.MM / YY.MM.patch tags (e.g. 25.07, 25.07.1) — version validation regex and release-listing grep are adjusted accordingly (no v prefix, 2- or 3-part numeric format)
  • .tar.xz format: Ensures xz-utils is present; uses tar -xJ instead of tar -xz
  • Runtime directory: The archive contains both the hx binary and a runtime/ directory required by Helix. The full archive is extracted to /usr/local/lib/helix/ (--strip-components=1), placing the runtime at /usr/local/lib/helix/runtime/ — the path Helix resolves automatically relative to its binary via current_exe()
  • Binary symlink: /usr/local/bin/hx → /usr/local/lib/helix/hx

Files

  • src/helix-editor.com/devcontainer-feature.json — feature definition, version option (default: latest)
  • src/helix-editor.com/install.sh — install script (executable bit set)
  • test/helix-editor.com/test.sh — validates hx --version outputs helix
  • .github/workflows/test.yaml — adds helix-editor.com to both test-autogenerated and test-scenarios matrix jobs
Original prompt

This section details on the original issue you should resolve

<issue_title>Add feature helix-editor.com</issue_title>
<issue_description>Files:

src/helix-editor.com/devcontainer-feature.json
src/helix-editor.com/install.sh (make sure to set executable bit)
test/helix-editor.com/test.sh
.github/workflows/test.yaml

Releases:

https://github.com/helix-editor/helix/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.

Co-authored-by: sebst <592313+sebst@users.noreply.github.com>
Copilot AI changed the title [WIP] Add feature helix-editor.com Add helix-editor.com devcontainer feature Mar 16, 2026
Copilot AI requested a review from sebst March 16, 2026 18:26

@sebst sebst left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

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

Copilot AI commented Mar 16, 2026 •

Copy link
Copy Markdown
Contributor Author

Tests fail

https://github.com/devcontainer-community/devcontainer-features/actions/runs/23159484913/job/67289185848#step:4:1

The failure was caused by a second GitHub API call in github_get_tag_for_version hitting rate limits in the CI environment. Since Helix release tags have no v prefix (the tag IS the version number, e.g. 25.07.1), I removed the redundant lookup and now set releaseTag = version directly — dropping from 2 API calls to 1. Fixed in 8c00303.

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/helix-editor/helix/releases
    • Triggering command: /usr/bin/curl curl -s REDACTED (http block)
  • https://api.github.com/repos/helix-editor/helix/releases/latest
    • Triggering command: /usr/bin/curl curl -s REDACTED (http block)

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

Copilot AI requested a review from sebst March 16, 2026 21:06
@sebst
sebst marked this pull request as ready for review March 18, 2026 12:22
@sebst
sebst merged commit 3a3faa9 into main Mar 18, 2026
128 of 129 checks passed
@sebst
sebst deleted the copilot/add-helix-editor-feature branch March 18, 2026 12:22
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 feature helix-editor.com

2 participants