Added Label property to Ports.#32
Merged
Merged
Conversation
Contributor
There was a problem hiding this comment.
Pull request overview
This pull request introduces editor-facing port labeling in the Statescript runtime by adding a Label property to Port and extending Node.CreatePort<T> to set that label, then updates built-in nodes and documentation to use/describe labeled ports.
Changes:
- Add
Port.Labelas an editor-facing label. - Extend
Node.CreatePort<T>with an optionallabelparameter and assign it to created ports. - Update built-in node port definitions and docs/templates to use and document port labels.
Reviewed changes
Copilot reviewed 15 out of 15 changed files in this pull request and generated 6 comments.
Show a summary per file
| File | Description |
|---|---|
| Forge/Statescript/Port.cs | Adds Label property to all ports. |
| Forge/Statescript/Node.cs | Extends CreatePort<T> to accept/assign an editor label. |
| Forge/Statescript/Nodes/StateNode.cs | Supplies labels for standard State node ports. |
| Forge/Statescript/Nodes/State/TimerNode.cs | Supplies label for the custom OnTimerEnd output. |
| Forge/Statescript/Nodes/State/EffectNode.cs | Supplies label for the custom OnEffectEnd output. |
| Forge/Statescript/Nodes/ExitNode.cs | Supplies label for the Exit node input port. |
| Forge/Statescript/Nodes/EntryNode.cs | Supplies label for the Entry node output port. |
| Forge/Statescript/Nodes/ConditionNode.cs | Supplies labels for Condition node ports. |
| Forge/Statescript/Nodes/ActionNode.cs | Supplies labels for Action node ports. |
| docs/statescript/templates/state-node-template.md | Notes that doc port names should match code labels. |
| docs/statescript/templates/condition-node-template.md | Notes that doc port names should match code labels. |
| docs/statescript/templates/action-node-template.md | Notes that doc port names should match code labels. |
| docs/statescript/subgraphs.md | Documents labeling custom event/subgraph ports for editor integrations. |
| docs/statescript/nodes/state/README.md | Documents how to label additional ports in custom state nodes. |
| docs/statescript/nodes/README.md | Adds an overview section explaining port labels and how editors consume them. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Added