fix: Improve json repair logic

[yehorkardash]

Summary

This PR introduces a new jsonrepair dependency that replaces unstable regex based repair logic

Related Linear tickets, Github issues, and Community forum posts

https://linear.app/n8n/issue/NODE-1773/community-issue-json-is-being-modified-incorrectly-invalidated-in-edit

closes #10972

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)

[bundlemon]

BundleMon

Unchanged files (2)

Status	Path	Size	Limits
✅	WASM Dependencies tree-sitter-bash.wasm	181.26KB	-
✅	WASM Dependencies tree-sitter.wasm	74.47KB	-

No change in files bundle size

Groups updated (2)

Status	Path	Size	Limits
✅	**/*.js	11.38MB (+91.49KB +0.79%)	-
✅	**/*.css	231.46KB (+10.68KB +4.84%)	-

Final result: ✅

View report in BundleMon website ➡️

---

Current branch size history | Target branch size history

[cubic-dev-ai]

3 issues found across 8 files

Prompt for AI agents (all 3 issues)

Understand the root cause of the following 3 issues and fix them.


<file name="packages/workflow/test/utils.test.ts">

<violation number="1" location="packages/workflow/test/utils.test.ts:194">
The apostrophe contraction test asserts `{ key: &#39;it&quot;s a test&#39; }`, forcing the parser to corrupt the original string instead of preserving `it&#39;s a test`; the expectation should keep the apostrophe so JSON repair fixes syntax without altering data.</violation>
</file>

<file name="packages/workflow/package.json">

<violation number="1" location="packages/workflow/package.json:51">
jsonrepair is declared as a devDependency even though src/utils.ts imports it at runtime, so downstream consumers of n8n-workflow will miss this module and `import { jsonrepair } from &#39;jsonrepair&#39;` will throw. Declare jsonrepair under dependencies instead of devDependencies.</violation>
</file>

<file name="packages/workflow/src/utils.ts">

<violation number="1" location="packages/workflow/src/utils.ts:144">
Rule violated: **Prefer Typeguards over Type casting**

Avoid asserting the repaired JSON result to `T`. There is no runtime verification that the repaired string matches the caller-specified type, so this cast hides type errors; let TypeScript accept the `any` result or introduce a guard before narrowing.</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 66.66667% with 4 lines in your changes missing coverage. Please review.

Files with missing lines	Patch %	Lines
...des-langchain/nodes/tools/ToolHttpRequest/utils.ts	0.00%	2 Missing ⚠️
packages/workflow/src/utils.ts	75.00%	2 Missing ⚠️

📢 Thoughts on this report? Let us know!

[RomanDavydchuk]

Works like a charm

Left one comment. In general, we try to be careful with adding new dependencies to the project, and avoid it, if it's possible. IMO this package is the best solution for this issue, so it's probably ok to add it, but did you discuss this with anyone else?

[yehorkardash]

but did you discuss this with anyone else

Mentioned that on the daily, didn't hear objections

[currents-bot]

✅

E2E Tests: n8n tests passed after 9m 13.7s

🟢 588 · 🔴 0 · ⚪️ 12 · 🟣 1

View Run Details

Run Details

• Project: n8n

• Groups: 2

• Framework: Playwright

• Run Status: Passed

• Commit: 21faede

• Spec files: 96

• Overall tests: 600

• Duration: 9m 13.7s

• Parallelization: 9

Groups

GroupId	Results	Spec Files Progress
ui	🟢 539 · 🔴 0 · ⚪️ 12 · 🟣 1	90 / 90
ui:isolated	🟢 49 · 🔴 0 · ⚪️ 0	6 / 6

---

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

[RomanDavydchuk]

LGTM

[n8n-assistant]

Got released with [email protected]