Skip to content

test(agentplugins): restore applied package before OpenCode collision repair - #323

Merged
777genius merged 1 commit into
refactor/installer-core-clean-architecturefrom
fix/opencode-native-repair-collision
Sep 19, 2026
Merged

777genius merged 1 commit into
refactor/installer-core-clean-architecturefrom
fix/opencode-native-repair-collision

Conversation

@777genius

Copy link
Copy Markdown
Owner

Summary

  • Hosted OpenCode native lifecycle still failed after test(agentplugins): expect OpenCode native collision to refuse before commit #322: update correctly refused the foreign api/server key before commit (apply_failed / unmanaged identity).
  • Repair then hit preflight_failed: repair must use the exact applied package revision because the fixture source had been rewritten to 3.0.0 and that revision was never applied.
  • Restore the managed package tree into the fixture before repair so the collision is observed on host identity, not a revision mismatch. Config and skill unchanged proofs stay.

Test plan

  • Isolated usecase install/repair: OpenCode, Windsurf, Claude, Cline
  • TestAgentpluginsNativeInstaller
  • CLI agentplugins tests
  • Native Client Runtime opencode runtime on darwin-arm64 and windows-amd64 after this lands on architecture
  • Do not merge refactor: installer core clean architecture refactor #288 to main

… repair

Update now refuses a foreign host MCP key before commit, so the fixture's 3.0.0 tree is not the applied revision. Copy the managed package back so repair preflight can observe that collision instead of failing on a revision mismatch.
@coderabbitai

coderabbitai Bot commented Sep 19, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Advanced

Run ID: 6ed49cf5-f3c2-45f6-97f8-dd36d701bd41

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

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

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

@777genius
777genius merged commit 32f9ccb into refactor/installer-core-clean-architecture Sep 19, 2026
14 checks passed
@777genius
777genius deleted the fix/opencode-native-repair-collision branch September 19, 2026 08:33
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.

1 participant