@kienntpixon/dieu-phoi 0.1.0 → 0.1.1

package/CHANGELOG.md

@@ -0,0 +1,11 @@
+# Changelog — @kienntpixon/dieu-phoi
+
+## 0.1.1
+
+- Sửa: Claude Code bắn `SubagentStop` cả cho subagent nội bộ của nó (không có transcript `agent-<id>.jsonl`, không qua Agent tool). Bản 0.1.0 ghi các lần này thành `complete` rỗng (`subagent_type ""`, `model null`, `tokens null`). Hook giờ bỏ qua khi transcript thiếu / rỗng / không có dòng assistant nào có usage **và** không có `dispatch` cho `agent_id` đó; số lần bỏ qua đếm ở `data/skipped-internal.json` (`doctor` hiển thị).
+- `stats` / `choose` / `retro` (và sink) bỏ qua các `complete` rỗng đã lỡ ghi bởi 0.1.0 — không cần sửa file log.
+- `fixtures/parity-events.jsonl` thêm hai bản ghi dạng này; kết quả parity không đổi.
+
+## 0.1.0
+
+- Bản đầu: hook PostToolUse(Agent) + SubagentStop, `setup/doctor/verdict/steer/stats/choose/plan-table/sync/retro/uninstall`.

package/README.md

@@ -82,6 +82,7 @@ Kết quả `--json`: `{"action":"dispatch","model":"sonnet","model_id":"claude-
 
 - Sự kiện: `~/.claude/dieu-phoi/data/events.jsonl` (`dispatch`, `complete`, `verdict`, `steer`, `override`; `schema: 1`), xem `CONTRACT.md`. File chỉ ghi nối (append-only), không bao giờ viết lại. Tiến độ đẩy lên sink nằm ở `data/sync-offset.json` (byte offset sau dòng cuối đã đẩy); trạng thái segment từng agent ở `data/segments/<agent_id>.json`.
 - Lỗi của hook: `~/.claude/dieu-phoi/hook-errors.log`.
+- Subagent nội bộ của Claude Code (không có transcript, không qua Agent tool) không được ghi; số lần bỏ qua ở `data/skipped-internal.json`. Log cũ của 0.1.0 có các `complete` rỗng loại này — `stats`/`retro` tự bỏ qua, không cần sửa file (từ 0.1.1).
 - Người dùng và phiên chỉ lưu dạng sha256 rút gọn 8 ký tự (người dùng = `git config user.email`, không có thì tên tài khoản máy).
 - Chi phí tính theo giá API trong `pricing.json` (USD / 1M token), đếm usage một lần cho mỗi `requestId`. Hệ số ghi cache (×1.25 cho 5 phút, ×2 cho 1 giờ) là **giả định**. Model không có giá → `cost_usd: null`. Ghi đè giá trong `config.json`:
 

package/hook.cjs

@@ -19,7 +19,7 @@ const path = require('path');
 const os = require('os');
 const crypto = require('crypto');
 
-const CORE_VERSION = '0.1.0';
+const CORE_VERSION = '0.1.1';
 const SCHEMA = 1;
 const TASK_TYPES = [
   'search', 'extract', 'mechanical-edit', 'feature-impl', 'debug', 'refactor',
@@ -119,6 +119,7 @@ function pathsFor(home) {
     data,
     events: path.join(data, 'events.jsonl'),
     segments: path.join(data, 'segments'),
+    skippedInternal: path.join(data, 'skipped-internal.json'),
     syncOffset: path.join(data, 'sync-offset.json'),
     syncSpawn: path.join(data, 'sync-spawn.json'),
     lastSync: path.join(data, 'last-sync.json'),
@@ -562,6 +563,33 @@ function handlePostToolUse(p, ctx) {
   return ev;
 }
 
+function hasAssistantUsage(lines) {
+  return lines.some((l) => l && l.type === 'assistant' && l.message && l.message.usage
+    && l.message.model !== '<synthetic>');
+}
+
+/** Whether events.jsonl holds a dispatch for this agent (cheap substring scan, rare path). */
+function hasDispatch(ctx, agentId) {
+  let txt;
+  try { txt = fs.readFileSync(ctx.paths.events, 'utf8'); } catch (_) { return false; }
+  const needle = `"agent_id":${JSON.stringify(agentId)}`;
+  let i = txt.indexOf(needle);
+  while (i >= 0) {
+    const start = txt.lastIndexOf('\n', i) + 1;
+    const end = txt.indexOf('\n', i);
+    if (txt.slice(start, end < 0 ? undefined : end).includes('"kind":"dispatch"')) return true;
+    i = txt.indexOf(needle, i + needle.length);
+  }
+  return false;
+}
+
+function countSkippedInternal(ctx) {
+  try {
+    const cur = readJsonFile(ctx.paths.skippedInternal, {}) || {};
+    writeFileAtomic(ctx.paths.skippedInternal, `${JSON.stringify({ count: (Number(cur.count) || 0) + 1, last_ts: nowIso() })}\n`);
+  } catch (_) { /* best effort */ }
+}
+
 function handleSubagentStop(p, ctx) {
   const agentId = typeof p.agent_id === 'string' && p.agent_id ? p.agent_id : null;
   if (!agentId) return null;
@@ -570,8 +598,15 @@ function handleSubagentStop(p, ctx) {
   let sum = null;
   let tag = null;
   const tpath = p.agent_transcript_path;
-  if (typeof tpath === 'string' && tpath && fs.existsSync(tpath)) {
-    const lines = readTranscript(tpath);
+  const exists = typeof tpath === 'string' && tpath && fs.existsSync(tpath);
+  const lines = exists ? readTranscript(tpath) : [];
+  // Claude Code also fires SubagentStop for its own internal subagents: no transcript on disk (or no
+  // API usage in it) and no Agent-tool dispatch. Those are not user work; record nothing.
+  if (!hasAssistantUsage(lines) && !hasDispatch(ctx, agentId)) {
+    countSkippedInternal(ctx);
+    return null;
+  }
+  if (exists) {
     const last = lastUuidOf(lines);
     if (last && seen.has(last)) return null; // same segment reported twice
     sum = summarizeTranscript(lines, { afterUuids: seen, pricing: loadPricing(ctx) });
@@ -815,7 +850,7 @@ module.exports = {
   priceFor, usageBuckets, requestCost, parseJsonl, readTranscript, lastUuidOf,
   tagFromTranscript, summarizeTranscript, appendEvent, eventLinesFrom, readEventLines, readEvents,
   loadSyncOffset, saveSyncOffset, recordRejected, unsyncedInfo, loadSegments, saveSegments, segmentFile,
-  extractAgentId, buildDispatch, handlePostToolUse, handleSubagentStop, maybeSpawnSync, runHook, runSync,
+  extractAgentId, hasAssistantUsage, hasDispatch, buildDispatch, handlePostToolUse, handleSubagentStop, maybeSpawnSync, runHook, runSync,
 };
 
 if (require.main === module) {

package/lib/aggregate.cjs

@@ -59,6 +59,19 @@ function normalizeVerdict(ev) {
   return { by, outcome: ev.outcome, cause, round: Number(ev.round) || null };
 }
 
+function blank(v) {
+  return v === '' || v === null || v === undefined;
+}
+
+/**
+ * `complete` written by core 0.1.0 for Claude Code's internal subagents (no transcript, no Agent-tool
+ * dispatch): subagent_type "" + tokens null + model null + no dispatch for that agent_id. Ignored.
+ */
+function isInternalComplete(ev, dispatched) {
+  return ev.kind === 'complete' && blank(ev.subagent_type) && blank(ev.model)
+    && (ev.tokens === null || ev.tokens === undefined) && !dispatched.has(ev.agent_id);
+}
+
 function steerIdOf(ev) {
   return typeof ev.steer_id === 'string' && ev.steer_id ? ev.steer_id : `${ev.agent_id}|${ev.ts}`;
 }
@@ -75,8 +88,10 @@ function buildAgents(events) {
     if (!map.has(id)) map.set(id, { agent_id: id, dispatch: null, completes: [], verdicts: [], steers: [] });
     return map.get(id);
   };
+  const dispatched = new Set(events.filter((ev) => ev && ev.kind === 'dispatch').map((ev) => ev.agent_id));
   events.forEach((ev, idx) => {
     if (!ev || typeof ev !== 'object') return;
+    if (isInternalComplete(ev, dispatched)) return;
     const e = Object.assign({ _idx: idx }, ev);
     if (ev.kind === 'override') { overrides.push(e); return; }
     if (ev.kind === 'verdict') {
@@ -332,6 +347,6 @@ function analyze(events, opts) {
 }
 
 module.exports = {
-  MIN_N, FAIL_OUTCOMES, HEAVY_STEERS, CANONICAL_CELL_FIELDS, normalizeVerdict, buildAgents, buildTasks,
+  MIN_N, FAIL_OUTCOMES, HEAVY_STEERS, CANONICAL_CELL_FIELDS, normalizeVerdict, isInternalComplete, buildAgents, buildTasks,
   buildCells, buildSteers, canonicalCells, analyze, cellKey, mean, percentile, tsOf, steerIdOf,
 };

package/lib/commands.cjs

@@ -120,6 +120,10 @@ async function cmdDoctor(opts) {
   let errSize = 0;
   try { errSize = fs.statSync(ctx.paths.errlog).size; } catch (_) { /* none */ }
   add(errSize === 0, 'hook-errors', errSize ? `${ctx.paths.errlog} có ${errSize} byte lỗi` : 'không có lỗi', 'CẢNH BÁO');
+  const skipped = core.readJsonFile(ctx.paths.skippedInternal, null);
+  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');
   if (ctx.config.sink_url && ctx.config.sink_token) {
     try {
       const u = new URL(ctx.config.sink_url);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@kienntpixon/dieu-phoi",
-  "version": "0.1.0",
+  "version": "0.1.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>",
@@ -15,7 +15,8 @@
     "hook.cjs",
     "pricing.json",
     "README.md",
-    "LICENSE"
+    "LICENSE",
+    "CHANGELOG.md"
   ],
   "engines": {
     "node": ">=18"