@dzhechkov/harness-cli 0.3.259 → 0.3.260

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@dzhechkov/harness-cli",
-  "version": "0.3.259",
+  "version": "0.3.260",
   "description": "The dz CLI — install AI skills for Claude Code, Codex, OpenCode, Hermes, OpenClaude, GitHub Copilot. 35 commands, 13 presets, 6 platform targets.",
   "type": "module",
   "license": "MIT",
@@ -41,7 +41,7 @@
     "sbom.json"
   ],
   "dependencies": {
-    "@dzhechkov/harness-core": "^0.3.149",
+    "@dzhechkov/harness-core": "^0.3.150",
     "@dzhechkov/harness-presets": "^0.5.0",
     "@dzhechkov/scout": "^0.8.0",
     "@dzhechkov/skills-devops": "^0.3.0",

package/src/cli.ts

@@ -275,6 +275,10 @@ import {
   makeBacklogIO,
   harmonizeBacklog,
   BACKLOG_BACKENDS,
+  applyDomainBoost,
+  countDisplacedByCut,
+  renderDomainBoostNote,
+  renderDomainCutNote,
   parseReqeDebt,
   buildReqeBrief,
   settleReqeDebt,
@@ -333,7 +337,7 @@ Usage:
   dz teach "<pattern>" [--reward <0-1>] [--domain <name>] [--type rule|success-pattern|lesson-learned] [--project <dir>] [--no-mirror]   (--project pins the learned store to <dir>/.dz, not the cwd — pin to a canonical brain)
   dz teach --from-json <file> [--project <dir>] [--no-mirror]   (bulk-import a 'dz recall --all --json' export — share a learned store across machines)
   dz consolidate [--sessions-dir <dir>] [--project <dir>] [--no-mirror] [--prune-noise [--apply]] [--prune-quarantine [--apply]]   (both prunes: DRY-RUN by default; --apply snapshots then deletes; prune-quarantine = expired unproven lessons ONLY, never coupled to noise)
-  dz recall "<query>" [--limit <N>] [--semantic | --no-semantic] [--books [--book <slug>]] [--project <dir>] | dz recall --all [--json] | dz recall --usage [--json] | dz recall --forget <dzId>[,<dzId>] [--apply] | dz recall --promote <dzId>[,<dzId>] [--apply]   (forget/promote: dry-run default; forget snapshots before removing; promote lifts lesson-quarantine)
+  dz recall "<query>" [--limit <N>] [--domain <name>] [--semantic | --no-semantic] [--books [--book <slug>]] [--project <dir>] | dz recall --all [--json] | dz recall --usage [--json] | dz recall --forget <dzId>[,<dzId>] [--apply] | dz recall --promote <dzId>[,<dzId>] [--apply]   (--domain <name> BOOSTS lessons of that domain without dropping foreign ones — a shared store keeps its cross-domain transfers; forget/promote: dry-run default; forget snapshots before removing; promote lifts lesson-quarantine)
   dz vector status [--project <dir>] [--json]                          (semantic tier: engine, mirrored vs lexical counts, pending queue)
   dz vector reindex [--project <dir>] [--json]                         (snapshot, re-embed learned-pattern vectors, stamp current model)
   dz vector export <path> [--project <dir>]                            (portable VECTOR form (.rvf, opt-in RVF engine); patterns ship via recall --all --json)
@@ -2241,13 +2245,31 @@ async function cmdRecall(options: Map<string, string>, flags: Set<string>, cwd:
     : flags.has('no-semantic') || flags.has('lexical')
       ? ('lexical' as const)
       : ('hybrid' as const);
-  const result = await recallHybrid(projectRoot, query, { limit, mode });
+  const wantedDomain = options.get('domain');
+  // OVER-FETCH before boosting (Codex QE #5): the boost used to run on hits ALREADY
+  // truncated to `limit`, so an exact-domain lesson sitting at rank limit+1 could
+  // never receive its promised lift — the feature was weakest in exactly the case it
+  // exists for (foreign-domain dilution pushing a relevant lesson just past the cut).
+  // Fetch a bounded surplus, re-rank, then trim to the limit the caller asked for.
+  const fetchLimit = wantedDomain !== undefined ? Math.min(limit * 3, limit + 20) : limit;
+  const result = await recallHybrid(projectRoot, query, { limit: fetchLimit, mode });
+
   if (mode === 'semantic' && result.vectorEngine === 'none') {
     // --semantic is an explicit ask — degrading it silently would be dishonest (FR-3).
     write(`dz recall --semantic: ${result.vectorReason ?? 'no vector engine available — run: dz setup --memory agentdb'}`);
     return 1;
   }
-  const hits = result.hits;
+  // Domain-aware re-ranking (health-advisor slice H): `--domain <name>` lifts lessons
+  // tagged with that domain WITHOUT dropping foreign ones — a boost, not a filter, so a
+  // shared store keeps the cross-domain transfers that make it worth more than two stores.
+  const boost = wantedDomain !== undefined ? applyDomainBoost(result.hits, wantedDomain) : null;
+  const hits = (boost ? boost.hits : result.hits).slice(0, limit);
+  // The boost never drops a hit, but the CUT still can: promoting a match into the top
+  // `limit` pushes the last one out, so a lesson visible WITHOUT --domain can vanish
+  // WITH it. Cross-model review called this out as a lie by omission — the note said
+  // "foreign-domain lessons kept" about the pre-cut list while the printed list was
+  // missing one. Count it and say so; the reader can act on it (raise --limit).
+  const displaced = boost !== null ? countDisplacedByCut(result.hits, boost.hits, limit) : 0;
   if (asJson) {
     // Portable contract UNCHANGED (I-7/AC-6): a plain PatternRecord[] — round-trips through
     // `dz teach --from-json` regardless of which backend ranked each hit.
@@ -2258,6 +2280,10 @@ async function cmdRecall(options: Map<string, string>, flags: Set<string>, cwd:
   if (hits.length === 0) {
     write(`dz recall "${query}"`);
     write(`  No matching patterns (teach some with \`dz teach\`, or consolidate sessions).`);
+    // The domain note must print here too (Codex QE #10): a --domain run with zero hits
+    // silently said nothing about the domain, so the reader could not tell whether the
+    // boost had been applied and found nothing, or had not run at all.
+    if (boost !== null && wantedDomain !== undefined) write(renderDomainBoostNote(boost, wantedDomain));
     return 0;
   }
   const vectorOn = result.vectorEngine !== 'none' && result.vectorError === undefined && mode !== 'lexical';
@@ -2269,12 +2295,25 @@ async function cmdRecall(options: Map<string, string>, flags: Set<string>, cwd:
     const backendTag = vectorOn ? ` ⟨${h.backend}⟩` : '';
     const qTag = h.quarantined === true ? ' ⚠q' : '';
     if (h.quarantined === true) sawQuarantined = true;
-    write(`  [${h.pattern.reward.toFixed(2)}] (${h.pattern.domain})${backendTag}${qTag} ${h.pattern.pattern.slice(0, 80)}`);
+    // ONE line per hit, always. A lesson may contain newlines, and printing them raw
+    // let stored CONTENT forge lines that look like the tool's own output — including
+    // the domain-boost note that learning_bridge.py reads as a capability probe. Any
+    // consumer that parses this output line-wise has the same exposure, so the fix
+    // belongs at the point of rendering rather than in each reader.
+    const oneLine = h.pattern.pattern.replace(/[\r\n]+/g, ' ⏎ ').slice(0, 80);
+    write(`  [${h.pattern.reward.toFixed(2)}] (${h.pattern.domain})${backendTag}${qTag} ${oneLine}`);
   }
   if (sawQuarantined) {
     // The loop stays VISIBLE (ADR D2): a quarantined hit is shown, marked, and explained.
     write('  ⚠q = quarantined (unproven hypothesis, rank damped) — confirm with dz teach --reinforce, or dz recall --promote <dzId> --apply');
   }
+  if (boost !== null && wantedDomain !== undefined) {
+    // Say what the boost did — INCLUDING when it did nothing. A silent reorder would
+    // let the reader believe the ranking was domain-aware when it had no match to work with.
+    write(renderDomainBoostNote(boost, wantedDomain));
+    const cutNote = renderDomainCutNote(displaced, limit);
+    if (cutNote !== '') write(cutNote);
+  }
   if (result.vectorError !== undefined && mode !== 'lexical') {
     // Engine present but the semantic leg failed/timed out — one honest line, exit 0 (05 §2.3).
     write(`  ℹ vector search degraded: ${result.vectorError} — showing lexical ranking`);