fix(sockets): duplicate block op should go through debounced path

[icecrasher321]

Description

Fix duplicate block operation for sockets client side handler to use collaborativeSetSubblockValue instead of directly setting store values. This makes sure it enters and is processed by ops queue correctly.

Type of change

• Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?

Screen.Recording.2025-07-31.at.3.42.42.PM.mov

Checklist:

• My code follows the style guidelines of this project
• I have performed a self-review of my own code
• I have commented my code, particularly in hard-to-understand areas
• I have added tests that prove my fix is effective or that my feature works
• All tests pass locally and in CI (bun run test)
• My changes generate no new warnings
• Any dependent changes have been merged and published in downstream modules
• I have updated version numbers as needed (if needed)
• I confirm that I have read and agree to the terms outlined in the Contributor License Agreement (CLA)

Security Considerations:

• My changes do not introduce any new security vulnerabilities
• I have considered the security implications of my changes

Additional Information:

Any additional information, configuration or data that might be necessary to reproduce the issue or use the feature.

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
sim	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Jul 31, 2025 10:43pm

1 Skipped Deployment

Name	Status	Preview	Comments	Updated (UTC)
docs	⬜️ Skipped (Inspect)			Jul 31, 2025 10:43pm

[icecrasher321]

main change -- rest is just rearranging

[greptile-apps]

Greptile Summary

This PR fixes a critical bug in the collaborative workflow system's duplicate block operation. When users duplicate blocks in the Sim workflow editor, the subblock values (configuration data within each block) need to be copied to the new block. Previously, the collaborativeDuplicateBlock function was directly calling subBlockStore.setValue() to copy these values, which bypassed the collaborative workflow's operation queue system.

The fix moves the collaborativeSetSubblockValue function definition earlier in the file (due to JavaScript hoisting requirements) and updates the duplicate operation to use this proper collaborative function instead of direct store access. This ensures that when a block is duplicated:

1. The subblock value updates go through the debounced operation queue
2. Other connected users receive the updates via socket synchronization
3. The operations benefit from retry mechanisms and proper error handling
4. State consistency is maintained across all collaborative sessions

This change integrates with Sim's real-time collaborative editing system, which uses WebSockets to synchronize workflow changes between multiple users. The collaborative workflow system maintains an operation queue that debounces rapid changes and ensures reliable delivery of updates to all connected clients.

Confidence score: 5/5

• This is a safe bug fix that improves consistency in the collaborative workflow system without breaking existing functionality
• The change follows established patterns in the codebase by using the proper collaborative function instead of bypassing the queue system
• No files need additional attention - the fix is well-contained and addresses the root cause directly

_{1 file reviewed, no comments}

_{Edit Code Review Bot Settings | Greptile}