fix(files): destroy round-trip probe editor on serialization error

Wrap the probe serialize() in try/finally so the throwaway Editor is always destroyed even if setContent/getMarkdown throws (addresses Greptile). Adds a test proving PipeSafeTable escapes only interior cell pipes, not structural delimiters.

Author: waleedlatif1

apps/sim/app/workspace/[workspaceId]/files/components/file-viewer/rich-markdown-editor/round-trip-safety.ts

@@ -60,10 +60,12 @@ function stripCode(content: string): string {
 function serialize(content: string): string {
   const { frontmatter, body } = splitFrontmatter(content)
   const editor = new Editor({ extensions: createMarkdownContentExtensions() })
-  editor.commands.setContent(body, { contentType: 'markdown' })
-  const out = applyFrontmatter(frontmatter, postProcessSerializedMarkdown(editor.getMarkdown()))
-  editor.destroy()
-  return out
+  try {
+    editor.commands.setContent(body, { contentType: 'markdown' })
+    return applyFrontmatter(frontmatter, postProcessSerializedMarkdown(editor.getMarkdown()))
+  } finally {
+    editor.destroy()
+  }
 }
 
 /**

apps/sim/app/workspace/[workspaceId]/files/components/file-viewer/rich-markdown-editor/round-trip.test.ts

@@ -206,6 +206,17 @@ describe('editor markdown round-trip', () => {
     expect(out).toContain('| --- |')
   })
 
+  it('escapes only interior cell pipes, not the structural delimiters', () => {
+    const out = roundTrip('| a | b |\n| --- | --- |\n| one \\| two | three |')
+    expect(out).toContain('one \\| two')
+    expect(out).toContain('| three |')
+    // Every row keeps exactly its two structural columns (3 pipes per line).
+    for (const line of out.trim().split('\n')) {
+      expect((line.match(/(?<!\\)\|/g) ?? []).length).toBe(3)
+    }
+    expect(roundTrip(out)).toBe(out)
+  })
+
   it('combines strikethrough with inline code (relaxed code mark)', () => {
     expect(roundTrip('~~`x`~~')).toContain('~~`x`~~')
     expect(roundTrip('# ~~`x`~~')).toContain('# ~~`x`~~')