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

Project	Deployment	Actions	Updated (UTC)
docs	Ready	Preview, Comment	Jun 9, 2026 11:58pm

PR Summary

Low Risk
Logic change is scoped to tool ID suffixing in transformBlockTool with new tests; remaining changes are SVG and presentation metadata only.

Overview
Fixes agent workflows where multiple table or knowledge-base tools registered the same LLM tool name because resource IDs lived on selector subblock keys (tableSelector, knowledgeBaseSelector) instead of canonical params (tableId, knowledgeBaseId). resolveCanonicalResourceParams mirrors basic/advanced mode resolution before suffixing unique tool IDs, preventing provider errors like duplicate tool names on Anthropic.

Also updates integration visuals: 1Password gets a full-color icon and white block/docs background; Prospeo and Neo4j icons are slightly scaled up via centered SVG transforms. Unit tests cover table and knowledge-base ID resolution paths.

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

Greptile Summary

This PR fixes a bug where multi-instance agent tools (table, knowledge-base, workflow) could produce duplicate LLM tool names when the resource ID was stored under a selector subblock key (e.g. tableSelector) rather than the canonical key (e.g. tableId), causing Anthropic to reject the request. It also updates the 1Password, Prospeo, and Neo4j icons.

• Introduces resolveCanonicalResourceParams, which non-destructively resolves canonical resource IDs from their basic/advanced selector sources before the unique-ID suffix is built, mirroring the existing paramsTransform logic so the suffix always matches the actual execution params.
• Moves canonicalGroups computation earlier in transformBlockTool so it can serve both the new resolution step and the existing paramsTransform; adds 8 unit tests covering table and knowledge-base selector resolution across all four selector modes.
• Swaps the 1Password icon to the full-color lock mark with a white bgColor, and slightly enlarges the Prospeo and Neo4j glyphs via a centered scale transform; updates docs and integration metadata to match.

Confidence Score: 5/5

Safe to merge — the fix is surgical, non-destructive, and fully tested.

The canonical-ID resolution logic is a shallow, non-destructive copy operation that only adds the canonical key to a temporary object used for building the tool-ID suffix; it does not touch userProvidedParams or the paramsTransform that drives actual execution. The change is a no-op for the ~90% of tools that already carry the canonical key directly. The eight new unit tests cover all four selector modes for both table and knowledge-base blocks, and the previously flagged gap (missing knowledge-base coverage) has been addressed in this PR. Icon and metadata changes are cosmetic with no runtime impact.

No files require special attention.

Important Files Changed

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[transformBlockTool called] --> B[Resolve toolId from block config]
    B --> C[Build canonicalGroups from subBlocks]
    C --> D[resolveCanonicalResourceParams]
    D --> E{Canonical key already in params?}
    E -- Yes --> F[Keep existing value]
    E -- No --> G[getCanonicalValues from selector subblock keys]
    G --> H{canonicalMode?}
    H -- basic --> I[Use basicValue e.g. tableSelector]
    H -- advanced --> J[Use advancedValue e.g. manualTableId]
    I --> K[Set canonical key in resolvedResourceParams]
    J --> K
    F --> L[Build uniqueToolId suffix]
    K --> L
    L --> M{Tool type?}
    M -- workflow_executor --> N[suffix = workflowId]
    M -- knowledge_* --> O[suffix = knowledgeBaseId]
    M -- table_* --> P[suffix = tableId]
    M -- other --> Q[no suffix]
    N & O & P & Q --> R[Return tool config with paramsTransform for execution]

_{Reviews (2): Last reviewed commit: "improvement(icons): new 1Password icon o..." | Re-trigger Greptile}

@greptile

@cursor review