igel-qe-core 1.0.12 → 1.0.16

package/dist/cli/index.js

@@ -221,30 +221,51 @@ function listFrameworkContextFiles(workspaceRoot) {
     return Array.from(new Set(found)).sort();
 }
 function isCopilotCliAvailable() {
-    const check = spawnSync("copilot", ["--version"], { encoding: "utf-8", shell: process.platform === "win32" });
-    return check.status === 0;
+    // Use a short timeout so this never blocks if copilot hangs on startup.
+    const check = spawnSync("copilot", ["--version"], {
+        encoding: "utf-8",
+        shell: process.platform === "win32",
+        timeout: 10_000,
+    });
+    return check.status === 0 && !check.error;
 }
-function isCopilotAuthenticated(workspaceRoot) {
-    const probe = spawnSync("copilot", ["-C", workspaceRoot, "-p", "Reply with exactly OK.", "--allow-all-tools", "--no-color", "-s"], { encoding: "utf-8", shell: process.platform === "win32" });
-    return probe.status === 0;
+function isCopilotAuthenticated() {
+    // Use a lightweight auth-status check instead of spawning a full agent session.
+    // "copilot auth status" exits 0 when authenticated, non-zero otherwise.
+    // Fallback: if the sub-command doesn't exist, treat as unauthenticated.
+    const probe = spawnSync("copilot", ["auth", "status"], {
+        encoding: "utf-8",
+        shell: process.platform === "win32",
+        timeout: 10_000, // 10 s max — simple status call should be instant
+    });
+    return probe.status === 0 && !probe.error;
 }
 function ensureCopilotLogin(workspaceRoot) {
     if (!isCopilotCliAvailable())
         return false;
-    if (isCopilotAuthenticated(workspaceRoot))
+    if (isCopilotAuthenticated())
         return true;
     console.log(chalk.yellow("   ⚠ GitHub Copilot CLI is not authenticated."));
     console.log(chalk.yellow("   → Please complete Copilot login to continue context enrichment."));
     const login = spawnSync("copilot", ["login"], { stdio: "inherit", encoding: "utf-8", shell: process.platform === "win32" });
     if (login.status !== 0)
         return false;
-    return isCopilotAuthenticated(workspaceRoot);
+    return isCopilotAuthenticated();
 }
+// Maximum number of context.md files to enrich per init run.
+// Prevents extremely long waits on repos with many folders.
+const MAX_ENRICHMENT_FILES = 15;
 function enrichWithCopilotCli(workspaceRoot, contextFiles) {
     let updated = 0;
     let failed = 0;
-    for (const absPath of contextFiles) {
-        const relPath = absPath.replace(`${workspaceRoot}/`, "");
+    // Limit the number of files processed in one init run.
+    const filesToProcess = contextFiles.slice(0, MAX_ENRICHMENT_FILES);
+    const total = filesToProcess.length;
+    for (let i = 0; i < filesToProcess.length; i++) {
+        const absPath = filesToProcess[i];
+        const relPath = absPath.replace(`${workspaceRoot}/`, "").replace(`${workspaceRoot}\\`, "");
+        // Show per-file progress so the user knows work is happening.
+        process.stdout.write(`   [${i + 1}/${total}] Enriching ${relPath}…`);
         const prompt = [
             `Update the markdown file at path '${relPath}' in place.`,
             "Goal: make it both human-understandable and LLM-ready for automation framework reasoning.",
@@ -262,11 +283,18 @@ function enrichWithCopilotCli(workspaceRoot, contextFiles) {
             "--allow-all-paths",
             "--no-color",
             "-s",
-        ], { encoding: "utf-8", shell: process.platform === "win32" });
-        if (result.status === 0) {
+        ], {
+            encoding: "utf-8",
+            shell: process.platform === "win32",
+            timeout: 120_000, // 2 min per file — prevents indefinite hangs
+            input: "", // close stdin immediately
+        });
+        if (result.status === 0 && !result.error) {
+            process.stdout.write(" ✓\n");
             updated += 1;
         }
         else {
+            process.stdout.write(" ✗\n");
             failed += 1;
         }
     }
@@ -388,7 +416,13 @@ program
     }
     else {
         try {
-            execSync(`${getPythonBin(workspaceRoot)} -m pip install -q -r \"${reqPath}\"`, {
+            const pyBin = getPythonBin(workspaceRoot);
+            // Wrap both the python path and requirements path in quotes so
+            // Windows paths that contain spaces (e.g. "IGEL Automation July 2026")
+            // are treated as a single argument by the shell.
+            // --prefer-binary: use pre-built wheels instead of compiling from source
+            // (avoids Rust/Cargo requirement for packages like tokenizers, crawl4ai)
+            execSync(`"${pyBin}" -m pip install -q --prefer-binary -r "${reqPath}"`, {
                 cwd: workspaceRoot, stdio: "inherit",
             });
             console.log(chalk.green("   ✓ Dependencies installed"));
@@ -415,29 +449,41 @@ program
     else {
         console.log(chalk.green("   ✓ Framework context files already up to date"));
     }
-    // Step 2d — Copilot CLI enrichment pass (asks login if needed), then DB sync.
+    // Step 3d — Copilot CLI enrichment pass.
+    // Only run when --with-copilot / --with-all is explicitly requested.
+    // Skipped silently on plain `igel-qe init` to avoid unexpected long waits.
+    const runEnrichment = options.withCopilot || options.withAll;
     console.log(chalk.yellow("3d. Enriching context via GitHub Copilot CLI…"));
-    try {
-        const allContextFiles = listFrameworkContextFiles(workspaceRoot);
-        if (allContextFiles.length === 0) {
-            console.log(chalk.yellow("   ⚠ No context.md files found for Copilot enrichment"));
-        }
-        else if (!isCopilotCliAvailable()) {
-            console.log(chalk.yellow("   ⚠ Copilot CLI not installed; skipping Copilot enrichment"));
-        }
-        else if (!ensureCopilotLogin(workspaceRoot)) {
-            console.log(chalk.yellow("   ⚠ Copilot login was not completed; skipping Copilot enrichment"));
-        }
-        else {
-            const run = enrichWithCopilotCli(workspaceRoot, allContextFiles);
-            console.log(chalk.green(`   ✓ Copilot updated ${run.updated} context.md file(s)`));
-            if (run.failed > 0) {
-                console.log(chalk.yellow(`   ⚠ Copilot failed to update ${run.failed} context.md file(s)`));
+    if (!runEnrichment) {
+        console.log(chalk.gray("   - Skipped (pass --with-copilot to enable Copilot context enrichment)"));
+    }
+    else {
+        try {
+            const allContextFiles = listFrameworkContextFiles(workspaceRoot);
+            if (allContextFiles.length === 0) {
+                console.log(chalk.yellow("   ⚠ No context.md files found for Copilot enrichment"));
+            }
+            else if (!isCopilotCliAvailable()) {
+                console.log(chalk.yellow("   ⚠ Copilot CLI not installed; skipping Copilot enrichment"));
+            }
+            else if (!ensureCopilotLogin(workspaceRoot)) {
+                console.log(chalk.yellow("   ⚠ Copilot login was not completed; skipping Copilot enrichment"));
+            }
+            else {
+                const cap = Math.min(allContextFiles.length, MAX_ENRICHMENT_FILES);
+                if (allContextFiles.length > MAX_ENRICHMENT_FILES) {
+                    console.log(chalk.gray(`   - ${allContextFiles.length} context files found; enriching first ${MAX_ENRICHMENT_FILES} (re-run to continue)`));
+                }
+                const run = enrichWithCopilotCli(workspaceRoot, allContextFiles);
+                console.log(chalk.green(`   ✓ Copilot updated ${run.updated}/${cap} context.md file(s)`));
+                if (run.failed > 0) {
+                    console.log(chalk.yellow(`   ⚠ Copilot failed to update ${run.failed} context.md file(s)`));
+                }
             }
         }
-    }
-    catch (e) {
-        console.log(chalk.yellow(`   ⚠ Copilot context enrichment skipped: ${e instanceof Error ? e.message : String(e)}`));
+        catch (e) {
+            console.log(chalk.yellow(`   ⚠ Copilot context enrichment skipped: ${e instanceof Error ? e.message : String(e)}`));
+        }
     }
     // Step 3 — write MCP config (default: Copilot when no explicit platform flags)
     console.log(chalk.yellow("4. Configuring MCP server…"));

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "igel-qe-core",
-  "version": "1.0.12",
+  "version": "1.0.16",
   "description": "IGEL QE Developer Experience Layer (CLI & MCP)",
   "type": "module",
   "publishConfig": {
@@ -16,6 +16,7 @@
   },
   "scripts": {
     "build": "tsc",
+    "prepublishOnly": "tsc",
     "start:mcp": "node ./dist/mcp/server.js"
   },
   "dependencies": {

package/src/cli/index.ts

@@ -232,37 +232,65 @@ function listFrameworkContextFiles(workspaceRoot: string): string[] {
 }
 
 function isCopilotCliAvailable(): boolean {
-  const check = spawnSync("copilot", ["--version"], { encoding: "utf-8", shell: process.platform === "win32" });
-  return check.status === 0;
+  // Use a short timeout so this never blocks if copilot hangs on startup.
+  const check = spawnSync("copilot", ["--version"], {
+    encoding: "utf-8",
+    shell: process.platform === "win32",
+    timeout: 10_000,
+  });
+  return check.status === 0 && !check.error;
 }
 
-function isCopilotAuthenticated(workspaceRoot: string): boolean {
+function isCopilotAuthenticated(): boolean {
+  // Use a lightweight auth-status check instead of spawning a full agent session.
+  // "copilot auth status" exits 0 when authenticated, non-zero otherwise.
+  // Fallback: if the sub-command doesn't exist, treat as unauthenticated.
   const probe = spawnSync(
     "copilot",
-    ["-C", workspaceRoot, "-p", "Reply with exactly OK.", "--allow-all-tools", "--no-color", "-s"],
-    { encoding: "utf-8", shell: process.platform === "win32" },
+    ["auth", "status"],
+    {
+      encoding: "utf-8",
+      shell: process.platform === "win32",
+      timeout: 10_000,   // 10 s max — simple status call should be instant
+    },
   );
-  return probe.status === 0;
+  return probe.status === 0 && !probe.error;
 }
 
 function ensureCopilotLogin(workspaceRoot: string): boolean {
   if (!isCopilotCliAvailable()) return false;
-  if (isCopilotAuthenticated(workspaceRoot)) return true;
+  if (isCopilotAuthenticated()) return true;
 
   console.log(chalk.yellow("   ⚠ GitHub Copilot CLI is not authenticated."));
   console.log(chalk.yellow("   → Please complete Copilot login to continue context enrichment."));
 
   const login = spawnSync("copilot", ["login"], { stdio: "inherit", encoding: "utf-8", shell: process.platform === "win32" });
   if (login.status !== 0) return false;
-  return isCopilotAuthenticated(workspaceRoot);
+  return isCopilotAuthenticated();
 }
 
-function enrichWithCopilotCli(workspaceRoot: string, contextFiles: string[]): { updated: number; failed: number } {
+// Maximum number of context.md files to enrich per init run.
+// Prevents extremely long waits on repos with many folders.
+const MAX_ENRICHMENT_FILES = 15;
+
+function enrichWithCopilotCli(
+  workspaceRoot: string,
+  contextFiles: string[],
+): { updated: number; failed: number } {
   let updated = 0;
   let failed = 0;
 
-  for (const absPath of contextFiles) {
-    const relPath = absPath.replace(`${workspaceRoot}/`, "");
+  // Limit the number of files processed in one init run.
+  const filesToProcess = contextFiles.slice(0, MAX_ENRICHMENT_FILES);
+  const total = filesToProcess.length;
+
+  for (let i = 0; i < filesToProcess.length; i++) {
+    const absPath = filesToProcess[i];
+    const relPath = absPath.replace(`${workspaceRoot}/`, "").replace(`${workspaceRoot}\\`, "");
+
+    // Show per-file progress so the user knows work is happening.
+    process.stdout.write(`   [${i + 1}/${total}] Enriching ${relPath}…`);
+
     const prompt = [
       `Update the markdown file at path '${relPath}' in place.`,
       "Goal: make it both human-understandable and LLM-ready for automation framework reasoning.",
@@ -284,12 +312,19 @@ function enrichWithCopilotCli(workspaceRoot: string, contextFiles: string[]): {
         "--no-color",
         "-s",
       ],
-      { encoding: "utf-8", shell: process.platform === "win32" },
+      {
+        encoding: "utf-8",
+        shell: process.platform === "win32",
+        timeout: 120_000,  // 2 min per file — prevents indefinite hangs
+        input: "",         // close stdin immediately
+      },
     );
 
-    if (result.status === 0) {
+    if (result.status === 0 && !result.error) {
+      process.stdout.write(" ✓\n");
       updated += 1;
     } else {
+      process.stdout.write(" ✗\n");
       failed += 1;
     }
   }
@@ -418,7 +453,13 @@ program
       console.log(chalk.yellow("   ⚠ requirements file not found; skipping Python dependency install"));
     } else {
       try {
-        execSync(`${getPythonBin(workspaceRoot)} -m pip install -q -r \"${reqPath}\"`, {
+        const pyBin = getPythonBin(workspaceRoot);
+        // Wrap both the python path and requirements path in quotes so
+        // Windows paths that contain spaces (e.g. "IGEL Automation July 2026")
+        // are treated as a single argument by the shell.
+        // --prefer-binary: use pre-built wheels instead of compiling from source
+        // (avoids Rust/Cargo requirement for packages like tokenizers, crawl4ai)
+        execSync(`"${pyBin}" -m pip install -q --prefer-binary -r "${reqPath}"`, {
           cwd: workspaceRoot, stdio: "inherit",
         });
         console.log(chalk.green("   ✓ Dependencies installed"));
@@ -446,25 +487,36 @@ program
     }
 
 
-    // Step 2d — Copilot CLI enrichment pass (asks login if needed), then DB sync.
+    // Step 3d — Copilot CLI enrichment pass.
+    // Only run when --with-copilot / --with-all is explicitly requested.
+    // Skipped silently on plain `igel-qe init` to avoid unexpected long waits.
+    const runEnrichment = options.withCopilot || options.withAll;
     console.log(chalk.yellow("3d. Enriching context via GitHub Copilot CLI…"));
-    try {
-      const allContextFiles = listFrameworkContextFiles(workspaceRoot);
-      if (allContextFiles.length === 0) {
-        console.log(chalk.yellow("   ⚠ No context.md files found for Copilot enrichment"));
-      } else if (!isCopilotCliAvailable()) {
-        console.log(chalk.yellow("   ⚠ Copilot CLI not installed; skipping Copilot enrichment"));
-      } else if (!ensureCopilotLogin(workspaceRoot)) {
-        console.log(chalk.yellow("   ⚠ Copilot login was not completed; skipping Copilot enrichment"));
-      } else {
-        const run = enrichWithCopilotCli(workspaceRoot, allContextFiles);
-        console.log(chalk.green(`   ✓ Copilot updated ${run.updated} context.md file(s)`));
-        if (run.failed > 0) {
-          console.log(chalk.yellow(`   ⚠ Copilot failed to update ${run.failed} context.md file(s)`));
+    if (!runEnrichment) {
+      console.log(chalk.gray("   - Skipped (pass --with-copilot to enable Copilot context enrichment)"));
+    } else {
+      try {
+        const allContextFiles = listFrameworkContextFiles(workspaceRoot);
+        if (allContextFiles.length === 0) {
+          console.log(chalk.yellow("   ⚠ No context.md files found for Copilot enrichment"));
+        } else if (!isCopilotCliAvailable()) {
+          console.log(chalk.yellow("   ⚠ Copilot CLI not installed; skipping Copilot enrichment"));
+        } else if (!ensureCopilotLogin(workspaceRoot)) {
+          console.log(chalk.yellow("   ⚠ Copilot login was not completed; skipping Copilot enrichment"));
+        } else {
+          const cap = Math.min(allContextFiles.length, MAX_ENRICHMENT_FILES);
+          if (allContextFiles.length > MAX_ENRICHMENT_FILES) {
+            console.log(chalk.gray(`   - ${allContextFiles.length} context files found; enriching first ${MAX_ENRICHMENT_FILES} (re-run to continue)`));
+          }
+          const run = enrichWithCopilotCli(workspaceRoot, allContextFiles);
+          console.log(chalk.green(`   ✓ Copilot updated ${run.updated}/${cap} context.md file(s)`));
+          if (run.failed > 0) {
+            console.log(chalk.yellow(`   ⚠ Copilot failed to update ${run.failed} context.md file(s)`));
+          }
         }
+      } catch (e: unknown) {
+        console.log(chalk.yellow(`   ⚠ Copilot context enrichment skipped: ${e instanceof Error ? e.message : String(e)}`));
       }
-    } catch (e: unknown) {
-      console.log(chalk.yellow(`   ⚠ Copilot context enrichment skipped: ${e instanceof Error ? e.message : String(e)}`));
     }
 
     // Step 3 — write MCP config (default: Copilot when no explicit platform flags)