feat(dashboard): Invalid variable pill improvements

[BiswaViraj]

What changed? Why was the change needed?

Screenshots

Expand for optional sections

Related enterprise PR

Special notes for your reviewer

[linear]

NV-5616 🐛 Do not create pill for {{payload}} or {{steps.digest.events.payload}}

[netlify]

❌ Deploy Preview for dashboard-v2-novu-staging failed. Why did it fail? →

Name	Link
🔨 Latest commit	ecee771
🔍 Latest deploy log	https://app.netlify.com/sites/dashboard-v2-novu-staging/deploys/67f7790429d5d500081a3f0a

[Copilot]

Copilot reviewed 5 out of 5 changed files in this pull request and generated no comments.

Comments suppressed due to low confidence (2)

apps/dashboard/src/utils/liquid-autocomplete.ts:275

• The wrapping logic conditionally appends punctuation, which could lead to inconsistent formatting of inserted variables. Please verify that when isInvalidValue is true, skipping the closing brackets or adding a dot is the intended behavior.

const wrappedValue = `${needsOpening ? '{{' : ''}${selectedValue}${isInvalidValue && !selectedValue.endsWith('.') ? '.' : ''}${needsClosing && !isInvalidValue ? '}}' : ''}`;

apps/dashboard/src/components/workflow-editor/steps/email/maily.tsx:226

• [nitpick] Concatenating the dynamic path root with a trimmed query may produce unexpected variable names when the query is empty. Consider adding a condition to only append the query if it is non-empty to ensure fully qualified suggestions.

name: value + query.trim(),