Improve command output handling: return specific success message for empty output

Author: habedi

src/agent/state.ts

@@ -240,6 +240,10 @@ export const useStore = create<AppState & AppActions>((set, get) => ({
                                 { toolName: toolCall.toolName, args: toolCall.input },
                                 config,
                             );
+                            const output =
+                                result === ""
+                                    ? "Tool executed successfully and produced no output."
+                                    : result;
                             toolResults.push({
                                 id: crypto.randomUUID(),
                                 role: "tool",
@@ -249,7 +253,7 @@ export const useStore = create<AppState & AppActions>((set, get) => ({
                                         toolCallId: toolCall.toolCallId,
                                         toolName: toolCall.toolName,
                                         // eslint-disable-next-line @typescript-eslint/no-explicit-any
-                                        output: result as any,
+                                        output: output as any,
                                     },
                                 ],
                             });

src/agent/tools/definitions/bash.ts

@@ -42,7 +42,11 @@ async function implementation(args: z.infer<typeof bashSchema>["arguments"]): Pr
 
         child.on("close", (code) => {
             if (code === 0) {
-                resolve(output);
+                if (output.trim() === "") {
+                    resolve("Command executed successfully with no output.");
+                } else {
+                    resolve(output);
+                }
             } else {
                 reject(new ToolError(`Command exited with code: ${code}\nOutput:\n${output}`));
             }

tests/agent/tools/definitions/bash.test.ts

@@ -53,7 +53,7 @@ describe("bash tool", () => {
 
         childProcessMock.emit("close", 0);
 
-        await expect(promise).resolves.toBe("");
+        await expect(promise).resolves.toBe("Command executed successfully with no output.");
         expect(spawnMock).toHaveBeenCalledWith("sleep 5", {
             cwd: process.cwd(),
             shell: "/bin/bash",