Skip to content

fix: Added oc-plugin key to package.json#25

Merged
dialupdisaster merged 2 commits intoDEVtheOPS:mainfrom
ndhansen:fix-missing-package-json-key
Apr 13, 2026
Merged

fix: Added oc-plugin key to package.json#25
dialupdisaster merged 2 commits intoDEVtheOPS:mainfrom
ndhansen:fix-missing-package-json-key

Conversation

@ndhansen
Copy link
Copy Markdown
Contributor

@ndhansen ndhansen commented Mar 31, 2026

Description

This PR adds an oc-plugin key to the package.json file, which is now required by opencode.

Type of change

  • Bug fix (non-breaking change that fixes an issue)
  • New feature (non-breaking change that adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation update
  • Refactoring (no functional changes)
  • Chore (dependency updates, etc.)

Checklist

  • I have read the CONTRIBUTING.md document
  • My code follows the style guidelines of this project
  • bun run typecheck passes with no errors
  • bun test passes with no errors
  • I have added tests that prove my fix is effective or that my feature works
  • I have updated the documentation accordingly
  • My commits follow the Conventional Commits specification

Related issues

#24

Additional context

I tested it by first installing it locally without my change:

$ opencode plugin ./opencode-plugin-otel/

┌  Install plugin ./opencode-plugin-otel/
│
◇  Plugin package ready
│
■  No plugin targets found
│
■  "./opencode-plugin-otel/" does not declare supported targets in package.json
│
●  Expected: "oc-plugin": ["server", "tui"] or tuples like [["tui", { ... }]].
│
└  Done

Then installing it locally with my change:

$ opencode plugin ./opencode-plugin-otel/

┌  Install plugin ./opencode-plugin-otel/
│
◇  Plugin package ready
│
◇  Detected server target
│
◇  Plugin config updated
│
●  Added to /home/tigadmin/code/.opencode/opencode.jsonc
│
◆  Installed ./opencode-plugin-otel/
│
●  Scope: local (/home/tigadmin/code/.opencode)
│
└  Done

I didn't add tests, because I don't believe the tests cover the installation in opencode, and my changes only affect the packaging.

Summary by CodeRabbit

  • Chores
    • Designated the package as a server-side plugin to ensure it runs in the appropriate runtime environment.

@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Mar 31, 2026

Caution

Review failed

Pull request was closed or merged during review

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 09b1d0a9-cfa9-41fa-8c07-eaa166905994

📥 Commits

Reviewing files that changed from the base of the PR and between 9975938 and a70a739.

📒 Files selected for processing (1)
  • package.json
✅ Files skipped from review due to trivial changes (1)
  • package.json

📝 Walkthrough

Walkthrough

Added a top-level oc-plugin field to package.json with value ["server"]. No other manifest fields, scripts, dependencies, or code were changed.

Changes

Cohort / File(s) Summary
Package Configuration
package.json
Added oc-plugin: ["server"] top-level field to mark the package as an OpenCode server plugin.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related issues

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately describes the main change: adding the oc-plugin key to package.json, which is the core modification in this pull request.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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 and usage tips.

dialupdisaster
dialupdisaster previously approved these changes Apr 13, 2026
@dialupdisaster dialupdisaster merged commit db66f87 into DEVtheOPS:main Apr 13, 2026
1 of 2 checks passed
@dialupdisaster
Copy link
Copy Markdown
Contributor

ty!

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