@kienntpixon/dieu-phoi 0.2.0 → 0.2.1

package/CHANGELOG.md

@@ -1,5 +1,13 @@
 # Changelog — @kienntpixon/dieu-phoi
 
+## 0.2.1
+
+- **Định dạng `task_id` được kiểm:** đúng 8 ký tự `[a-z0-9]`. Một phiên thật đã tự bịa `almanac-cat-1791568325` làm root id mà không gì chặn. `plan-table --approve` (cả `--root` lẫn `task_id` từng bước), `choose --task`, `verdict --task`, `steer --task` giờ từ chối id sai: exit ≠ 0, lỗi tiếng Việt nêu id sai và gợi ý một id mới hợp lệ (`task_id "…" không hợp lệ: cần đúng 8 ký tự a-z0-9 (vd. k3f9x2ab)`), không ghi gì. `plan-table` không `--approve` chỉ cảnh báo trên stderr.
+- Lệnh mới `dieu-phoi new-id [--count n] [--json]`: in task_id mới hợp lệ, mỗi dòng một id (sinh bằng `crypto`), để điều phối viên không bao giờ tự bịa id.
+- Core (`hook.cjs`) export `isValidTaskId`, `newTaskId`, `taskIdError` — một chỗ duy nhất cho cả CLI và hook.
+- Hook không từ chối hay bỏ sự kiện: vẫn nhận dạng cũ `[A-Za-z0-9_-]{1,32}` để log/phiên cũ còn đọc được, nhưng `dispatch` có id sai định dạng mang thêm `task_id_invalid: true` (trường bổ sung, `schema` vẫn 1). `doctor` có dòng `task-ids`: CẢNH BÁO khi có dispatch như vậy trong 7 ngày gần đây.
+- Hook `SessionStart(compact)` thêm trường `systemMessage`: một dòng tiếng Việt cho người dùng, vd. `dieu-phoi: đã nạp lại trạng thái điều phối sau khi nén (6 task, 2 agent đang chạy, 2 bước chưa giao)` — cùng số đếm với dòng tiêu đề trạng thái. Phiên không có dispatch của dieu-phoi vẫn không in gì.
+
 ## 0.2.0
 
 - **Nâng cấp:** **dán lại `sheet/Code.gs` và deploy bản mới TRƯỚC khi nâng CLI** — Code.gs cũ bỏ qua (làm rơi) các trường mới `session_id, orch_ctx_tokens, orch_cum_usd, orch_ctx_baseline, orch_compactions` của bản ghi được đẩy lên trong lúc chưa cập nhật; sau đó `npm i -g @kienntpixon/dieu-phoi@0.2.0` rồi `dieu-phoi setup`.

package/README.md

@@ -43,7 +43,8 @@ Khởi động lại Claude Code sau khi cài. Kiểm tra bằng `dieu-phoi doct
 | `choose --type T --score N [--urgency U] [--reversible y\|n] [--verifiable y\|n] [--role R] [--task ID] [--override M/E \| --approved M/E] [--skill-dir P] [--seed S] [--json]` | đề xuất model + effort kèm lý do, `action`, `role`, ước tính chi phí/thời gian. `--override` cần `--task`. Kèm `orch_ctx_tokens`, `orch_ctx_growth`, `orch_ctx_over` (như `ctx`); vượt trần mềm → thêm một dòng cảnh báo |
 | `ctx [--session <id>] [--json]` | context phiên điều phối: `ctx_tokens` (request cuối), `ctx_baseline` (request đầu: system prompt + tools), `ctx_growth`, chi phí cộng dồn `cum_usd`, số lần đã nén `compactions`; `over` = `ctx_growth` > `orch_ctx_soft_cap` (profile, mặc định 60000 token tăng, chỉ được hạ). Session mặc định = `$CLAUDE_CODE_SESSION_ID`. JSON: `session_id, ctx_tokens, ctx_baseline, ctx_growth, cum_usd, compactions, soft_cap, over, advice`. Vượt trần → khuyên gõ `/compact` (trạng thái điều phối được nạp lại tự động). Đọc tăng dần qua cache `data/orch/`. Không đo được → exit 0, các số `null`, `over: false` |
 | `state [--session <id>] [--json]` | trạng thái điều phối của phiên dựng lại chỉ từ log: mỗi task — cổng (1/2/3/xong), vòng, tier, các lượt (agent, role, try, model/effort, running/done/pass/fail/timeout), verdict người dùng, steer cuối, thư mục chạy; nhóm theo kế hoạch đã duyệt (tier đã duyệt, bước "chưa giao", thư mục chạy theo root). Hook `SessionStart(compact)` tự nạp bản này vào ngữ cảnh sau `/compact` hoặc auto-compact. Bản chữ ≤ 8.000 ký tự |
-| `plan-table --file <steps.json> [--json] [--approve --root <root_task_id>]` | bảng phân công tiếng Việt (Bước \| Model/effort \| Rubric: điểm · kiểm chứng · đảo ngược \| Lý do \| Ước tính) + tổng chi phí + thời gian đường găng. Bước lỗi (override sai) hiện \"từ chối: …\", các bước khác vẫn hiện. Sau khi người dùng duyệt: `--approve --root R` ghi kế hoạch đã duyệt vào `data/plans/R.json` (cục bộ; `approval` = số lần duyệt, duyệt lại = +1, lần cũ vào `history`) và in thêm "Đã ghi kế hoạch đã duyệt (root R, lần duyệt n)". Từ chối (không ghi gì) khi bước thiếu `task_id`, `task_id` lặp, hoặc trùng `--root` |
+| `plan-table --file <steps.json> [--json] [--approve --root <root_task_id>]` | bảng phân công tiếng Việt (Bước \| Model/effort \| Rubric: điểm · kiểm chứng · đảo ngược \| Lý do \| Ước tính) + tổng chi phí + thời gian đường găng. Bước lỗi (override sai) hiện \"từ chối: …\", các bước khác vẫn hiện. Sau khi người dùng duyệt: `--approve --root R` ghi kế hoạch đã duyệt vào `data/plans/R.json` (cục bộ; `approval` = số lần duyệt, duyệt lại = +1, lần cũ vào `history`) và in thêm "Đã ghi kế hoạch đã duyệt (root R, lần duyệt n)". Từ chối (không ghi gì) khi bước thiếu `task_id`, `task_id` sai định dạng, `task_id` lặp, hoặc trùng `--root`; `--root` phải đúng 8 ký tự `a-z0-9` |
+| `new-id [--count n] [--json]` | in task_id mới hợp lệ (đúng 8 ký tự `a-z0-9`), mỗi dòng một id (mặc định 1, tối đa 1000; `--json`: mảng). Lấy mọi `task_id` / `root_task_id` từ đây — không tự bịa. `choose/verdict/steer --task` và `plan-table --approve` từ chối id sai định dạng (kèm gợi ý id mới); hook vẫn ghi id cũ nhưng gắn `task_id_invalid: true`, `doctor` cảnh báo |
 | `sync [--json]` | đẩy bản ghi chưa đồng bộ lên sink, lô ≤ 200, có thử lại. Bản ghi sink từ chối → `data/sync-rejected.jsonl`, `doctor` báo số lượng |
 | `retro [--json]` | tóm tắt cho vòng retro: số liệu từng ô, đếm `reason_code`, danh sách loại task, tóm tắt điều phối (`orchestration`) |
 | `uninstall [--purge]` | gỡ hook và đúng các quyền thư mục `runs` đã thêm khỏi settings (có sao lưu); `--purge` xoá cả `hook.cjs`. Dữ liệu giữ lại |

package/bin/dieu-phoi.cjs

@@ -30,6 +30,8 @@ Cách dùng: dieu-phoi <lệnh> [tuỳ chọn]
                          trạng thái điều phối của phiên dựng lại từ log (cũng được nạp tự động sau /compact)
   plan-table --file <steps.json> [--json] [--approve --root <root_task_id>]
                          bảng phân công: Bước | Model/effort | Lý do | Ước tính + tổng + đường găng
+  new-id [--count n] [--json]
+                         in task_id mới hợp lệ (8 ký tự a-z0-9), mỗi dòng một id — không tự bịa id
   sync                   đẩy bản ghi chưa đồng bộ lên sink
   retro [--json]         tóm tắt cho vòng retro (không chứa nội dung)
   uninstall [--purge]    gỡ hook khỏi settings.json (--purge xoá cả hook.cjs; dữ liệu giữ lại)
@@ -64,6 +66,7 @@ async function main(argv) {
     verdict: c.cmdVerdict,
     steer: c.cmdSteer,
     'plan-table': c.cmdPlanTable,
+    'new-id': c.cmdNewId,
     stats: c.cmdStats,
     choose: c.cmdChoose,
     ctx: c.cmdCtx,

package/hook.cjs

@@ -19,7 +19,7 @@ const path = require('path');
 const os = require('os');
 const crypto = require('crypto');
 
-const CORE_VERSION = '0.2.0';
+const CORE_VERSION = '0.2.1';
 const SCHEMA = 1;
 const TASK_TYPES = [
   'search', 'extract', 'mechanical-edit', 'feature-impl', 'debug', 'refactor',
@@ -226,6 +226,34 @@ function parseTag(text) {
   return tag;
 }
 
+// ------------------------------------------------------------------ task_id
+
+// CONTRACT: task_id = exactly 8 chars [a-z0-9], obtained from `dieu-phoi new-id` (never invented).
+// The tag parser still accepts the legacy 1–32 [A-Za-z0-9_-] form so old logs/sessions parse; such
+// dispatches are flagged `task_id_invalid: true` instead of being dropped.
+const TASK_ID_RE = /^[a-z0-9]{8}$/;
+const TASK_ID_ALPHABET = 'abcdefghijklmnopqrstuvwxyz0123456789';
+
+function isValidTaskId(s) {
+  return typeof s === 'string' && TASK_ID_RE.test(s);
+}
+
+/** Fresh random task_id: 8 chars [a-z0-9], unbiased (rejection sampling on crypto bytes). */
+function newTaskId() {
+  let out = '';
+  while (out.length < 8) {
+    for (const b of crypto.randomBytes(16)) {
+      if (b < 252 && out.length < 8) out += TASK_ID_ALPHABET[b % 36];
+    }
+  }
+  return out;
+}
+
+/** Vietnamese error for a bad id, with a freshly generated valid suggestion. */
+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\``;
+}
+
 // ------------------------------------------------------------------ models / pricing
 
 function normalizeModel(m) {
@@ -1056,11 +1084,23 @@ function stateBody(state, kept) {
  * Vietnamese text of a session state, at most `cap` chars. When too long, done tasks go first
  * (oldest first), then the oldest open ones; the count of omitted tasks is stated.
  */
-function formatSessionState(state, cap) {
-  const limit = cap || STATE_TEXT_CAP;
+/** Counts shared by the state header and the SessionStart systemMessage. */
+function stateCounts(state) {
   const open = state.tasks.filter((t) => t.gate !== 'done' && !t.pending);
   const pending = state.tasks.filter((t) => t.pending).length;
   const running = state.tasks.reduce((n, t) => n + t.attempts.filter((a) => a.status === 'running').length, 0);
+  return { tasks: state.tasks.length, open, pending, running };
+}
+
+/** One short Vietnamese line shown to the user after /compact re-injected the state. */
+function compactSystemMessage(state) {
+  const c = stateCounts(state);
+  return `dieu-phoi: đã nạp lại trạng thái điều phối sau khi nén (${c.tasks} task, ${c.running} agent đang chạy, ${c.pending} bước chưa giao)`;
+}
+
+function formatSessionState(state, cap) {
+  const limit = cap || STATE_TEXT_CAP;
+  const { open, pending, running } = stateCounts(state);
   const header = `Phiên ${state.session_id}: ${state.tasks.length} task (${open.length} chưa xong, ${pending} bước chưa giao, ${running} agent đang chạy). `
     + `Thư mục chạy: ${state.runs_dir} (chỉ trỏ tới, không tự đọc).`;
   if (!state.tasks.length) return `${header}\nChưa có task nào được giao trong phiên này.`;
@@ -1093,7 +1133,10 @@ function handleSessionStart(p, ctx) {
   const state = buildSessionState(events, sid, { plansDir: ctx.paths.plans });
   const text = `${COMPACT_PREFIX}\n${formatSessionState(state, STATE_TEXT_CAP)}`;
   return {
-    stdout: JSON.stringify({ hookSpecificOutput: { hookEventName: 'SessionStart', additionalContext: text } }),
+    stdout: JSON.stringify({
+      systemMessage: compactSystemMessage(state),
+      hookSpecificOutput: { hookEventName: 'SessionStart', additionalContext: text },
+    }),
   };
 }
 
@@ -1133,6 +1176,7 @@ function buildDispatch(p, ctx) {
     agent_id: agentId,
     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 } : {}),
     task_type: tag.task_type,
     score: tag.score,
     urgency: tag.urgency,
@@ -1444,10 +1488,10 @@ async function runSync(ctx, opts) {
 // ------------------------------------------------------------------ exports / main
 
 module.exports = {
-  runsDir, loadPlans, buildSessionState, formatSessionState, handleSessionStart, dispatchInSession, STATE_TEXT_CAP, COMPACT_PREFIX,
+  runsDir, loadPlans, 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, defaultClaudeDir, resolveHome, pathsFor, loadConfig, makeCtx,
-  logError, computeUserHash, userHash, commonFields, parseTag, normalizeModel, loadPricing,
+  logError, computeUserHash, userHash, commonFields, parseTag, TASK_ID_RE, isValidTaskId, newTaskId, taskIdError, normalizeModel, loadPricing,
   priceFor, usageBuckets, requestCost, parseJsonl, readTranscript, lastUuidOf,
   tagFromTranscript, summarizeTranscript, orchestratorUsage, orchCacheFile, findSessionTranscript, SESSION_ID_RE, HOOK_COLD_MAX_BYTES, appendEvent, eventLinesFrom, readEventLines, readEvents,
   loadSyncOffset, saveSyncOffset, recordRejected, unsyncedInfo, loadSegments, saveSegments, segmentFile,

package/lib/commands.cjs

@@ -15,6 +15,14 @@ const USER_OUTCOMES = ['accept', 'reject'];
 const CAUSES = ['quality', 'scope'];
 const STEER_KINDS = ['minor', 'plan', 'redirect'];
 const VERIFY = ['tests', 'lint', 'review', 'judgment', 'none'];
+const INVALID_ID_WINDOW_DAYS = 7;
+const NEW_ID_MAX = 1000;
+
+/** Throw the Vietnamese task_id error (with a fresh suggestion) unless `id` is a valid task_id. */
+function requireTaskId(id) {
+  if (!core.isValidTaskId(id)) throw new Error(core.taskIdError(id));
+  return id;
+}
 
 function skillDirOf(opts, ctx) {
   if (typeof opts['skill-dir'] === 'string') return path.resolve(opts['skill-dir']);
@@ -129,6 +137,11 @@ async function cmdDoctor(opts) {
   add(true, 'internal-subagents', skipped && skipped.count
     ? `bỏ qua ${skipped.count} SubagentStop của subagent nội bộ Claude Code (lần cuối ${skipped.last_ts})`
     : 'chưa bỏ qua SubagentStop nội bộ nào');
+  const since = Date.now() - INVALID_ID_WINDOW_DAYS * 86400000;
+  const badIds = events.filter((e) => e.kind === 'dispatch' && e.task_id_invalid === true && !(Date.parse(e.ts) < since));
+  add(!badIds.length, 'task-ids', badIds.length
+    ? `${badIds.length} dispatch trong ${INVALID_ID_WINDOW_DAYS} ngày gần đây có task_id sai định dạng (${Array.from(new Set(badIds.map((e) => e.task_id))).slice(0, 3).map((x) => `"${x}"`).join(', ')}) — task_id phải đúng 8 ký tự a-z0-9, lấy bằng \`dieu-phoi new-id\``
+    : `mọi dispatch trong ${INVALID_ID_WINDOW_DAYS} ngày gần đây có task_id đúng định dạng`, 'CẢNH BÁO');
   const orch = measureOrch(opts, ctx, profileOrNull(opts, ctx));
   if (orch.ctx_tokens !== null) {
     add(!orch.over, 'orch-ctx', `session ${orch.session_id}: ${orch.ctx_tokens} token, tăng ${orch.ctx_growth}/${orch.soft_cap}, ≈ ${fmtUsd(orch.cum_usd)} — ${orch.advice}`, 'CẢNH BÁO');
@@ -255,6 +268,7 @@ function verdictUser(opts, ctx, events) {
   const taskId = opts.task;
   const outcome = opts._[0];
   if (typeof taskId !== 'string' || !taskId) throw new Error('--task cần task_id');
+  requireTaskId(taskId);
   if (!USER_OUTCOMES.includes(outcome)) throw new Error(`verdict --task: outcome phải là ${USER_OUTCOMES.join('|')}`);
   let cause = typeof opts.cause === 'string' ? opts.cause : null;
   if (outcome === 'reject' && !CAUSES.includes(cause)) throw new Error(`reject cần --cause ${CAUSES.join('|')} (scope = yêu cầu đổi/thiếu, không phải lỗi model)`);
@@ -307,6 +321,7 @@ function cmdSteer(opts) {
   const kind = opts.kind;
   if (!ref && !opts.task) throw new Error('Cú pháp: dieu-phoi steer <agent_id> | --task <task_id> --kind minor|plan|redirect [--reason R]');
   if (!STEER_KINDS.includes(kind)) throw new Error(`--kind phải là ${STEER_KINDS.join('|')}`);
+  if (!ref) requireTaskId(String(opts.task));
   const ctx = ctxOf(opts);
   const events = core.readEvents(ctx);
   const steerId = crypto.randomBytes(4).toString('hex');
@@ -557,6 +572,7 @@ function cmdChoose(opts) {
   if (opts.override !== undefined && typeof opts.task !== 'string') {
     throw new Error('--override cần --task <task_id> (để học override theo từng task)');
   }
+  if (opts.task !== undefined) requireTaskId(typeof opts.task === 'string' ? opts.task : '');
   const input = chooseInput(opts, ctx, core.readEvents(ctx));
   const res = chooser.choose(input);
   const orch = measureOrch(opts, ctx, input.profile);
@@ -682,8 +698,6 @@ function buildPlanTable(rawSteps, base) {
   };
 }
 
-const ROOT_RE = /^[A-Za-z0-9_-]{1,32}$/;
-
 /**
  * Record the plan the user approved at gate 1: <data>/plans/<root>.json (local only — not an event, never
  * synced). Each re-approval increments `approval` (number of approvals for this root) and overwrites the
@@ -704,6 +718,9 @@ function planTaskIdProblems(plan, root) {
   const problems = [];
   const missing = plan.steps.filter((r) => !r.task_id).map((r) => r.step);
   if (missing.length) problems.push(`thiếu task_id ở bước ${missing.join(', ')}`);
+  for (const r of plan.steps) {
+    if (r.task_id && !core.isValidTaskId(r.task_id)) problems.push(`bước ${r.step}: ${core.taskIdError(r.task_id)}`);
+  }
   const by = new Map();
   for (const r of plan.steps) if (r.task_id) by.set(r.task_id, (by.get(r.task_id) || []).concat(r.step));
   for (const [id, steps] of by) if (steps.length > 1) problems.push(`task_id "${id}" bị dùng lại ở các bước ${steps.join(', ')}`);
@@ -738,15 +755,29 @@ function recordApprovedPlan(ctx, root, stepsFile, plan) {
   return Object.assign({ file }, rec);
 }
 
+// ------------------------------------------------------------------ new-id
+
+/** `dieu-phoi new-id [--count n]`: fresh valid task_ids, one per line (the orchestrator never invents ids). */
+function cmdNewId(opts) {
+  const raw = opts.count === undefined ? '1' : String(opts.count);
+  const n = /^\d+$/.test(raw) ? Number(raw) : NaN;
+  if (!Number.isInteger(n) || n < 1 || n > NEW_ID_MAX) throw new Error(`--count phải là số nguyên 1–${NEW_ID_MAX}`);
+  const ids = new Set();
+  while (ids.size < n) ids.add(core.newTaskId());
+  const list = Array.from(ids);
+  return { code: 0, out: opts.json ? JSON.stringify(list) : list.join('\n') };
+}
+
 function mdCell(s) {
   return String(s).replace(/\|/g, '\\|').replace(/\r?\n/g, ' ');
 }
 
 function cmdPlanTable(opts) {
   if (typeof opts.file !== 'string') throw new Error('Cú pháp: dieu-phoi plan-table --file <steps.json> [--json] [--approve --root <root_task_id>]');
-  if (opts.approve && (typeof opts.root !== 'string' || !ROOT_RE.test(opts.root))) {
-    throw new Error('--approve cần --root <root_task_id> (chữ, số, - _; tối đa 32 ký tự)');
+  if (opts.approve && typeof opts.root !== 'string') {
+    throw new Error(`--approve cần --root <root_task_id> (đúng 8 ký tự a-z0-9, vd. ${core.newTaskId()})`);
   }
+  if (opts.approve) requireTaskId(opts.root);
   let raw;
   try {
     raw = JSON.parse(fs.readFileSync(path.resolve(opts.file), 'utf8').replace(/^/, ''));
@@ -759,7 +790,7 @@ function cmdPlanTable(opts) {
   const plan = buildPlanTable(raw, base);
   const problems = planTaskIdProblems(plan, opts.approve ? String(opts.root) : null);
   if (opts.approve && problems.length) {
-    throw new Error(`plan-table --approve: không ghi kế hoạch — ${problems.join('; ')}. Mỗi bước cần task_id riêng, khác --root.`);
+    throw new Error(`plan-table --approve: không ghi kế hoạch — ${problems.join('; ')}. Mỗi bước cần task_id riêng (8 ký tự a-z0-9, lấy bằng \`dieu-phoi new-id\`), khác --root.`);
   }
   if (problems.length) process.stderr.write(`dieu-phoi plan-table: cảnh báo — ${problems.join('; ')} (cần sửa trước khi --approve)\n`);
   const rec = opts.approve ? recordApprovedPlan(ctx, String(opts.root), path.resolve(opts.file), plan) : null;
@@ -790,5 +821,5 @@ function cmdPlanTable(opts) {
 
 module.exports = {
   cmdSetup, cmdUninstall, cmdDoctor, cmdHook, cmdSync, cmdVerdict, cmdSteer, cmdStats, cmdRetro, cmdChoose, cmdCtx, cmdState, measureOrch,
-  cmdPlanTable, buildPlanTable, recordApprovedPlan, planTaskIdProblems, criticalPath, resolveAgentId, AGENT_OUTCOMES, USER_OUTCOMES, VERIFY,
+  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.2.0",
+  "version": "0.2.1",
   "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>",