Add uuid4FromHash utility to codegen (node_id skipping disabled pending vembda fix)

[devin-ai-integration]

Summary

Duplicates the uuid4_from_hash utility from ee/vellum_ee into ee/codegen as a TypeScript implementation. The utility is designed to skip generating the node_id attribute in Node Display classes when the UUID matches what would be generated from the node's definition module path and class name (matching the Python SDK's uuid4_from_hash(f"{node_class.__module__}.{node_class.__qualname__}") pattern).

Note: The node_id skipping check is currently commented out pending vembda-side fixes. The utility and tests are in place but disabled until those issues are resolved.

Review & Testing Checklist for Human

• Verify the TypeScript uuid4FromHash implementation produces identical output to the Python uuid4_from_hash for the same inputs (tested with my_nodes.my_custom_node.MyCustomNode → 1be19097-e8d7-4acc-8bb5-2a99dc12dece)
• Confirm the getNodeIdFromDefinition helper correctly handles undefined definitions by returning undefined
• When vembda-side issues are resolved, unskip the tests and uncomment the check in base.ts

Notes

• The node_id skipping logic in base.ts is commented out with TODO markers
• Tests in generic-node.test.ts are skipped with describe.skip until vembda-side issues are resolved
• Pre-existing test failures in base-case-merge.test.ts (7 tests) are unrelated to this change
• Requested by: @dvargas92495
• Session: https://app.devin.ai/sessions/f1c7ee093d264ac5bae7236faa8a8b19

[devin-ai-integration]

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

• Address comments on this PR that start with 'DevinAI' or '@devin'.
• Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

• Disable automatic comment and CI monitoring

[dvargas92495]

DevinAI add a snapshot test to generic-node.test.ts to show this in action

[devin-ai-integration]

Added snapshot tests to generic-node.test.ts demonstrating the node_id skipping behavior:

1. "should skip node_id when it matches hash" - Uses node ID 1be19097-e8d7-4acc-8bb5-2a99dc12dece which matches the hash of my_nodes.my_custom_node.MyCustomNode, so node_id is omitted from the display class
2. "should include node_id when it does not match hash" - Uses node ID aaaaaaaa-bbbb-cccc-dddd-eeeeeeeeeeee which doesn't match, so node_id is included

Also fixed the implementation to use nodeData.definition.module and nodeData.definition.name (matching Python SDK's node_class.__module__ and node_class.__qualname__) instead of the generated module path.

[dvargas92495]

DevinAI update this helper to accept the full definition instead of forcing the consumer to split the fields

[devin-ai-integration]

Addressed all feedback:

1. Updated helper to accept full definition - Renamed getNodeIdFromModuleAndName to getNodeIdFromDefinition and updated it to accept the full definition object (with module and name properties) instead of separate fields

2. Commented out the node_id check - The check in base.ts is now commented out with a TODO marker pending vembda-side fixes

3. Skipped the tests - The describe block for node_id skipping tests is now using describe.skip with a TODO marker

[dvargas92495]

DevinAI comment out this check for now - there are some vembda-side problems we need to solve first

[dvargas92495]

DevinAI keep this test but skip it for now while we solve some issues on the vembda side

[NgoHarrison]

✅