From b54777c731b64e2b21fe8e9c2b14acb21fc085a4 Mon Sep 17 00:00:00 2001 From: Seongho Bae Date: Mon, 13 Jul 2026 02:38:16 +0900 Subject: [PATCH] fix(opencode): forward pnpm coverage flags correctly --- .github/workflows/opencode-review.yml | 2 +- tests/test_opencode_agent_contract.py | 6 ++++++ 2 files changed, 7 insertions(+), 1 deletion(-) 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