@ddtcorex/dsh-maestro-memory 1.1.0 → 1.2.0

package/src/host/memory/store.ts

@@ -373,6 +373,22 @@ export class MaestroMemoryStore {
     const t = normalizeTarget(target)
     let content = String(entry ?? '').trim()
     if (!content) return { ok: false, error: 'empty content' }
+    // Guard against idle-loop spam: reject trivial placeholder entries that carry
+    // no new information (e.g. "Idle", "Idle — no change"). The snapshot
+    // discipline now says to skip when idle, but keep a code-level guard as
+    // defense-in-depth so a misbehaving model cannot flood daily/project.
+    const trivialStripped = content.replace(/^\[.*?\]\s*/g, '').trim().toLowerCase()
+    if (
+      trivialStripped.startsWith('idle') ||
+      trivialStripped.startsWith('no change') ||
+      trivialStripped.startsWith('awaiting') ||
+      trivialStripped.startsWith('waiting')
+    ) {
+      // Idle/placeholder entries carry no new information; treat as duplicate/no-op
+      // rather than error so the caller can proceed without retrying. Cap at 150
+      // chars so legitimate long entries that merely mention "idle" are not blocked.
+      if (trivialStripped.length < 150) return { ok: true, duplicate: true }
+    }
     // Desensitize by default (opt-out via {desensitize:false} for tests/internal)
     const doDesensitize = opts.desensitize !== false
     if (doDesensitize) {

package/src/host/prompt/snapshot.ts

@@ -99,12 +99,15 @@ export function renderSnapshot(
 
   // Recent daily slot (512B) — last 2 days' newest entries
   // Keeps recent context without exceeding cap; full logs remain query-only.
+  // Use local calendar (matching store.todayStamp) to avoid UTC/local drift near midnight.
   try {
     const recentDaily: string[] = []
     for (let i = 0; i < 2; i++) {
       const d = new Date()
       d.setDate(d.getDate() - i)
-      const ds = d.toISOString().slice(0, 10)
+      const mm = String(d.getMonth() + 1).padStart(2, '0')
+      const dd = String(d.getDate()).padStart(2, '0')
+      const ds = `${d.getFullYear()}-${mm}-${dd}`
       try {
         const list: string[] = store.list('daily', undefined, { date: ds } as any)
         if (list.length) recentDaily.push(list[list.length - 1])
@@ -117,8 +120,9 @@ export function renderSnapshot(
   } catch {}
 
   // End-of-turn discipline note — verbatim contract (hardened: exactly once, always last)
+  // Conditional to avoid idle loops: only when turn produced meaningful progress.
   const discipline =
-    `---\nEnd of every turn you must: 1. Write daily+project via memory entries (daily+project in one call) 2. Check dtodo list (bounded, max 8)\nFor important project decisions (convention, incident, infra) use memory_suggest target=key with reason, not memory add.`
+    `---\nEnd of every turn — if this turn produced meaningful progress (code, decisions, learnings, or next steps): 1. Write daily+project via memory entries (daily+project in one call, skip if idle/waiting or no new information — never write entries containing only 'Idle' or placeholders) 2. Check dtodo list only if relevant (bounded, max 8)\nFor important project decisions (convention, incident, infra) use memory_suggest target=key with reason, not memory add.`
   // Defensive: strip any pre-existing discipline entry (should never occur — parts is fresh per call)
   // then append exactly once so the note is guaranteed last even for empty stores or repeated calls.
   const deduped = parts.filter((p) => p !== discipline)