✨ [Frontend] RTC: Support for adding nodes

[odeimaiz]

What do these changes do?

This pull request adds support for adding nodes in Real-Time Collaboration (RTC) mode. The changes focus on handling collaborative node creation and updates through improved serialization, event handling, and UI updates.

• Refactors node creation, update asynchronous node metadata fetching and real-time node addition support
• Implements throttled project document coming updates to handle concurrent modifications
• Adds support for unknown/placeholder nodes during metadata loading

Next step:

• Handle "unknown" node creation on clientB side

Related issue/s

• related to Simultaneous access to a pipeline in view-only mode (Aug.) osparc-issues#1647

How to test

Dev-ops

[Copilot]

Pull Request Overview

This pull request adds support for adding nodes in Real-Time Collaboration (RTC) mode, enabling collaborative editing features in the frontend. The changes focus on handling collaborative node creation and updates through improved serialization, event handling, and UI updates.

• Introduces asynchronous node metadata fetching and real-time node addition support
• Implements throttled project document updates to handle concurrent modifications
• Adds support for unknown/placeholder nodes during metadata loading

Reviewed Changes

Copilot reviewed 31 out of 31 changed files in this pull request and generated 7 comments.

Show a summary per file

File	Description
services/static-webserver/client/source/class/osparc/workbench/WorkbenchUI.js	Refactors node creation to use new addNode method and updates event listener names
services/static-webserver/client/source/class/osparc/workbench/NodeUI.js	Major refactoring of node UI creation with new type system and metadata handling
services/static-webserver/client/source/class/osparc/data/model/Workbench.js	Extensive changes to support async node creation and RTC patch handling
services/static-webserver/client/source/class/osparc/data/model/Node.js	Refactors constructor and metadata handling for async loading
services/static-webserver/client/source/class/osparc/data/model/NodeUnknown.js	New class for handling nodes with unknown metadata
services/static-webserver/client/source/class/osparc/desktop/StudyEditor.js	Implements throttled project document updates for RTC
Multiple other files	Various supporting changes including method renames, UI updates, and bug fixes

[sonarqubecloud]

Quality Gate passed

Issues
0 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

[odeimaiz]

@mergify queue

[mergify]

queue

🟠 Waiting for conditions to match

• -closed [📌 queue requirement]
• any of: [🔀 queue conditions]
  • all of: [📌 queue conditions of queue default]
    • label=🤖-automerge
    • #approved-reviews-by >= 2 [🛡 GitHub branch protection]
    • #approved-reviews-by>=2
    • #changes-requested-reviews-by = 0 [🛡 GitHub branch protection]
    • #changes-requested-reviews-by=0
    • #review-threads-unresolved = 0 [🛡 GitHub branch protection]
    • #review-threads-unresolved=0
    • -conflict
    • -draft
    • base=master
    • branch-protection-review-decision = APPROVED [🛡 GitHub branch protection]
    • label!=🤖-do-not-merge
    • any of: [🛡 GitHub branch protection]
      • check-skipped = deploy to dockerhub
      • check-neutral = deploy to dockerhub
      • check-success = deploy to dockerhub
    • any of: [🛡 GitHub branch protection]
      • check-success = system-tests
      • check-neutral = system-tests
      • check-skipped = system-tests
    • any of: [🛡 GitHub branch protection]
      • check-success = unit-tests
      • check-neutral = unit-tests
      • check-skipped = unit-tests
    • any of: [🛡 GitHub branch protection]
      • check-success = check OAS' are up to date
      • check-neutral = check OAS' are up to date
      • check-skipped = check OAS' are up to date
    • any of: [🛡 GitHub branch protection]
      • check-success = integration-tests
      • check-neutral = integration-tests
      • check-skipped = integration-tests
    • any of: [🛡 GitHub branch protection]
      • check-success = build-test-images (frontend) / build-test-images
      • check-neutral = build-test-images (frontend) / build-test-images
      • check-skipped = build-test-images (frontend) / build-test-images
    • any of: [🛡 GitHub branch protection]
      • check-success = SonarCloud Code Analysis
      • check-neutral = SonarCloud Code Analysis
      • check-skipped = SonarCloud Code Analysis
• -conflict [📌 queue requirement]
• -draft [📌 queue requirement]
• any of: [📌 queue -> configuration change requirements]
  • -mergify-configuration-changed
  • check-success = Configuration changed

[matusdrobuliak66]

Nice!