Skip to content

feat: Download prebuilt plugin binaries instead of building with Go - #19

Merged
erezrokah merged 7 commits into
mainfrom
claude/modest-maxwell-462744
Oct 21, 2026
Merged

erezrokah merged 7 commits into
mainfrom
claude/modest-maxwell-462744

Conversation

@erezrokah

@erezrokah erezrokah commented Oct 18, 2026 •

Copy link
Copy Markdown
Contributor

Change the plugin to download the release binary for the agent architecture (the release that matches the plugin version, set in PLUGIN_DEFAULT_VERSION and checked against the tag by the release workflow, or the new optional version input), verify its SHA-256 checksum, and fall back to the old build from source (cloning the default branch if the version tag does not exist) when any step fails, so it no longer installs Go on every run, and make the test flow use the published plugin; users who copied the old script into their env0.yaml must switch to use: https://github.com/env0/env0-import-variable-plugin to benefit.

@env0

env0 Bot commented Oct 18, 2026 •

Copy link
Copy Markdown

🚀  env0 had composed a PR Plan for environment ACME Financial Services / Demos / Plugins Demo / env0-import-variable-plugin test :

No changes. Infrastructure is up-to-date.
There are no cost estimation changes

Full PR Plan logs on env0

Get instant insights with AI Summary

@erezrokah
erezrokah force-pushed the claude/modest-maxwell-462744 branch from dc69dc8 to 7b0740b Compare October 20, 2026 09:39
@erezrokah
erezrokah marked this pull request as ready for review October 20, 2026 14:49
@erezrokah
erezrokah requested a review from dcelasun October 21, 2026 07:32

@dcelasun dcelasun left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

@erezrokah LGTM. Notes:

  • env0.plugin.yaml: use: ...@0.5.0 still downloads releases/latest unless inputs.version is also set, so a pinned ref doesn't pin the binary. Derive it from the ref, or document both together.
  • The README example drops the @<tag> pin.
  • test/env0.yaml now runs main's plugin, so PR plans no longer exercise the PR's own script.
  • Nit: curl has no --connect-timeout/--max-time, so a firewalled self-hosted agent can stall before it falls back.

@erezrokah
erezrokah merged commit be0fc2c into main Oct 21, 2026
9 checks passed
@erezrokah
erezrokah deleted the claude/modest-maxwell-462744 branch October 21, 2026 12:30
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