From 44b2cceebe19338eb90b3d46ede85e1b108cc985 Mon Sep 17 00:00:00 2001
From: Jarvis AI Team <jarvis@openclaw.ai>
Date: Sat, 16 May 2026 22:22:21 +0800
Subject: [PATCH 1/2] fix: do not escape callstack fields in sanitizeMetadata
 (#103)

---
 src/util.ts                     | 12 ++++++++-
 tests/sanitize-metadata.test.ts | 45 +++++++++++++++++++++++++++++++++
 2 files changed, 56 insertions(+), 1 deletion(-)
 create mode 100644 tests/sanitize-metadata.test.ts

diff --git a/src/util.ts b/src/util.ts
index 538f985..f355833 100644
--- a/src/util.ts
+++ b/src/util.ts
@@ -21,7 +21,17 @@ export interface Options<TEnvSchema extends TSchema = TAnySchema, TSettingsSchem
 }
 
 export function sanitizeMetadata(obj: LogReturn["metadata"]): string {
-  return JSON.stringify(obj, null, 2).replace(/</g, "&lt;").replace(/>/g, "&gt;").replace(/--/g, "&#45;&#45;");
+  if (!obj) return "null";
+  // Extract callstack-related fields that should be preserved unescaped for linking
+  const { stack, callstack, caller, ...content } = obj;
+  // Escape content fields normally
+  const escapedContent = JSON.stringify(content, null, 2).replace(/</g, "&lt;").replace(/>/g, "&gt;").replace(/--/g, "&#45;&#45;");
+  const contentObj = JSON.parse(escapedContent);
+  // Merge callstack fields back without additional escaping (they are safe values)
+  if (stack !== undefined) contentObj.stack = stack;
+  if (callstack !== undefined) contentObj.callstack = callstack;
+  if (caller !== undefined) contentObj.caller = caller;
+  return JSON.stringify(contentObj, null, 2);
 }
 
 /**
diff --git a/tests/sanitize-metadata.test.ts b/tests/sanitize-metadata.test.ts
new file mode 100644
index 0000000..dd5626d
--- /dev/null
+++ b/tests/sanitize-metadata.test.ts
@@ -0,0 +1,45 @@
+import { describe, expect, it } from "@jest/globals";
+import { sanitizeMetadata } from "../src/util";
+
+describe("sanitizeMetadata", () => {
+  it("escapes dangerous characters in content fields", () => {
+    const obj = {
+      message: "<script>alert('xss')</script>",
+      status: 200,
+    };
+    const result = sanitizeMetadata(obj);
+    expect(result).toContain("&lt;script&gt;");
+    expect(result).not.toContain("<script>");
+    expect(result).toContain("200");
+  });
+
+  it("does not escape stack, callstack, or caller fields", () => {
+    const obj = {
+      message: "Error occurred",
+      stack: "Error: msg\n    at foo (file.js:10:5)\n    at bar (file.js:20:10)",
+      callstack: ["foo", "bar"],
+      caller: "myFunction",
+    };
+    const result = sanitizeMetadata(obj);
+    // stack should NOT be escaped (no &lt;/&gt;)
+    expect(result).toContain('"stack":');
+    expect(result).toContain("(file.js:10:5)");
+    expect(result).toContain("callstack");
+    expect(result).toContain("caller");
+    // content message should be escaped if it had special chars
+    expect(result).toContain("Error occurred");
+  });
+
+  it("handles null/undefined gracefully", () => {
+    expect(sanitizeMetadata(null)).toBe("null");
+    expect(sanitizeMetadata(undefined)).toBe("null");
+  });
+
+  it("escapes double-dash sequences in content", () => {
+    const obj = {
+      message: "text with -- double dash",
+    };
+    const result = sanitizeMetadata(obj);
+    expect(result).toContain("&#45;&#45;");
+  });
+});

From 3896070e080cec739f26d35f45fa6207d6b74b35 Mon Sep 17 00:00:00 2001
From: wbobbynmworley <wbobbynmworley@gmail.com>
Date: Sat, 16 May 2026 22:24:56 +0800
Subject: [PATCH 2/2] fix: allow string in postComment instead of requiring
 logger.info (#50)

---
 src/comment.ts | 9 +++++++--
 1 file changed, 7 insertions(+), 2 deletions(-)

diff --git a/src/comment.ts b/src/comment.ts
index 1cc40a1..fa06497 100644
--- a/src/comment.ts
+++ b/src/comment.ts
@@ -379,7 +379,7 @@ export class CommentHandler {
 
   async postComment(
     context: Context,
-    message: LogReturn | Error,
+    message: LogReturn | Error | string,
     options: CommentOptions = { updateComment: true, raw: false }
   ): Promise<WithIssueNumber<PostedGithubComment> | null> {
     await this._applyCommandResponsePolicy(context);
@@ -390,8 +390,13 @@ export class CommentHandler {
       return null;
     }
 
+    // Convert string to LogReturn-like object
+    const logMessage: LogReturn | Error = typeof message === "string"
+      ? ({ raw: message, diff: message, metadata: {} } as LogReturn)
+      : message;
+
     const shouldTagCommandResponse = this._shouldApplyCommandResponsePolicy(context);
-    const body = this._createCommentBody(context, message, {
+    const body = this._createCommentBody(context, logMessage, {
       ...options,
       commentKind: options.commentKind ?? (shouldTagCommandResponse ? COMMAND_RESPONSE_KIND : undefined),
     });