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

1 Skipped Deployment

Project	Deployment	Actions	Updated (UTC)
docs	Skipped		May 8, 2026 5:18pm

PR Summary

Medium Risk
Touches table grid rendering/selection behavior and significantly extends workflow search/replace and function-code variable interpolation, which could introduce subtle UI/behavior regressions despite being well-tested.

Overview
Fixes several table UX issues: boolean cells now expose a stable click target, the sticky row-number gutter width is computed from maxRows digit count (instead of a coarse threshold), and selection overlays/tints align correctly at the left edge. Column/workflow sidebars also avoid double error toasts on validation failures and make minor header spacing adjustments.

Improves workflow editor search/replace: introduces a shared highlight style and supports nested field highlighting for input-format, response-format, and eval-input (including unmasking password fields when highlighted). Search/replace resource handling is expanded to hydrate and offer replacements for tables, workspace files, and MCP servers/tools (with server-scoped MCP tool options), refactors resource utilities under search-replace/resources, and adds broader indexer coverage/edge-case tests (visibility rules, structured subblocks, tool params, selector contexts).

Hardens execution variable interpolation for function blocks by detecting whether a <Block.ref> occurs inside JS/Python string literals, template literals, or comments and emitting safer replacements (e.g., JSON-stringifying when embedded in quotes). Also adds optimistic-collab safeguards by skipping batch subblock/subflow updates when expected values no longer match.

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

Greptile Summary

This PR delivers a set of targeted bug fixes and improvements across the table module and the workflow search-replace system. The most substantial work is in the search-replace indexer: it now handles tool-input sub-blocks, JSON-structured sub-block types (condition-input, router-input, etc.), and visibility/feature-flag filters when indexing matches. The variable resolver gains a proper string-context scanner that correctly wraps block references in string-break expressions when inserted inside JavaScript or Python literals.

• Table grid: boolean cell click is fixed with a data-boolean-cell-toggle attribute; the row-number column width is now computed dynamically from Math.floor(Math.log10(maxRows)) + 1 digits rather than a hard-coded two-bucket lookup; selection overlay left-edge bleeding is fixed per column index.
• Search-replace: new files (json-value-fields.ts, resources/registry.ts, resources/references.ts, resources/resolvers.ts, resources/validation.ts) modularize resource handling; getWorkflowSearchDependentClears is upgraded to BFS so transitive dependency clears propagate correctly.
• Resolver: getCodeStringQuoteContext is a new state-machine scanner that tracks single/double/triple-quote Python strings, JS template literals, template expressions, line and block comments, correctly enabling json.dumps/JSON.stringify wrapping only when the reference appears inside a string literal.

Confidence Score: 5/5

This PR is safe to merge. All changes are well-scoped bug fixes and refactors with good test coverage.

The resolver scanner is the highest-risk new logic; it is exercised by a comprehensive suite of unit tests covering JS strings, template literals, template expressions, Python single/double/triple-quoted strings, escaped characters, and comment handling. The search-replace refactoring consolidates previously scattered files into a clean resource registry and is backed by extensive indexer and replacements tests. The table-grid changes are mechanical and easy to verify visually.

No files require special attention; the indexer and resolver changes are the most complex but are well covered by tests.

Important Files Changed

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[indexWorkflowSearchMatches] --> B{subBlock type?}
    B -->|tool-input| C[addToolInputMatches]
    B -->|display-only| D[skip]
    B -->|other| E{mode?}
    C --> F[parseStoredToolInputValue]
    F --> G[getToolInputParamConfigs]
    G --> H[buildToolInputSearchConfig per param]
    H --> I[addTextMatches / addInlineReferenceMatches]
    E -->|text mode| J[getTextLeaves]
    J --> K{isSearchableJsonValueSubBlock?}
    K -->|yes| L[getSearchableJsonStringLeaves]
    K -->|no| M[getSearchableStringLeaves]
    L --> N[addTextMatches]
    M --> N
    E -->|resource mode| O[getSearchableStringLeaves reference]
    O --> P[parseInlineReferences]
    O --> Q[parseStructuredResourceReferences]
    P --> R[environment / workflow-reference matches]
    Q --> S[structured resource matches]

    subgraph Resolver
        T[formatCodeContextVariableReference] --> U[getCodeStringQuoteContext]
        U --> V{context?}
        V -->|template| W[wrap in dollar-brace]
        V -->|single/double/triple| X[break string + dumps/stringify]
        V -->|null code context| Y[plain globalThis/globals ref]
    end

_{Reviews (7): Last reviewed commit: "escaped behaviour" | Re-trigger Greptile}

@greptile

@cursor review

@cursor review @greptile

@greptile

@greptile

bugbot run

@greptile

@cursor review

@greptile