diff --git a/.github/workflows/opencode-review.yml b/.github/workflows/opencode-review.yml index da8c08c2..ce492f71 100644 --- a/.github/workflows/opencode-review.yml +++ b/.github/workflows/opencode-review.yml @@ -1402,7 +1402,7 @@ jobs: else case "$package_runner" in npm) run_and_capture "JavaScript/TypeScript test coverage" npm test -- --coverage ;; - pnpm) run_and_capture "JavaScript/TypeScript test coverage" pnpm test -- --coverage ;; + pnpm) run_and_capture "JavaScript/TypeScript test coverage" pnpm run test --coverage ;; yarn) run_and_capture "JavaScript/TypeScript test coverage" yarn test --coverage ;; esac fi diff --git a/tests/test_opencode_agent_contract.py b/tests/test_opencode_agent_contract.py index fb1b2a55..ee9d0627 100644 --- a/tests/test_opencode_agent_contract.py +++ b/tests/test_opencode_agent_contract.py @@ -295,6 +295,12 @@ def test_opencode_coverage_does_not_duplicate_existing_javascript_coverage(): 'npm) run_and_capture "JavaScript/TypeScript test coverage" npm test -- --coverage ;;' in measure_step ) + assert ( + 'pnpm) run_and_capture "JavaScript/TypeScript test coverage" pnpm run test --coverage ;;' + in measure_step + ) + assert "pnpm test --coverage" not in measure_step + assert "pnpm test -- --coverage" not in measure_step assert 'test("(^|[[:space:]])--coverage([.=[:space:]]|$)' in measure_step assert '|c8([[:space:]]|$)|nyc([[:space:]]|$)")' in measure_step