karajan-code 1.53.0 → 1.54.0

package/README.md

@@ -258,7 +258,7 @@ Not nostalgia, not stubbornness. I've been using JavaScript since 1997, when Bre
 git clone https://github.com/manufosela/karajan-code.git
 cd karajan-code
 npm install
-npm test              # Run ~2473 tests with Vitest
+npm test              # Run ~2599 tests with Vitest
 npm run validate      # Lint + test
 ```
 

package/docs/README.es.md

@@ -231,7 +231,7 @@ No es nostalgia ni cabezoneria. Es que llevo usando JavaScript desde 1997, cuand
 git clone https://github.com/manufosela/karajan-code.git
 cd karajan-code
 npm install
-npm test              # Ejecutar ~2473 tests con Vitest
+npm test              # Ejecutar ~2599 tests con Vitest
 npm run validate      # Lint + test
 ```
 

package/package.json

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

package/src/cli.js

@@ -117,6 +117,7 @@ program
   .option("--no-auto-simplify", "Force full pipeline regardless of triage level")
   .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("--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/mcp/response-compressor.js

@@ -0,0 +1,81 @@
+/**
+ * MCP response compressor — reduces token consumption by stripping
+ * verbose fields from list payloads and truncating oversized arrays.
+ */
+
+// Fields to strip from list items (keep in single-item detail views)
+const STRIP_FROM_LIST_ITEMS = [
+  "descriptionStructured", "acceptanceCriteriaStructured",
+  "implementationPlan", "implementationNotes",
+  "developmentInstructions", "raw", "textSummary",
+  "workCycles", "config_snapshot", "paused_state",
+  "checkpoints"
+];
+
+// Fields to strip from all responses (never useful to the LLM)
+const ALWAYS_STRIP = [
+  "firebaseId", "cardType", "group", "createdBy", "updatedBy",
+  "_instance"
+];
+
+const MAX_ARRAY_ITEMS = 20;
+const MAX_COMMITS = 5;
+const MAX_FINDINGS = 10;
+
+export function compressResponse(data, toolName = "") {
+  if (!data || typeof data !== "object") return data;
+  if (Array.isArray(data)) return compressArray(data);
+
+  const compressed = { ...data };
+
+  // Strip always-remove fields
+  for (const key of ALWAYS_STRIP) delete compressed[key];
+
+  // Compress nested arrays and objects
+  for (const [key, value] of Object.entries(compressed)) {
+    if (Array.isArray(value)) {
+      if (key === "commits") compressed[key] = truncateFromEnd(value, MAX_COMMITS);
+      else if (key === "findings" || key === "blocking_issues") compressed[key] = truncateFromStart(value, MAX_FINDINGS);
+      else compressed[key] = compressArray(value);
+    } else if (value && typeof value === "object") {
+      compressed[key] = compressResponse(value, toolName);
+    }
+  }
+
+  return compressed;
+}
+
+function compressArray(arr) {
+  if (!Array.isArray(arr) || arr.length === 0) return arr;
+
+  // Strip verbose fields from list items
+  const stripped = arr.map(item => {
+    if (!item || typeof item !== "object") return item;
+    const clean = { ...item };
+    for (const key of STRIP_FROM_LIST_ITEMS) delete clean[key];
+    for (const key of ALWAYS_STRIP) delete clean[key];
+    return clean;
+  });
+
+  if (stripped.length <= MAX_ARRAY_ITEMS) return stripped;
+
+  const truncated = stripped.slice(0, MAX_ARRAY_ITEMS);
+  truncated.push({ _truncated: true, _total: arr.length, _showing: MAX_ARRAY_ITEMS });
+  return truncated;
+}
+
+function truncateFromEnd(arr, max) {
+  if (arr.length <= max) return arr;
+  return [{ _note: `${arr.length - max} earlier items omitted` }, ...arr.slice(-max)];
+}
+
+function truncateFromStart(arr, max) {
+  if (arr.length <= max) return arr;
+  const truncated = arr.slice(0, max);
+  truncated.push({ _note: `... and ${arr.length - max} more` });
+  return truncated;
+}
+
+export function compactStringify(data) {
+  return JSON.stringify(data);
+}

package/src/mcp/run-kj.js

@@ -43,6 +43,7 @@ export async function runKjCommand({ command, commandArgs = [], options = {}, en
   normalizeBoolFlag(options.enableTester, "--enable-tester", args);
   normalizeBoolFlag(options.enableSecurity, "--enable-security", args);
   normalizeBoolFlag(options.enableImpeccable, "--enable-impeccable", args);
+  normalizeBoolFlag(options.design, "--design", args);
   normalizeBoolFlag(options.enableTriage, "--enable-triage", args);
   normalizeBoolFlag(options.enableDiscover, "--enable-discover", args);
   normalizeBoolFlag(options.enableArchitect, "--enable-architect", args);

package/src/mcp/server-handlers.js

@@ -9,6 +9,7 @@ import fs from "node:fs/promises";
 import { loadConfig, applyRunOverrides, validateConfig } from "../config.js";
 import { currentBranch } from "../utils/git.js";
 import { buildProgressNotifier } from "./progress.js";
+import { compressResponse, compactStringify } from "./response-compressor.js";
 
 // ── Sub-module re-exports ────────────────────────────────────────────
 export { handleRunDirect, handleResumeDirect, validateResumeAnswer, handleRun, handleResume } from "./handlers/run-handler.js";
@@ -58,8 +59,9 @@ export function asObject(value) {
 }
 
 export function responseText(payload) {
+  const compressed = compressResponse(payload);
   return {
-    content: [{ type: "text", text: JSON.stringify(payload, null, 2) }]
+    content: [{ type: "text", text: compactStringify(compressed) }]
   };
 }
 

package/src/mcp/tools.js

@@ -100,6 +100,7 @@ export const tools = [
         taskType: { type: "string", enum: ["sw", "infra", "doc", "add-tests", "refactor"], description: "Explicit task type for policy resolution. Overrides triage classification." },
         quiet: { type: "boolean", description: "Suppress raw agent output lines, show only stage status (default: true). Set false for verbose output." },
         noSonar: { type: "boolean" },
+        design: { type: "boolean", description: "Activate design refactoring mode. Impeccable role applies design changes instead of just auditing." },
         enableSonarcloud: { type: "boolean", description: "Enable SonarCloud scan (complementary to SonarQube)" },
         kjHome: { type: "string" },
         sonarToken: { type: "string" },

package/src/orchestrator/post-loop-stages.js

@@ -247,17 +247,21 @@ export async function runSecurityStage({ config, logger, emitter, eventBase, ses
   return { action: "ok", stageResult: { ok: true, summary: securityOutput.summary || "No vulnerabilities found" } };
 }
 
-export async function runImpeccableStage({ config, logger, emitter, eventBase, session, coderRole, trackBudget, iteration, task, diff }) {
+export async function runImpeccableStage({ config, logger, emitter, eventBase, session, coderRole, trackBudget, iteration, task, diff, mode = "audit" }) {
   logger.setContext({ iteration, stage: "impeccable" });
+  const isRefactoring = mode === "refactoring";
+  const startMessage = isRefactoring
+    ? "Impeccable applying design refactoring"
+    : "Impeccable auditing frontend design quality";
   emitProgress(
     emitter,
     makeEvent("impeccable:start", { ...eventBase, stage: "impeccable" }, {
-      message: "Impeccable auditing frontend design quality",
-      detail: { provider: config?.roles?.impeccable?.provider || coderRole.provider, executorType: "agent" }
+      message: startMessage,
+      detail: { provider: config?.roles?.impeccable?.provider || coderRole.provider, executorType: "agent", mode }
     })
   );
 
-  const impeccable = new ImpeccableRole({ config, logger, emitter });
+  const impeccable = new ImpeccableRole({ config, logger, emitter, mode });
   await impeccable.init({ task, iteration });
   const impeccableStart = Date.now();
   let impeccableOutput;

package/src/orchestrator.js

@@ -262,6 +262,12 @@ function applyFlagOverrides(pipelineFlags, flags) {
   if (flags.enableTester !== undefined) pipelineFlags.testerEnabled = Boolean(flags.enableTester);
   if (flags.enableSecurity !== undefined) pipelineFlags.securityEnabled = Boolean(flags.enableSecurity);
   if (flags.enableImpeccable !== undefined) pipelineFlags.impeccableEnabled = Boolean(flags.enableImpeccable);
+
+  // --design flag: force-enable impeccable in refactoring mode
+  if (flags.design) {
+    pipelineFlags.impeccableEnabled = true;
+    pipelineFlags.impeccableMode = "refactoring";
+  }
 }
 
 function resolvePipelinePolicies({ flags, config, stageResults, emitter, eventBase, session, pipelineFlags }) {
@@ -1149,9 +1155,10 @@ async function runQualityGateStages({ config, logger, emitter, eventBase, sessio
 
   if (pipelineFlags?.impeccableEnabled) {
     const diff = await generateDiff({ baseRef: session.session_start_sha });
+    const impeccableMode = pipelineFlags?.impeccableMode || "audit";
     const impeccableResult = await runImpeccableStage({
       config, logger, emitter, eventBase, session, coderRole, trackBudget,
-      iteration: i, task, diff
+      iteration: i, task, diff, mode: impeccableMode
     });
     if (impeccableResult.stageResult) {
       stageResults.impeccable = impeccableResult.stageResult;

package/src/roles/impeccable-role.js

@@ -59,9 +59,11 @@ function buildSummary(parsed) {
 }
 
 export class ImpeccableRole extends BaseRole {
-  constructor({ config, logger, emitter = null, createAgentFn = null }) {
-    super({ name: "impeccable", config, logger, emitter });
+  constructor({ config, logger, emitter = null, createAgentFn = null, mode = "audit" }) {
+    const roleName = mode === "refactoring" ? "impeccable-design" : "impeccable";
+    super({ name: roleName, config, logger, emitter });
     this._createAgent = createAgentFn || defaultCreateAgent;
+    this.mode = mode;
   }
 
   async execute(input) {

package/templates/roles/impeccable-design.md

@@ -0,0 +1,87 @@
+# Impeccable Design (Refactoring Mode)
+
+You are refactoring the frontend design. Apply ALL of these improvements:
+
+## Scope constraint
+
+- **ONLY modify files present in the diff.** Do not touch files that were not changed.
+- If no frontend files (.html, .css, .astro, .jsx, .tsx, .vue, .svelte, .lit, .js with DOM manipulation) are in the diff, report APPROVED immediately with 0 issues.
+
+## Input
+
+- **Task**: {{task}}
+- **Diff**: {{diff}}
+- **Context**: {{context}}
+
+## Visual Hierarchy
+- Establish clear heading levels (size, weight, color)
+- Group related elements with consistent spacing
+- Use whitespace to separate sections
+
+## Spacing & Alignment
+- Consistent padding/margin using a spacing scale (4px, 8px, 16px, 24px, 32px)
+- Align elements to a grid
+- Remove arbitrary pixel values
+
+## Responsive
+- Mobile-first approach
+- Fluid layouts with relative units (rem, %, vw)
+- Breakpoints at 640px, 768px, 1024px, 1280px
+
+## Accessibility
+- Color contrast ratio >= 4.5:1 for text
+- All interactive elements keyboard-focusable
+- Labels on all form inputs
+- Alt text on all images
+
+## Micro-interactions
+- Hover/focus states on all interactive elements
+- Smooth transitions (150-300ms)
+- Loading states for async operations
+
+## Theming
+- CSS custom properties for colors, fonts, spacing
+- Dark mode support if applicable
+- Consistent color palette (max 5 primary colors)
+
+## Rules
+- Each fix must be minimal and targeted (Edit, not Write)
+- Only use Read, Edit, Grep, Glob, and Bash tools
+- Verify each fix with `git diff` to confirm only intended lines changed
+- Apply changes directly. Do not just list issues.
+
+## Report
+
+Output a strict JSON object:
+
+```json
+{
+  "ok": true,
+  "result": {
+    "verdict": "IMPROVED",
+    "issuesFound": 3,
+    "issuesFixed": 3,
+    "categories": {
+      "visualHierarchy": 0,
+      "spacing": 1,
+      "responsive": 1,
+      "a11y": 1,
+      "microInteractions": 0,
+      "theming": 0
+    },
+    "changes": [
+      {
+        "file": "src/components/Button.astro",
+        "issue": "Arbitrary padding values",
+        "fix": "Replaced with spacing scale (16px)",
+        "category": "spacing"
+      }
+    ]
+  },
+  "summary": "3 design issues found and fixed (1 spacing, 1 responsive, 1 a11y)"
+}
+```
+
+### Verdict rules
+- **APPROVED** — No frontend design issues found (issuesFound === 0)
+- **IMPROVED** — Issues were found and fixes were applied (issuesFixed > 0)