Skip to content

feat: add OpenCode target support for install and integration#170

Closed
timvw wants to merge 1 commit intomicrosoft:mainfrom
timvw:feat/opencode-support
Closed

feat: add OpenCode target support for install and integration#170
timvw wants to merge 1 commit intomicrosoft:mainfrom
timvw:feat/opencode-support

Conversation

@timvw
Copy link
Contributor

@timvw timvw commented Mar 5, 2026

Summary

  • add opencode as a first-class target in detection and compile/install routing with parity to existing target behavior
  • integrate skills and commands for OpenCode paths (.opencode/skills, .opencode/commands) and extend uninstall/sync cleanup to include OpenCode-managed artifacts
  • extend unit coverage for target detection, compile flag behavior, install bootstrap logic, and integration cleanup across command/skill flows

Test Plan

  • uv run pytest tests/unit/integration/test_command_integrator.py tests/unit/test_uninstall_reintegration.py -q
  • uv run pytest tests/unit/core tests/unit/compilation tests/unit/integration tests/unit/test_install_command.py tests/unit/test_uninstall_reintegration.py tests/unit/test_install_bootstrap_dirs.py -q

@timvw timvw requested a review from danielmeppiel as a code owner March 5, 2026 09:25
@danielmeppiel danielmeppiel added the enhancement New feature or request label Mar 5, 2026
@danielmeppiel danielmeppiel added this to the 0.8.0 milestone Mar 5, 2026
@danielmeppiel
Copy link
Collaborator

This is great @timvw . We need to merge #163 first and then rebase this one - as it refactors the integration logic. That will require resolving some conflicts on this one. I'll wait until #163 is merged and then review this PR, but definitely one I want to see merged for the next release. Thanks!

@timvw timvw force-pushed the feat/opencode-support branch from e87452d to ec45086 Compare March 5, 2026 10:45
@timvw
Copy link
Contributor Author

timvw commented Mar 5, 2026

Rebased this PR on latest main and force-updated the branch to remove the accidental merge-from-main commit and unrelated drift.

Current PR now contains a single feature commit on top of main only:

  • ec45086 feat: add opencode target support for install and integration

Also validated post-rebase with:

  • uv run pytest tests/unit/core tests/unit/compilation tests/unit/integration tests/unit/test_install_command.py tests/unit/test_uninstall_reintegration.py tests/unit/test_install_bootstrap_dirs.py -q
  • Result: 559 passed

I saw the note about waiting for #163 first. Since #163 is still open, this is rebased to current main now; I can rebase again after #163 lands.

Copy link
Collaborator

@danielmeppiel danielmeppiel left a comment

Choose a reason for hiding this comment

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

@timvw #163 has been merged so once you resolve the current conflicts (we improved the integrator logic) I'll be good to review. To say we "fully support OpenCode" we should add automatic integration for the following primitives:

  • Agents
  • Commands (prompts)
  • Skills

The above should work with transitive dependencies and skills within APM packages like in https://github.com/microsoft/apm-sample-package. This is what we got for VSCode too.

Let's leave MCP out for now, instructions will be supported through the AGENTS.md standard with apm compile out of the box. You can use https://github.com/microsoft/apm-sample-package for tests.

Looking forward to this one! 👍

@JustinGOSSES
Copy link

@microsoft-github-policy-service rerun

@timvw timvw force-pushed the feat/opencode-support branch from ec45086 to a267224 Compare March 9, 2026 09:43
@timvw timvw force-pushed the feat/opencode-support branch from a267224 to 4132152 Compare March 9, 2026 11:46
@timvw
Copy link
Contributor Author

timvw commented Mar 12, 2026

Update: I prepared a clean OpenCode-target rewrite commit on a fresh branch (feat/opencode-target-standalone-v2) with:

  • standalone opencode target detection and compile routing
  • .opencode/agents, .opencode/commands, .opencode/skills integration plus uninstall sync
  • focused unit test updates

I could not safely fast-forward this into the existing PR branch without large historical conflict churn due to branch drift and unrelated base changes.

I recommend replacing this PR with a fresh PR from the clean branch so review scope is isolated to OpenCode target work. If you want, I can open that replacement PR immediately and link it here.

@timvw
Copy link
Contributor Author

timvw commented Mar 12, 2026

Superseded by #257, which refreshes the OpenCode standalone target work on top of current main and resolves merge conflicts from branch drift.

@timvw timvw closed this Mar 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Support OpenCode.ai skills directory during install [FEATURE] Opencode support

3 participants