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

1 Skipped Deployment

Project	Deployment	Actions	Updated (UTC)
docs	Skipped		Jun 15, 2026 9:27pm

PR Summary

Medium Risk
Large surface-area refactor across table CRUD, jobs, imports, and workflow execution paths; risk is import/mock miswiring rather than new logic, but regressions would affect core data and background job flows.

Overview
This refactor decomposes the table service god-file into subdomain modules (mirroring lib/knowledge/): rows/service (CRUD/query), rows/ordering and rows/executions (position keys and workflow exec sidecar), columns/service, jobs/service, import-data, plus a slim service.ts for table definition CRUD and shared withLockedTable / schema helpers.

Call sites across table API routes, v1 APIs, background workers (workflow-column-execution, resume-execution), copilot tools, and runners now import from the specific module instead of a single @/lib/table/service. The @/lib/table barrel re-exports the new services so external imports stay stable; Vitest mocks were split to match (e.g. job routes mock @/lib/table/jobs/service, row routes mock @/lib/table/rows/service).

No intended behavior change—logic is moved verbatim into the new files with dependency ordering preserved (e.g. column ops still use withLockedTable from the root service).

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

Greptile Summary

This PR decomposes the monolithic lib/table/service.ts (5,324 lines, 58 exports) into six per-concern modules: rows/service.ts, rows/ordering.ts, rows/executions.ts, columns/service.ts, jobs/service.ts, workflow-groups/service.ts, plus shared tx.ts and import-data.ts helpers. The barrel (@/lib/table) is updated to re-export all six new modules, and every consumer is repointed to the correct leaf module.

• Code-motion only: function bodies are verbatim copies; the dependency graph is a valid DAG (backfill-runner cycle preserved via lazy dynamic import); no logic was altered beyond two relative → absolute dynamic-import strings.
• Consumers updated: all API routes, background runners, copilot tools, and test mocks are repointed to the new module paths.
• Three previously-private helpers (EMPTY_JOB_FIELDS, latestJobForTable, latestJobsForTables) are now exported from the barrel as a side-effect of export * from '@/lib/table/jobs/service'; this is additive and harmless but technically widens the public surface beyond what the PR description claims.

Confidence Score: 5/5

Safe to merge — pure code-motion with no logic changes, a valid acyclic dependency graph, and all consumers correctly repointed.

Every moved function body is verbatim from the original file. The dependency graph is a strict DAG with the backfill-runner cycle preserved via lazy dynamic import. No consumer imports a symbol missing from the new barrel or sub-module paths. The only difference from the stated intent is that three previously-private helpers become reachable via the barrel, which is additive and cannot regress existing callers.

The new barrel (index.ts) is the only place worth a second glance to confirm intentional surface exposure of the three newly-public job helpers.

Important Files Changed

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    barrel["@/lib/table (index.ts)\n— public barrel —"]
    service["service.ts\nroot CRUD + withLockedTable\naddTableColumnsWithTx"]
    rows_svc["rows/service.ts\ninsertRow / updateRow / deleteRow\nqueryRows / findRowMatches"]
    cols_svc["columns/service.ts\naddTableColumn / renameColumn\ndeleteColumn / updateColumnType"]
    jobs_svc["jobs/service.ts\nmarkTableJobRunning / releaseJobClaim\nselectExportRowPage / getTableJob"]
    wfg_svc["workflow-groups/service.ts\naddWorkflowGroup / updateWorkflowGroup\npruneStaleWorkflowGroupOutputs"]
    import_data["import-data.ts\nbulkInsertImportBatch\nimportAppendRows / importReplaceRows"]
    rows_ord["rows/ordering.ts internal\nacquireRowOrderLock / selectRowIdPage"]
    rows_exec["rows/executions.ts internal\nloadExecutionsByRow / applyExecutionsPatch"]
    tx["tx.ts internal\nsetTableTxTimeouts / scaledStatementTimeoutMs"]
    backfill["backfill-runner.ts\n— lazy dynamic import —"]
    barrel --> service
    barrel --> rows_svc
    barrel --> cols_svc
    barrel --> jobs_svc
    barrel --> wfg_svc
    barrel --> import_data
    service --> jobs_svc
    cols_svc --> service
    wfg_svc --> service
    import_data --> service
    import_data --> rows_svc
    jobs_svc --> rows_svc
    rows_svc --> rows_ord
    rows_svc --> rows_exec
    cols_svc --> rows_exec
    wfg_svc --> rows_exec
    service --> tx
    rows_svc --> tx
    cols_svc --> tx
    rows_ord --> tx
    wfg_svc --> tx
    wfg_svc -. "await import()" .-> backfill
    backfill --> jobs_svc
    backfill --> rows_svc

_{Reviews (3): Last reviewed commit: "refactor(table): use absolute imports th..." | Re-trigger Greptile}

@greptile

@cursor review

Re: the partial @/lib/table/rows/service mock in import/route.test.ts — confirmed safe. The import route imports exactly one symbol from rows/service: dispatchAfterBatchInsert (called at route.ts:287), and the mock provides precisely that. The route's other @/lib/table barrel imports are releaseJobClaim (jobs/service), sanitizeName/validateMapping, and types — none from rows/service, and it calls no row CRUD/query function. So the mock nulls nothing the test path exercises (27/27 pass). It's also a test-only mock, so it has no production/integration impact regardless.

This whole PR is mechanically verified pure code-motion: of the 93 functions in the original service.ts, 90 are token-identical in the new modules, 1 (updateWorkflowGroup) differs only by two dynamic-import() path strings resolving to the same modules, and 2 were intentionally-removed dead helpers. All 14 non-function symbols are byte-identical.

Re-triggering on the latest commit (a5a2b7e) — the summary footer was still on 059b682.

@greptile

@cursor review