fix: fix cleanEdges to work with group_outputs logic, make tests

[lucaseduoli]

This pull request improves how custom component outputs are handled and tested in the flow editor, especially regarding grouped outputs and their persistence across page reloads. The main changes include refining the logic for matching and displaying outputs in cleanEdges, and adding a comprehensive regression test to ensure output connections persist as expected for custom components with multiple outputs.

Improvements to output handling logic:

• Refined the output matching logic in cleanEdges (in reactflowUtils.ts) for genericNode types to better handle grouped and non-grouped outputs, ensuring correct visibility and selection behavior for outputs with and without the group_outputs property.

Testing and regression coverage:

• Added a new Playwright regression test (general-bugs-clean-edges.spec.ts) that creates a custom component with multiple outputs (including grouped outputs), verifies correct UI rendering (separate outputs and dropdowns), tests connecting outputs to other components, and ensures that connections persist correctly after page reloads.

[coderabbitai]

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

✨ Finishing touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch fix/output_selection

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[mfortman11]

nicely done

[github-actions]

Frontend Unit Test Coverage Report

Coverage Summary

Lines	Statements	Branches	Functions
	9.58% (2522/26308)	3.9% (793/20313)	4.04% (229/5660)

Unit Test Results

Tests	Skipped	Failures	Errors	Time
831	0 💤	0 ❌	0 🔥	14.467s ⏱️

[codecov]

Codecov Report

❌ Patch coverage is 0% with 106 lines in your changes missing coverage. Please review.
⚠️ Please upload report for BASE (release-1.6.5@7125c60). Learn more about missing BASE report.

Files with missing lines	Patch %	Lines
.../components/NodeOutputParameter/nodeOutputUtils.ts	0.00%	61 Missing ⚠️
...ode/components/NodeOutputParameter/NodeOutputs.tsx	0.00%	16 Missing ⚠️
src/frontend/src/CustomNodes/GenericNode/index.tsx	0.00%	11 Missing ⚠️
...FlowPage/components/nodeToolbarComponent/index.tsx	0.00%	10 Missing ⚠️
src/frontend/src/utils/reactflowUtils.ts	0.00%	7 Missing ⚠️
...s/GenericNode/components/NodeOutputfield/index.tsx	0.00%	1 Missing ⚠️

❌ Your patch status has failed because the patch coverage (0.00%) is below the target coverage (40.00%). You can increase the patch coverage or adjust the target coverage.

Additional details and impacted files

@@               Coverage Diff                @@
##             release-1.6.5   #10188   +/-   ##
================================================
  Coverage                 ?   35.68%           
================================================
  Files                    ?     1221           
  Lines                    ?    58147           
  Branches                 ?     5549           
================================================
  Hits                     ?    20747           
  Misses                   ?    37239           
  Partials                 ?      161           

Flag	Coverage Δ
frontend	8.81% <0.00%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines	Coverage Δ
...nericNode/components/NodeOutputParameter/index.tsx	0.00% <ø> (ø)
...s/GenericNode/components/NodeOutputfield/index.tsx	0.00% <0.00%> (ø)
src/frontend/src/utils/reactflowUtils.ts	9.87% <0.00%> (ø)
...FlowPage/components/nodeToolbarComponent/index.tsx	0.00% <0.00%> (ø)
src/frontend/src/CustomNodes/GenericNode/index.tsx	0.00% <0.00%> (ø)
...ode/components/NodeOutputParameter/NodeOutputs.tsx	0.00% <0.00%> (ø)
.../components/NodeOutputParameter/nodeOutputUtils.ts	0.00% <0.00%> (ø)

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[sonarqubecloud]

Quality Gate passed

Issues
2 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud