Add flowIn/flowOut Flow attribute pins to every node, displayed in the node header

[Copilot]

Add the a connection pin in the node's header, to force a flow evaluation, when nodes are not naturally connected by their attributes

Flow attributes are internal attributes:

• Rename FlowAttribute → Flow in desc/attribute.py and attribute.py (+ backward-compat alias)
• Update all imports, isinstance checks, comments, and QML type strings to use "Flow"
• Move flow input/output attrs from _attributes to _internalAttributes in node.py
• Update node.py _applyExpr() to also iterate over _internalAttributes
• Fix attribute.py _applyExpr() to search _internalAttributes when looking for source attr
• Update QML Node.qml to use internalAttribute()/hasInternalAttribute() for flowIn/flowOut
• Update nodeFactory.py compatibility checks to account for flow attrs in internalInputs
• Remove if v is not None filter from outputs in toDict()
• Update tests (test_flowAttribute.py, test_compatibility.py, test_pipeline.py)
• All 257 backend tests pass, CodeQL: 0 alerts

---

💬 Send tasks to Copilot coding agent from Slack and Teams to turn conversations into code. Copilot posts an update in your thread when it's finished.

[codecov]

Codecov Report

❌ Patch coverage is 96.15385% with 11 lines in your changes missing coverage. Please review.
✅ Project coverage is 84.30%. Comparing base (a33c099) to head (26e4a54).
✅ All tests successful. No failed tests found.

Files with missing lines	Patch %	Lines
meshroom/core/attribute.py	77.77%	4 Missing ⚠️
meshroom/core/nodeFactory.py	87.50%	3 Missing ⚠️
meshroom/core/graph.py	71.42%	2 Missing ⚠️
meshroom/core/node.py	92.00%	2 Missing ⚠️

Additional details and impacted files

@@             Coverage Diff             @@
##           develop    #3048      +/-   ##
===========================================
+ Coverage    83.98%   84.30%   +0.32%     
===========================================
  Files           83       84       +1     
  Lines        10829    11101     +272     
===========================================
+ Hits          9095     9359     +264     
- Misses        1734     1742       +8     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[Copilot]

Pull request overview

This PR adds automatic flowIn/flowOut pins to nodes and moves those flow attributes into internalAttributes so they can be shown in the node header while remaining separate from normal data attributes.

Changes:

• Introduces Flow/FlowAttribute descriptor/runtime classes and auto-injects internal flow input/output attrs on nodes.
• Updates serialization, expression application, compatibility checks, and UI command lookup to account for internal flow attrs.
• Updates QML node rendering and backend tests for the new flow-pin behavior.

Reviewed changes

Copilot reviewed 13 out of 13 changed files in this pull request and generated 3 comments.

Show a summary per file

File	Description
tests/test_pipeline.py	Adjusts pipeline test to ignore Flow outputs in attribute iteration.
tests/test_flowAttribute.py	Adds coverage for Flow descriptor behavior, linking, and graph serialization.
tests/test_compatibility.py	Extends compatibility expectations for internal flow attrs.
meshroom/ui/qml/GraphEditor/Node.qml	Moves flow pins into the node header and filters them from regular attribute lists.
meshroom/ui/qml/GraphEditor/AttributePin.qml	Adds compact pin mode for header rendering.
meshroom/ui/qml/GraphEditor/AttributeItemDelegate.qml	Handles Flow attrs as non-data UI elements.
meshroom/ui/commands.py	Resolves edge commands against regular or internal attributes.
meshroom/core/nodeFactory.py	Updates compatibility/name checks for internal flow serialization.
meshroom/core/node.py	Applies expressions to internal attrs and auto-creates internal flow pins.
meshroom/core/desc/node.py	Defines internal flow attrs on node descriptors.
meshroom/core/desc/attribute.py	Adds the Flow descriptor class and alias.
meshroom/core/desc/init.py	Re-exports Flow and FlowAttribute.
meshroom/core/attribute.py	Resolves link expressions against internal attrs and adds runtime Flow.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Alxiice]

yes indeed there's a problem here

[Alxiice]

but deleting these lines is even worse

[Alxiice]

Here is a gif a bit more detailed to show some functionnalities :

• the input flow can be connected to multiple outputs
• here the flow invalidates the node. Note that I finally decided remove it (because this would switch to compatibility mode all nodes in existing pipelines). This is up to discussion but I think it's better for the flow not to invalidate the node.
• the serialization works well, and the flow is only serialized if not empty

[Alxiice]

There are behaviours that still doesn't work :

• copy node with flow connection -> the connection is somewhat restored but the UI fails to recreate it
• delete/recreate connection : also fails