Stop escaping generated patches in comments (#18)

* Stop escaping generated patches in comments

* Verify rendered patch writes preserve quotes

* Bundle

* Update AGENTS.md

Author: shuyangli

AGENTS.md

@@ -49,3 +49,8 @@ requests should describe the motivation, summarize testing (`npm test`,
 `npm run bundle`), and link the tracking issue. Attach screenshots for
 user-visible output changes and ensure generated `dist/` artifacts are updated
 alongside source when behavior changes.
+
+## IMPORTANT RULES
+
+Before submitting a PR, run `npm run bundle` to update the javascript bundles.
+Otherwise, CI will fail.

dist/generate-pr-patch/index.js

@@ -1,3 +1,7 @@
+import fs from 'fs'
+import os from 'os'
+import path from 'path'
+
 import { renderComment } from './pullRequestCommentTemplate.js'
 
 describe('renderComment', () => {
@@ -53,6 +57,43 @@ describe('renderComment', () => {
     expect(result).toContain('patch contents')
   })
 
+  it('renders the generated patch without HTML escaping', () => {
+    const result = renderComment({
+      generatedCommentBody: 'CI failed on the lint step',
+      followupPrCreationError: 'Authentication failed',
+      generatedPatch:
+        'diff --git a/example.ts b/example.ts\n@@ -1,4 +1,4 @@\n-import { something } from "old"\n+import { something } from "new"'
+    })
+
+    expect(result).toContain('import { something } from "old"')
+    expect(result).toContain('import { something } from "new"')
+    expect(result).not.toContain('"')
+  })
+
+  it('preserves quotes when the rendered comment is written to disk', () => {
+    const result = renderComment({
+      generatedCommentBody: 'CI failed on the lint step',
+      followupPrCreationError: 'Authentication failed',
+      generatedPatch:
+        'diff --git a/example.ts b/example.ts\n@@ -1,4 +1,4 @@\n-import { something } from "old"\n+import { something } from "new"'
+    })
+
+    expect(result).toBeDefined()
+
+    const tmpDir = fs.mkdtempSync(path.join(os.tmpdir(), 'comment-test-'))
+    try {
+      const filePath = path.join(tmpDir, 'comment.md')
+      fs.writeFileSync(filePath, result ?? '', { encoding: 'utf-8' })
+
+      const fileContents = fs.readFileSync(filePath, { encoding: 'utf-8' })
+      expect(fileContents).toContain('import { something } from "old"')
+      expect(fileContents).toContain('import { something } from "new"')
+      expect(fileContents).not.toContain('"')
+    } finally {
+      fs.rmSync(tmpDir, { recursive: true, force: true })
+    }
+  })
+
   it('does not render patch if passed', () => {
     const result = renderComment({
       generatedCommentBody: 'CI failed on the lint step',

dist/generate-pr-patch/index.js.map

@@ -27,7 +27,7 @@ I've opened an automated follow-up PR #{{followupPrNumber}} with proposed fixes.
 {{#if generatedPatch}}
 The patch I tried to generate is as follows:
 \`\`\`diff
-{{generatedPatch}}
+{{{generatedPatch}}}
 \`\`\`
 {{else}}
 No patch was generated.