fix: ensure results are arrays for applicable tools

Author: Mubashwer

src/tools/listMobSessionCoauthorTrailers.test.ts

@@ -55,7 +55,13 @@ describe("[tools] listMobSessionCoauthorTrailers", () => {
       expect(listMobSessionCoauthorTrailers).toHaveBeenCalledWith();
       expect(result).toEqual({
         isError: false,
-        content: [{ type: "text", text: trailers }],
+        content: [
+          { type: "text", text: "Co-authored-by: John Doe <john@example.com>" },
+          {
+            type: "text",
+            text: "Co-authored-by: Jane Smith <jane@example.com>",
+          },
+        ],
       });
     });
 
@@ -74,20 +80,5 @@ describe("[tools] listMobSessionCoauthorTrailers", () => {
         content: [{ type: "text", text: errorMessage }],
       });
     });
-
-    it("should handle null/undefined value by returning empty string", async () => {
-      mockListMobSessionCoauthorTrailers.mockResolvedValueOnce({
-        ok: true,
-        value: null,
-      });
-
-      const result = await tool.callback({});
-
-      expect(listMobSessionCoauthorTrailers).toHaveBeenCalledWith();
-      expect(result).toEqual({
-        isError: false,
-        content: [{ type: "text", text: "" }],
-      });
-    });
   });
 });

src/tools/listMobSessionCoauthorTrailers.ts

@@ -23,7 +23,16 @@ const annotations: ToolAnnotations = {
 
 const callback: ToolCallback<typeof inputSchema> = async () => {
   const { ok, value } = await listMobSessionCoauthorTrailers();
-  return { isError: !ok, content: [{ type: "text", text: value || "" }] };
+
+  if (!ok) {
+    return { isError: true, content: [{ type: "text", text: value }] };
+  }
+
+  const lines = value.split("\n").filter((line) => line.trim() !== "");
+  return {
+    isError: false,
+    content: lines.map((line) => ({ type: "text", text: line })),
+  };
 };
 
 const tool: GitMobTool<typeof inputSchema, Record<string, never>> = {

src/tools/listMobSessionCoauthors.test.ts

@@ -53,7 +53,10 @@ describe("[tools] listMobSessionCoauthors", () => {
       expect(listMobSessionCoauthors).toHaveBeenCalledWith();
       expect(result).toEqual({
         isError: false,
-        content: [{ type: "text", text: coauthors }],
+        content: [
+          { type: "text", text: "john John Doe john@example.com" },
+          { type: "text", text: "jane Jane Smith jane@example.com" },
+        ],
       });
     });
 
@@ -72,20 +75,5 @@ describe("[tools] listMobSessionCoauthors", () => {
         content: [{ type: "text", text: errorMessage }],
       });
     });
-
-    it("should handle null/undefined value by returning empty string", async () => {
-      mockListMobSessionCoauthors.mockResolvedValueOnce({
-        ok: true,
-        value: null,
-      });
-
-      const result = await tool.callback({});
-
-      expect(listMobSessionCoauthors).toHaveBeenCalledWith();
-      expect(result).toEqual({
-        isError: false,
-        content: [{ type: "text", text: "" }],
-      });
-    });
   });
 });

src/tools/listMobSessionCoauthors.ts

@@ -22,7 +22,16 @@ const annotations: ToolAnnotations = {
 
 const callback: ToolCallback<typeof inputSchema> = async () => {
   const { ok, value } = await listMobSessionCoauthors();
-  return { isError: !ok, content: [{ type: "text", text: value || "" }] };
+
+  if (!ok) {
+    return { isError: true, content: [{ type: "text", text: value }] };
+  }
+
+  const lines = value.split("\n").filter((line) => line.trim() !== "");
+  return {
+    isError: false,
+    content: lines.map((line) => ({ type: "text", text: line })),
+  };
 };
 
 const tool: GitMobTool<typeof inputSchema, Record<string, never>> = {

src/tools/listTeamMembers.test.ts

@@ -51,7 +51,10 @@ describe("[tools] listTeamMembers", () => {
       expect(listCoauthors).toHaveBeenCalledWith();
       expect(result).toEqual({
         isError: false,
-        content: [{ type: "text", text: teamMembers }],
+        content: [
+          { type: "text", text: "john John Doe john@example.com" },
+          { type: "text", text: "jane Jane Smith jane@example.com" },
+        ],
       });
     });
 
@@ -70,17 +73,5 @@ describe("[tools] listTeamMembers", () => {
         content: [{ type: "text", text: errorMessage }],
       });
     });
-
-    it("should handle null/undefined value by returning empty string", async () => {
-      mockListCoauthors.mockResolvedValueOnce({ ok: true, value: null });
-
-      const result = await tool.callback({});
-
-      expect(listCoauthors).toHaveBeenCalledWith();
-      expect(result).toEqual({
-        isError: false,
-        content: [{ type: "text", text: "" }],
-      });
-    });
   });
 });

src/tools/listTeamMembers.ts

@@ -23,7 +23,16 @@ const annotations: ToolAnnotations = {
 
 const callback: ToolCallback<typeof inputSchema> = async () => {
   const { ok, value } = await listCoauthors();
-  return { isError: !ok, content: [{ type: "text", text: value || "" }] };
+
+  if (!ok) {
+    return { isError: true, content: [{ type: "text", text: value }] };
+  }
+
+  const lines = value.split("\n").filter((line) => line.trim() !== "");
+  return {
+    isError: false,
+    content: lines.map((line) => ({ type: "text", text: line })),
+  };
 };
 
 const tool: GitMobTool<typeof inputSchema, Record<string, never>> = {