pi-usereq 0.48.0 → 0.50.0

package/pi.dev-src/pi-mono/coding-agent/test/git-update.test.ts

@@ -1,411 +0,0 @@
-/**
- * Tests for git-based extension updates, specifically handling force-push scenarios.
- *
- * These tests verify that DefaultPackageManager.update() handles:
- * - Normal git updates (no force-push)
- * - Force-pushed remotes gracefully (currently fails, fix needed)
- */
-
-import { spawnSync } from "node:child_process";
-import { createHash } from "node:crypto";
-import { existsSync, mkdirSync, readFileSync, rmSync, writeFileSync } from "node:fs";
-import { tmpdir } from "node:os";
-import { join } from "node:path";
-import { afterEach, beforeEach, describe, expect, it } from "vitest";
-import { DefaultPackageManager } from "../src/core/package-manager.js";
-import { SettingsManager } from "../src/core/settings-manager.js";
-
-// Helper to run git commands in a directory
-function git(args: string[], cwd: string): string {
-	const result = spawnSync("git", args, {
-		cwd,
-		encoding: "utf-8",
-	});
-	if (result.status !== 0) {
-		throw new Error(`Command failed: git ${args.join(" ")}\n${result.stderr}`);
-	}
-	return result.stdout.trim();
-}
-
-function initGitRepo(repoDir: string): void {
-	git(["init", "--initial-branch=main"], repoDir);
-	git(["config", "--local", "user.email", "test@test.com"], repoDir);
-	git(["config", "--local", "user.name", "Test"], repoDir);
-}
-
-// Helper to create a commit with a file
-function createCommit(repoDir: string, filename: string, content: string, message: string): string {
-	writeFileSync(join(repoDir, filename), content);
-	git(["add", filename], repoDir);
-	git(["commit", "-m", message], repoDir);
-	return git(["rev-parse", "HEAD"], repoDir);
-}
-
-// Helper to get current commit hash
-function getCurrentCommit(repoDir: string): string {
-	return git(["rev-parse", "HEAD"], repoDir);
-}
-
-// Helper to get file content
-function getFileContent(repoDir: string, filename: string): string {
-	return readFileSync(join(repoDir, filename), "utf-8");
-}
-
-describe("DefaultPackageManager git update", () => {
-	let tempDir: string;
-	let remoteDir: string; // Simulates the "remote" repository
-	let agentDir: string; // The agent directory where extensions are installed
-	let installedDir: string; // The installed extension directory
-	let settingsManager: SettingsManager;
-	let packageManager: DefaultPackageManager;
-
-	// Git source that maps to our installed directory structure.
-	// Must use "git:" prefix so parseSource() treats it as a git source
-	// (bare "github.com/..." is not recognized as a git URL).
-	const gitSource = "git:github.com/test/extension";
-
-	beforeEach(() => {
-		tempDir = join(tmpdir(), `git-update-test-${Date.now()}-${Math.random().toString(36).slice(2)}`);
-		mkdirSync(tempDir, { recursive: true });
-		remoteDir = join(tempDir, "remote");
-		agentDir = join(tempDir, "agent");
-
-		// This matches the path structure: agentDir/git/<host>/<path>
-		installedDir = join(agentDir, "git", "github.com", "test", "extension");
-
-		mkdirSync(agentDir, { recursive: true });
-
-		settingsManager = SettingsManager.inMemory();
-		packageManager = new DefaultPackageManager({
-			cwd: tempDir,
-			agentDir,
-			settingsManager,
-		});
-	});
-
-	afterEach(() => {
-		if (tempDir && existsSync(tempDir)) {
-			rmSync(tempDir, { recursive: true, force: true });
-		}
-	});
-
-	/**
-	 * Sets up a "remote" repository and clones it to the installed directory.
-	 * This simulates what packageManager.install() would do.
-	 * @param sourceOverride Optional source string to use instead of gitSource (e.g., with @ref for pinned tests)
-	 */
-	function setupRemoteAndInstall(sourceOverride?: string): void {
-		// Create "remote" repository
-		mkdirSync(remoteDir, { recursive: true });
-		initGitRepo(remoteDir);
-		createCommit(remoteDir, "extension.ts", "// v1", "Initial commit");
-
-		// Clone to installed directory (simulating what install() does)
-		mkdirSync(join(agentDir, "git", "github.com", "test"), { recursive: true });
-		git(["clone", remoteDir, installedDir], tempDir);
-		git(["config", "--local", "user.email", "test@test.com"], installedDir);
-		git(["config", "--local", "user.name", "Test"], installedDir);
-
-		// Add to global packages so update() processes this source
-		settingsManager.setPackages([sourceOverride ?? gitSource]);
-	}
-
-	describe("normal updates (no force-push)", () => {
-		it("should skip reset, clean, and install when already up to date", async () => {
-			mkdirSync(remoteDir, { recursive: true });
-			initGitRepo(remoteDir);
-			writeFileSync(join(remoteDir, "package.json"), JSON.stringify({ name: "test-extension", version: "1.0.0" }));
-			createCommit(remoteDir, "extension.ts", "// v1", "Initial commit");
-
-			mkdirSync(join(agentDir, "git", "github.com", "test"), { recursive: true });
-			git(["clone", remoteDir, installedDir], tempDir);
-			settingsManager.setPackages([gitSource]);
-
-			const executedCommands: string[] = [];
-			const managerWithInternals = packageManager as unknown as {
-				runCommand: (command: string, args: string[], options?: { cwd?: string }) => Promise<void>;
-			};
-			managerWithInternals.runCommand = async (command, args, options) => {
-				executedCommands.push(`${command} ${args.join(" ")}`);
-				if (command === "npm") {
-					return;
-				}
-				const result = spawnSync(command, args, {
-					cwd: options?.cwd,
-					encoding: "utf-8",
-				});
-				if (result.status !== 0) {
-					throw new Error(`Command failed: ${command} ${args.join(" ")}\n${result.stderr}`);
-				}
-			};
-
-			await packageManager.update();
-
-			expect(executedCommands).toContain(
-				"git fetch --prune --no-tags origin +refs/heads/main:refs/remotes/origin/main",
-			);
-			expect(executedCommands).not.toContain("git fetch --prune origin");
-			expect(executedCommands).not.toContain("git reset --hard @{upstream}");
-			expect(executedCommands).not.toContain("git reset --hard origin/HEAD");
-			expect(executedCommands).not.toContain("git clean -fdx");
-			expect(executedCommands).not.toContain("npm install");
-		});
-
-		it("should update to latest commit when remote has new commits", async () => {
-			setupRemoteAndInstall();
-			expect(getFileContent(installedDir, "extension.ts")).toBe("// v1");
-
-			// Add a new commit to remote
-			const newCommit = createCommit(remoteDir, "extension.ts", "// v2", "Second commit");
-
-			// Update via package manager (no args = uses settings)
-			await packageManager.update();
-
-			// Verify update succeeded
-			expect(getCurrentCommit(installedDir)).toBe(newCommit);
-			expect(getFileContent(installedDir, "extension.ts")).toBe("// v2");
-		});
-
-		it("should handle multiple commits ahead", async () => {
-			setupRemoteAndInstall();
-
-			// Add multiple commits to remote
-			createCommit(remoteDir, "extension.ts", "// v2", "Second commit");
-			createCommit(remoteDir, "extension.ts", "// v3", "Third commit");
-			const latestCommit = createCommit(remoteDir, "extension.ts", "// v4", "Fourth commit");
-
-			await packageManager.update();
-
-			expect(getCurrentCommit(installedDir)).toBe(latestCommit);
-			expect(getFileContent(installedDir, "extension.ts")).toBe("// v4");
-		});
-
-		it("should update even when local checkout has no upstream", async () => {
-			setupRemoteAndInstall();
-			createCommit(remoteDir, "extension.ts", "// v2", "Second commit");
-			const latestCommit = createCommit(remoteDir, "extension.ts", "// v3", "Third commit");
-
-			const detachedCommit = getCurrentCommit(installedDir);
-			git(["checkout", detachedCommit], installedDir);
-
-			const executedCommands: string[] = [];
-			const managerWithInternals = packageManager as unknown as {
-				runCommand: (command: string, args: string[], options?: { cwd?: string }) => Promise<void>;
-			};
-			managerWithInternals.runCommand = async (command, args, options) => {
-				executedCommands.push(`${command} ${args.join(" ")}`);
-				const result = spawnSync(command, args, {
-					cwd: options?.cwd,
-					encoding: "utf-8",
-				});
-				if (result.status !== 0) {
-					throw new Error(`Command failed: ${command} ${args.join(" ")}\n${result.stderr}`);
-				}
-			};
-
-			await packageManager.update();
-
-			expect(executedCommands).toContain(
-				"git fetch --prune --no-tags origin +refs/heads/main:refs/remotes/origin/main",
-			);
-			expect(getCurrentCommit(installedDir)).toBe(latestCommit);
-			expect(getFileContent(installedDir, "extension.ts")).toBe("// v3");
-		});
-	});
-
-	describe("force-push scenarios", () => {
-		it("should recover when remote history is rewritten", async () => {
-			setupRemoteAndInstall();
-			const initialCommit = getCurrentCommit(remoteDir);
-
-			// Add commit to remote
-			createCommit(remoteDir, "extension.ts", "// v2", "Commit to keep");
-
-			// Update to get the new commit
-			await packageManager.update();
-			expect(getFileContent(installedDir, "extension.ts")).toBe("// v2");
-
-			// Now force-push to rewrite history on remote
-			git(["reset", "--hard", initialCommit], remoteDir);
-			const rewrittenCommit = createCommit(remoteDir, "extension.ts", "// v2-rewritten", "Rewritten commit");
-
-			// Update should succeed despite force-push
-			await packageManager.update();
-
-			expect(getCurrentCommit(installedDir)).toBe(rewrittenCommit);
-			expect(getFileContent(installedDir, "extension.ts")).toBe("// v2-rewritten");
-		});
-
-		it("should recover when local commit no longer exists in remote", async () => {
-			setupRemoteAndInstall();
-
-			// Add commits to remote
-			createCommit(remoteDir, "extension.ts", "// v2", "Commit A");
-			createCommit(remoteDir, "extension.ts", "// v3", "Commit B");
-
-			// Update to get all commits
-			await packageManager.update();
-			expect(getFileContent(installedDir, "extension.ts")).toBe("// v3");
-
-			// Force-push remote to remove commits A and B
-			git(["reset", "--hard", "HEAD~2"], remoteDir);
-			const newCommit = createCommit(remoteDir, "extension.ts", "// v2-new", "New commit replacing A and B");
-
-			// Update should succeed - the commits we had locally no longer exist
-			await packageManager.update();
-
-			expect(getCurrentCommit(installedDir)).toBe(newCommit);
-			expect(getFileContent(installedDir, "extension.ts")).toBe("// v2-new");
-		});
-
-		it("should handle complete history rewrite", async () => {
-			setupRemoteAndInstall();
-
-			// Remote gets several commits
-			createCommit(remoteDir, "extension.ts", "// v2", "v2");
-			createCommit(remoteDir, "extension.ts", "// v3", "v3");
-
-			await packageManager.update();
-			expect(getFileContent(installedDir, "extension.ts")).toBe("// v3");
-
-			// Maintainer force-pushes completely different history
-			git(["reset", "--hard", "HEAD~2"], remoteDir);
-			createCommit(remoteDir, "extension.ts", "// rewrite-a", "Rewrite A");
-			const finalCommit = createCommit(remoteDir, "extension.ts", "// rewrite-b", "Rewrite B");
-
-			// Should handle this gracefully
-			await packageManager.update();
-
-			expect(getCurrentCommit(installedDir)).toBe(finalCommit);
-			expect(getFileContent(installedDir, "extension.ts")).toBe("// rewrite-b");
-		});
-	});
-
-	describe("pinned sources", () => {
-		it("should not update pinned git sources (with @ref)", async () => {
-			// Create remote repo first to get the initial commit
-			mkdirSync(remoteDir, { recursive: true });
-			initGitRepo(remoteDir);
-			const initialCommit = createCommit(remoteDir, "extension.ts", "// v1", "Initial commit");
-
-			// Install with pinned ref from the start - full clone to ensure commit is available
-			mkdirSync(join(agentDir, "git", "github.com", "test"), { recursive: true });
-			git(["clone", remoteDir, installedDir], tempDir);
-			git(["checkout", initialCommit], installedDir);
-			git(["config", "--local", "user.email", "test@test.com"], installedDir);
-			git(["config", "--local", "user.name", "Test"], installedDir);
-
-			// Add to global packages with pinned ref
-			settingsManager.setPackages([`${gitSource}@${initialCommit}`]);
-
-			// Add new commit to remote
-			createCommit(remoteDir, "extension.ts", "// v2", "Second commit");
-
-			// Update should be skipped for pinned sources
-			await packageManager.update();
-
-			// Should still be on initial commit
-			expect(getCurrentCommit(installedDir)).toBe(initialCommit);
-			expect(getFileContent(installedDir, "extension.ts")).toBe("// v1");
-		});
-	});
-
-	describe("temporary git sources", () => {
-		it("should refresh cached temporary git sources when resolving", async () => {
-			const gitHost = "github.com";
-			const gitPath = "test/extension";
-			const hash = createHash("sha256").update(`git-${gitHost}-${gitPath}`).digest("hex").slice(0, 8);
-			const cachedDir = join(tmpdir(), "pi-extensions", `git-${gitHost}`, hash, gitPath);
-			const extensionFile = join(cachedDir, "pi-extensions", "session-breakdown.ts");
-
-			rmSync(cachedDir, { recursive: true, force: true });
-			mkdirSync(join(cachedDir, "pi-extensions"), { recursive: true });
-			writeFileSync(
-				join(cachedDir, "package.json"),
-				JSON.stringify({ pi: { extensions: ["./pi-extensions"] } }, null, 2),
-			);
-			writeFileSync(extensionFile, "// stale");
-
-			const executedCommands: string[] = [];
-			const managerWithInternals = packageManager as unknown as {
-				runCommand: (command: string, args: string[], options?: { cwd?: string }) => Promise<void>;
-				runCommandCapture: (command: string, args: string[], options?: { cwd?: string }) => Promise<string>;
-			};
-			managerWithInternals.runCommand = async (command, args) => {
-				executedCommands.push(`${command} ${args.join(" ")}`);
-				if (command === "git" && args[0] === "reset") {
-					writeFileSync(extensionFile, "// fresh");
-				}
-			};
-			managerWithInternals.runCommandCapture = async (_command, args) => {
-				if (args[0] === "rev-parse" && args[1] === "HEAD") {
-					return "local-head";
-				}
-				if (args[0] === "rev-parse" && args[1] === "@{upstream}") {
-					return "remote-head";
-				}
-				if (args[0] === "rev-parse" && args[1] === "--abbrev-ref") {
-					return "origin/main";
-				}
-				return "";
-			};
-
-			await packageManager.resolveExtensionSources([gitSource], { temporary: true });
-
-			expect(executedCommands).toContain(
-				"git fetch --prune --no-tags origin +refs/heads/main:refs/remotes/origin/main",
-			);
-			expect(getFileContent(cachedDir, "pi-extensions/session-breakdown.ts")).toBe("// fresh");
-		});
-
-		it("should not refresh pinned temporary git sources", async () => {
-			const gitHost = "github.com";
-			const gitPath = "test/extension";
-			const hash = createHash("sha256").update(`git-${gitHost}-${gitPath}`).digest("hex").slice(0, 8);
-			const cachedDir = join(tmpdir(), "pi-extensions", `git-${gitHost}`, hash, gitPath);
-			const extensionFile = join(cachedDir, "pi-extensions", "session-breakdown.ts");
-
-			rmSync(cachedDir, { recursive: true, force: true });
-			mkdirSync(join(cachedDir, "pi-extensions"), { recursive: true });
-			writeFileSync(
-				join(cachedDir, "package.json"),
-				JSON.stringify({ pi: { extensions: ["./pi-extensions"] } }, null, 2),
-			);
-			writeFileSync(extensionFile, "// pinned");
-
-			const executedCommands: string[] = [];
-			const managerWithInternals = packageManager as unknown as {
-				runCommand: (command: string, args: string[], options?: { cwd?: string }) => Promise<void>;
-			};
-			managerWithInternals.runCommand = async (command, args) => {
-				executedCommands.push(`${command} ${args.join(" ")}`);
-			};
-
-			await packageManager.resolveExtensionSources([`${gitSource}@main`], { temporary: true });
-
-			expect(executedCommands).toEqual([]);
-			expect(getFileContent(cachedDir, "pi-extensions/session-breakdown.ts")).toBe("// pinned");
-		});
-	});
-
-	describe("scope-aware update", () => {
-		it("should not install locally when source is only registered globally", async () => {
-			setupRemoteAndInstall();
-
-			// Add a new commit to remote
-			createCommit(remoteDir, "extension.ts", "// v2", "Second commit");
-
-			// The project-scope install path should not exist before or after update
-			const projectGitDir = join(tempDir, ".pi", "git", "github.com", "test", "extension");
-			expect(existsSync(projectGitDir)).toBe(false);
-
-			await packageManager.update(gitSource);
-
-			// Global install should be updated
-			expect(getFileContent(installedDir, "extension.ts")).toBe("// v2");
-
-			// Project-scope directory should NOT have been created
-			expect(existsSync(projectGitDir)).toBe(false);
-		});
-	});
-});

package/pi.dev-src/pi-mono/coding-agent/test/image-processing.test.ts

@@ -1,145 +0,0 @@
-/**
- * Tests for image processing utilities using Photon.
- */
-
-import { describe, expect, it } from "vitest";
-import { convertToPng } from "../src/utils/image-convert.js";
-import { formatDimensionNote, resizeImage } from "../src/utils/image-resize.js";
-
-// Small 2x2 red PNG image (base64) - generated with ImageMagick
-const TINY_PNG =
-	"iVBORw0KGgoAAAANSUhEUgAAAAIAAAACAQMAAABIeJ9nAAAAIGNIUk0AAHomAACAhAAA+gAAAIDoAAB1MAAA6mAAADqYAAAXcJy6UTwAAAAGUExURf8AAP///0EdNBEAAAABYktHRAH/Ai3eAAAAB3RJTUUH6gEOADM5Ddoh/wAAAAxJREFUCNdjYGBgAAAABAABJzQnCgAAACV0RVh0ZGF0ZTpjcmVhdGUAMjAyNi0wMS0xNFQwMDo1MTo1NyswMDowMOnKzHgAAAAldEVYdGRhdGU6bW9kaWZ5ADIwMjYtMDEtMTRUMDA6NTE6NTcrMDA6MDCYl3TEAAAAKHRFWHRkYXRlOnRpbWVzdGFtcAAyMDI2LTAxLTE0VDAwOjUxOjU3KzAwOjAwz4JVGwAAAABJRU5ErkJggg==";
-
-// Small 2x2 blue JPEG image (base64) - generated with ImageMagick
-const TINY_JPEG =
-	"/9j/4AAQSkZJRgABAQAAAQABAAD/2wBDAAMCAgMCAgMDAwMEAwMEBQgFBQQEBQoHBwYIDAoMDAsKCwsNDhIQDQ4RDgsLEBYQERMUFRUVDA8XGBYUGBIUFRT/2wBDAQMEBAUEBQkFBQkUDQsNFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBT/wAARCAACAAIDAREAAhEBAxEB/8QAFAABAAAAAAAAAAAAAAAAAAAACf/EABQQAQAAAAAAAAAAAAAAAAAAAAD/xAAVAQEBAAAAAAAAAAAAAAAAAAAGCf/EABQRAQAAAAAAAAAAAAAAAAAAAAD/2gAMAwEAAhEDEQA/AD3VTB3/2Q==";
-
-// 100x100 gray PNG
-const MEDIUM_PNG_100x100 =
-	"iVBORw0KGgoAAAANSUhEUgAAAGQAAABkCAAAAABVicqIAAAAAmJLR0QA/4ePzL8AAAAHdElNRQfqAQ4AMzkN2iH/AAAAP0lEQVRo3u3NQQEAAAQEMASXXYrz2gqst/Lm4ZBIJBKJRCKRSCQSiUQikUgkEolEIpFIJBKJRCKRSCQSiSTsAP1cAUZeKtreAAAAJXRFWHRkYXRlOmNyZWF0ZQAyMDI2LTAxLTE0VDAwOjUxOjU3KzAwOjAw6crMeAAAACV0RVh0ZGF0ZTptb2RpZnkAMjAyNi0wMS0xNFQwMDo1MTo1NyswMDowMJiXdMQAAAAodEVYdGRhdGU6dGltZXN0YW1wADIwMjYtMDEtMTRUMDA6NTE6NTcrMDA6MDDPglUbAAAAAElFTkSuQmCC";
-
-// 200x200 colored PNG
-const LARGE_PNG_200x200 =
-	"iVBORw0KGgoAAAANSUhEUgAAAMgAAADIEAIAAAByquWKAAAAIGNIUk0AAHomAACAhAAA+gAAAIDoAAB1MAAA6mAAADqYAAAXcJy6UTwAAAAGYktHRP///////wlY99wAAAAHdElNRQfqAQ4AMzqU03BFAAAEEElEQVR42u3dvWqUURSF4XXMRAyIhSAWQvwnWtjaWusFx9bW1kKD/wELESxEiBjDjJdwysOC57mCXW32+zGcGbtdACpsdi+T8Xz1GABzm91xMl6sHgNgblzcT/Y+rh4DYG6cv0/2H64eA2BOEgI1fHQHaoyzbXIwVo8BMCcJgRqSEKgxft9Lrn5aPQbA3Ph1klw7Wj0GwJwkBGpYWECN8fMiub63egyAORcWUMPCAmqMH3eTG59XjwEwN76/S24+Wj0GwJwkBGpYWECN8e1fcmuzegyAORcWUMPCAmqM09vJ4dfVYwDMjS9vkzuPV48BMCcJgRqeSAZqjA/nyYP91WMAzElCoIYkBGqMk8Pk6HT1GABzm+3L5JIkBApIQqCGj+5AjfHmb/Lk8uoxAOYkIVBDEgI1LCygxmZ7nFyShEABFxZQw8ICaozXf5KnV1aPATDnwgJqWFhADQsLqOFnDUANFxZQw8ICaoxXZ8mzg9VjAMy5sIAaFhZQw3tYQA1/QgHUkIRADUkI1JCEQA1JCNSQhEANFxZQwzcsoIYkBGpIQqCGJARqSEKghiQEalhYQA1/QgHUcGEBNSwsoIYkBGq4sIAaFhZQw8ICaviGBdRwYQE1LCyghiQEariwgBoWFlDDe1hADS+OAjUkIVBDEgI1JCFQQxICNSQhUMOFBdTwDQuoIQmBGpIQqCEJgRqSEKghCYEaFhZQw4ujQA0XFlDDwgJqSEKghgsLqGFhATUsLKCGb1hADRcWUMPCAmpIQqCGCwuoYWEBNbyHBdTw4ihQQxICNSQhUEMSAjUkIVBDEgI1XFhADd+wgBqSEKghCYEakhCoIQmBGpIQqGFhATW8OArUcGEBNSwsoIYkBGq4sIAaFhZQw8ICaviGBdRwYQE1LCyghiQEariwgBoWFlDDe1hADS+OAjUkIVBDEgI1JCFQQxICNSQhUMOFBdTwDQuoIQmBGpIQqCEJgRqSEKghCYEa49lB8ups9RgAc14cBWpIQqCGhQXUkIRADRcWUMPCAmqMp1eS139WjwEw5xsWUEMSAjUsLKCGJARquLCAGhYWUGM8uZy8+bt6DIA5L44CNSQhUMMTyUCNcXSanByuHgNgThICNSQhUGM82E8+nK8eA2DOzxqAGpIQqDHuPE6+vF09BsDcOPyanN5ePQbAnCQEavgdFlBj3Nok3/6tHgNgzoujQA1JCNQYNx8l39+tHgNgbtz4nPy4u3oMgDlJCNSwsIAa4/pe8vNi9RgAc37WANSQhECNce0o+XWyegyAuXH1U/L73uoxAOYkIVDDwgJqjIORnG1XjwEw508ogBqSEKgx9h8m5+9XjwEwN/Y+Jhf3V48BMCcJgRpjPE+2x6vHAJgbSbLbrR4DYO4/GqiSgXN+ksgAAAAldEVYdGRhdGU6Y3JlYXRlADIwMjYtMDEtMTRUMDA6NTE6NTcrMDA6MDDpysx4AAAAJXRFWHRkYXRlOm1vZGlmeQAyMDI2LTAxLTE0VDAwOjUxOjU3KzAwOjAwmJd0xAAAACh0RVh0ZGF0ZTp0aW1lc3RhbXAAMjAyNi0wMS0xNFQwMDo1MTo1NyswMDowMM+CVRsAAAAASUVORK5CYII=";
-
-describe("convertToPng", () => {
-	it("should return original data for PNG input", async () => {
-		const result = await convertToPng(TINY_PNG, "image/png");
-		expect(result).not.toBeNull();
-		expect(result!.data).toBe(TINY_PNG);
-		expect(result!.mimeType).toBe("image/png");
-	});
-
-	it("should convert JPEG to PNG", async () => {
-		const result = await convertToPng(TINY_JPEG, "image/jpeg");
-		expect(result).not.toBeNull();
-		expect(result!.mimeType).toBe("image/png");
-		// Result should be valid base64
-		expect(() => Buffer.from(result!.data, "base64")).not.toThrow();
-		// PNG magic bytes
-		const buffer = Buffer.from(result!.data, "base64");
-		expect(buffer[0]).toBe(0x89);
-		expect(buffer[1]).toBe(0x50); // 'P'
-		expect(buffer[2]).toBe(0x4e); // 'N'
-		expect(buffer[3]).toBe(0x47); // 'G'
-	});
-});
-
-describe("resizeImage", () => {
-	it("should return original image if within limits", async () => {
-		const result = await resizeImage(
-			{ type: "image", data: TINY_PNG, mimeType: "image/png" },
-			{ maxWidth: 100, maxHeight: 100, maxBytes: 1024 * 1024 },
-		);
-
-		expect(result).not.toBeNull();
-		expect(result!.wasResized).toBe(false);
-		expect(result!.data).toBe(TINY_PNG);
-		expect(result!.originalWidth).toBe(2);
-		expect(result!.originalHeight).toBe(2);
-		expect(result!.width).toBe(2);
-		expect(result!.height).toBe(2);
-	});
-
-	it("should resize image exceeding dimension limits", async () => {
-		const result = await resizeImage(
-			{ type: "image", data: MEDIUM_PNG_100x100, mimeType: "image/png" },
-			{ maxWidth: 50, maxHeight: 50, maxBytes: 1024 * 1024 },
-		);
-
-		expect(result).not.toBeNull();
-		expect(result!.wasResized).toBe(true);
-		expect(result!.originalWidth).toBe(100);
-		expect(result!.originalHeight).toBe(100);
-		expect(result!.width).toBeLessThanOrEqual(50);
-		expect(result!.height).toBeLessThanOrEqual(50);
-	});
-
-	it("should resize image exceeding byte limit", async () => {
-		const originalBuffer = Buffer.from(LARGE_PNG_200x200, "base64");
-		const originalSize = originalBuffer.length;
-
-		// Set maxBytes to less than the original encoded image size
-		const result = await resizeImage(
-			{ type: "image", data: LARGE_PNG_200x200, mimeType: "image/png" },
-			{ maxWidth: 2000, maxHeight: 2000, maxBytes: Math.floor(LARGE_PNG_200x200.length * 0.9) },
-		);
-
-		// Should have tried to reduce size
-		expect(result).not.toBeNull();
-		const resultBuffer = Buffer.from(result!.data, "base64");
-		expect(resultBuffer.length).toBeLessThan(originalSize);
-		expect(result!.data.length).toBeLessThan(LARGE_PNG_200x200.length);
-	});
-
-	it("should return null when image cannot be resized below maxBytes", async () => {
-		const result = await resizeImage(
-			{ type: "image", data: LARGE_PNG_200x200, mimeType: "image/png" },
-			{ maxWidth: 2000, maxHeight: 2000, maxBytes: 1 },
-		);
-
-		expect(result).toBeNull();
-	});
-
-	it("should handle JPEG input", async () => {
-		const result = await resizeImage(
-			{ type: "image", data: TINY_JPEG, mimeType: "image/jpeg" },
-			{ maxWidth: 100, maxHeight: 100, maxBytes: 1024 * 1024 },
-		);
-
-		expect(result).not.toBeNull();
-		expect(result!.wasResized).toBe(false);
-		expect(result!.originalWidth).toBe(2);
-		expect(result!.originalHeight).toBe(2);
-	});
-});
-
-describe("formatDimensionNote", () => {
-	it("should return undefined for non-resized images", () => {
-		const note = formatDimensionNote({
-			data: "",
-			mimeType: "image/png",
-			originalWidth: 100,
-			originalHeight: 100,
-			width: 100,
-			height: 100,
-			wasResized: false,
-		});
-		expect(note).toBeUndefined();
-	});
-
-	it("should return formatted note for resized images", () => {
-		const note = formatDimensionNote({
-			data: "",
-			mimeType: "image/png",
-			originalWidth: 2000,
-			originalHeight: 1000,
-			width: 1000,
-			height: 500,
-			wasResized: true,
-		});
-		expect(note).toContain("original 2000x1000");
-		expect(note).toContain("displayed at 1000x500");
-		expect(note).toContain("2.00"); // scale factor
-	});
-});

package/pi.dev-src/pi-mono/coding-agent/test/image-resize-callers.test.ts

@@ -1,53 +0,0 @@
-import { mkdirSync, rmSync, writeFileSync } from "node:fs";
-import { tmpdir } from "node:os";
-import { join } from "node:path";
-import { afterEach, beforeEach, describe, expect, it, vi } from "vitest";
-
-vi.mock("../src/utils/image-resize.js", () => ({
-	resizeImage: vi.fn(),
-	formatDimensionNote: vi.fn(() => undefined),
-}));
-
-import { processFileArguments } from "../src/cli/file-processor.js";
-import { createReadTool } from "../src/core/tools/read.js";
-import { resizeImage } from "../src/utils/image-resize.js";
-
-const TINY_PNG_BASE64 =
-	"iVBORw0KGgoAAAANSUhEUgAAAAEAAAABCAYAAAAfFcSJAAAADUlEQVR42mP8z8DwHwAFBQIAX8jx0gAAAABJRU5ErkJggg==";
-
-describe("image resize callers", () => {
-	let testDir: string;
-
-	beforeEach(() => {
-		testDir = join(tmpdir(), `image-resize-callers-${Date.now()}`);
-		mkdirSync(testDir, { recursive: true });
-		vi.mocked(resizeImage).mockReset();
-		vi.mocked(resizeImage).mockResolvedValue(null);
-	});
-
-	afterEach(() => {
-		rmSync(testDir, { recursive: true, force: true });
-	});
-
-	it("read tool returns text-only output when auto-resize cannot produce a safe image", async () => {
-		const imagePath = join(testDir, "test.png");
-		writeFileSync(imagePath, Buffer.from(TINY_PNG_BASE64, "base64"));
-
-		const tool = createReadTool(testDir);
-		const result = await tool.execute("test-read-image", { path: imagePath });
-
-		expect(result.content).toHaveLength(1);
-		expect(result.content[0].type).toBe("text");
-		expect((result.content[0] as { type: "text"; text: string }).text).toContain("Image omitted");
-	});
-
-	it("file processor omits image attachments when auto-resize cannot produce a safe image", async () => {
-		const imagePath = join(testDir, "test.png");
-		writeFileSync(imagePath, Buffer.from(TINY_PNG_BASE64, "base64"));
-
-		const result = await processFileArguments([imagePath]);
-
-		expect(result.images).toHaveLength(0);
-		expect(result.text).toContain("Image omitted");
-	});
-});

package/pi.dev-src/pi-mono/coding-agent/test/initial-message.test.ts

@@ -1,48 +0,0 @@
-import { describe, expect, test } from "vitest";
-import type { Args } from "../src/cli/args.js";
-import { buildInitialMessage } from "../src/cli/initial-message.js";
-
-function createArgs(messages: string[] = []): Args {
-	return {
-		messages: [...messages],
-		fileArgs: [],
-		unknownFlags: new Map(),
-		diagnostics: [],
-	};
-}
-
-describe("buildInitialMessage", () => {
-	test("merges piped stdin with the first CLI message into one prompt", () => {
-		const parsed = createArgs(["Summarize the text given"]);
-		const result = buildInitialMessage({
-			parsed,
-			stdinContent: "README contents\n",
-		});
-
-		expect(result.initialMessage).toBe("README contents\nSummarize the text given");
-		expect(parsed.messages).toEqual([]);
-	});
-
-	test("uses stdin as the initial prompt when no CLI message is present", () => {
-		const parsed = createArgs();
-		const result = buildInitialMessage({
-			parsed,
-			stdinContent: "README contents",
-		});
-
-		expect(result.initialMessage).toBe("README contents");
-		expect(parsed.messages).toEqual([]);
-	});
-
-	test("combines stdin, file text, and first CLI message in one prompt", () => {
-		const parsed = createArgs(["Explain it", "Second message"]);
-		const result = buildInitialMessage({
-			parsed,
-			stdinContent: "stdin\n",
-			fileText: "file\n",
-		});
-
-		expect(result.initialMessage).toBe("stdin\nfile\nExplain it");
-		expect(parsed.messages).toEqual(["Second message"]);
-	});
-});