File size: 4,230 Bytes
3201ca6 | 1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33 34 35 36 37 38 39 40 41 42 43 44 45 46 47 48 49 50 51 52 53 54 55 56 57 58 59 60 61 62 63 64 65 66 67 68 69 70 71 72 73 74 75 76 77 78 79 80 81 82 83 84 85 86 87 88 89 90 91 92 93 94 95 96 97 98 99 100 101 102 103 104 105 106 107 108 109 110 111 112 113 114 115 116 117 118 119 120 121 122 123 124 125 126 127 128 129 130 131 132 133 134 135 136 137 | import { HooksClient } from "@openhands/typescript-client/clients";
import { afterEach, beforeEach, describe, expect, it, vi } from "vitest";
import {
__resetActiveStoreForTests,
setActiveSelection,
setRegisteredBackends,
} from "#/api/backend-registry/active-store";
import { DEFAULT_WORKING_DIR } from "#/api/agent-server-config";
import type { Backend } from "#/api/backend-registry/types";
const { mockLoadHooks } = vi.hoisted(() => ({
mockLoadHooks: vi.fn(),
}));
vi.mock("@openhands/typescript-client/clients", () => ({
HooksClient: vi.fn(function HooksClientMock() {
return { loadHooks: mockLoadHooks };
}),
}));
import HooksService from "#/api/hooks-service";
const localBackend: Backend = {
id: "local",
name: "Local",
host: "http://127.0.0.1:8000",
apiKey: "",
kind: "local",
};
const cloudBackend: Backend = {
id: "cloud",
name: "Cloud",
host: "https://openhands.dev",
apiKey: "cloud-key",
kind: "cloud",
};
beforeEach(() => {
window.localStorage.clear();
__resetActiveStoreForTests();
setRegisteredBackends([localBackend]);
setActiveSelection({ backendId: localBackend.id });
mockLoadHooks.mockReset();
vi.mocked(HooksClient).mockClear();
});
afterEach(() => {
__resetActiveStoreForTests();
});
describe("HooksService.loadWorkspaceHooks", () => {
it("loads workspace hooks from the agent-server when present", async () => {
const mockHookConfig = {
session_start: [
{
matcher: "*",
hooks: [{ command: "cat AGENTS.md", type: "command" }],
},
],
pre_tool_use: [],
post_tool_use: [],
user_prompt_submit: [],
session_end: [],
stop: [],
};
mockLoadHooks.mockResolvedValue({ hook_config: mockHookConfig });
const result = await HooksService.loadWorkspaceHooks(
"/workspace/test-project",
);
expect(mockLoadHooks).toHaveBeenCalledTimes(1);
expect(mockLoadHooks).toHaveBeenCalledWith({
project_dir: "/workspace/test-project",
});
expect(result).toEqual(mockHookConfig);
});
it("defaults project_dir to getAgentServerWorkingDir() when omitted", async () => {
mockLoadHooks.mockResolvedValue({ hook_config: null });
const result = await HooksService.loadWorkspaceHooks();
expect(mockLoadHooks).toHaveBeenCalledTimes(1);
expect(mockLoadHooks).toHaveBeenCalledWith({
project_dir: DEFAULT_WORKING_DIR,
});
expect(result).toBeNull();
});
it("gracefully returns null and warns when the agent-server throws an error", async () => {
const warn = vi.spyOn(console, "warn").mockImplementation(() => {});
mockLoadHooks.mockRejectedValue(new Error("500 Internal Server Error"));
const result = await HooksService.loadWorkspaceHooks("/workspace/broken");
expect(result).toBeNull();
expect(warn).toHaveBeenCalled();
warn.mockRestore();
});
// The lookup is on the launch critical path; the SDK's 60s default would
// stall a launch against a wedged agent-server.
it("bounds the request so a wedged agent-server cannot stall a launch", async () => {
mockLoadHooks.mockResolvedValue({ hook_config: null });
await HooksService.loadWorkspaceHooks("/workspace/test-project");
const [clientOptions] = vi.mocked(HooksClient).mock.calls[0];
expect(clientOptions.timeout).toBeLessThanOrEqual(10_000);
});
it("returns null immediately for cloud backend without calling HooksClient", async () => {
setRegisteredBackends([cloudBackend]);
setActiveSelection({ backendId: cloudBackend.id });
const result = await HooksService.loadWorkspaceHooks("/workspace/cloud");
expect(mockLoadHooks).not.toHaveBeenCalled();
expect(result).toBeNull();
});
it("returns null for unseeded/empty registry (NO_BACKEND sentinel) without calling HooksClient", async () => {
// Simulate a fresh install with no backends registered.
// setRegisteredBackends([]) computes the snapshot with NO_BACKEND,
// so getEffectiveLocalBackend() returns null.
setRegisteredBackends([]);
const result = await HooksService.loadWorkspaceHooks("/workspace/test");
expect(mockLoadHooks).not.toHaveBeenCalled();
expect(result).toBeNull();
});
});
|