pi-crew 0.9.55 → 0.9.57

package/scripts/check-all-skills.ts

@@ -1,294 +0,0 @@
-#!/usr/bin/env node
-
-/**
- * Check All Skills Script
- *
- * Runs verify-skill.ts against all skills and produces a summary report.
- *
- * Usage:
- *   node scripts/check-all-skills.ts
- */
-
-import { spawnSync } from "node:child_process";
-import * as fs from "node:fs";
-import * as path from "node:path";
-
-interface SkillSummary {
-	name: string;
-	path: string;
-	status: "pass" | "fail" | "warning";
-	warnings: string[];
-	errors: string[];
-}
-
-interface Report {
-	total: number;
-	passed: number;
-	failed: number;
-	warningsOnly: number;
-	skills: SkillSummary[];
-	timestamp: string;
-}
-
-const SKILLS_DIR = path.join(process.cwd(), "skills");
-
-/**
- * Get all skill directories
- */
-function getSkillDirs(): string[] {
-	if (!fs.existsSync(SKILLS_DIR)) {
-		console.error("Skills directory not found: " + SKILLS_DIR);
-		return [];
-	}
-
-	const dirs: string[] = [];
-	const entries = fs.readdirSync(SKILLS_DIR, { withFileTypes: true });
-
-	for (const entry of entries) {
-		if (entry.isDirectory()) {
-			const skillPath = path.join(SKILLS_DIR, entry.name, "SKILL.md");
-			if (fs.existsSync(skillPath)) {
-				dirs.push(entry.name);
-			}
-		}
-	}
-
-	return dirs.sort();
-}
-
-/**
- * Run verify-skill.ts for a single skill
- */
-function verifySingleSkill(skillName: string): {
-	passed: boolean;
-	warnings: string[];
-	errors: string[];
-} {
-	const skillPath = path.join(SKILLS_DIR, skillName, "SKILL.md");
-
-	const result = spawnSync("node", ["scripts/verify-skill.ts", skillPath], {
-		cwd: process.cwd(),
-		encoding: "utf-8",
-	});
-
-	const output = result.stdout + result.stderr;
-	const warnings: string[] = [];
-	const errors: string[] = [];
-
-	// Parse output
-	const warningMatches = output.match(/[^\n]*⚠️[^\n]*/g);
-	const errorMatches = output.match(/[^\n]+FAIL[^\n]+/g);
-
-	if (warningMatches) {
-		warnings.push(...warningMatches.map((w) => w.trim()));
-	}
-	if (errorMatches) {
-		errors.push(...errorMatches.map((e) => e.trim()));
-	}
-
-	const passed = result.status === 0;
-
-	return { passed, warnings, errors };
-}
-
-/**
- * Generate markdown report
- */
-function generateMarkdownReport(report: Report): string {
-	const lines: string[] = [];
-
-	lines.push("# Skill Verification Report");
-	lines.push("");
-	lines.push("Generated: " + report.timestamp);
-	lines.push("");
-
-	// Summary table
-	lines.push("## Summary");
-	lines.push("");
-	lines.push("| Status | Count |");
-	lines.push("|--------|-------|");
-	lines.push("| PASS | " + report.passed + " |");
-	lines.push("| FAIL | " + report.failed + " |");
-	lines.push("| Warnings | " + report.warningsOnly + " |");
-	lines.push("| **Total** | **" + report.total + "** |");
-	lines.push("");
-
-	// Skill list by status
-	lines.push("## Skills by Status");
-	lines.push("");
-
-	// Passed skills
-	const passedSkills = report.skills.filter((s) => s.status === "pass");
-	if (passedSkills.length > 0) {
-		lines.push("### Passing Skills");
-		lines.push("");
-		for (const skill of passedSkills) {
-			lines.push("- " + skill.name);
-		}
-		lines.push("");
-	}
-
-	// Failed skills
-	const failedSkills = report.skills.filter((s) => s.status === "fail");
-	if (failedSkills.length > 0) {
-		lines.push("### Failing Skills");
-		lines.push("");
-		for (const skill of failedSkills) {
-			lines.push("- **" + skill.name + "**");
-			for (const error of skill.errors) {
-				lines.push("  - " + error);
-			}
-		}
-		lines.push("");
-	}
-
-	// Warning skills
-	const warningSkills = report.skills.filter((s) => s.status === "warning");
-	if (warningSkills.length > 0) {
-		lines.push("### Skills with Warnings");
-		lines.push("");
-		for (const skill of warningSkills) {
-			lines.push("- **" + skill.name + "**");
-			for (const warning of skill.warnings) {
-				lines.push("  - " + warning);
-			}
-		}
-		lines.push("");
-	}
-
-	// Recommendations
-	lines.push("## Recommendations");
-	lines.push("");
-	if (failedSkills.length > 0) {
-		lines.push("1. Fix " + failedSkills.length + " failing skills to have proper RED/GREEN gates");
-		lines.push("2. Add trigger sections (When to Activate) if missing");
-		lines.push("3. Add anti-patterns sections to prevent common mistakes");
-		lines.push("");
-	}
-	if (warningSkills.length > 0) {
-		lines.push("4. Review " + warningSkills.length + " skills with warnings for improvements");
-	}
-
-	return lines.join("\n");
-}
-
-/**
- * Generate JSON report
- */
-function generateJsonReport(report: Report): string {
-	return JSON.stringify(report, null, 2);
-}
-
-/**
- * Main entry point
- */
-async function main() {
-	console.log("Checking all skills...\n");
-
-	const skillNames = getSkillDirs();
-	console.log("Found " + skillNames.length + " skills\n");
-
-	const report: Report = {
-		total: skillNames.length,
-		passed: 0,
-		failed: 0,
-		warningsOnly: 0,
-		skills: [],
-		timestamp: new Date().toISOString(),
-	};
-
-	for (const skillName of skillNames) {
-		process.stdout.write(".");
-
-		const result = verifySingleSkill(skillName);
-
-		let status: "pass" | "fail" | "warning" = "pass";
-		if (!result.passed && result.errors.length > 0) {
-			status = "fail";
-			report.failed++;
-		} else if (result.warnings.length > 0) {
-			status = "warning";
-			report.warningsOnly++;
-		} else {
-			report.passed++;
-		}
-
-		report.skills.push({
-			name: skillName,
-			path: path.join("skills", skillName, "SKILL.md"),
-			status,
-			warnings: result.warnings,
-			errors: result.errors,
-		});
-	}
-
-	console.log("\n\n");
-
-	// Print summary
-	console.log("=== Skill Verification Summary ===");
-	console.log("Total: " + report.total);
-	console.log("Passed: " + report.passed);
-	console.log("Failed: " + report.failed);
-	console.log("Warnings only: " + report.warningsOnly);
-	console.log("");
-
-	// List failed skills
-	if (report.failed > 0) {
-		console.log("Failing skills:");
-		for (const skill of report.skills.filter((s) => s.status === "fail")) {
-			console.log("  - " + skill.name);
-			for (const error of skill.errors) {
-				console.log("    " + error);
-			}
-		}
-		console.log("");
-	}
-
-	// List skills with warnings
-	if (report.warningsOnly > 0) {
-		console.log("Skills with warnings:");
-		for (const skill of report.skills.filter((s) => s.status === "warning")) {
-			console.log("  - " + skill.name);
-			for (const warning of skill.warnings) {
-				console.log("    " + warning);
-			}
-		}
-		console.log("");
-	}
-
-	// Write reports
-	const reportDir = path.join(process.cwd(), "reports");
-	if (!fs.existsSync(reportDir)) {
-		fs.mkdirSync(reportDir, { recursive: true });
-	}
-
-	const timestamp = new Date().toISOString().replace(/[:.]/g, "-");
-	const mdReportPath = path.join(reportDir, "skill-verification-" + timestamp + ".md");
-	const jsonReportPath = path.join(reportDir, "skill-verification-" + timestamp + ".json");
-	const latestMdPath = path.join(reportDir, "skill-verification-latest.md");
-	const latestJsonPath = path.join(reportDir, "skill-verification-latest.json");
-
-	fs.writeFileSync(mdReportPath, generateMarkdownReport(report));
-	fs.writeFileSync(jsonReportPath, generateJsonReport(report));
-	fs.writeFileSync(latestMdPath, generateMarkdownReport(report));
-	fs.writeFileSync(latestJsonPath, generateJsonReport(report));
-
-	console.log("Reports written to:");
-	console.log("  - " + mdReportPath);
-	console.log("  - " + jsonReportPath);
-	console.log("  - " + latestMdPath + " (latest)");
-	console.log("  - " + latestJsonPath + " (latest)");
-
-	// Exit with appropriate code
-	if (report.failed > 0) {
-		process.exit(1);
-	} else if (report.warningsOnly > 0) {
-		process.exit(2);
-	}
-	process.exit(0);
-}
-
-main().catch((err) => {
-	console.error("Fatal error:", err);
-	process.exit(1);
-});

package/scripts/check-bundle-size.mjs

@@ -1,38 +0,0 @@
-#!/usr/bin/env node
-/**
- * CI gate: enforce a max size budget on dist/index.mjs.
- *
- * The bundle is the cold-start entrypoint (preferred over strip-types
- * loading since v0.9.17). A growing bundle directly increases parse
- * time and npm install cost. This guard FAILS the build if the bundle
- * exceeds the budget, surfacing regressions early.
- *
- * Budget: 3.5 MB (see ROADMAP-2026-07-27 Sprint 6 CI-4).
- *
- * Exits:
- *   0 — bundle is within budget (or absent, deferring to build:bundle)
- *   1 — bundle exceeds budget
- */
-
-import { existsSync, statSync } from "node:fs";
-
-const BUDGET_BYTES = 3.5 * 1024 * 1024; // 3.5 MB
-const distPath = "dist/index.mjs";
-
-if (!existsSync(distPath)) {
-	console.log("[check-bundle-size] dist/index.mjs absent — nothing to check. OK.");
-	process.exit(0);
-}
-
-const sizeBytes = statSync(distPath).size;
-
-if (sizeBytes > BUDGET_BYTES) {
-	console.error(
-		`[check-bundle-size] FAIL: dist/index.mjs (${(sizeBytes / 1024 / 1024).toFixed(2)} MB) exceeds budget (${(BUDGET_BYTES / 1024 / 1024).toFixed(1)} MB).`,
-	);
-	process.exit(1);
-}
-
-console.log(
-	`[check-bundle-size] OK: dist/index.mjs is ${(sizeBytes / 1024 / 1024).toFixed(2)} MB (budget ${(BUDGET_BYTES / 1024 / 1024).toFixed(1)} MB).`,
-);

package/scripts/check-bundle-staleness.mjs

@@ -1,97 +0,0 @@
-#!/usr/bin/env node
-/**
- * CI gate: detect stale dist/index.mjs vs src/.
- *
- * Compares the newest mtime in src/ against dist/index.mjs mtime. If
- * src/ is newer, the bundle is stale (someone edited source without
- * rebuilding). Exits non-zero in that case.
- *
- * Why: in v0.9.17 the entrypoint prefers the bundle (with strip-types
- * fallback). Stale bundles silently run old code — see Phase 5 H2
- * investigation (2026-07-01) for the regression risk.
- *
- * Skips if dist/index.mjs does not exist (the strip-types fallback path
- * is fine and the build:bundle automation will produce it).
- *
- * Exits:
- *   0 — dist is fresh OR absent
- *   1 — dist is stale (run `npm run build:bundle`)
- */
-
-import { execSync } from "node:child_process";
-import { existsSync, statSync } from "node:fs";
-
-const distPath = "dist/index.mjs";
-if (!existsSync(distPath)) {
-	console.log("[check-bundle-staleness] dist/index.mjs absent — strip-types fallback will be used. OK.");
-	process.exit(0);
-}
-
-const distMtimeMs = statSync(distPath).mtimeMs;
-
-// `git ls-files -o -- src` lists untracked files in src/. We want every
-// tracked file's mtime plus the newest untracked one. Cheap heuristic:
-// use `find` style by asking git for the full list, then statSync each.
-let trackedFiles;
-try {
-	trackedFiles = execSync(`git ls-files src`, { encoding: "utf-8" })
-		.split("\n")
-		.filter(Boolean);
-} catch {
-	// Not a git repo (e.g. npm pack test) — skip.
-	console.log("[check-bundle-staleness] not a git repo; skipping staleness check.");
-	process.exit(0);
-}
-
-let newestMtimeMs = 0;
-let newestFile = "";
-for (const f of trackedFiles) {
-	try {
-		const mt = statSync(f).mtimeMs;
-		if (mt > newestMtimeMs) {
-			newestMtimeMs = mt;
-			newestFile = f;
-		}
-	} catch {
-		// File listed but missing on disk — ignore.
-	}
-}
-
-// Add untracked .ts files in src/ (devs editing a new file)
-try {
-	const untracked = execSync(`git ls-files -o --exclude-standard src`, { encoding: "utf-8" })
-		.split("\n")
-		.filter(Boolean);
-	for (const f of untracked) {
-		if (!f.endsWith(".ts")) continue;
-		try {
-			const mt = statSync(f).mtimeMs;
-			if (mt > newestMtimeMs) {
-				newestMtimeMs = mt;
-				newestFile = f;
-			}
-		} catch {
-			// ignore
-		}
-	}
-} catch {
-	// git may fail; ignore
-}
-
-if (newestMtimeMs === 0) {
-	console.log("[check-bundle-staleness] no src/*.ts files found; skipping.");
-	process.exit(0);
-}
-
-if (newestMtimeMs > distMtimeMs) {
-	console.error(
-		`[check-bundle-staleness] FAIL: dist/index.mjs is stale.\n` +
-			`  Newest src file: ${newestFile} (mtime=${newestMtimeMs.toFixed(0)})\n` +
-			`  dist/index.mjs mtime: ${distMtimeMs.toFixed(0)}\n` +
-			`  → run \`npm run build:bundle\` to refresh.`,
-	);
-	process.exit(1);
-}
-
-const ageSec = (distMtimeMs - newestMtimeMs) / 1000;
-console.log(`[check-bundle-staleness] OK: dist/index.mjs is ${ageSec.toFixed(1)}s newer than newest src/ file.`);

package/scripts/check-conflict-markers.mjs

@@ -1,91 +0,0 @@
-#!/usr/bin/env node
-/**
- * CI gate: detect unresolved git merge conflict markers in source.
- *
- * Scans tracked source files for `<<<<<<<`, `=======` (lone), and
- * `>>>>>>>` patterns that indicate a botched merge. Fails CI if any
- * are found so they can't reach the bundle (which esbuild would
- * choke on with a parse error).
- *
- * History: 2026-07-01 CI #28498831579 (Ubuntu) failed because
- * src/runtime/child-pi.ts and src/extension/pi-api.ts shipped with
- * unresolved conflict markers left over from earlier stash/pop cycles
- * during Phase 2-5 implementation. The esbuild transformer in the
- * test runner hit `Expected identifier but found "<<"` at
- * src/runtime/child-pi.ts:1073 — a syntax error caused by the markers.
- *
- * Scope:
- *   - Tracks files in src/, test/, scripts/, workflows/, teams/,
- *     agents/, themes/ — anywhere a conflict would land.
- *   - Excludes test/unit/conflict-detect.test.ts which INTENTIONALLY
- *     contains `<<<<<<< HEAD` etc. as test fixtures (test the
-     detector itself).
- *   - Detects both conflict-style (`<<<<<<<`, `=======`, `>>>>>>>`)
- *     and diff3-style (`|||||||`) markers.
- *
- * Exits:
- *   0 — clean (no markers found, or only allowed locations)
- *   1 — markers found (with file:line listing)
- */
-
-import { execSync } from "node:child_process";
-import { readFileSync } from "node:fs";
-
-const MARKER_PATTERNS = [
-	/^<<<<<<< /m, // ours/incoming/conflict markers
-	/^=======$/m, // conflict separator (lone, not bash =======)
-	/^>>>>>>> /m, // theirs/outgoing markers
-	/^\|\|\|\|\|\|\| /m, // diff3 base marker
-];
-
-const SCAN_PATHS = ["src/", "test/", "scripts/", "workflows/", "teams/", "agents/", "themes/"];
-const EXCLUDE_FILES = new Set([
-	// Intentionally contains conflict markers as test fixtures.
-	"test/unit/conflict-detect.test.ts",
-]);
-
-function listTrackedFiles() {
-	try {
-		const all = execSync("git ls-files", { encoding: "utf-8" });
-		return all
-			.split("\n")
-			.filter(Boolean)
-			.filter((f) => SCAN_PATHS.some((p) => f.startsWith(p) || f === p.replace(/\/$/, "")))
-			.filter((f) => !EXCLUDE_FILES.has(f));
-	} catch {
-		console.error("[conflict-markers] not a git repo; skipping check.");
-		process.exit(0);
-	}
-}
-
-const files = listTrackedFiles();
-const offenders = [];
-
-for (const file of files) {
-	let content;
-	try {
-		content = readFileSync(file, "utf-8");
-	} catch {
-		// File listed but unreadable — skip silently.
-		continue;
-	}
-	const lines = content.split(/\r?\n/);
-	for (let i = 0; i < lines.length; i += 1) {
-		const line = lines[i];
-		if (MARKER_PATTERNS.some((re) => re.test(line))) {
-			offenders.push({ file, line: i + 1, text: line });
-		}
-	}
-}
-
-if (offenders.length === 0) {
-	console.log(`[conflict-markers] OK: scanned ${files.length} files, no markers found.`);
-	process.exit(0);
-}
-
-console.error(`[conflict-markers] FAIL: ${offenders.length} conflict marker(s) found.`);
-for (const o of offenders) {
-	console.error(`  ${o.file}:${o.line}: ${o.text.trim().slice(0, 80)}`);
-}
-console.error("\nResolve with: git checkout --ours|--theirs or git add ... after manual edit.");
-process.exit(1);

package/scripts/check-lazy-imports.mjs

@@ -1,28 +0,0 @@
-import { execSync } from "node:child_process";
-import { readFileSync } from "node:fs";
-
-const out = execSync(
-	`git grep -nE "await import\\(" -- "src/**/*.ts"`,
-	{ encoding: "utf-8" },
-);
-
-const bad = [];
-const fileCache = new Map();
-
-for (const line of out.split("\n").filter(Boolean)) {
-	if (line.includes("// LAZY:")) continue;
-	const m = line.match(/^([^:]+):(\d+):/);
-	if (!m) continue;
-	const [, file, lineNum] = m;
-	if (!fileCache.has(file)) fileCache.set(file, readFileSync(file, "utf-8").split(/\r?\n/));
-	const lines = fileCache.get(file);
-	const prevLine = lines[Number(lineNum) - 2] ?? "";
-	if (!prevLine.includes("// LAZY:")) bad.push(line);
-}
-
-if (bad.length > 0) {
-	console.error("Dynamic imports without `// LAZY:` marker:\n" + bad.join("\n"));
-	process.exit(1);
-}
-
-console.log("All dynamic imports have `// LAZY:` marker.");

package/scripts/dev-runner.mjs

@@ -1,106 +0,0 @@
-#!/usr/bin/env node
-/**
- * Dev runner — spawn `watch:bundle` and `test:watch` in parallel.
- *
- * Replaces the `npm-run-all --parallel watch:bundle test:watch` form that
- * would require an additional dev-dependency. Uses Node's stdlib
- * `child_process.spawn` so we keep the dev tooling dependency-free.
- *
- * Behavior:
- *   - Spawns both child processes with stdio: inherit (the parent's TTY
- *     sees both streams interleaved).
- *   - Forwards Ctrl+C (SIGINT) and SIGTERM to both children so they shut
- *     down cleanly when the user aborts.
- *   - Exits with a non-zero code if EITHER child exits with a non-zero
- *     status (mimics `npm-run-all --parallel` semantics).
- *
- * Usage:
- *   npm run dev
- *   node scripts/dev-runner.mjs
- */
-
-import { spawn } from "node:child_process";
-import process from "node:process";
-
-const tasks = [
-	{ name: "watch:bundle", script: "watch:bundle" },
-	{ name: "test:watch", script: "test:watch" },
-];
-
-/**
- * @typedef {{name: string, child: import("node:child_process").ChildProcess, exited: boolean, code: number|null, signal: NodeJS.Signals|null}} Task
- */
-
-/** @type {Task[]} */
-const children = tasks.map((t) => ({
-	name: t.name,
-	child: spawnNpm(t.script),
-	exited: false,
-	code: null,
-	signal: null,
-}));
-
-let shuttingDown = false;
-let exitCode = 0;
-
-function spawnNpm(script) {
-	return spawn("npm", ["run", "--silent", script], {
-		stdio: "inherit",
-		env: process.env,
-	});
-}
-
-function maybeExit() {
-	if (!shuttingDown) return;
-	if (children.some((c) => !c.exited)) return;
-	process.exit(exitCode);
-}
-
-function shutdown(reason) {
-	if (shuttingDown) return;
-	shuttingDown = true;
-	process.stderr.write(`\n[dev-runner] received ${reason}, forwarding to children...\n`);
-	for (const c of children) {
-		if (c.exited) continue;
-		try {
-			c.child.kill("SIGTERM");
-		} catch {
-			/* already dead */
-		}
-	}
-	// Hard-kill grace window: if children ignore SIGTERM for 3s, send SIGKILL.
-	setTimeout(() => {
-		for (const c of children) {
-			if (c.exited) continue;
-			try {
-				c.child.kill("SIGKILL");
-			} catch {
-				/* already dead */
-			}
-		}
-	}, 3000).unref();
-	maybeExit();
-}
-
-for (const c of children) {
-	c.child.on("exit", (code, signal) => {
-		c.exited = true;
-		c.code = code;
-		c.signal = signal;
-		const note = signal ? `signal=${signal}` : `code=${code}`;
-		process.stderr.write(`[dev-runner] ${c.name} exited (${note})\n`);
-		// Non-zero is only meaningful when the child exited via code (not signal).
-		if (signal === null && code !== 0) exitCode = exitCode || (code ?? 1);
-		maybeExit();
-	});
-
-	c.child.on("error", (err) => {
-		process.stderr.write(`[dev-runner] failed to spawn ${c.name}: ${err.message}\n`);
-		c.exited = true;
-		exitCode = exitCode || 1;
-		maybeExit();
-	});
-}
-
-process.on("SIGINT", () => shutdown("SIGINT"));
-process.on("SIGTERM", () => shutdown("SIGTERM"));