fix(copilot): mask credentials fix

[Sg312]

Summary

Fix latency wit mask credentials

Type of Change

• Bug fix

Testing

Manually with @icecrasher321

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	Review	Updated (UTC)
docs	Skipped		Jan 24, 2026 3:31am

[greptile-apps]

Greptile Summary

Improved copilot message rendering performance by addressing two latency issues:

• Component render optimization: Changed maskCredentialValue from destructured store access to selector-based access (useCopilotStore((s) => s.maskCredentialValue)), and removed it from the useMemo dependency array in copilot-message.tsx:269. This prevents unnecessary re-computation of rendered content when unrelated store state changes.

• Non-blocking initialization: Converted loadSensitiveCredentialIds() and loadAutoAllowedTools() from blocking await calls to non-blocking fire-and-forget calls with error logging in store.ts:2740-2745. This eliminates latency when sending messages, as these operations now run in parallel without blocking the message send flow.

Both changes maintain the same functionality while significantly reducing perceived latency. The masking function still operates correctly by reading the latest sensitiveCredentialIds state via get() when called, and the credential/tool loading continues in the background with proper error handling.

Confidence Score: 5/5

• Safe to merge - well-tested performance optimization with proper error handling
• Both changes are performance optimizations that maintain existing functionality. The selector pattern is a standard Zustand best practice, and the non-blocking async pattern includes proper error logging. Manually tested by the team.
• No files require special attention

Important Files Changed

Filename	Overview
apps/sim/app/workspace/[workspaceId]/w/[workflowId]/components/panel/components/copilot/components/copilot-message/copilot-message.tsx	Optimized maskCredentialValue to use selector pattern instead of destructuring, removed from useMemo deps to prevent unnecessary re-renders
apps/sim/stores/panel/copilot/store.ts	Changed credential/tool loading from blocking await to non-blocking fire-and-forget with error handling to reduce message send latency

Sequence Diagram

sequenceDiagram
    participant User
    participant Component as CopilotMessage
    participant Store as useCopilotStore
    participant API as Backend API

    Note over Component,Store: BEFORE (Blocking)
    User->>Store: sendMessage()
    Store->>API: await loadSensitiveCredentialIds()
    API-->>Store: credential IDs
    Store->>API: await loadAutoAllowedTools()
    API-->>Store: allowed tools
    Note over Store: Blocked until both complete
    Store->>Store: Create & stream message
    Store-->>Component: Update messages state
    Component->>Component: Re-render (all store changes)
    Component->>Store: maskCredentialValue (from destructure)
    Component->>Component: useMemo recalculates

    Note over Component,Store: AFTER (Non-blocking + Optimized)
    User->>Store: sendMessage()
    Store->>API: loadSensitiveCredentialIds() [fire & forget]
    Store->>API: loadAutoAllowedTools() [fire & forget]
    Store->>Store: Create & stream message immediately
    Store-->>Component: Update messages state
    API-->>Store: credential IDs (async)
    API-->>Store: allowed tools (async)
    Component->>Component: Re-render (only on relevant changes)
    Component->>Store: maskCredentialValue (via selector)
    Note over Component: useMemo stable - no recalculation

Loading