Skip to content

Add client information to nodes#117

Merged
atti92 merged 3 commits into
mainfrom
issue-116
Apr 25, 2026
Merged

Add client information to nodes#117
atti92 merged 3 commits into
mainfrom
issue-116

Conversation

@atti92

@atti92 atti92 commented Apr 10, 2026

Copy link
Copy Markdown
Contributor

Implement #116

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Implements issue #116 by propagating operation “category” into scenario graph node data and surfacing it in the node UI (as a “Client” badge / field), so mixed-client scenarios are visually distinguishable.

Changes:

  • Add category to NodeData and propagate operation.category in YAML→graph conversion and drag/drop node creation.
  • Render a client/category badge in CustomNode and show client info in the Properties panel.
  • Add unit tests for category propagation and UI badge rendering.

Reviewed changes

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

Show a summary per file
File Description
web-editor/src/utils/scenarioConversion.ts Propagates category from Operation into node data during YAML→graph conversion.
web-editor/src/utils/tests/scenarioConversion.test.ts Adds tests verifying category is set/undefined appropriately (including group child steps).
web-editor/src/types/scenario.ts Extends NodeData with optional category.
web-editor/src/styles/Node.module.css Adds styling for the client/category badge.
web-editor/src/hooks/useScenarioGraph.ts Sets category when creating nodes via drag/drop.
web-editor/src/components/ScenarioEditor/PropertiesPanel.tsx Displays client/category for the selected node.
web-editor/src/components/ScenarioEditor/CustomNode.tsx Renders a client/category badge on the node.
web-editor/src/components/ScenarioEditor/tests/CustomNode.test.tsx Adds tests for conditional rendering of the client badge.
.vite/deps/package.json Adds Vite cache artifact (should not be committed).
.vite/deps/_metadata.json Adds Vite cache artifact (should not be committed).

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

Comment thread web-editor/src/utils/scenarioConversion.ts Outdated
Comment thread .vite/deps/package.json
Comment thread .vite/deps/_metadata.json
@atti92
atti92 merged commit 29b54ed into main Apr 25, 2026
2 checks passed
@atti92
atti92 deleted the issue-116 branch April 25, 2026 18:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants