@dzhechkov/harness-cli 0.3.259 → 0.3.261

package/.dz-manifest.json

@@ -9,7 +9,7 @@
       },
       {
         "path": "README.md",
-        "sha256": "6b3bcc252253732b9703f1eb43ac1f0e9f0599979b09d0fea3208932fd0f33a8"
+        "sha256": "61c71e355d633a88401a19c247e517d69f7a8d1483ea7549b3505eccb2a8740b"
       },
       {
         "path": "coverage/coverage-final.json",
@@ -41,15 +41,15 @@
       },
       {
         "path": "dist/cli.d.ts.map",
-        "sha256": "4c5fa8c51cc350874413efceb0a2e4dce2f0c71f39914ddd8be425336a37ae2d"
+        "sha256": "942d09c054994014e50139b807e1984586e172809b21e309114a5bf2ea92667c"
       },
       {
         "path": "dist/cli.js",
-        "sha256": "a75034dfe70f0b57811dfad831366bd5c7995583a57db0f218621b026ac8d854"
+        "sha256": "43b1280af2070d30296162cc33a045a23452201b2da2ce7aec12eea1900451f3"
       },
       {
         "path": "dist/cli.js.map",
-        "sha256": "dbe8fa418aedcb77dead3a374bba7cbf0fb5e305dd9a06002946a19604279684"
+        "sha256": "d0847603ca911ccca9e0838d54dadd1924ac0d9008e103c4edaef16dc4be0678"
       },
       {
         "path": "dist/index.d.ts",
@@ -73,7 +73,7 @@
       },
       {
         "path": "package.json",
-        "sha256": "72858a1f0199832c7ad781f892b7b117ec980af33ff9598587e1ed1544046a35"
+        "sha256": "582e9b8e1b919650b444b6be90cc3a1ae54e2fd190524fdb3ffa46c8f2058990"
       },
       {
         "path": "src/bin.ts",
@@ -81,7 +81,7 @@
       },
       {
         "path": "src/cli.ts",
-        "sha256": "1481238a2fc3a60051f678dcf6b415f8052ce954a9e55ee7577d8c3b2261e062"
+        "sha256": "e6b54081432afb9d3f1ee8a851c0e6b5ec8012dad4b9ae02a0d9036772226044"
       },
       {
         "path": "src/index.ts",
@@ -109,5 +109,5 @@
       }
     ]
   },
-  "signature": "uYnxiEzDxjwdXlp1SGaC0Zv2x/MW39skbUKw3+xpvnLcoNsEV4pp8kFD5q7urBvC87fC+lRKsIpatoXCNGzwBA=="
+  "signature": "y8gV9ZSni/bgEg5MSCmyhQ+Nr9Q22yNs4Ac2MNR5JFmL6hsn8VsOzxFdWlog2Fw2oyZN4ZuprbK4lUVU4GLSDw=="
 }

package/README.md

@@ -999,7 +999,7 @@ dz teach             "<pattern>" [--reward <0-1>] [--domain <name>] [--type rule
 dz teach --reinforce "<dzId-or-exact-text>" [--project <dir>] # bump an existing learned pattern instead of writing a near-duplicate
 dz teach --from-json <file> [--project <dir>] [--no-mirror] [--harmonize]   # bulk-import a `dz recall --all --json` export; prints a harmonize dry-run advisory
 dz consolidate       [--sessions-dir <dir>] [--project <dir>] [--no-mirror]
-dz recall            "<query>" [--limit <N>] [--semantic | --no-semantic] [--project <dir>]   # hybrid lexical+vector when the vector tier is enabled
+dz recall            "<query>" [--limit <N>] [--domain <name>] [--semantic | --no-semantic] [--project <dir>]   # hybrid lexical+vector when the vector tier is enabled; --domain BOOSTS same-domain lessons (never filters)
 dz recall --all      [--json] [--stats]             # export the WHOLE learned store, or inspect learning stats/top uses
 dz recall --books    "<query>" [--book <slug>]      # digitized-book KUs; --book narrows to one book
 dz vector status     [--project <dir>] [--json]     # semantic tier: engine availability, mirrored vs lexical counts, pending queue
@@ -2347,6 +2347,40 @@ guard, `--loc-cap <n>` tunes, default 700), **secret-scan** (the same high-signa
 edit caps/paths/waivers in `guards.config.json`; the project-critic role should then *not* re-flag these
 rules, only waivers without a reason.
 
+### Domain-scoped recall — `--domain <name>` (a boost, never a filter)
+
+**When to use it.** One brain store, more than one *kind* of work. `dz teach --domain` has always
+recorded a domain, but until now the ranking ignored it — so medical-research lessons and coding
+lessons came back interleaved, and the dilution grew with the store.
+
+```bash
+dz recall "signature" --limit 3 --domain security
+#   dz recall "signature"  —  3 hit(s), SQLite FTS5 + vector (agentdb) ranking
+#     [0.90] (security) ⟨both⟩ Crypto anti-hallucination anti-pattern: verifying a signature …
+#     [0.85] (publishing) ⟨both⟩ ⚠q 'Re-sign LAST' applies to EVERY signed pack a sweep touches …
+#     [0.80] (testing) ⟨vector⟩ A structural registry-coverage test that only checks key-SET …
+#     domain "security": among 6 candidate(s) — 1 exact match(es), 3 changed position;
+#     foreign-domain lessons kept (a boost, not a filter)
+#     1 lower-ranked lesson fell past the --limit 3 cut to make room — raise --limit to see them
+
+dz recall "signature" --limit 3 --domain health-research
+#     domain "health-research": no lesson in this result carries it — order unchanged, nothing was hidden
+```
+
+**Why a boost and not a filter.** Cross-domain transfer is the reason one shared store beats two
+separate ones: *"a reviewer's evidence needs the same execute-don't-describe discipline as your own
+claims"* was learned reviewing code and applies verbatim to medical sources. A filter would cut
+exactly those transfers. So matching hits move UP — at most **2 places** for an exact domain, **1**
+for a hierarchical relative (`health` ↔ `health-research`) — and nothing is ever dropped: the hit
+count with `--domain` equals the hit count without it.
+
+The bound matters: a domain tag is a hint about relevance, not evidence of it, so it breaks near-ties
+instead of overruling lexical ranking. And the note is honest in three directions — it says when
+nothing matched, it distinguishes *matched* from *actually moved*, and it counts what the `--limit`
+cut hid. That last one is the subtle part: the boost never drops a hit, but promoting a match into
+the top N does push the last one out of the printed list, so "nothing was dropped" would be true of
+the ranked list and false of the one you are reading. The note names the difference instead.
+
 ### Semantic recall (vector tier)
 
 `dz recall` is **hybrid** when the vector tier is available and **exactly the old lexical command** when it is not — enabling it never changes behavior for projects that skip it.

package/dist/cli.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"cli.d.ts","sourceRoot":"","sources":["../src/cli.ts"],"names":[],"mappings":"AAAA;;;;GAIG;AA0XH,2EAA2E;AAC3E,MAAM,WAAW,KAAK;IACpB,QAAQ,CAAC,GAAG,CAAC,EAAE,MAAM,CAAC;IACtB,QAAQ,CAAC,KAAK,CAAC,EAAE,CAAC,IAAI,EAAE,MAAM,KAAK,IAAI,CAAC;IACxC;;;;OAIG;IACH,QAAQ,CAAC,KAAK,CAAC,EAAE,MAAM,CAAC;IACxB;;;;;OAKG;IACH,QAAQ,CAAC,aAAa,CAAC,EAAE,iBAAiB,CAAC;CAC5C;AAED,yFAAyF;AACzF,MAAM,MAAM,iBAAiB,GAAG,CAC9B,GAAG,EAAE,MAAM,EACX,IAAI,EAAE;IAAE,QAAQ,CAAC,GAAG,EAAE,MAAM,CAAC;IAAC,QAAQ,CAAC,SAAS,EAAE,MAAM,CAAA;CAAE,KACvD;IAAE,QAAQ,EAAE,MAAM,CAAC;IAAC,MAAM,EAAE,MAAM,CAAC;IAAC,MAAM,EAAE,MAAM,CAAC;IAAC,QAAQ,CAAC,EAAE,OAAO,CAAA;CAAE,CAAC;AA+0O9E,wBAAsB,MAAM,CAAC,IAAI,EAAE,MAAM,EAAE,EAAE,EAAE,GAAE,KAAU,GAAG,OAAO,CAAC,MAAM,CAAC,CA8J5E"}
+{"version":3,"file":"cli.d.ts","sourceRoot":"","sources":["../src/cli.ts"],"names":[],"mappings":"AAAA;;;;GAIG;AA8XH,2EAA2E;AAC3E,MAAM,WAAW,KAAK;IACpB,QAAQ,CAAC,GAAG,CAAC,EAAE,MAAM,CAAC;IACtB,QAAQ,CAAC,KAAK,CAAC,EAAE,CAAC,IAAI,EAAE,MAAM,KAAK,IAAI,CAAC;IACxC;;;;OAIG;IACH,QAAQ,CAAC,KAAK,CAAC,EAAE,MAAM,CAAC;IACxB;;;;;OAKG;IACH,QAAQ,CAAC,aAAa,CAAC,EAAE,iBAAiB,CAAC;CAC5C;AAED,yFAAyF;AACzF,MAAM,MAAM,iBAAiB,GAAG,CAC9B,GAAG,EAAE,MAAM,EACX,IAAI,EAAE;IAAE,QAAQ,CAAC,GAAG,EAAE,MAAM,CAAC;IAAC,QAAQ,CAAC,SAAS,EAAE,MAAM,CAAA;CAAE,KACvD;IAAE,QAAQ,EAAE,MAAM,CAAC;IAAC,MAAM,EAAE,MAAM,CAAC;IAAC,MAAM,EAAE,MAAM,CAAC;IAAC,QAAQ,CAAC,EAAE,OAAO,CAAA;CAAE,CAAC;AAu3O9E,wBAAsB,MAAM,CAAC,IAAI,EAAE,MAAM,EAAE,EAAE,EAAE,GAAE,KAAU,GAAG,OAAO,CAAC,MAAM,CAAC,CA8J5E"}

package/dist/cli.js

@@ -15,7 +15,7 @@ assembleCandidates, renderPromotionReport, renderPromotionAdr, normalizePromotio
 // Cold-vs-warm EPOCH RUNNER (feature epoch-replay) — orchestrates + scores, never calls a model.
 replayableInstances, buildWorkOrder, buildJudgePrompts, unblindJudgments, verifyWorkOrder, isValidMargin, DIGEST_HONEST_SCOPE, scoreEpochReplay, generateMockOutcomes, renderEpochReplayResult, renderWorkOrderSummary, renderJudgePromptsSummary, WORK_ORDER_KIND, DEFAULT_MOCK_N, DEFAULT_MOCK_SEED, scoreRun, renderScorecard, renderCompoundingReport, readReinforcementState, readQuarantineState, registrationExitCode, renderRegistrationReport, 
 // Smart Backlog (feature smart-backlog) — goal-directed idea pipeline over the Brain vector engine.
-readBacklogConfig, readIdeas, writeIdeas, ideaId, dedupIdea, readGoalMap, readGoalMapDetailed, parseEffort, ensureBacklogGitignored, isSafeId, alignIdea, mirrorIdeaVector, snapshotIdeas, spinRoulette, rankRoulette, seededRng, eligibleIdeas, stageEnrichment, buildJiraDraft, resolveJiraAdapter, makeBacklogIO, harmonizeBacklog, BACKLOG_BACKENDS, parseReqeDebt, buildReqeBrief, settleReqeDebt, renderReqeList, REQE_SCOPE, } from '@dzhechkov/harness-core';
+readBacklogConfig, readIdeas, writeIdeas, ideaId, dedupIdea, readGoalMap, readGoalMapDetailed, parseEffort, ensureBacklogGitignored, isSafeId, alignIdea, mirrorIdeaVector, snapshotIdeas, spinRoulette, rankRoulette, seededRng, eligibleIdeas, stageEnrichment, buildJiraDraft, resolveJiraAdapter, makeBacklogIO, harmonizeBacklog, BACKLOG_BACKENDS, applyDomainBoost, countDisplacedByCut, renderDomainBoostNote, renderDomainCutNote, parseReqeDebt, buildReqeBrief, settleReqeDebt, renderReqeList, REQE_SCOPE, } from '@dzhechkov/harness-core';
 import { getPreset, PRESET_NAMES } from '@dzhechkov/harness-presets';
 import { scanGitHub, analyzeRepo, generateReport, deepAnalyze, scanAllSources, ScoutMemory } from '@dzhechkov/scout';
 const USAGE = `dz - DZ cross-platform harness CLI
@@ -61,7 +61,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)
@@ -1842,13 +1842,30 @@ async function cmdRecall(options, flags, cwd, write) {
         : flags.has('no-semantic') || flags.has('lexical')
             ? 'lexical'
             : 'hybrid';
-    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.
@@ -1858,6 +1875,11 @@ async function cmdRecall(options, flags, cwd, write) {
     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';
@@ -1870,12 +1892,31 @@ async function cmdRecall(options, flags, cwd, write) {
         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.
+        // Sanitise EVERY stored field that reaches the line, and every character a reader
+        // might treat as a break — not just CR/LF. Round 4 forged the domain-boost note
+        // twice over: once through `domain`, which was rendered raw, and once through a
+        // U+2028 that Python's splitlines() honours and this replace did not.
+        const oneLine = (v) => v.replace(/[\r\n\u2028\u2029\u0085\v\f]+/g, ' ⏎ ');
+        const shown = oneLine(h.pattern.pattern).slice(0, 80);
+        write(`  [${h.pattern.reward.toFixed(2)}] (${oneLine(h.pattern.domain)})${backendTag}${qTag} ${shown}`);
     }
     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`);