Make the node graph's "Merge Selected Nodes" target the right-clicked node, and fix it crashing#4252
Conversation
There was a problem hiding this comment.
Code Review
This pull request introduces several improvements and bug fixes to the node graph and document management systems. It adds a test to ensure merging un-exported nodes during inspection does not crash, updates right-click behavior to auto-select unselected nodes for context actions, and prevents dangling monitor node inputs by verifying subnetwork exports before monitoring. Additionally, it refactors the document name generation logic. Feedback is provided regarding a potential naming collision bug in the document name generation logic, where the binary search on a transformed array with duplicates can behave unpredictably; a simpler linear scan is suggested instead.
Important
The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.
5ce9590 to
da568e1
Compare
… node, and fix it crashing
da568e1 to
8090e61
Compare
Fixes #4250.