thincoder 0.12.27 → 0.12.28

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "thincoder",
-  "version": "0.12.27",
+  "version": "0.12.28",
   "description": "Thin coding agent - zero dependencies, no build step, Node.js native. Sharp code, zero bloat.",
   "keywords": [
     "ai",

package/src/agent/setup.mjs

@@ -124,13 +124,6 @@ export async function prepareRun(agent, input, callbacks, {
         })
       }
     }
-    // Time grounding for EVERY agent depth (subagents are short-lived; they also need
-    // to know "now"). Transient — dropped on persist, fresh at every run start.
-    agent.history.push({
-      role: "user",
-      content: `[System reminder: current time is ${timeNowLocal()} (local; timezone ${Intl.DateTimeFormat().resolvedOptions().timeZone || "local"}).]`,
-      transient: true,
-    })
     if (depth === 0) {
       // Checklist injection: inject pending + in_progress items from .thincoder/checklist.md
       try {
@@ -146,6 +139,15 @@ export async function prepareRun(agent, input, callbacks, {
       } catch { /* checklist not available — suppress error */ }
     }
     pushReal(agent, { role: "user", content: input })
+    // Time grounding for EVERY agent depth, pushed LAST (after the user input): transient,
+    // dropped on persist, fresh at every run start. Tail position keeps the second-precision
+    // content out of any prefix — caches stay hit (plugin hit a position-drift regression
+    // when the reminder was interleaved before the input; CLI is aligned preemptively).
+    agent.history.push({
+      role: "user",
+      content: `[System reminder: current time is ${timeNowLocal()} (local; timezone ${Intl.DateTimeFormat().resolvedOptions().timeZone || "local"}).]`,
+      transient: true,
+    })
   }
 
   if (agent._pendingReminders.length > 0) {

package/src/provider/core.mjs

@@ -240,7 +240,9 @@ export function normalizeToolPairing(messages) {
       rest.push(m)
     }
   }
-  if (toolById.size === 0) return messages // no tool messages — nothing to enforce
+  if (toolById.size === 0 && !messages.some((m) => m.role === "assistant" && m.tool_calls?.length)) {
+    return messages // no tool messages AND no tool_calls declared — nothing to enforce
+  }
   const out = []
   for (const m of rest) {
     out.push(m)

package/src/session.mjs

@@ -325,8 +325,14 @@ export function saveSession(agent) {
   // _fullHistory is written at the source via pushReal — no flush needed here.
   // history        = FULL, never-compacted (human-readable; VS Code panel & CLI resume read this)
   // contextHistory = machine context (possibly compacted) so CLI resume keeps the token savings
+  // Human line: transient machine injections never enter the readable record.
   const history = (agent._fullHistory ?? agent.history).filter((m) => !m.transient && !isLegacyTransient(m))
-  const contextHistory = agent.history.filter((m) => !m.transient && !isLegacyTransient(m))
+  // Machine line (contextHistory): KEEP transient messages — resume must rebuild the
+  // machine line byte-identical to what the provider cache saw. Dropping them made every
+  // process restart diverge at the first injection position (git/OS/time reminders are
+  // re-injected with FRESH content) → whole-prefix cache miss on the CLI's very first
+  // request of each session (2026-08-16 cache-hit report; Kimi review).
+  const contextHistory = agent.history.filter((m) => !isLegacyTransient(m))
   const data = {
     version: 2,
     cwd: agent.cwd,