geneseed 4.10.0 → 4.15.1

package/js/web/catalog.mjs

@@ -24,6 +24,7 @@ import { readText, withDiscardableStderr, isDir, isFile } from '../lib/fs.mjs';
 import { comparePaths, normcase, within } from '../lib/paths.mjs';
 import { stripWhitespace } from '../lib/text.mjs';
 import { readJsonc } from '../hosts/settings.mjs';
+import { loadKits } from '../build/source.mjs';
 
 /** A requested catalog section or item that does not exist. */
 export class NotFound extends Error {}
@@ -466,16 +467,29 @@ function constitutionItems(inv) {
   ];
 }
 
+/** `Map<'skills/x' | 'agents/y', kit>` from the shipped manifests; a broken manifest means no badges, not a 500. */
+function kitOwners() {
+  const owner = new Map();
+  try {
+    for (const [kit, m] of loadKits()) {
+      for (const n of m.skills ?? []) owner.set(`skill:${n}`, kit);
+      for (const n of m.agents ?? []) owner.set(`agent:${n}`, kit);
+    }
+  } catch { /* shortcut: unreadable manifests hide the badge; apiKits reports the error */ }
+  return owner;
+}
+
 export function apiCatalog(state, section) {
   if (!SECTIONS.includes(section)) throw new NotFound(section);
   const inv = state.inventory;
   let items;
+  const owner = kitOwners();
   if (section === 'agents') {
     items = inv.agents.map((e) => ({ name: e.name, title: e.name, desc: e.desc,
-      source: e.source ?? null, status: e.status ?? 'unknown' }));
+      source: e.source ?? null, status: e.status ?? 'unknown', kit: owner.get(`agent:${e.name}`) ?? null }));
   } else if (section === 'skills') {
     items = inv.skills.map((e) => ({ name: e.name, title: e.name, desc: e.desc,
-      source: e.source ?? null, klass: e.klass ?? 'build', status: e.status ?? 'unknown' }));
+      source: e.source ?? null, klass: e.klass ?? 'build', status: e.status ?? 'unknown', kit: owner.get(`skill:${e.name}`) ?? null }));
   } else if (section === 'laws') {
     items = constitutionItems(inv);
   } else if (section === 'memory') {
@@ -499,6 +513,7 @@ export function apiItem(state, type, name) {
       links: resolveLinks(state, e.body), source: e.source ?? null };
     if (type === 'skill') out.klass = e.klass ?? 'build';
     out.status = e.status ?? 'unknown';
+    out.kit = kitOwners().get(`${type}:${name}`) ?? null;
     return out;
   }
   if (type === 'law') {

package/js/web/handler.mjs

@@ -303,6 +303,7 @@ export function makeHandler(state, jm, token, dist, holder = null) {
     // Build resolves its argv off the install (`buildActionCmds`); the other actions take no
     // axes and self-resolve the deployed theme downstream.
     const cmds = action === 'build' ? buildActionCmds(state, body) : actionCommands(action);
+    if (cmds && Object.hasOwn(cmds, 'error')) return sendJson(res, cmds, 400, ae);
     if (!isTruthy(cmds)) {
       return sendJson(res, { error: `unknown action ${action}` }, 404, ae);
     }

package/js/web/server.mjs

@@ -27,7 +27,7 @@ import { isFile, printOut } from '../lib/fs.mjs';
 import { which } from '../lib/paths.mjs';
 import { parseIntStrict } from '../lib/text.mjs';
 import { promptLine } from '../lib/prompt.mjs';
-import { warmDoctor, webState } from './state.mjs';
+import { fallbackView, warmDoctor, webState } from './state.mjs';
 import { clearDaemon, codeStamp, openUrl, readDaemon, restartDaemon, startDaemon, statusDaemon, stopDaemon, writeDaemon } from './daemon.mjs';
 import { makeHandler } from './handler.mjs';
 import { JobManager } from './jobs.mjs';
@@ -132,6 +132,12 @@ export async function serve({ theme = null, port = 4747, openBrowser = true,
     }
   }
   const state = webState(theme);
+  // The daemon record and the job file stay at the DEFAULT target, captured before the view
+  // moves: `geneseed web stop`/`status` find them through `webState(theme).target`, which never
+  // moves, and a view that followed a Bob-only install into ~/.bob would hide the record.
+  const home = state.target;
+  const fallback = fallbackView(home);
+  if (fallback) state.selectView(...fallback);
   if (!existsSync(join(state.target, GLOBAL_MANIFEST))) {
     printOut(`[web] no deployed harness at ${state.target}.\n`);
     printOut('      Run `geneseed setup` first — serving anyway (read-only UI).\n');
@@ -143,7 +149,7 @@ export async function serve({ theme = null, port = 4747, openBrowser = true,
   const holder = {};
   // The console's job list survives a reload and a restart because it is a FILE on disk,
   // not held in process memory.
-  const jm = new JobManager(join(state.target, '.geneseed-web-runs.json'));
+  const jm = new JobManager(join(home, '.geneseed-web-runs.json'));
   const srv = createServer(makeHandler(state, jm, token, dist, holder));
   holder.srv = srv;
   return new Promise((done) => {
@@ -157,7 +163,7 @@ export async function serve({ theme = null, port = 4747, openBrowser = true,
       const hostPort = /** @type {import('node:net').AddressInfo} */ (srv.address()).port;
       const url = `http://127.0.0.1:${hostPort}`;
       if (daemon) {
-        writeDaemon(state.target, {
+        writeDaemon(home, {
           pid: process.pid, port: hostPort, url,
           token, theme: state.theme, started: Math.floor(Date.now() / 1000), code: startCode,
         });
@@ -177,8 +183,8 @@ export async function serve({ theme = null, port = 4747, openBrowser = true,
       });
       srv.on('close', () => {
         if (daemon) {
-          const st = readDaemon(state.target);
-          if (!st || st.pid === process.pid) clearDaemon(state.target);
+          const st = readDaemon(home);
+          if (!st || st.pid === process.pid) clearDaemon(home);
         }
         done(0);
       });

package/js/web/state.mjs

@@ -8,10 +8,10 @@
 import { existsSync } from 'node:fs';
 import path from 'node:path';
 
-import { opencodeConfigDir } from '../hosts/hosts.mjs';
+import { CLAUDE_STYLE, GLOBAL_MANIFEST, HOSTS, opencodeConfigDir } from '../hosts/hosts.mjs';
 import {
-  EMIT_HOST_SCOPE, footprintOfDir, installedDefaults, modeOfDir, postureOfDir, themeOfDir,
-  trustOfDir,
+  EMIT_HOST_SCOPE, footprintOfDir, installState, installTargets, installedDefaults, modeOfDir,
+  postureOfDir, themeOfDir, trustOfDir,
 } from '../hosts/installs.mjs';
 import { doctorCollect } from '../inspect/doctor.mjs';
 import { DEFAULT_PRESET } from '../loop/score.mjs';
@@ -19,6 +19,36 @@ import { readText, isFile } from '../lib/fs.mjs';
 import { stripWhitespace } from '../lib/text.mjs';
 import { inventoryFor } from './catalog.mjs';
 
+/**
+ * The data dir an install's inventory/memory/diff is read from. Host-driven, so a new
+ * nested-marker host cannot silently read the bare root.
+ */
+export function viewCfg(host, scope, root) {
+  if (scope === 'project' && CLAUDE_STYLE.includes(host)) {
+    return path.join(root, HOSTS.find((h) => h.host === host).projectMarker);
+  }
+  return root;
+}
+
+/**
+ * `[target, root]` for the install the console should show when nothing picked one, or null
+ * when the current target is already deployed or nothing is.
+ *
+ * The console used to open on the OpenCode global dir unconditionally. On a machine with only
+ * IBM Bob (reported 2026-10) nothing is deployed there, so Overview showed the onboarding form
+ * over a perfectly good `~/.bob`, and the selector that could re-point it hides itself below two
+ * installs. So when `current` holds no manifest, the first ACTIVE install `installTargets`
+ * finds is shown instead — a global before a project, in its host order.
+ */
+export function fallbackView(current, targets = installTargets()) {
+  if (existsSync(path.join(current, GLOBAL_MANIFEST))) return null;
+  const active = targets.filter(([host, scope, root]) => {
+    try { return installState(root, host, scope) === 'active'; } catch { return false; }
+  });
+  const pick = active.find(([, scope]) => scope === 'global') || active[0];
+  return pick ? [viewCfg(...pick), pick[2]] : null;
+}
+
 /**
  * `YYYY-MM-DD HH:MM`, LOCAL time — `toISOString` would be UTC, which reads wrong in every
  * timezone but one for a user-facing timestamp.
@@ -154,6 +184,11 @@ export function webState(theme = null, target = null) {
     st._doctor = null;
   };
   st.refresh = ({ doctor = true } = {}) => {
+    // A job may have deployed the first install while the view still points at an empty
+    // default (the onboarding Build), or removed the one in view: follow it to one that exists.
+    // Only on a full refresh: `{ doctor: false }` marks a mutation that cannot change installs.
+    const fallback = doctor ? fallbackView(st.target) : null;
+    if (fallback) { st.selectView(...fallback); return; }
     st._inv = null;
     if (doctor) st._doctor = null;
     st.emit = st.detectEmit() || st.emit;

package/package.json

@@ -1,7 +1,7 @@
 {
   "//": "Every field here is argued and gated in tests/unit/package_manifest.test.mjs. `files`, `bin` and `engines` each have a test that fails when reality drifts from this file; `version` mirrors harness.config.json and is gated too. Read that docblock before editing this one.",
   "name": "geneseed",
-  "version": "4.10.0",
+  "version": "4.15.1",
   "description": "A portable, theme-able harness you implant once and use everywhere to grow a disciplined AI coding agent.",
   "keywords": [
     "ai",

package/registry.json

@@ -99,6 +99,20 @@
       "added": "2026-06-08",
       "last_verified": ""
     },
+    "agents/react-engineer": {
+      "status": "experimental",
+      "version": "1.0.0",
+      "owner": "Guillaume Lemer",
+      "added": "2026-10-21",
+      "last_verified": ""
+    },
+    "agents/react-reviewer": {
+      "status": "experimental",
+      "version": "1.0.0",
+      "owner": "Guillaume Lemer",
+      "added": "2026-10-21",
+      "last_verified": ""
+    },
     "agents/researcher": {
       "status": "approved",
       "version": "1.0.0",
@@ -400,6 +414,20 @@
       "added": "2026-09-06",
       "last_verified": ""
     },
+    "skills/react": {
+      "status": "experimental",
+      "version": "1.0.0",
+      "owner": "Guillaume Lemer",
+      "added": "2026-10-21",
+      "last_verified": ""
+    },
+    "skills/react-testing": {
+      "status": "experimental",
+      "version": "1.0.0",
+      "owner": "Guillaume Lemer",
+      "added": "2026-10-21",
+      "last_verified": ""
+    },
     "skills/react-view-transitions": {
       "status": "approved",
       "version": "1.0.0",

package/src/AGENT.md.tmpl

@@ -187,9 +187,9 @@ contract you do not already know.
 **Rule of delegation:** read-only investigation can be dispatched freely; any
 {{AGENT}} that *writes* must return a summary of exactly what it changed. Every
 {{AGENT}} spec ends with a **self-improvement loop**: when a report carries a
-`spec-feedback:` line, weigh it — fold a real flaw back into that {{AGENT}}'s
-file (with the user's assent); {{MEMORY}} gets a record only when it clears
-{{DOCTRINE:persist-insight}}'s bar.
+`spec-feedback:` line, weigh it — record a real flaw (with the user's assent) in
+`{{DIR_NOTEBOOK}}/agent-notes/<name>.md` and pass it to later dispatches;
+{{MEMORY}} gets a record only when it clears {{DOCTRINE:persist-insight}}'s bar.
 
 <!-- LEAN:begin -->
 **Dispatching a multi-part goal.** When a goal spans several capabilities,
@@ -293,8 +293,8 @@ When a task repeats and no {{SKILL}} covers it, forge one through the
 [skill-forge]({{DIR_SKILLS}}/skill-forge.md) {{SKILL}}: copy
 [`{{DIR_SKILLS}}/_template.md`]({{DIR_SKILLS}}/_template.md), keep it to one coherent
 domain ({{DOCTRINE:automate-repetition}}), and list it wherever this file carries a catalogue. Every {{SKILL}} ends with a
-**self-improvement loop** — honour it: fold what a run teaches back into the
-file, or into {{MEMORY}} ({{DOCTRINE:persist-insight}}).
+**self-improvement loop** — honour it: a lesson goes to its notes, never into
+the file, or to {{MEMORY}} ({{DOCTRINE:persist-insight}}).
 
 ---
 

package/src/agents/_self-improvement.md

@@ -0,0 +1,11 @@
+## Self-improvement
+
+If this spec misled you — an input you needed but were not given, a boundary
+that proved wrong, a step you could not execute — end your report with one line:
+`spec-feedback: [agent|general] <the one-line fix> — seen: <what happened, against
+what this spec led you to expect>`. `general` marks a fix other {{AGENTS}} need too.
+Omit the line when there is no friction. The caller weighs the feedback and, with the
+user's assent, writes a real flaw, dated, into `{{DIR_NOTEBOOK}}/agent-notes/<name>.md`
+— never into this rendered file, which a rebuild overwrites — and hands those notes
+to every later dispatch of this {{AGENT}}. It records the feedback to {{MEMORY}} only if it clears
+{{DOCTRINE:persist-insight}}'s bar — most reports carry no feedback at all.

package/src/agents/_template.md

@@ -67,11 +67,4 @@ tells pipeline orchestration who to recruit; it changes nothing about how this
 - **Delivers:** what it must hand the next seat or the parent — same shape as
   its own output contract above, nothing looser.
 
-## Self-improvement
-
-If this spec misled you — an input you needed but were not given, a boundary
-that proved wrong, a step you could not execute — end your report with one line:
-`spec-feedback: <what failed — the one-line fix>`. Omit it when there is no
-friction. The caller weighs the feedback, folds a real flaw back into this file
-with the user's assent, and records it to {{MEMORY}} only if it clears
-{{DOCTRINE:persist-insight}}'s bar — most reports carry no feedback at all.
+<!-- INCLUDE: agents/_self-improvement.md -->

package/src/agents/advocate.md

@@ -26,11 +26,4 @@
 - A tight brief: the position in one line, the 2–4 load-bearing arguments for it, the key supporting evidence (`file:line` or facts), and the one objection it must beat. No hedging, no filler.
 - If the motion has fewer than two defensible upsides, report that the case cannot honestly be made — an invented argument loses the council more than a conceded seat.
 
-## Self-improvement
-
-If this spec misled you — an input you needed but were not given, a boundary
-that proved wrong, a step you could not execute — end your report with one line:
-`spec-feedback: <what failed — the one-line fix>`. Omit it when there is no
-friction. The caller weighs the feedback, folds a real flaw back into this file
-with the user's assent, and records it to {{MEMORY}} only if it clears
-{{DOCTRINE:persist-insight}}'s bar — most reports carry no feedback at all.
+<!-- INCLUDE: agents/_self-improvement.md -->

package/src/agents/architect.md

@@ -48,11 +48,4 @@ independently.)*
 - **Delivers:** the plan for the developer seat to implement, same shape as its
   own output contract above.
 
-## Self-improvement
-
-If this spec misled you — an input you needed but were not given, a boundary
-that proved wrong, a step you could not execute — end your report with one line:
-`spec-feedback: <what failed — the one-line fix>`. Omit it when there is no
-friction. The caller weighs the feedback, folds a real flaw back into this file
-with the user's assent, and records it to {{MEMORY}} only if it clears
-{{DOCTRINE:persist-insight}}'s bar — most reports carry no feedback at all.
+<!-- INCLUDE: agents/_self-improvement.md -->

package/src/agents/developer.md

@@ -56,11 +56,4 @@ independently.)*
 - **Delivers:** the changed files and a one-line summary, handed to the tester
   seat for validation.
 
-## Self-improvement
-
-If this spec misled you — an input you needed but were not given, a boundary
-that proved wrong, a step you could not execute — end your report with one line:
-`spec-feedback: <what failed — the one-line fix>`. Omit it when there is no
-friction. The caller weighs the feedback, folds a real flaw back into this file
-with the user's assent, and records it to {{MEMORY}} only if it clears
-{{DOCTRINE:persist-insight}}'s bar — most reports carry no feedback at all.
+<!-- INCLUDE: agents/_self-improvement.md -->

package/src/agents/devops-engineer.md

@@ -34,12 +34,4 @@ Builds CI/CD pipelines and delivery automation for Spring Boot applications.
 ## Output contract
 - Files changed, the stage list and what was validated.
 
-## Self-improvement
-
-If this spec misled you — an input you needed but were not given, a boundary
-that proved wrong, a step you could not execute — end your report with one line:
-`spec-feedback: <what failed — the one-line fix>`. Omit it when there is no
-friction. The caller weighs the feedback, folds a real flaw back into this file
-with the user's assent, and records it to {{MEMORY}} only if it clears
-{{DOCTRINE:persist-insight}}'s bar — most reports carry no feedback at all.
-
+<!-- INCLUDE: agents/_self-improvement.md -->

package/src/agents/docker-expert.md

@@ -34,12 +34,4 @@ Builds small, safe container images for Spring Boot 4 applications.
 ## Output contract
 - Files changed, the build command and whether the image built and started.
 
-## Self-improvement
-
-If this spec misled you — an input you needed but were not given, a boundary
-that proved wrong, a step you could not execute — end your report with one line:
-`spec-feedback: <what failed — the one-line fix>`. Omit it when there is no
-friction. The caller weighs the feedback, folds a real flaw back into this file
-with the user's assent, and records it to {{MEMORY}} only if it clears
-{{DOCTRINE:persist-insight}}'s bar — most reports carry no feedback at all.
-
+<!-- INCLUDE: agents/_self-improvement.md -->

package/src/agents/docs.md

@@ -49,11 +49,4 @@ independently.)*
 - **Receives:** the analyst's brief describing what changed and what needs documenting.
 - **Delivers:** the doc files written/updated, ready for the parent's merge decision.
 
-## Self-improvement
-
-If this spec misled you — an input you needed but were not given, a boundary
-that proved wrong, a step you could not execute — end your report with one line:
-`spec-feedback: <what failed — the one-line fix>`. Omit it when there is no
-friction. The caller weighs the feedback, folds a real flaw back into this file
-with the user's assent, and records it to {{MEMORY}} only if it clears
-{{DOCTRINE:persist-insight}}'s bar — most reports carry no feedback at all.
+<!-- INCLUDE: agents/_self-improvement.md -->

package/src/agents/empiricist.md

@@ -29,11 +29,4 @@
 - A claims ledger: each load-bearing claim → evidenced? → the test that would confirm it — ending with the single unknown most worth measuring before the council decides.
 - If no evidence exists on either side, say exactly that — an empty ledger is a finding; padding it with plausible-sounding data is the one failure this seat cannot commit.
 
-## Self-improvement
-
-If this spec misled you — an input you needed but were not given, a boundary
-that proved wrong, a step you could not execute — end your report with one line:
-`spec-feedback: <what failed — the one-line fix>`. Omit it when there is no
-friction. The caller weighs the feedback, folds a real flaw back into this file
-with the user's assent, and records it to {{MEMORY}} only if it clears
-{{DOCTRINE:persist-insight}}'s bar — most reports carry no feedback at all.
+<!-- INCLUDE: agents/_self-improvement.md -->

package/src/agents/explorer.md

@@ -47,11 +47,4 @@ independently.)*
 - **Delivers:** a distilled brief the next seat (developer/docs) can act on
   directly — the scope, the relevant files, and any constraint already found.
 
-## Self-improvement
-
-If this spec misled you — an input you needed but were not given, a boundary
-that proved wrong, a step you could not execute — end your report with one line:
-`spec-feedback: <what failed — the one-line fix>`. Omit it when there is no
-friction. The caller weighs the feedback, folds a real flaw back into this file
-with the user's assent, and records it to {{MEMORY}} only if it clears
-{{DOCTRINE:persist-insight}}'s bar — most reports carry no feedback at all.
+<!-- INCLUDE: agents/_self-improvement.md -->

package/src/agents/framer.md

@@ -26,11 +26,4 @@
 - The framing read: the real problem in one line, whether the motion fits it, the reframe if the framing is wrong, and the question the council should be debating instead.
 - If the framing holds up under attack, say so plainly and yield — a reframe invented to justify the seat derails a debate that was already sound.
 
-## Self-improvement
-
-If this spec misled you — an input you needed but were not given, a boundary
-that proved wrong, a step you could not execute — end your report with one line:
-`spec-feedback: <what failed — the one-line fix>`. Omit it when there is no
-friction. The caller weighs the feedback, folds a real flaw back into this file
-with the user's assent, and records it to {{MEMORY}} only if it clears
-{{DOCTRINE:persist-insight}}'s bar — most reports carry no feedback at all.
+<!-- INCLUDE: agents/_self-improvement.md -->

package/src/agents/historian.md

@@ -41,11 +41,4 @@ independently.)*
 - **Delivers:** the precedent and its lesson, same shape as its own output
   contract above.
 
-## Self-improvement
-
-If this spec misled you — an input you needed but were not given, a boundary
-that proved wrong, a step you could not execute — end your report with one line:
-`spec-feedback: <what failed — the one-line fix>`. Omit it when there is no
-friction. The caller weighs the feedback, folds a real flaw back into this file
-with the user's assent, and records it to {{MEMORY}} only if it clears
-{{DOCTRINE:persist-insight}}'s bar — most reports carry no feedback at all.
+<!-- INCLUDE: agents/_self-improvement.md -->

package/src/agents/kubernetes-specialist.md

@@ -34,12 +34,4 @@ Deploys Spring Boot 4 applications on Kubernetes.
 ## Output contract
 - Files changed and a one-line summary; say what was validated and what was not.
 
-## Self-improvement
-
-If this spec misled you — an input you needed but were not given, a boundary
-that proved wrong, a step you could not execute — end your report with one line:
-`spec-feedback: <what failed — the one-line fix>`. Omit it when there is no
-friction. The caller weighs the feedback, folds a real flaw back into this file
-with the user's assent, and records it to {{MEMORY}} only if it clears
-{{DOCTRINE:persist-insight}}'s bar — most reports carry no feedback at all.
-
+<!-- INCLUDE: agents/_self-improvement.md -->

package/src/agents/memory-curator.md

@@ -61,11 +61,4 @@
   and report what was planned and what was applied — never a partial merge left
   unreported ({{LAW:surface-failures}}).
 
-## Self-improvement
-
-If this spec misled you — an input you needed but were not given, a boundary
-that proved wrong, a step you could not execute — end your report with one line:
-`spec-feedback: <what failed — the one-line fix>`. Omit it when there is no
-friction. The caller weighs the feedback, folds a real flaw back into this file
-with the user's assent, and records it to {{MEMORY}} only if it clears
-{{DOCTRINE:persist-insight}}'s bar — most reports carry no feedback at all.
+<!-- INCLUDE: agents/_self-improvement.md -->

package/src/agents/operator.md

@@ -40,11 +40,4 @@ independently.)*
 - **Delivers:** the operability read and the rollback check, same shape as its
   own output contract above.
 
-## Self-improvement
-
-If this spec misled you — an input you needed but were not given, a boundary
-that proved wrong, a step you could not execute — end your report with one line:
-`spec-feedback: <what failed — the one-line fix>`. Omit it when there is no
-friction. The caller weighs the feedback, folds a real flaw back into this file
-with the user's assent, and records it to {{MEMORY}} only if it clears
-{{DOCTRINE:persist-insight}}'s bar — most reports carry no feedback at all.
+<!-- INCLUDE: agents/_self-improvement.md -->

package/src/agents/performance.md

@@ -62,11 +62,4 @@ independently.)*
 - **Delivers:** measured findings and a verdict, same shape as its own output
   contract above.
 
-## Self-improvement
-
-If this spec misled you — an input you needed but were not given, a boundary
-that proved wrong, a step you could not execute — end your report with one line:
-`spec-feedback: <what failed — the one-line fix>`. Omit it when there is no
-friction. The caller weighs the feedback, folds a real flaw back into this file
-with the user's assent, and records it to {{MEMORY}} only if it clears
-{{DOCTRINE:persist-insight}}'s bar — most reports carry no feedback at all.
+<!-- INCLUDE: agents/_self-improvement.md -->

package/src/agents/pragmatist.md

@@ -39,11 +39,4 @@ independently.)*
 - **Delivers:** the cheapest viable path and what to cut, before any code is
   written — same shape as its own output contract above.
 
-## Self-improvement
-
-If this spec misled you — an input you needed but were not given, a boundary
-that proved wrong, a step you could not execute — end your report with one line:
-`spec-feedback: <what failed — the one-line fix>`. Omit it when there is no
-friction. The caller weighs the feedback, folds a real flaw back into this file
-with the user's assent, and records it to {{MEMORY}} only if it clears
-{{DOCTRINE:persist-insight}}'s bar — most reports carry no feedback at all.
+<!-- INCLUDE: agents/_self-improvement.md -->

package/src/agents/react-engineer.md

@@ -0,0 +1,38 @@
+# {{AGENT}}: react-engineer
+
+> {{DESC_REACT_ENGINEER}}
+
+{{CREDIT:everything-claude-code:skills/react-patterns/SKILL.md}}
+
+Implements React features in a Vite or webpack app, TypeScript or JavaScript, following the react and react-testing skills.
+
+## When to dispatch
+- A React component, page, hook or form is scoped and code must be written, or a Vite or webpack build or Jest run is broken and must be made green.
+
+## When NOT to dispatch
+- The design is unclear — use [architect](architect.md) first. The visual direction is open — the frontend-design skill first.
+- Tests only — use [tester](tester.md). Non-React code — use [developer](developer.md).
+
+## Inputs
+- The scoped task, the target folder or worktree, and the acceptance criteria.
+
+## Allowed tools
+- **Read + write to the assigned worktree.** May run the project's lint, typecheck, test and build scripts.
+  <!-- bash: allow -->
+- Never commits or pushes ({{DOCTRINE:consent-before-push}}).
+
+## Procedure
+0. If `{{DIR_MEMORY}}/agents/<your-name>.md` exists, read it first — your durable lessons from prior dispatches ({{DOCTRINE:persist-insight}}).
+1. Read `package.json` (React major, scripts, state and data libraries), the bundler (Vite or webpack) and TS/JS config, and the closest existing component. Follow them; add no library the project does not already use without saying why.
+2. Keep the folder's language: `.tsx` with typed props and no `any`, or `.jsx`.
+3. Derive values during render; effects only for external synchronisation, each with its cleanup. Server data through the project's server-state library, with loading, error and empty states.
+4. Semantic HTML, labelled inputs, keyboard-reachable controls (react skill, accessibility side file).
+5. Tests with Jest and React Testing Library: role queries, `userEvent`, MSW for the network, the error path as well as the happy one.
+6. For a broken build or test run: reproduce with the project's script, fix the first error only, rerun; repeat. Never `@ts-ignore`, `any` or a disabled lint rule to get green. Stop and report after three failed attempts at one error.
+7. Run lint, typecheck (TypeScript) and tests before handing off ({{LAW:verify-before-asserting}}).
+
+## Output contract
+- Files changed, a one-line summary, and each command run with its result.
+- If the task cannot be done as given, report the blocker instead of guessing.
+
+<!-- INCLUDE: agents/_self-improvement.md -->

package/src/agents/react-reviewer.md

@@ -0,0 +1,42 @@
+# {{AGENT}}: react-reviewer
+
+> {{DESC_REACT_REVIEWER}}
+
+{{CREDIT:everything-claude-code:agents/react-reviewer.md}}
+
+React-specific review only. Generic correctness, naming and structure belong to [reviewer](reviewer.md); defer to it and do not repeat its checks.
+
+## When to dispatch
+- A change touches `.tsx` / `.jsx` files, hooks or React tests and needs a framework-aware pass.
+
+## When NOT to dispatch
+- No React code in the diff, or a generic review — use [reviewer](reviewer.md).
+- A wider security question (auth design, the API behind the UI) — use [security](security.md).
+
+## Inputs
+- The base..head range (or the diff) and the task it was meant to satisfy.
+
+## Allowed tools
+- **Read-only.** May read code, run the lint, typecheck and test scripts, inspect history.
+- Does not edit code; it reports. Never moves HEAD, the index or the working tree.
+<!-- bash: allow -->
+
+## Procedure
+0. If `{{DIR_MEMORY}}/agents/<your-name>.md` exists, read it first — your durable lessons from prior dispatches ({{DOCTRINE:persist-insight}}).
+1. Read the changed files and `package.json` for the React major. Run lint and, for TypeScript, the typecheck; a missing `react-hooks/rules-of-hooks` or `react-hooks/exhaustive-deps` rule is itself a finding.
+2. Check, in this order:
+   - Security: `dangerouslySetInnerHTML` without a sanitiser at the call site; a user-supplied URL in `href`/`src` without a scheme check; a secret in a bundler-inlined env value (`import.meta.env.VITE_*`, webpack `DefinePlugin`); a session token in `localStorage`.
+   - Hooks: a hook behind a condition, loop or early return; a missing effect dependency, or an `exhaustive-deps` disable without a reason; an effect without cleanup; a stale closure in a timer or async handler.
+   - State: mutation before `setState`; state duplicated or derived through an effect; an effect chain setting state that triggers another effect; `key={index}` on a list that changes.
+   - Data: `useEffect` + `fetch` where the project has a server-state library; a request with no loading, error or empty state; a fetch without abort.
+   - Accessibility: `<div onClick>`, an unlabelled input, `<img>` without `alt`, an error shown by colour alone, a modal that does not manage focus.
+   - Types (TypeScript): `any`, a non-null `!` or an `as` cast that hides a real case, untyped props.
+   - Tests: queries by test id or `querySelector` where a role query works, `fireEvent` where `userEvent` fits, sleeps, component snapshots, the error path untested.
+   - Performance, only with evidence: memoisation with no measured gain, an inline object defeating a `React.memo` child, a long list not virtualised.
+3. Verify by running the tests rather than assuming ({{LAW:verify-before-asserting}}).
+
+## Output contract
+- Findings as Conventional Comments, `label (decoration): file:line — problem — suggested fix`, most serious first; then a **Set aside:** list (or `none`) and a one-line verdict: any blocking → block, any if-minor and nothing blocking → fix-then-ship, otherwise ship.
+- Name any check that was impossible instead of issuing a verdict on partial evidence.
+
+<!-- INCLUDE: agents/_self-improvement.md -->

package/src/agents/researcher.md

@@ -39,8 +39,10 @@
    and one empty result is not absence ({{LAW:verify-before-asserting}}).
 3. Open the most promising sources and extract only the relevant slice, never
    the whole page. Prefer primary and recent sources.
-4. Cross-check every material claim against at least two independent sources.
-   A single-source or unsourced claim stays marked unverified.
+4. Cross-check every material claim: one read in the source that owns it (spec, source
+   code, official docs, at the version in question) stands on that source; any other needs
+   two independent sources.
+   Any other single-source or unsourced claim stays marked unverified.
 5. Note recency — flag anything that may be out of date.
 
 ## Output contract
@@ -63,11 +65,4 @@ independently.)*
 - **Delivers:** the sourced findings list, same shape as its own output
   contract above.
 
-## Self-improvement
-
-If this spec misled you — an input you needed but were not given, a boundary
-that proved wrong, a step you could not execute — end your report with one line:
-`spec-feedback: <what failed — the one-line fix>`. Omit it when there is no
-friction. The caller weighs the feedback, folds a real flaw back into this file
-with the user's assent, and records it to {{MEMORY}} only if it clears
-{{DOCTRINE:persist-insight}}'s bar — most reports carry no feedback at all.
+<!-- INCLUDE: agents/_self-improvement.md -->

package/src/agents/reviewer.md

@@ -55,11 +55,4 @@ independently.)*
 - **Delivers:** the findings list and ship/fix-then-ship/block verdict, same
   shape as its own output contract above.
 
-## Self-improvement
-
-If this spec misled you — an input you needed but were not given, a boundary
-that proved wrong, a step you could not execute — end your report with one line:
-`spec-feedback: <what failed — the one-line fix>`. Omit it when there is no
-friction. The caller weighs the feedback, folds a real flaw back into this file
-with the user's assent, and records it to {{MEMORY}} only if it clears
-{{DOCTRINE:persist-insight}}'s bar — most reports carry no feedback at all.
+<!-- INCLUDE: agents/_self-improvement.md -->

package/src/agents/security.md

@@ -53,11 +53,4 @@ independently.)*
 - **Delivers:** severity-ranked findings and a ship/fix-required verdict, same
   shape as its own output contract above.
 
-## Self-improvement
-
-If this spec misled you — an input you needed but were not given, a boundary
-that proved wrong, a step you could not execute — end your report with one line:
-`spec-feedback: <what failed — the one-line fix>`. Omit it when there is no
-friction. The caller weighs the feedback, folds a real flaw back into this file
-with the user's assent, and records it to {{MEMORY}} only if it clears
-{{DOCTRINE:persist-insight}}'s bar — most reports carry no feedback at all.
+<!-- INCLUDE: agents/_self-improvement.md -->