Skip to content
This repository was archived by the owner on Mar 18, 2026. It is now read-only.

fix: Agent Xgen#261

Merged
CocoRoF merged 2 commits intodeployfrom
main
Oct 22, 2025
Merged

fix: Agent Xgen#261
CocoRoF merged 2 commits intodeployfrom
main

Conversation

@CocoRoF
Copy link
Collaborator

@CocoRoF CocoRoF commented Oct 22, 2025

This pull request refactors the Canvas node management and rendering logic by removing the AgentXgen-specific output update functionality and introducing a unified approach for filtering node ports based on parameter dependencies. The changes enhance maintainability and flexibility by centralizing port dependency logic and ensuring only relevant ports are rendered for each node type.

Removal of AgentXgen-specific output update logic

  • Removed the onOutputsUpdate prop and related handling from CanvasNodes, as well as the updateNodeOutputs function from useNodeManagement and its usage throughout the Canvas component hierarchy. This eliminates special-case code for AgentXgen nodes, streamlining node output management. [1] [2] [3] [4] [5] [6] [7] [8]

Port dependency filtering for node rendering

  • Added a new utility module portUtils.ts with functions to filter ports based on their dependencies and node parameters, ensuring only ports with satisfied dependencies are rendered.
  • Updated the NodePorts, NodePortsCollapsed, and RouterNodePorts components to use the new filtering logic, passing parameters as props and rendering only filtered ports. [1] [2] [3] [4] [5] [6] [7] [8] [9] [10] [11] [12] [13]

Propagation of parameter data for port filtering

  • Modified node rendering in Node, RouterNode, and SchemaProviderNode components to pass parameters to port components, enabling correct dependency filtering. [1] [2] [3] [4] [5]
  • Updated the NodePortsProps type definition to include an optional parameters field for dependency checking.

@CocoRoF CocoRoF merged commit 403bc47 into deploy Oct 22, 2025
4 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant