pi-usereq 0.49.0 → 0.50.0

package/pi.dev-src/pi-mono/coding-agent/test/fixtures/skills/consecutive-hyphens/SKILL.md

@@ -1,8 +0,0 @@
----
-name: bad--name
-description: A skill with consecutive hyphens in the name.
----
-
-# Consecutive Hyphens
-
-This skill has consecutive hyphens in its name.

package/pi.dev-src/pi-mono/coding-agent/test/fixtures/skills/disable-model-invocation/SKILL.md

@@ -1,9 +0,0 @@
----
-name: disable-model-invocation
-description: A skill that cannot be invoked by the model.
-disable-model-invocation: true
----
-
-# Manual Only Skill
-
-This skill can only be invoked via /skill:disable-model-invocation.

package/pi.dev-src/pi-mono/coding-agent/test/fixtures/skills/invalid-name-chars/SKILL.md

@@ -1,8 +0,0 @@
----
-name: Invalid_Name
-description: A skill with invalid characters in the name.
----
-
-# Invalid Name
-
-This skill has uppercase and underscore in the name.

package/pi.dev-src/pi-mono/coding-agent/test/fixtures/skills/invalid-yaml/SKILL.md

@@ -1,8 +0,0 @@
----
-name: invalid-yaml
-description: [unclosed bracket
----
-
-# Invalid YAML Skill
-
-This skill has invalid YAML in the frontmatter.

package/pi.dev-src/pi-mono/coding-agent/test/fixtures/skills/long-name/SKILL.md

@@ -1,8 +0,0 @@
----
-name: this-is-a-very-long-skill-name-that-exceeds-the-sixty-four-character-limit-set-by-the-standard
-description: A skill with a name that exceeds 64 characters.
----
-
-# Long Name
-
-This skill's name is too long.

package/pi.dev-src/pi-mono/coding-agent/test/fixtures/skills/missing-description/SKILL.md

@@ -1,7 +0,0 @@
----
-name: missing-description
----
-
-# Missing Description
-
-This skill has no description field.

package/pi.dev-src/pi-mono/coding-agent/test/fixtures/skills/multiline-description/SKILL.md

@@ -1,11 +0,0 @@
----
-name: multiline-description
-description: |
-  This is a multiline description.
-  It spans multiple lines.
-  And should be normalized.
----
-
-# Multiline Description Skill
-
-This skill tests that multiline YAML descriptions are normalized to single lines.

package/pi.dev-src/pi-mono/coding-agent/test/fixtures/skills/name-mismatch/SKILL.md

@@ -1,8 +0,0 @@
----
-name: different-name
-description: A skill with a name that doesn't match the directory.
----
-
-# Name Mismatch
-
-This skill's name doesn't match its parent directory.

package/pi.dev-src/pi-mono/coding-agent/test/fixtures/skills/nested/child-skill/SKILL.md

@@ -1,8 +0,0 @@
----
-name: child-skill
-description: A nested skill in a subdirectory.
----
-
-# Child Skill
-
-This skill is nested in a subdirectory.

package/pi.dev-src/pi-mono/coding-agent/test/fixtures/skills/no-frontmatter/SKILL.md

@@ -1,3 +0,0 @@
-# No Frontmatter
-
-This skill has no YAML frontmatter at all.

package/pi.dev-src/pi-mono/coding-agent/test/fixtures/skills/root-skill-preferred/SKILL.md

@@ -1,3 +0,0 @@
----
-description: Root skill should win.
----

package/pi.dev-src/pi-mono/coding-agent/test/fixtures/skills/root-skill-preferred/nested-child/SKILL.md

@@ -1,3 +0,0 @@
----
-description: Nested skill should be ignored.
----

package/pi.dev-src/pi-mono/coding-agent/test/fixtures/skills/unknown-field/SKILL.md

@@ -1,10 +0,0 @@
----
-name: unknown-field
-description: A skill with an unknown frontmatter field.
-author: someone
-version: 1.0
----
-
-# Unknown Field
-
-This skill has non-standard frontmatter fields.

package/pi.dev-src/pi-mono/coding-agent/test/fixtures/skills/valid-skill/SKILL.md

@@ -1,8 +0,0 @@
----
-name: valid-skill
-description: A valid skill for testing purposes.
----
-
-# Valid Skill
-
-This is a valid skill that follows the Agent Skills standard.

package/pi.dev-src/pi-mono/coding-agent/test/fixtures/skills-collision/first/calendar/SKILL.md

@@ -1,8 +0,0 @@
----
-name: calendar
-description: First calendar skill.
----
-
-# Calendar (First)
-
-This is the first calendar skill.

package/pi.dev-src/pi-mono/coding-agent/test/fixtures/skills-collision/second/calendar/SKILL.md

@@ -1,8 +0,0 @@
----
-name: calendar
-description: Second calendar skill.
----
-
-# Calendar (Second)
-
-This is the second calendar skill.

package/pi.dev-src/pi-mono/coding-agent/test/footer-data-provider.test.ts

@@ -1,265 +0,0 @@
-import { execFile, spawnSync } from "child_process";
-import { existsSync, type FSWatcher, mkdirSync, mkdtempSync, rmSync, writeFileSync } from "fs";
-import { tmpdir } from "os";
-import { join } from "path";
-import { afterEach, beforeEach, describe, expect, it, vi } from "vitest";
-
-let resolvedBranch = "main";
-
-vi.mock("child_process", () => ({
-	execFile: vi.fn(
-		(
-			_command: string,
-			args: readonly string[],
-			_options: unknown,
-			callback: (error: Error | null, stdout: string, stderr: string) => void,
-		) => {
-			if (args[1] === "symbolic-ref") {
-				setTimeout(
-					() =>
-						callback(
-							resolvedBranch ? null : new Error("detached"),
-							resolvedBranch ? `${resolvedBranch}\n` : "",
-							"",
-						),
-					0,
-				);
-				return;
-			}
-			setTimeout(() => callback(new Error("unsupported"), "", ""), 0);
-		},
-	),
-	spawnSync: vi.fn((_command: string, args: readonly string[]) => {
-		if (args[1] === "symbolic-ref") {
-			return { status: resolvedBranch ? 0 : 1, stdout: resolvedBranch ? `${resolvedBranch}\n` : "", stderr: "" };
-		}
-		return { status: 1, stdout: "", stderr: "" };
-	}),
-}));
-
-import { FooterDataProvider } from "../src/core/footer-data-provider.js";
-
-type WorktreeFixture = {
-	worktreeDir: string;
-	reftableDir: string;
-};
-
-function createPlainReftableRepo(tempDir: string): string {
-	const repoDir = join(tempDir, "repo");
-	mkdirSync(join(repoDir, ".git", "reftable"), { recursive: true });
-	writeFileSync(join(repoDir, ".git", "HEAD"), "ref: refs/heads/.invalid\n");
-	return repoDir;
-}
-
-function createPlainRepo(tempDir: string): string {
-	const repoDir = join(tempDir, "repo");
-	mkdirSync(join(repoDir, ".git"), { recursive: true });
-	writeFileSync(join(repoDir, ".git", "HEAD"), "ref: refs/heads/main\n");
-	return repoDir;
-}
-
-function createReftableWorktree(tempDir: string): WorktreeFixture {
-	const repoDir = join(tempDir, "repo");
-	const commonGitDir = join(repoDir, ".git");
-	const gitDir = join(commonGitDir, "worktrees", "src");
-	const worktreeDir = join(tempDir, "worktree");
-	const reftableDir = join(commonGitDir, "reftable");
-
-	mkdirSync(gitDir, { recursive: true });
-	mkdirSync(reftableDir, { recursive: true });
-	mkdirSync(worktreeDir, { recursive: true });
-
-	writeFileSync(join(worktreeDir, ".git"), `gitdir: ${gitDir}\n`);
-	writeFileSync(join(gitDir, "HEAD"), "ref: refs/heads/.invalid\n");
-	writeFileSync(join(gitDir, "commondir"), "../..\n");
-	writeFileSync(join(reftableDir, "tables.list"), "0\n");
-
-	return { worktreeDir, reftableDir };
-}
-
-async function waitFor(condition: () => boolean, timeoutMs = 3000): Promise<void> {
-	const startedAt = Date.now();
-	while (!condition()) {
-		if (Date.now() - startedAt > timeoutMs) {
-			throw new Error("Timed out waiting for condition");
-		}
-		await new Promise((resolve) => setTimeout(resolve, 10));
-	}
-}
-
-describe("FooterDataProvider reftable branch detection", () => {
-	let originalCwd: string;
-	let tempDir: string;
-
-	beforeEach(() => {
-		originalCwd = process.cwd();
-		tempDir = mkdtempSync(join(tmpdir(), "footer-data-provider-"));
-		resolvedBranch = "main";
-		vi.mocked(spawnSync).mockClear();
-		vi.mocked(execFile).mockClear();
-	});
-
-	afterEach(() => {
-		process.chdir(originalCwd);
-		if (tempDir && existsSync(tempDir)) {
-			rmSync(tempDir, { recursive: true, force: true });
-		}
-	});
-
-	it("uses HEAD directly in a regular repo from a nested directory", () => {
-		const repoDir = createPlainRepo(tempDir);
-		const nestedDir = join(repoDir, "src", "nested");
-		mkdirSync(nestedDir, { recursive: true });
-		process.chdir(nestedDir);
-
-		const provider = new FooterDataProvider(nestedDir);
-		try {
-			expect(provider.getGitBranch()).toBe("main");
-			expect(vi.mocked(spawnSync)).not.toHaveBeenCalled();
-		} finally {
-			provider.dispose();
-		}
-	});
-
-	it("resolves the branch via git when HEAD is .invalid in a reftable repo", () => {
-		const repoDir = createPlainReftableRepo(tempDir);
-		process.chdir(repoDir);
-
-		const provider = new FooterDataProvider(repoDir);
-		try {
-			expect(provider.getGitBranch()).toBe("main");
-			expect(vi.mocked(spawnSync)).toHaveBeenCalledWith(
-				"git",
-				["--no-optional-locks", "symbolic-ref", "--quiet", "--short", "HEAD"],
-				expect.objectContaining({
-					cwd: expect.stringMatching(/repo$/),
-					encoding: "utf8",
-					stdio: ["ignore", "pipe", "ignore"],
-				}),
-			);
-		} finally {
-			provider.dispose();
-		}
-	});
-
-	it("resolves the branch via git in a reftable-backed worktree", () => {
-		const { worktreeDir } = createReftableWorktree(tempDir);
-		process.chdir(worktreeDir);
-
-		const provider = new FooterDataProvider(worktreeDir);
-		try {
-			expect(provider.getGitBranch()).toBe("main");
-		} finally {
-			provider.dispose();
-		}
-	});
-
-	it("treats an unresolved .invalid reftable HEAD as detached", () => {
-		const repoDir = createPlainReftableRepo(tempDir);
-		process.chdir(repoDir);
-		resolvedBranch = "";
-
-		const provider = new FooterDataProvider(repoDir);
-		try {
-			expect(provider.getGitBranch()).toBe("detached");
-		} finally {
-			provider.dispose();
-		}
-	});
-
-	it("does not notify listeners when reftable updates keep the same branch", async () => {
-		const { worktreeDir, reftableDir } = createReftableWorktree(tempDir);
-		process.chdir(worktreeDir);
-
-		const provider = new FooterDataProvider(worktreeDir);
-		try {
-			expect(provider.getGitBranch()).toBe("main");
-			vi.mocked(spawnSync).mockClear();
-			const onBranchChange = vi.fn();
-			provider.onBranchChange(onBranchChange);
-
-			writeFileSync(join(reftableDir, "tables.list"), "1\n");
-			await waitFor(() => vi.mocked(execFile).mock.calls.length === 1);
-
-			expect(vi.mocked(execFile)).toHaveBeenCalledTimes(1);
-			expect(vi.mocked(spawnSync)).not.toHaveBeenCalled();
-			expect(provider.getGitBranch()).toBe("main");
-			expect(onBranchChange).not.toHaveBeenCalled();
-		} finally {
-			provider.dispose();
-		}
-	});
-
-	it("debounces rapid reftable updates into a single async refresh", async () => {
-		const { worktreeDir, reftableDir } = createReftableWorktree(tempDir);
-		process.chdir(worktreeDir);
-
-		const provider = new FooterDataProvider(worktreeDir);
-		try {
-			expect(provider.getGitBranch()).toBe("main");
-			vi.mocked(execFile).mockClear();
-
-			writeFileSync(join(reftableDir, "tables.list"), "1\n");
-			writeFileSync(join(reftableDir, "tables.list"), "2\n");
-			writeFileSync(join(reftableDir, "tables.list"), "3\n");
-			await waitFor(() => vi.mocked(execFile).mock.calls.length === 1);
-			await new Promise((resolve) => setTimeout(resolve, 650));
-
-			expect(vi.mocked(execFile)).toHaveBeenCalledTimes(1);
-		} finally {
-			provider.dispose();
-		}
-	});
-
-	it("updates the cached branch when the reftable directory changes", async () => {
-		const { worktreeDir, reftableDir } = createReftableWorktree(tempDir);
-		process.chdir(worktreeDir);
-
-		const provider = new FooterDataProvider(worktreeDir);
-		try {
-			expect(provider.getGitBranch()).toBe("main");
-			resolvedBranch = "foo";
-			const onBranchChange = vi.fn();
-			provider.onBranchChange(onBranchChange);
-
-			writeFileSync(join(reftableDir, "tables.list"), "1\n");
-			await waitFor(() => vi.mocked(execFile).mock.calls.length === 1);
-			await waitFor(() => provider.getGitBranch() === "foo");
-
-			expect(vi.mocked(execFile)).toHaveBeenCalledTimes(1);
-			expect(provider.getGitBranch()).toBe("foo");
-			expect(onBranchChange).toHaveBeenCalledTimes(1);
-		} finally {
-			provider.dispose();
-		}
-	});
-
-	it("retries git watchers 5 seconds after an async fs.watch error", async () => {
-		vi.useFakeTimers();
-		const repoDir = createPlainRepo(tempDir);
-		process.chdir(repoDir);
-
-		const provider = new FooterDataProvider(repoDir);
-		try {
-			const providerWithInternals = provider as unknown as {
-				headWatcher: FSWatcher | null;
-			};
-			const originalWatcher = providerWithInternals.headWatcher;
-			expect(originalWatcher).not.toBeNull();
-			expect(originalWatcher?.listenerCount("error")).toBeGreaterThan(0);
-
-			originalWatcher?.emit("error", new Error("simulated EMFILE"));
-			expect(providerWithInternals.headWatcher).toBeNull();
-
-			await vi.advanceTimersByTimeAsync(4999);
-			expect(providerWithInternals.headWatcher).toBeNull();
-
-			await vi.advanceTimersByTimeAsync(1);
-			expect(providerWithInternals.headWatcher).not.toBeNull();
-			expect(providerWithInternals.headWatcher).not.toBe(originalWatcher);
-		} finally {
-			provider.dispose();
-			vi.useRealTimers();
-		}
-	});
-});

package/pi.dev-src/pi-mono/coding-agent/test/footer-width.test.ts

@@ -1,115 +0,0 @@
-import { visibleWidth } from "@mariozechner/pi-tui";
-import { beforeAll, describe, expect, it } from "vitest";
-import type { AgentSession } from "../src/core/agent-session.js";
-import type { ReadonlyFooterDataProvider } from "../src/core/footer-data-provider.js";
-import { FooterComponent } from "../src/modes/interactive/components/footer.js";
-import { initTheme } from "../src/modes/interactive/theme/theme.js";
-
-type AssistantUsage = {
-	input: number;
-	output: number;
-	cacheRead: number;
-	cacheWrite: number;
-	cost: { total: number };
-};
-
-function createSession(options: {
-	sessionName: string;
-	modelId?: string;
-	provider?: string;
-	reasoning?: boolean;
-	thinkingLevel?: string;
-	usage?: AssistantUsage;
-}): AgentSession {
-	const usage = options.usage;
-	const entries =
-		usage === undefined
-			? []
-			: [
-					{
-						type: "message",
-						message: {
-							role: "assistant",
-							usage,
-						},
-					},
-				];
-
-	const session = {
-		state: {
-			model: {
-				id: options.modelId ?? "test-model",
-				provider: options.provider ?? "test",
-				contextWindow: 200_000,
-				reasoning: options.reasoning ?? false,
-			},
-			thinkingLevel: options.thinkingLevel ?? "off",
-		},
-		sessionManager: {
-			getEntries: () => entries,
-			getSessionName: () => options.sessionName,
-			getCwd: () => "/tmp/project",
-		},
-		getContextUsage: () => ({ contextWindow: 200_000, percent: 12.3 }),
-		modelRegistry: {
-			isUsingOAuth: () => false,
-		},
-	};
-
-	return session as unknown as AgentSession;
-}
-
-function createFooterData(providerCount: number): ReadonlyFooterDataProvider {
-	const provider = {
-		getGitBranch: () => "main",
-		getExtensionStatuses: () => new Map<string, string>(),
-		getAvailableProviderCount: () => providerCount,
-		onBranchChange: (callback: () => void) => {
-			void callback;
-			return () => {};
-		},
-	};
-
-	return provider;
-}
-
-describe("FooterComponent width handling", () => {
-	beforeAll(() => {
-		initTheme(undefined, false);
-	});
-
-	it("keeps all lines within width for wide session names", () => {
-		const width = 93;
-		const session = createSession({ sessionName: "한글".repeat(30) });
-		const footer = new FooterComponent(session, createFooterData(1));
-
-		const lines = footer.render(width);
-		for (const line of lines) {
-			expect(visibleWidth(line)).toBeLessThanOrEqual(width);
-		}
-	});
-
-	it("keeps stats line within width for wide model and provider names", () => {
-		const width = 60;
-		const session = createSession({
-			sessionName: "",
-			modelId: "模".repeat(30),
-			provider: "공급자",
-			reasoning: true,
-			thinkingLevel: "high",
-			usage: {
-				input: 12_345,
-				output: 6_789,
-				cacheRead: 0,
-				cacheWrite: 0,
-				cost: { total: 1.234 },
-			},
-		});
-		const footer = new FooterComponent(session, createFooterData(2));
-
-		const lines = footer.render(width);
-		for (const line of lines) {
-			expect(visibleWidth(line)).toBeLessThanOrEqual(width);
-		}
-	});
-});

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

@@ -1,60 +0,0 @@
-import { describe, expect, it } from "vitest";
-import { parseFrontmatter, stripFrontmatter } from "../src/utils/frontmatter.js";
-
-describe("parseFrontmatter", () => {
-	it("parses keys, strips quotes, and returns body", () => {
-		const input = "---\nname: \"skill-name\"\ndescription: 'A desc'\nfoo-bar: value\n---\n\nBody text";
-		const { frontmatter, body } = parseFrontmatter<Record<string, string>>(input);
-		expect(frontmatter.name).toBe("skill-name");
-		expect(frontmatter.description).toBe("A desc");
-		expect(frontmatter["foo-bar"]).toBe("value");
-		expect(body).toBe("Body text");
-	});
-
-	it("normalizes newlines and handles CRLF", () => {
-		const input = "---\r\nname: test\r\n---\r\nLine one\r\nLine two";
-		const { body } = parseFrontmatter<Record<string, string>>(input);
-		expect(body).toBe("Line one\nLine two");
-	});
-
-	it("throws on invalid YAML frontmatter", () => {
-		const input = "---\nfoo: [bar\n---\nBody";
-		expect(() => parseFrontmatter<Record<string, string>>(input)).toThrow(/at line 1, column 10/);
-	});
-
-	it("parses | multiline yaml syntax", () => {
-		const input = "---\ndescription: |\n  Line one\n  Line two\n---\n\nBody";
-		const { frontmatter, body } = parseFrontmatter<Record<string, string>>(input);
-		expect(frontmatter.description).toBe("Line one\nLine two\n");
-		expect(body).toBe("Body");
-	});
-
-	it("returns original content when frontmatter is missing or unterminated", () => {
-		const noFrontmatter = "Just text\nsecond line";
-		const missingEnd = "---\nname: test\nBody without terminator";
-		const resultNoFrontmatter = parseFrontmatter<Record<string, string>>(noFrontmatter);
-		const resultMissingEnd = parseFrontmatter<Record<string, string>>(missingEnd);
-		expect(resultNoFrontmatter.body).toBe("Just text\nsecond line");
-		expect(resultMissingEnd.body).toBe(
-			"---\nname: test\nBody without terminator".replace(/\r\n/g, "\n").replace(/\r/g, "\n"),
-		);
-	});
-
-	it("returns empty object for empty or comment-only frontmatter", () => {
-		const input = "---\n# just a comment\n---\nBody";
-		const { frontmatter } = parseFrontmatter(input);
-		expect(frontmatter).toEqual({});
-	});
-});
-
-describe("stripFrontmatter", () => {
-	it("removes frontmatter and trims body", () => {
-		const input = "---\nkey: value\n---\n\nBody\n";
-		expect(stripFrontmatter(input)).toBe("Body");
-	});
-
-	it("returns body when no frontmatter present", () => {
-		const input = "\n  No frontmatter body  \n";
-		expect(stripFrontmatter(input)).toBe("\n  No frontmatter body  \n");
-	});
-});

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

@@ -1,78 +0,0 @@
-import { describe, expect, it } from "vitest";
-import { parseGitUrl } from "../src/utils/git.js";
-
-describe("Git URL Parsing", () => {
-	describe("protocol URLs (accepted without git: prefix)", () => {
-		it("should parse HTTPS URL", () => {
-			const result = parseGitUrl("https://github.com/user/repo");
-			expect(result).toMatchObject({
-				host: "github.com",
-				path: "user/repo",
-				repo: "https://github.com/user/repo",
-			});
-		});
-
-		it("should parse ssh:// URL", () => {
-			const result = parseGitUrl("ssh://git@github.com/user/repo");
-			expect(result).toMatchObject({
-				host: "github.com",
-				path: "user/repo",
-				repo: "ssh://git@github.com/user/repo",
-			});
-		});
-
-		it("should parse protocol URL with ref", () => {
-			const result = parseGitUrl("https://github.com/user/repo@v1.0.0");
-			expect(result).toMatchObject({
-				host: "github.com",
-				path: "user/repo",
-				ref: "v1.0.0",
-				repo: "https://github.com/user/repo",
-			});
-		});
-	});
-
-	describe("shorthand URLs (accepted only with git: prefix)", () => {
-		it("should parse git@host:path with git: prefix", () => {
-			const result = parseGitUrl("git:git@github.com:user/repo");
-			expect(result).toMatchObject({
-				host: "github.com",
-				path: "user/repo",
-				repo: "git@github.com:user/repo",
-			});
-		});
-
-		it("should parse host/path shorthand with git: prefix", () => {
-			const result = parseGitUrl("git:github.com/user/repo");
-			expect(result).toMatchObject({
-				host: "github.com",
-				path: "user/repo",
-				repo: "https://github.com/user/repo",
-			});
-		});
-
-		it("should parse shorthand with ref and git: prefix", () => {
-			const result = parseGitUrl("git:git@github.com:user/repo@v1.0.0");
-			expect(result).toMatchObject({
-				host: "github.com",
-				path: "user/repo",
-				ref: "v1.0.0",
-				repo: "git@github.com:user/repo",
-			});
-		});
-	});
-
-	describe("unsupported without git: prefix", () => {
-		it("should reject git@host:path without git: prefix", () => {
-			expect(parseGitUrl("git@github.com:user/repo")).toBeNull();
-		});
-
-		it("should reject host/path shorthand without git: prefix", () => {
-			expect(parseGitUrl("github.com/user/repo")).toBeNull();
-		});
-
-		it("should reject user/repo shorthand", () => {
-			expect(parseGitUrl("user/repo")).toBeNull();
-		});
-	});
-});