@minhpnq1807/contextos 0.5.32 → 0.5.34

package/CHANGELOG.md

@@ -1,5 +1,12 @@
 # Changelog
 
+## 0.5.34
+
+- **Real-time streaming output during install/setup:** Replaced `captureSetupOutput` (buffered) with `streamSetupOutput` — now prints each line immediately with `│  ` prefix as it arrives, eliminating the perceived "hang" during long-running downloads and installs.
+- **Fix codex CLI output missing `│` prefix:** Changed `runCodex` from `stdio: "inherit"` to `stdio: ["ignore", "pipe", "pipe"]`. Output now flows through `console.log` → `streamSetupOutput` → `│  ` prefix, ensuring lines like "Added marketplace..." are consistently formatted.
+- **Async streaming for skillshare/ruler install:** Replaced blocking `execSync`/`runShell` calls in `installSkillshare` and `installRuler` with async `spawn` + line-by-line streaming. Download progress from PowerShell/curl/npm is now visible in real time instead of being buffered until completion.
+- **Fix `skillshare init` hang on Windows:** `skillshare init` is interactive by default (prompts for copy source, git, skill install). With stdin routed to NUL (deadlock prevention), the Go binary hangs waiting for terminal input that never arrives. Fixed by passing `--no-copy --no-git --no-skill --all-targets` flags for fully non-interactive initialization.
+
 ## 0.5.32
 
 - **Fix Windows terminal hang during skillshare/ruler install:** `execSync` with `stdio: "pipe"` creates a stdin pipe whose write-end is held by Node while it blocks on `waitpid`. If the child process (PowerShell installer, npm, etc.) reads from stdin, it blocks waiting for data/EOF that never comes — classic deadlock. Fixed by normalizing `stdio: "pipe"` to `["ignore", "pipe", "pipe"]` in both `runCommand` and `runShell`. This routes stdin to NUL (`/dev/null`) for immediate EOF, while still capturing stdout/stderr through pipes for `◇`/`│` formatting.

package/bin/ctx.js

@@ -97,19 +97,23 @@ function normalizeInstallAgent(agent) {
   if (normalized === "antigravity") return "agy";
   return normalized;
 }
-
 /**
- * Capture all console.log and process.stderr.write output from an async fn.
- * Returns an array of clean text lines (no \r, no trailing whitespace).
+ * Intercept console.log and process.stderr.write from an async fn,
+ * printing each line immediately with "│  " prefix for real-time feedback.
+ * Returns the collected lines array (for callers that inspect it).
  */
-async function captureSetupOutput(fn) {
+async function streamSetupOutput(fn) {
   const lines = [];
   const origLog = console.log;
   const origStderrWrite = process.stderr.write;
-  console.log = (...args) => lines.push(args.map(String).join(" "));
+  const emit = (text) => {
+    lines.push(text);
+    origLog(`│  ${text}`);
+  };
+  console.log = (...args) => emit(args.map(String).join(" "));
   process.stderr.write = (chunk) => {
     const text = String(chunk).replace(/\r/g, "").trim();
-    if (text) lines.push(text);
+    if (text) emit(text);
     return true;
   };
   try {
@@ -363,8 +367,23 @@ function tryRunCodex(args) {
 
 function runCodex(args) {
   try {
-    execFileSync("codex", args, { stdio: "inherit", shell: true });
+    const stdout = execFileSync("codex", args, {
+      stdio: ["ignore", "pipe", "pipe"],
+      encoding: "utf8",
+      shell: true
+    });
+    if (stdout && stdout.trim()) {
+      for (const line of stdout.trim().split(/\r?\n/)) {
+        console.log(line);
+      }
+    }
   } catch (error) {
+    // Log any output captured before the error
+    if (error.stdout && error.stdout.trim()) {
+      for (const line of error.stdout.trim().split(/\r?\n/)) {
+        console.log(line);
+      }
+    }
     const status = typeof error.status === "number" ? error.status : 1;
     throw new Error(`codex ${args.join(" ")} failed with exit code ${status}. Make sure Codex CLI is installed and authenticated.`);
   }
@@ -561,8 +580,7 @@ async function setup({ args = [], cwd = process.cwd() } = {}) {
 
   for (const agent of options.agents) {
     console.log(`◇ Setting up ${agent}...`);
-    const lines = await captureSetupOutput(() => install({ agent, copy: false }));
-    for (const line of lines) console.log(`│  ${line}`);
+    await streamSetupOutput(() => install({ agent, copy: false }));
   }
 
   if (options.syncRules) {
@@ -570,8 +588,7 @@ async function setup({ args = [], cwd = process.cwd() } = {}) {
     const syncAgents = options.agents.map((agent) => agent === "agy" ? "antigravity" : agent).join(",");
     const syncArgs = ["--rules", "--agents", syncAgents];
     if (options.yes) syncArgs.push("--yes");
-    const lines = await captureSetupOutput(() => syncRules({ cwd, rootDir, args: syncArgs }));
-    for (const line of lines) console.log(`│  ${line}`);
+    await streamSetupOutput(() => syncRules({ cwd, rootDir, args: syncArgs }));
   }
 
   if (options.syncSkills) {
@@ -579,7 +596,7 @@ async function setup({ args = [], cwd = process.cwd() } = {}) {
     const skillAgents = options.agents.map((agent) => agent === "agy" ? "antigravity" : agent).join(",");
     const syncArgs = ["--skills", "--agents", skillAgents];
     if (options.yes) syncArgs.push("--yes");
-    const lines = await captureSetupOutput(() => syncSkills({
+    await streamSetupOutput(() => syncSkills({
       cwd,
       args: syncArgs,
       rebuildSkillEmbeddings: async ({ cwd: skillCwd, sourceDir }) => warmSkillEmbeddings({
@@ -589,7 +606,6 @@ async function setup({ args = [], cwd = process.cwd() } = {}) {
         skills: scanSkills({ cwd: skillCwd, roots: [sourceDir] })
       })
     }));
-    for (const line of lines) console.log(`│  ${line}`);
   }
 
   console.log("");
@@ -620,8 +636,7 @@ try {
     if (explicitAgent) {
       // Direct mode: ctx install --agent <name>
       console.log(`◇ Installing ${explicitAgent}...`);
-      const lines = await captureSetupOutput(() => install({ copy, agent: explicitAgent }));
-      for (const line of lines) console.log(`│  ${line}`);
+      await streamSetupOutput(() => install({ copy, agent: explicitAgent }));
       console.log("");
     } else {
       // Interactive mode: ctx install
@@ -634,8 +649,7 @@ try {
       } else {
         for (const agent of selected) {
           console.log(`◇ Installing ${agent}...`);
-          const lines = await captureSetupOutput(() => install({ copy, agent }));
-          for (const line of lines) console.log(`│  ${line}`);
+          await streamSetupOutput(() => install({ copy, agent }));
           console.log("");
         }
       }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@minhpnq1807/contextos",
-  "version": "0.5.32",
+  "version": "0.5.34",
   "description": "Task-aware AGENTS.md context injection and compliance reporting for Codex, Claude Code, and Antigravity.",
   "type": "module",
   "bin": {

package/plugins/ctx/.codex-plugin/plugin.json

@@ -1,6 +1,6 @@
 {
   "name": "ctx",
-  "version": "0.5.32",
+  "version": "0.5.34",
   "description": "Inject task-relevant AGENTS.md rules into Codex through plugin hooks.",
   "author": {
     "name": "ContextOS"

package/plugins/ctx/lib/ruler-sync.js

@@ -3,7 +3,7 @@ import os from "node:os";
 import path from "node:path";
 import readline from "node:readline/promises";
 import { stdin as input, stdout as output } from "node:process";
-import { execFileSync } from "node:child_process";
+import { execFileSync, spawn } from "node:child_process";
 
 import { defaultDataRoot } from "./workspace-data.js";
 
@@ -98,7 +98,46 @@ export async function installRuler({ run = runCommand, yes = false, dryRun = fal
   if (!accepted) {
     throw new Error("Ruler is required for ctx sync --rules. Install it with `npm install -g @intellectronica/ruler` or rerun with --yes.");
   }
-  run("npm", ["install", "-g", "@intellectronica/ruler"], { stdio: "pipe", dryRun });
+  if (dryRun) {
+    run("npm", ["install", "-g", "@intellectronica/ruler"], { stdio: "pipe", dryRun });
+  } else {
+    console.log("Installing ruler...");
+    await spawnCommand("npm", ["install", "-g", "@intellectronica/ruler"]);
+  }
+}
+
+/**
+ * Spawn a child process and stream stdout/stderr line-by-line in real time.
+ * stdin is closed immediately to prevent deadlocks on Windows.
+ */
+function spawnCommand(command, args = []) {
+  return new Promise((resolve, reject) => {
+    const child = spawn(command, args, {
+      stdio: ["ignore", "pipe", "pipe"],
+      shell: true
+    });
+    const streamLines = (stream) => {
+      let buffer = "";
+      stream.on("data", (chunk) => {
+        buffer += chunk.toString();
+        const lines = buffer.split(/\r?\n/);
+        buffer = lines.pop() || "";
+        for (const line of lines) {
+          if (line.trim()) console.log(line);
+        }
+      });
+      stream.on("end", () => {
+        if (buffer.trim()) console.log(buffer.trim());
+      });
+    };
+    if (child.stdout) streamLines(child.stdout);
+    if (child.stderr) streamLines(child.stderr);
+    child.on("close", (code) => {
+      if (code === 0) resolve();
+      else reject(new Error(`${command} ${args.join(" ")} exited with code ${code}`));
+    });
+    child.on("error", reject);
+  });
 }
 
 export function ensureRulerInit({ cwd = process.cwd(), run = runCommand, dryRun = false } = {}) {

package/plugins/ctx/lib/skillshare-sync.js

@@ -3,7 +3,7 @@ import os from "node:os";
 import path from "node:path";
 import readline from "node:readline/promises";
 import { stdin as input, stdout as output } from "node:process";
-import { execFileSync, execSync } from "node:child_process";
+import { execFileSync, execSync, spawn } from "node:child_process";
 
 const DEFAULT_AGENTS = ["codex", "claude", "antigravity"];
 const INSTALL_SH_URL = "https://raw.githubusercontent.com/runkids/skillshare/main/install.sh";
@@ -127,17 +127,31 @@ export async function installSkillshare({
   }
 
   const osName = detectOS(platform);
-  if (osName === "windows") {
-    runShellCommand(`powershell -NoProfile -ExecutionPolicy Bypass -Command "irm ${INSTALL_PS_URL} | iex"`, { stdio: "pipe", dryRun });
-    // The installer adds to the system PATH, but the current Node process
-    // still has the old PATH. Inject the known install dir so subsequent
-    // skillshare calls in this session can resolve the binary.
+
+  if (dryRun) {
+    // dry-run keeps the old sync path
+    if (osName === "windows") {
+      runShellCommand(`powershell -NoProfile -ExecutionPolicy Bypass -Command "irm ${INSTALL_PS_URL} | iex"`, { stdio: "pipe", dryRun });
+    } else {
+      runShellCommand(`curl -fsSL ${INSTALL_SH_URL} | sh`, { stdio: "pipe", dryRun });
+    }
+  } else {
+    console.log("Installing skillshare...");
+    if (osName === "windows") {
+      await spawnShellStreaming("powershell", ["-NoProfile", "-ExecutionPolicy", "Bypass", "-Command", `irm ${INSTALL_PS_URL} | iex`]);
+    } else {
+      await spawnShellStreaming("sh", ["-c", `curl -fsSL ${INSTALL_SH_URL} | sh`]);
+    }
+  }
+
+  // The installer adds to the system PATH, but the current Node process
+  // still has the old PATH. Inject the known install dir so subsequent
+  // skillshare calls in this session can resolve the binary.
+  if (!dryRun && osName === "windows") {
     const winInstallDir = path.join(os.homedir(), "AppData", "Local", "Programs", "skillshare");
-    if (!dryRun && !process.env.PATH.includes(winInstallDir)) {
+    if (!process.env.PATH.includes(winInstallDir)) {
       process.env.PATH = `${winInstallDir}${path.delimiter}${process.env.PATH}`;
     }
-  } else {
-    runShellCommand(`curl -fsSL ${INSTALL_SH_URL} | sh`, { stdio: "pipe", dryRun });
   }
 
   const check = checkSkillshareInstalled({ run });
@@ -147,6 +161,44 @@ export async function installSkillshare({
   return check;
 }
 
+/**
+ * Spawn a child process and stream its stdout/stderr line-by-line in real time
+ * via console.log (which will be intercepted by streamSetupOutput for │ prefix).
+ * stdin is closed immediately to prevent deadlocks.
+ */
+function spawnShellStreaming(command, args = []) {
+  return new Promise((resolve, reject) => {
+    const child = spawn(command, args, {
+      stdio: ["ignore", "pipe", "pipe"],
+      shell: false
+    });
+
+    const streamLines = (stream) => {
+      let buffer = "";
+      stream.on("data", (chunk) => {
+        buffer += chunk.toString();
+        const lines = buffer.split(/\r?\n/);
+        buffer = lines.pop() || "";
+        for (const line of lines) {
+          if (line.trim()) console.log(line);
+        }
+      });
+      stream.on("end", () => {
+        if (buffer.trim()) console.log(buffer.trim());
+      });
+    };
+
+    if (child.stdout) streamLines(child.stdout);
+    if (child.stderr) streamLines(child.stderr);
+
+    child.on("close", (code) => {
+      if (code === 0) resolve();
+      else reject(new Error(`${command} exited with code ${code}`));
+    });
+    child.on("error", reject);
+  });
+}
+
 export function detectExistingSkills({ cwd = process.cwd(), home = os.homedir() } = {}) {
   return skillRoots({ cwd, home })
     .map((root) => ({ path: root, count: countSkillFiles(root) }))
@@ -360,7 +412,10 @@ export async function syncSkills({
       logger("[ctx] No existing skills found.");
     }
 
-    run("skillshare", ["init"], { cwd, stdio: "pipe", dryRun: options.dryRun });
+    // --no-copy --no-git --no-skill --all-targets: fully non-interactive init.
+    // skillshare init is interactive by default; with stdin routed to NUL
+    // (deadlock prevention) the Go binary hangs waiting for terminal input.
+    run("skillshare", ["init", "--no-copy", "--no-git", "--no-skill", "--all-targets"], { cwd, stdio: "pipe", dryRun: options.dryRun });
     logger(statusLine("Initializing skillshare...", options.dryRun ? "dry-run" : "✓ initialized"));
 
     if (existing.length && !options.noCollect) {