@agent-plan/core 0.2.19-next.14 → 0.2.19-next.16

package/dist/refs.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"refs.d.ts","sourceRoot":"","sources":["../src/refs.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;GAYG;AACH,OAAO,KAAK,EAAE,KAAK,EAAE,OAAO,EAAE,MAAM,aAAa,CAAC;AAKlD;;;;;;;GAOG;AACH,wBAAgB,cAAc,CAC5B,MAAM,EAAE,KAAK,EAAE,EACf,QAAQ,EAAE,OAAO,EAAE,EACnB,GAAG,EAAE,MAAM,GACV,KAAK,GAAG,SAAS,CAgCnB;AACD;;;;;;;;;;;;;;;;;GAiBG;AACH,OAAO,KAAK,EAAE,IAAI,EAAE,MAAM,aAAa,CAAC;AAexC,wBAAgB,aAAa,CAC3B,MAAM,EAAE,KAAK,EAAE,EACf,QAAQ,EAAE,OAAO,EAAE,EACnB,GAAG,EAAE,MAAM,GACV;IAAE,KAAK,EAAE,KAAK,CAAC;IAAC,IAAI,EAAE,IAAI,CAAA;CAAE,GAAG,SAAS,CAiD1C"}
+{"version":3,"file":"refs.d.ts","sourceRoot":"","sources":["../src/refs.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;GAYG;AACH,OAAO,KAAK,EAAE,KAAK,EAAE,OAAO,EAAE,MAAM,aAAa,CAAC;AAKlD;;;;;;;GAOG;AACH,wBAAgB,cAAc,CAC5B,MAAM,EAAE,KAAK,EAAE,EACf,QAAQ,EAAE,OAAO,EAAE,EACnB,GAAG,EAAE,MAAM,GACV,KAAK,GAAG,SAAS,CAgCnB;AACD;;;;;;;;;;;;;;;;;GAiBG;AACH,OAAO,KAAK,EAAE,IAAI,EAAE,MAAM,aAAa,CAAC;AAiBxC,wBAAgB,aAAa,CAC3B,MAAM,EAAE,KAAK,EAAE,EACf,QAAQ,EAAE,OAAO,EAAE,EACnB,GAAG,EAAE,MAAM,GACV;IAAE,KAAK,EAAE,KAAK,CAAC;IAAC,IAAI,EAAE,IAAI,CAAA;CAAE,GAAG,SAAS,CA2D1C"}

package/dist/refs.js

@@ -43,14 +43,19 @@ export function findPhaseByRef(phases, features, ref) {
 // Extract optional feature/phase/task numbers from a composite ref.
 // Accepts 1+ digits so "t3" == "t003". Numbers are GLOBAL (post-migration).
 function parseTaskComposite(ref) {
-    const t = ref.match(/t0*(\d+)/);
-    const p = ref.match(/p0*(\d+)/);
-    const f = ref.match(/f0*(\d+)/);
-    return {
-        taskNum: t ? parseInt(t[1], 10) : undefined,
-        phaseNum: p ? parseInt(p[1], 10) : undefined,
-        featureNum: f ? parseInt(f[1], 10) : undefined,
-    };
+    // Anchored patterns only: composite refs (F00x/P00x/T00x, P00x(F00x)/T00x,
+    // P00x/T00x) and bare T00x. A 5-char shortId (e.g. "HT23X") or a title never
+    // matches these because they are anchored to the WHOLE string.
+    const composite = ref.match(/^f0*(\d+)\/p0*(\d+)\/t0*(\d+)$/);
+    if (composite)
+        return { featureNum: parseInt(composite[1], 10), phaseNum: parseInt(composite[2], 10), taskNum: parseInt(composite[3], 10) };
+    const phaseTask = ref.match(/^p0*(\d+)(?:\(f0*(\d+)\))?\/t0*(\d+)$/);
+    if (phaseTask)
+        return { phaseNum: parseInt(phaseTask[1], 10), featureNum: phaseTask[2] ? parseInt(phaseTask[2], 10) : undefined, taskNum: parseInt(phaseTask[3], 10) };
+    const bare = ref.match(/^t0*(\d+)$/);
+    if (bare)
+        return { taskNum: parseInt(bare[1], 10) };
+    return {};
 }
 export function findTaskByRef(phases, features, ref) {
     const normalized = ref.trim().toLowerCase();
@@ -62,6 +67,16 @@ export function findTaskByRef(phases, features, ref) {
         if (task)
             return { phase, task };
     }
+    // 1b. ShortId (5-char) — check BEFORE composite so a shortId like "HT23X"
+    // or "T1234" is never misread as a numeric task ref (regexes are anchored now,
+    // but a 5-char ref can still look numeric). SHORT_ID_LENGTH is always 5.
+    if (normalized.length === 5) {
+        for (const phase of phases) {
+            const task = phase.tasks.find((t) => t.shortId && t.shortId.toLowerCase() === normalized);
+            if (task)
+                return { phase, task };
+        }
+    }
     // 2. Composite / short ref with a T segment
     const { featureNum, phaseNum, taskNum } = parseTaskComposite(normalized);
     if (taskNum !== undefined) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@agent-plan/core",
-  "version": "0.2.19-next.14",
+  "version": "0.2.19-next.16",
   "private": false,
   "description": "Harness-agnostic core for Agent Plan: schemas, persistence, ordering, status rollups, and markdown rendering.",
   "license": "MIT",