memgres 0.13.1__tar.gz → 0.13.2__tar.gz

{memgres-0.13.1 → memgres-0.13.2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: memgres
-Version: 0.13.1
+Version: 0.13.2
 Summary: Drop-in memory for AI agents: one Postgres, lexical + semantic recall, diff-versioned history, GDPR-erasable.
 Author: mozgsml
 License-Expression: MIT

{memgres-0.13.1 → memgres-0.13.2}/memgres/_version.py

@@ -8,4 +8,4 @@ here at release; nowhere else carries the number.
 PEP 440: a ``.devN`` suffix marks an unreleased build ahead of the last tag.
 """
 
-__version__ = "0.13.1"
+__version__ = "0.13.2"

{memgres-0.13.1 → memgres-0.13.2}/memgres/migrations/0023_relink_after_parser_fix.sql

@@ -13,4 +13,17 @@
 --
 -- Additive by nature — an older client reading this database is no worse off
 -- than it was, so the compatibility floor does not move.
-UPDATE memgres_meta SET links_built = false;
+-- ONCE. Every migration file runs again at every start (they are meant to be
+-- idempotent), and a bare UPDATE here cleared the flag on every start — so every
+-- start rebuilt the whole link graph, and two servers starting together raced on
+-- it. The marker column makes the clearing happen the first time only; on a
+-- database that has been through this file before, it happens one last time.
+DO $$
+BEGIN
+    IF NOT EXISTS (SELECT 1 FROM information_schema.columns
+                   WHERE table_schema = current_schema() AND table_name = 'memgres_meta'
+                     AND column_name = 'relinked_after_0023') THEN
+        ALTER TABLE memgres_meta ADD COLUMN relinked_after_0023 boolean NOT NULL DEFAULT true;
+        UPDATE memgres_meta SET links_built = false;
+    END IF;
+END $$;

{memgres-0.13.1 → memgres-0.13.2}/memgres/relink.py

@@ -20,6 +20,12 @@ _log = logging.getLogger("memgres.relink")
 
 BATCH = 500
 
+# One rebuild at a time across processes. Two servers starting together (the MCP
+# door and the REST/panel door) both found the flag clear and rebuilt at once,
+# and one of them died on the other's rows. A session-level advisory lock: held
+# for the rebuild, gone with the connection if the process is.
+LOCK_KEY = 0x6D656D67_6C6E6B   # "memglnk"
+
 
 class _NoEmbedder:
     """Stands in for an embedder so `Store` does not build the real one. The
@@ -55,11 +61,32 @@ def rebuild(conn, cfg, *, force: bool = False) -> int:
     the flag false — so the server came up serving a link graph that was perfectly
     empty while saying it had built one. Owning the commit here means the work is
     durable when this returns, whatever the caller does next."""
+    with conn.cursor() as cur:
+        cur.execute("SELECT pg_try_advisory_lock(%s)", (LOCK_KEY,))
+        mine = cur.fetchone()[0]
+    if not mine:
+        conn.commit()
+        _log.info("the link index is being rebuilt by another process; not starting a second pass")
+        return 0
+    try:
+        return _rebuild(conn, cfg, force=force)
+    finally:
+        try:
+            with conn.cursor() as cur:
+                cur.execute("SELECT pg_advisory_unlock(%s)", (LOCK_KEY,))
+            conn.commit()
+        except Exception:          # a broken connection releases the lock by closing
+            pass
+
+
+def _rebuild(conn, cfg, *, force: bool) -> int:
     from .store import Store
     with conn.cursor() as cur:
+        # read under the lock: another process may have just finished
         cur.execute("SELECT links_built FROM memgres_meta")
         row = cur.fetchone()
         if row and row[0] and not force:
+            conn.commit()
             return 0
 
     # `_sync_links` touches neither embeddings nor vectors; passing a

{memgres-0.13.1 → memgres-0.13.2}/memgres/web/static/app.css

@@ -487,3 +487,5 @@ a.person:hover { color: var(--focus); box-shadow: inset 0 -1px 0 var(--focus); }
 .node.hubnode .halo { opacity: .12; stroke-width: 18; }
 .node.hubnode.sel .halo { opacity: .22; }
 @media (prefers-reduced-motion: reduce) { .spinner, .search.busy::before { animation-duration: 2.4s; } }
+/* the results header — the query and × — stays put while the list scrolls under it */
+.results > .res-head { position: sticky; top: -14px; z-index: 2; margin: -14px -14px 0; padding: 14px 14px 10px; background: var(--panel); border-radius: 14px 14px 0 0; box-shadow: 0 1px 0 var(--line-soft); }

{memgres-0.13.1 → memgres-0.13.2}/memgres.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: memgres
-Version: 0.13.1
+Version: 0.13.2
 Summary: Drop-in memory for AI agents: one Postgres, lexical + semantic recall, diff-versioned history, GDPR-erasable.
 Author: mozgsml
 License-Expression: MIT

{memgres-0.13.1 → memgres-0.13.2}/tests/test_links.py

@@ -778,3 +778,31 @@ def test_an_expired_referrer_is_left_alone(monkeypatch):
         body, seq = cur.fetchone()
     assert body == "see [[ops.a]]" and seq == 1
     conn.close()
+
+
+def test_starting_again_does_not_clear_the_flag(store):
+    """Migrations run at every start. The one that asked for a rebuild after the
+    parser fix used to clear the flag every time, so every start rebuilt the
+    whole graph — and two servers starting together raced on it."""
+    from memgres.relink import rebuild
+    store.write(body="see [[ops.deploy]]", path="notes.a")
+    rebuild(store._conn, store.cfg, force=True)
+    with psycopg.connect(DSN) as conn:
+        migrate(conn, store.cfg)
+        conn.commit()
+        with conn.cursor() as cur:
+            cur.execute("SELECT links_built FROM memgres_meta")
+            assert cur.fetchone()[0] is True
+
+
+def test_a_second_process_does_not_rebuild_alongside_the_first(store):
+    from memgres.relink import LOCK_KEY, rebuild
+    store.write(body="see [[ops.deploy]]", path="notes.a")
+    with store._conn.cursor() as cur:
+        cur.execute("UPDATE memgres_meta SET links_built = false")
+    store._conn.commit()
+    with psycopg.connect(DSN, autocommit=True) as other:
+        other.execute("SELECT pg_advisory_lock(%s)", (LOCK_KEY,))       # "another server is rebuilding"
+        assert rebuild(store._conn, store.cfg) == 0                      # skips, no error
+        other.execute("SELECT pg_advisory_unlock(%s)", (LOCK_KEY,))
+    assert rebuild(store._conn, store.cfg) == 1                          # and runs once it is free