fix(Data Table Node): Convert dates to ISO strings in the output

[dariacodes]

Summary

Before:

After:

Related Linear tickets, Github issues, and Community forum posts

Closes ADO-4482

Review / Merge checklist

• PR title and summary are descriptive. (conventions)
• Docs updated or follow-up ticket created.
• Tests included.
• PR Labeled with release/backport (if the PR is an urgent fix that needs to be backported)

[Joffcom]

@dariacodes this feels like a potential breaking change, If a user has worked around this in their workflow the new output may be unexpected. This will likely need a light version and a test to cover the change.

[cubic-dev-ai]

1 issue found across 1 file

Prompt for AI agents (all 1 issues)

Check if these issues are valid — if so, understand the root cause of each and fix them.


<file name="packages/nodes-base/nodes/Set/v2/helpers/utils.ts">

<violation number="1" location="packages/nodes-base/nodes/Set/v2/helpers/utils.ts:186">
P1: Rule violated: **Tests**

This bug fix adds Date-to-string conversion but lacks a corresponding test. Existing tests in `packages/nodes-base/nodes/Set/test/v2/utils.test.ts` cover all other type conversions (number, array, object, boolean, undefined) for `validateEntry`, but Date is missing. Add a test like:

```typescript
it(&#39;should convert Date to ISO string&#39;, () =&gt; {
  const testDate = new Date(&#39;2025-01-15T12:00:00.000Z&#39;);
  const result = validateEntry(&#39;foo&#39;, &#39;string&#39;, testDate, node, 0);

  expect(result).toEqual({
    name: &#39;foo&#39;,
    value: &#39;2025-01-15T12:00:00.000Z&#39;,
  });
});
```</violation>
</file>

_{Reply to cubic to teach it or ask questions. Re-run a review with @cubic-dev-ai review this PR}

[codecov]

Codecov Report

❌ Patch coverage is 75.00000% with 4 lines in your changes missing coverage. Please review.

Files with missing lines	Patch %	Lines
...es/nodes-base/nodes/DataTable/common/selectMany.ts	75.00%	3 Missing and 1 partial ⚠️

📢 Thoughts on this report? Let us know!

[currents-bot]

✅

E2E Tests: n8n tests passed after 5m 17.3s

🟢 69 · 🔴 0 · ⚪️ 1 · 🟣 1

View Run Details

Run Details

• Project: n8n

• Groups: 1

• Framework: Playwright

• Run Status: Passed

• Commit: f0612d6

• Spec files: 121

• Overall tests: 559

• Duration: 5m 17.3s

• Parallelization: 1

---

This message was posted automatically by currents.dev | Integration Settings

[dariacodes]

@Joffcom Good point! I ended up fixing this on Data Table node and created a new version. I had to introduce a conditional logic in the code, but I don't see a better way

[cubic-dev-ai]

No issues found across 3 files

[autologie]

👍

[n8n-assistant]

Got released with n8n@n8n@2.1.0