sim icon indicating copy to clipboard operation
sim copied to clipboard

[BUG] Period in Node title leads to broken variable reference

Open mkummer225 opened this issue 8 months ago • 1 comments

Describe the bug When adding a period to a node's title there's a bug when referencing that node as a variable.

To Reproduce Steps to reproduce the behavior:

  • Create a new File node connected to your starter node
  • Change the title to "Perf. Node"
  • Create an Agent node with input from the "Perf. Node"
  • Reference the output from "Perf. Node" in the Agent's prompt

Expected behavior Appropriately able to reference the File node whether or not there's a period in it's title

Additional context Potential quick solution: simply replace periods / other non-char symbols that cause issues with underscores or hyphens when building their handle. ie, "Perf. Node" would become "perf_node..." instead of creating an invalid reference to "perf.node..."

mkummer225 avatar May 29 '25 05:05 mkummer225

@mkummer225

Thank you for your bug report! We'll replace the value with something else.

emir-karabeg avatar May 31 '25 03:05 emir-karabeg

A period in a node title breaking variable references indicates an issue with parsing or sanitization. My experience in handling data input, variable parsing, and ensuring robust system references makes me well-suited. I'll focus on identifying where the node title is processed into a variable reference and implement a fix to properly handle or sanitize special characters like periods to prevent invalid references.

Ebube540 avatar Jun 01 '25 11:06 Ebube540

@waleedlatif1 will do some more testing on my end – thanks for the quick turnaround!

mkummer225 avatar Jun 03 '25 17:06 mkummer225