@mjasnikovs/pi-task 0.18.44 → 0.18.46

package/dist/task/batch-test-task.d.ts

@@ -15,8 +15,12 @@ export declare function mandatesTestsInSameChange(decisions: string, spec?: stri
  * the whole project, and it is not test INFRASTRUCTURE. A per-feature task that
  * carries "+ tests" never matches (its head names the feature), which is the
  * point: those are the cadence the decision asks for.
+ *
+ * `spec` is optional only so the detector stays callable on a bare title list;
+ * pass it whenever it is available, or unmarked spec echoes read as scope (see
+ * `stripSpecEchoes`).
  */
-export declare function findBatchTestTitles(titles: string[]): number[];
+export declare function findBatchTestTitles(titles: string[], spec?: string): number[];
 /**
  * The spec's own coverage source — the command whose report scopes the sweep.
  *

package/dist/task/batch-test-task.js

@@ -88,9 +88,11 @@ function splitDecisions(title) {
     const decisions = DECISIONS_CLAUSE_RE.exec(title);
     return { body, tail: decisions ? title.slice(decisions.index) : '' };
 }
+/** How decompose separates a title's head from its detail segments. */
+const SEGMENT_SPLIT_RE = /\s+[—–-]\s+|\s*\|\s*/;
 /** The head of a title: everything before the " — <detail>" separator. */
 function head(body) {
-    return body.split(/\s+[—–-]\s+|\s*\|\s*/)[0];
+    return body.split(SEGMENT_SPLIT_RE)[0];
 }
 /**
  * Remove double-quoted spans — they are QUOTED SPEC TEXT, never the task's own
@@ -105,6 +107,61 @@ function head(body) {
 function stripQuotedSpans(s) {
     return s.replace(/"[^"]*"/g, ' ').replace(/[“][^”]*[”]/g, ' ');
 }
+/**
+ * Normalize for echo comparison: case, markdown emphasis, backticks and all
+ * punctuation collapse away, so a title that reflows a spec line across a newline
+ * still matches the spec's own wording.
+ */
+function normalizeForEcho(s) {
+    return s
+        .toLowerCase()
+        .replace(/[^a-z0-9]+/g, ' ')
+        .trim();
+}
+/**
+ * A detail segment shorter than this is not treated as a citation even when it
+ * appears in the spec — short phrases ("with screenshot baselines") are shared
+ * vocabulary, not provenance, and stripping them would blind the scope check.
+ */
+const MIN_ECHO_CHARS = 40;
+/**
+ * Drop detail segments that are VERBATIM spec text.
+ *
+ * Third variant of the same failure, and the one the syntactic defenses miss:
+ * measured live (rep 4, Qwen3.6-27B on the mx5 fixture), the model appended its
+ * citation as BARE PROSE — no quotes, no `[source: …]` wrapper — and then
+ * repeated it inside a `[source: "…"]` clause:
+ *
+ *   "Implement Login page with phone/password form and tests — **Client/UI:**
+ *    Playwright `1.61.1` React component tests … — every component/page test
+ *    captures a screenshot committed as a baseline. [source: "…"]"
+ *
+ * `stripQuotedSpans` sees no quotes and `splitDecisions` cuts only at the clause,
+ * so the borrowed "every component/page" survived as if it were the task's own
+ * scope, and a correctly per-change-scoped Login task was DROPPED — the exact
+ * deletion this module exists to avoid.
+ *
+ * The general rule the two earlier guards were reaching for: text the title shares
+ * verbatim with the spec is provenance, whoever failed to mark it as such. So the
+ * check is semantic, against the spec, rather than one more citation dialect.
+ *
+ * The HEAD segment is never stripped: it is the task's own claim of what it
+ * delivers, and a batch title that happens to quote the spec must still be caught.
+ */
+function stripSpecEchoes(body, specNorm) {
+    if (specNorm === '')
+        return body;
+    const segments = body.split(SEGMENT_SPLIT_RE);
+    if (segments.length < 2)
+        return body;
+    const kept = segments.filter((seg, i) => {
+        if (i === 0)
+            return true;
+        const norm = normalizeForEcho(seg);
+        return norm.length < MIN_ECHO_CHARS || !specNorm.includes(norm);
+    });
+    return kept.join(' — ');
+}
 /** A title whose DELIVERABLE is tests: an authoring verb whose object is tests,
  *  with nothing else claimed in between ("Write component and page tests" ✓,
  *  "Add listings CRUD + tests" ✗ — the `+` marks tests as an additive constraint
@@ -133,12 +190,17 @@ const WHOLE_SCOPE_RE = /\b(?:all|every|each|entire|whole|comprehensive)\b[\w\s,/
  * the whole project, and it is not test INFRASTRUCTURE. A per-feature task that
  * carries "+ tests" never matches (its head names the feature), which is the
  * point: those are the cadence the decision asks for.
+ *
+ * `spec` is optional only so the detector stays callable on a bare title list;
+ * pass it whenever it is available, or unmarked spec echoes read as scope (see
+ * `stripSpecEchoes`).
  */
-export function findBatchTestTitles(titles) {
+export function findBatchTestTitles(titles, spec = '') {
+    const specNorm = normalizeForEcho(spec);
     const out = [];
     for (let i = 0; i < titles.length; i++) {
         const { body } = splitDecisions(titles[i]);
-        const scope = stripQuotedSpans(body);
+        const scope = stripSpecEchoes(stripQuotedSpans(body), specNorm);
         const h = head(scope);
         if (!TEST_AUTHORING_RE.test(h))
             continue;
@@ -278,7 +340,7 @@ export function buildSweepTitle(orphaned, coverageSource) {
 export function rewriteBatchTestPlan(titles, decisions, spec, requirementQuotes, isCrossCutting) {
     if (!mandatesTestsInSameChange(decisions, spec))
         return { titles, actions: [] };
-    const batch = findBatchTestTitles(titles);
+    const batch = findBatchTestTitles(titles, spec);
     if (batch.length === 0)
         return { titles, actions: [] };
     const batchSet = new Set(batch);

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@mjasnikovs/pi-task",
-    "version": "0.18.44",
+    "version": "0.18.46",
     "description": "Deterministic task planning and spec-orchestration for local models — crash-safe /task pipelines with verify/enforce gates, a real-time remote web view, and web/docs/fetch/worker subagent tools.",
     "type": "module",
     "main": "./dist/index.js",