Skip to content

Commit 8c2d056

Browse files
ci(node): run vitest suite instead of missing test_omnibus.ts
`packages/openmemory-js/tests/test_omnibus.ts` was renamed to `tests/omnibus.test.ts` in cbc8c84 ("port ad-hoc tsx scripts to vitest specs"), but the CI step kept calling the old path with `npx tsx`. The "Test Node.js SDK" job has been red on every PR since. Switch the step to `npm test` so the omnibus spec, the temporal / multilingual / webhook / verify suites, and any new vitest specs (e.g. the per-tenant MCP regression added in the parent commit) all run. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
1 parent af4b3bb commit 8c2d056

1 file changed

Lines changed: 5 additions & 1 deletion

File tree

.github/workflows/ci.yml

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -54,6 +54,10 @@ jobs:
5454
npm ci
5555
5656
- name: Run Verification Test
57+
# `tests/test_omnibus.ts` was ported to vitest as
58+
# `tests/omnibus.test.ts` in cbc8c84; this step has been failing
59+
# ever since. Run the whole vitest suite so omnibus + the rest of
60+
# the per-tenant / temporal / multilingual specs are exercised.
5761
run: |
5862
cd packages/openmemory-js
59-
npx tsx tests/test_omnibus.ts
63+
npm test

0 commit comments

Comments
 (0)