@kienntpixon/dieu-phoi 0.5.2 → 0.5.3

package/CHANGELOG.md

@@ -1,5 +1,16 @@
 # Changelog — @kienntpixon/dieu-phoi
 
+## 0.5.3 (2026-10-22)
+
+Đề xuất core 1–3 của retro 2026-10-22 (`~/.dieu-phoi/runs/258ajcrr/retro-2026-10-22.md`). `CORE_VERSION` 0.5.3; đi cùng skill 1.4.3. Không cần deploy lại `sheet/Code.gs`.
+
+- **Thẻ sai được gắn cờ thay vì mất task_id:** 76 dp-exec ($407, 25% chi phí 10-19..21) mất `task_id` vì brief đặt id vào ô loại (`[dp:<id>|…]`, thiếu `task=`). Hook mới có `resolveTag` (`parseTag` giữ nguyên, gate gen-2 vẫn dùng bản chặt): `dispatch`/`complete` mang `tag_issue` = `type-is-id|bad-type|missing-task|no-tag` (nối `+`); thiếu `task=` mà ô loại là id (đúng định dạng, không phải loại đã biết, có chữ số hoặc được nhắc trong `dp-out-<id>`/`handoff-<id>`/`task=<id>` của brief) → `task_id` = id đó + `task_id_inferred: true`, loại lấy từ phần khác của thẻ nếu có. Id bắt đầu bằng chữ số (trước đây `parseTag` trả null, agent thành `untagged` và không có watchdog) giờ cũng được nhận. Loại hợp lệ = 12 loại cố định + `art-gen` + `extra_task_types` của profile.
+- **Cảnh báo trả về điều phối viên:** PostToolUse(Agent) in `additionalContext` khi thẻ bị gắn cờ (nêu lỗi, id đã suy hoặc "không vào stats", mẫu thẻ đúng, danh sách loại). Agent không phải `dp-*` và không có thẻ: không cờ, không in.
+- **Dispatch ghi tier `choose` đã đề xuất:** `choose --task` và `plan-table --approve` để lại `data/suggest/<task>-<role>.json`; dispatch kế tiếp của task + role đó ghi `suggested_tier` (tier choose tự chọn, trước override), `choose_tier` (tier choose trả về), `suggested_source`, `dispatched_tier`, `tier_mismatch`. Bản ghi > 14 ngày bị bỏ qua.
+- **Trần chuỗi handoff:** dispatch exec ghi `task_handoffs` (số agent đã handoff của task); từ handoff thứ 3 hook cảnh báo dừng tiếp nối và giao planner chia phần còn lại (retro: 15/63 task feature-impl handoff ≥ 2 lần; cq7yn1ws 10 handoff, $70, fail).
+- **Sửa hai lỗi nhỏ của cảnh báo thẻ:** `bad-type` nêu đúng token sai (trường `bad_type`) kể cả khi thẻ có thêm một loại hợp lệ (`[dp:foo|review|…]` báo "foo", không phải "review"); watchdog (`readHeadTag`) nay truyền `extra_task_types` của profile nên loại của profile không bị coi là `bad-type`.
+- Sink không thêm cột (các trường mới là chẩn đoán cục bộ; `suggested_tier` trùng tên cột sẵn có của `override`). Test mới `test/retro-1022.test.cjs`.
+
 ## 0.5.2
 
 - **`dieu-phoi prune --after-retro <file> [--yes]`:** dọn sau retro, dry-run mặc định. Run dir của task đã có verdict người dùng và mọi sự kiện ≤ mốc cắt (dòng `cutoff:` trong file retro, không có thì mtime) chỉ còn `retro-*.md` + dp-out exec/verify cuối (giữ = try cao nhất VÀ file mới nhất theo mtime cho mỗi task+role; file legacy không phân loại được và kết quả inline cuối cũng giữ); xoá file plan hỏng (tên không phải task id hoặc không đọc được); task còn mở không đụng. Event của task đã dọn chuyển vào `data/archive/events-<ngày>.jsonl.gz` (ghi + xác minh archive trước, rồi ghi lại `events.jsonl` qua file tạm + rename, giữ khoá sync); `readEvents` đọc cả archive (chống trùng nếu prune chết giữa chừng) nên retro/stats/choose/state không đổi số; sync chỉ đọc `events.jsonl`, offset được dịch theo phần đã bỏ, event chưa đẩy lên sink (khi có sink) không bị chuyển. Run dir có file mới hơn mốc cắt (việc đang chạy chưa vào log) không bị dọn. Từ chối gốc ổ đĩa, không đi theo symlink/junction, chỉ đụng runs dir + data dir.

package/hook.cjs

@@ -9,7 +9,8 @@
  * handoff watchdog), `--gate` (PreToolUse(Agent), gen-2 helper rules) and `--gate-release`. It must stay
  * self-contained: Node built-ins only, no requires of other package files.
  *
- * Hook rules: stdout only for the JSON of SessionStart(compact) / UserPromptSubmit, always exit 0,
+ * Hook rules: stdout only for the JSON of SessionStart(compact) / UserPromptSubmit / a PostToolUse(Agent) warning
+ * (core 0.5.3: bad dp tag, handoff chain), always exit 0,
  * no network.
  * Errors go to <home>/hook-errors.log.
  *
@@ -25,7 +26,7 @@ const path = require('path');
 const os = require('os');
 const crypto = require('crypto');
 
-const CORE_VERSION = '0.5.2';
+const CORE_VERSION = '0.5.3';
 const SCHEMA = 1;
 const TASK_TYPES = [
   'search', 'extract', 'mechanical-edit', 'feature-impl', 'debug', 'refactor',
@@ -126,6 +127,7 @@ function pathsFor(home) {
     events: path.join(data, 'events.jsonl'),
     segments: path.join(data, 'segments'),
     plans: path.join(data, 'plans'),
+    suggest: path.join(data, 'suggest'),
     skippedInternal: path.join(data, 'skipped-internal.json'),
     syncOffset: path.join(data, 'sync-offset.json'),
     syncSpawn: path.join(data, 'sync-spawn.json'),
@@ -263,6 +265,87 @@ function taskIdError(id) {
   return `task_id "${id}" không hợp lệ: cần đúng 8 ký tự a-z0-9 (vd. ${newTaskId()}) — lấy id bằng \`dieu-phoi new-id\``;
 }
 
+// ------------------------------------------------------------------ tag check (core 0.5.3, retro 2026-10-22)
+//
+// 76 dp-exec ($407, 25% of 10-19..21 cost) lost their task_id because the brief put the id in the type slot
+// (`[dp:<id>|…]`, no `task=`). parseTag stays strict (the gen-2 gate relies on it); resolveTag diagnoses the tag,
+// flags it (`tag_issue`) and recovers the task_id only when the guess is safe.
+
+const DEFAULT_EXTRA_TYPES = ['art-gen'];
+
+/** Fixed task types + the profile's extra_task_types (art-gen when no profile is readable). */
+function knownTaskTypes(ctx) {
+  const out = TASK_TYPES.concat(DEFAULT_EXTRA_TYPES);
+  const sd = ctx && ctx.config && ctx.config.skill_dir;
+  if (sd) {
+    const prof = readJsonFile(path.join(sd, 'profile', 'profile.json'), null);
+    const extra = prof && Array.isArray(prof.extra_task_types) ? prof.extra_task_types : [];
+    for (const t of extra) if (typeof t === 'string' && !out.includes(t)) out.push(t);
+  }
+  return out;
+}
+
+/**
+ * parseTag + diagnosis. null when the first line has no `[dp:` tag at all. Otherwise a tag (never dropped), plus:
+ * - `tag_issue`: `type-is-id` (an 8-char [a-z0-9] id in the type slot), `bad-type` (type not a known task type),
+ *   `missing-task` (no `task=`), joined with `+`;
+ * - `task_id_inferred: true` + `task_id` = the type-slot token, only when `task=` is missing, the token matches the
+ *   task_id format, is not a known type, and either holds a digit or the prompt names it in a run-file path /
+ *   `task=` (`dp-out-<id>`, `handoff-<id>`) — the same test that tells `type-is-id` from `bad-type` (an 8-letter label
+ *   such as "planning" stays a label). Then task_type = another tag part that is a known type, else `untagged`.
+ */
+function resolveTag(text, known) {
+  if (typeof text !== 'string') return null;
+  const body = text.replace(/^/, '');
+  const first = body.split(/\r?\n/, 1)[0].trim();
+  const m = /^\[dp:([^\]]*)\]/.exec(first);
+  if (!m) return null;
+  const types = Array.isArray(known) && known.length ? known : TASK_TYPES.concat(DEFAULT_EXTRA_TYPES);
+  const parts = m[1].split('|').map((s) => s.trim());
+  const head = (parts[0] || '').toLowerCase();
+  const rest = parts.slice(1);
+  let tag = parseTag(body);
+  // type slot unusable for parseTag (e.g. an id starting with a digit): parse the other parts on their own
+  if (!tag) tag = parseTag(`[dp:untagged|${rest.join('|')}]`);
+  const issues = [];
+  const altType = rest.map((p) => p.toLowerCase()).find((p) => types.includes(p)) || null;
+  // an id, not a label: id format, not a known type, and a digit or a run-file / task= mention ("planning" is a label)
+  const named = new RegExp(`(?:dp-out-|handoff-|task=)${escRe(head)}(?![a-z0-9])`).test(body.slice(first.length));
+  const idLike = TASK_ID_RE.test(head) && !types.includes(head) && (/\d/.test(head) || named);
+  if (idLike) issues.push('type-is-id');
+  else if (!types.includes(head)) {
+    issues.push('bad-type');
+    tag.bad_type = parts[0]; // the offending token (task_type may be replaced by a valid altType below)
+  }
+  if (!tag.task_id) {
+    issues.push('missing-task');
+    if (idLike) {
+      tag.task_id = head;
+      tag.task_id_inferred = true;
+    }
+  }
+  if (idLike || tag.task_type === 'untagged') tag.task_type = altType || 'untagged';
+  else if (!types.includes(tag.task_type) && altType) tag.task_type = altType;
+  if (issues.length) tag.tag_issue = issues.join('+');
+  return tag;
+}
+
+/** One Vietnamese line for the orchestrator about a flagged tag (null when the tag is fine). */
+function tagIssueMessage(tag, known) {
+  if (!tag || !tag.tag_issue) return null;
+  const what = [];
+  const codes = tag.tag_issue.split('+');
+  if (codes.includes('no-tag')) what.push('agent dp-* được giao mà không có thẻ `[dp:...]` ở dòng đầu prompt');
+  if (codes.includes('type-is-id')) what.push('ô đầu của thẻ là một task id, không phải loại task');
+  if (codes.includes('bad-type')) what.push(`loại "${tag.bad_type || tag.task_type}" không nằm trong danh sách loại hợp lệ`);
+  if (codes.includes('missing-task')) what.push('thiếu `task=<id>`');
+  const got = tag.task_id_inferred
+    ? ` Hook đã suy task_id=${tag.task_id} từ ô loại (log ghi \`task_id_inferred\`), loại ghi là "${tag.task_type}".`
+    : (tag.task_id ? '' : ' Lượt này KHÔNG có task_id: không vào stats/choose/retro.');
+  const types = (Array.isArray(known) && known.length ? known : TASK_TYPES.concat(DEFAULT_EXTRA_TYPES)).join(', ');
+  return `dieu-phoi: thẻ của agent vừa giao sai (tag_issue=${tag.tag_issue}): ${what.join('; ')}.${got} Lần giao sau dùng đúng \`[dp:<task_type>|s<n>|<urgency>|try<n>|task=<id>|role=<role>]\`, task_type là một trong: ${types}; id lấy bằng \`dieu-phoi new-id\`.`;
+}
+
 // ------------------------------------------------------------------ models / pricing
 
 function normalizeModel(m) {
@@ -387,10 +470,13 @@ function firstPromptText(lines) {
   return null;
 }
 
-/** Tag from the first user message of the transcript (the original prompt). */
-function tagFromTranscript(lines) {
+/**
+ * Tag from the first user message of the transcript (the original prompt). Core 0.5.3: resolveTag, so a tag with the
+ * id in the type slot still yields its task_id (complete events, watchdog handoff file name, inline results).
+ */
+function tagFromTranscript(lines, known) {
   const text = firstPromptText(lines);
-  return text === null ? null : parseTag(text);
+  return text === null ? null : resolveTag(text, known);
 }
 
 /**
@@ -1409,11 +1495,105 @@ function extractAgentId(tr) {
   return m ? m[1] : null;
 }
 
+// ------------------------------------------------------------------ core 0.5.3: choose's tier per task (retro 2026-10-22)
+//
+// `choose --task <id>` and `plan-table --approve` leave data/suggest/<task>-<role>.json; the next dispatch of that
+// task + role copies it as `suggested_tier` (choose's own pick, before a user override) and `choose_tier` (what
+// choose answered), next to the tier actually dispatched. Older than SUGGEST_MAX_AGE_MS -> ignored.
+
+const SUGGEST_MAX_AGE_MS = 14 * 24 * 3600 * 1000;
+
+function suggestFile(ctx, taskId, role) {
+  return path.join(ctx.paths.suggest, `${safeName(taskId)}-${safeName(role || 'exec')}.json`);
+}
+
+/**
+ * rec: { task_id, role?, suggested_tier, choose_tier?, source: 'choose'|'plan', keep_suggested? }. keep_suggested:
+ * a `choose --approved` answer carries no Thompson pick, so an earlier record's suggested_tier (from the plan table)
+ * is kept. Best effort: returns the written record or null.
+ */
+function recordSuggestion(ctx, rec) {
+  if (!rec || !isValidTaskId(rec.task_id) || typeof rec.suggested_tier !== 'string') return null;
+  const role = rec.role || 'exec';
+  const file = suggestFile(ctx, rec.task_id, role);
+  const prev = rec.keep_suggested ? readSuggestion(ctx, rec.task_id, role) : null;
+  const out = {
+    task_id: rec.task_id,
+    role,
+    suggested_tier: prev && prev.suggested_tier ? prev.suggested_tier : rec.suggested_tier,
+    choose_tier: rec.choose_tier || rec.suggested_tier,
+    source: prev && prev.suggested_tier ? prev.source : (rec.source || 'choose'),
+    ts: nowIso(),
+  };
+  try {
+    fs.mkdirSync(ctx.paths.suggest, { recursive: true });
+    writeFileAtomic(file, `${JSON.stringify(out)}\n`);
+    pruneSuggest(ctx);
+  } catch (e) {
+    logError(ctx.home, e, 'suggest');
+    return null;
+  }
+  return out;
+}
+
+function readSuggestion(ctx, taskId, role) {
+  if (!isValidTaskId(taskId)) return null;
+  const r = readJsonFile(suggestFile(ctx, taskId, role), null);
+  if (!r || typeof r !== 'object' || typeof r.suggested_tier !== 'string') return null;
+  const t = Date.parse(r.ts);
+  if (!(Date.now() - t <= SUGGEST_MAX_AGE_MS)) return null;
+  return r;
+}
+
+function pruneSuggest(ctx) {
+  let names = [];
+  try { names = fs.readdirSync(ctx.paths.suggest); } catch (_) { return; }
+  const now = Date.now();
+  for (const n of names) {
+    const f = path.join(ctx.paths.suggest, n);
+    try { if (now - fs.statSync(f).mtimeMs > 2 * SUGGEST_MAX_AGE_MS) fs.unlinkSync(f); } catch (_) { /* gone */ }
+  }
+}
+
+// ------------------------------------------------------------------ core 0.5.3: handoff chains (retro 2026-10-22)
+//
+// 15/63 feature-impl tasks handed off >= 2 times (19% of feature-impl cost); one chain ran 10 handoffs ($70) and
+// failed. From the 3rd handoff of a task the orchestrator is told to stop continuing and have a planner re-split.
+
+const HANDOFF_CHAIN_WARN = 3;
+
+/** Number of agents whose `complete` event has handoff:true for this task (substring scan of events.jsonl). */
+function countTaskHandoffs(ctx, taskId) {
+  if (!taskId) return 0;
+  let txt;
+  try { txt = fs.readFileSync(ctx.paths.events, 'utf8'); } catch (_) { return 0; }
+  const needle = `"task_id":${JSON.stringify(taskId)}`;
+  const agents = new Set();
+  let i = txt.indexOf(needle);
+  while (i >= 0) {
+    const start = txt.lastIndexOf('\n', i) + 1;
+    const end = txt.indexOf('\n', i);
+    const line = txt.slice(start, end < 0 ? undefined : end);
+    if (line.includes('"kind":"complete"') && line.includes('"handoff":true')) {
+      const m = /"agent_id":("(?:[^"\\]|\\.)*")/.exec(line);
+      agents.add(m ? m[1] : `@${start}`);
+    }
+    i = end < 0 ? -1 : txt.indexOf(needle, end);
+  }
+  return agents.size;
+}
+
+function handoffChainMessage(taskId, n) {
+  return `dieu-phoi: task ${taskId} đã handoff ${n} lần (trần ${HANDOFF_CHAIN_WARN}). Đừng nối tiếp thêm: nếu agent vừa giao là agent tiếp nối, cân nhắc dừng nó và giao planner (dp-scout) đọc handoff-${taskId}.md rồi chia phần còn lại thành bước mới (task_id mới từ \`dieu-phoi new-id\`).`;
+}
+
 function buildDispatch(p, ctx) {
   const ti = p.tool_input && typeof p.tool_input === 'object' ? p.tool_input : {};
   const tr = p.tool_response;
   const trObj = tr && typeof tr === 'object' && !Array.isArray(tr) ? tr : {};
-  const tag = parseTag(ti.prompt) || UNTAGGED;
+  const subagentType = typeof ti.subagent_type === 'string' && ti.subagent_type ? ti.subagent_type : 'general-purpose';
+  let tag = resolveTag(ti.prompt, knownTaskTypes(ctx));
+  if (!tag) tag = /^dp-/.test(subagentType) ? Object.assign({}, UNTAGGED, { tag_issue: 'no-tag' }) : UNTAGGED;
   const explicitEffort = typeof ti.effort === 'string' && ti.effort ? ti.effort : null;
   const parentEffort = p.effort && typeof p.effort === 'object' ? p.effort.level : null;
   // agent_id must never be empty (the sink keys on it): agentId -> tool_use_id -> random.
@@ -1432,6 +1612,8 @@ function buildDispatch(p, ctx) {
     session: p.session_id ? sha8(p.session_id) : null,
     task_id: tag.task_id,
     ...(tag.task_id && !isValidTaskId(tag.task_id) ? { task_id_invalid: true } : {}),
+    ...(tag.task_id_inferred ? { task_id_inferred: true } : {}),
+    ...(tag.tag_issue ? { tag_issue: tag.tag_issue } : {}),
     task_type: tag.task_type,
     score: tag.score,
     urgency: tag.urgency,
@@ -1440,7 +1622,7 @@ function buildDispatch(p, ctx) {
     model: normalizeModel(trObj.resolvedModel) || normalizeModel(ti.model) || null,
     effort: explicitEffort || parentEffort || null,
     effort_inherited: !explicitEffort,
-    subagent_type: typeof ti.subagent_type === 'string' && ti.subagent_type ? ti.subagent_type : 'general-purpose',
+    subagent_type: subagentType,
     background: ti.run_in_background === true || trObj.isAsync === true,
     // Orchestrator (main session) at the moment of this dispatch. Raw session_id joins with `dieu-phoi ctx`.
     session_id: typeof p.session_id === 'string' && p.session_id ? p.session_id : null,
@@ -1449,15 +1631,54 @@ function buildDispatch(p, ctx) {
     orch_compactions: orch.compactions,
     orch_cum_usd: orch.cum_usd,
   }, tag.parent_task_id ? { parent_task_id: tag.parent_task_id } : {},
+  tierFields(ctx, tag, normalizeModel(trObj.resolvedModel) || normalizeModel(ti.model) || null, explicitEffort || parentEffort || null),
   // core 0.5.0: an Agent call made inside a subagent (gen-2) carries the caller's agent_id
   typeof p.agent_id === 'string' && p.agent_id ? { parent_agent_id: p.agent_id } : {},
   commonFields(ctx));
 }
 
+/** suggested_tier / choose_tier / suggested_source / dispatched_tier / tier_mismatch, only when choose left a record. */
+function tierFields(ctx, tag, model, effort) {
+  const s = tag.task_id ? readSuggestion(ctx, tag.task_id, tag.role) : null;
+  if (!s) return {};
+  const dispatched = tierOf(model, effort);
+  const out = {
+    suggested_tier: s.suggested_tier,
+    choose_tier: s.choose_tier || s.suggested_tier,
+    suggested_source: s.source || null,
+    dispatched_tier: dispatched,
+  };
+  if (dispatched && !dispatched.includes('?')) out.tier_mismatch = dispatched !== out.choose_tier;
+  return out;
+}
+
+/**
+ * Core 0.5.3: besides the dispatch event, a dieu-phoi dispatch with a flagged tag (tag_issue) or a task already on
+ * its HANDOFF_CHAIN_WARN-th handoff gets a warning back to the caller as PostToolUse additionalContext (the
+ * returned event carries it as a non-enumerable `stdout`, never written to the log).
+ */
 function handlePostToolUse(p, ctx) {
   if (p.tool_name !== 'Agent' && p.tool_name !== 'Task') return null;
   const ev = buildDispatch(p, ctx);
+  const notes = [];
+  if (ev.tag_issue) {
+    const ti = p.tool_input && typeof p.tool_input === 'object' ? p.tool_input : {};
+    const known = knownTaskTypes(ctx);
+    const tag = resolveTag(ti.prompt, known) || { tag_issue: ev.tag_issue, task_id: null, task_type: ev.task_type };
+    notes.push(tagIssueMessage(tag, known));
+  }
+  if (ev.task_id && ev.role === 'exec') {
+    const n = countTaskHandoffs(ctx, ev.task_id);
+    if (n) ev.task_handoffs = n;
+    if (n >= HANDOFF_CHAIN_WARN) notes.push(handoffChainMessage(ev.task_id, n));
+  }
   appendEvent(ctx, ev);
+  if (notes.length) {
+    Object.defineProperty(ev, 'stdout', {
+      enumerable: false,
+      value: JSON.stringify({ hookSpecificOutput: { hookEventName: 'PostToolUse', additionalContext: notes.join('\n') } }),
+    });
+  }
   return ev;
 }
 
@@ -1626,7 +1847,7 @@ function handleSubagentStop(p, ctx) {
     const last = lastUuidOf(lines);
     if (last && seen.has(last)) return null; // same segment reported twice
     sum = summarizeTranscript(lines, { afterUuids: seen, pricing: loadPricing(ctx) });
-    tag = tagFromTranscript(lines);
+    tag = tagFromTranscript(lines, knownTaskTypes(ctx));
   }
   const t = tag || UNTAGGED;
   const ev = Object.assign({
@@ -1648,7 +1869,7 @@ function handleSubagentStop(p, ctx) {
     tokens: sum ? sum.tokens : null,
     cost_usd: sum ? sum.cost_usd : null,
     model: sum ? sum.model : null,
-  }, commonFields(ctx));
+  }, t.task_id_inferred ? { task_id_inferred: true } : {}, t.tag_issue ? { tag_issue: t.tag_issue } : {}, commonFields(ctx));
   // Core 0.5.2 (retro 2026-10-18): how the segment ended, measured here instead of re-reading transcripts later.
   // Labels and numbers only, never text.
   const why = exists ? abortReasonOf(lines) : null;
@@ -1821,7 +2042,7 @@ function readSlice(fd, start, len) {
  * Tag of the subagent's first prompt, read from the head of its transcript only.
  * Returns { found: false } while no user line is on disk yet (transcript written async), else { found: true, tag|null }.
  */
-function readHeadTag(file) {
+function readHeadTag(file, known) {
   let fd;
   try { fd = fs.openSync(file, 'r'); } catch (_) { return { found: false, tag: null }; }
   try {
@@ -1831,7 +2052,7 @@ function readHeadTag(file) {
       const text = readSlice(fd, 0, len).toString('utf8');
       const complete = len === size ? text : text.slice(0, text.lastIndexOf('\n') + 1);
       const lines = parseJsonl(complete);
-      if (lines.some((l) => l && l.type === 'user' && l.message)) return { found: true, tag: tagFromTranscript(lines), prompt: firstPromptText(lines) };
+      if (lines.some((l) => l && l.type === 'user' && l.message)) return { found: true, tag: tagFromTranscript(lines, known), prompt: firstPromptText(lines) };
       if (len >= size || len >= WATCH_HEAD_MAX) return { found: false, tag: null };
     }
   } finally {
@@ -1902,7 +2123,7 @@ function watchState(p, ctx, agentId) {
   if (!st || typeof st !== 'object') {
     st = null;
     if (tpath) {
-      const h = readHeadTag(tpath);
+      const h = readHeadTag(tpath, knownTaskTypes(ctx));
       if (h.found) {
         st = h.tag
           ? { dp: true, task_id: h.tag.task_id, run_id: h.tag.task_id ? null : runIdFromPrompt(h.prompt), role: h.tag.role, parent_task_id: h.tag.parent_task_id || null, fired: 0, t: Date.now() }
@@ -2328,7 +2549,8 @@ module.exports = {
   runsDir, loadPlans, isDieuPhoiPrompt, foldPrompt, sinkStatusCached, sinkUrlKey, SINK_CACHE_TTL_MS, scoutBriefPath, quickHealth, startRun, startInfo, orchLessons, orchLessonLines, abortReasonOf, completionStatus, logGate, QUOTA_PAUSE_MARKER, stepIds, START_STEP_IDS, healthLine, formatStartBlock, handleUserPromptSubmit, START_PREFIX, buildSessionState, formatSessionState, stateCounts, compactSystemMessage, handleSessionStart, dispatchInSession, STATE_TEXT_CAP, COMPACT_PREFIX,
   CORE_VERSION, SCHEMA, TASK_TYPES, URGENCIES, ROLES, MODEL_ALIASES, UNTAGGED,
   sha8, nowIso, readJsonFile, writeFileAtomic, renameWithRetry, acquireLock, defaultClaudeDir, resolveHome, pathsFor, loadConfig, makeCtx,
-  logError, computeUserHash, userHash, commonFields, parseTag, TASK_ID_RE, isValidTaskId, newTaskId, taskIdError, normalizeModel, loadPricing,
+  logError, computeUserHash, userHash, commonFields, parseTag, resolveTag, knownTaskTypes, tagIssueMessage, DEFAULT_EXTRA_TYPES,
+  recordSuggestion, readSuggestion, suggestFile, SUGGEST_MAX_AGE_MS, countTaskHandoffs, handoffChainMessage, HANDOFF_CHAIN_WARN, tierFields, TASK_ID_RE, isValidTaskId, newTaskId, taskIdError, normalizeModel, loadPricing,
   priceFor, usageBuckets, requestCost, parseJsonl, readTranscript, lastUuidOf,
   firstPromptText, tagFromTranscript, summarizeTranscript,
   RESULT_ROLES, BLOCKED_RESULT_NAME_RE, resultFileName, findResultFiles, resultRunDir, finalAssistantText, persistInlineResult, orchestratorUsage, orchCacheFile, findSessionTranscript, SESSION_ID_RE, HOOK_COLD_MAX_BYTES, appendEvent, eventLinesFrom, readEventLines, readEvents, readEventsBuffer, archiveFiles, readArchiveLines,

package/lib/commands.cjs

@@ -186,13 +186,13 @@ const TMP_STALE_MS = 3600 * 1000;
 
 /**
  * Leftovers of writeFileAtomic (tmp written, rename never happened: process killed, or rename lost a race on
- * Windows) under data/, data/orch, data/segments, data/plans. Older than 1 h → removed; newer → reported.
+ * Windows) under data/, data/orch, data/segments, data/plans, data/suggest. Older than 1 h → removed; newer → reported.
  */
 function sweepStaleTmp(ctx) {
   let removed = 0;
   const left = [];
   const now = Date.now();
-  for (const dir of [ctx.paths.data, path.join(ctx.paths.data, 'orch'), ctx.paths.segments, ctx.paths.plans]) {
+  for (const dir of [ctx.paths.data, path.join(ctx.paths.data, 'orch'), ctx.paths.segments, ctx.paths.plans, ctx.paths.suggest]) {
     let names = [];
     try { names = fs.readdirSync(dir); } catch (_) { continue; }
     for (const n of names) {
@@ -788,6 +788,13 @@ function cmdChoose(opts) {
     const tid = input.taskId;
     core.appendEvent(ctx, overrideEvent(ctx, tid, res.task_type || String(input.type).toLowerCase(), res.suggested_tier, res.tier));
   }
+  // core 0.5.3: the next dispatch of this task + role logs suggested_tier / choose_tier next to the tier it ran on
+  if (input.taskId && res.action === 'dispatch' && res.tier) {
+    core.recordSuggestion(ctx, {
+      task_id: input.taskId, role: res.role || input.role, suggested_tier: res.suggested_tier || res.tier, choose_tier: res.tier,
+      source: 'choose', keep_suggested: input.approved !== null && !res.overridden,
+    });
+  }
   if (opts.json) return { code: 0, out: JSON.stringify(res) };
   const typeWarn = res.type_warning && !String(res.reason).includes(res.type_warning) ? `\nCẢNH BÁO: ${res.type_warning}` : '';
   const warn = `${typeWarn}${res.orch_ctx_warn ? `\nCẢNH BÁO: ${orch.advice}` : ''}`;
@@ -954,6 +961,19 @@ function planOverrideEvents(ctx, plan, events) {
   return out;
 }
 
+/** Core 0.5.3: each approved step with a task_id leaves its choose tier for the step's dispatch (data/suggest/). */
+function recordPlanSuggestions(ctx, plan) {
+  let n = 0;
+  for (const r of plan.steps) {
+    if (!r.task_id || !r.tier || r.action !== 'dispatch') continue;
+    const ok = core.recordSuggestion(ctx, {
+      task_id: r.task_id, role: r.role || 'exec', suggested_tier: r.suggested_tier || r.tier, choose_tier: r.tier, source: 'plan',
+    });
+    if (ok) n++;
+  }
+  return n;
+}
+
 function recordApprovedPlan(ctx, root, stepsFile, plan) {
   const file = path.join(ctx.paths.plans, `${root}.json`);
   const prev = core.readJsonFile(file, null);
@@ -1021,6 +1041,7 @@ function cmdPlanTable(opts) {
   const rec = opts.approve ? recordApprovedPlan(ctx, String(opts.root), path.resolve(opts.file), plan) : null;
   const ovs = rec ? planOverrideEvents(ctx, plan, base.events) : [];
   for (const ev of ovs) core.appendEvent(ctx, ev);
+  if (rec) recordPlanSuggestions(ctx, plan);
   if (opts.json) {
     const out = rec ? Object.assign({}, plan, {
       approved: { root_task_id: rec.root_task_id, approval: rec.approval, record: rec.file, overrides_logged: ovs.length },
@@ -1050,6 +1071,6 @@ function cmdPlanTable(opts) {
 }
 
 module.exports = {
-  cmdSetup, cmdUninstall, cmdDoctor, cmdStart, checkSinkLive, cmdHook, cmdSync, cmdVerdict, cmdSteer, cmdDiscard, ctxWarnOnce, ctxWarnLevel, sweepStaleTmp, planOverrideEvents, cmdStats, cmdRetro, cmdPrune: (o) => require('./prune.cjs').cmdPrune(o), cmdChoose, cmdCtx, cmdState, measureOrch,
+  cmdSetup, cmdUninstall, cmdDoctor, cmdStart, checkSinkLive, cmdHook, cmdSync, cmdVerdict, cmdSteer, cmdDiscard, ctxWarnOnce, ctxWarnLevel, sweepStaleTmp, planOverrideEvents, recordPlanSuggestions, cmdStats, cmdRetro, cmdPrune: (o) => require('./prune.cjs').cmdPrune(o), cmdChoose, cmdCtx, cmdState, measureOrch,
   cmdPlanTable, cmdNewId, requireTaskId, buildPlanTable, recordApprovedPlan, planTaskIdProblems, criticalPath, resolveAgentId, AGENT_OUTCOMES, USER_OUTCOMES, VERIFY,
 };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@kienntpixon/dieu-phoi",
-  "version": "0.5.2",
+  "version": "0.5.3",
   "description": "Hook + CLI ghi log subagent của Claude Code và chọn model/effort theo chi phí + thời gian tới lúc nghiệm thu.",
   "license": "MIT",
   "author": "kiennt <kiennt@pixon.games>",