improvement(tables): add per-column-type behavior registry

[j15z]

Summary

• Add a per-column-type behavior registry (column-types.ts) so each of the 6 column types' SQL cast, parse, validate, format, and conversion-compatibility rules live in one place instead of ~10 scattered switch statements across sql.ts, validation.ts, and columns/service.ts
• Migrate those four call sites to delegate to the registry
• Consolidate duplicated select-value helpers into select-values.ts
• Fix CsvColumnType type drift (was missing select) by splitting it into ColumnType (coercion target) and InferableColumnType (CSV type-inference output); fix a bug where a whitespace-only CSV number cell imported as 0 instead of null
• Fix the same select drift in the test-fixture factory (packages/testing)

Type of Change

• Improvement (internal refactor + two small bug fixes, no new column type yet)

Testing

• bun run type-check (sim, @sim/testing) — clean
• bun run lint — clean
• All CI check:* audits — clean
• lib/table suite: 536 passing (was 486 before this PR; +50 new — 48 direct column-types.ts tests, 2 for the CSV whitespace fix)
• @sim/testing suite: 39 passing
• No behavior change for the 6 existing column types except the CSV whitespace fix

Checklist

• Code follows project style guidelines
• Self-reviewed my changes
• Tests added/updated and passing
• No new warnings introduced
• I confirm that I have read and agree to the terms outlined in the Contributor License Agreement (CLA)

[vercel]

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

1 Skipped Deployment

Project	Deployment	Actions	Updated (UTC)
docs	Skipped		Jul 30, 2026 9:59pm

[cursor]

PR Summary

Medium Risk
Touches core row validation, coercion, SQL filtering/sorting casts, and column type-change compatibility across the table stack; behavior is intended to be equivalent aside from the CSV whitespace fix, but regressions would affect writes, imports, and schema migrations.

Overview
Introduces column-types.ts, a registry where each column type defines SQL cast, parse, validate, format, and type-conversion compatibility in one place, keyed off COLUMN_TYPES so a missing type fails at compile time.

validation.ts, sql.ts, and columns/service.ts no longer duplicate large switch blocks; they delegate to parseColumnValue, isValidColumnValue, sqlCastForColumnType, and isValueCompatibleWithColumnType. resolveSelectOptionId and splitMultiSelectInput move from validation.ts into select-values.ts so select logic stays shared without pulling @sim/db into the pure registry.

CSV import fixes type drift: coercion targets use ColumnType derived from COLUMN_TYPES (includes select), inference uses a separate InferableColumnType, whitespace-only number cells import as null instead of 0, and select CSV values pass through as text until schema coercion. The testing factory’s TableColumnType adds select. Large column-types.test.ts coverage plus import/select test updates.

^{Reviewed by Cursor Bugbot for commit a15b7d4. Bugbot is set up for automated code reviews on this repo. Configure here.}

[greptile-apps]

Greptile Summary

The PR centralizes table column behavior while preserving existing behavior for all registered column types.

• Adds a registry for SQL casting, parsing, validation, formatting, and conversion compatibility.
• Routes validation, SQL generation, and column conversion through the registry.
• Consolidates select-value helpers into a dependency-light module.
• Corrects CSV and test-factory type unions to include select columns.
• Treats whitespace-only numeric CSV cells as empty instead of zero.

Confidence Score: 5/5

The PR appears safe to merge with no actionable behavioral, build, data-integrity, or security defects identified.

The registered column types retain their previous coercion, validation, compatibility, and SQL-cast behavior, while the CSV changes correctly feed select values into schema-level resolution and prevent whitespace-only numbers from becoming zero.

Important Files Changed

Filename	Overview
apps/sim/lib/table/column-types.ts	Introduces the six-type behavior registry and preserves the prior parsing, validation, SQL-cast, and conversion semantics for registered types.
apps/sim/lib/table/validation.ts	Replaces duplicated parsing and validation switches with registry delegation without changing behavior for supported schemas.
apps/sim/lib/table/columns/service.ts	Delegates type-conversion compatibility checks to behavior definitions equivalent to the removed switch branches.
apps/sim/lib/table/import.ts	Aligns CSV coercion types with supported column types, passes select text to schema-level resolution, and maps whitespace-only numeric cells to null.
apps/sim/lib/table/select-values.ts	Moves select option resolution and multi-select splitting into a shared pure helper module with equivalent behavior.
apps/sim/lib/table/sql.ts	Delegates numeric and timestamp SQL cast selection to the registry while preserving generated query behavior.
packages/testing/src/factories/table.factory.ts	Aligns testing fixtures with the production column-type union by supporting select columns.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart LR
  Input[Raw cell value] --> Registry[Column type registry]
  Registry --> Parse[Parse and coerce]
  Registry --> Validate[Validate stored value]
  Registry --> Format[Format display value]
  Registry --> Compatible[Check type conversion]
  Registry --> Cast[Select SQL cast]
  Parse --> Persist[Row persistence]
  Validate --> Persist
  Compatible --> Migration[Column type migration]
  Cast --> Query[Filter and sort query]

Loading

_{Reviews (1): Last reviewed commit: "fix(testing): add missing select type to..." | Re-trigger Greptile}

[cursor]

Cursor Bugbot has reviewed your changes using high effort and found 1 potential issue.

^{❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.}

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

[cursor]

Duplicate ColumnType barrel export

Medium Severity

The new ColumnType export in import.ts duplicates an existing ColumnType alias already available from types.ts (via ColumnDefinition['type']). This redundancy causes a name collision when both are re-exported from the barrel, leading to an ambiguous export.

 

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

[j15z]

#6106