GeezerVison

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

1 Skipped Deployment

Project	Deployment	Actions	Updated (UTC)
docs	Skipped		Apr 24, 2026 11:15pm

PR Summary

Low Risk
Low risk, localized UI-state read change: filter/sort builders now resolve tableId via canonicalParamId mapping, which could affect which sub-block value wins in edge cases but doesn’t touch auth/data writes.

Overview
Fixes Table block filter/sort builders failing to load column options when the selected table is stored under canonical basic/advanced inputs.

Adds useCanonicalSubBlockValue to read a sub-block value via canonicalParamId resolution (using the existing canonical index + resolveDependencyValue), and switches FilterBuilder/SortBuilder to use it when deriving tableId for useTableColumns.

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

Greptile Summary

This bug fix resolves a longstanding issue where the column picker in the Table block's filter and sort builders always showed "no options available" because useSubBlockValue was looking up 'tableId' as a raw sub-block ID — which doesn't exist — while the actual values live under 'tableSelector' (basic) or 'manualTableId' (advanced) via canonicalParamId. The new useCanonicalSubBlockValue hook reuses buildCanonicalIndex / resolveDependencyValue from the existing visibility module to resolve whichever canonical member currently holds the value, exactly as the dropdown dependsOn logic does.

Confidence Score: 5/5

Safe to merge — targeted bug fix with no breaking changes and no P0/P1 issues found.

The implementation follows established patterns (useStoreWithEqualityFn + isEqual, buildCanonicalIndex/resolveDependencyValue), handles null gracefully (useTableColumns already accepts null via ?? undefined), and the change is minimal and focused. All remaining observations are P2 or lower.

No files require special attention.

Important Files Changed

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A["FilterBuilder / SortBuilder\nuseCanonicalSubBlockValue(blockId, 'tableId')"] --> B["buildCanonicalIndex(blockConfig.subBlocks)"]
    B --> C{canonicalId found\nfor 'tableId'?}
    C -- Yes --> D["getCanonicalValues(group, blockValues)"]
    D --> E["resolveCanonicalMode\n(basic or advanced?)"]
    E -- basic --> F["values['tableSelector']"]
    E -- advanced --> G["values['manualTableId']"]
    F --> H["tableIdValue → useTableColumns"]
    G --> H
    C -- No --> I["values['tableId'] (undefined fallback)"]
    I --> J["tableIdValue = null\n(columns stay empty)"]
    H --> K["Column options populated ✓"]

_{Reviews (1): Last reviewed commit: "fix(table-block): resolve canonical tabl..." | Re-trigger Greptile}