Skip to content

Conversation

@Minamiyama
Copy link
Contributor

Summary

When generating default variable names, check for existing names and append a suffix if needed to prevent duplicates

fixes #22037

Screenshots

Before

befor.auto-naming.mp4

After:

after.auto-naming.mp4

Checklist

  • This change requires a documentation update, included: Dify Document
  • I understand that this PR may be closed in case there was no previous discussion or issues. (This doesn't apply to typos!)
  • I've added a test for each change that was introduced, and I tried as much as possible to make a single atomic change.
  • I've updated the documentation accordingly.
  • I ran dev/reformat(backend) and cd web && npx lint-staged(frontend) to appease the lint gods

When generating default variable names, check for existing names and append a suffix if needed to prevent duplicates
@dosubot dosubot bot added size:S This PR changes 10-29 lines, ignoring generated files. ☕️ typescript labels Jul 8, 2025
@crazywoola crazywoola merged commit f925869 into langgenius:main Jul 8, 2025
5 checks passed
@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Jul 8, 2025
yijiaquan pushed a commit to yijiaquan/dify that referenced this pull request Jul 9, 2025
tutkun pushed a commit to tutkun/dify that referenced this pull request Aug 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lgtm This PR has been approved by a maintainer size:S This PR changes 10-29 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Auto-generated Variable Name from Variable Picker Conflicts with Existing Variables

2 participants