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

1 Skipped Deployment

Project	Deployment	Review	Updated (UTC)
docs	Skipped		Dec 18, 2025 8:04pm

Greptile Summary

Fixed unnecessary webhook API calls during block re-rendering by adding a useWebhookUrl flag to the useWebhookManagement hook. Previously, every block using this hook would trigger a GET /api/webhooks call on focus/blur events, even when the webhook URL wasn't needed.

Key changes:

• Added optional useWebhookUrl parameter (defaults to false) to control when webhook data should be loaded
• ShortInput component passes useWebhookUrl prop through to the hook (defaults to false)
• TriggerSave component explicitly sets useWebhookUrl: true since it needs the webhook URL for display and management
• Wrapped loadWebhookOrGenerateUrl() call in conditional check

Issue found:

• The useEffect dependency array is missing useWebhookUrl, which could cause stale closures if the prop changes dynamically

Confidence Score: 4/5

• This PR is safe to merge after fixing the missing dependency in the useEffect
• The fix correctly addresses the performance issue by preventing unnecessary API calls. The logic is sound and component changes are minimal. However, missing useWebhookUrl from the dependency array is a React Hooks best practice violation that should be fixed to avoid potential bugs.
• Pay attention to apps/sim/hooks/use-webhook-management.ts - the dependency array needs to be updated to include useWebhookUrl

Important Files Changed

Sequence Diagram

sequenceDiagram
    participant ShortInput
    participant TriggerSave
    participant useWebhookManagement
    participant API
    participant SubBlockStore

    Note over ShortInput,TriggerSave: Components using webhook management

    ShortInput->>useWebhookManagement: call hook with useWebhookUrl=false
    Note over useWebhookManagement: useWebhookUrl is false
    useWebhookManagement->>useWebhookManagement: Skip loadWebhookOrGenerateUrl()
    useWebhookManagement-->>ShortInput: Return webhook state (no API call)

    TriggerSave->>useWebhookManagement: call hook with useWebhookUrl=true
    Note over useWebhookManagement: useWebhookUrl is true
    useWebhookManagement->>SubBlockStore: Set loading state
    useWebhookManagement->>API: GET /api/webhooks?workflowId&blockId
    API-->>useWebhookManagement: Return webhook data
    useWebhookManagement->>SubBlockStore: Store webhookId, triggerPath, triggerConfig
    useWebhookManagement->>SubBlockStore: Mark as checked
    useWebhookManagement-->>TriggerSave: Return webhook state with URL