The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment

Project	Deployment	Actions	Updated (UTC)
docs	Skipped		Jul 22, 2026 9:44pm

PR Summary

Low Risk
Test-only refactor of mocks and assertions; production code is untouched aside from shared testing utilities.

Overview
Tranche 1 moves ~19 API, lib, and background Vitest suites off hand-rolled @sim/db mocks onto the shared dbChainMock stack (queueTableRows, resetDbChainMock, dbChainMockFns, schemaMock). That deletes a large amount of per-file select-chain factories and FIFO selectResults arrays in favor of table-keyed row queues and shared spies.

The shared mock gains .limit(n).for('update') support (plus a contract test), and queueTableRows docs call out the join-table queue fallback footgun. Migrated tests follow the same shape: vi.mock('@sim/db', () => dbChainMock), reset queues each run, queue rows per schema table (often with small helpers like queueFolderLookup / queueOwnerAndOrg), and assert via dbChainMockFns instead of local DB stubs. Several org/roster/invitation tests now import real @sim/db/schema tables for queue keys instead of duplicating schema mocks.

No production routes or services change—only packages/testing mock behavior and test wiring, aimed at fixing shared-worker pollution between rival suites (knowledge, billing/org, workflows/background).

^{Reviewed by Cursor Bugbot for commit 00a06cb. Configure here.}

Greptile Summary

This PR moves the first group of database tests onto the shared mock infrastructure. The main changes are:

• Migrates 19 suites to dbChainMock and table-specific row queues.
• Adds .limit(...).for(...) row-lock chaining to the shared mock.
• Adds contract coverage for the new builder path.
• Documents join-table queue fallback behavior.

Confidence Score: 5/5

This looks safe to merge.

• No blocking issues found in the changed code.
• The new row-lock builder keeps one lazy row supplier across the chain.
• The migrated suites reset shared queue and spy state between tests.

Important Files Changed

_{Reviews (1): Last reviewed commit: "improvement(tests): db-mock migration tr..." | Re-trigger Greptile}