test(opencode): separate plugin test suite #253

Merged

jwilger merged 1 commit from issue-248-opencode-plugin-tests into main

2026-05-18 11:55:21 -07:00

jwilger commented

2026-05-18 11:52:30 -07:00

Owner

Why this change

Issue #248 asks for opencode plugin/harness tests to be kept visible without mixing them into the Rust application test layout.

What changed

Moved the remaining opencode RGR task-gate test from tests/ to .opencode/plugins/.
Added just opencode-test as the explicit plugin test runner, backed by tracked .opencode/package*.json metadata.
Added Node to the Nix dev shell for the plugin harness.
Added a separate Forgejo CI job for just opencode-test and made semantic review/artifact jobs depend on it.
Updated contributor, agent, command, and PR-template guidance to distinguish just test from just opencode-test.

Scope of this PR

Test layout and developer tooling only.
CI workflow wiring for the new opencode plugin test suite.
Documentation/guidance for contributors and agents.

Consequences

just test remains the Rust application suite.
just opencode-test runs the .opencode/plugins/*.test.ts harness suite and installs its npm dependency with npm ci.

Verification

just opencode-test
just fmt
just test
just clippy
just ci

Closes #248

## Why this change Issue #248 asks for opencode plugin/harness tests to be kept visible without mixing them into the Rust application test layout. ## What changed - Moved the remaining opencode RGR task-gate test from `tests/` to `.opencode/plugins/`. - Added `just opencode-test` as the explicit plugin test runner, backed by tracked `.opencode/package*.json` metadata. - Added Node to the Nix dev shell for the plugin harness. - Added a separate Forgejo CI job for `just opencode-test` and made semantic review/artifact jobs depend on it. - Updated contributor, agent, command, and PR-template guidance to distinguish `just test` from `just opencode-test`. ## Scope of this PR - Test layout and developer tooling only. - CI workflow wiring for the new opencode plugin test suite. - Documentation/guidance for contributors and agents. ## Consequences - `just test` remains the Rust application suite. - `just opencode-test` runs the `.opencode/plugins/*.test.ts` harness suite and installs its npm dependency with `npm ci`. ## Verification - [x] `just opencode-test` - [x] `just fmt` - [x] `just test` - [x] `just clippy` - [x] `just ci` Closes #248

jwilger added 1 commit

2026-05-18 11:52:30 -07:00

test(opencode): separate plugin test suite

CI / Format check (pull_request) Successful in 12s

Details

CI / Clippy (pull_request) Successful in 40s

Details

CI / opencode plugin tests (pull_request) Successful in 9s

Details

CI / Dependency policy (pull_request) Successful in 10s

Details

CI / Test (pull_request) Successful in 49s

Details

CI / Build (pull_request) Successful in 32s

Details

CI / Request auto_review semantic review (pull_request) Successful in 1s

Details

CI / Build PR artifacts (no token) (pull_request) Successful in 2s

Details

auto_review auto_review: no findings

bd81bc1a78

Why:
- Issue #248 asks for opencode harness/plugin tests to be visible without being mixed into the Rust application test tree.
- A dedicated test recipe and CI job make the boundary explicit for contributors and reviewers.

What:
- Move the remaining opencode RGR plugin test beside the project plugins.
- Add just opencode-test with tracked npm metadata and Node in the Nix dev shell.
- Wire the opencode plugin test gate into CI and update developer guidance.

Validation:
- just opencode-test
- just fmt
- just test
- just clippy
- just ci

auto-review approved these changes

2026-05-18 11:54:20 -07:00

auto-review left a comment

This PR separates the opencode plugin test suite from the Rust application tests, introducing a new just opencode-test command and CI job. The changes appear well-integrated and maintain the project's testing standards.

Walkthrough

CI Workflow Changes:
- Added a new opencode-test job to run plugin tests separately.
- Updated dependencies for semantic-review and pr-artifact-build jobs to include opencode-test.
Justfile and Commands:
- Introduced just opencode-test for running plugin tests.
- Updated documentation to reflect the new command and its usage.
File Relocation:
- Moved plugin tests to .opencode/plugins/ to keep them distinct from Rust tests.

This PR separates the opencode plugin test suite from the Rust application tests, introducing a new `just opencode-test` command and CI job. The changes appear well-integrated and maintain the project's testing standards. ## Walkthrough - **CI Workflow Changes**: - Added a new `opencode-test` job to run plugin tests separately. - Updated dependencies for `semantic-review` and `pr-artifact-build` jobs to include `opencode-test`. - **Justfile and Commands**: - Introduced `just opencode-test` for running plugin tests. - Updated documentation to reflect the new command and its usage. - **File Relocation**: - Moved plugin tests to `.opencode/plugins/` to keep them distinct from Rust tests.

jwilger merged commit 79056f5361 into main

2026-05-18 11:55:21 -07:00

jwilger referenced this pull request from a commit

2026-05-18 11:55:22 -07:00

test(opencode): separate plugin test suite (#253)