@oh-my-pi/pi-coding-agent 5.7.67 → 5.7.69

package/src/core/tools/python-renderer.test.ts

@@ -1,36 +0,0 @@
-import { describe, expect, it } from "bun:test";
-import stripAnsi from "strip-ansi";
-import { getThemeByName } from "../../modes/interactive/theme/theme";
-import { pythonToolRenderer } from "./python";
-import { truncateTail } from "./truncate";
-
-describe("pythonToolRenderer", () => {
-	it("renders truncated output when collapsed and full output when expanded", () => {
-		const theme = getThemeByName("dark");
-		expect(theme).toBeDefined();
-		const uiTheme = theme!;
-
-		const fullOutput = ["line 1", "line 2", "line 3", "line 4"].join("\n");
-		const truncation = truncateTail(fullOutput, { maxLines: 2, maxBytes: 128 });
-
-		const result = {
-			content: [{ type: "text", text: truncation.content }],
-			details: {
-				truncation,
-				fullOutput,
-			},
-		};
-
-		const collapsed = pythonToolRenderer.renderResult(result, { expanded: false, isPartial: false }, uiTheme);
-		const collapsedLines = stripAnsi(collapsed.render(80).join("\n"));
-		expect(collapsedLines).toContain("line 4");
-		expect(collapsedLines).not.toContain("line 1");
-		expect(collapsedLines).toContain("Truncated:");
-
-		const expanded = pythonToolRenderer.renderResult(result, { expanded: true, isPartial: false }, uiTheme);
-		const expandedLines = stripAnsi(expanded.render(80).join("\n"));
-		expect(expandedLines).toContain("line 1");
-		expect(expandedLines).toContain("line 4");
-		expect(expandedLines).not.toContain("Truncated:");
-	});
-});

package/src/core/tools/python-tool-mode.test.ts

@@ -1,43 +0,0 @@
-import { describe, expect, it } from "bun:test";
-import { createTools, type ToolSession } from "./index";
-
-function createSession(overrides: Partial<ToolSession> = {}): ToolSession {
-	return {
-		cwd: "/tmp/test",
-		hasUI: false,
-		getSessionFile: () => null,
-		getSessionSpawns: () => "*",
-		settings: {
-			getImageAutoResize: () => true,
-			getLspFormatOnWrite: () => true,
-			getLspDiagnosticsOnWrite: () => true,
-			getLspDiagnosticsOnEdit: () => false,
-			getEditFuzzyMatch: () => true,
-			getGitToolEnabled: () => true,
-			getBashInterceptorEnabled: () => true,
-			getBashInterceptorSimpleLsEnabled: () => true,
-			getBashInterceptorRules: () => [],
-			getPythonToolMode: () => "bash-only",
-			getPythonKernelMode: () => "session",
-		},
-		...overrides,
-	};
-}
-
-describe("createTools python fallback", () => {
-	it("falls back to bash when python is requested but disabled", async () => {
-		const previous = process.env.OMP_PYTHON_SKIP_CHECK;
-		process.env.OMP_PYTHON_SKIP_CHECK = "1";
-		const session = createSession();
-		const tools = await createTools(session, ["python"]);
-		const names = tools.map((tool) => tool.name);
-
-		expect(names).toEqual(["bash"]);
-
-		if (previous === undefined) {
-			delete process.env.OMP_PYTHON_SKIP_CHECK;
-		} else {
-			process.env.OMP_PYTHON_SKIP_CHECK = previous;
-		}
-	});
-});

package/src/core/tools/python.test.ts

@@ -1,121 +0,0 @@
-import { afterAll, beforeAll, describe, expect, it, vi } from "bun:test";
-import * as pythonExecutor from "../python-executor";
-import { createTools, type ToolSession } from "./index";
-import { createPythonTool } from "./python";
-
-let previousSkipCheck: string | undefined;
-
-beforeAll(() => {
-	previousSkipCheck = process.env.OMP_PYTHON_SKIP_CHECK;
-	process.env.OMP_PYTHON_SKIP_CHECK = "1";
-});
-
-afterAll(() => {
-	if (previousSkipCheck === undefined) {
-		delete process.env.OMP_PYTHON_SKIP_CHECK;
-		return;
-	}
-	process.env.OMP_PYTHON_SKIP_CHECK = previousSkipCheck;
-});
-
-function createSession(overrides: Partial<ToolSession> = {}): ToolSession {
-	return {
-		cwd: "/tmp/test",
-		hasUI: false,
-		getSessionFile: () => null,
-		getSessionSpawns: () => "*",
-		...overrides,
-	};
-}
-
-function createSettings(toolMode: "ipy-only" | "bash-only" | "both") {
-	return {
-		getImageAutoResize: () => true,
-		getLspFormatOnWrite: () => true,
-		getLspDiagnosticsOnWrite: () => true,
-		getLspDiagnosticsOnEdit: () => false,
-		getEditFuzzyMatch: () => true,
-		getGitToolEnabled: () => true,
-		getBashInterceptorEnabled: () => true,
-		getBashInterceptorSimpleLsEnabled: () => true,
-		getBashInterceptorRules: () => [],
-		getPythonToolMode: () => toolMode,
-		getPythonKernelMode: () => "session" as const,
-	};
-}
-
-describe("python tool schema", () => {
-	it("exposes expected parameters", () => {
-		const tool = createPythonTool(createSession());
-		const schema = tool.parameters as {
-			type: string;
-			properties: Record<string, { type: string; description?: string }>;
-			required?: string[];
-		};
-
-		expect(schema.type).toBe("object");
-		expect(schema.properties.code.type).toBe("string");
-		expect(schema.properties.timeout.type).toBe("number");
-		expect(schema.properties.workdir.type).toBe("string");
-		expect(schema.properties.reset.type).toBe("boolean");
-		expect(schema.required).toEqual(["code"]);
-	});
-});
-
-describe("python tool docs template", () => {
-	it("renders dynamic helper docs", () => {
-		const docs = [
-			{
-				name: "read",
-				signature: "(path)",
-				docstring: "Read file contents.",
-				category: "File I/O",
-			},
-		];
-		const spy = vi.spyOn(pythonExecutor, "getPreludeDocs").mockReturnValue(docs);
-
-		const tool = createPythonTool(createSession());
-
-		expect(tool.description).toContain("### File I/O");
-		expect(tool.description).toContain("read(path)");
-		expect(tool.description).toContain("Read file contents.");
-
-		spy.mockRestore();
-	});
-
-	it("renders fallback when docs are unavailable", () => {
-		const spy = vi.spyOn(pythonExecutor, "getPreludeDocs").mockReturnValue([]);
-
-		const tool = createPythonTool(createSession());
-
-		expect(tool.description).toContain("Documentation unavailable — Python kernel failed to start");
-
-		spy.mockRestore();
-	});
-});
-
-describe("python tool exposure", () => {
-	it("includes python only in ipy-only mode", async () => {
-		const session = createSession({ settings: createSettings("ipy-only") });
-		const tools = await createTools(session);
-		const names = tools.map((tool) => tool.name);
-		expect(names).toContain("python");
-		expect(names).not.toContain("bash");
-	});
-
-	it("includes bash only in bash-only mode", async () => {
-		const session = createSession({ settings: createSettings("bash-only") });
-		const tools = await createTools(session);
-		const names = tools.map((tool) => tool.name);
-		expect(names).toContain("bash");
-		expect(names).not.toContain("python");
-	});
-
-	it("includes bash and python in both mode", async () => {
-		const session = createSession({ settings: createSettings("both") });
-		const tools = await createTools(session);
-		const names = tools.map((tool) => tool.name);
-		expect(names).toContain("bash");
-		expect(names).toContain("python");
-	});
-});