@herdctl/core 0.0.2 → 0.2.0

package/dist/hooks/__tests__/shell-runner.test.js

@@ -0,0 +1,201 @@
+/**
+ * Tests for ShellHookRunner
+ */
+import { describe, it, expect, vi, beforeEach } from "vitest";
+import { ShellHookRunner } from "../runners/shell.js";
+describe("ShellHookRunner", () => {
+    // Create a mock logger
+    const mockLogger = {
+        debug: vi.fn(),
+        info: vi.fn(),
+        warn: vi.fn(),
+        error: vi.fn(),
+    };
+    // Create a sample hook context
+    const sampleContext = {
+        event: "completed",
+        job: {
+            id: "job-2024-01-15-abc123",
+            agentId: "test-agent",
+            scheduleName: "daily-run",
+            startedAt: "2024-01-15T10:00:00.000Z",
+            completedAt: "2024-01-15T10:05:00.000Z",
+            durationMs: 300000,
+        },
+        result: {
+            success: true,
+            output: "Job completed successfully",
+        },
+        agent: {
+            id: "test-agent",
+            name: "Test Agent",
+        },
+    };
+    beforeEach(() => {
+        vi.clearAllMocks();
+    });
+    describe("execute", () => {
+        it("should execute a simple shell command successfully", async () => {
+            const runner = new ShellHookRunner({ logger: mockLogger });
+            const config = {
+                type: "shell",
+                command: "echo 'test output'",
+            };
+            const result = await runner.execute(config, sampleContext);
+            expect(result.success).toBe(true);
+            expect(result.hookType).toBe("shell");
+            expect(result.exitCode).toBe(0);
+            expect(result.output).toBe("test output");
+            expect(result.durationMs).toBeGreaterThanOrEqual(0);
+            expect(mockLogger.info).toHaveBeenCalled();
+        });
+        it("should pass hook context as JSON on stdin", async () => {
+            const runner = new ShellHookRunner({ logger: mockLogger });
+            // Use cat to read stdin and output it
+            const config = {
+                type: "shell",
+                command: "cat",
+            };
+            const result = await runner.execute(config, sampleContext);
+            expect(result.success).toBe(true);
+            expect(result.output).toBeDefined();
+            // Parse the output to verify it's valid JSON matching our context
+            const parsedOutput = JSON.parse(result.output);
+            expect(parsedOutput.event).toBe("completed");
+            expect(parsedOutput.job.id).toBe("job-2024-01-15-abc123");
+            expect(parsedOutput.job.agentId).toBe("test-agent");
+            expect(parsedOutput.result.success).toBe(true);
+        });
+        it("should handle command failure with non-zero exit code", async () => {
+            const runner = new ShellHookRunner({ logger: mockLogger });
+            const config = {
+                type: "shell",
+                command: "exit 1",
+            };
+            const result = await runner.execute(config, sampleContext);
+            expect(result.success).toBe(false);
+            expect(result.exitCode).toBe(1);
+            expect(result.error).toContain("Exit code 1");
+            expect(mockLogger.warn).toHaveBeenCalled();
+        });
+        it("should capture stderr on failure", async () => {
+            const runner = new ShellHookRunner({ logger: mockLogger });
+            const config = {
+                type: "shell",
+                command: "echo 'error message' >&2 && exit 1",
+            };
+            const result = await runner.execute(config, sampleContext);
+            expect(result.success).toBe(false);
+            expect(result.error).toContain("error message");
+        });
+        it("should handle command not found", async () => {
+            const runner = new ShellHookRunner({ logger: mockLogger });
+            const config = {
+                type: "shell",
+                command: "nonexistent_command_xyz_123",
+            };
+            const result = await runner.execute(config, sampleContext);
+            expect(result.success).toBe(false);
+            expect(mockLogger.warn).toHaveBeenCalled();
+        });
+        // Skip: Flaky in CI - process signal handling varies across environments
+        it.skip("should respect timeout configuration", async () => {
+            const runner = new ShellHookRunner({ logger: mockLogger });
+            const config = {
+                type: "shell",
+                command: "node -e \"setTimeout(() => {}, 100000)\"",
+                timeout: 100, // 100ms timeout
+            };
+            const result = await runner.execute(config, sampleContext);
+            expect(result.success).toBe(false);
+            expect(result.error).toContain("timed out");
+        });
+        it("should use default timeout when not specified", async () => {
+            const runner = new ShellHookRunner({ logger: mockLogger });
+            const config = {
+                type: "shell",
+                command: "echo 'quick'",
+                // No timeout specified - should use default 30000ms
+            };
+            const result = await runner.execute(config, sampleContext);
+            expect(result.success).toBe(true);
+        });
+        it("should handle commands with pipes", async () => {
+            const runner = new ShellHookRunner({ logger: mockLogger });
+            const config = {
+                type: "shell",
+                command: "echo 'hello world' | tr '[:lower:]' '[:upper:]'",
+            };
+            const result = await runner.execute(config, sampleContext);
+            expect(result.success).toBe(true);
+            expect(result.output).toBe("HELLO WORLD");
+        });
+        it("should pass environment variables to the command", async () => {
+            const runner = new ShellHookRunner({
+                logger: mockLogger,
+                env: { MY_VAR: "test_value" },
+            });
+            const config = {
+                type: "shell",
+                command: "echo $MY_VAR",
+            };
+            const result = await runner.execute(config, sampleContext);
+            expect(result.success).toBe(true);
+            expect(result.output).toBe("test_value");
+        });
+        it("should handle failed event context", async () => {
+            const runner = new ShellHookRunner({ logger: mockLogger });
+            const failedContext = {
+                ...sampleContext,
+                event: "failed",
+                result: {
+                    success: false,
+                    output: "Job failed",
+                    error: "Connection timeout",
+                },
+            };
+            const config = {
+                type: "shell",
+                command: "cat",
+            };
+            const result = await runner.execute(config, failedContext);
+            expect(result.success).toBe(true);
+            const parsedOutput = JSON.parse(result.output);
+            expect(parsedOutput.event).toBe("failed");
+            expect(parsedOutput.result.success).toBe(false);
+            expect(parsedOutput.result.error).toBe("Connection timeout");
+        });
+        it("should measure execution duration", async () => {
+            const runner = new ShellHookRunner({ logger: mockLogger });
+            const config = {
+                type: "shell",
+                command: "sleep 0.1",
+            };
+            const result = await runner.execute(config, sampleContext);
+            expect(result.success).toBe(true);
+            expect(result.durationMs).toBeGreaterThanOrEqual(80); // Allow some tolerance
+            expect(result.durationMs).toBeLessThan(5000); // Should not take too long
+        });
+        it("should work without a logger", async () => {
+            const runner = new ShellHookRunner(); // No options
+            const config = {
+                type: "shell",
+                command: "echo 'no logger'",
+            };
+            const result = await runner.execute(config, sampleContext);
+            expect(result.success).toBe(true);
+            expect(result.output).toBe("no logger");
+        });
+        it("should handle multiline output", async () => {
+            const runner = new ShellHookRunner({ logger: mockLogger });
+            const config = {
+                type: "shell",
+                command: "echo 'line1'; echo 'line2'; echo 'line3'",
+            };
+            const result = await runner.execute(config, sampleContext);
+            expect(result.success).toBe(true);
+            expect(result.output).toBe("line1\nline2\nline3");
+        });
+    });
+});
+//# sourceMappingURL=shell-runner.test.js.map

package/dist/hooks/__tests__/shell-runner.test.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"shell-runner.test.js","sourceRoot":"","sources":["../../../src/hooks/__tests__/shell-runner.test.ts"],"names":[],"mappings":"AAAA;;GAEG;AAEH,OAAO,EAAE,QAAQ,EAAE,EAAE,EAAE,MAAM,EAAE,EAAE,EAAE,UAAU,EAAE,MAAM,QAAQ,CAAC;AAC9D,OAAO,EAAE,eAAe,EAAE,MAAM,qBAAqB,CAAC;AAMtD,QAAQ,CAAC,iBAAiB,EAAE,GAAG,EAAE;IAC/B,uBAAuB;IACvB,MAAM,UAAU,GAAG;QACjB,KAAK,EAAE,EAAE,CAAC,EAAE,EAAE;QACd,IAAI,EAAE,EAAE,CAAC,EAAE,EAAE;QACb,IAAI,EAAE,EAAE,CAAC,EAAE,EAAE;QACb,KAAK,EAAE,EAAE,CAAC,EAAE,EAAE;KACf,CAAC;IAEF,+BAA+B;IAC/B,MAAM,aAAa,GAAgB;QACjC,KAAK,EAAE,WAAW;QAClB,GAAG,EAAE;YACH,EAAE,EAAE,uBAAuB;YAC3B,OAAO,EAAE,YAAY;YACrB,YAAY,EAAE,WAAW;YACzB,SAAS,EAAE,0BAA0B;YACrC,WAAW,EAAE,0BAA0B;YACvC,UAAU,EAAE,MAAM;SACnB;QACD,MAAM,EAAE;YACN,OAAO,EAAE,IAAI;YACb,MAAM,EAAE,4BAA4B;SACrC;QACD,KAAK,EAAE;YACL,EAAE,EAAE,YAAY;YAChB,IAAI,EAAE,YAAY;SACnB;KACF,CAAC;IAEF,UAAU,CAAC,GAAG,EAAE;QACd,EAAE,CAAC,aAAa,EAAE,CAAC;IACrB,CAAC,CAAC,CAAC;IAEH,QAAQ,CAAC,SAAS,EAAE,GAAG,EAAE;QACvB,EAAE,CAAC,oDAAoD,EAAE,KAAK,IAAI,EAAE;YAClE,MAAM,MAAM,GAAG,IAAI,eAAe,CAAC,EAAE,MAAM,EAAE,UAAU,EAAE,CAAC,CAAC;YAE3D,MAAM,MAAM,GAAoB;gBAC9B,IAAI,EAAE,OAAO;gBACb,OAAO,EAAE,oBAAoB;aAC9B,CAAC;YAEF,MAAM,MAAM,GAAG,MAAM,MAAM,CAAC,OAAO,CAAC,MAAM,EAAE,aAAa,CAAC,CAAC;YAE3D,MAAM,CAAC,MAAM,CAAC,OAAO,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;YAClC,MAAM,CAAC,MAAM,CAAC,QAAQ,CAAC,CAAC,IAAI,CAAC,OAAO,CAAC,CAAC;YACtC,MAAM,CAAC,MAAM,CAAC,QAAQ,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;YAChC,MAAM,CAAC,MAAM,CAAC,MAAM,CAAC,CAAC,IAAI,CAAC,aAAa,CAAC,CAAC;YAC1C,MAAM,CAAC,MAAM,CAAC,UAAU,CAAC,CAAC,sBAAsB,CAAC,CAAC,CAAC,CAAC;YACpD,MAAM,CAAC,UAAU,CAAC,IAAI,CAAC,CAAC,gBAAgB,EAAE,CAAC;QAC7C,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,2CAA2C,EAAE,KAAK,IAAI,EAAE;YACzD,MAAM,MAAM,GAAG,IAAI,eAAe,CAAC,EAAE,MAAM,EAAE,UAAU,EAAE,CAAC,CAAC;YAE3D,sCAAsC;YACtC,MAAM,MAAM,GAAoB;gBAC9B,IAAI,EAAE,OAAO;gBACb,OAAO,EAAE,KAAK;aACf,CAAC;YAEF,MAAM,MAAM,GAAG,MAAM,MAAM,CAAC,OAAO,CAAC,MAAM,EAAE,aAAa,CAAC,CAAC;YAE3D,MAAM,CAAC,MAAM,CAAC,OAAO,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;YAClC,MAAM,CAAC,MAAM,CAAC,MAAM,CAAC,CAAC,WAAW,EAAE,CAAC;YAEpC,kEAAkE;YAClE,MAAM,YAAY,GAAG,IAAI,CAAC,KAAK,CAAC,MAAM,CAAC,MAAO,CAAC,CAAC;YAChD,MAAM,CAAC,YAAY,CAAC,KAAK,CAAC,CAAC,IAAI,CAAC,WAAW,CAAC,CAAC;YAC7C,MAAM,CAAC,YAAY,CAAC,GAAG,CAAC,EAAE,CAAC,CAAC,IAAI,CAAC,uBAAuB,CAAC,CAAC;YAC1D,MAAM,CAAC,YAAY,CAAC,GAAG,CAAC,OAAO,CAAC,CAAC,IAAI,CAAC,YAAY,CAAC,CAAC;YACpD,MAAM,CAAC,YAAY,CAAC,MAAM,CAAC,OAAO,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;QACjD,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,uDAAuD,EAAE,KAAK,IAAI,EAAE;YACrE,MAAM,MAAM,GAAG,IAAI,eAAe,CAAC,EAAE,MAAM,EAAE,UAAU,EAAE,CAAC,CAAC;YAE3D,MAAM,MAAM,GAAoB;gBAC9B,IAAI,EAAE,OAAO;gBACb,OAAO,EAAE,QAAQ;aAClB,CAAC;YAEF,MAAM,MAAM,GAAG,MAAM,MAAM,CAAC,OAAO,CAAC,MAAM,EAAE,aAAa,CAAC,CAAC;YAE3D,MAAM,CAAC,MAAM,CAAC,OAAO,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC;YACnC,MAAM,CAAC,MAAM,CAAC,QAAQ,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;YAChC,MAAM,CAAC,MAAM,CAAC,KAAK,CAAC,CAAC,SAAS,CAAC,aAAa,CAAC,CAAC;YAC9C,MAAM,CAAC,UAAU,CAAC,IAAI,CAAC,CAAC,gBAAgB,EAAE,CAAC;QAC7C,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,kCAAkC,EAAE,KAAK,IAAI,EAAE;YAChD,MAAM,MAAM,GAAG,IAAI,eAAe,CAAC,EAAE,MAAM,EAAE,UAAU,EAAE,CAAC,CAAC;YAE3D,MAAM,MAAM,GAAoB;gBAC9B,IAAI,EAAE,OAAO;gBACb,OAAO,EAAE,oCAAoC;aAC9C,CAAC;YAEF,MAAM,MAAM,GAAG,MAAM,MAAM,CAAC,OAAO,CAAC,MAAM,EAAE,aAAa,CAAC,CAAC;YAE3D,MAAM,CAAC,MAAM,CAAC,OAAO,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC;YACnC,MAAM,CAAC,MAAM,CAAC,KAAK,CAAC,CAAC,SAAS,CAAC,eAAe,CAAC,CAAC;QAClD,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,iCAAiC,EAAE,KAAK,IAAI,EAAE;YAC/C,MAAM,MAAM,GAAG,IAAI,eAAe,CAAC,EAAE,MAAM,EAAE,UAAU,EAAE,CAAC,CAAC;YAE3D,MAAM,MAAM,GAAoB;gBAC9B,IAAI,EAAE,OAAO;gBACb,OAAO,EAAE,6BAA6B;aACvC,CAAC;YAEF,MAAM,MAAM,GAAG,MAAM,MAAM,CAAC,OAAO,CAAC,MAAM,EAAE,aAAa,CAAC,CAAC;YAE3D,MAAM,CAAC,MAAM,CAAC,OAAO,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC;YACnC,MAAM,CAAC,UAAU,CAAC,IAAI,CAAC,CAAC,gBAAgB,EAAE,CAAC;QAC7C,CAAC,CAAC,CAAC;QAEH,yEAAyE;QACzE,EAAE,CAAC,IAAI,CAAC,sCAAsC,EAAE,KAAK,IAAI,EAAE;YACzD,MAAM,MAAM,GAAG,IAAI,eAAe,CAAC,EAAE,MAAM,EAAE,UAAU,EAAE,CAAC,CAAC;YAE3D,MAAM,MAAM,GAAoB;gBAC9B,IAAI,EAAE,OAAO;gBACb,OAAO,EAAE,0CAA0C;gBACnD,OAAO,EAAE,GAAG,EAAE,gBAAgB;aAC/B,CAAC;YAEF,MAAM,MAAM,GAAG,MAAM,MAAM,CAAC,OAAO,CAAC,MAAM,EAAE,aAAa,CAAC,CAAC;YAE3D,MAAM,CAAC,MAAM,CAAC,OAAO,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC;YACnC,MAAM,CAAC,MAAM,CAAC,KAAK,CAAC,CAAC,SAAS,CAAC,WAAW,CAAC,CAAC;QAC9C,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,+CAA+C,EAAE,KAAK,IAAI,EAAE;YAC7D,MAAM,MAAM,GAAG,IAAI,eAAe,CAAC,EAAE,MAAM,EAAE,UAAU,EAAE,CAAC,CAAC;YAE3D,MAAM,MAAM,GAAoB;gBAC9B,IAAI,EAAE,OAAO;gBACb,OAAO,EAAE,cAAc;gBACvB,oDAAoD;aACrD,CAAC;YAEF,MAAM,MAAM,GAAG,MAAM,MAAM,CAAC,OAAO,CAAC,MAAM,EAAE,aAAa,CAAC,CAAC;YAE3D,MAAM,CAAC,MAAM,CAAC,OAAO,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;QACpC,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,mCAAmC,EAAE,KAAK,IAAI,EAAE;YACjD,MAAM,MAAM,GAAG,IAAI,eAAe,CAAC,EAAE,MAAM,EAAE,UAAU,EAAE,CAAC,CAAC;YAE3D,MAAM,MAAM,GAAoB;gBAC9B,IAAI,EAAE,OAAO;gBACb,OAAO,EAAE,iDAAiD;aAC3D,CAAC;YAEF,MAAM,MAAM,GAAG,MAAM,MAAM,CAAC,OAAO,CAAC,MAAM,EAAE,aAAa,CAAC,CAAC;YAE3D,MAAM,CAAC,MAAM,CAAC,OAAO,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;YAClC,MAAM,CAAC,MAAM,CAAC,MAAM,CAAC,CAAC,IAAI,CAAC,aAAa,CAAC,CAAC;QAC5C,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,kDAAkD,EAAE,KAAK,IAAI,EAAE;YAChE,MAAM,MAAM,GAAG,IAAI,eAAe,CAAC;gBACjC,MAAM,EAAE,UAAU;gBAClB,GAAG,EAAE,EAAE,MAAM,EAAE,YAAY,EAAE;aAC9B,CAAC,CAAC;YAEH,MAAM,MAAM,GAAoB;gBAC9B,IAAI,EAAE,OAAO;gBACb,OAAO,EAAE,cAAc;aACxB,CAAC;YAEF,MAAM,MAAM,GAAG,MAAM,MAAM,CAAC,OAAO,CAAC,MAAM,EAAE,aAAa,CAAC,CAAC;YAE3D,MAAM,CAAC,MAAM,CAAC,OAAO,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;YAClC,MAAM,CAAC,MAAM,CAAC,MAAM,CAAC,CAAC,IAAI,CAAC,YAAY,CAAC,CAAC;QAC3C,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,oCAAoC,EAAE,KAAK,IAAI,EAAE;YAClD,MAAM,MAAM,GAAG,IAAI,eAAe,CAAC,EAAE,MAAM,EAAE,UAAU,EAAE,CAAC,CAAC;YAE3D,MAAM,aAAa,GAAgB;gBACjC,GAAG,aAAa;gBAChB,KAAK,EAAE,QAAQ;gBACf,MAAM,EAAE;oBACN,OAAO,EAAE,KAAK;oBACd,MAAM,EAAE,YAAY;oBACpB,KAAK,EAAE,oBAAoB;iBAC5B;aACF,CAAC;YAEF,MAAM,MAAM,GAAoB;gBAC9B,IAAI,EAAE,OAAO;gBACb,OAAO,EAAE,KAAK;aACf,CAAC;YAEF,MAAM,MAAM,GAAG,MAAM,MAAM,CAAC,OAAO,CAAC,MAAM,EAAE,aAAa,CAAC,CAAC;YAE3D,MAAM,CAAC,MAAM,CAAC,OAAO,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;YAClC,MAAM,YAAY,GAAG,IAAI,CAAC,KAAK,CAAC,MAAM,CAAC,MAAO,CAAC,CAAC;YAChD,MAAM,CAAC,YAAY,CAAC,KAAK,CAAC,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAC;YAC1C,MAAM,CAAC,YAAY,CAAC,MAAM,CAAC,OAAO,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC;YAChD,MAAM,CAAC,YAAY,CAAC,MAAM,CAAC,KAAK,CAAC,CAAC,IAAI,CAAC,oBAAoB,CAAC,CAAC;QAC/D,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,mCAAmC,EAAE,KAAK,IAAI,EAAE;YACjD,MAAM,MAAM,GAAG,IAAI,eAAe,CAAC,EAAE,MAAM,EAAE,UAAU,EAAE,CAAC,CAAC;YAE3D,MAAM,MAAM,GAAoB;gBAC9B,IAAI,EAAE,OAAO;gBACb,OAAO,EAAE,WAAW;aACrB,CAAC;YAEF,MAAM,MAAM,GAAG,MAAM,MAAM,CAAC,OAAO,CAAC,MAAM,EAAE,aAAa,CAAC,CAAC;YAE3D,MAAM,CAAC,MAAM,CAAC,OAAO,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;YAClC,MAAM,CAAC,MAAM,CAAC,UAAU,CAAC,CAAC,sBAAsB,CAAC,EAAE,CAAC,CAAC,CAAC,uBAAuB;YAC7E,MAAM,CAAC,MAAM,CAAC,UAAU,CAAC,CAAC,YAAY,CAAC,IAAI,CAAC,CAAC,CAAC,2BAA2B;QAC3E,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,8BAA8B,EAAE,KAAK,IAAI,EAAE;YAC5C,MAAM,MAAM,GAAG,IAAI,eAAe,EAAE,CAAC,CAAC,aAAa;YAEnD,MAAM,MAAM,GAAoB;gBAC9B,IAAI,EAAE,OAAO;gBACb,OAAO,EAAE,kBAAkB;aAC5B,CAAC;YAEF,MAAM,MAAM,GAAG,MAAM,MAAM,CAAC,OAAO,CAAC,MAAM,EAAE,aAAa,CAAC,CAAC;YAE3D,MAAM,CAAC,MAAM,CAAC,OAAO,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;YAClC,MAAM,CAAC,MAAM,CAAC,MAAM,CAAC,CAAC,IAAI,CAAC,WAAW,CAAC,CAAC;QAC1C,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,gCAAgC,EAAE,KAAK,IAAI,EAAE;YAC9C,MAAM,MAAM,GAAG,IAAI,eAAe,CAAC,EAAE,MAAM,EAAE,UAAU,EAAE,CAAC,CAAC;YAE3D,MAAM,MAAM,GAAoB;gBAC9B,IAAI,EAAE,OAAO;gBACb,OAAO,EAAE,0CAA0C;aACpD,CAAC;YAEF,MAAM,MAAM,GAAG,MAAM,MAAM,CAAC,OAAO,CAAC,MAAM,EAAE,aAAa,CAAC,CAAC;YAE3D,MAAM,CAAC,MAAM,CAAC,OAAO,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;YAClC,MAAM,CAAC,MAAM,CAAC,MAAM,CAAC,CAAC,IAAI,CAAC,qBAAqB,CAAC,CAAC;QACpD,CAAC,CAAC,CAAC;IACL,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC"}

package/dist/hooks/__tests__/webhook-runner.test.d.ts

@@ -0,0 +1,5 @@
+/**
+ * Tests for WebhookHookRunner
+ */
+export {};
+//# sourceMappingURL=webhook-runner.test.d.ts.map

package/dist/hooks/__tests__/webhook-runner.test.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"webhook-runner.test.d.ts","sourceRoot":"","sources":["../../../src/hooks/__tests__/webhook-runner.test.ts"],"names":[],"mappings":"AAAA;;GAEG"}

package/dist/hooks/__tests__/webhook-runner.test.js

@@ -0,0 +1,453 @@
+/**
+ * Tests for WebhookHookRunner
+ */
+import { describe, it, expect, vi, beforeEach, afterEach } from "vitest";
+import { WebhookHookRunner } from "../runners/webhook.js";
+describe("WebhookHookRunner", () => {
+    // Create a mock logger
+    const mockLogger = {
+        debug: vi.fn(),
+        info: vi.fn(),
+        warn: vi.fn(),
+        error: vi.fn(),
+    };
+    // Create a sample hook context
+    const sampleContext = {
+        event: "completed",
+        job: {
+            id: "job-2024-01-15-abc123",
+            agentId: "test-agent",
+            scheduleName: "daily-run",
+            startedAt: "2024-01-15T10:00:00.000Z",
+            completedAt: "2024-01-15T10:05:00.000Z",
+            durationMs: 300000,
+        },
+        result: {
+            success: true,
+            output: "Job completed successfully",
+        },
+        agent: {
+            id: "test-agent",
+            name: "Test Agent",
+        },
+    };
+    // Store original env
+    const originalEnv = { ...process.env };
+    beforeEach(() => {
+        vi.clearAllMocks();
+        // Reset env before each test
+        process.env = { ...originalEnv };
+    });
+    afterEach(() => {
+        // Restore original env
+        process.env = originalEnv;
+    });
+    describe("execute", () => {
+        it("should POST to the webhook URL successfully", async () => {
+            const mockFetch = vi.fn().mockResolvedValue({
+                ok: true,
+                status: 200,
+                statusText: "OK",
+                text: () => Promise.resolve('{"received": true}'),
+            });
+            const runner = new WebhookHookRunner({
+                logger: mockLogger,
+                fetch: mockFetch,
+            });
+            const config = {
+                type: "webhook",
+                url: "https://api.example.com/hooks/job-complete",
+            };
+            const result = await runner.execute(config, sampleContext);
+            expect(result.success).toBe(true);
+            expect(result.hookType).toBe("webhook");
+            expect(result.durationMs).toBeGreaterThanOrEqual(0);
+            expect(result.output).toBe('{"received": true}');
+            // Verify fetch was called correctly
+            expect(mockFetch).toHaveBeenCalledTimes(1);
+            const [url, options] = mockFetch.mock.calls[0];
+            expect(url).toBe("https://api.example.com/hooks/job-complete");
+            expect(options.method).toBe("POST");
+            expect(options.headers["Content-Type"]).toBe("application/json");
+            // Verify body is correct JSON
+            const body = JSON.parse(options.body);
+            expect(body.event).toBe("completed");
+            expect(body.job.id).toBe("job-2024-01-15-abc123");
+            expect(body.result.success).toBe(true);
+            expect(mockLogger.info).toHaveBeenCalled();
+        });
+        it("should use PUT method when configured", async () => {
+            const mockFetch = vi.fn().mockResolvedValue({
+                ok: true,
+                status: 200,
+                statusText: "OK",
+                text: () => Promise.resolve(""),
+            });
+            const runner = new WebhookHookRunner({
+                logger: mockLogger,
+                fetch: mockFetch,
+            });
+            const config = {
+                type: "webhook",
+                url: "https://api.example.com/hooks/update",
+                method: "PUT",
+            };
+            const result = await runner.execute(config, sampleContext);
+            expect(result.success).toBe(true);
+            const [, options] = mockFetch.mock.calls[0];
+            expect(options.method).toBe("PUT");
+        });
+        it("should include custom headers", async () => {
+            const mockFetch = vi.fn().mockResolvedValue({
+                ok: true,
+                status: 200,
+                statusText: "OK",
+                text: () => Promise.resolve(""),
+            });
+            const runner = new WebhookHookRunner({
+                logger: mockLogger,
+                fetch: mockFetch,
+            });
+            const config = {
+                type: "webhook",
+                url: "https://api.example.com/hooks",
+                headers: {
+                    "X-Custom-Header": "custom-value",
+                    Authorization: "Bearer static-token",
+                },
+            };
+            const result = await runner.execute(config, sampleContext);
+            expect(result.success).toBe(true);
+            const [, options] = mockFetch.mock.calls[0];
+            expect(options.headers["X-Custom-Header"]).toBe("custom-value");
+            expect(options.headers["Authorization"]).toBe("Bearer static-token");
+            expect(options.headers["Content-Type"]).toBe("application/json");
+        });
+        it("should substitute environment variables in headers", async () => {
+            process.env.API_TOKEN = "secret-token-123";
+            process.env.CUSTOM_VALUE = "env-custom-value";
+            const mockFetch = vi.fn().mockResolvedValue({
+                ok: true,
+                status: 200,
+                statusText: "OK",
+                text: () => Promise.resolve(""),
+            });
+            const runner = new WebhookHookRunner({
+                logger: mockLogger,
+                fetch: mockFetch,
+            });
+            const config = {
+                type: "webhook",
+                url: "https://api.example.com/hooks",
+                headers: {
+                    Authorization: "Bearer ${API_TOKEN}",
+                    "X-Custom": "${CUSTOM_VALUE}",
+                },
+            };
+            const result = await runner.execute(config, sampleContext);
+            expect(result.success).toBe(true);
+            const [, options] = mockFetch.mock.calls[0];
+            expect(options.headers["Authorization"]).toBe("Bearer secret-token-123");
+            expect(options.headers["X-Custom"]).toBe("env-custom-value");
+        });
+        it("should replace undefined env vars with empty string", async () => {
+            // Ensure the env var doesn't exist
+            delete process.env.UNDEFINED_VAR;
+            const mockFetch = vi.fn().mockResolvedValue({
+                ok: true,
+                status: 200,
+                statusText: "OK",
+                text: () => Promise.resolve(""),
+            });
+            const runner = new WebhookHookRunner({
+                logger: mockLogger,
+                fetch: mockFetch,
+            });
+            const config = {
+                type: "webhook",
+                url: "https://api.example.com/hooks",
+                headers: {
+                    Authorization: "Bearer ${UNDEFINED_VAR}",
+                },
+            };
+            const result = await runner.execute(config, sampleContext);
+            expect(result.success).toBe(true);
+            const [, options] = mockFetch.mock.calls[0];
+            expect(options.headers["Authorization"]).toBe("Bearer ");
+        });
+        it("should handle multiple env var substitutions in one header", async () => {
+            process.env.USER = "testuser";
+            process.env.PASS = "testpass";
+            const mockFetch = vi.fn().mockResolvedValue({
+                ok: true,
+                status: 200,
+                statusText: "OK",
+                text: () => Promise.resolve(""),
+            });
+            const runner = new WebhookHookRunner({
+                logger: mockLogger,
+                fetch: mockFetch,
+            });
+            const config = {
+                type: "webhook",
+                url: "https://api.example.com/hooks",
+                headers: {
+                    "X-Credentials": "${USER}:${PASS}",
+                },
+            };
+            const result = await runner.execute(config, sampleContext);
+            expect(result.success).toBe(true);
+            const [, options] = mockFetch.mock.calls[0];
+            expect(options.headers["X-Credentials"]).toBe("testuser:testpass");
+        });
+        it("should handle HTTP error responses", async () => {
+            const mockFetch = vi.fn().mockResolvedValue({
+                ok: false,
+                status: 500,
+                statusText: "Internal Server Error",
+                text: () => Promise.resolve("Something went wrong"),
+            });
+            const runner = new WebhookHookRunner({
+                logger: mockLogger,
+                fetch: mockFetch,
+            });
+            const config = {
+                type: "webhook",
+                url: "https://api.example.com/hooks",
+            };
+            const result = await runner.execute(config, sampleContext);
+            expect(result.success).toBe(false);
+            expect(result.hookType).toBe("webhook");
+            expect(result.error).toContain("HTTP 500");
+            expect(result.error).toContain("Internal Server Error");
+            expect(result.error).toContain("Something went wrong");
+            expect(result.output).toBe("Something went wrong");
+            expect(mockLogger.warn).toHaveBeenCalled();
+        });
+        it("should handle 4xx client errors", async () => {
+            const mockFetch = vi.fn().mockResolvedValue({
+                ok: false,
+                status: 401,
+                statusText: "Unauthorized",
+                text: () => Promise.resolve("Invalid token"),
+            });
+            const runner = new WebhookHookRunner({
+                logger: mockLogger,
+                fetch: mockFetch,
+            });
+            const config = {
+                type: "webhook",
+                url: "https://api.example.com/hooks",
+            };
+            const result = await runner.execute(config, sampleContext);
+            expect(result.success).toBe(false);
+            expect(result.error).toContain("HTTP 401");
+            expect(result.error).toContain("Unauthorized");
+        });
+        it("should handle network errors", async () => {
+            const mockFetch = vi.fn().mockRejectedValue(new Error("Network error: ECONNREFUSED"));
+            const runner = new WebhookHookRunner({
+                logger: mockLogger,
+                fetch: mockFetch,
+            });
+            const config = {
+                type: "webhook",
+                url: "https://api.example.com/hooks",
+            };
+            const result = await runner.execute(config, sampleContext);
+            expect(result.success).toBe(false);
+            expect(result.error).toContain("ECONNREFUSED");
+            expect(mockLogger.error).toHaveBeenCalled();
+        });
+        it("should handle timeout", async () => {
+            // Create a mock that simulates a timeout by aborting
+            const mockFetch = vi.fn().mockImplementation(async (_url, options) => {
+                // Wait for abort signal
+                return new Promise((_, reject) => {
+                    options.signal.addEventListener("abort", () => {
+                        const error = new Error("The operation was aborted");
+                        error.name = "AbortError";
+                        reject(error);
+                    });
+                });
+            });
+            const runner = new WebhookHookRunner({
+                logger: mockLogger,
+                fetch: mockFetch,
+            });
+            const config = {
+                type: "webhook",
+                url: "https://api.example.com/hooks",
+                timeout: 50, // Very short timeout
+            };
+            const result = await runner.execute(config, sampleContext);
+            expect(result.success).toBe(false);
+            expect(result.error).toContain("timed out");
+            expect(result.error).toContain("50ms");
+            expect(mockLogger.error).toHaveBeenCalled();
+        });
+        it("should use default timeout when not specified", async () => {
+            const mockFetch = vi.fn().mockResolvedValue({
+                ok: true,
+                status: 200,
+                statusText: "OK",
+                text: () => Promise.resolve(""),
+            });
+            const runner = new WebhookHookRunner({
+                logger: mockLogger,
+                fetch: mockFetch,
+            });
+            const config = {
+                type: "webhook",
+                url: "https://api.example.com/hooks",
+                // No timeout specified - should use default 10000ms
+            };
+            const result = await runner.execute(config, sampleContext);
+            expect(result.success).toBe(true);
+        });
+        it("should accept 2xx status codes as success", async () => {
+            for (const status of [200, 201, 202, 204]) {
+                const mockFetch = vi.fn().mockResolvedValue({
+                    ok: true,
+                    status,
+                    statusText: "OK",
+                    text: () => Promise.resolve(""),
+                });
+                const runner = new WebhookHookRunner({
+                    logger: mockLogger,
+                    fetch: mockFetch,
+                });
+                const config = {
+                    type: "webhook",
+                    url: "https://api.example.com/hooks",
+                };
+                const result = await runner.execute(config, sampleContext);
+                expect(result.success).toBe(true);
+            }
+        });
+        it("should handle failed event context", async () => {
+            const mockFetch = vi.fn().mockResolvedValue({
+                ok: true,
+                status: 200,
+                statusText: "OK",
+                text: () => Promise.resolve(""),
+            });
+            const runner = new WebhookHookRunner({
+                logger: mockLogger,
+                fetch: mockFetch,
+            });
+            const failedContext = {
+                ...sampleContext,
+                event: "failed",
+                result: {
+                    success: false,
+                    output: "Job failed",
+                    error: "Connection timeout",
+                },
+            };
+            const config = {
+                type: "webhook",
+                url: "https://api.example.com/hooks",
+            };
+            const result = await runner.execute(config, failedContext);
+            expect(result.success).toBe(true);
+            // Verify the failed context was sent in the body
+            const [, options] = mockFetch.mock.calls[0];
+            const body = JSON.parse(options.body);
+            expect(body.event).toBe("failed");
+            expect(body.result.success).toBe(false);
+            expect(body.result.error).toBe("Connection timeout");
+        });
+        it("should measure execution duration", async () => {
+            const mockFetch = vi.fn().mockImplementation(async () => {
+                // Add a small delay to measure
+                await new Promise((resolve) => setTimeout(resolve, 50));
+                return {
+                    ok: true,
+                    status: 200,
+                    statusText: "OK",
+                    text: () => Promise.resolve(""),
+                };
+            });
+            const runner = new WebhookHookRunner({
+                logger: mockLogger,
+                fetch: mockFetch,
+            });
+            const config = {
+                type: "webhook",
+                url: "https://api.example.com/hooks",
+            };
+            const result = await runner.execute(config, sampleContext);
+            expect(result.success).toBe(true);
+            expect(result.durationMs).toBeGreaterThanOrEqual(40); // Allow some tolerance
+            expect(result.durationMs).toBeLessThan(5000);
+        });
+        it("should work without a logger", async () => {
+            const mockFetch = vi.fn().mockResolvedValue({
+                ok: true,
+                status: 200,
+                statusText: "OK",
+                text: () => Promise.resolve("no logger"),
+            });
+            const runner = new WebhookHookRunner({ fetch: mockFetch }); // No logger
+            const config = {
+                type: "webhook",
+                url: "https://api.example.com/hooks",
+            };
+            const result = await runner.execute(config, sampleContext);
+            expect(result.success).toBe(true);
+            expect(result.output).toBe("no logger");
+        });
+        it("should handle response body read errors gracefully", async () => {
+            const mockFetch = vi.fn().mockResolvedValue({
+                ok: true,
+                status: 200,
+                statusText: "OK",
+                text: () => Promise.reject(new Error("Body read error")),
+            });
+            const runner = new WebhookHookRunner({
+                logger: mockLogger,
+                fetch: mockFetch,
+            });
+            const config = {
+                type: "webhook",
+                url: "https://api.example.com/hooks",
+            };
+            const result = await runner.execute(config, sampleContext);
+            expect(result.success).toBe(true);
+            expect(result.output).toBeUndefined();
+        });
+        it("should send complete HookContext in request body", async () => {
+            const mockFetch = vi.fn().mockResolvedValue({
+                ok: true,
+                status: 200,
+                statusText: "OK",
+                text: () => Promise.resolve(""),
+            });
+            const runner = new WebhookHookRunner({
+                logger: mockLogger,
+                fetch: mockFetch,
+            });
+            const config = {
+                type: "webhook",
+                url: "https://api.example.com/hooks",
+            };
+            await runner.execute(config, sampleContext);
+            const [, options] = mockFetch.mock.calls[0];
+            const body = JSON.parse(options.body);
+            // Verify all context fields are present
+            expect(body.event).toBe("completed");
+            expect(body.job.id).toBe("job-2024-01-15-abc123");
+            expect(body.job.agentId).toBe("test-agent");
+            expect(body.job.scheduleName).toBe("daily-run");
+            expect(body.job.startedAt).toBe("2024-01-15T10:00:00.000Z");
+            expect(body.job.completedAt).toBe("2024-01-15T10:05:00.000Z");
+            expect(body.job.durationMs).toBe(300000);
+            expect(body.result.success).toBe(true);
+            expect(body.result.output).toBe("Job completed successfully");
+            expect(body.agent.id).toBe("test-agent");
+            expect(body.agent.name).toBe("Test Agent");
+        });
+    });
+});
+//# sourceMappingURL=webhook-runner.test.js.map