memory: comprehensive week retrospective -- PT/orama collab Lessons

.agent/harness/hooks/_episodic_io.py

@@ -72,6 +72,22 @@ def episodic_lock(path: str, *, exclusive: bool = True) -> Iterator[None]:
     )
 
 
+def is_legacy_episodic_row(entry: dict) -> bool:
+    """True for pre-canonical rows that used date/summary instead of timestamp/action.
+
+    Replay and dream clustering must skip these — they lack evidence_ids and the
+    fields downstream loaders expect. Canonical re-encodes live as normal rows.
+    """
+    if not isinstance(entry, dict):
+        return False
+    return (
+        "date" in entry
+        and "summary" in entry
+        and "timestamp" not in entry
+        and "action" not in entry
+    )
+
+
 def append_jsonl(path: str, entry: dict) -> dict:
     """Append one UTF-8 JSON line while excluding concurrent rewrites/appends."""
     payload = (json.dumps(entry) + "\n").encode("utf-8")

.agent/memory/auto_dream.py

@@ -35,7 +35,7 @@
 HARNESS_HOOKS = os.path.join(AGENT_ROOT, "harness", "hooks")
 if HARNESS_HOOKS not in sys.path:
     sys.path.insert(0, HARNESS_HOOKS)
-from _episodic_io import episodic_lock  # noqa: E402
+from _episodic_io import episodic_lock, is_legacy_episodic_row  # noqa: E402
 from path_hygiene import sanitize_json_strings  # noqa: E402
 
 EPISODIC = os.path.join(ROOT, "episodic/AGENT_LEARNINGS.jsonl")
@@ -63,9 +63,12 @@ def _load_entries_locked(_fd):
                 if not line:
                     continue
                 try:
-                    entries.append(json.loads(line))
+                    entry = json.loads(line)
                 except json.JSONDecodeError:
                     continue
+                if is_legacy_episodic_row(entry):
+                    continue
+                entries.append(entry)
     except FileNotFoundError:
         pass
     return entries

.agent/memory/candidates/graduated/2d53056e593c.json

@@ -0,0 +1,42 @@
+{
+  "id": "2d53056e593c",
+  "key": "manual_2d5305",
+  "name": "manual_2d5305",
+  "claim": "Several tooling and API gotchas recur across a working session even after being individually caught once, because a single encounter doesn't automatically generalize into a standing rule: the PR-list API's merged field being unreliable (must check the single-PR endpoint); squash-merged branches never showing as git ancestors (verify by content/ID presence, not ancestry); a stale local checkout after an earlier push in the same session diverging silently; and diff-scoped CI lint checking a whole touched file, not just the changed hunk. Each of these needs to be treated as a standing checklist item to consult before the relevant operation, not a one-off lesson learned and then re-derived from scratch the next time it's encountered.",
+  "conditions": [
+    "pr-list-merged-field",
+    "squash-merge-ancestry",
+    "diff-scoped-lint",
+    "stale-checkout",
+    "recurring-gotcha"
+  ],
+  "evidence_ids": [
+    "2026-08-01T06:53:43.671436+00:00"
+  ],
+  "cluster_size": 1,
+  "canonical_salience": 8.0,
+  "staged_at": "2026-08-01T06:53:43.671436+00:00",
+  "status": "accepted",
+  "decisions": [
+    {
+      "ts": "2026-08-01T06:53:43.671436+00:00",
+      "action": "staged",
+      "reviewer": "learn"
+    },
+    {
+      "ts": "2026-08-01T06:53:50.650448+00:00",
+      "action": "graduated",
+      "reviewer": "host-agent",
+      "notes": "Several gotchas recurred even after individual capture -- worth the meta-lesson that a checklist consulted before the operation, not recall alone, is what actually prevents repetition.",
+      "provisional": false,
+      "evidence_snapshot": [
+        "2026-08-01T06:53:43.671436+00:00"
+      ],
+      "lessons_sha": "4bce83c74754"
+    }
+  ],
+  "rejection_count": 0,
+  "accepted_at": "2026-08-01T06:53:50.650434+00:00",
+  "reviewer": "host-agent",
+  "rationale": "Several gotchas recurred even after individual capture -- worth the meta-lesson that a checklist consulted before the operation, not recall alone, is what actually prevents repetition."
+}

.agent/memory/candidates/graduated/718c9430f44b.json

@@ -0,0 +1,42 @@
+{
+  "id": "718c9430f44b",
+  "key": "manual_718c94",
+  "name": "manual_718c94",
+  "claim": "When resolving real overlap between a new draft and an existing, more comprehensive doctrine, \"avoid duplication\" does not automatically mean \"make the newer, simpler thing subordinate to the older, more complex one.\" It means determining which document actually serves which situation and sizing each to its own job. A heavyweight protocol built for a rare, hard problem (e.g. concurrent multi-agent edits to the same repo, or reconciling a fork separated from its source by months of drift) should not become the default an agent reads first for the common, simple case (one branch, one agent, no concurrent editing) just because it existed first and is more thorough. Getting this backwards is not caught by re-reading a project's own stated single-source-of-truth value, since that value is genuinely being honored in some sense (no literal duplication) even while the sizing is wrong -- it may require direct correction from a human collaborator who can see the actual audience mismatch.",
+  "conditions": [
+    "doctrine-overlap",
+    "thin-addendum",
+    "subordinate-doc",
+    "single-source-of-truth",
+    "worktree-merge-card"
+  ],
+  "evidence_ids": [
+    "2026-08-01T06:53:43.608740+00:00"
+  ],
+  "cluster_size": 1,
+  "canonical_salience": 8.0,
+  "staged_at": "2026-08-01T06:53:43.608740+00:00",
+  "status": "accepted",
+  "decisions": [
+    {
+      "ts": "2026-08-01T06:53:43.608740+00:00",
+      "action": "staged",
+      "reviewer": "learn"
+    },
+    {
+      "ts": "2026-08-01T06:53:50.501493+00:00",
+      "action": "graduated",
+      "reviewer": "host-agent",
+      "notes": "The hierarchy-inversion mistake this session, requiring direct human correction rather than self-catching -- worth a durable lesson about resolving overlap correctly, not just avoiding literal duplication.",
+      "provisional": false,
+      "evidence_snapshot": [
+        "2026-08-01T06:53:43.608740+00:00"
+      ],
+      "lessons_sha": "c7f4daa0894a"
+    }
+  ],
+  "rejection_count": 0,
+  "accepted_at": "2026-08-01T06:53:50.501478+00:00",
+  "reviewer": "host-agent",
+  "rationale": "The hierarchy-inversion mistake this session, requiring direct human correction rather than self-catching -- worth a durable lesson about resolving overlap correctly, not just avoiding literal duplication."
+}