fix(editor): Prevent duplicate values in preview for SQL editor

[elsmr]

Summary

SQL query preview should not include duplicates (specifically brackets for this issue).

• Exclude bracket segments from preview
• Harden the ExpressionEditorOutput to not display duplicate segments

Related tickets and issues

https://linear.app/n8n/issue/NODE-1286/postgres-sqleditor-duplicate-values-in-preview
https://community.n8n.io/t/postgresql-node-using-array-something-duplicates-the-output/43373

Review / Merge checklist

• PR title and summary are descriptive. Remember, the title automatically goes into the changelog. Use (no-changelog) otherwise. (conventions)
• Docs updated or follow-up ticket created.
• Tests included.

  A bug is not considered fixed, unless a test is added to prevent it from happening again.
  A feature is not complete without tests.

[netroy]

resolvedExpression in packages/editor-ui/src/components/ExpressionEditorModal/ExpressionEditorModalOutput.vue has the exact same code. Do we not need to update there as well?

[elsmr]

resolvedExpression in packages/editor-ui/src/components/ExpressionEditorModal/ExpressionEditorModalOutput.vue has the exact same code. Do we not need to update there as well?

Good point, I extracted the common logic into a new component

[cypress]

1 flaky test on run #4735 ↗︎

0

356

12

0

1

Details:

🌳 🖥️ browsers:node18.12.0-chrome107 🤖 elsmr 🗃️ e2e/*
Project: n8n	Commit: b4359a042d
Status: Passed	Duration: 04:35 💡
Started: Apr 22, 2024 8:21 AM	Ended: Apr 22, 2024 8:25 AM

  cypress/e2e/5-ndv.cy.ts • 1 flaky test

View Output Video

Test		Artifacts
NDV > Stop listening for trigger event from NDV		Screenshots Video

Review all test suite changes for PR #9129 ↗︎

[github-actions]

✅ All Cypress E2E specs passed

[janober]

Got released with n8n@1.39.0