karajan-code 1.48.0 → 1.50.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "karajan-code",
-  "version": "1.48.0",
+  "version": "1.50.0",
   "description": "Local multi-agent coding orchestrator with TDD, SonarQube, and code review pipeline",
   "type": "module",
   "license": "AGPL-3.0",

package/src/agents/resolve-bin.js

@@ -26,7 +26,7 @@ function getNvmDirs() {
   const versionsDir = path.join(nvmDir, "versions", "node");
   try {
     return readdirSync(versionsDir).map(v => path.join(versionsDir, v, "bin"));
-  } catch {
+  } catch { /* nvm versions dir may not exist */
     return [];
   }
 }

package/src/audit/basal-cost.js

@@ -1,8 +1,11 @@
-import fs from "node:fs";
+import fs from "node:fs/promises";
 import path from "node:path";
-import { execFile, execFileSync } from "node:child_process";
+import { execFile } from "node:child_process";
+import { promisify } from "node:util";
 import { getKarajanHome } from "../utils/paths.js";
 
+const execFileAsync = promisify(execFile);
+
 const SOURCE_EXTENSIONS = new Set([
   ".js", ".mjs", ".cjs", ".ts", ".mts", ".cts", ".tsx", ".jsx",
   ".py", ".rb", ".go", ".rs", ".java", ".kt", ".swift", ".c", ".cpp", ".h",
@@ -18,11 +21,11 @@ function slugify(projectDir) {
   return path.basename(projectDir).replace(/[^a-zA-Z0-9_-]/g, "_");
 }
 
-function walkSourceFiles(dir) {
+async function walkSourceFiles(dir) {
   // Use git ls-files for speed (instant vs recursive walk)
   try {
-    const output = execFileSync("git", ["ls-files", "--cached", "--others", "--exclude-standard"], { cwd: dir, encoding: "utf8", timeout: 5000 });
-    return output.trim().split("\n")
+    const { stdout } = await execFileAsync("git", ["ls-files", "--cached", "--others", "--exclude-standard"], { cwd: dir, encoding: "utf8", timeout: 5000 });
+    return stdout.trim().split("\n")
       .filter(f => f && SOURCE_EXTENSIONS.has(path.extname(f)) && !EXCLUDE_DIRS.has(f.split("/")[0]))
       .map(f => path.join(dir, f));
   } catch {
@@ -30,15 +33,15 @@ function walkSourceFiles(dir) {
     const files = [];
     let entries;
     try {
-      entries = fs.readdirSync(dir, { withFileTypes: true });
-    } catch {
+      entries = await fs.readdir(dir, { withFileTypes: true });
+    } catch { /* directory not readable */
       return files;
     }
     for (const entry of entries) {
       if (EXCLUDE_DIRS.has(entry.name)) continue;
       const full = path.join(dir, entry.name);
       if (entry.isDirectory()) {
-        files.push(...walkSourceFiles(full));
+        files.push(...await walkSourceFiles(full));
       } else if (SOURCE_EXTENSIONS.has(path.extname(entry.name))) {
         files.push(full);
       }
@@ -47,19 +50,19 @@ function walkSourceFiles(dir) {
   }
 }
 
-function countLines(filePath) {
+async function countLines(filePath) {
   try {
-    const content = fs.readFileSync(filePath, "utf8");
+    const content = await fs.readFile(filePath, "utf8");
     return content.split("\n").length;
   } catch {
     return 0;
   }
 }
 
-function countDependencies(projectDir) {
+async function countDependencies(projectDir) {
   const pkgPath = path.join(projectDir, "package.json");
   try {
-    const pkg = JSON.parse(fs.readFileSync(pkgPath, "utf8"));
+    const pkg = JSON.parse(await fs.readFile(pkgPath, "utf8"));
     const deps = Object.keys(pkg.dependencies || {});
     const devDeps = Object.keys(pkg.devDependencies || {});
     return { dependencies: deps.length, devDependencies: devDeps.length, total: deps.length + devDeps.length };
@@ -89,7 +92,7 @@ function runDepcheck(projectDir) {
   });
 }
 
-function findDeadExports(sourceFiles) {
+async function findDeadExports(sourceFiles) {
   const exportMap = new Map(); // file -> [exportName]
   const importedNames = new Set();
 
@@ -101,7 +104,7 @@ function findDeadExports(sourceFiles) {
   for (const file of sourceFiles) {
     let content;
     try {
-      content = fs.readFileSync(file, "utf8");
+      content = await fs.readFile(file, "utf8");
     } catch {
       continue;
     }
@@ -139,15 +142,15 @@ function findDeadExports(sourceFiles) {
 }
 
 export async function measureBasalCost(projectDir) {
-  const sourceFiles = walkSourceFiles(projectDir);
+  const sourceFiles = await walkSourceFiles(projectDir);
   let totalLines = 0;
   for (const f of sourceFiles) {
-    totalLines += countLines(f);
+    totalLines += await countLines(f);
   }
 
-  const depInfo = countDependencies(projectDir);
+  const depInfo = await countDependencies(projectDir);
   const depcheck = await runDepcheck(projectDir);
-  const deadExports = findDeadExports(sourceFiles);
+  const deadExports = await findDeadExports(sourceFiles);
 
   return {
     totalLines,
@@ -162,20 +165,20 @@ function auditDir() {
   return path.join(getKarajanHome(), "audits");
 }
 
-export function loadPreviousAudit(projectDir) {
+export async function loadPreviousAudit(projectDir) {
   const file = path.join(auditDir(), slugify(projectDir), "latest.json");
   try {
-    return JSON.parse(fs.readFileSync(file, "utf8"));
+    return JSON.parse(await fs.readFile(file, "utf8"));
   } catch {
     return null;
   }
 }
 
-export function saveAuditSnapshot(projectDir, metrics) {
+export async function saveAuditSnapshot(projectDir, metrics) {
   const dir = path.join(auditDir(), slugify(projectDir));
-  fs.mkdirSync(dir, { recursive: true });
+  await fs.mkdir(dir, { recursive: true });
   const snapshot = { ...metrics, timestamp: new Date().toISOString() };
-  fs.writeFileSync(path.join(dir, "latest.json"), JSON.stringify(snapshot, null, 2));
+  await fs.writeFile(path.join(dir, "latest.json"), JSON.stringify(snapshot, null, 2));
   return snapshot;
 }
 

package/src/bootstrap.js

@@ -38,7 +38,7 @@ async function checkGitRepo() {
   let ok = false;
   try {
     ok = await ensureGitRepo();
-  } catch {
+  } catch { /* git may not be installed */
     ok = false;
   }
   return {
@@ -156,7 +156,7 @@ async function readBootstrapFile(projectDir) {
   try {
     const raw = await fs.readFile(bootstrapPath(projectDir), "utf8");
     return JSON.parse(raw);
-  } catch {
+  } catch { /* bootstrap file may not exist or contain invalid JSON */
     return null;
   }
 }

package/src/commands/agents.js

@@ -51,7 +51,7 @@ export async function setAgent(role, provider, { global: isGlobal = false } = {}
     const { setSessionOverride } = await import("../mcp/preflight.js");
     setSessionOverride(role, provider);
     return { role, provider, scope: "session" };
-  } catch {
+  } catch { /* preflight module not available in CLI mode */
     // preflight module not available (CLI mode) — write to project config
     const projectConfigPath = getProjectConfigPath();
     const projectConfig = (await loadProjectConfig()) || {};

package/src/commands/board.js

@@ -10,7 +10,7 @@ function readPid() {
   try {
     const pid = Number.parseInt(fs.readFileSync(PID_FILE, "utf8").trim(), 10);
     return Number.isFinite(pid) ? pid : null;
-  } catch {
+  } catch { /* PID file does not exist */
     return null;
   }
 }
@@ -19,7 +19,7 @@ function isProcessAlive(pid) {
   try {
     process.kill(pid, 0);
     return true;
-  } catch {
+  } catch { /* process does not exist */
     return false;
   }
 }
@@ -55,7 +55,7 @@ export async function stopBoard() {
 
   try {
     process.kill(pid, "SIGTERM");
-  } catch {
+  } catch { /* process already dead */
     // already dead
   }
   try { fs.unlinkSync(PID_FILE); } catch { /* ignore */ }

package/src/commands/code.js

@@ -13,7 +13,7 @@ export async function codeCommand({ task, config, logger }) {
   if (config.coder_rules) {
     try {
       coderRules = await fs.readFile(config.coder_rules, "utf8");
-    } catch {
+    } catch { /* configured coder_rules path not found — try fallback */
       try { coderRules = await fs.readFile("coder-rules.md", "utf8"); } catch { /* no coder rules file */ }
     }
   }

package/src/commands/doctor.js

@@ -44,7 +44,7 @@ async function checkGitRepo() {
   let gitOk = false;
   try {
     gitOk = await ensureGitRepo();
-  } catch {
+  } catch { /* git check failed */
     gitOk = false;
   }
   return {
@@ -79,7 +79,7 @@ async function checkSonarQube(config) {
   if (config.sonarqube?.enabled !== false) {
     try {
       sonarOk = await isSonarReachable(sonarHost);
-    } catch {
+    } catch { /* sonar check failed */
       sonarOk = false;
     }
   }
@@ -130,7 +130,7 @@ async function checkSerena() {
   try {
     const serenaCheck = await runCommand("serena", ["--version"]);
     serenaOk = serenaCheck.exitCode === 0;
-  } catch {
+  } catch { /* serena not installed */
     serenaOk = false;
   }
   return {
@@ -181,7 +181,7 @@ async function checkBecariaSecrets() {
       detail: secretsOk ? "BECARIA_APP_ID + BECARIA_APP_PRIVATE_KEY found" : `Missing: ${missing}`,
       fix: secretsOk ? null : "Add BECARIA_APP_ID and BECARIA_APP_PRIVATE_KEY as GitHub repository secrets"
     };
-  } catch {
+  } catch { /* GitHub API error */
     return null;
   }
 }

package/src/commands/init.js

@@ -163,7 +163,7 @@ async function ensureCoderRules(coderRulesPath, logger) {
   let content;
   try {
     content = await fs.readFile(templatePath, "utf8");
-  } catch {
+  } catch { /* template not found, use inline default */
     content = [
       "# Coder Rules",
       "",
@@ -333,7 +333,7 @@ export async function initCommand({ logger, flags = {} }) {
   try {
     const rtkRes = await runCommand("rtk", ["--version"]);
     hasRtk = rtkRes.exitCode === 0;
-  } catch {
+  } catch { /* rtk not installed */
     hasRtk = false;
   }
   if (!hasRtk) {

package/src/commands/report.js

@@ -312,7 +312,7 @@ async function findSessionsByPgTask(dir, pgTask) {
       if (session.pg_task_id === pgTask) {
         matches.push(entry);
       }
-    } catch {
+    } catch { /* skip malformed session files */
       // skip malformed sessions
     }
   }

package/src/commands/review.js

@@ -53,7 +53,7 @@ export async function reviewCommand({ task, config, logger, baseRef }) {
       let review;
       try {
         review = JSON.parse(result.output);
-      } catch {
+      } catch { /* output is not JSON */
         review = { approved: true, summary: result.output };
       }
 

package/src/commands/roles.js

@@ -60,7 +60,7 @@ export async function showRole(roleName, config) {
     try {
       await fs.readFile(candidates[1], "utf8");
       source = "user";
-    } catch {
+    } catch { /* user role file not found */
       source = "built-in";
     }
   }

package/src/hu/store.js

@@ -1,6 +1,9 @@
 import fs from "node:fs/promises";
 import path from "node:path";
-import { execSync } from "node:child_process";
+import { execFile } from "node:child_process";
+import { promisify } from "node:util";
+
+const execFileAsync = promisify(execFile);
 import { getKarajanHome } from "../utils/paths.js";
 
 // FUTURE: hu-storage adapter for PG/Trello/etc — currently local files only
@@ -237,13 +240,14 @@ export function addSplittingMetadata(batch, huId, metadata) {
 /**
  * Detect project info from the given directory.
  * @param {string} cwd - Directory to detect project from.
- * @returns {{ name: string, remoteUrl: string|null }}
+ * @returns {Promise<{ name: string, remoteUrl: string|null }>}
  */
-export function detectProject(cwd) {
+export async function detectProject(cwd) {
   const name = path.basename(cwd);
   let remoteUrl = null;
   try {
-    remoteUrl = execSync("git remote get-url origin", { cwd, encoding: "utf8", stdio: ["pipe", "pipe", "pipe"] }).trim();
+    const { stdout } = await execFileAsync("git", ["remote", "get-url", "origin"], { cwd, encoding: "utf8" });
+    remoteUrl = stdout.trim();
   } catch { /* not a git repo or no remote */ }
   return { name, remoteUrl };
 }
@@ -276,7 +280,7 @@ async function loadProjectBatch(projectDir) {
   try {
     const raw = await fs.readFile(filePath, "utf8");
     return JSON.parse(raw);
-  } catch {
+  } catch { /* no saved batch for this project */
     return {
       project: projectSlug(projectDir),
       created_at: new Date().toISOString(),