karajan-code 1.59.0 → 1.59.1

package/package.json

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

package/src/agents/claude-agent.js

@@ -1,7 +1,6 @@
 import { BaseAgent } from "./base-agent.js";
 import { runCommand } from "../utils/process.js";
 import { resolveBin } from "./resolve-bin.js";
-import { getProxyEnv } from "../proxy/proxy-lifecycle.js";
 
 /**
  * Safely parse a JSON line, returning null on failure.
@@ -12,6 +11,37 @@ function tryParseJson(line) {
   } catch { return null; }
 }
 
+/**
+ * Extract a human-readable error message from Claude's raw NDJSON stderr output.
+ * Claude emits system init, api_retry, assistant error, and result lines.
+ * We extract just the meaningful error text, discarding the JSON noise.
+ */
+export function sanitizeClaudeError(raw) {
+  if (!raw) return "Unknown error";
+  const lines = raw.split("\n").filter(Boolean);
+
+  // Look for a result line with the actual error
+  for (let i = lines.length - 1; i >= 0; i--) {
+    const obj = tryParseJson(lines[i]);
+    if (!obj) continue;
+    if (obj.type === "result" && obj.result) return String(obj.result);
+    if (obj.type === "assistant" && obj.message?.content) {
+      const text = obj.message.content
+        .filter(b => b.type === "text" && b.text)
+        .map(b => b.text)
+        .join(" ");
+      if (text) return text;
+    }
+  }
+
+  // If no JSON parsed, return first non-JSON line or truncated raw
+  for (const line of lines) {
+    if (!line.startsWith("{")) return line.slice(0, 200);
+  }
+
+  return raw.slice(0, 200);
+}
+
 /**
  * Try to extract a result string from a parsed JSON object.
  * Returns the result string or null if the object is not a result message.
@@ -135,8 +165,9 @@ function createStreamJsonFilter(onOutput) {
  */
 function cleanExecaOpts(extra = {}) {
   const { CLAUDECODE, ...env } = process.env;
-  const proxyEnv = getProxyEnv();
-  if (proxyEnv) Object.assign(env, proxyEnv);
+  // NOTE: Claude Code CLI does not respect ANTHROPIC_BASE_URL — proxy env
+  // injection is skipped for claude to avoid ECONNREFUSED errors.
+  // Other agents (codex, gemini, aider) use SDKs that do respect base URL vars.
   return { env, stdin: "ignore", ...extra };
 }
 
@@ -197,7 +228,7 @@ export class ClaudeAgent extends BaseAgent {
       const raw = pickOutput(res);
       const output = extractTextFromStreamJson(raw);
       const usage = extractUsageFromStreamJson(raw);
-      return { ok: res.exitCode === 0, output, error: res.exitCode === 0 ? "" : raw, exitCode: res.exitCode, ...usage };
+      return { ok: res.exitCode === 0, output, error: res.exitCode === 0 ? "" : sanitizeClaudeError(raw), exitCode: res.exitCode, ...usage };
     }
 
     // Without streaming, use json output to get structured response via stderr
@@ -206,7 +237,7 @@ export class ClaudeAgent extends BaseAgent {
     const raw = pickOutput(res);
     const output = extractTextFromStreamJson(raw);
     const usage = extractUsageFromStreamJson(raw);
-    return { ok: res.exitCode === 0, output, error: res.exitCode === 0 ? "" : raw, exitCode: res.exitCode, ...usage };
+    return { ok: res.exitCode === 0, output, error: res.exitCode === 0 ? "" : sanitizeClaudeError(raw), exitCode: res.exitCode, ...usage };
   }
 
   async _reviewTaskExec(task, model) {

package/src/cli.js

@@ -127,6 +127,7 @@ program
   .option("--smart-models", "Enable smart model selection based on triage complexity")
   .option("--no-smart-models", "Disable smart model selection")
   .option("--design", "Activate design refactoring mode (impeccable role applies design changes)")
+  .option("--domain <text-or-path>", "Domain knowledge: inline text or path to .md file")
   .option("--dry-run", "Show what would be executed without running anything")
   .option("--json", "Output JSON only (no styled display)")
   .option("-q, --quiet", "Show only stage status lines, suppress raw agent output (default)")

package/src/domains/domain-loader.js

@@ -5,8 +5,8 @@
  * Project-local domains override user-global domains by directory name.
  */
 
-import { readdir, readFile } from "node:fs/promises";
-import { join } from "node:path";
+import { readdir, readFile, mkdir, writeFile, stat } from "node:fs/promises";
+import { join, isAbsolute } from "node:path";
 import yaml from "js-yaml";
 import { getKarajanHome } from "../utils/paths.js";
 
@@ -103,8 +103,52 @@ function parseSections(content) {
   return sections;
 }
 
+/**
+ * Auto-detect project context files (README.md, CLAUDE.md) as lightweight domains.
+ * Used as fallback when no explicit .karajan/domains/ are configured.
+ *
+ * @param {string} projectDir — absolute path to project root
+ * @returns {Promise<DomainFile[]>}
+ */
+export async function autoDetectDomains(projectDir) {
+  if (!projectDir) return [];
+
+  const candidates = ["README.md", "CLAUDE.md", "docs/README.md"];
+  const detected = [];
+
+  for (const rel of candidates) {
+    const filePath = join(projectDir, rel);
+    let content;
+    try {
+      content = await readFile(filePath, "utf-8");
+    } catch { continue; }
+
+    if (!content || content.trim().length < 20) continue;
+
+    // Truncate to first 4000 chars to avoid bloating context
+    const trimmed = content.length > 4000 ? content.slice(0, 4000) + "\n\n[...truncated]" : content;
+
+    detected.push({
+      name: `auto:${rel}`,
+      description: `Auto-detected from ${rel}`,
+      tags: [],
+      version: "0.0.0",
+      author: "",
+      visibility: "private",
+      sources: [{ type: "auto-detect", note: rel }],
+      content: trimmed,
+      sections: parseSections(trimmed),
+      filePath,
+      origin: "auto-detect"
+    });
+  }
+
+  return detected;
+}
+
 /**
  * Load all domains from project-local and user-global directories.
+ * Falls back to auto-detecting README.md/CLAUDE.md when no explicit domains found.
  * Project-local domains override user-global domains when directory names match.
  *
  * @param {string|null} projectDir — absolute path to project root (null = user-global only)
@@ -119,7 +163,6 @@ export async function loadDomains(projectDir) {
   const projectDomains = projectDomainDir ? await scanDomainDir(projectDomainDir, "project") : [];
 
   // Build a map keyed by directory name for merge.
-  // We track by directory name (not domain name) because the override is by directory.
   const merged = new Map();
 
   for (const { dirName, domain } of userDomains) {
@@ -129,7 +172,14 @@ export async function loadDomains(projectDir) {
     merged.set(dirName, domain); // project overrides user
   }
 
-  return Array.from(merged.values());
+  const explicit = Array.from(merged.values());
+
+  // Fallback: auto-detect README.md, CLAUDE.md when no explicit domains found
+  if (explicit.length === 0 && projectDir) {
+    return autoDetectDomains(projectDir);
+  }
+
+  return explicit;
 }
 
 /**
@@ -162,3 +212,34 @@ async function scanDomainDir(dir, origin) {
 
   return results;
 }
+
+/**
+ * Persist an inline domain (text or file path) to .karajan/domains/inline/DOMAIN.md.
+ * If domainInput is a path to an existing .md file, reads its content.
+ * Otherwise treats it as inline text.
+ *
+ * @param {string} domainInput — inline text or absolute path to a .md file
+ * @param {string} projectDir — project root
+ */
+export async function persistInlineDomain(domainInput, projectDir) {
+  if (!domainInput || !projectDir) return;
+
+  let content = domainInput;
+
+  // If it looks like a file path, try to read it
+  if (isAbsolute(domainInput) && domainInput.endsWith(".md")) {
+    try {
+      const s = await stat(domainInput);
+      if (s.isFile()) {
+        content = await readFile(domainInput, "utf-8");
+      }
+    } catch { /* treat as inline text */ }
+  }
+
+  const domainDir = join(projectDir, ".karajan", "domains", "inline");
+  await mkdir(domainDir, { recursive: true });
+
+  const domainFile = join(domainDir, DOMAIN_FILE);
+  const wrapped = `---\nname: inline-domain\ndescription: Domain knowledge provided via --domain parameter\ntags: []\nversion: "0.0.0"\n---\n\n${content}\n`;
+  await writeFile(domainFile, wrapped, "utf-8");
+}

package/src/mcp/sovereignty-guard.js

@@ -24,7 +24,7 @@ const ALLOWED_PARAMS = new Set([
   "autoCommit", "autoPush", "autoPr", "autoRebase", "branchPrefix",
   "autoSimplify", "smartModels", "checkpointInterval",
   "taskType", "quiet", "noSonar", "enableSonarcloud",
-  "kjHome", "sonarToken", "timeoutMs",
+  "kjHome", "sonarToken", "timeoutMs", "domain",
   // Sovereign flags — validated below but still "known"
   "enableTriage", "enableHuReviewer",
 ]);

package/src/mcp/tools.js

@@ -104,7 +104,8 @@ export const tools = [
         enableSonarcloud: { type: "boolean", description: "Enable SonarCloud scan (complementary to SonarQube)" },
         kjHome: { type: "string" },
         sonarToken: { type: "string" },
-        timeoutMs: { type: "number" }
+        timeoutMs: { type: "number" },
+        domain: { type: "string", description: "Domain knowledge: inline text describing the project domain, or absolute path to a .md file. Auto-saved to .karajan/domains/ for the curator." }
       }
     }
   },

package/src/orchestrator.js

@@ -35,6 +35,7 @@ import { msg, getLang } from "./utils/messages.js";
 import { PipelineContext } from "./orchestrator/pipeline-context.js";
 import { runTriageStage, runResearcherStage, runArchitectStage, runPlannerStage, runDiscoverStage, runHuReviewerStage } from "./orchestrator/pre-loop-stages.js";
 import { runDomainCuratorStage } from "./orchestrator/stages/domain-curator-stage.js";
+import { persistInlineDomain } from "./domains/domain-loader.js";
 import { runCoderStage, runRefactorerStage, runTddCheckStage, runSonarStage, runSonarCloudStage, runReviewerStage } from "./orchestrator/iteration-stages.js";
 import { runTesterStage, runSecurityStage, runImpeccableStage, runFinalAuditStage } from "./orchestrator/post-loop-stages.js";
 import { needsSubPipeline, runHuSubPipeline } from "./orchestrator/hu-sub-pipeline.js";
@@ -940,6 +941,15 @@ async function runPreLoopStages({ config, logger, emitter, eventBase, session, f
     logger.warn(`Skill auto-install failed (non-blocking): ${err.message}`);
   }
 
+  // --- Persist inline domain if provided via --domain flag ---
+  if (flags?.domain) {
+    try {
+      await persistInlineDomain(flags.domain, config.projectDir || process.cwd());
+    } catch (err) {
+      logger.warn(`Failed to persist inline domain: ${err.message}`);
+    }
+  }
+
   // --- Domain Curator (after triage + skill auto-install, before planning phases) ---
   const domainHints = triageResult.stageResult?.domainHints || [];
   if (domainHints.length > 0 || config.projectDir) {

package/src/proxy/proxy-lifecycle.js

@@ -144,6 +144,9 @@ export function getProxyEnv() {
 
   const base = `http://127.0.0.1:${proxyPort}`;
   return {
+    // SDK-specific base URL vars — used by SDKs that respect them.
+    // Claude Code CLI does NOT respect ANTHROPIC_BASE_URL, so claude-agent
+    // must skip proxy injection until upstream support is confirmed.
     ANTHROPIC_BASE_URL: base,
     OPENAI_BASE_URL: base,
     GEMINI_API_BASE: base,

package/src/roles/impeccable-role.js

@@ -1,6 +1,7 @@
 import { BaseRole } from "./base-role.js";
 import { createAgent as defaultCreateAgent } from "../agents/index.js";
 import { isDevToolsMcpAvailable, ensureWebPerfSkills } from "../webperf/devtools-detect.js";
+import { extractFirstJson } from "../utils/json-extract.js";
 
 const SUBAGENT_PREAMBLE = [
   "IMPORTANT: You are running as a Karajan sub-agent.",
@@ -35,10 +36,7 @@ function buildPrompt({ task, diff, instructions }) {
 }
 
 function parseImpeccableOutput(raw) {
-  const text = raw?.trim() || "";
-  const jsonMatch = /\{[\s\S]*\}/.exec(text);
-  if (!jsonMatch) return null;
-  return JSON.parse(jsonMatch[0]);
+  return extractFirstJson(raw);
 }
 
 function buildSummary(parsed) {

package/src/roles/researcher-role.js

@@ -1,5 +1,6 @@
 import { BaseRole } from "./base-role.js";
 import { createAgent as defaultCreateAgent } from "../agents/index.js";
+import { extractFirstJson } from "../utils/json-extract.js";
 
 const SUBAGENT_PREAMBLE = [
   "IMPORTANT: You are running as a Karajan sub-agent.",
@@ -34,10 +35,7 @@ function buildPrompt({ task, instructions }) {
 }
 
 function parseResearchOutput(raw) {
-  const text = raw?.trim() || "";
-  const jsonMatch = /\{[\s\S]*\}/.exec(text);
-  if (!jsonMatch) return null;
-  return JSON.parse(jsonMatch[0]);
+  return extractFirstJson(raw);
 }
 
 function buildSummary(parsed) {

package/src/roles/reviewer-role.js

@@ -1,6 +1,7 @@
 import { BaseRole } from "./base-role.js";
 import { createAgent as defaultCreateAgent } from "../agents/index.js";
 import { buildRtkInstructions } from "../prompts/rtk-snippet.js";
+import { extractFirstJson } from "../utils/json-extract.js";
 
 const MAX_DIFF_LENGTH = 12000;
 
@@ -65,12 +66,11 @@ function buildPrompt({ task, diff, reviewRules, reviewMode, instructions, rtkAva
 }
 
 function parseReviewOutput(raw) {
-  const text = raw?.trim() || "";
-  const jsonMatch = /\{[\s\S]*\}/.exec(text);
-  if (!jsonMatch) {
+  const parsed = extractFirstJson(raw);
+  if (!parsed) {
     throw new Error(`Failed to parse reviewer output: no JSON found`);
   }
-  return JSON.parse(jsonMatch[0]);
+  return parsed;
 }
 
 export class ReviewerRole extends BaseRole {

package/src/roles/solomon-role.js

@@ -1,5 +1,6 @@
 import { BaseRole } from "./base-role.js";
 import { createAgent as defaultCreateAgent } from "../agents/index.js";
+import { extractFirstJson } from "../utils/json-extract.js";
 
 const SUBAGENT_PREAMBLE = [
   "IMPORTANT: You are running as a Karajan sub-agent.",
@@ -98,10 +99,7 @@ function buildPrompt({ conflict, task, instructions }) {
 }
 
 function parseSolomonOutput(raw) {
-  const text = raw?.trim() || "";
-  const jsonMatch = /\{[\s\S]*\}/.exec(text);
-  if (!jsonMatch) return null;
-  return JSON.parse(jsonMatch[0]);
+  return extractFirstJson(raw);
 }
 
 function buildSummary(parsed) {

package/src/utils/banner.js

@@ -0,0 +1,66 @@
+/**
+ * Karajan Code ASCII banner with brand colors.
+ *
+ * Brand palette:
+ *   #AACCEE  steel blue (primary text)
+ *   #88CC88  green (accents, robot eyes)
+ *   #464000  dark olive
+ *   #444411  charcoal (secondary text)
+ */
+
+const C = {
+  blue:     "\x1b[38;2;170;204;238m",  // #AACCEE
+  green:    "\x1b[38;2;100;190;100m",   // green accents
+  gray:     "\x1b[38;2;140;140;140m",   // borders/dim
+  white:    "\x1b[38;2;220;220;220m",   // bright text
+  charcoal: "\x1b[38;2;100;100;80m",    // #444411 approx
+  bold:     "\x1b[1m",
+  dim:      "\x1b[2m",
+  reset:    "\x1b[0m"
+};
+
+const R = C.reset;
+
+const BANNER_LINES = [
+  ``,
+  `${C.gray}        ╭───────────╮`,
+  `${C.gray}        │  ${C.green}~${C.gray}        │`,
+  `${C.gray}        │  ${C.green}|${C.gray}     ${C.green}|${C.gray}  │  ${C.charcoal}|`,
+  `${C.gray}        |     ${C.green}v${C.gray}     │ ${C.charcoal}/`,
+  `${C.gray}      ${C.charcoal}\\${C.gray} ╰───────────╯${C.charcoal}/`,
+  `${C.charcoal}       \\${C.gray} ╭───${C.white}⋈${C.gray}───╮ ${C.charcoal}/`,
+  `${C.charcoal}        \\${C.gray}        ${C.charcoal}/`,
+  `${C.gray}         ╰───────╯`,
+  ``,
+  `  ${C.bold}${C.blue}╦╔═ ╔═╗ ╦═╗ ╔═╗  ╦ ╔═╗ ╔╗╔${R}`,
+  `  ${C.bold}${C.blue}╠╩╗ ╠═╣ ╠╦╝ ╠═╣  ║ ╠═╣ ║║║${R}`,
+  `  ${C.bold}${C.blue}╩ ╩ ╩ ╩ ╩╚═ ╩ ╩ ╚╝ ╩ ╩ ╝╚╝${R}`,
+  `  ${C.bold}${C.white}        C O D E${R}  %%VERSION%%`,
+  `  ${C.charcoal}multiagent coding orchestrator${R}`,
+  ``,
+];
+
+/**
+ * Print the Karajan Code ASCII banner.
+ * @param {string} version - Package version to display
+ * @param {object} [opts]
+ * @param {boolean} [opts.compact] - Skip mascot, show only text
+ * @param {boolean} [opts.force] - Print even without TTY
+ */
+export function printBanner(version, opts = {}) {
+  if (!opts.force && !process.stdout.isTTY) return;
+
+  const versionTag = `${C.dim}v${version}${R}`;
+
+  if (opts.compact) {
+    console.log();
+    for (const line of BANNER_LINES.slice(10)) {
+      console.log(line.replace("%%VERSION%%", versionTag));
+    }
+    return;
+  }
+
+  for (const line of BANNER_LINES) {
+    console.log(line.replace("%%VERSION%%", versionTag));
+  }
+}

package/src/utils/display.js

@@ -1,6 +1,7 @@
 import path from "node:path";
 import { readFileSync } from "node:fs";
 import { fileURLToPath } from "node:url";
+import { printBanner } from "./banner.js";
 
 // TODO: i18n display messages
 const DISPLAY_PKG_PATH = path.resolve(path.dirname(fileURLToPath(import.meta.url)), "../../package.json");
@@ -83,9 +84,7 @@ const BAR = `${ANSI.dim}\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u
 
 export function printHeader({ task, config }) {
   const version = DISPLAY_VERSION;
-  console.log(BAR);
-  console.log(`${ANSI.bold}${ANSI.cyan}\u25b6 Karajan Code v${version}${ANSI.reset}`);
-  console.log(BAR);
+  printBanner(version);
   console.log(`${ANSI.bold}Task:${ANSI.reset} ${task}`);
   console.log(
     `${ANSI.bold}Coder:${ANSI.reset} ${config.roles?.coder?.provider || config.coder} ${ANSI.dim}|${ANSI.reset} ${ANSI.bold}Reviewer:${ANSI.reset} ${config.roles?.reviewer?.provider || config.reviewer}`

package/src/utils/welcome.js

@@ -1,3 +1,5 @@
+import { printBanner } from "./banner.js";
+
 const A = {
   reset:   "\x1b[0m",
   bold:    "\x1b[1m",
@@ -7,8 +9,6 @@ const A = {
   yellow:  "\x1b[33m",
 };
 
-const BAR = `${A.dim}${"─".repeat(44)}${A.reset}`;
-
 const QUICK_START = [
   ["kj run <task>",  "Run the full coder+reviewer pipeline"],
   ["kj init",        "Initialize config and setup"],
@@ -24,12 +24,7 @@ const QUICK_START = [
  * @param {object} [opts.config]  - Loaded KJ config (optional, shows configured agents)
  */
 export function printWelcomeScreen({ version, config = null }) {
-  console.log();
-  console.log(BAR);
-  console.log(`  ${A.bold}${A.cyan}▶ Karajan Code${A.reset}  ${A.dim}v${version}${A.reset}`);
-  console.log(`  ${A.dim}AI-powered coding pipeline${A.reset}`);
-  console.log(BAR);
-  console.log();
+  printBanner(version, { force: true });
 
   if (config) {
     const coder    = config.roles?.coder?.provider    || config.coder    || "claude";