karajan-code 2.29.0 → 2.30.0

package/README.md

@@ -345,6 +345,13 @@ Opt out: set `telemetry: false` in `~/.karajan/kj.config.yml`
 
 ## Recent releases
 
+> **v2.30.0 released** — Minor. **Writable config UI on HU Board** — settings modal with grouped sections, atomic-write backend, and a global vs per-project scope toggle. No more hand-editing the YAML.
+>
+> - **Pipeline role toggles** (PR #854): 8 nuevos booleanos en el modal (`planner`, `researcher`, `architect`, `tester`, `security`, `refactorer`, `impeccable`, `brain`) reflejan los defaults reales de `src/config/defaults.js`.
+> - **RAG controls** (PR #855): `rag.preload.{enabled,topK,scope}` + `rag.embedder.provider` editables sin abrir el archivo. El dropdown del provider lista los 6 embedders (ollama / openai / voyage / cohere / mistral / onnx).
+> - **Grouped sections** (PR #856): los campos se agrupan por categoría (Agentes y modelos, Roles del pipeline, RAG, Tiempos de sesión, Calidad) con iconos y orden determinista. Campos nuevos sin categoría caen en "Otros" — defensivo.
+> - **Scope toggle global vs per-project** (PR #857): pill toggle en el header del modal. Switchea entre `~/.karajan/kj.config.yml` (global, default) y `<projectDir>/.karajan/kj.config.yml` (override del proyecto, se crea al guardar). Atomic-write + `.bak` aplican a ambos.
+>
 > **v2.29.0 released** — Minor. **RAG quality lift** — dashboard + three new providers + metadata filter + rerank.
 >
 > - **Retrieval dashboard on HU Board** (PR #843): nuevo `/rag.html` con embedder activo, tamaño de DB, last-index, chunks por kind, chunks por proyecto. Primera pieza del config UI que llega completo en v2.30.

package/package.json

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

package/packages/hu-board/public/app.js

@@ -3272,10 +3272,14 @@ function handleRoute() {
 // row. `select` → <select>, `boolean` → <input type=checkbox>,
 // `number` → <input type=number min/max>. Help text under each
 // label as a small grey paragraph.
-async function showConfigEditor() {
+async function showConfigEditor(scope = 'global') {
+  // v2.30.0 PR4 — scope toggle. 'global' edita ~/.karajan/kj.config.yml
+  // (afecta a todos los proyectos); 'project' edita
+  // <projectDir>/.karajan/kj.config.yml (sólo el repo actual). El backend
+  // resuelve <projectDir> desde KJ_PROJECT_DIR || cwd.
   let cfg;
   try {
-    const r = await fetch('/api/config');
+    const r = await fetch(`/api/config?scope=${encodeURIComponent(scope)}`);
     if (!r.ok) throw new Error(`HTTP ${r.status}`);
     cfg = await r.json();
   } catch (err) {
@@ -3306,14 +3310,66 @@ async function showConfigEditor() {
       </div>
     `;
   };
+
+  // v2.30.0 — agrupa los campos por `f.category`. El backend declara
+  // las categorías ordenadas en `cfg.categories`; cualquier campo sin
+  // categoría cae a "Otros" (defensivo, no se pierde si olvidamos
+  // etiquetar uno nuevo). Mantiene el orden interno del whitelist
+  // dentro de cada sección.
+  const cats = Array.isArray(cfg.categories) && cfg.categories.length > 0
+    ? cfg.categories
+    : [{ id: 'misc', label: 'Otros', icon: '⚙', order: 999 }];
+  const byCat = new Map(cats.map((c) => [c.id, []]));
+  byCat.set('misc', byCat.get('misc') || []);
+  for (const f of cfg.fields) {
+    const id = byCat.has(f.category) ? f.category : 'misc';
+    if (!byCat.has(id)) byCat.set(id, []);
+    byCat.get(id).push(f);
+  }
+  const renderSection = (cat) => {
+    const fields = byCat.get(cat.id) || [];
+    if (fields.length === 0) return '';
+    return `
+      <section style="margin-bottom:22px;">
+        <h3 style="display:flex;align-items:center;gap:8px;margin:0 0 12px;padding-bottom:6px;border-bottom:1px solid var(--border);font-size:0.95rem;font-weight:700;color:var(--text);">
+          <span aria-hidden="true">${esc(cat.icon || '⚙')}</span>
+          <span>${esc(cat.label)}</span>
+        </h3>
+        ${fields.map(renderField).join('')}
+      </section>
+    `;
+  };
+  const allCats = [...cats];
+  if ((byCat.get('misc') || []).length > 0 && !allCats.some((c) => c.id === 'misc')) {
+    allCats.push({ id: 'misc', label: 'Otros', icon: '⚙', order: 999 });
+  }
+  const sectionsHtml = allCats.map(renderSection).join('');
+
+  // PR4 — toggle global / proyecto. Cambiarlo recarga el modal contra el
+  // otro fichero. Visual: dos pills mutuamente excluyentes.
+  const pill = (id, value, label, active) => `
+    <button data-scope="${value}" id="${id}" type="button"
+      style="padding:4px 10px;border-radius:999px;border:1px solid var(--border);
+      background:${active ? 'var(--color-blue)' : 'var(--bg-primary)'};
+      color:${active ? '#fff' : 'var(--text)'};font-size:0.75rem;cursor:pointer;">
+      ${esc(label)}
+    </button>`;
+  const scopeToggle = `
+    <div style="display:flex;align-items:center;gap:4px;margin-left:auto;" role="radiogroup" aria-label="Ámbito de configuración">
+      ${pill('cfg-scope-global',  'global',  'Global',         scope === 'global')}
+      ${pill('cfg-scope-project', 'project', 'Este proyecto',  scope === 'project')}
+    </div>`;
   dlg.innerHTML = `
     <div style="padding:14px 18px;border-bottom:1px solid var(--border);font-weight:700;display:flex;align-items:center;gap:10px;">
       <span style="font-size:1.2rem;">⚙</span>
       <span>Configuración de Karajan</span>
-      <span style="margin-left:auto;font-family:var(--font-mono,monospace);font-size:0.75rem;color:var(--text-muted);">${esc(cfg.path || '')}</span>
+      ${scopeToggle}
+    </div>
+    <div style="padding:6px 18px 0;font-family:var(--font-mono,monospace);font-size:0.7rem;color:var(--text-muted);">
+      ${esc(cfg.path || '')}${cfg.exists === false ? ' · (aún no existe, se creará al guardar)' : ''}
     </div>
     <div style="padding:14px 18px;max-height:65vh;overflow:auto;">
-      ${cfg.fields.map(renderField).join('')}
+      ${sectionsHtml}
       <p style="margin:14px 0 0;font-size:0.75rem;color:var(--text-muted);">
         Los cambios se guardan en <code>kj.config.yml</code> de forma atómica (con copia de seguridad <code>.bak</code>).
         El próximo <code>kj run</code> los aplicará — no hace falta reiniciar el board.
@@ -3325,6 +3381,15 @@ async function showConfigEditor() {
     </div>
   `;
   if (typeof dlg.showModal === 'function' && !dlg.open) dlg.showModal();
+  // PR4 — al cambiar de pill, recargar el modal con el otro scope.
+  for (const btn of dlg.querySelectorAll('[data-scope]')) {
+    btn.addEventListener('click', () => {
+      const next = btn.getAttribute('data-scope');
+      if (next === scope) return;
+      try { dlg.close(); } catch { /* ignore */ }
+      showConfigEditor(next);
+    }, { once: true });
+  }
   dlg.querySelector('#config-cancel')?.addEventListener('click', () => { try { dlg.close(); } catch { /* ignore */ } }, { once: true });
   dlg.querySelector('#config-save')?.addEventListener('click', async () => {
     // Build the patch from the form, comparing against initial values
@@ -3347,7 +3412,7 @@ async function showConfigEditor() {
       const r = await fetch('/api/config', {
         method: 'PUT',
         headers: { 'Content-Type': 'application/json' },
-        body: JSON.stringify({ patch }),
+        body: JSON.stringify({ patch, scope }),
       });
       if (!r.ok) {
         const body = await r.json().catch(() => ({}));

package/packages/hu-board/src/config-yaml.js

@@ -34,7 +34,29 @@ import { tmpdir } from 'node:os';
 import yaml from 'js-yaml';
 import { getKjHome } from './db.js';
 
-function configPath() {
+/**
+ * v2.30.0 PR4 — scope resolver.
+ *
+ *   'global'  → ~/.karajan/kj.config.yml (mismo path que el resto de
+ *               Karajan; afecta a todos los proyectos).
+ *   'project' → <projectDir>/.karajan/kj.config.yml (override por
+ *               proyecto; cuando existe, el loader del padre lo prefiere
+ *               sobre el global).
+ *
+ * projectDir se resuelve desde `KJ_PROJECT_DIR` (ya usado por
+ * journal-parser.js) con fallback a `process.cwd()` — el board se
+ * arranca desde el repo del usuario, así que cwd es el proyecto.
+ */
+export const SCOPES = ['global', 'project'];
+
+function getProjectDir() {
+  return process.env.KJ_PROJECT_DIR || process.cwd();
+}
+
+function configPath(scope = 'global') {
+  if (scope === 'project') {
+    return join(getProjectDir(), '.karajan', 'kj.config.yml');
+  }
   // Same path the parent uses (src/utils/paths.js::getKarajanHome).
   // KJC-TSK-0420: delegate to db.js::getKjHome — gives us KARAJAN_HOME
   // priority + KJ_HOME fallback + VITEST guard from one place, instead
@@ -42,13 +64,28 @@ function configPath() {
   return join(getKjHome(), 'kj.config.yml');
 }
 
-function backupPath() { return `${configPath()}.bak`; }
+function backupPath(scope = 'global') { return `${configPath(scope)}.bak`; }
+
+/**
+ * v2.30.0 — Categorías para agrupar los campos en el modal. El backend
+ * sólo declara la metadata (id, label, icon, order). El front (app.js)
+ * itera sobre EDITABLE_FIELDS, agrupa por `f.category` y dibuja un
+ * <h3> por sección. Si un campo nuevo no declara categoría, cae en
+ * "Otros" (defensivo: no se pierde aunque se nos olvide etiquetarlo).
+ */
+export const CATEGORIES = [
+  { id: 'agents',   label: 'Agentes y modelos',        icon: '🤖', order: 1 },
+  { id: 'pipeline', label: 'Roles del pipeline',       icon: '🔌', order: 2 },
+  { id: 'rag',      label: 'RAG (búsqueda de contexto)', icon: '📚', order: 3 },
+  { id: 'session',  label: 'Tiempos de sesión',        icon: '⏱',  order: 4 },
+  { id: 'quality',  label: 'Calidad',                  icon: '✅', order: 5 },
+];
 
 /**
  * Whitelist of fields the board's modal exposes. Everything else
  * stays untouched in the yml. Each entry says where in the yml the
  * field lives (`path`) and how to render it for the UI (`type`,
- * `label`, `options` for selects).
+ * `label`, `options` for selects, `category` for grouping).
  *
  * The path is a dotted notation we resolve manually (so we never
  * need to take a runtime dep on lodash for `_.get`/`_.set`).
@@ -57,6 +94,7 @@ export const EDITABLE_FIELDS = [
   {
     key: 'coder',
     path: 'coder',
+    category: 'agents',
     type: 'select',
     label: 'Agente para el rol "coder"',
     help: 'Quién escribe el código. Claude es el más fiable, Codex es más rápido.',
@@ -66,6 +104,7 @@ export const EDITABLE_FIELDS = [
   {
     key: 'reviewer',
     path: 'reviewer',
+    category: 'agents',
     type: 'select',
     label: 'Agente para el rol "reviewer"',
     help: 'Quién revisa el código del coder. Conviene que sea un agente distinto al coder para detectar errores.',
@@ -75,6 +114,7 @@ export const EDITABLE_FIELDS = [
   {
     key: 'modelMode',
     path: '_modelMode_virtual',
+    category: 'agents',
     type: 'select',
     label: 'Estrategia de modelos',
     help: 'Auto = Karajan elige el modelo según la complejidad (haiku/sonnet/opus). Sonnet siempre = barato, ideal para pruebas. Opus siempre = caro, máxima calidad.',
@@ -86,6 +126,7 @@ export const EDITABLE_FIELDS = [
   {
     key: 'maxIterationMinutes',
     path: 'session.max_iteration_minutes',
+    category: 'session',
     type: 'number',
     label: 'Tope de minutos por iteración',
     help: 'Si una iteración (coder + reviewer) supera este tiempo, Karajan la corta y Solomon decide qué hacer.',
@@ -96,6 +137,7 @@ export const EDITABLE_FIELDS = [
   {
     key: 'maxTotalMinutes',
     path: 'session.max_total_minutes',
+    category: 'session',
     type: 'number',
     label: 'Tope de minutos total por sesión',
     help: 'Tope absoluto del wall-clock de la sesión. Si se llega aquí, Karajan para aunque queden HUs.',
@@ -106,11 +148,133 @@ export const EDITABLE_FIELDS = [
   {
     key: 'sonarEnabled',
     path: 'sonarqube.enabled',
+    category: 'quality',
     type: 'boolean',
     label: 'Activar análisis SonarQube',
     help: 'Pasa el código por SonarQube y bloquea si el quality gate falla. Requiere SonarQube corriendo.',
     default: false,
   },
+  // v2.30.0 — pipeline role toggles. Cada uno controla si una etapa
+  // del pipeline se ejecuta. Brain está OFF por defecto porque sigue
+  // siendo opt-in (v2 feature); el resto reflejan los defaults de
+  // src/config/defaults.js.
+  {
+    key: 'plannerEnabled',
+    path: 'pipeline.planner.enabled',
+    category: 'pipeline',
+    type: 'boolean',
+    label: 'Activar rol planner',
+    help: 'Genera un plan de implementación antes de codificar. Útil en tareas con >2 archivos o devPoints ≥ 3.',
+    default: true,
+  },
+  {
+    key: 'researcherEnabled',
+    path: 'pipeline.researcher.enabled',
+    category: 'pipeline',
+    type: 'boolean',
+    label: 'Activar rol researcher',
+    help: 'Investiga la codebase antes del coder (lectura de archivos clave, patrones existentes).',
+    default: true,
+  },
+  {
+    key: 'architectEnabled',
+    path: 'pipeline.architect.enabled',
+    category: 'pipeline',
+    type: 'boolean',
+    label: 'Activar rol architect',
+    help: 'Diseña la arquitectura (capas, patrones, contratos) antes de codificar. Recomendado en cambios estructurales.',
+    default: true,
+  },
+  {
+    key: 'testerEnabled',
+    path: 'pipeline.tester.enabled',
+    category: 'pipeline',
+    type: 'boolean',
+    label: 'Activar rol tester',
+    help: 'Genera tests automáticos durante o después del coder. Refuerza la calidad.',
+    default: true,
+  },
+  {
+    key: 'securityEnabled',
+    path: 'pipeline.security.enabled',
+    category: 'pipeline',
+    type: 'boolean',
+    label: 'Activar rol security',
+    help: 'Audita el código contra OWASP-style issues, secret leaks, dependencias vulnerables.',
+    default: true,
+  },
+  {
+    key: 'refactorerEnabled',
+    path: 'pipeline.refactorer.enabled',
+    category: 'pipeline',
+    type: 'boolean',
+    label: 'Activar rol refactorer',
+    help: 'Pasa una iteración de limpieza tras el coder (extract function, rename, etc.).',
+    default: false,
+  },
+  {
+    key: 'impeccableEnabled',
+    path: 'pipeline.impeccable.enabled',
+    category: 'pipeline',
+    type: 'boolean',
+    label: 'Activar rol impeccable',
+    help: 'Auditoría UI/UX automática (Lighthouse, a11y, contraste). Solo útil en frontend.',
+    default: false,
+  },
+  {
+    key: 'brainEnabled',
+    path: 'pipeline.brain.enabled',
+    category: 'pipeline',
+    type: 'boolean',
+    label: 'Activar Karajan Brain (v2)',
+    help: 'Orquestador IA central: enriquece feedback, comprime outputs y verifica cambios reales. Opt-in.',
+    default: false,
+  },
+  // v2.30.0 — RAG controls. Sólo se exponen las claves que el código
+  // realmente lee hoy (rag.preload.* en rag-context-stage.js y
+  // rag.embedder.provider en embedders/factory.js). mode/alpha/rerank
+  // se controlan vía flags CLI; cuando se cableen en config se añadirán
+  // aquí (ver task #99).
+  {
+    key: 'ragPreloadEnabled',
+    path: 'rag.preload.enabled',
+    category: 'rag',
+    type: 'boolean',
+    label: 'Activar pre-carga RAG en el pipeline',
+    help: 'Si está activo, antes de cada iteración se inyecta contexto del vector store en el prompt del coder.',
+    default: false,
+  },
+  {
+    key: 'ragPreloadTopK',
+    path: 'rag.preload.topK',
+    category: 'rag',
+    type: 'number',
+    label: 'Nº de chunks a pre-cargar (topK)',
+    help: 'Cuántos fragmentos del vector store se inyectan como contexto. 3-5 funciona bien; subir gasta tokens.',
+    min: 1,
+    max: 20,
+    default: 5,
+  },
+  {
+    key: 'ragPreloadScope',
+    path: 'rag.preload.scope',
+    category: 'rag',
+    type: 'select',
+    label: 'Scope de la pre-carga RAG',
+    help: '"all" mezcla todo. "code" solo busca en el código indexado, "plans" en planes previos, "onboarding" en el brief del proyecto.',
+    options: ['all', 'code', 'plans', 'onboarding'],
+    default: 'all',
+  },
+  {
+    key: 'ragEmbedderProvider',
+    path: 'rag.embedder.provider',
+    category: 'rag',
+    type: 'select',
+    label: 'Proveedor de embeddings RAG',
+    help: 'Ollama local es gratis. OpenAI/Voyage/Cohere/Mistral son cloud (requieren API key). ONNX corre 100% local con @huggingface/transformers.',
+    options: ['ollama', 'openai', 'voyage', 'cohere', 'mistral', 'onnx'],
+    default: 'ollama',
+  },
 ];
 
 const ROLES_FOR_MODEL_MODE = [
@@ -203,8 +367,13 @@ function applyModelMode(parsed, mode) {
  *
  * Returns sane defaults when the file doesn't exist yet.
  */
-export function readConfig() {
-  const p = configPath();
+export function readConfig({ scope = 'global' } = {}) {
+  // PR4 — accept scope. Default 'global' preserves the previous behavior;
+  // 'project' reads <projectDir>/.karajan/kj.config.yml.
+  if (!SCOPES.includes(scope)) {
+    throw new Error(`Scope inválido: ${scope}. Permitidos: ${SCOPES.join(', ')}`);
+  }
+  const p = configPath(scope);
   let parsed = {};
   let exists = false;
   if (existsSync(p)) {
@@ -222,7 +391,11 @@ export function readConfig() {
     if (value === undefined) value = f.default;
     return { ...f, value };
   });
-  return { path: p, exists, fields, raw: parsed };
+  // v2.30.0 — also expose CATEGORIES so the modal can render grouped
+  // sections without re-importing the constant client-side. Ordered
+  // by `order` for deterministic UI.
+  const categories = [...CATEGORIES].sort((a, b) => (a.order ?? 999) - (b.order ?? 999));
+  return { path: p, scope, exists, fields, categories, raw: parsed };
 }
 
 /**
@@ -232,16 +405,20 @@ export function readConfig() {
  *
  * @returns {{ path: string, written: boolean, applied: object[], errors: string[] }}
  */
-export function writeConfigPatch(patch) {
+export function writeConfigPatch(patch, { scope = 'global' } = {}) {
+  // PR4 — same shape as readConfig: scope decides target file.
+  if (!SCOPES.includes(scope)) {
+    return { path: '', scope, written: false, applied: [], errors: [`Scope inválido: ${scope}. Permitidos: ${SCOPES.join(', ')}`] };
+  }
   if (!patch || typeof patch !== 'object') {
-    return { path: configPath(), written: false, applied: [], errors: ['patch debe ser un objeto'] };
+    return { path: configPath(scope), scope, written: false, applied: [], errors: ['patch debe ser un objeto'] };
   }
   // Read current state (or empty if file doesn't exist yet).
-  const p = configPath();
+  const p = configPath(scope);
   let parsed = {};
   if (existsSync(p)) {
     try { parsed = yaml.load(readFileSync(p, 'utf8'), { json: true }) || {}; }
-    catch (err) { return { path: p, written: false, applied: [], errors: [`No se pudo parsear el yml actual: ${err.message}`] }; }
+    catch (err) { return { path: p, scope, written: false, applied: [], errors: [`No se pudo parsear el yml actual: ${err.message}`] }; }
   }
   const errors = [];
   const applied = [];
@@ -268,18 +445,18 @@ export function writeConfigPatch(patch) {
     else setDeep(parsed, field.path, field.type === 'number' ? Number(val) : val);
     applied.push({ key, value: val });
   }
-  if (errors.length > 0) return { path: p, written: false, applied, errors };
+  if (errors.length > 0) return { path: p, scope, written: false, applied, errors };
 
   // Write atomically: serialize → write to .tmp → rename. Backup the
   // existing file first so the user can recover.
   const dir = dirname(p);
   if (!existsSync(dir)) mkdirSync(dir, { recursive: true });
-  if (existsSync(p)) { try { copyFileSync(p, backupPath()); } catch { /* best-effort */ } }
+  if (existsSync(p)) { try { copyFileSync(p, backupPath(scope)); } catch { /* best-effort */ } }
   const dump = yaml.dump(parsed, { lineWidth: 120 });
   // mkstemp would be nicer but we want to stay in the same dir for
   // a guaranteed-atomic rename across the same filesystem.
   const tmpFile = join(dir, `.kj.config.yml.${process.pid}.${Date.now()}.tmp`);
   writeFileSync(tmpFile, dump, 'utf8');
   renameSync(tmpFile, p);
-  return { path: p, written: true, applied, errors: [] };
+  return { path: p, scope, written: true, applied, errors: [] };
 }

package/packages/hu-board/src/routes/api.js

@@ -1031,29 +1031,41 @@ router.post('/projects/:id/run', (req, res) => {
 });
 
 /**
- * GET /api/config — read the editable subset of ~/.karajan/kj.config.yml.
+ * GET /api/config — read the editable subset of the kj.config.yml.
  * Powers the board's settings modal; non-editable keys in the yml are
  * left untouched on PUT (see writeConfigPatch's whitelist semantics).
+ *
+ * v2.30.0 PR4 — supports `?scope=global|project`:
+ *   - `global` (default): reads ~/.karajan/kj.config.yml.
+ *   - `project`: reads <projectDir>/.karajan/kj.config.yml.
+ * Unknown scopes are 400'd. Response includes the resolved `scope` and
+ * `path` so the UI can show which file backs the current view.
  */
-router.get('/config', (_req, res) => {
+router.get('/config', (req, res) => {
   try {
-    const data = readConfig();
+    const scope = req.query?.scope || 'global';
+    const data = readConfig({ scope });
     res.json(data);
   } catch (err) {
-    res.status(500).json({ error: err.message });
+    res.status(400).json({ error: err.message });
   }
 });
 
 /**
- * PUT /api/config — apply a patch to the yml. Body is `{ patch: { key: value } }`
- * with the keys defined in EDITABLE_FIELDS. Unknown keys are 400'd; type
- * errors are 400'd; validation errors are 400'd. Writes are atomic
- * (tmp + rename) and a `.bak` is kept of the previous content.
+ * PUT /api/config — apply a patch to the yml. Body is
+ * `{ patch: { key: value }, scope?: 'global'|'project' }` with the keys
+ * defined in EDITABLE_FIELDS. Unknown keys are 400'd; type errors are
+ * 400'd; validation errors are 400'd. Writes are atomic (tmp + rename)
+ * and a `.bak` is kept of the previous content.
+ *
+ * v2.30.0 PR4 — `scope` decides which file is patched. Default 'global'
+ * preserves the previous behavior.
  */
 router.put('/config', (req, res) => {
   try {
     const patch = req.body?.patch;
-    const result = writeConfigPatch(patch);
+    const scope = req.body?.scope || 'global';
+    const result = writeConfigPatch(patch, { scope });
     if (!result.written) {
       return res.status(400).json({ error: 'No se pudo guardar la configuración', details: result });
     }