From 4aab22fc1c8c75290007d887f115a3d17b3aa09c Mon Sep 17 00:00:00 2001 From: RBKunnela Date: Fri, 22 May 2026 22:19:15 +0300 Subject: [PATCH] ci: gate paybot-sdk on npm test (closes #4) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Wire vitest run into the build matrix between type-check and build, across Node 18 + 20. paybot-sdk auto-publishes to npm on every push to main; before this change CI never ran the test suite, so code could ship to customers without its own tests ever executing. Test suite: 102 tests / 7 files in tests/, vitest.config.ts scoped to tests/**/*.test.ts. Local: 102/102 pass in 1.72s. Not in this PR (deliberately split per Orion routing during Phase 1 discovery): - Task #14: dual-mode dead-code bug in src/x402-v2.ts:251 (no-dupe-else-if surfaces it; requires semantic decision on what dual-mode should emit — separate @dev story) - Task #15: coverage gate at 80% threshold + tests for x402-v2.ts and payment-engine.ts (currently 0% covered, 805 LOC combined) - Task #16: coverage/ gitignore hygiene No change to required_status_checks contexts: the new step runs inside the existing build (18) + build (20) matrix entries, adding substance without adding new context names. 5th application of automated-pr-merge-authority.md. --- .github/workflows/ci.yml | 3 +++ 1 file changed, 3 insertions(+) diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml index 29f5ff1..2a53438 100644 --- a/.github/workflows/ci.yml +++ b/.github/workflows/ci.yml @@ -34,6 +34,9 @@ jobs: - name: Type check run: npm run type-check + - name: Run tests + run: npm test + - name: Build run: npm run build