You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
This pull request introduces a new validator to ensure node names are unique within a graph and adds comprehensive test coverage for this validation. The main changes include the implementation of the validator, a new invalid test resource, and an additional test case.
Validation logic:
Added UniqueNodeNameValidator in unique_node_name.py to check that all node names (excluding "Input" and "Output") are unique within the graph, returning a detailed error message if duplicates are found.
Testing:
Added a new invalid graph resource duplicate-node-names.json containing nodes with duplicate names to test the validator.
Extended the test suite in test_executor.py to include a case that expects a duplicate node name error when processing the new resource.
Base automatically changed from
f/improve-execution-serialization to
mainJanuary 29, 2026 13:39
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
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.
This pull request introduces a new validator to ensure node names are unique within a graph and adds comprehensive test coverage for this validation. The main changes include the implementation of the validator, a new invalid test resource, and an additional test case.
Validation logic:
UniqueNodeNameValidatorinunique_node_name.pyto check that all node names (excluding "Input" and "Output") are unique within the graph, returning a detailed error message if duplicates are found.Testing:
duplicate-node-names.jsoncontaining nodes with duplicate names to test the validator.test_executor.pyto include a case that expects a duplicate node name error when processing the new resource.