Skip to content

fix: resolve mise environment for OpenCode plugin enablement - #7516

Merged
disintegrator merged 2 commits into
mainfrom
fix/opencode-resolved-plugin-env
Oct 14, 2026
Merged

disintegrator merged 2 commits into
mainfrom
fix/opencode-resolved-plugin-env

Conversation

@disintegrator

@disintegrator disintegrator commented Oct 14, 2026 •

Copy link
Copy Markdown
Contributor

Summary

  • Read GRAM_OPENCODE_PLUGINS through mise env --json using a shared isPluginEnabled helper in .opencode/lib/plugins.js, reused by the pause-stack and Worktrunk plugins and parsing JSON directly in JavaScript.
  • Preserve exact colon-separated plugin ID matching and keep plugins disabled when the value is unset or lookup fails.
  • Cover enablement and failure cases for both plugins, and update the documented append command to preserve resolved values.

Motivation

The previous mise set lookup read configured values without resolving the environment correctly. Using mise's resolved environment honors local overrides and templates without adding a jq subprocess to plugin setup.

@disintegrator
disintegrator requested a review from a team as a code owner October 14, 2026 12:18
@disintegrator disintegrator added the bug Something isn't working label Oct 14, 2026
@changeset-bot

changeset-bot Bot commented Oct 14, 2026 •

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: 171d4d6

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

No issues found across 5 files

View guided diff | Turn on auto-fix | Re-trigger cubic

@disintegrator disintegrator added the review:bypass Merge without human review approval. Required status checks still apply. label Oct 14, 2026

@gram-bot gram-bot Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Auto-approved: review:bypass label applied by @disintegrator. Required status checks still gate this merge.

@disintegrator
disintegrator added this pull request to the merge queue Oct 14, 2026
Merged via the queue into main with commit 5742095 Oct 14, 2026
49 of 50 checks passed
@disintegrator
disintegrator deleted the fix/opencode-resolved-plugin-env branch October 14, 2026 12:33
@github-actions github-actions Bot locked and limited conversation to collaborators Oct 14, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

bug Something isn't working review:bypass Merge without human review approval. Required status checks still apply.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant