oh-skillhub 0.1.14 → 0.1.16

package/bin/oh-skillhub.js

@@ -1,8 +1,8 @@
 #!/usr/bin/env node
 
-const { main } = require("../src/cli");
+const { main, renderStatusLine } = require("../src/cli");
 
 main(process.argv.slice(2)).catch((error) => {
-  console.error(error.message);
+  process.stderr.write(`${renderStatusLine("failure", "INSTALL FAILED", error.message, process.stderr)}\n`);
   process.exitCode = 1;
 });

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "oh-skillhub",
-  "version": "0.1.14",
+  "version": "0.1.16",
   "description": "OpenHarmony Skills installer for Codex, Claude Code, and OpenCode.",
   "type": "commonjs",
   "bin": {

package/src/cli.js

@@ -8,7 +8,7 @@ const { resolveAgentTargets } = require("./agents");
 const { applyCleanPlan, planClean, scanInstalledSkills } = require("./cleaner");
 const { loadLocalManifest, loadProfiles, selectSkills } = require("./manifest");
 const { applyInstallPlan, planInstall } = require("./planner");
-const { ensureSkillSourceRoot } = require("./source");
+const { ensureSkillSourceRoot, ensureSkillSourceRootAsync } = require("./source");
 const { buildTelemetryEvent, enqueueTelemetryEvent, telemetryStatus } = require("./telemetry");
 
 const packageJson = require("../package.json");
@@ -29,6 +29,7 @@ const ANSI = {
   dim: "\x1b[2m",
   cyan: "\x1b[36m",
   green: "\x1b[32m",
+  red: "\x1b[31m",
   reverse: "\x1b[7m",
 };
 const AGENT_CHOICES = [
@@ -392,9 +393,39 @@ async function installInteractiveSelection(manifest, choices, agent, selectedInd
     throw new Error("No skills matched the selected groups.");
   }
   output.write(`\nInstalling ${selectedChoices.map((choice) => choice.path).join(", ")} for ${agent}:user...\n`);
-  output.write(`Preparing skill source from ${manifest.source}#${manifest.ref}...\n`);
-  output.write(`${renderInstallForSkills(skills, { agent, scope: "user" })}\n`);
-  output.write(`${renderInteractiveCompletion(skills, { agent, scope: "user" })}\n`);
+  const sourceRoot = await withSpinner(output, `Preparing skill source from ${manifest.source}#${manifest.ref}`, () =>
+    ensureSkillSourceRootAsync(manifest, { env: process.env, skills }),
+  );
+  output.write(`${renderInstallForSkills(skills, { agent, scope: "user" }, sourceRoot)}\n`);
+  output.write(`${renderInteractiveCompletion(skills, { agent, scope: "user" }, output)}\n`);
+}
+
+async function withSpinner(output, message, task, options = {}) {
+  if (!output.isTTY) {
+    output.write(`${message}...\n`);
+    return task();
+  }
+
+  const frames = options.frames || ["|", "/", "-", "\\"];
+  const intervalMs = options.intervalMs || 120;
+  let index = 0;
+  const render = () => {
+    output.write(`\r${frames[index % frames.length]} ${message}...`);
+    index += 1;
+  };
+
+  render();
+  const timer = setInterval(render, intervalMs);
+  try {
+    const result = await task();
+    clearInterval(timer);
+    output.write(`\r${" ".repeat(message.length + 8)}\r✓ ${message}\n`);
+    return result;
+  } catch (error) {
+    clearInterval(timer);
+    output.write(`\r${" ".repeat(message.length + 8)}\r! ${message}\n`);
+    throw error;
+  }
 }
 
 function readAll(input) {
@@ -918,7 +949,7 @@ function selectSkillsForChoices(manifest, choices) {
   return Array.from(selected.values()).sort((left, right) => left.name.localeCompare(right.name));
 }
 
-function renderInstallForSkills(skills, targetOptions) {
+function renderInstallForSkills(skills, targetOptions, sourceRootOverride = null) {
   const targets = resolveAgentTargets({
     agent: targetOptions.agent,
     scope: targetOptions.scope,
@@ -928,7 +959,7 @@ function renderInstallForSkills(skills, targetOptions) {
   });
   const plan = planInstall(skills, targets);
   const manifest = loadLocalManifest();
-  const sourceRoot = ensureSkillSourceRoot(manifest, { env: process.env, skills });
+  const sourceRoot = sourceRootOverride || ensureSkillSourceRoot(manifest, { env: process.env, skills });
   const applied = applyInstallPlan(plan, { sourceRoot });
   const telemetryEnabled = process.env.OH_SKILLHUB_NO_TELEMETRY !== "1";
   for (const operation of applied) {
@@ -950,14 +981,24 @@ function renderInstallForSkills(skills, targetOptions) {
   return renderInstallPlan("Install summary", plan);
 }
 
-function renderInteractiveCompletion(skills, targetOptions) {
+function renderInteractiveCompletion(skills, targetOptions, output = process.stdout) {
   return [
     "",
-    `Done. Installed ${skills.length} skill(s) for ${targetOptions.agent}:${targetOptions.scope}.`,
-    "You can close this terminal or start using the installed skills.",
+    renderStatusLine(
+      "success",
+      "INSTALL SUCCESS",
+      `Installed ${skills.length} skill(s) for ${targetOptions.agent}:${targetOptions.scope}.`,
+      output,
+    ),
   ].join("\n");
 }
 
+function renderStatusLine(kind, title, detail, output = process.stdout) {
+  const color = kind === "failure" ? ANSI.red : ANSI.green;
+  const heading = output.isTTY ? colorize(title, color, ANSI.bold) : title;
+  return detail ? `${heading}\n${detail}` : heading;
+}
+
 function renderInstallPlan(title, plan) {
   const lines = [title];
   for (const operation of plan.operations) {
@@ -1038,6 +1079,7 @@ module.exports = {
   renderInstall,
   renderList,
   renderTelemetry,
+  renderStatusLine,
   buildRepositoryChoices,
   renderAgentMenu,
   renderActionMenu,
@@ -1048,5 +1090,6 @@ module.exports = {
   parseSelection,
   renderRawTuiMenu,
   renderTuiMenu,
+  withSpinner,
   selectSkillsForChoices,
 };

package/src/source.js

@@ -1,4 +1,4 @@
-const { spawnSync } = require("node:child_process");
+const { spawn, spawnSync } = require("node:child_process");
 const crypto = require("node:crypto");
 const fs = require("node:fs");
 const os = require("node:os");
@@ -48,6 +48,61 @@ function ensureSkillSourceRoot(manifest, options = {}) {
   return checkout;
 }
 
+async function ensureSkillSourceRootAsync(manifest, options = {}) {
+  const env = options.env || process.env;
+  if (env.OH_SKILLHUB_SOURCE_DIR) {
+    return requireDirectory(env.OH_SKILLHUB_SOURCE_DIR, "OH_SKILLHUB_SOURCE_DIR");
+  }
+
+  const source = manifest.source;
+  const ref = manifest.ref || "release";
+  if (!source) {
+    throw new Error("Manifest does not define a skill source repository.");
+  }
+
+  const cacheRoot = env.OH_SKILLHUB_CACHE_DIR || path.join(os.homedir(), ".oh-skillhub", "cache");
+  const sparsePaths = selectedSparsePaths(options.skills);
+  const checkout = path.join(cacheRoot, `${repositorySlug(source)}-${sanitize(ref)}-${sparseKey(sparsePaths)}`);
+  if (isUsableCheckout(checkout, sparsePaths)) {
+    return checkout;
+  }
+
+  fs.mkdirSync(cacheRoot, { recursive: true });
+  fs.rmSync(checkout, { recursive: true, force: true });
+  const tempCheckout = path.join(cacheRoot, `.tmp-${Date.now()}-${Math.random().toString(16).slice(2)}`);
+  const clone = await runGitAsync([
+    "-c",
+    "core.longpaths=true",
+    "clone",
+    "--depth",
+    "1",
+    "--branch",
+    ref,
+    "--no-checkout",
+    source,
+    tempCheckout,
+  ]);
+  if (clone.status !== 0) {
+    fs.rmSync(tempCheckout, { recursive: true, force: true });
+    const detail = gitDetail(clone);
+    throw new Error(`Failed to download skill source from ${source}#${ref}.${detail ? ` ${detail}` : ""}`);
+  }
+  const sparseInit = await runGitAsync(["-C", tempCheckout, "-c", "core.longpaths=true", "sparse-checkout", "init", "--no-cone"]);
+  const sparseSet = sparseInit.status === 0
+    ? await runGitAsync(["-C", tempCheckout, "-c", "core.longpaths=true", "sparse-checkout", "set", ...sparsePaths])
+    : sparseInit;
+  const checkoutResult = sparseSet.status === 0
+    ? await runGitAsync(["-C", tempCheckout, "-c", "core.longpaths=true", "checkout"])
+    : sparseSet;
+  if (checkoutResult.status !== 0) {
+    fs.rmSync(tempCheckout, { recursive: true, force: true });
+    const detail = gitDetail(checkoutResult);
+    throw new Error(`Failed to checkout selected skill source from ${source}#${ref}.${detail ? ` ${detail}` : ""}`);
+  }
+  fs.renameSync(tempCheckout, checkout);
+  return checkout;
+}
+
 function requireDirectory(value, label) {
   const dir = path.resolve(value);
   if (!fs.existsSync(dir) || !fs.statSync(dir).isDirectory()) {
@@ -82,10 +137,33 @@ function runGit(args) {
   return spawnSync("git", args, { encoding: "utf8", shell: false });
 }
 
+function runGitAsync(args) {
+  return new Promise((resolve) => {
+    const child = spawn("git", args, { shell: false });
+    let stdout = "";
+    let stderr = "";
+    child.stdout.setEncoding("utf8");
+    child.stderr.setEncoding("utf8");
+    child.stdout.on("data", (chunk) => {
+      stdout += chunk;
+    });
+    child.stderr.on("data", (chunk) => {
+      stderr += chunk;
+    });
+    child.on("error", (error) => {
+      resolve({ status: 1, stdout, stderr: `${stderr}${error.message}` });
+    });
+    child.on("close", (status) => {
+      resolve({ status, stdout, stderr });
+    });
+  });
+}
+
 function gitDetail(result) {
   return (result.stderr || result.stdout || "").trim();
 }
 
 module.exports = {
   ensureSkillSourceRoot,
+  ensureSkillSourceRootAsync,
 };