refactor(tool-input): subblock-first rendering, component extraction, bug fixes

[waleedlatif1]

Summary

• Replace ~440-line SyncWrapper system with ToolSubBlockRenderer that delegates to <SubBlock> for full rendering parity
• Restore "(optional)" indicator on tool params via labelSuffix prop on SubBlock
• Fix folder selector missing for tools with canonicalParamId (e.g. Google Drive list files)
• Fix canonical toggle (basic/advanced switch) not clickable in tool-input context
• Extract ParameterWithLabel, ToolSubBlockRenderer, ToolCredentialSelector to components/tools/
• Extract StoredTool to types.ts, selection helpers to utils.ts
• Remove dead code and strengthen typing (no any)

Type of Change

• Refactor
• Bug fix

Testing

• tsc --noEmit — zero errors
• 42 tool-input tests pass
• 31 params tests pass
• Tested manually: registry tools, custom tools, MCP tools, canonical toggle, folder selector, optional indicator

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		Feb 13, 2026 2:32am

[greptile-apps]

Greptile Overview

Greptile Summary

This PR successfully refactors the tool-input system by replacing a ~440-line SyncWrapper with a cleaner ToolSubBlockRenderer component that delegates to <SubBlock> for full rendering parity. The refactor reduces tool-input.tsx from ~2400 to ~1100 lines while extracting reusable components (ParameterWithLabel, ToolCredentialSelector) and utilities to separate files.

Key improvements:

• Extracted StoredTool type and selection utilities to dedicated files for better code organization
• Added canonical mode handling in providers/utils.ts for basic/advanced parameter switching
• Fixed canonical toggle clickability by properly passing props through the component hierarchy
• Fixed folder selector visibility for tools with canonicalParamId (e.g. Google Drive)
• Added labelSuffix and dependencyContext props to SubBlock component
• Removed any types and strengthened type safety throughout

Issues found:

• The PR description claims to restore "(optional)" indicator via labelSuffix, but ToolSubBlockRenderer doesn't pass this prop to SubBlock. Parameters with user-or-llm visibility currently show no indicator (line 90-107 in sub-block-renderer.tsx)

Confidence Score: 4/5

• This PR is generally safe to merge with one logic issue that needs fixing
• The refactor is well-structured and tests pass (42 tool-input tests, 31 params tests), but there's a critical logic issue where the "(optional)" indicator is missing from user-or-llm visibility parameters despite being mentioned as fixed in the PR description. The issue is isolated to one component and has a straightforward fix.
• The sub-block-renderer.tsx file needs the labelSuffix prop added to properly display "(optional)" indicators

Important Files Changed

Filename	Overview
apps/sim/app/workspace/[workspaceId]/w/[workflowId]/components/panel/components/editor/components/sub-block/components/tool-input/components/tools/sub-block-renderer.tsx	New component that bridges SubBlock rendering with tool params; missing labelSuffix prop for "(optional)" indicator on user-or-llm visibility params
apps/sim/app/workspace/[workspaceId]/w/[workflowId]/components/panel/components/editor/components/sub-block/components/tool-input/tool-input.tsx	Major refactor from ~2400 to ~1100 lines by delegating to ToolSubBlockRenderer; removes SyncWrapper system and improves code organization
apps/sim/tools/params.ts	Added wandConfig to UIComponentConfig and new utility functions for filtering SubBlocks in tool-input context
apps/sim/providers/utils.ts	Added canonical mode handling and paramsTransform logic for tool execution; complex but necessary changes

Sequence Diagram

sequenceDiagram
    participant TI as ToolInput
    participant TSR as ToolSubBlockRenderer
    participant SB as SubBlock
    participant Store as SubBlockStore
    
    TI->>TSR: Render tool parameter<br/>(with subBlock config)
    Note over TSR: Create synthetic ID:<br/>${subBlockId}-tool-${toolIndex}-${paramId}
    TSR->>Store: useSubBlockValue(syntheticId)
    Store-->>TSR: [storeValue, setStoreValue]
    
    Note over TSR: Sync tool.params → store
    TSR->>TSR: Parse JSON if object type<br/>(file-upload, table, etc.)
    TSR->>Store: setStoreValue(parsed/string)
    
    TSR->>SB: <SubBlock config={...} canonicalToggle={...}/>
    Note over SB: Renders using native<br/>SubBlock components
    SB->>Store: Updates store on user input
    
    Store-->>TSR: storeValue changed
    Note over TSR: Sync store → tool.params
    TSR->>TSR: Stringify if object type
    TSR->>TI: onParamChange(toolIndex, paramId, stringValue)

Loading

_{Last reviewed commit: a76e6e9}

[greptile-apps]

_{11 files reviewed, 1 comment}

_{Edit Code Review Agent Settings | Greptile}

[waleedlatif1]

@cursor review
@greptile

[greptile-apps]

_{12 files reviewed, no comments}

_{Edit Code Review Agent Settings | Greptile}

[waleedlatif1]

@cursor review
@greptile

[greptile-apps]

_{17 files reviewed, no comments}

_{Edit Code Review Agent Settings | Greptile}

[waleedlatif1]

@cursor review
@greptile

[greptile-apps]

_{25 files reviewed, 1 comment}

_{Edit Code Review Agent Settings | Greptile}

[waleedlatif1]

@greptile
@cursor review

[greptile-apps]

_{30 files reviewed, 1 comment}

_{Edit Code Review Agent Settings | Greptile}

[waleedlatif1]

https://github.com/cursor review

[waleedlatif1]

@cursor review

[cursor]

Cursor Bugbot has reviewed your changes and found 2 potential issues.

^{Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.}