@bigking67/pi-67 0.10.0

package/src/commands/update.mjs

@@ -0,0 +1,138 @@
+import { parseCommandOptions } from "../lib/args.mjs";
+import { buildUpdatePlan } from "../lib/update-plan.mjs";
+import { printJson, section, keyValue, pass, warn, info } from "../lib/output.mjs";
+import { runDistroScript } from "../lib/distro-scripts.mjs";
+import { isWindows } from "../lib/platform.mjs";
+import { runCommand } from "../lib/shell-runner.mjs";
+import { writeState } from "../lib/state-store.mjs";
+import { beginUpdateLifecycle } from "../lib/update-safety.mjs";
+
+export async function updateCommand(ctx, argv) {
+  const { options } = parseCommandOptions(argv, {
+    bools: [
+      "check",
+      "repair",
+      "dry-run",
+      "json",
+      "no-remote",
+      "no-npm",
+      "allow-dirty",
+      "include-pi",
+      "include-external",
+      "all",
+      "yes",
+      "strict-shared-skills",
+    ],
+  });
+  const dryRun = ctx.dryRun || options.dryRun;
+  const json = ctx.json || options.json;
+  if (options.check) {
+    const plan = buildUpdatePlan(ctx, {
+      noRemote: ctx.noRemote || options.noRemote,
+      strictSharedSkills: options.strictSharedSkills,
+    });
+    if (json) {
+      printJson(plan);
+      return;
+    }
+    printPlan(plan);
+    return;
+  }
+
+  const plan = buildUpdatePlan(ctx, {
+    noRemote: ctx.noRemote || options.noRemote,
+    strictSharedSkills: options.strictSharedSkills,
+  });
+  const lifecycle = beginUpdateLifecycle(ctx, {
+    operation: options.repair ? "repair" : "update",
+    dryRun,
+    plan,
+  });
+  if (!dryRun && lifecycle.backedUp.length > 0) {
+    info(`Preserved runtime backup: ${lifecycle.backupDir}`);
+  }
+
+  const args = isWindows()
+    ? buildWindowsUpdateArgs(ctx, options, dryRun)
+    : buildBashUpdateArgs(ctx, options, dryRun);
+  try {
+    runDistroScript(ctx, { sh: "pi67-update.sh", ps1: "pi67-update.ps1" }, args, { dryRun: false });
+    if (!dryRun) writeState(ctx, options.repair ? "repair" : "update");
+  } finally {
+    lifecycle.release();
+  }
+
+  if (options.includePi || options.all) {
+    if (ctx.yes || options.yes) {
+      runCommand("pi", ["update", "--all"], { dryRun });
+    } else {
+      warn("Pi upstream update skipped; rerun with --include-pi --yes if you really want `pi update --all`.");
+    }
+  }
+  if (options.includeExternal || options.all) {
+    warn("External repo updates are explicit; run `pi-67 external update browser67` or `pi-67 external update design-craft`.");
+  }
+}
+
+function buildBashUpdateArgs(ctx, options, dryRun) {
+  const args = ["--agent-dir", ctx.agentDir, "--repo-root", ctx.repoRoot, "--skills-dir", ctx.skillsDir];
+  if (dryRun) args.push("--dry-run");
+  if (options.repair) args.push("--force-npm");
+  if (options.noNpm) args.push("--no-npm");
+  if (options.allowDirty) args.push("--allow-dirty");
+  if (options.strictSharedSkills) args.push("--strict-shared-skills");
+  return args;
+}
+
+function buildWindowsUpdateArgs(ctx, options, dryRun) {
+  const args = ["-AgentDir", ctx.agentDir, "-RepoRoot", ctx.repoRoot, "-SkillsDir", ctx.skillsDir];
+  if (dryRun) args.push("-DryRun");
+  if (options.repair) args.push("-ForceNpm");
+  if (options.noNpm) args.push("-NoNpm");
+  if (options.allowDirty) args.push("-AllowDirty");
+  if (options.strictSharedSkills) args.push("-StrictSharedSkills");
+  return args;
+}
+
+function printPlan(plan) {
+  section("pi-67 update check");
+  keyValue("Agent dir", plan.paths.agentDir);
+  keyValue("Repo root", plan.paths.repoRoot);
+  keyValue("Manager", `${plan.manager.package}@${plan.manager.version}`);
+  if (plan.manager.registry?.skipped) {
+    keyValue("Manager latest", "skipped");
+  } else if (plan.manager.registry?.ok) {
+    keyValue("Manager latest", `${plan.manager.registry.latestVersion}${plan.manager.registry.outdated ? " update available" : ""}`);
+  } else if (plan.manager.registry?.message) {
+    keyValue("Manager latest", `unknown (${plan.manager.registry.message})`);
+  }
+  keyValue("Distro", plan.distro.version || "unknown");
+  keyValue("Git", plan.git?.isRepo ? `${plan.git.branchLine || plan.git.branch || ""} ${plan.git.commit || ""}` : "not a git repo");
+  keyValue("Dirty", plan.git?.dirty ? "yes" : "no");
+  keyValue("Remote", plan.remote?.skipped ? "skipped" : (plan.remote?.commit || plan.remote?.message || "unknown"));
+  keyValue("Provider", plan.settings.defaultProvider || "unset");
+  keyValue("Model", plan.settings.defaultModel || "unset");
+  keyValue("Theme", plan.settings.theme || "unset");
+  keyValue("Theme installed", plan.settings.themeInstalled ? "yes" : "no");
+  keyValue("Shared skills", `${plan.skills.identical} ok, ${plan.skills.missing} missing, ${plan.skills.conflicts} conflicts`);
+  for (const repo of plan.external) {
+    keyValue(`External ${repo.name}`, repo.exists ? `${repo.git?.dirty ? "dirty" : "clean"} ${repo.git?.commit || ""}` : "missing");
+  }
+  if (plan.actions?.length > 0) {
+    section("Planned safe actions");
+    for (const action of plan.actions) {
+      info(`${action.id}: ${action.operation}; writes=${action.writes.join(", ")}; preserves=${action.preserves.join(", ")}`);
+    }
+  }
+  if (plan.blocked?.length > 0) {
+    section("Blocked actions");
+    for (const item of plan.blocked) warn(`${item.id}: ${item.reason}`);
+  }
+  if (plan.warnings?.length > 0) {
+    section("Warnings");
+    for (const item of plan.warnings) warn(item);
+  }
+  section("Recommendations");
+  for (const item of plan.recommendations) info(item);
+  if (!plan.git?.dirty) pass("update check completed without local dirty blocker");
+}

package/src/commands/version.mjs

@@ -0,0 +1,51 @@
+import { captureCommand } from "../lib/shell-runner.mjs";
+import { gitStatus } from "../lib/git.mjs";
+import { currentTheme } from "../lib/theme-policy.mjs";
+import { keyValue, printJson } from "../lib/output.mjs";
+import { platformName } from "../lib/platform.mjs";
+import { readCliPackageJson, readTextIfExists } from "../lib/paths.mjs";
+import path from "node:path";
+
+export async function versionCommand(ctx, argv) {
+  const json = ctx.json || argv.includes("--json");
+  const pkg = readCliPackageJson();
+  const git = gitStatus(ctx.repoRoot);
+  const pi = captureCommand("pi", ["--version"]);
+  const data = {
+    schema: "pi67.version.v1",
+    manager: {
+      package: pkg.name,
+      version: pkg.version,
+    },
+    distro: {
+      version: readTextIfExists(path.join(ctx.repoRoot, "VERSION")).trim(),
+      commit: git.commit || "",
+      branch: git.branch || "",
+      dirty: Boolean(git.dirty),
+    },
+    runtime: {
+      node: process.version,
+      platform: platformName(),
+      pi: pi.ok ? pi.stdout.trim() || pi.stderr.trim() : "",
+    },
+    paths: {
+      agentDir: ctx.agentDir,
+      repoRoot: ctx.repoRoot,
+      skillsDir: ctx.skillsDir,
+      packagesDir: ctx.packagesDir,
+    },
+    theme: currentTheme(ctx),
+  };
+  if (json) {
+    printJson(data);
+    return;
+  }
+  keyValue("manager", `${data.manager.package}@${data.manager.version}`);
+  keyValue("pi-67 distro", data.distro.version || "unknown");
+  keyValue("git", `${data.distro.commit || "unknown"}${data.distro.dirty ? " dirty" : ""}`);
+  keyValue("pi", data.runtime.pi || "not found");
+  keyValue("node", data.runtime.node);
+  keyValue("platform", data.runtime.platform);
+  keyValue("agentDir", data.paths.agentDir);
+  keyValue("theme", data.theme || "unset");
+}

package/src/commands/xtalpi.mjs

@@ -0,0 +1,69 @@
+import { parseCommandOptions } from "../lib/args.mjs";
+import { runCommand } from "../lib/shell-runner.mjs";
+import { scriptPath } from "../lib/paths.mjs";
+import { isWindows, findPowerShell } from "../lib/platform.mjs";
+import { CliError } from "../lib/output.mjs";
+
+export async function xtalpiCommand(ctx, argv) {
+  const [sub = "health", ...rest] = argv;
+  if (sub === "health") return health(ctx, rest);
+  if (sub === "smoke") return smoke(ctx, rest);
+  if (sub === "capability") return capability(ctx, rest);
+  throw new CliError(`unknown xtalpi command: ${sub}`, 2);
+}
+
+function health(ctx, argv) {
+  const { options } = parseCommandOptions(argv, {
+    strings: ["model", "provider", "timeout-ms", "attempts"],
+    bools: ["dry-run"],
+  });
+  const args = [scriptPath(ctx, "pi67-xtalpi-provider-health.mjs"), "--agent-dir", ctx.agentDir];
+  if (options.provider) args.push("--provider", options.provider);
+  if (options.model) args.push("--model", options.model);
+  if (options.timeoutMs) args.push("--timeout-ms", options.timeoutMs);
+  if (options.attempts) args.push("--attempts", options.attempts);
+  runCommand("node", args, { cwd: ctx.repoRoot, dryRun: ctx.dryRun || options.dryRun });
+}
+
+function smoke(ctx, argv) {
+  const { options } = parseCommandOptions(argv, {
+    strings: ["case", "profile"],
+    bools: ["quick", "extension-low-risk", "extension-expanded", "self-test", "dry-run"],
+  });
+  if (isWindows()) {
+    const pwsh = findPowerShell();
+    if (!pwsh) throw new CliError("PowerShell executable not found");
+    const args = ["-NoProfile", "-ExecutionPolicy", "Bypass", "-File", scriptPath(ctx, "pi67-xtalpi-pi-tools-smoke.ps1")];
+    if (options.selfTest) args.push("-SelfTest");
+    else if (options.case) args.push("-Case", options.case);
+    else args.push("-Profile", profileFromOptions(options));
+    runCommand(pwsh, args, { cwd: ctx.repoRoot, dryRun: ctx.dryRun || options.dryRun });
+    return;
+  }
+  const args = [scriptPath(ctx, "pi67-xtalpi-pi-tools-smoke.sh")];
+  if (options.selfTest) args.push("--self-test");
+  else if (options.case) args.push("--case", options.case);
+  else args.push("--profile", profileFromOptions(options));
+  runCommand("bash", args, { cwd: ctx.repoRoot, dryRun: ctx.dryRun || options.dryRun });
+}
+
+function capability(ctx, argv) {
+  const { options } = parseCommandOptions(argv, {
+    strings: ["model", "provider", "timeout-ms"],
+    bools: ["dry-run", "self-test"],
+  });
+  const args = [scriptPath(ctx, "pi67-xtalpi-provider-capability-probe.mjs"), "--agent-dir", ctx.agentDir];
+  if (options.selfTest) args.push("--self-test");
+  if (options.provider) args.push("--provider", options.provider);
+  if (options.model) args.push("--model", options.model);
+  if (options.timeoutMs) args.push("--timeout-ms", options.timeoutMs);
+  runCommand("node", args, { cwd: ctx.repoRoot, dryRun: ctx.dryRun || options.dryRun });
+}
+
+function profileFromOptions(options) {
+  if (options.profile) return options.profile;
+  if (options.extensionLowRisk) return "extension-low-risk";
+  if (options.extensionExpanded) return "extension-expanded";
+  if (options.quick) return "quick";
+  return "quick";
+}

package/src/data/distro-manifest.json

@@ -0,0 +1,85 @@
+{
+  "schema": "pi67.distro-manifest.base.v1",
+  "ownership": {
+    "pi67Managed": "Installed, repaired, and smoked by pi-67; updates must preserve local user runtime choices.",
+    "userManaged": "Reported only; pi-67 must not remove, downgrade, or overwrite without an explicit user command."
+  },
+  "commands": {
+    "update": "pi-67 update",
+    "repair": "pi-67 update --repair",
+    "alwaysFreshRepair": "npx -y @bigking67/pi-67@latest update --repair",
+    "upstreamPiExtensions": "pi update --extensions"
+  },
+  "runtimeFiles": {
+    "preserve": [
+      "settings.json",
+      "models.json",
+      "auth.json",
+      "mcp.json",
+      "image-gen.json",
+      "settings.json.theme"
+    ],
+    "templateOnly": [
+      "models.example.json",
+      "auth.example.json",
+      "mcp.example.json",
+      "image-gen.example.json"
+    ]
+  },
+  "theme": {
+    "packageName": "@victor-software-house/pi-curated-themes",
+    "preserveSetting": "settings.json:theme",
+    "selection": {
+      "file": "settings.json",
+      "path": "theme"
+    },
+    "legacyPreserveFiles": [
+      "settings.json.theme"
+    ],
+    "policy": "install-theme-package-only-never-select-theme-on-update",
+    "explicitCommands": [
+      "pi-67 themes current",
+      "pi-67 themes list",
+      "pi-67 themes set <name>"
+    ]
+  },
+  "sharedSkills": {
+    "sourceDir": "shared-skills",
+    "activeDirDefault": "~/.agents/skills",
+    "policy": "copy-by-default-preserve-different-existing-skills-unless-strict"
+  },
+  "localExtensions": [
+    {
+      "name": "xtalpi-pi-tools",
+      "path": "extensions/xtalpi-pi-tools",
+      "required": true,
+      "policy": "pi-local-json-action-provider"
+    },
+    {
+      "name": "pi-rules-loader",
+      "path": "extensions/pi-rules-loader",
+      "required": true,
+      "policy": "local-rules-index-extension"
+    }
+  ],
+  "runtimePackageSpecs": [
+    {
+      "spec": "git:github.com/justhil/pi-image-gen",
+      "packageName": "pi-image-gen",
+      "owner": "pi67-managed",
+      "policy": "runtime-package-preserve-user-config"
+    },
+    {
+      "spec": "git:github.com/arpagon/pi-rewind",
+      "packageName": "pi-rewind",
+      "owner": "pi67-managed",
+      "policy": "runtime-package-preserve-user-config"
+    }
+  ],
+  "externalReposPolicy": {
+    "defaultRoot": "~/.agents/packages",
+    "dirtyRepo": "preserve-and-block-update",
+    "detachedRepo": "preserve-and-block-update",
+    "explicitCommand": "pi-67 external update <name>"
+  }
+}

package/src/data/extension-registry.json

@@ -0,0 +1,147 @@
+{
+  "schema": "pi67.extension-registry.v1",
+  "governance": {
+    "defaultUserConfigPolicy": "preserve-and-merge-only",
+    "forbiddenUpdateBehavior": [
+      "overwrite-user-config",
+      "select-theme-during-update",
+      "overwrite-different-shared-skill",
+      "update-dirty-external-repo",
+      "fallback-to-upstream-pi-extension-update"
+    ],
+    "configPatchModes": [
+      "merge-preserve",
+      "template-only",
+      "report-only"
+    ]
+  },
+  "extensions": [
+    {
+      "id": "xtalpi-pi-tools",
+      "kind": "local-provider",
+      "owner": "pi67-managed",
+      "path": "extensions/xtalpi-pi-tools",
+      "installStrategy": "package-owned-clean-overwrite",
+      "updateStrategy": "update-owned-code-preserve-user-config",
+      "repairStrategy": "local-json-action-protocol-repair",
+      "configPatches": [
+        {
+          "file": "models.json",
+          "path": "providers.xtalpi-pi-tools",
+          "mode": "merge-preserve"
+        },
+        {
+          "file": "settings.json",
+          "path": "defaultProvider",
+          "mode": "merge-preserve"
+        }
+      ],
+      "smoke": [
+        "pi-67 xtalpi health",
+        "pi-67 xtalpi smoke --quick"
+      ],
+      "notes": "xtalpi is treated as plain chat-completions transport; Pi local adapter owns JSON action protocol, repair, validation, and smoke gates."
+    },
+    {
+      "id": "pi-rules-loader",
+      "kind": "local-extension",
+      "owner": "pi67-managed",
+      "path": "extensions/pi-rules-loader",
+      "installStrategy": "package-owned-clean-overwrite",
+      "updateStrategy": "update-owned-code-preserve-user-config",
+      "repairStrategy": "rules-index-refresh",
+      "configPatches": [],
+      "smoke": [
+        "pi-67 manifest"
+      ],
+      "notes": "Rules index extension is package-owned; user AGENTS/settings remain user-owned."
+    },
+    {
+      "id": "pi-curated-themes",
+      "kind": "theme-package",
+      "owner": "pi67-managed",
+      "packageName": "@victor-software-house/pi-curated-themes",
+      "installStrategy": "install-theme-package",
+      "updateStrategy": "install-theme-package-only-never-select-theme-on-update",
+      "repairStrategy": "verify-theme-assets-without-changing-selection",
+      "configPatches": [
+        {
+          "file": "settings.json",
+          "path": "theme",
+          "mode": "report-only"
+        }
+      ],
+      "explicitCommands": [
+        "pi-67 themes current",
+        "pi-67 themes list",
+        "pi-67 themes doctor",
+        "pi-67 themes set <name>"
+      ],
+      "smoke": [
+        "pi-67 themes current",
+        "pi-67 themes doctor"
+      ],
+      "notes": "Update may install theme assets but must never change the selected theme."
+    },
+    {
+      "id": "shared-skills",
+      "kind": "skill-pack",
+      "owner": "pi67-managed",
+      "path": "shared-skills",
+      "target": "~/.agents/skills",
+      "installStrategy": "copy-missing-skills",
+      "updateStrategy": "copy-by-default-preserve-different-existing-skills-unless-strict",
+      "repairStrategy": "inventory-and-copy-missing-only",
+      "configPatches": [],
+      "explicitCommands": [
+        "pi-67 skills inventory",
+        "pi-67 skills sync",
+        "pi-67 skills migrate"
+      ],
+      "smoke": [
+        "pi-67 skills inventory"
+      ],
+      "notes": "Different existing shared skills are user-owned by default; strict mode blocks instead of overwriting."
+    },
+    {
+      "id": "browser67",
+      "kind": "external-repo",
+      "owner": "external",
+      "repoUrl": "https://github.com/bigKING67/browser67.git",
+      "target": "~/.agents/packages/browser67",
+      "installStrategy": "explicit-external-install",
+      "updateStrategy": "preserve-and-block-update-when-dirty",
+      "repairStrategy": "doctor-only-when-dirty",
+      "configPatches": [],
+      "explicitCommands": [
+        "pi-67 external list",
+        "pi-67 external doctor",
+        "pi-67 external update browser67"
+      ],
+      "smoke": [
+        "pi-67 external list"
+      ],
+      "notes": "External repo updates are explicit and must not reset, clean, or overwrite dirty worktrees."
+    },
+    {
+      "id": "design-craft",
+      "kind": "external-repo",
+      "owner": "external",
+      "repoUrl": "https://github.com/bigKING67/design-craft",
+      "target": "~/.agents/packages/design-craft",
+      "installStrategy": "explicit-external-install",
+      "updateStrategy": "preserve-and-block-update-when-dirty",
+      "repairStrategy": "doctor-only-when-dirty",
+      "configPatches": [],
+      "explicitCommands": [
+        "pi-67 external list",
+        "pi-67 external doctor",
+        "pi-67 external update design-craft"
+      ],
+      "smoke": [
+        "pi-67 external list"
+      ],
+      "notes": "Frontend skill repo is external; pi-67 reports dirty state and blocks destructive updates."
+    }
+  ]
+}

package/src/lib/args.mjs

@@ -0,0 +1,87 @@
+import { CliError } from "./output.mjs";
+
+const VALUE_GLOBALS = new Set(["agent-dir", "repo-root", "skills-dir", "packages-dir"]);
+const BOOL_GLOBALS = new Set(["json", "dry-run", "yes", "help", "no-remote"]);
+
+export function splitGlobalArgs(argv) {
+  const globals = {};
+  const rest = [];
+  for (let index = 0; index < argv.length; index += 1) {
+    const arg = argv[index];
+    if (arg === "-h") {
+      globals.help = true;
+      continue;
+    }
+    if (!arg.startsWith("--")) {
+      rest.push(arg);
+      continue;
+    }
+
+    const raw = arg.slice(2);
+    const eq = raw.indexOf("=");
+    const name = eq === -1 ? raw : raw.slice(0, eq);
+    const inlineValue = eq === -1 ? undefined : raw.slice(eq + 1);
+
+    if (VALUE_GLOBALS.has(name)) {
+      const value = inlineValue ?? argv[index + 1];
+      if (!value || value.startsWith("--")) {
+        throw new CliError(`--${name} requires a value`, 2);
+      }
+      globals[toCamel(name)] = value;
+      if (inlineValue === undefined) index += 1;
+      continue;
+    }
+    if (BOOL_GLOBALS.has(name)) {
+      globals[toCamel(name)] = true;
+      continue;
+    }
+    rest.push(arg);
+  }
+  return { globals, rest };
+}
+
+export function parseCommandOptions(argv, spec = {}) {
+  const strings = new Set(spec.strings || []);
+  const bools = new Set(spec.bools || []);
+  const positionals = [];
+  const options = {};
+
+  for (let index = 0; index < argv.length; index += 1) {
+    const arg = argv[index];
+    if (arg === "-h") {
+      options.help = true;
+      continue;
+    }
+    if (!arg.startsWith("--")) {
+      positionals.push(arg);
+      continue;
+    }
+
+    const raw = arg.slice(2);
+    const eq = raw.indexOf("=");
+    const name = eq === -1 ? raw : raw.slice(0, eq);
+    const key = toCamel(name);
+    const inlineValue = eq === -1 ? undefined : raw.slice(eq + 1);
+
+    if (strings.has(name)) {
+      const value = inlineValue ?? argv[index + 1];
+      if (!value || value.startsWith("--")) {
+        throw new CliError(`--${name} requires a value`, 2);
+      }
+      options[key] = value;
+      if (inlineValue === undefined) index += 1;
+      continue;
+    }
+    if (bools.has(name)) {
+      options[key] = true;
+      continue;
+    }
+    throw new CliError(`unknown option: --${name}`, 2);
+  }
+
+  return { options, positionals };
+}
+
+export function toCamel(name) {
+  return name.replace(/-([a-z])/g, (_, char) => char.toUpperCase());
+}

package/src/lib/config-json.mjs

@@ -0,0 +1,20 @@
+import fs from "node:fs";
+import path from "node:path";
+
+export function readJsonFile(file) {
+  const raw = fs.readFileSync(file, "utf8").replace(/^\uFEFF/, "");
+  return JSON.parse(raw);
+}
+
+export function readJsonFileIfExists(file) {
+  if (!fs.existsSync(file)) return null;
+  return readJsonFile(file);
+}
+
+export function writeJsonAtomic(file, value) {
+  const dir = path.dirname(file);
+  fs.mkdirSync(dir, { recursive: true });
+  const tmp = path.join(dir, `.${path.basename(file)}.${process.pid}.tmp`);
+  fs.writeFileSync(tmp, `${JSON.stringify(value, null, 2)}\n`, "utf8");
+  fs.renameSync(tmp, file);
+}