fix(loop-ui): fixed some UI issues, added tests for parallel and loop nodes

[aadamgough]

Description

Fixed UI issues with resizing the node, dragging content into the node, and auto connecting the nodes.

Type of change

Please delete options that are not relevant.

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

How Has This Been Tested?

Confirmed that each change I made worked on both the parallel and loop nodes. I also wrote two test files.

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

[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	May 26, 2025 6:39pm

1 Skipped Deployment

Name	Status	Preview	Comments	Updated (UTC)
docs	⬜️ Skipped (Inspect)			May 26, 2025 6:39pm

[vercel]

Someone is attempting to deploy a commit to the Sim Studio Team on Vercel.

A member of the Team first needs to authorize it.

[greptile-apps]

PR Summary

This PR addresses UI issues in loop and parallel nodes with comprehensive test coverage. Here are the key changes:

• Improved container node resizing by removing problematic CSS transitions and adding proper dimension calculations with padding to prevent sidebar overlap
• Enhanced drag-and-drop functionality with better event propagation control and parent-child relationship handling
• Added proper source handle determination for loop/parallel nodes using specific handles ('loop-end-source'/'parallel-end-source')
• Added comprehensive test suites for both loop and parallel nodes covering component structure, props validation, and edge cases
• Fixed auto-connection logic for nested nodes by prioritizing container start nodes and preventing invalid connections

_{8 file(s) reviewed, 6 comment(s)
Edit PR Review Bot Settings | Greptile}