okstra 0.173.0 → 0.175.1

package/src/lib/python-helper.mjs

@@ -1,142 +0,0 @@
-import { spawn } from "node:child_process";
-import { existsSync } from "node:fs";
-import { join, resolve as resolvePath } from "node:path";
-import { fileURLToPath } from "node:url";
-import { buildPythonpath, resolvePaths } from "./paths.mjs";
-
-export function resolveInstalledScript(paths, scriptName) {
-  // Prefer the installed copy under ~/.okstra/bin (what production users run);
-  // fall back to the in-repo source when invoked from a checkout that has not
-  // been installed (dev / CI).
-  const installed = join(paths.bin, scriptName);
-  if (existsSync(installed)) return installed;
-  const repoRoot = fileURLToPath(new URL("../..", import.meta.url));
-  const dev = resolvePath(repoRoot, "scripts", scriptName);
-  return existsSync(dev) ? dev : null;
-}
-
-export function resolveInstalledValidator(paths, validatorName) {
-  // `okstra install` lands the validators tree at ~/.okstra/lib/validators;
-  // a checkout that has not been installed still has the in-repo source.
-  const installed = join(paths.home, "lib", "validators", validatorName);
-  if (existsSync(installed)) return installed;
-  const repoRoot = fileURLToPath(new URL("../..", import.meta.url));
-  const dev = resolvePath(repoRoot, "validators", validatorName);
-  return existsSync(dev) ? dev : null;
-}
-
-async function spawnPythonEntry(entry, args, paths) {
-  return await new Promise((resolve) => {
-    const child = spawn("python3", [entry, ...args], {
-      stdio: "inherit",
-      env: { ...process.env, PYTHONPATH: buildPythonpath(paths) },
-    });
-    child.on("error", (err) => {
-      process.stderr.write(`error: failed to spawn python3: ${err.message}\n`);
-      resolve(1);
-    });
-    child.on("close", (code) => resolve(typeof code === "number" ? code : 1));
-  });
-}
-
-// Thin spawn shim shared by every `okstra <cmd>` subcommand that fronts a
-// `scripts/okstra-*.py` entry point. Centralizing it keeps PYTHONPATH wiring
-// and installed/dev resolution in one place so skills call `okstra <cmd>`
-// instead of emitting `python3 "$HOME/..."` (which breaks `Bash(okstra:*)`
-// permission matching and prompts on every call).
-export async function runInstalledScript({ scriptName, args, usage, emptyArgsCode = 2 }) {
-  if (args.length === 0) {
-    process.stdout.write(usage);
-    return emptyArgsCode;
-  }
-  // Only a bare `--help` / `-h` prints the wrapper usage. A `--help` that
-  // follows a subcommand (e.g. `error-log append-observed --help`) must reach
-  // the python helper so its own per-subcommand help shows through.
-  if (args.length === 1 && (args[0] === "--help" || args[0] === "-h")) {
-    process.stdout.write(usage);
-    return 0;
-  }
-  const paths = await resolvePaths();
-  const entry = resolveInstalledScript(paths, scriptName);
-  if (!entry) {
-    process.stderr.write(
-      `error: ${scriptName} not found — run 'okstra install' (or 'okstra ensure-installed') first\n`,
-    );
-    return 1;
-  }
-  return await spawnPythonEntry(entry, args, paths);
-}
-
-export async function runInstalledValidator({ validatorName, args }) {
-  const paths = await resolvePaths();
-  const entry = resolveInstalledValidator(paths, validatorName);
-  if (!entry) {
-    process.stderr.write(
-      `error: ${validatorName} not found — run 'okstra install' (or 'okstra ensure-installed') first\n`,
-    );
-    return 1;
-  }
-  return await spawnPythonEntry(entry, args, paths);
-}
-
-export async function runPythonSnippet({ script, args = [], extraEnv = {} }) {
-  const paths = await resolvePaths();
-  return new Promise((resolve) => {
-    const child = spawn("python3", ["-c", script, ...args], {
-      stdio: ["ignore", "pipe", "pipe"],
-      env: { ...process.env, PYTHONPATH: buildPythonpath(paths), ...extraEnv },
-    });
-    // Collect chunks and join once: repeated `s += chunk` is O(n^2) for the
-    // large rendered prompts some capture-mode callers stream through here.
-    const outChunks = [];
-    const errChunks = [];
-    child.stdout.on("data", (b) => outChunks.push(b));
-    child.stderr.on("data", (b) => errChunks.push(b));
-    child.on("error", (err) =>
-      resolve({ code: -1, stdout: Buffer.concat(outChunks).toString(), stderr: err.message }),
-    );
-    child.on("close", (code) =>
-      resolve({
-        code,
-        stdout: Buffer.concat(outChunks).toString(),
-        stderr: Buffer.concat(errChunks).toString(),
-      }),
-    );
-  });
-}
-
-export async function runPythonModule({ module, args = [], extraEnv = {}, stdio = "inherit-stdout" }) {
-  const paths = await resolvePaths();
-  return new Promise((resolve) => {
-    const child = spawn("python3", ["-m", module, ...args], {
-      stdio: stdio === "capture" ? ["ignore", "pipe", "pipe"] : ["ignore", "inherit", "inherit"],
-      env: { ...process.env, PYTHONPATH: buildPythonpath(paths), ...extraEnv },
-    });
-    // Collect chunks and join once: repeated `s += chunk` is O(n^2) for the
-    // large rendered prompts some capture-mode callers stream through here.
-    const outChunks = [];
-    const errChunks = [];
-    if (stdio === "capture") {
-      child.stdout.on("data", (b) => outChunks.push(b));
-      child.stderr.on("data", (b) => errChunks.push(b));
-    }
-    child.on("error", (err) =>
-      resolve({ code: -1, stdout: Buffer.concat(outChunks).toString(), stderr: err.message }),
-    );
-    // A signal-killed child reports code === null; normalize to a non-zero exit
-    // here so every consumer inherits correct failure semantics from one place.
-    child.on("close", (code) =>
-      resolve({
-        code: typeof code === "number" ? code : 1,
-        stdout: Buffer.concat(outChunks).toString(),
-        stderr: Buffer.concat(errChunks).toString(),
-      }),
-    );
-  });
-}
-
-export function emitJsonError({ stage, reason, extra = {} }) {
-  process.stdout.write(
-    JSON.stringify({ ok: false, stage, reason, ...extra }, null, 2) + "\n",
-  );
-}

package/src/lib/runtime-manifest.mjs

@@ -1,70 +0,0 @@
-export const RUNTIMES_MANIFEST_REL = "installed-runtimes.json";
-export const RUNTIME_MANIFEST_SCHEMA_VERSION = 2;
-export function runtimeListForSelection(runtime, resolution = null) {
-  if (runtime === "all") return [...(resolution?.catalogHostIds ?? [])];
-  if (runtime === "auto") {
-    const resolved = resolution?.resolvedRuntime;
-    return resolved && resolved !== "all" ? [resolved] : [];
-  }
-  return [runtime];
-}
-
-function includesClaudeAssets(resolution) {
-  return resolution?.installTargets?.includes("claude") === true;
-}
-
-function defaultResolution(runtime) {
-  if (runtime === "auto") return null;
-  return {
-    requestedRuntime: runtime,
-    resolvedRuntime: runtime,
-    launchMode: null,
-    installTargets: [],
-    host: "explicit",
-    confidence: "high",
-    reason: `Explicit runtime '${runtime}' requested.`,
-    fallbackFrom: null,
-    warnings: [],
-  };
-}
-
-export function buildRuntimeManifest(
-  runtime,
-  installedAt = new Date().toISOString(),
-  resolution = null,
-  installedAssets = null,
-) {
-  const runtimeResolution = resolution ?? defaultResolution(runtime);
-  const includesClaude = includesClaudeAssets(runtimeResolution);
-  const defaultAssets = {
-    sharedRuntime: true,
-    claudeSkills: includesClaude,
-    claudeAgents: includesClaude,
-    binWrappers: true,
-    templates: true,
-    schemas: true,
-    validators: true,
-  };
-  return {
-    schemaVersion: RUNTIME_MANIFEST_SCHEMA_VERSION,
-    installedAt,
-    installRequest: runtime,
-    runtimeResolution,
-    installedRuntimes: runtimeListForSelection(runtime, runtimeResolution),
-    installedAssets: { ...defaultAssets, ...(installedAssets ?? {}) },
-  };
-}
-
-export function installedRuntimesFromManifest(data) {
-  if (Array.isArray(data?.installedRuntimes)) return data.installedRuntimes;
-  if (Array.isArray(data?.runtimes)) return data.runtimes;
-  if (typeof data?.runtime === "string" && data.runtime) return runtimeListForSelection(data.runtime);
-  return [];
-}
-
-export function runtimeManifestIncludesClaudeAssets(data) {
-  if (data?.schemaVersion === RUNTIME_MANIFEST_SCHEMA_VERSION) {
-    return data?.installedAssets?.claudeSkills === true || data?.installedAssets?.claudeAgents === true;
-  }
-  return data?.assets?.claudeSkills === true || data?.assets?.claudeAgents === true;
-}

package/src/lib/skill-catalog.mjs

@@ -1,53 +0,0 @@
-// Single source of truth for which okstra skills are user-facing and which
-// former or renamed skill directories must be pruned from agent skill homes on
-// (re)install. The Claude plugin manifest (.claude-plugin/plugin.json) and the
-// installer both derive their skill list from here, so the public surface
-// cannot drift between discovery and install.
-
-export const USER_SKILL_NAMES = Object.freeze([
-  "okstra-setup",
-  "okstra-brief-gen",
-  "okstra-run",
-  "okstra-manager",
-  "okstra-memory",
-  "okstra-inspect",
-  "okstra-rollup",
-  "okstra-usage",
-  "okstra-schedule-gen",
-  "okstra-container-build",
-  "okstra-pr-gen",
-  "okstra-user-response",
-  "okstra-code-review",
-]);
-
-// Names okstra used to install as skills before they were renamed or moved
-// into the runtime resource tree. Install prunes these exact directories from
-// each target skill home so upgrades do not leave stale slash commands behind.
-// Exact-name only — never wildcard.
-export const OBSOLETE_SKILL_NAMES = Object.freeze([
-  "okstra",
-  "okstra-context-loader",
-  "okstra-team-contract",
-  "okstra-convergence",
-  "okstra-report-writer",
-  "okstra-coding-preflight",
-  "okstra-container",
-  "okstra-brief",
-  "okstra-schedule",
-  // Pre-merge status/history/logs/report/time skills, since folded into
-  // okstra-inspect. Absent here, upgrades left them stranded in agent skill
-  // homes and their pre-.okstra-migration bodies kept triggering.
-  "okstra-status",
-  "okstra-history",
-  "okstra-logs",
-  "okstra-report-finder",
-  "okstra-time-summary",
-]);
-
-export function userSkillNames() {
-  return [...USER_SKILL_NAMES];
-}
-
-export function obsoleteSkillNames() {
-  return [...OBSOLETE_SKILL_NAMES];
-}

package/src/lib/version.mjs

@@ -1,20 +0,0 @@
-import { readFile } from "node:fs/promises";
-import { fileURLToPath } from "node:url";
-import { dirname, join } from "node:path";
-
-const here = dirname(fileURLToPath(import.meta.url));
-const packageJsonPath = join(here, "..", "..", "package.json");
-
-let cachedVersion = null;
-
-export async function getPackageVersion() {
-  if (cachedVersion) return cachedVersion;
-  const raw = await readFile(packageJsonPath, "utf8");
-  const { version } = JSON.parse(raw);
-  cachedVersion = version;
-  return version;
-}
-
-export function getPackageRoot() {
-  return join(here, "..", "..");
-}