shellbrain 0.1.13__tar.gz → 0.1.14__tar.gz

{shellbrain-0.1.13 → shellbrain-0.1.14}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: shellbrain
-Version: 0.1.13
+Version: 0.1.14
 Summary: Repo-scoped Shellbrain CLI with explicit evidence-backed writes.
 Requires-Python: >=3.11
 Description-Content-Type: text/markdown
@@ -96,6 +96,19 @@ Fish PATH setup is written to `~/.config/fish/conf.d/shellbrain.fish`.
 
 ---
 
+## DB-backed tests
+
+**Live memories and DB-backed tests now use different Postgres hosts.**
+
+- managed local Shellbrain keeps durable memories on the machine-owned managed instance
+- DB-backed tests and scratch validation should use the dedicated repo-owned test host from `docker-compose.test.yml`
+- `scripts/run_tests` provisions a disposable test database on that dedicated host by default
+- `scripts/storage_status` shows the live managed target, the dedicated test host, and any legacy local test host that is still hanging around
+
+If you are running managed local Shellbrain, do not leave a stale `SHELLBRAIN_DB_DSN` export in your shell profile that points at the old local compose database. The machine config wins anyway, and the stale env var just makes the storage layout harder to reason about.
+
+---
+
 ## Docs
 
 - [shellbrain.ai/humans](https://shellbrain.ai/humans/) — install, upgrade, getting started

{shellbrain-0.1.13 → shellbrain-0.1.14}/README.md

@@ -82,6 +82,19 @@ Fish PATH setup is written to `~/.config/fish/conf.d/shellbrain.fish`.
 
 ---
 
+## DB-backed tests
+
+**Live memories and DB-backed tests now use different Postgres hosts.**
+
+- managed local Shellbrain keeps durable memories on the machine-owned managed instance
+- DB-backed tests and scratch validation should use the dedicated repo-owned test host from `docker-compose.test.yml`
+- `scripts/run_tests` provisions a disposable test database on that dedicated host by default
+- `scripts/storage_status` shows the live managed target, the dedicated test host, and any legacy local test host that is still hanging around
+
+If you are running managed local Shellbrain, do not leave a stale `SHELLBRAIN_DB_DSN` export in your shell profile that points at the old local compose database. The machine config wins anyway, and the stale env var just makes the storage layout harder to reason about.
+
+---
+
 ## Docs
 
 - [shellbrain.ai/humans](https://shellbrain.ai/humans/) — install, upgrade, getting started

{shellbrain-0.1.13 → shellbrain-0.1.14}/app/core/policies/_shared/executor.py

@@ -52,7 +52,7 @@ def apply_side_effects(
         if effect_type == "memory_evidence.attach":
             refs = params["refs"]
             assert isinstance(refs, list)
-            for ref in refs:
+            for ref in sorted(str(ref) for ref in refs):
                 evidence = uow.evidence.upsert_ref(repo_id=str(params["repo_id"]), ref=str(ref))
                 uow.evidence.link_memory_evidence(memory_id=str(params["memory_id"]), evidence_id=evidence.id)
             continue
@@ -124,7 +124,7 @@ def apply_side_effects(
             )
             evidence_refs = params.get("evidence_refs", [])
             assert isinstance(evidence_refs, list)
-            for ref in evidence_refs:
+            for ref in sorted(str(ref) for ref in evidence_refs):
                 evidence = uow.evidence.upsert_ref(repo_id=str(params["repo_id"]), ref=str(ref))
                 uow.evidence.link_association_edge_evidence(edge_id=edge.id, evidence_id=evidence.id)
             continue

{shellbrain-0.1.13 → shellbrain-0.1.14}/app/periphery/admin/analytics_diagnostics.py

@@ -13,7 +13,7 @@ def classify_operation_failure(
     """Return one stable diagnosis payload for an operation failure."""
 
     message = (error_message or "").lower()
-    if "uq_evidence_repo_ref" in message:
+    if "uq_evidence_repo_ref" in message or "uq_evidence_repo_episode_event" in message:
         return _diagnosis(
             category="duplicate_evidence_ref",
             summary="Evidence refs are being inserted twice for the same repo/event pair.",

{shellbrain-0.1.13 → shellbrain-0.1.14}/app/periphery/admin/instance_guard.py

@@ -45,17 +45,36 @@ def dsn_fingerprint(dsn: str) -> str:
     return hashlib.sha256(normalize_dsn(dsn).encode("utf-8")).hexdigest()
 
 
+def host_port_from_dsn(dsn: str) -> tuple[str, int]:
+    """Extract the normalized host/port pair from one DSN."""
+
+    parsed = urlparse(dsn.replace("+psycopg", ""))
+    return ((parsed.hostname or "").lower(), parsed.port or 5432)
+
+
 def database_name_from_dsn(dsn: str) -> str:
     """Extract the target database name from one DSN."""
 
     return urlparse(dsn.replace("+psycopg", "")).path.lstrip("/")
 
 
-def assert_disposable_test_dsn(*, test_dsn: str, protected_dsn: str | None = None) -> None:
+def assert_disposable_test_dsn(
+    *,
+    test_dsn: str,
+    protected_dsn: str | None = None,
+    protected_host_ports: set[tuple[str, int]] | None = None,
+) -> None:
     """Refuse to treat a protected or production-shaped database as disposable."""
 
     if protected_dsn and dsn_fingerprint(test_dsn) == dsn_fingerprint(protected_dsn):
         raise RuntimeError("Refusing destructive test setup against the protected live database DSN.")
+    protected_pairs = set(protected_host_ports or set())
+    if protected_dsn:
+        protected_pairs.add(host_port_from_dsn(protected_dsn))
+    if host_port_from_dsn(test_dsn) in protected_pairs:
+        raise RuntimeError(
+            "Refusing destructive test setup against the protected live database host/port."
+        )
     db_name = database_name_from_dsn(test_dsn).lower()
     if db_name in PROTECTED_DB_NAMES:
         raise RuntimeError(

{shellbrain-0.1.13 → shellbrain-0.1.14}/app/periphery/db/repos/relational/evidence_repo.py

@@ -3,7 +3,7 @@
 from datetime import datetime, timezone
 from uuid import uuid4
 
-from sqlalchemy import select, update
+from sqlalchemy import select, text, update
 from sqlalchemy.dialects.postgresql import insert
 
 from app.core.entities.evidence import EvidenceRef
@@ -24,6 +24,7 @@ class EvidenceRepo(IEvidenceRepo):
     def upsert_ref(self, repo_id: str, ref: str) -> EvidenceRef:
         """This method inserts or returns a canonical evidence reference row."""
 
+        self._acquire_ref_guard(repo_id=repo_id, ref=ref)
         existing = (
             self._session.execute(
                 select(evidence_refs).where(
@@ -63,6 +64,14 @@ class EvidenceRepo(IEvidenceRepo):
         )
         return EvidenceRef(id=evidence_id, repo_id=repo_id, ref=ref, episode_event_id=ref)
 
+    def _acquire_ref_guard(self, *, repo_id: str, ref: str) -> None:
+        """Serialize concurrent writes for one repo/ref pair within the active transaction."""
+
+        self._session.execute(
+            text("SELECT pg_advisory_xact_lock(hashtext(:repo_id), hashtext(:ref))"),
+            {"repo_id": repo_id, "ref": ref},
+        )
+
     def link_memory_evidence(self, memory_id: str, evidence_id: str) -> None:
         """This method creates shellbrain-to-evidence link rows."""
 

{shellbrain-0.1.13 → shellbrain-0.1.14}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "shellbrain"
-version = "0.1.13"
+version = "0.1.14"
 description = "Repo-scoped Shellbrain CLI with explicit evidence-backed writes."
 readme = "README.md"
 requires-python = ">=3.11"

{shellbrain-0.1.13 → shellbrain-0.1.14}/shellbrain.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: shellbrain
-Version: 0.1.13
+Version: 0.1.14
 Summary: Repo-scoped Shellbrain CLI with explicit evidence-backed writes.
 Requires-Python: >=3.11
 Description-Content-Type: text/markdown
@@ -96,6 +96,19 @@ Fish PATH setup is written to `~/.config/fish/conf.d/shellbrain.fish`.
 
 ---
 
+## DB-backed tests
+
+**Live memories and DB-backed tests now use different Postgres hosts.**
+
+- managed local Shellbrain keeps durable memories on the machine-owned managed instance
+- DB-backed tests and scratch validation should use the dedicated repo-owned test host from `docker-compose.test.yml`
+- `scripts/run_tests` provisions a disposable test database on that dedicated host by default
+- `scripts/storage_status` shows the live managed target, the dedicated test host, and any legacy local test host that is still hanging around
+
+If you are running managed local Shellbrain, do not leave a stale `SHELLBRAIN_DB_DSN` export in your shell profile that points at the old local compose database. The machine config wins anyway, and the stale env var just makes the storage layout harder to reason about.
+
+---
+
 ## Docs
 
 - [shellbrain.ai/humans](https://shellbrain.ai/humans/) — install, upgrade, getting started