portable-agent-layer 0.85.9 → 0.85.11

package/src/cli/skill.ts

@@ -76,11 +76,6 @@ function isPlainSegment(part: string): boolean {
   );
 }
 
-/** Playwright tools ship compiled as .mjs and need Node; everything else is Bun. */
-function runtimeFor(file: string): string {
-  return file.endsWith(".mjs") ? "node" : "bun";
-}
-
 function toolFileName(tool: string): string {
   return new RegExp(/\.[a-z]+$/).test(tool) ? tool : `${tool}.ts`;
 }
@@ -96,7 +91,7 @@ function runSkillTool(skill: string, tool: string, toolArgs: string[]): number {
     log.error(`No tool '${file}' in skill '${skill}' — looked in ${path}`);
     return 1;
   }
-  const { status } = spawnSync(runtimeFor(file), [path, ...toolArgs], {
+  const { status } = spawnSync("bun", [path, ...toolArgs], {
     stdio: "inherit",
   });
   return status ?? 1;

package/src/hooks/lib/log.ts

@@ -79,9 +79,10 @@ function rotateIfNeeded(path: string): void {
   }
 }
 
-export interface HookHealth {
-  totalErrors: number;
-  lastError: string | null;
+export interface HookErrorGroup {
+  source: string;
+  count: number;
+  last: string;
 }
 
 const DAY_MS = 24 * 60 * 60 * 1000;
@@ -107,18 +108,30 @@ function loggedAt(line: string): number {
   return match ? Date.parse(`${match[1]}T${match[2]}Z`) : 0;
 }
 
-export function recentHookErrors(now: number = Date.now()): HookHealth {
+function groupBySource(errors: string[]): HookErrorGroup[] {
+  const groups = new Map<string, HookErrorGroup>();
+  for (const error of errors) {
+    const source = /^([^:]+):/.exec(error)?.[1] ?? "unknown";
+    const group = groups.get(source) ?? { source, count: 0, last: "" };
+    group.count++;
+    group.last = error
+      .slice(source.length + 1)
+      .trim()
+      .slice(0, 120);
+    groups.set(source, group);
+  }
+  return [...groups.values()].sort((a, b) => b.count - a.count);
+}
+
+export function recentHookErrors(now: number = Date.now()): HookErrorGroup[] {
   try {
     const recent = readAllLogs()
       .split("\n")
-      .filter((line) => line.includes("] ERROR ") && loggedAt(line) > now - DAY_MS);
-    const last = recent.at(-1);
-    return {
-      totalErrors: recent.length,
-      lastError: last ? last.replace(/^\[.*?\] ERROR /, "").slice(0, 120) : null,
-    };
+      .filter((line) => line.includes("] ERROR ") && loggedAt(line) > now - DAY_MS)
+      .map((line) => line.replace(/^\[.*?\] ERROR /, ""));
+    return groupBySource(recent);
   } catch {
-    return { totalErrors: 0, lastError: null };
+    return [];
   }
 }
 

package/src/hooks/lib/projects.ts

@@ -572,22 +572,6 @@ export function auditBindings(
   return issues;
 }
 
-/**
- * Every issue names the command that fixes it. PAL runs inside agents, where a
- * hook cannot ask a question — so a suggestion is always a command the user can
- * choose to run, never something applied on their behalf.
- */
-export function describeBindingIssue(issue: BindingIssue): string {
-  const fix = (name: string) => `run 'project set-path ${name} <path>'`;
-  if (issue.kind === "unlocatable")
-    return `${issue.project} — not checked out here (${fix(issue.project)})`;
-  // "points at" rather than "bound to": the path may equally have come from a
-  // legacy record's own field, which is not a binding.
-  if (issue.kind === "missing")
-    return `${issue.project} — points at ${issue.path}, which does not exist here (${fix(issue.project)})`;
-  return `${issue.projects.join(" and ")} — both point at ${issue.path}; rebind whichever is wrong (${fix(issue.projects[0])})`;
-}
-
 export type BindingProposal = {
   state: "unbound";
   confidence: "strong" | "weak";

package/assets/skills/consulting-report/tools/generate-pdf.mjs

@@ -1,226 +0,0 @@
-import { spawnSync } from "node:child_process";
-import { createReadStream, constants as fsConstants, realpathSync } from "node:fs";
-import { access, readFile, stat, unlink, writeFile } from "node:fs/promises";
-import { createServer } from "node:http";
-import { extname, join, resolve } from "node:path";
-import { fileURLToPath, pathToFileURL } from "node:url";
-import { PDFDocument } from "pdf-lib";
-import { chromium } from "playwright";
-const COLOR = {
-  navy: "#0f172a",
-  blue: "#1d4ed8",
-  red: "#dc2626"
-};
-async function exists(p) {
-  try {
-    await access(p, fsConstants.F_OK);
-    return true;
-  } catch {
-    return false;
-  }
-}
-function escapeHtml(s) {
-  return s.replaceAll("&", "&amp;").replaceAll("<", "&lt;").replaceAll(">", "&gt;").replaceAll('"', "&quot;");
-}
-function slugify(s) {
-  return s.toLowerCase().replace(/[^a-z0-9]+/g, "-").replace(/^-|-$/g, "");
-}
-async function logoDataUri(outDir, publicPath) {
-  if (!publicPath)
-    return null;
-  const filePath = join(outDir, publicPath);
-  if (!await exists(filePath))
-    return null;
-  const buf = await readFile(filePath);
-  const ext = extname(filePath).toLowerCase();
-  let mime = "image/jpeg";
-  if (ext === ".svg") {
-    mime = "image/svg+xml";
-  } else if (ext === ".png") {
-    mime = "image/png";
-  }
-  return `data:${mime};base64,${buf.toString("base64")}`;
-}
-async function loadMeta(reportDir) {
-  const dataPath = join(reportDir, "lib", "report-data.ts");
-  if (!await exists(dataPath)) {
-    throw new Error(`lib/report-data.ts not found at ${dataPath}`);
-  }
-  const mod = await import(pathToFileURL(dataPath).href);
-  if (!mod.reportData) {
-    throw new Error(`lib/report-data.ts must export a named 'reportData' constant`);
-  }
-  return mod.reportData;
-}
-function buildNext(reportDir) {
-  const result = spawnSync("bun", ["run", "build"], {
-    cwd: reportDir,
-    stdio: "inherit",
-    shell: true
-  });
-  if (result.status !== 0) {
-    throw new Error(`next build failed (exit ${result.status})`);
-  }
-}
-function serveStatic(rootDir) {
-  const MIME = {
-    ".html": "text/html; charset=utf-8",
-    ".css": "text/css; charset=utf-8",
-    ".js": "application/javascript; charset=utf-8",
-    ".json": "application/json; charset=utf-8",
-    ".woff": "font/woff",
-    ".woff2": "font/woff2",
-    ".svg": "image/svg+xml",
-    ".png": "image/png",
-    ".jpg": "image/jpeg",
-    ".jpeg": "image/jpeg",
-    ".webp": "image/webp",
-    ".ico": "image/x-icon"
-  };
-  return new Promise((res) => {
-    const server = createServer((req, response) => {
-      const urlPath = decodeURIComponent((req.url ?? "/").split("?")[0]);
-      let filePath = join(rootDir, urlPath === "/" ? "/index.html" : urlPath);
-      if (filePath.endsWith("/"))
-        filePath = join(filePath, "index.html");
-      const ext = extname(filePath).toLowerCase();
-      response.setHeader("Content-Type", MIME[ext] ?? "application/octet-stream");
-      const stream = createReadStream(filePath);
-      stream.on("error", () => {
-        response.statusCode = 404;
-        response.end();
-      });
-      stream.pipe(response);
-    });
-    server.listen(0, "127.0.0.1", () => {
-      const addr = server.address();
-      const port = typeof addr === "object" && addr ? addr.port : 0;
-      res({ server, url: `http://127.0.0.1:${port}/` });
-    });
-  });
-}
-async function renderPdf(htmlPath, pdfPath, meta) {
-  const outDir = resolve(htmlPath, "..");
-  const { server, url } = await serveStatic(outDir);
-  const browser = await chromium.launch();
-  try {
-    const page = await browser.newPage();
-    await page.goto(url, { waitUntil: "networkidle" });
-    await page.evaluate(async () => {
-      await document.fonts?.ready;
-      const imgs = Array.from(document.querySelectorAll("img"));
-      await Promise.all(imgs.map((img) => img.complete ? Promise.resolve() : new Promise((res) => {
-        img.onload = () => res(null);
-        img.onerror = () => res(null);
-        setTimeout(() => res(null), 5000);
-      })));
-    });
-    const [clientUri, consultancyUri] = await Promise.all([
-      logoDataUri(outDir, meta.clientLogoSrc),
-      logoDataUri(outDir, meta.consultancyLogoSrc)
-    ]);
-    const clientSlot = clientUri ? `<img src="${clientUri}" style="height:18px; width:auto; object-fit:contain; display:block;">` : `<span style="font-weight:600; color:${COLOR.navy}; letter-spacing:0.05em;">${escapeHtml(meta.clientName.toUpperCase())}</span>`;
-    const consultancySlot = consultancyUri ? `<img src="${consultancyUri}" style="height:14px; width:auto; object-fit:contain; display:block;">` : `<span style="color:${COLOR.navy};">${escapeHtml(meta.consultancyName)}</span>`;
-    const header = `
-<div style="width:100%; font-family:Inter,'Helvetica Neue',Arial,sans-serif; font-size:7.5pt; padding:0 0.7in; display:flex; justify-content:space-between; align-items:center;">
-  ${clientSlot}
-  <span style="color:#94a3b8;">${escapeHtml(meta.reportTitle)}</span>
-</div>`;
-    const footer = `
-<div style="width:100%; font-family:Inter,'Helvetica Neue',Arial,sans-serif; font-size:7.5pt; padding:0 0.7in; display:flex; justify-content:space-between; align-items:center;">
-  ${consultancySlot}
-  <span style="color:${COLOR.navy};"><span class="pageNumber"></span></span>
-</div>`;
-    const margin = { top: "0.7in", right: "0.7in", bottom: "0.7in", left: "0.7in" };
-    const tmpCover = `${pdfPath}.cover.tmp.pdf`;
-    const tmpBody = `${pdfPath}.body.tmp.pdf`;
-    await Promise.all([
-      page.pdf({
-        path: tmpCover,
-        format: "A4",
-        pageRanges: "1",
-        printBackground: true,
-        displayHeaderFooter: false,
-        margin: { top: "0", right: "0", bottom: "0", left: "0" },
-        preferCSSPageSize: false
-      }),
-      page.pdf({
-        path: tmpBody,
-        format: "A4",
-        pageRanges: "2-",
-        printBackground: true,
-        displayHeaderFooter: true,
-        headerTemplate: header,
-        footerTemplate: footer,
-        margin,
-        preferCSSPageSize: false
-      })
-    ]);
-    const [coverBytes, bodyBytes] = await Promise.all([
-      readFile(tmpCover),
-      readFile(tmpBody)
-    ]);
-    const [coverDoc, bodyDoc] = await Promise.all([
-      PDFDocument.load(coverBytes),
-      PDFDocument.load(bodyBytes)
-    ]);
-    const [coverPage] = await bodyDoc.copyPages(coverDoc, [0]);
-    bodyDoc.insertPage(0, coverPage);
-    await writeFile(pdfPath, await bodyDoc.save());
-    await Promise.all([unlink(tmpCover), unlink(tmpBody)]);
-  } finally {
-    await browser.close();
-    server.close();
-  }
-}
-async function generate(opts) {
-  const dir = resolve(opts.reportDir);
-  if (!await exists(join(dir, "package.json"))) {
-    throw new Error(`not a scaffolded report (missing package.json): ${dir}`);
-  }
-  if (!opts.skipBuild) {
-    buildNext(dir);
-  }
-  const htmlPath = join(dir, "out", "index.html");
-  if (!await exists(htmlPath)) {
-    throw new Error(`static export missing: ${htmlPath} — run without --skip-build`);
-  }
-  const meta = await loadMeta(dir);
-  const slug = `${slugify(meta.clientName)}-${slugify(meta.reportTitle)}-${slugify(new Date().toISOString().slice(0, 10))}`;
-  const pdfPath = opts.pdfOut ? resolve(opts.pdfOut) : join(dir, `${slug}.pdf`);
-  await renderPdf(htmlPath, pdfPath, meta);
-  return { htmlPath, pdfPath };
-}
-function parseArgs(argv) {
-  if (argv.length === 0) {
-    throw new Error("usage: generate-pdf.ts <report-dir> [--pdf <out>] [--skip-build]");
-  }
-  const opts = { reportDir: argv[0] };
-  for (let i = 1;i < argv.length; i++) {
-    if (argv[i] === "--pdf")
-      opts.pdfOut = argv[++i];
-    else if (argv[i] === "--html")
-      opts.htmlOut = argv[++i];
-    else if (argv[i] === "--skip-build")
-      opts.skipBuild = true;
-  }
-  return opts;
-}
-async function run(argv = process.argv.slice(2)) {
-  const opts = parseArgs(argv);
-  const { htmlPath, pdfPath } = await generate(opts);
-  const [htmlStat, pdfStat] = await Promise.all([stat(htmlPath), stat(pdfPath)]);
-  console.log(`HTML: ${htmlPath} (${(htmlStat.size / 1024).toFixed(1)} KB)`);
-  console.log(`PDF:  ${pdfPath} (${(pdfStat.size / 1024).toFixed(1)} KB)`);
-}
-function realResolve(p) {
-  try {
-    return realpathSync(resolve(p));
-  } catch {
-    return resolve(p);
-  }
-}
-const isMain = process.argv[1] && realResolve(process.argv[1]) === realResolve(fileURLToPath(import.meta.url));
-if (isMain) {
-  await run();
-}

package/assets/skills/create-pdf/tools/md-to-html-pdf.mjs

@@ -1,103 +0,0 @@
-import { readFile, stat, writeFile } from "node:fs/promises";
-import { basename, dirname, extname, resolve } from "node:path";
-import { pathToFileURL } from "node:url";
-import { marked } from "marked";
-import { chromium } from "playwright";
-const args = process.argv.slice(2);
-if (args.length === 0) {
-  console.error("usage: md-to-html-pdf.ts <input.md> [--html <out>] [--pdf <out>] [--margin <css>] [--header <html|file>] [--footer <html|file>]");
-  process.exit(1);
-}
-const input = resolve(args[0]);
-let htmlOut = "";
-let pdfOut = "";
-let margin = "25mm";
-let headerArg = "";
-let footerArg = "";
-for (let i = 1;i < args.length; i++) {
-  if (args[i] === "--html")
-    htmlOut = resolve(args[++i]);
-  else if (args[i] === "--pdf")
-    pdfOut = resolve(args[++i]);
-  else if (args[i] === "--margin")
-    margin = args[++i];
-  else if (args[i] === "--header")
-    headerArg = args[++i];
-  else if (args[i] === "--footer")
-    footerArg = args[++i];
-}
-const stem = basename(input, extname(input));
-const dir = dirname(input);
-htmlOut ||= resolve(dir, `${stem}.html`);
-pdfOut ||= resolve(dir, `${stem}.pdf`);
-async function resolveTemplate(value) {
-  if (!value)
-    return "";
-  try {
-    return await readFile(resolve(value), "utf8");
-  } catch {
-    return value;
-  }
-}
-const headerTemplate = await resolveTemplate(headerArg);
-const footerTemplate = await resolveTemplate(footerArg);
-const displayHeaderFooter = Boolean(headerTemplate || footerTemplate);
-const md = await readFile(input, "utf8");
-marked.setOptions({ gfm: true, breaks: false });
-const body = await marked.parse(md);
-const css = `
-html { -webkit-print-color-adjust: exact; print-color-adjust: exact; }
-body {
-  font-family: -apple-system, BlinkMacSystemFont, "Segoe UI", Helvetica, Arial, sans-serif;
-  font-size: 11px; line-height: 1.6; color: #1a1a1a; margin: 0;
-}
-h1 { font-size: 22px; border-bottom: 2px solid #333; padding-bottom: 8px; margin-top: 0; }
-h2 { font-size: 17px; margin-top: 1.6em; }
-h3 { font-size: 14px; }
-h1, h2, h3, h4 { page-break-after: avoid; }
-p, li { orphans: 3; widows: 3; }
-hr { border: none; border-top: 1px solid #ccc; margin: 20px 0; }
-a { color: #0366d6; text-decoration: none; }
-blockquote { border-left: 3px solid #666; padding-left: 12px; color: #444; margin: 12px 0; }
-code { background: #f4f4f4; padding: 1px 4px; border-radius: 3px; font-size: 10px; }
-pre { background: #f4f4f4; padding: 10px; border-radius: 4px; overflow-x: auto; }
-pre code { background: transparent; padding: 0; }
-table { border-collapse: collapse; width: 100%; margin: 12px 0; page-break-inside: avoid; }
-th, td { border: 1px solid #ccc; padding: 6px 10px; text-align: left; font-size: 10px; vertical-align: top; }
-th { background: #f0f0f0; }
-tr { page-break-inside: avoid; }
-ul, ol { padding-left: 1.4em; }
-`;
-const html = `<!doctype html>
-<html>
-<head>
-<meta charset="utf-8">
-<title>${stem}</title>
-<style>${css}</style>
-</head>
-<body>
-${body}
-</body>
-</html>
-`;
-await writeFile(htmlOut, html, "utf8");
-const browser = await chromium.launch();
-try {
-  const page = await browser.newPage();
-  await page.goto(pathToFileURL(htmlOut).href, { waitUntil: "networkidle" });
-  await page.pdf({
-    path: pdfOut,
-    format: "A4",
-    margin: { top: margin, right: margin, bottom: margin, left: margin },
-    printBackground: true,
-    preferCSSPageSize: false,
-    displayHeaderFooter,
-    headerTemplate: headerTemplate || "<span></span>",
-    footerTemplate: footerTemplate || "<span></span>"
-  });
-} finally {
-  await browser.close();
-}
-const [htmlStat, pdfStat] = await Promise.all([stat(htmlOut), stat(pdfOut)]);
-console.log(`HTML: ${htmlOut} (${(htmlStat.size / 1024).toFixed(1)} KB)`);
-console.log(`PDF:  ${pdfOut} (${(pdfStat.size / 1024).toFixed(1)} KB)`);

package/src/cli/doctor-tools.ts

@@ -1,133 +0,0 @@
-/**
- * The doctor's git and gh checks. Neither is required: without git, project
- * matching by remote and the update checks skip; without gh, nothing does yet.
- */
-
-import { readFileSync } from "node:fs";
-import { findBinaryOnPath } from "../hooks/lib/which";
-
-export interface DoctorLine {
-  level: "ok" | "warn" | "info";
-  text: string;
-}
-
-type LinuxFamily = "debian" | "fedora" | "arch" | "alpine" | "suse";
-
-interface Host {
-  platform: NodeJS.Platform;
-  osRelease: string;
-  timeoutMs: number;
-}
-
-const GH_LINUX_DOCS = "https://github.com/cli/cli/blob/trunk/docs/install_linux.md";
-
-function osReleaseIds(osRelease: string): string[] {
-  const field = (key: string) =>
-    osRelease.match(new RegExp(String.raw`^${key}="?([^"\n]*)"?$`, "m"))?.[1] ?? "";
-  return `${field("ID")} ${field("ID_LIKE")}`.split(/\s+/).filter(Boolean);
-}
-
-function linuxFamily(osRelease: string): LinuxFamily | null {
-  const ids = osReleaseIds(osRelease);
-  const families: [LinuxFamily, string[]][] = [
-    ["debian", ["debian", "ubuntu"]],
-    ["fedora", ["fedora", "rhel", "centos"]],
-    ["arch", ["arch"]],
-    ["alpine", ["alpine"]],
-    ["suse", ["suse", "opensuse"]],
-  ];
-  return families.find(([, names]) => names.some((n) => ids.includes(n)))?.[0] ?? null;
-}
-
-const GIT_ON_LINUX: Record<LinuxFamily, string> = {
-  debian: "sudo apt install git",
-  fedora: "sudo dnf install git",
-  arch: "sudo pacman -S git",
-  alpine: "sudo apk add git",
-  suse: "sudo zypper install git",
-};
-
-const GH_ON_LINUX: Partial<Record<LinuxFamily, string>> = {
-  fedora: "sudo dnf install gh",
-  arch: "sudo pacman -S github-cli",
-};
-
-export function gitInstallHint(platform: NodeJS.Platform, osRelease: string): string {
-  if (platform === "win32") return "`winget install Git.Git`";
-  if (platform === "darwin") return "`brew install git` or `xcode-select --install`";
-  const family = linuxFamily(osRelease);
-  return family ? `\`${GIT_ON_LINUX[family]}\`` : "install git with your package manager";
-}
-
-export function ghInstallHint(platform: NodeJS.Platform, osRelease: string): string {
-  if (platform === "win32") return "`winget install GitHub.cli`";
-  if (platform === "darwin") return "`brew install gh`";
-  const family = linuxFamily(osRelease);
-  const command = family && GH_ON_LINUX[family];
-  return command ? `\`${command}\`` : `see ${GH_LINUX_DOCS}`;
-}
-
-function run(binary: string, args: string[], timeoutMs: number) {
-  const result = Bun.spawnSync([binary, ...args], {
-    stdout: "pipe",
-    stderr: "ignore",
-    timeout: timeoutMs,
-    windowsHide: true,
-  });
-  return {
-    answered: !result.exitedDueToTimeout,
-    ok: result.success,
-    firstLine: result.stdout.toString().trim().split("\n")[0] ?? "",
-  };
-}
-
-function installed(name: string, timeoutMs: number) {
-  const binary = findBinaryOnPath(name);
-  if (!binary) return null;
-  const version = run(binary, ["--version"], timeoutMs);
-  return version.ok ? { binary, version: version.firstLine } : null;
-}
-
-function gitLine(host: Host): DoctorLine {
-  const git = installed("git", host.timeoutMs);
-  if (git) return { level: "ok", text: git.version };
-  return {
-    level: "warn",
-    text: `git — not found; project matching by remote, the update check and auto-update skip without it. ${gitInstallHint(host.platform, host.osRelease)}`,
-  };
-}
-
-function ghLine(host: Host): DoctorLine {
-  const gh = installed("gh", host.timeoutMs);
-  if (!gh)
-    return {
-      level: "info",
-      text: `gh — not installed (optional; lets PAL see PR, CI and release state — ${ghInstallHint(host.platform, host.osRelease)})`,
-    };
-  const auth = run(gh.binary, ["auth", "status"], host.timeoutMs);
-  if (!auth.answered)
-    return { level: "info", text: `${gh.version} — could not check the login in time` };
-  if (!auth.ok)
-    return {
-      level: "warn",
-      text: `${gh.version} — not logged in; run \`gh auth login\``,
-    };
-  return { level: "ok", text: `${gh.version}, logged in` };
-}
-
-function readOsRelease(): string {
-  try {
-    return readFileSync("/etc/os-release", "utf-8");
-  } catch {
-    return "";
-  }
-}
-
-export function versionControlLines(host: Partial<Host> = {}): DoctorLine[] {
-  const resolved: Host = {
-    platform: host.platform ?? process.platform,
-    osRelease: host.osRelease ?? (process.platform === "linux" ? readOsRelease() : ""),
-    timeoutMs: host.timeoutMs ?? 5000,
-  };
-  return [gitLine(resolved), ghLine(resolved)];
-}