fix(workflow): use panel-aware viewport center for paste and block placement

[waleedlatif1]

Summary

• Use panel-aware getViewportCenter from useCanvasViewport hook instead of naive rect.width/2 calculation
• Blocks now paste/place at the true visible center accounting for sidebar, terminal, and side panel
• Refactored getVisibleCanvasCenter to reuse getVisibleCanvasBounds (removed duplicate code)
• Removed unused isInvitationsDisabled and handleContextInvite

Type of Change

• Bug fix

Testing

Tested manually

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 27, 2026 8:17pm

[greptile-apps]

Greptile Overview

Greptile Summary

This PR fixes viewport center calculation for block paste and placement operations by using panel-aware bounds from the useCanvasViewport hook instead of naive rect.width/2 calculations.

Key Changes:

• Replaced local getViewportCenter function with getViewportCenter() from useCanvasViewport hook that properly accounts for sidebar, terminal, and side panel overlays
• Refactored getVisibleCanvasCenter in the hook to reuse getVisibleCanvasBounds, eliminating duplicate bounds calculation logic
• Enhanced executePasteOperation to support pasting into subflows with proper validation (prevents triggers and nested subflows) and position clamping
• Added smart offset handling in regenerateBlockIds - caps large viewport-based offsets to DEFAULT_DUPLICATE_OFFSET (180,20) for blocks remaining in existing subflows, since large offsets don't make sense for relative positions
• Removed unused isInvitationsDisabled, handleContextInvite, and deleted entire server-utils.ts file as wrapper functions are no longer needed
• Added comprehensive test coverage for the new regenerateBlockIds logic

Impact:
Blocks now paste/place at the true visible center of the canvas, improving UX when panels are open.

Confidence Score: 5/5

• This PR is safe to merge - well-tested refactoring with clear improvements to viewport calculations
• Score reflects comprehensive test coverage, clean refactoring that eliminates code duplication, proper validation logic for subflow pasting, and smart offset handling that accounts for different paste contexts. The changes are backward-compatible and improve user experience.
• No files require special attention

Important Files Changed

Filename	Overview
apps/sim/hooks/use-canvas-viewport.ts	Refactored getVisibleCanvasCenter to reuse getVisibleCanvasBounds, eliminating duplicate logic. Clean refactoring with no issues.
apps/sim/app/workspace/[workspaceId]/w/[workflowId]/workflow.tsx	Replaced naive viewport center calculation with panel-aware getViewportCenter hook, removed unused code, added subflow paste support with proper validation.
apps/sim/stores/workflows/utils.ts	Added smart offset handling for viewport-based vs duplicate offsets, improved parentId preservation logic for subflow children, removed unused code.

Sequence Diagram

sequenceDiagram
    participant User
    participant Workflow
    participant useCanvasViewport
    participant getVisibleCanvasBounds
    participant calculatePasteOffset
    participant executePasteOperation
    participant regenerateBlockIds

    User->>Workflow: Paste blocks (Cmd+V or Context Menu)
    Workflow->>useCanvasViewport: getViewportCenter()
    useCanvasViewport->>getVisibleCanvasBounds: Get visible bounds
    Note over getVisibleCanvasBounds: Accounts for sidebar, panel,<br/>and terminal overlays
    getVisibleCanvasBounds-->>useCanvasViewport: Returns visible bounds
    useCanvasViewport->>useCanvasViewport: Calculate center from bounds
    useCanvasViewport->>Workflow: screenToFlowPosition(center)
    useCanvasViewport-->>Workflow: Viewport center in flow coords
    
    Workflow->>calculatePasteOffset: Calculate offset
    Note over calculatePasteOffset: Calculate clipboard center<br/>from block positions
    calculatePasteOffset-->>Workflow: Paste offset
    
    Workflow->>executePasteOperation: Execute paste
    
    alt Pasting into subflow
        executePasteOperation->>executePasteOperation: Validate (no triggers/subflows)
        executePasteOperation->>executePasteOperation: Adjust positions to relative coords
        executePasteOperation->>executePasteOperation: Clamp to container bounds
        executePasteOperation->>executePasteOperation: Set parentId and extent
    end
    
    executePasteOperation->>regenerateBlockIds: Generate new IDs
    
    alt Block has parent in existing workflow
        Note over regenerateBlockIds: Cap large viewport offset to<br/>DEFAULT_DUPLICATE_OFFSET (180,20)
        regenerateBlockIds->>regenerateBlockIds: Use capped offset
        regenerateBlockIds->>regenerateBlockIds: Preserve parentId
    else Block has parent in paste set
        regenerateBlockIds->>regenerateBlockIds: Keep relative position
        regenerateBlockIds->>regenerateBlockIds: Remap parentId to new ID
    else Top-level block
        regenerateBlockIds->>regenerateBlockIds: Apply full viewport offset
    end
    
    regenerateBlockIds-->>executePasteOperation: New blocks with remapped IDs
    executePasteOperation->>Workflow: Add blocks to workflow
    Workflow-->>User: Blocks pasted at viewport center

Loading

[greptile-apps]

_{1 file reviewed, 1 comment}

_{Edit Code Review Agent Settings | Greptile}

[waleedlatif1]

@greptile

[greptile-apps]

_{No files reviewed, no comments}

_{Edit Code Review Agent Settings | Greptile}

[cursor]

Cursor Bugbot has reviewed your changes and found 1 potential issue.

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