@petercjl/topazlabscli 0.2.1 → 0.2.2

package/README.md

@@ -15,7 +15,7 @@ npm install --global @petercjl/topazlabscli
 topazlabscli skill install --agent all
 ```
 
-The CLI checks npm for a newer stable release before operational commands, at most once every six hours. When an update is available it upgrades itself, refreshes installed Agent Skills, and then resumes the original command. It discovers npm through the running Node installation, preserves SealSeek's managed global prefix/cache, and discovers Windows OpenSSH through the standard system location, so it also works in Agent runtimes with a restricted `PATH`. A temporary npm outage does not block video processing. `topazlabscli update` forces an immediate manual update.
+The CLI checks npm for a newer stable release before operational commands, at most once every six hours. It first uses the registry already configured for npm and automatically tries `https://registry.npmmirror.com/` if that registry is unavailable. The successful registry is also used for installation, without changing the user's `.npmrc`. When an update is available the CLI upgrades itself, refreshes installed Agent Skills, and then resumes the original command. It discovers npm through the running Node installation, preserves SealSeek's managed global prefix/cache, and discovers Windows OpenSSH through the standard system location, so it also works in Agent runtimes with a restricted `PATH`. A temporary registry outage does not block video processing. `topazlabscli update` forces an immediate manual update.
 
 On Windows, SealSeek Skills are installed into `%USERPROFILE%\.sealseek\workspace\skills` when that workspace is present. The CLI automatically uses a managed copy because SealSeek rejects junctions that resolve outside the workspace Skill root; subsequent CLI updates refresh the copy from the npm package. The copy includes a local runtime manifest so the Agent can invoke the canonical package even when its PATH is restricted. `SEALSEEK_SKILLS_HOME` remains available as an explicit override.
 
@@ -84,4 +84,6 @@ Configuration is stored outside the package:
 - macOS/Linux: `${XDG_CONFIG_HOME:-~/.config}/topazlabscli/config.json`
 - Override for testing or automation: `TOPAZLABSCLI_CONFIG`
 
+Update registry selection is CLI-local. `settings set update-registry <url>` sets a preferred registry, and `settings set update-registry auto` restores automatic selection. `TOPAZLABSCLI_UPDATE_REGISTRY` can provide one or more comma-separated preferred registries for managed environments.
+
 Do not publish configuration files, keys, internal addresses, media, Topaz model files, or authentication data.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@petercjl/topazlabscli",
-  "version": "0.2.1",
+  "version": "0.2.2",
   "description": "Cross-Agent CLI and portable Skill for queued remote Topaz Video AI processing",
   "type": "module",
   "bin": {

package/skill/topazlabscli/SKILL.md

@@ -39,7 +39,7 @@ Use the CLI as the single execution surface. Do not reproduce SSH, SFTP, queue,
 
 Configuration, hostnames, addresses, usernames, SSH identities, VPN details, media, Topaz binaries, models, and credentials are external to this Skill and npm package. Installation does not grant access to a workstation. Treat the configured server and Topaz license as user-managed resources.
 
-Before operational commands, the CLI performs a cached npm update check. A newer stable package is installed automatically, installed Agent Skills are refreshed, and the original command resumes under the new version. The CLI resolves npm through the running Agent's Node installation when PATH is restricted. Registry, npm, and Skill-refresh failures produce a warning and continue with the installed version.
+Before operational commands, the CLI performs a cached npm update check. It tries the user's current npm registry and then its built-in reachable-registry fallback without changing the user's global npm configuration. A newer stable package is installed from the same registry that answered the version check, installed Agent Skills are refreshed, and the original command resumes under the new version. The CLI resolves npm through the running Agent's Node installation when PATH is restricted. Registry, npm, and Skill-refresh failures produce a warning and continue with the installed version. Treat `doctor`'s `updates.registry` check as advisory; a failed update source does not make video processing unavailable.
 
 Do not overwrite a local output unless the user has authorized that exact existing target. The remote worker retains job inputs, outputs, status, and logs for operator review; cleanup is an administrative action outside version 0.2.
 

package/src/cli.mjs

@@ -8,7 +8,7 @@ import { CliError, requireValue } from "./errors.mjs";
 import { run } from "./process.mjs";
 import { psLiteral, runPowerShell, selectEndpoint, sftpGet, sftpPut, startPowerShellDetached } from "./ssh.mjs";
 import { skillInstall, skillSource, skillStatus } from "./skill.mjs";
-import { maybeAutoUpdate } from "./update.mjs";
+import { installLatestPackage, maybeAutoUpdate, queryLatestVersion, updateRegistryWarning } from "./update.mjs";
 
 const require = createRequire(import.meta.url);
 const pkg = require("../package.json");
@@ -19,7 +19,7 @@ const CAPABILITIES = {
   package: pkg.name,
   version: pkg.version,
   commands: ["version", "capabilities", "doctor", "settings", "target", "connection", "worker", "model", "job", "process", "skill", "update"],
-  automatic_updates: { enabled_by_default: true, registry_check_hours: 6, refreshes_installed_skills: true },
+  automatic_updates: { enabled_by_default: true, registry_check_hours: 6, registry_fallback: true, refreshes_installed_skills: true },
   presets: [{ id: PRESET, model: "prob-4", output: "aspect-preserving 1080p", fps: "source", concurrency: 1 }],
   agents: { codex: "tested", sealseek_windows: "tested" },
   worker_os: ["windows"],
@@ -60,7 +60,7 @@ function help() {
   return `topazlabscli ${pkg.version}\n\n` +
     `Commands:\n` +
     `  version | capabilities | doctor\n` +
-    `  settings show | set auto-update <on|off> | set update-check-hours <hours>\n` +
+    `  settings show | set auto-update <on|off> | set update-check-hours <hours> | set update-registry <auto|url>\n` +
     `  target add <name> --endpoint <label=host>... --user <user> [--identity <path>] [--workspace <windows-path>] [--default]\n` +
     `  target list\n` +
     `  connection check [--target <name>]\n` +
@@ -152,7 +152,7 @@ export function defaultOutputPath(inputPath) {
   return path.join(path.dirname(resolved), `${path.basename(resolved, path.extname(resolved))}-topaz-1080p${extension}`);
 }
 
-async function doctor(requestedTarget) {
+async function doctor(requestedTarget, updateInfo = null) {
   const checks = [];
   for (const command of ["ssh", "sftp", "node", "npm"]) {
     const args = command === "node" || command === "npm" ? ["--version"] : command === "sftp" ? ["-h"] : ["-V"];
@@ -169,6 +169,16 @@ async function doctor(requestedTarget) {
     });
   }
   checks.push({ id: "config", ok: fs.existsSync(configPath()), detail: configPath() });
+  if (updateInfo) {
+    checks.push({
+      id: "updates.registry",
+      ok: Boolean(updateInfo.registry),
+      required: false,
+      detail: updateInfo.registry
+        ? { registry: updateInfo.registry, latest: updateInfo.latest || null, checked: updateInfo.checked }
+        : { warning: updateInfo.warning || "Update registry was not checked.", attempts: updateInfo.attempts || [] }
+    });
+  }
   if (requestedTarget || fs.existsSync(configPath())) {
     try {
       const { target, endpoint } = await getConnectedTarget(requestedTarget);
@@ -183,7 +193,7 @@ async function doctor(requestedTarget) {
       checks.push({ id: "connection", ok: false, detail: error.details || error.message });
     }
   }
-  return { ok: checks.every((item) => item.ok), checks };
+  return { ok: checks.filter((item) => item.required !== false).every((item) => item.ok), checks };
 }
 
 export async function main(rawArgs) {
@@ -194,11 +204,12 @@ export async function main(rawArgs) {
   if (command === "version" || command === "--version" || command === "-V") return output(pkg.version, json);
   if (command === "capabilities") return output(CAPABILITIES, json);
 
+  let updateInfo = null;
   if (!["update", "settings"].includes(command)) {
-    const update = await maybeAutoUpdate(rawArgs, pkg);
-    if (update.warning) process.stderr.write(`[AUTO_UPDATE_WARNING] ${update.warning}\n`);
-    if (update.reexecuted) {
-      process.exitCode = update.exitCode;
+    updateInfo = await maybeAutoUpdate(rawArgs, pkg);
+    if (updateInfo.warning) process.stderr.write(`[AUTO_UPDATE_WARNING] ${updateInfo.warning}\n`);
+    if (updateInfo.reexecuted) {
+      process.exitCode = updateInfo.exitCode;
       return;
     }
   }
@@ -217,6 +228,9 @@ export async function main(rawArgs) {
         const hours = Number(value);
         if (!Number.isFinite(hours) || hours < 0) throw new CliError("SETTING_INVALID", "update-check-hours must be zero or a positive number.");
         config.settings.update_check_hours = hours;
+      } else if (name === "update-registry") {
+        if (value !== "auto" && !/^https?:\/\//i.test(value)) throw new CliError("SETTING_INVALID", "update-registry must be auto or an http(s) URL.");
+        config.settings.update_registry = value;
       } else throw new CliError("SETTING_UNKNOWN", `Unknown setting: ${name}`);
       const saved = saveConfig(config);
       return output({ path: saved, settings: config.settings }, json);
@@ -224,7 +238,7 @@ export async function main(rawArgs) {
     throw new CliError("COMMAND_UNKNOWN", `Unknown settings action: ${action}`);
   }
 
-  if (command === "doctor") return output(await doctor(option(args, "--target")), json);
+  if (command === "doctor") return output(await doctor(option(args, "--target"), updateInfo), json);
 
   if (command === "target") {
     const action = args.shift();
@@ -319,14 +333,17 @@ export async function main(rawArgs) {
   }
 
   if (command === "update") {
+    const config = loadConfig();
+    const query = await queryLatestVersion(pkg, config);
+    if (!query.ok) throw new CliError("UPDATE_FAILED", updateRegistryWarning(query.attempts), { attempts: query.attempts });
     const previousSkills = skillStatus("all");
-    const result = await run("npm", ["install", "-g", `${pkg.name}@latest`]);
+    const result = await installLatestPackage(pkg, query.registry);
     if (result.code !== 0) throw new CliError("UPDATE_FAILED", result.stderr.trim() || "npm update failed.");
     const skills = [];
     for (const existing of previousSkills.filter((item) => item.installed)) {
       skills.push(...skillInstall(existing.agent, existing.mode || "link", true));
     }
-    return output({ package: pkg.name, updated: true, skills, detail: result.stdout.trim() }, json);
+    return output({ package: pkg.name, updated: true, registry: query.registry, latest: query.latest, skills, detail: result.stdout.trim() }, json);
   }
   throw new CliError("COMMAND_UNKNOWN", `Unknown command: ${command}`);
 }

package/src/config.mjs

@@ -8,7 +8,7 @@ export function emptyConfig() {
     schema_version: 1,
     default_target: null,
     targets: {},
-    settings: { auto_update: true, update_check_hours: 6 }
+    settings: { auto_update: true, update_check_hours: 6, update_registry: "auto" }
   };
 }
 
@@ -29,7 +29,10 @@ export function loadConfig({ required = false } = {}) {
         auto_update: parsed.settings?.auto_update !== false,
         update_check_hours: Number.isFinite(parsed.settings?.update_check_hours)
           ? parsed.settings.update_check_hours
-          : 6
+          : 6,
+        update_registry: typeof parsed.settings?.update_registry === "string"
+          ? parsed.settings.update_registry
+          : "auto"
       }
     };
   } catch (error) {

package/src/update.mjs

@@ -6,6 +6,8 @@ import { run, runInherited } from "./process.mjs";
 import { skillInstall, skillStatus } from "./skill.mjs";
 
 const DEFAULT_INTERVAL_HOURS = 6;
+const DEFAULT_MIRROR_REGISTRY = "https://registry.npmmirror.com/";
+const UPDATE_TIMEOUT_MS = 8000;
 const UPDATE_GUARD = "TOPAZLABSCLI_AUTO_UPDATE_GUARD";
 
 function numericParts(version) {
@@ -41,6 +43,87 @@ function automaticUpdateEnabled(config, env) {
   return config.settings?.auto_update !== false;
 }
 
+function normalizeRegistry(value) {
+  const text = String(value || "").trim();
+  if (!/^https?:\/\//i.test(text)) return null;
+  return text.endsWith("/") ? text : `${text}/`;
+}
+
+function uniqueRegistries(values) {
+  return [...new Set(values.map(normalizeRegistry).filter(Boolean))];
+}
+
+export async function resolveUpdateRegistries(config, dependencies = {}) {
+  const env = dependencies.env || process.env;
+  const execute = dependencies.run || run;
+  const configured = config.settings?.update_registry;
+  const overrides = String(env.TOPAZLABSCLI_UPDATE_REGISTRY || "")
+    .split(",")
+    .map((item) => item.trim())
+    .filter(Boolean);
+  const preferred = configured && configured !== "auto" ? [configured] : [];
+  let npmRegistry = null;
+  try {
+    const result = await execute("npm", ["config", "get", "registry"], {
+      env,
+      timeoutMs: dependencies.timeoutMs || UPDATE_TIMEOUT_MS
+    });
+    if (result.code === 0) npmRegistry = result.stdout.trim();
+  } catch {}
+  return uniqueRegistries([...overrides, ...preferred, npmRegistry, DEFAULT_MIRROR_REGISTRY]);
+}
+
+export async function queryLatestVersion(pkg, config, dependencies = {}) {
+  const env = dependencies.env || process.env;
+  const execute = dependencies.run || run;
+  const registries = dependencies.registries || await resolveUpdateRegistries(config, dependencies);
+  const attempts = [];
+  for (const registry of registries) {
+    let result;
+    try {
+      result = await execute("npm", ["view", pkg.name, "version", "--json", "--registry", registry], {
+        env: {
+          ...env,
+          npm_config_fetch_timeout: env.npm_config_fetch_timeout || "5000",
+          npm_config_fetch_retries: "0"
+        },
+        timeoutMs: dependencies.timeoutMs || UPDATE_TIMEOUT_MS
+      });
+    } catch (error) {
+      attempts.push({ registry, ok: false, detail: error.message });
+      continue;
+    }
+    if (result.code !== 0) {
+      attempts.push({
+        registry,
+        ok: false,
+        detail: result.timedOut ? "timed out" : (result.stderr.trim() || "registry query failed")
+      });
+      continue;
+    }
+    let latest;
+    try { latest = JSON.parse(result.stdout.trim()); }
+    catch { latest = result.stdout.trim().replace(/^"|"$/g, ""); }
+    attempts.push({ registry, ok: true, latest });
+    return { ok: true, latest, registry, attempts };
+  }
+  return { ok: false, attempts };
+}
+
+function registryFailureMessage(attempts) {
+  if (!attempts.length) return "No valid npm update registry is configured.";
+  return `Unable to check npm for updates: ${attempts.map((item) => `${item.registry} (${item.detail})`).join("; ")}`;
+}
+
+export async function installLatestPackage(pkg, registry, dependencies = {}) {
+  const env = dependencies.env || process.env;
+  const execute = dependencies.run || run;
+  return execute("npm", ["install", "--global", `${pkg.name}@latest`, "--registry", registry], {
+    env,
+    timeoutMs: dependencies.installTimeoutMs
+  });
+}
+
 export async function maybeAutoUpdate(rawArgs, pkg, dependencies = {}) {
   const env = dependencies.env || process.env;
   if (env[UPDATE_GUARD] === "1") return { checked: false, reason: "guard" };
@@ -54,39 +137,27 @@ export async function maybeAutoUpdate(rawArgs, pkg, dependencies = {}) {
   const intervalMs = Math.max(0, intervalHours) * 60 * 60 * 1000;
   const state = readState(stateFile);
   if (intervalMs > 0 && Number.isFinite(state.last_checked_at) && now - state.last_checked_at < intervalMs) {
-    return { checked: false, reason: "fresh", latest: state.latest || null };
+    return { checked: false, reason: "fresh", latest: state.latest || null, registry: state.registry || null };
   }
 
-  const execute = dependencies.run || run;
-  let query;
-  try {
-    query = await execute("npm", ["view", pkg.name, "version", "--json"], {
-      env: { ...env, npm_config_fetch_timeout: env.npm_config_fetch_timeout || "5000", npm_config_fetch_retries: "0" }
-    });
-  } catch (error) {
-    return { checked: true, warning: `Unable to check npm for updates: ${error.message}` };
-  }
-  if (query.code !== 0) {
-    return { checked: true, warning: query.stderr.trim() || "Unable to check npm for updates." };
-  }
+  const query = await queryLatestVersion(pkg, config, dependencies);
+  if (!query.ok) return { checked: true, warning: registryFailureMessage(query.attempts), attempts: query.attempts };
 
-  let latest;
-  try { latest = JSON.parse(query.stdout.trim()); }
-  catch { latest = query.stdout.trim().replace(/^"|"$/g, ""); }
-  writeState(stateFile, { last_checked_at: now, latest, current: pkg.version });
-  if (!isNewerVersion(latest, pkg.version)) return { checked: true, updated: false, latest };
+  const { latest, registry } = query;
+  writeState(stateFile, { last_checked_at: now, latest, current: pkg.version, registry });
+  if (!isNewerVersion(latest, pkg.version)) return { checked: true, updated: false, latest, registry, attempts: query.attempts };
 
   const getSkillStatus = dependencies.skillStatus || skillStatus;
   const installSkill = dependencies.skillInstall || skillInstall;
   const installedSkills = getSkillStatus("all").filter((item) => item.installed);
   let install;
   try {
-    install = await execute("npm", ["install", "--global", `${pkg.name}@latest`], { env });
+    install = await installLatestPackage(pkg, registry, dependencies);
   } catch (error) {
-    return { checked: true, warning: `Automatic npm update failed: ${error.message}`, latest };
+    return { checked: true, warning: `Automatic npm update failed: ${error.message}`, latest, registry };
   }
   if (install.code !== 0) {
-    return { checked: true, warning: install.stderr.trim() || "Automatic npm update failed.", latest };
+    return { checked: true, warning: install.stderr.trim() || "Automatic npm update failed.", latest, registry };
   }
   const warnings = [];
   for (const item of installedSkills) {
@@ -99,9 +170,13 @@ export async function maybeAutoUpdate(rawArgs, pkg, dependencies = {}) {
     const child = await reexecute(process.execPath, [binScript, ...rawArgs], {
       env: { ...env, [UPDATE_GUARD]: "1" }
     });
-    return { checked: true, updated: true, latest, reexecuted: true, exitCode: child.code ?? 1, warnings };
+    return { checked: true, updated: true, latest, registry, reexecuted: true, exitCode: child.code ?? 1, warnings };
   } catch (error) {
     warnings.push(`Updated package could not restart the command: ${error.message}`);
-    return { checked: true, updated: true, latest, warning: warnings.join(" ") };
+    return { checked: true, updated: true, latest, registry, warning: warnings.join(" ") };
   }
 }
+
+export function updateRegistryWarning(attempts) {
+  return registryFailureMessage(attempts);
+}