loop-memory 0.4.16__tar.gz → 0.4.17__tar.gz

{loop_memory-0.4.16 → loop_memory-0.4.17}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: loop-memory
-Version: 0.4.16
+Version: 0.4.17
 Summary: A general-purpose, local memory system for every AI agent you run. Loop Memory auto-captures conversations from Codex / Claude / Hermes / OpenClaw, scores them by importance × recency × usage × feedback, distils them into a curated wiki, and serves everything from a single web UI.
 Author: Loop Memory contributors <loop-memory@users.noreply.github.com>
 License-Expression: MIT
@@ -470,7 +470,10 @@ never-used stale row is expired. Age alone never decides — that is the
 whole point of "In Defense of Forgotten" (arXiv 2605.27366).
 
 The pass is a **dry run by default**: `--apply` (CLI) or `"apply": true`
-(HTTP) is what actually writes.
+(HTTP) is what actually writes. A dry run changes **no memory data** --
+no expiry, no supersession, no rescore -- it only appends a
+`cognitive_audit` row per proposed action (`action="suggest"`), which
+is the reviewable plan the dashboard shows before you commit.
 
 ```bash
 loop-memory dream --json                     # machine-readable dry run

{loop_memory-0.4.16 → loop_memory-0.4.17}/README.md

@@ -426,7 +426,10 @@ never-used stale row is expired. Age alone never decides — that is the
 whole point of "In Defense of Forgotten" (arXiv 2605.27366).
 
 The pass is a **dry run by default**: `--apply` (CLI) or `"apply": true`
-(HTTP) is what actually writes.
+(HTTP) is what actually writes. A dry run changes **no memory data** --
+no expiry, no supersession, no rescore -- it only appends a
+`cognitive_audit` row per proposed action (`action="suggest"`), which
+is the reviewable plan the dashboard shows before you commit.
 
 ```bash
 loop-memory dream --json                     # machine-readable dry run

{loop_memory-0.4.16 → loop_memory-0.4.17}/loop_memory/jobs/dream.py

@@ -30,7 +30,9 @@ What this job does
 pass:
 
 1. **Decay.** Rescore every row with the existing exponential
-   half-life model so recency-sensitive ordering stays honest.
+   half-life model so recency-sensitive ordering stays honest. A dry
+   run computes the new scores and reports how many rows *would*
+   change, but writes nothing (audit 2026-10-25, fix).
 
 2. **Utility-aware selective forgetting.** A row is a *forgetting
    candidate* once it is past its own TTL (or past the sweep's
@@ -48,7 +50,12 @@ pass:
    ``recall(latest_only=True)`` can collapse to one answer.
 
 Everything is opt-in: ``apply=False`` (the default) produces a dry-run
-report and touches nothing.
+report and **changes no memory data** -- no expiry, no supersession,
+no rescore. The only write a dry run performs is one
+``cognitive_audit`` row per proposed action (with ``action="suggest"``
+instead of ``"applied"``), because that row *is* the reviewable plan:
+it is what lets the dashboard show what a real run would do before
+you commit to it. Use ``record_audit=False`` to suppress even that.
 """
 
 from __future__ import annotations
@@ -282,7 +289,17 @@ class Dreamer:
     def _stage_decay(self, report: DreamReport) -> None:
         t = time.time()
         try:
-            report.rescored = self.store.rescore_all(self.half_life_days)
+            # Audit 2026-10-25 (fix) -- the decay stage used to write
+            # unconditionally, so a *dry run* still rewrote the
+            # ``score`` column of every memory. That broke this
+            # module's own contract ("nothing is written unless
+            # apply=True") and the README / CHANGELOG wording. The
+            # store call now runs in dry-run mode, which computes the
+            # new scores and reports how many rows *would* change
+            # without touching the database.
+            report.rescored = self.store.rescore_all(
+                self.half_life_days, dry_run=not self.apply
+            )
         except Exception:  # pragma: no cover - decay must never abort the pass
             log.warning("dream: rescore_all failed; continuing", exc_info=True)
         report.stages["decay"] = round((time.time() - t) * 1000, 1)

{loop_memory-0.4.16 → loop_memory-0.4.17}/loop_memory/storage/sqlite_store.py

@@ -5110,10 +5110,18 @@ class MemoryStore:
         blended = 0.35 * importance + 0.65 * recency
         return max(0.0, min(1.0, blended))
 
-    def rescore_all(self, half_life_days: float = 30.0) -> int:
+    def rescore_all(self, half_life_days: float = 30.0,
+                    dry_run: bool = False) -> int:
         """Recompute score for every memory using v2 (importance × recency
         × usage × feedback). One row at a time keeps WAL writes small
-        and lets us skip rows whose components didn't change."""
+        and lets us skip rows whose components didn't change.
+
+        ``dry_run=True`` computes every new score but writes nothing,
+        and returns how many rows *would* change. The dream pass uses
+        it to honour its own "nothing is written unless apply=True"
+        contract while still reporting the size of the decay job
+        (audit 2026-10-25).
+        """
         updated = 0
         now = time.time()
         with self._conn() as c:
@@ -5143,10 +5151,11 @@ class MemoryStore:
                     "SELECT score FROM memories WHERE id=?", (r["id"],)
                 ).fetchone()
                 if cur is None or abs((cur["score"] or 0) - new) > 1e-3:
-                    c.execute(
-                        "UPDATE memories SET score=? WHERE id=?",
-                        (new, r["id"]),
-                    )
+                    if not dry_run:
+                        c.execute(
+                            "UPDATE memories SET score=? WHERE id=?",
+                            (new, r["id"]),
+                        )
                     updated += 1
         return updated
 

{loop_memory-0.4.16 → loop_memory-0.4.17}/loop_memory.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: loop-memory
-Version: 0.4.16
+Version: 0.4.17
 Summary: A general-purpose, local memory system for every AI agent you run. Loop Memory auto-captures conversations from Codex / Claude / Hermes / OpenClaw, scores them by importance × recency × usage × feedback, distils them into a curated wiki, and serves everything from a single web UI.
 Author: Loop Memory contributors <loop-memory@users.noreply.github.com>
 License-Expression: MIT
@@ -470,7 +470,10 @@ never-used stale row is expired. Age alone never decides — that is the
 whole point of "In Defense of Forgotten" (arXiv 2605.27366).
 
 The pass is a **dry run by default**: `--apply` (CLI) or `"apply": true`
-(HTTP) is what actually writes.
+(HTTP) is what actually writes. A dry run changes **no memory data** --
+no expiry, no supersession, no rescore -- it only appends a
+`cognitive_audit` row per proposed action (`action="suggest"`), which
+is the reviewable plan the dashboard shows before you commit.
 
 ```bash
 loop-memory dream --json                     # machine-readable dry run

{loop_memory-0.4.16 → loop_memory-0.4.17}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "loop-memory"
-version = "0.4.16"
+version = "0.4.17"
 description = "A general-purpose, local memory system for every AI agent you run. Loop Memory auto-captures conversations from Codex / Claude / Hermes / OpenClaw, scores them by importance × recency × usage × feedback, distils them into a curated wiki, and serves everything from a single web UI."
 readme = "README.md"
 requires-python = ">=3.10"

{loop_memory-0.4.16 → loop_memory-0.4.17}/tests/test_dream_2026_10_25.py

@@ -8,7 +8,9 @@ contract:
   "what did we believe back then?";
 * **utility-aware, not age-only** -- a row that is past its deadline
   but is recalled often is *protected*;
-* **dry run by default** -- nothing is written unless ``apply=True``;
+* **dry run by default** -- no memory *data* is changed unless
+  ``apply=True`` (the only write a dry run performs is the
+  ``cognitive_audit`` ``suggest`` row that records the plan);
 * **audited** -- every decision lands in ``cognitive_audit``.
 """
 from __future__ import annotations
@@ -84,6 +86,41 @@ class DreamPassTests(unittest.TestCase):
         self.assertFalse(report.applied)
         self.assertIsNone(self.store.get_memory(m.id).invalid_at)
 
+    def test_dry_run_does_not_touch_the_score_column(self) -> None:
+        """Regression (audit 2026-10-25, fix): the decay stage used to
+        call ``rescore_all`` unconditionally, so a dry run still
+        rewrote ``memories.score`` for every row -- breaking the
+        "nothing is written unless apply=True" contract."""
+        m = self.store.upsert_memory(kind="fact", text="a fact worth scoring")
+        with self.store._conn() as c:
+            c.execute("UPDATE memories SET score=0.01 WHERE id=?", (m.id,))
+        before = self.store.get_memory(m.id).score
+        self.assertEqual(before, 0.01)
+
+        report = dream(self.store, apply=False)
+        self.assertEqual(
+            self.store.get_memory(m.id).score, before,
+            "a dry run must not rewrite the score column",
+        )
+        # ...but it still reports the size of the decay job.
+        self.assertGreaterEqual(report.rescored, 1)
+
+    def test_apply_does_rescore(self) -> None:
+        m = self.store.upsert_memory(kind="fact", text="a fact worth scoring")
+        with self.store._conn() as c:
+            c.execute("UPDATE memories SET score=0.01 WHERE id=?", (m.id,))
+        report = dream(self.store, apply=True)
+        self.assertGreaterEqual(report.rescored, 1)
+        self.assertNotEqual(self.store.get_memory(m.id).score, 0.01)
+
+    def test_rescore_all_dry_run_writes_nothing(self) -> None:
+        m = self.store.upsert_memory(kind="fact", text="a fact worth scoring")
+        with self.store._conn() as c:
+            c.execute("UPDATE memories SET score=0.01 WHERE id=?", (m.id,))
+        n = self.store.rescore_all(dry_run=True)
+        self.assertGreaterEqual(n, 1)
+        self.assertEqual(self.store.get_memory(m.id).score, 0.01)
+
     def test_dry_run_reports_the_action(self) -> None:
         self.store.upsert_memory(
             kind="fact", text="the old build server is decommissioned",