@klhapp/skillmux 1.12.0 → 1.13.0

package/src/commands/context.ts

@@ -1,104 +0,0 @@
-import {
-  addContext,
-  getCurrentContext,
-  listContexts,
-  removeContext,
-  useContext,
-  type ResolvedContext,
-} from "../context";
-import {
-  emitSuccess,
-  renderTable,
-  renderContextBanner,
-  unknownSubcommandError,
-} from "../output";
-
-export async function handleContextCommand(
-  sub: string,
-  args: string[],
-  ctx: { context: ResolvedContext; isJson: boolean },
-) {
-  if (sub === "list") {
-    const contexts = await listContexts();
-    emitSuccess({ isJson: ctx.isJson, context: ctx.context }, contexts, () => {
-      renderContextBanner(ctx.context);
-      renderTable(
-        [
-          { key: "name", header: "NAME" },
-          { key: "server", header: "SERVER" },
-          { key: "token_env", header: "TOKEN_ENV" },
-          { key: "isDefault", header: "DEFAULT" },
-        ],
-        contexts.map((c) => ({
-          ...c,
-          token_env: c.token_env ?? "-",
-          isDefault: c.isDefault ? "*" : "",
-        })),
-      );
-    });
-    return;
-  }
-
-  if (sub === "current") {
-    const current = await getCurrentContext();
-    emitSuccess({ isJson: ctx.isJson, context: ctx.context }, current, () => {
-      renderContextBanner(ctx.context);
-      console.log(`Current context: ${current.name} (${current.server})`);
-    });
-    return;
-  }
-
-  if (sub === "add") {
-    const name = args[0];
-    let server: string | undefined;
-    let tokenEnv: string | undefined;
-    for (let i = 1; i < args.length; i++) {
-      if (args[i] === "--server") server = args[++i];
-      else if (args[i] === "--token-env") tokenEnv = args[++i];
-    }
-    if (!name || !server) {
-      throw new Error(
-        "usage: skillmux context add <name> --server <url> [--token-env <env_name>]",
-      );
-    }
-    await addContext(name, { server, token_env: tokenEnv });
-    emitSuccess(
-      { isJson: ctx.isJson, context: ctx.context },
-      { name, server, token_env: tokenEnv },
-      () => {
-        console.log(`Added context "${name}" -> ${server}`);
-      },
-    );
-    return;
-  }
-
-  if (sub === "use") {
-    const name = args[0];
-    if (!name) throw new Error("usage: skillmux context use <name>");
-    await useContext(name);
-    emitSuccess(
-      { isJson: ctx.isJson, context: ctx.context },
-      { default_context: name },
-      () => {
-        console.log(`Switched default context to "${name}"`);
-      },
-    );
-    return;
-  }
-
-  if (sub === "remove") {
-    const name = args[0];
-    if (!name) throw new Error("usage: skillmux context remove <name>");
-    await removeContext(name);
-    emitSuccess(
-      { isJson: ctx.isJson, context: ctx.context },
-      { removed: name },
-      () => {
-        console.log(`Removed context "${name}"`);
-      },
-    );
-    return;
-  }
-
-  throw unknownSubcommandError("context", sub, ["add", "list", "current", "use", "remove"]);
-}

package/src/commands/core.ts

@@ -1,56 +0,0 @@
-import { expandHome } from "../config";
-import { pinCore, unpinCore, validateManifest, writeManifestAtomic } from "../manifest";
-import { emitSuccess, unknownSubcommandError } from "../output";
-import { confirmIfNeeded, loadManifestContext } from "./shared";
-export async function runCore(
-  subCommand: string,
-  args: string[],
-  options: { isJson: boolean; dryRun: boolean },
-): Promise<void> {
-  if (subCommand !== "pin" && subCommand !== "unpin") {
-    throw unknownSubcommandError("core", subCommand, ["pin", "unpin"]);
-  }
-  const skillIds = args.filter((arg) => !arg.startsWith("-"));
-  if (skillIds.length === 0) {
-    throw new Error(`usage: skillmux core ${subCommand} <skill_id>... --yes`);
-  }
-  const yes = args.includes("--yes");
-  const { config, vaultPath, manifestPath, manifest } =
-    await loadManifestContext();
-  let updated = manifest;
-  for (const skillId of skillIds) {
-    updated =
-      subCommand === "pin"
-        ? pinCore(updated, skillId)
-        : unpinCore(updated, skillId);
-  }
-  validateManifest(
-    updated,
-    vaultPath,
-    config.local_vault_paths.map(expandHome),
-  );
-  if (options.dryRun) {
-    emitSuccess(
-      { isJson: options.isJson },
-      { subcommand: subCommand, skill_ids: skillIds },
-      () =>
-        console.log(`${subCommand}: [core] ${skillIds.join(", ")} (dry-run)`),
-    );
-    return;
-  }
-  if (
-    !(await confirmIfNeeded({
-      confirmed: yes,
-      isJson: options.isJson,
-      prompt: `${subCommand} ${skillIds.join(", ")} in [core]?`,
-      nonInteractiveError: `skillmux core ${subCommand} requires --yes when run non-interactively`,
-    }))
-  )
-    return;
-  writeManifestAtomic(manifestPath, updated);
-  emitSuccess(
-    { isJson: options.isJson },
-    { subcommand: subCommand, skill_ids: skillIds },
-    () => console.log(`${subCommand}: [core] ${skillIds.join(", ")}`),
-  );
-}

package/src/commands/doctor.ts

@@ -1,97 +0,0 @@
-import { resolveConfigPath } from "../config";
-import { diagnose } from "../doctor";
-import { getEffectiveConfig } from "../config-service";
-import type { ContextAdapter } from "../adapters";
-import type { ResolvedContext } from "../context";
-import { isGlobalFlag, isGlobalFlagWithValue } from "../global-flags";
-import {
-  emitSuccess,
-  green,
-  red,
-  renderContextBanner,
-} from "../output";
-
-/**
- * doctor takes no options of its own, but it still has to reject unknown ones
- * rather than silently ignoring them the way every other command does.
- */
-export function parseDoctorArgs(args: readonly string[]): void {
-  for (let i = 0; i < args.length; i++) {
-    const option = args[i];
-    if (isGlobalFlag(option, "--json", "--allow-insecure", "--verbose")) {
-      // handled globally by main(); recognized here so it isn't rejected
-    } else if (isGlobalFlagWithValue(option)) {
-      // handled globally by main()'s resolveContext(); skip its value too
-      i++;
-    } else {
-      throw new Error(`unknown doctor option: ${option}`);
-    }
-  }
-}
-
-export async function runDoctor(options: {
-  isJson: boolean;
-  context: ResolvedContext;
-  adapter: ContextAdapter;
-  args?: readonly string[];
-}): Promise<void> {
-  parseDoctorArgs(options.args ?? []);
-  if (options.context.type === "remote") {
-    const context = options.context;
-    const [status, caps] = await Promise.all([
-      options.adapter.configStatus(),
-      options.adapter.getCapabilities(),
-    ]);
-    const remoteReport = {
-      target: context.name || context.server,
-      server: context.server,
-      version: status.version,
-      deployment_runtime: status.deployment_runtime,
-      image_variant: status.image_variant ?? null,
-      runtime: status.runtime,
-      readiness: status.readiness,
-      active_revision: status.active_revision,
-      capabilities: caps,
-      restart_required_keys: status.restart_required_keys,
-      last_reload_error: status.last_reload_error,
-    };
-    emitSuccess({ isJson: options.isJson, target: options.context }, remoteReport, () => {
-      renderContextBanner(options.context);
-      console.log(`server: ${remoteReport.server}`);
-      console.log(`version: ${remoteReport.version}`);
-      console.log(`deployment runtime: ${remoteReport.deployment_runtime}`);
-      console.log(`image variant: ${remoteReport.image_variant ?? "none"}`);
-      console.log(`runtime: ${remoteReport.runtime}`);
-      console.log(`readiness: ${remoteReport.readiness.status} (${remoteReport.readiness.capability})`);
-      console.log(`active revision: ${remoteReport.active_revision}`);
-      console.log(`persistence: ${caps.persistence}`);
-      console.log(`config read: ${caps.config_read}`);
-      console.log(`config write: ${caps.config_write}`);
-      if (status.last_reload_error) {
-        console.log(`last reload error: ${status.last_reload_error}`);
-      }
-      if (status.restart_required_keys.length > 0) {
-        console.log(`restart required for: ${status.restart_required_keys.join(", ")}`);
-      }
-    });
-    return;
-  }
-
-  const effective = await getEffectiveConfig(resolveConfigPath());
-  const report = await diagnose(effective.effective, process.env, effective.sources);
-  emitSuccess({ isJson: options.isJson }, report, () => {
-    console.log(`version: ${report.version}`);
-    console.log(`runtime: ${report.runtime}`);
-    console.log(`image variant: ${report.image_variant ?? "none"}`);
-    console.log(`vault path: ${report.vault_path}`);
-    console.log(`state directory: ${report.state_dir}`);
-    console.log(`inference mode: ${report.mode}`);
-    console.log(`routing capability: ${report.capability}`);
-    console.log(`retrieval capability: ${report.retrieval_capability}`);
-    for (const check of report.checks)
-      console.log(
-        `${check.ok ? green("ok") : red("fail")}: ${check.name} - ${check.detail}`,
-      );
-  });
-  if (report.checks.some((check) => !check.ok)) process.exitCode = 1;
-}

package/src/commands/eval.ts

@@ -1,88 +0,0 @@
-import { existsSync, readFileSync, writeFileSync } from "node:fs";
-import { join } from "node:path";
-import { expandHome, loadConfig } from "../config";
-import { excludeExistingCases, parseEvalCases } from "../eval";
-import { emitSuccess, warn } from "../output";
-import { parseSince } from "../stats";
-import { confirmIfNeeded } from "./shared";
-import type { ContextAdapter } from "../adapters";
-import { isGlobalFlag, isGlobalFlagWithValue } from "../global-flags";
-
-export async function runEvalPromote(
-  args: string[],
-  options: { isJson: boolean; dryRun: boolean; adapter: ContextAdapter },
-): Promise<void> {
-  let since: string | undefined;
-  let out: string | undefined;
-  let target: string | undefined;
-  let dryRun = options.dryRun;
-  let yes = false;
-  for (let i = 0; i < args.length; i++) {
-    const arg = args[i];
-    if (arg === "--since") since = args[++i];
-    else if (arg === "--out") out = args[++i];
-    else if (arg === "--target") target = args[++i];
-    else if (arg === "--dry-run") dryRun = true;
-    else if (arg === "--yes") yes = true;
-    else if (isGlobalFlag(arg, "--json", "--allow-insecure", "--verbose")) {
-      // handled globally by main()'s isJson flag; recognized here so it isn't rejected
-    } else if (isGlobalFlagWithValue(arg)) {
-      i++; // skip flag value
-    } else if (arg?.startsWith("--")) {
-      throw new Error(`unknown eval promote option: ${arg}`);
-    }
-  }
-  if (!since) {
-    throw new Error("usage: skillmux eval promote --since <window> [--out <path>] [--dry-run] [--yes] [--json]");
-  }
-
-  if (out !== undefined && target !== undefined && out !== target) {
-    throw new Error("cannot specify conflicting --out and --target");
-  }
-
-  if (target !== undefined) {
-    warn("--target is deprecated, use --out instead");
-  }
-
-  const config = await loadConfig();
-  const stateDir = expandHome(config.state_dir);
-  const targetPath = out ?? target ?? join(stateDir, "eval-observed.json");
-  const sinceDate = parseSince(since);
-  const sinceIso = sinceDate.toISOString();
-
-  const candidates = await options.adapter.evalPromote(since);
-
-  const existing = existsSync(targetPath) ? parseEvalCases(JSON.parse(readFileSync(targetPath, "utf-8"))) : [];
-  const { cases: newCases, skipped } = excludeExistingCases(candidates, existing);
-
-  warn("promoted eval cases contain raw user queries");
-
-  if (dryRun) {
-    emitSuccess(
-      { isJson: options.isJson },
-      { dry_run: true, since: sinceIso, target_path: targetPath, promoted: newCases.length, skipped_existing: skipped },
-      () => console.log(`promote: would write ${newCases.length} case(s) to ${targetPath} (skipped_existing=${skipped})`),
-    );
-    return;
-  }
-
-  if (
-    !(await confirmIfNeeded({
-      confirmed: yes,
-      isJson: options.isJson,
-      prompt: `promote ${newCases.length} eval case(s) to ${targetPath}?`,
-      nonInteractiveError: "skillmux eval promote requires --yes when run non-interactively",
-    }))
-  )
-    return;
-
-  if (newCases.length > 0) {
-    writeFileSync(targetPath, JSON.stringify([...existing, ...newCases], null, 2) + "\n");
-  }
-
-  emitSuccess(
-    { isJson: options.isJson },
-    { dry_run: false, since: sinceIso, target_path: targetPath, promoted: newCases.length, skipped_existing: skipped },
-    () => console.log(`promote: wrote ${newCases.length} case(s) to ${targetPath} (skipped_existing=${skipped})`),
-  );
-}