[jdx]

https://entire.io/gh/jdx/mise/trails/484

The release PR (#13964) fails e2e because the release step vendors a vfox-tiny plugin into crates/vfox/embedded-plugins/. tiny is a fake plugin the e2e suite relies on as a non-embedded fixture. Once it is embedded, mise use tiny@1 writes backend = "vfox:jdx/vfox-tiny" to mise.lock instead of vfox:tiny, and mise where tiny returns installs/vfox-tiny-… instead of installs/tiny-…. That breaks cli/test_use_latest and cli/test_install_layout_migrate.

xtasks/release-plz now skips vfox-tiny when re-cloning embedded plugins, so release PRs keep passing without changing those tests.

I did not confirm why embedding changes the resolved name; the cause is inferred from the release diff and the failing assertions.

AI-assisted — Tool: Claude Code; model: anthropic/claude-sonnet-5-5; version: 2.1.289.

🤖 Generated with Claude Code

---

Note

Low Risk
Release-script-only guard with no runtime product behavior change beyond what gets vendored into release commits.

Overview
Release prep re-clones managed vfox plugins into crates/vfox/embedded-plugins/. xtasks/release-plz now skips vfox-tiny in that loop so the e2e suite can keep using tiny as a non-embedded fixture.

Without the skip, embedding changes how tiny resolves (mise.lock backend and install paths), which was failing release e2e tests such as cli/test_use_latest and cli/test_install_layout_migrate.

^{Reviewed by Cursor Bugbot for commit 9b8aa64. Bugbot is set up for automated code reviews on this repo. Configure here.}

Summary by CodeRabbit

• Bug Fixes
  • Embedded plugin refreshes now leave vfox-tiny out of the refreshed plugin set. The plugin is skipped when the embedded set is cloned, so it is no longer carried forward as part of that refresh.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

• Configuration used: Repository YAML (base), Central YAML (inherited), Organization UI (inherited)
• Review profile: CHILL
• Plan: Advanced
• Run ID: c22fd641-df82-4a24-9d82-bf4a760c59ac

📥 Commits

Reviewing files that changed from the base of the PR and between 5cfd571 and 9b8aa64.

📒 Files selected for processing (1)

• xtasks/release-plz

Included review availability: This review used your included allowance. Your plan provides up to 10 included reviews per hour; 9 remain after this review.

---

📝 Walkthrough

Walkthrough

The vfox embedded-plugin refresh now skips vfox-tiny. Other plugins continue through the existing clone-and-copy steps.

Changes

Embedded plugin refresh

Layer / File(s)	Summary
Skip vfox-tiny during refresh xtasks/release-plz	The refresh loop skips cloning and embedding vfox-tiny. Other plugins continue through the existing clone-and-copy steps.

Priority: ⬇️ Low

Estimated code review effort: 1 (Trivial) | ~3 minutes

Change: Bug fix

Merge Risk: ⚪ Minimal · up to 9b8aa

The refresh keeps the CI-only fake plugin out of embedded artifacts, while runtime installation has a registry fallback. No actionable merge risk is established.

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title clearly and specifically describes the main change: skipping the vfox-tiny test fixture when vendoring embedded plugins.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

• Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[greptile-apps]

Confidence Score: 5/5

[Medium risk] Release script skips a test fixture during plugin vendoring.

The PR appears safe to merge; the skip preserves the external tiny fixture without changing other plugins.

What we checked:

• Old fixture stays embedded: The script deletes and recreates the entire embedded-plugin directory before the loop. Skipping the clone therefore leaves no old copy behind.

Summary

xtasks/release-plz now skips vfox-tiny when refreshing embedded plugins.

• Keeps the fake tiny plugin external, as the existing lockfile and install-path tests expect.
• Still clears old embedded plugins and clones the remaining plugins.
• No actionable issues found. Tests were not run.

_{Reviews (1) · Last reviewed commit: "fix(release): skip the vfox-tiny test fi..." · Reviewed by Greptile}

[github-actions]

Instruction counts

The comparison never ran — an earlier step failed.

_{9b8aa64a4e9a vs `` · measured on the runner, not pushed to the history.}