skalpel 4.0.49 → 4.0.50

package/codex-normalize.mjs

@@ -141,13 +141,28 @@ export function loadCodexEntries(file, maxBytes = DEFAULT_TAIL_BYTES) {
     }
     if (e.type !== "response_item") continue;
     const p = e.payload || {};
+    // The rollout event's OWN wall-clock timestamp (top-level ISO string on every response_item —
+    // verified against real ~/.codex rollouts). Claude transcript entries carry `.timestamp` natively
+    // (loadClaudeEntries pushes the raw object), and downstream consumers (efficacy-eval, first-run,
+    // eval-harness) index `e.timestamp` per entry to compute the MEASURED fix-delta: t(later same-proof
+    // PASS) − t(the false "done"). Codex normalized entries omitted it, so every Codex session was silently
+    // forced to count-only even when the rollout recorded the real times. Propagate each event's own
+    // timestamp — the claim entry gets the claim's time, the proof-output entry gets the completion time —
+    // so a Codex fix-delta measures from the same two real timestamps a Claude one does. Only a string is
+    // carried (matching the consumers' `typeof === "string"` guard); anything else is left absent, exactly
+    // as a Claude entry with no timestamp — never an invented time.
+    const ts = typeof e.timestamp === "string" ? e.timestamp : undefined;
     if (p.type === "message") {
       const role = p.role === "assistant" ? "assistant" : "user";
       const text = Array.isArray(p.content)
         ? p.content.map((b) => (b && typeof b.text === "string" ? b.text : "")).join("")
         : "";
       if (text.trim())
-        entries.push({ type: role, message: { role, content: [{ type: "text", text }] } });
+        entries.push({
+          type: role,
+          timestamp: ts,
+          message: { role, content: [{ type: "text", text }] },
+        });
     } else if (p.type === "function_call" && p.name === "exec_command" && p.call_id) {
       let cmd = "";
       let workdir = null;
@@ -165,6 +180,7 @@ export function loadCodexEntries(file, maxBytes = DEFAULT_TAIL_BYTES) {
       if (cmd)
         entries.push({
           cwd: workdir,
+          timestamp: ts,
           message: {
             role: "assistant",
             content: [{ type: "tool_use", name: "Bash", id: p.call_id, input: { command: cmd } }],
@@ -173,6 +189,7 @@ export function loadCodexEntries(file, maxBytes = DEFAULT_TAIL_BYTES) {
     } else if (p.type === "function_call_output" && p.call_id) {
       const { exitCode, body } = parseCodexOutput(p.output);
       entries.push({
+        timestamp: ts,
         message: {
           role: "user",
           content: [

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "skalpel",
-  "version": "4.0.49",
+  "version": "4.0.50",
   "description": "Behavioral graph for AI-assisted coding — learns how you work and steers Claude Code + Codex in real time.",
   "type": "module",
   "bin": {