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

1 Skipped Deployment

Project	Deployment	Actions	Updated (UTC)
docs	Skipped		May 28, 2026 6:26pm

PR Summary

Medium Risk
Touches core table grid layout, column order/metadata persistence, and undo paths; no auth or data-model changes, but incorrect sticky/offset logic could affect editing and scrolling UX.

Overview
Adds column freezing to the workspace table grid: users can pin columns (or whole workflow output groups) to the left via Freeze / Unfreeze column in the column options menu.

Frozen columns use position: sticky with computed left offsets (after the checkbox gutter), a separator on the rightmost frozen column, and state persisted in TableMetadata.frozenColumns (schema + load/rename/delete/reorder paths). Freezing also moves frozen columns to the front of columnOrder; drag-reorder re-sorts so frozen columns stay left. Horizontal scroll-to-reveal now accounts for the full frozen zone instead of only the row-number column. Undo for column delete restores prior frozen metadata; DataRow memoization keys off frozen offset maps with a width fingerprint to limit re-renders on non-frozen resizes.

Also removes in-header change column type wiring (onChangeType / handleChangeType) and switches the header chevron to emcn icons.

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

Greptile Summary

This PR adds a "freeze column" (pin-to-left) feature to the Tables grid, persisting state to TableMetadata via frozenColumns?: string[]. Frozen columns use position: sticky with computed left offsets stacked after the checkbox gutter, and workflow-output columns freeze/unfreeze as a group.

• handleFreezeToggle reorders frozen columns to the front of columnOrder on freeze and updates metadata; frozenOffsets, lastFrozenColKey, and frozenStickyLeftEdge are derived via useMemo and flow down to ColumnHeaderMenu, WorkflowGroupMetaCell, and DataRow.
• The undo hook is extended with getFrozenColumns / onFrozenColumnsChange so delete-column redo correctly strips the deleted column from current (not snapshot) frozen state — fixing the stale-snapshot bug from the prior review cycle.
• onChangeType / handleChangeType are removed from the header menu path; the ColumnOptionsMenu docstring confirms type changes were already relocated to the column config sidebar.

Confidence Score: 5/5

Safe to merge — the freeze feature is well-contained, no data mutation paths are affected, and the previously identified undo/redo stale-snapshot bug has been addressed.

All changes are UI-preference state (sticky offsets, column ordering) with no write path to row data. The undo/redo extension for delete-column correctly reads live frozen state via a getter ref, matching the pattern already used for column widths. The onChangeType removal is intentional per the component docstring. No regressions found in the freeze toggle logic, metadata seeding, or scroll-to-reveal update.

No files require special attention.

Important Files Changed

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[User clicks Freeze/Unfreeze in column menu] --> B{Column has workflowGroupId?}
    B -- Yes --> C[Collect all sibling column names in group]
    B -- No --> D[Use single column name]
    C --> E{Currently frozen?}
    D --> E
    E -- Unfreeze --> F[Filter siblings out of frozenColumns]
    E -- Freeze --> G[Append siblings to frozenColumns]
    G --> H[Re-enforce frozen-at-front in columnOrder]
    H --> I[setColumnOrder + columnOrderRef]
    F --> J[setFrozenColumns + frozenColumnsRef]
    I --> J
    J --> K[updateMetadata with frozenColumns + columnOrder]
    K --> L[Server persists to TableMetadata]
    J --> M[frozenColumnSet useMemo]
    M --> N[frozenOffsets useMemo key to sticky left px]
    M --> O[lastFrozenColKey useMemo]
    N --> P[frozenStickyLeftEdge useMemo]
    N --> Q[ColumnHeaderMenu stickyLeft + isLastFrozen]
    N --> R[WorkflowGroupMetaCell stickyLeft + isLastFrozen]
    N --> S[DataRow frozenOffsets + lastFrozenColKey]
    P --> T[Scroll-to-reveal inset]

_{Reviews (5): Last reviewed commit: "fix(tables): use current frozenColumns o..." | Re-trigger Greptile}

@greptile

@cursor review

@greptile

@cursor review

@greptile

@cursor review

@greptile

@cursor review

@greptile

@cursor review