fix: stop error when run has no output

Instead, only error when the run has an error

Signed-off-by: Donnie Adams <donnie@acorn.io>

Author: thedadams

src/gptscript.ts

@@ -480,11 +480,11 @@ export class Run {
 
                 res.on("end", () => {
                     if (this.state === RunState.Running || this.state === RunState.Finished || this.state === RunState.Continue) {
-                        if (this.stdout) {
+                        if (this.stdout || !this.stderr) {
                             if (this.state !== RunState.Continue) {
                                 this.state = RunState.Finished
                             }
-                            resolve(this.stdout)
+                            resolve(this.stdout || "")
                         } else {
                             this.state = RunState.Error
                             reject(new Error(this.stderr))

tests/gptscript.test.ts

@@ -1,5 +1,5 @@
 import * as gptscript from "../src/gptscript"
-import {ArgumentSchemaType, getEnv, PropertyType, RunEventType, TextType, ToolType} from "../src/gptscript"
+import {ArgumentSchemaType, getEnv, PropertyType, RunEventType, TextType, ToolDef, ToolType} from "../src/gptscript"
 import path from "path"
 import {fileURLToPath} from "url"
 import * as fs from "node:fs"
@@ -85,6 +85,29 @@ describe("gptscript module", () => {
         expect(await run.text()).toContain("Calvin Coolidge")
     })
 
+    test("evaluate executes subtool with empty instructions", async () => {
+        const tools = [
+            {
+                type: "tool",
+                tools: ["new-tool-1"],
+                instructions: "Ask the user for their 'first name'. Then reply hello to the user.",
+            } as ToolDef,
+            {
+                type: "tool",
+                name: "new-tool-1",
+            } as ToolDef,
+        ]
+        const run = await g.evaluate(tools, {
+            input: "{}",
+            disableCache: true,
+            workspace: "",
+            subTool: "new-tool-1",
+        })
+
+        expect(run).toBeDefined()
+        expect(await run.text()).toContain("Understood.")
+    })
+
     test("evaluate executes and streams a prompt correctly", async () => {
         let out = ""
         let err = undefined