pi-crew 0.1.41 → 0.1.43

package/src/utils/git.ts

@@ -1,262 +1,262 @@
-import { createRequire } from "node:module";
-
-type HostedGitInfoResult = {
-	repo?: unknown;
-	domain?: string;
-	project?: string;
-	user?: string;
-	committish?: string;
-};
-
-type HostedGitInfoModule = {
-	fromUrl: (url: string) => HostedGitInfoResult | null;
-};
-
-/**
- * Parsed git source information.
- */
-export type GitSource = {
-	/**
-	 * Always "git" for git sources.
-	 */
-	type: "git";
-	/**
-	 * Clone URL.
-	 */
-	repo: string;
-	/**
-	 * Git host domain (for example, "github.com").
-	 */
-	host: string;
-	/**
-	 * Repository path (for example, "org/repo").
-	 */
-	path: string;
-	/**
-	 * Git ref (branch, tag, or commit) if specified.
-	 */
-	ref?: string;
-	/**
-	 * Whether the source includes a pinned ref.
-	 */
-	pinned: boolean;
-};
-
-function getHostedGitInfo(): HostedGitInfoModule | null {
-	const require = createRequire(import.meta.url);
-	try {
-		return require("hosted-git-info") as HostedGitInfoModule;
-	} catch {
-		return null;
-	}
-}
-
-const hostedGitInfo = getHostedGitInfo();
-
-function splitRef(url: string): { repo: string; ref?: string } {
-	const scpLikeMatch = url.match(/^git@([^:]+):(.+)$/);
-	if (scpLikeMatch) {
-		const pathWithMaybeRef = scpLikeMatch[2] ?? "";
-		const atRefIndex = pathWithMaybeRef.indexOf("@");
-		const hashRefIndex = pathWithMaybeRef.indexOf("#");
-		let refSeparator = -1;
-		if (atRefIndex >= 0 && hashRefIndex >= 0) {
-			refSeparator = Math.min(atRefIndex, hashRefIndex);
-		} else {
-			refSeparator = atRefIndex >= 0 ? atRefIndex : hashRefIndex;
-		}
-		if (refSeparator < 0) {
-			return { repo: url };
-		}
-
-		const repoPath = pathWithMaybeRef.slice(0, refSeparator);
-		const ref = pathWithMaybeRef.slice(refSeparator + 1);
-		if (!repoPath || !ref) {
-			return { repo: url };
-		}
-
-		return {
-			repo: `git@${scpLikeMatch[1] ?? ""}:${repoPath}`,
-			ref,
-		};
-	}
-
-	if (url.includes("://")) {
-		try {
-			const parsed = new URL(url);
-			const fragmentRef = parsed.hash.startsWith("#") ? parsed.hash.slice(1) : "";
-			parsed.hash = "";
-			const pathWithMaybeRef = parsed.pathname.replace(/^\/+/, "");
-			const atRefIndex = pathWithMaybeRef.indexOf("@");
-			if (atRefIndex >= 0) {
-				const repoPath = pathWithMaybeRef.slice(0, atRefIndex);
-				const ref = pathWithMaybeRef.slice(atRefIndex + 1);
-				if (!repoPath || !ref) {
-					return { repo: parsed.toString().replace(/\/$/, "") };
-				}
-				parsed.pathname = `/${repoPath}`;
-				return {
-					repo: parsed.toString().replace(/\/$/, ""),
-					ref,
-				};
-			}
-
-			if (fragmentRef) {
-				return {
-					repo: parsed.toString().replace(/\/$/, ""),
-					ref: fragmentRef,
-				};
-			}
-
-			return { repo: parsed.toString().replace(/\/$/, "") };
-		} catch {
-			return { repo: url };
-		}
-	}
-
-	const slashIndex = url.indexOf("/");
-	if (slashIndex < 0) {
-		return { repo: url };
-	}
-
-	const pathWithMaybeRef = url.slice(slashIndex + 1);
-	const atRefIndex = pathWithMaybeRef.indexOf("@");
-	const hashRefIndex = pathWithMaybeRef.indexOf("#");
-	let refSeparator = -1;
-	if (atRefIndex >= 0 && hashRefIndex >= 0) {
-		refSeparator = Math.min(atRefIndex, hashRefIndex);
-	} else {
-		refSeparator = atRefIndex >= 0 ? atRefIndex : hashRefIndex;
-	}
-	if (refSeparator < 0) {
-		return { repo: url };
-	}
-
-	const repoPath = pathWithMaybeRef.slice(0, refSeparator);
-	const ref = pathWithMaybeRef.slice(refSeparator + 1);
-	if (!repoPath || !ref) {
-		return { repo: url };
-	}
-
-	return {
-		repo: `${url.slice(0, slashIndex)}/${repoPath}`,
-		ref,
-	};
-}
-
-function parseGenericGitUrl(url: string): GitSource | null {
-	const { repo: repoWithoutRef, ref } = splitRef(url);
-	let repo = repoWithoutRef;
-	let host = "";
-	let path = "";
-
-	const scpLikeMatch = repoWithoutRef.match(/^git@([^:]+):(.+)$/);
-	if (scpLikeMatch) {
-		host = scpLikeMatch[1] ?? "";
-		path = scpLikeMatch[2] ?? "";
-	} else if (
-		repoWithoutRef.startsWith("https://") ||
-		repoWithoutRef.startsWith("http://") ||
-		repoWithoutRef.startsWith("ssh://") ||
-		repoWithoutRef.startsWith("git://")
-	) {
-		try {
-			const parsed = new URL(repoWithoutRef);
-			host = parsed.hostname;
-			path = parsed.pathname.replace(/^\/+/, "");
-		} catch {
-			return null;
-		}
-	} else {
-		const slashIndex = repoWithoutRef.indexOf("/");
-		if (slashIndex < 0) {
-			return null;
-		}
-
-		host = repoWithoutRef.slice(0, slashIndex);
-		path = repoWithoutRef.slice(slashIndex + 1);
-		if (!host.includes(".") && host !== "localhost") {
-			return null;
-		}
-
-		repo = `https://${repoWithoutRef}`;
-	}
-
-	const normalizedPath = path.replace(/\.git$/, "").replace(/^\/+/, "");
-	if (!host || !normalizedPath || normalizedPath.split("/").length < 2) {
-		return null;
-	}
-
-	return {
-		type: "git",
-		repo,
-		host,
-		path: normalizedPath,
-		ref,
-		pinned: Boolean(ref),
-	};
-}
-
-/**
- * Parse git source into normalized source information.
- */
-export function parseGitUrl(source: string): GitSource | null {
-	const trimmed = source.trim();
-	const hasGitPrefix = trimmed.startsWith("git:");
-	const normalizedSource = hasGitPrefix ? trimmed.slice(4).trim() : trimmed.replace(/^git\+/i, "");
-	const url = normalizedSource;
-
-	if (
-		!hasGitPrefix &&
-		!/^(https?|ssh|git):\/\//i.test(url) &&
-		!/^git@[^:]+:/.test(url)
-	) {
-		return null;
-	}
-
-	const split = splitRef(url);
-
-	const parseCandidate = (candidate: string): GitSource | null => {
-		if (!hostedGitInfo) {
-			return null;
-		}
-		const info = hostedGitInfo.fromUrl(candidate) as HostedGitInfoResult | null;
-		if (!info) return null;
-		if (split.ref && info.project?.includes("@")) {
-			return null;
-		}
-		return {
-			type: "git",
-			repo: split.repo.startsWith("http://") || split.repo.startsWith("https://") || split.repo.startsWith("ssh://") || split.repo.startsWith("git://") || split.repo.startsWith("git@")
-				? split.repo
-				: `https://${split.repo}`,
-			host: info.domain ?? "",
-			path: `${info.user ?? ""}/${info.project ?? ""}`.replace(/\.git$/, ""),
-			ref: info.committish || split.ref || undefined,
-			pinned: Boolean(info.committish || split.ref),
-		};
-	};
-
-	const hostedCandidates = [split.ref ? `${split.repo}#${split.ref}` : undefined, url].filter((value): value is string =>
-		Boolean(value),
-	);
-	for (const candidate of hostedCandidates) {
-		const info = parseCandidate(candidate);
-		if (info) {
-			return info;
-		}
-	}
-
-	const httpsCandidates = [
-		split.ref ? `https://${split.repo}#${split.ref}` : undefined,
-		`https://${url}`,
-	].filter((value): value is string => Boolean(value));
-	for (const candidate of httpsCandidates) {
-		const info = parseCandidate(candidate);
-		if (info) {
-			return info;
-		}
-	}
-
-	return parseGenericGitUrl(url);
-}
+import { createRequire } from "node:module";
+
+type HostedGitInfoResult = {
+	repo?: unknown;
+	domain?: string;
+	project?: string;
+	user?: string;
+	committish?: string;
+};
+
+type HostedGitInfoModule = {
+	fromUrl: (url: string) => HostedGitInfoResult | null;
+};
+
+/**
+ * Parsed git source information.
+ */
+export type GitSource = {
+	/**
+	 * Always "git" for git sources.
+	 */
+	type: "git";
+	/**
+	 * Clone URL.
+	 */
+	repo: string;
+	/**
+	 * Git host domain (for example, "github.com").
+	 */
+	host: string;
+	/**
+	 * Repository path (for example, "org/repo").
+	 */
+	path: string;
+	/**
+	 * Git ref (branch, tag, or commit) if specified.
+	 */
+	ref?: string;
+	/**
+	 * Whether the source includes a pinned ref.
+	 */
+	pinned: boolean;
+};
+
+function getHostedGitInfo(): HostedGitInfoModule | null {
+	const require = createRequire(import.meta.url);
+	try {
+		return require("hosted-git-info") as HostedGitInfoModule;
+	} catch {
+		return null;
+	}
+}
+
+const hostedGitInfo = getHostedGitInfo();
+
+function splitRef(url: string): { repo: string; ref?: string } {
+	const scpLikeMatch = url.match(/^git@([^:]+):(.+)$/);
+	if (scpLikeMatch) {
+		const pathWithMaybeRef = scpLikeMatch[2] ?? "";
+		const atRefIndex = pathWithMaybeRef.indexOf("@");
+		const hashRefIndex = pathWithMaybeRef.indexOf("#");
+		let refSeparator = -1;
+		if (atRefIndex >= 0 && hashRefIndex >= 0) {
+			refSeparator = Math.min(atRefIndex, hashRefIndex);
+		} else {
+			refSeparator = atRefIndex >= 0 ? atRefIndex : hashRefIndex;
+		}
+		if (refSeparator < 0) {
+			return { repo: url };
+		}
+
+		const repoPath = pathWithMaybeRef.slice(0, refSeparator);
+		const ref = pathWithMaybeRef.slice(refSeparator + 1);
+		if (!repoPath || !ref) {
+			return { repo: url };
+		}
+
+		return {
+			repo: `git@${scpLikeMatch[1] ?? ""}:${repoPath}`,
+			ref,
+		};
+	}
+
+	if (url.includes("://")) {
+		try {
+			const parsed = new URL(url);
+			const fragmentRef = parsed.hash.startsWith("#") ? parsed.hash.slice(1) : "";
+			parsed.hash = "";
+			const pathWithMaybeRef = parsed.pathname.replace(/^\/+/, "");
+			const atRefIndex = pathWithMaybeRef.indexOf("@");
+			if (atRefIndex >= 0) {
+				const repoPath = pathWithMaybeRef.slice(0, atRefIndex);
+				const ref = pathWithMaybeRef.slice(atRefIndex + 1);
+				if (!repoPath || !ref) {
+					return { repo: parsed.toString().replace(/\/$/, "") };
+				}
+				parsed.pathname = `/${repoPath}`;
+				return {
+					repo: parsed.toString().replace(/\/$/, ""),
+					ref,
+				};
+			}
+
+			if (fragmentRef) {
+				return {
+					repo: parsed.toString().replace(/\/$/, ""),
+					ref: fragmentRef,
+				};
+			}
+
+			return { repo: parsed.toString().replace(/\/$/, "") };
+		} catch {
+			return { repo: url };
+		}
+	}
+
+	const slashIndex = url.indexOf("/");
+	if (slashIndex < 0) {
+		return { repo: url };
+	}
+
+	const pathWithMaybeRef = url.slice(slashIndex + 1);
+	const atRefIndex = pathWithMaybeRef.indexOf("@");
+	const hashRefIndex = pathWithMaybeRef.indexOf("#");
+	let refSeparator = -1;
+	if (atRefIndex >= 0 && hashRefIndex >= 0) {
+		refSeparator = Math.min(atRefIndex, hashRefIndex);
+	} else {
+		refSeparator = atRefIndex >= 0 ? atRefIndex : hashRefIndex;
+	}
+	if (refSeparator < 0) {
+		return { repo: url };
+	}
+
+	const repoPath = pathWithMaybeRef.slice(0, refSeparator);
+	const ref = pathWithMaybeRef.slice(refSeparator + 1);
+	if (!repoPath || !ref) {
+		return { repo: url };
+	}
+
+	return {
+		repo: `${url.slice(0, slashIndex)}/${repoPath}`,
+		ref,
+	};
+}
+
+function parseGenericGitUrl(url: string): GitSource | null {
+	const { repo: repoWithoutRef, ref } = splitRef(url);
+	let repo = repoWithoutRef;
+	let host = "";
+	let path = "";
+
+	const scpLikeMatch = repoWithoutRef.match(/^git@([^:]+):(.+)$/);
+	if (scpLikeMatch) {
+		host = scpLikeMatch[1] ?? "";
+		path = scpLikeMatch[2] ?? "";
+	} else if (
+		repoWithoutRef.startsWith("https://") ||
+		repoWithoutRef.startsWith("http://") ||
+		repoWithoutRef.startsWith("ssh://") ||
+		repoWithoutRef.startsWith("git://")
+	) {
+		try {
+			const parsed = new URL(repoWithoutRef);
+			host = parsed.hostname;
+			path = parsed.pathname.replace(/^\/+/, "");
+		} catch {
+			return null;
+		}
+	} else {
+		const slashIndex = repoWithoutRef.indexOf("/");
+		if (slashIndex < 0) {
+			return null;
+		}
+
+		host = repoWithoutRef.slice(0, slashIndex);
+		path = repoWithoutRef.slice(slashIndex + 1);
+		if (!host.includes(".") && host !== "localhost") {
+			return null;
+		}
+
+		repo = `https://${repoWithoutRef}`;
+	}
+
+	const normalizedPath = path.replace(/\.git$/, "").replace(/^\/+/, "");
+	if (!host || !normalizedPath || normalizedPath.split("/").length < 2) {
+		return null;
+	}
+
+	return {
+		type: "git",
+		repo,
+		host,
+		path: normalizedPath,
+		ref,
+		pinned: Boolean(ref),
+	};
+}
+
+/**
+ * Parse git source into normalized source information.
+ */
+export function parseGitUrl(source: string): GitSource | null {
+	const trimmed = source.trim();
+	const hasGitPrefix = trimmed.startsWith("git:");
+	const normalizedSource = hasGitPrefix ? trimmed.slice(4).trim() : trimmed.replace(/^git\+/i, "");
+	const url = normalizedSource;
+
+	if (
+		!hasGitPrefix &&
+		!/^(https?|ssh|git):\/\//i.test(url) &&
+		!/^git@[^:]+:/.test(url)
+	) {
+		return null;
+	}
+
+	const split = splitRef(url);
+
+	const parseCandidate = (candidate: string): GitSource | null => {
+		if (!hostedGitInfo) {
+			return null;
+		}
+		const info = hostedGitInfo.fromUrl(candidate) as HostedGitInfoResult | null;
+		if (!info) return null;
+		if (split.ref && info.project?.includes("@")) {
+			return null;
+		}
+		return {
+			type: "git",
+			repo: split.repo.startsWith("http://") || split.repo.startsWith("https://") || split.repo.startsWith("ssh://") || split.repo.startsWith("git://") || split.repo.startsWith("git@")
+				? split.repo
+				: `https://${split.repo}`,
+			host: info.domain ?? "",
+			path: `${info.user ?? ""}/${info.project ?? ""}`.replace(/\.git$/, ""),
+			ref: info.committish || split.ref || undefined,
+			pinned: Boolean(info.committish || split.ref),
+		};
+	};
+
+	const hostedCandidates = [split.ref ? `${split.repo}#${split.ref}` : undefined, url].filter((value): value is string =>
+		Boolean(value),
+	);
+	for (const candidate of hostedCandidates) {
+		const info = parseCandidate(candidate);
+		if (info) {
+			return info;
+		}
+	}
+
+	const httpsCandidates = [
+		split.ref ? `https://${split.repo}#${split.ref}` : undefined,
+		`https://${url}`,
+	].filter((value): value is string => Boolean(value));
+	for (const candidate of httpsCandidates) {
+		const info = parseCandidate(candidate);
+		if (info) {
+			return info;
+		}
+	}
+
+	return parseGenericGitUrl(url);
+}

package/src/utils/ids.ts

@@ -1,12 +1,12 @@
-import { randomBytes } from "node:crypto";
-
-export function createRunId(prefix = "team"): string {
-	const stamp = new Date().toISOString().replace(/[-:.TZ]/g, "").slice(0, 14);
-	const suffix = randomBytes(4).toString("hex");
-	return `${prefix}_${stamp}_${suffix}`;
-}
-
-export function createTaskId(stepId: string, index: number): string {
-	const normalized = stepId.toLowerCase().replace(/[^a-z0-9-]+/g, "-").replace(/^-+|-+$/g, "") || "task";
-	return `${String(index + 1).padStart(2, "0")}_${normalized}`;
-}
+import { randomBytes } from "node:crypto";
+
+export function createRunId(prefix = "team"): string {
+	const stamp = new Date().toISOString().replace(/[-:.TZ]/g, "").slice(0, 14);
+	const suffix = randomBytes(8).toString("hex");
+	return `${prefix}_${stamp}_${suffix}`;
+}
+
+export function createTaskId(stepId: string, index: number): string {
+	const normalized = stepId.toLowerCase().replace(/[^a-z0-9-]+/g, "-").replace(/^-+|-+$/g, "") || "task";
+	return `${String(index + 1).padStart(2, "0")}_${normalized}`;
+}

package/src/utils/internal-error.ts

@@ -1,6 +1,6 @@
 export function logInternalError(scope: string, error: unknown, details?: string): void {
 	if (!process.env.PI_TEAMS_DEBUG) return;
-	const message = error instanceof Error ? error.message : String(error);
+	const message = error instanceof Error ? error.message : (typeof error === "object" && error !== null ? JSON.stringify(error) : String(error));
 	const suffix = details ? `: ${details}` : "";
 	console.error(`[pi-crew:${scope}] ${message}${suffix}`);
 }

package/src/utils/names.ts

@@ -1,26 +1,27 @@
-export function sanitizeName(name: string): string {
-	return name.toLowerCase().trim().replace(/\s+/g, "-").replace(/[^a-z0-9-]/g, "").replace(/-+/g, "-").replace(/^-+|-+$/g, "");
-}
-
-export function requireString(value: unknown, label: string): { value?: string; error?: string } {
-	if (typeof value !== "string" || !value.trim()) return { error: `${label} must be a non-empty string.` };
-	return { value: value.trim() };
-}
-
-export function parseConfigObject(config: unknown): { value?: Record<string, unknown>; error?: string } {
-	let parsed = config;
-	if (typeof parsed === "string") {
-		try {
-			parsed = JSON.parse(parsed) as unknown;
-		} catch (error) {
-			const message = error instanceof Error ? error.message : String(error);
-			return { error: `config must be valid JSON: ${message}` };
-		}
-	}
-	if (!parsed || typeof parsed !== "object" || Array.isArray(parsed)) return { error: "config must be an object." };
-	return { value: parsed as Record<string, unknown> };
-}
-
-export function hasOwn(obj: Record<string, unknown>, key: string): boolean {
-	return Object.prototype.hasOwnProperty.call(obj, key);
-}
+export function sanitizeName(name: string): string {
+	const result = name.toLowerCase().trim().replace(/\s+/g, "-").replace(/[^a-z0-9-]/g, "").replace(/-+/g, "-").replace(/^-+|-+$/g, "");
+	return result || "unnamed";
+}
+
+export function requireString(value: unknown, label: string): { value?: string; error?: string } {
+	if (typeof value !== "string" || !value.trim()) return { error: `${label} must be a non-empty string.` };
+	return { value: value.trim() };
+}
+
+export function parseConfigObject(config: unknown): { value?: Record<string, unknown>; error?: string } {
+	let parsed = config;
+	if (typeof parsed === "string") {
+		try {
+			parsed = JSON.parse(parsed) as unknown;
+		} catch (error) {
+			const message = error instanceof Error ? error.message : String(error);
+			return { error: `config must be valid JSON: ${message}` };
+		}
+	}
+	if (!parsed || typeof parsed !== "object" || Array.isArray(parsed)) return { error: "config must be an object." };
+	return { value: parsed as Record<string, unknown> };
+}
+
+export function hasOwn(obj: Record<string, unknown>, key: string): boolean {
+	return Object.prototype.hasOwnProperty.call(obj, key);
+}

package/src/utils/paths.ts

@@ -31,8 +31,8 @@ export function findRepoRoot(cwd: string): string | undefined {
 	const home = path.resolve(os.homedir());
 	const tempRoot = path.resolve(os.tmpdir());
 	while (current !== root) {
-		if (current === home || current === tempRoot) return undefined;
 		if (hasProjectMarker(current)) return current;
+		if (current === home || current === tempRoot) return undefined;
 		const parent = path.dirname(current);
 		if (parent === current) break;
 		current = parent;