velocity-python 0.1.110__tar.gz → 0.1.112__tar.gz

{velocity_python-0.1.110/src/velocity_python.egg-info → velocity_python-0.1.112}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: velocity-python
-Version: 0.1.110
+Version: 0.1.112
 Summary: A rapid application development library for interfacing with data storage
 Author-email: Velocity Team <info@codeclubs.org>
 License-Expression: MIT

{velocity_python-0.1.110 → velocity_python-0.1.112}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "velocity-python"
-version = "0.1.110"
+version = "0.1.112"
 authors = [
   { name="Velocity Team", email="info@codeclubs.org" },
 ]

{velocity_python-0.1.110 → velocity_python-0.1.112}/src/velocity/__init__.py

@@ -1,4 +1,4 @@
-__version__ = version = "0.1.110"
+__version__ = version = "0.1.112"
 
 import importlib as _importlib
 

{velocity_python-0.1.110 → velocity_python-0.1.112}/src/velocity/aws/capabilities/base.py

@@ -55,7 +55,16 @@ TEXT = "text"
 BOOLEAN = "boolean"
 CHOICE = "choice"  # one of `options`
 LIST = "list"  # free strings
-POLICY_KINDS = (CIDR_LIST, COUNTRY_LIST, TABLE_DAYS, NUMBER, TEXT, BOOLEAN, CHOICE, LIST)
+POLICY_KINDS = (
+    CIDR_LIST,
+    COUNTRY_LIST,
+    TABLE_DAYS,
+    NUMBER,
+    TEXT,
+    BOOLEAN,
+    CHOICE,
+    LIST,
+)
 
 IDENTIFIER = re.compile(r"^[a-z_][a-z0-9_]*$")
 COUNTRY_CODE = re.compile(r"^[A-Z]{2}$")
@@ -145,7 +154,9 @@ def describe(cap) -> dict:
 def catalog() -> list:
     """Every registered capability, described, ordered by label — the
     pick-list an admin UI shows."""
-    return sorted((describe(cap) for cap in REGISTRY.values()), key=lambda d: d["label"])
+    return sorted(
+        (describe(cap) for cap in REGISTRY.values()), key=lambda d: d["label"]
+    )
 
 
 def load_config(raw) -> dict:
@@ -232,8 +243,7 @@ def validate_policy(name, config) -> list:
         if value in (None, "", [], {}):
             continue
         problems.extend(
-            f"{field['label']}: {problem}"
-            for problem in _check_field(field, value)
+            f"{field['label']}: {problem}" for problem in _check_field(field, value)
         )
     return problems
 

{velocity_python-0.1.110 → velocity_python-0.1.112}/src/velocity/aws/capabilities/security.py

@@ -62,6 +62,7 @@ KNOBS = {
     "security_events_days": "90",
 }
 
+
 def clear_cache():
     """Forget the cached site row AND the shared flags row (tests)."""
     read_cache.invalidate(SITE_TABLE)
@@ -173,7 +174,13 @@ def check_rate(tx, bucket, *, limit, now, table=RATE_TABLE):
 def _cached(key, loader, *, now=None):
     """The process read cache (velocity.cache), tagged with the site table
     so provisioning's own writes to it in this process drop the entry."""
-    return memo(("security",) + tuple(key), loader, CACHE_TTL_SECONDS, tables=(SITE_TABLE,), now=now)
+    return memo(
+        ("security",) + tuple(key),
+        loader,
+        CACHE_TTL_SECONDS,
+        tables=(SITE_TABLE,),
+        now=now,
+    )
 
 
 def _site_row(tx):

{velocity_python-0.1.110 → velocity_python-0.1.112}/src/velocity/aws/feature_flags.py

@@ -109,7 +109,11 @@ def clear_cache(table: str = DEFAULT_TABLE) -> None:
 
 
 def get_cached(
-    tx, system: str | None = None, *, table: str = DEFAULT_TABLE, now: float | None = None
+    tx,
+    system: str | None = None,
+    *,
+    table: str = DEFAULT_TABLE,
+    now: float | None = None,
 ) -> dict:
     """:func:`get` through the process read cache (:mod:`velocity.cache`)
     — the hot-path reader.

{velocity_python-0.1.110 → velocity_python-0.1.112}/src/velocity/aws/handlers/context.py

@@ -102,7 +102,9 @@ def user_cache_seconds() -> float:
             raw = None
         try:
             _user_cache_seconds = (
-                float(raw) if raw not in (None, "") else float(DEFAULT_USER_CACHE_SECONDS)
+                float(raw)
+                if raw not in (None, "")
+                else float(DEFAULT_USER_CACHE_SECONDS)
             )
         except (TypeError, ValueError):
             _user_cache_seconds = float(DEFAULT_USER_CACHE_SECONDS)
@@ -861,7 +863,9 @@ class Context:
                     )
                     self.perf.log("cognito list_users")
                 except ClientError as exc:
-                    message = exc.response.get("Error", {}).get("Message", "Unknown error")
+                    message = exc.response.get("Error", {}).get(
+                        "Message", "Unknown error"
+                    )
                     raise AlertError(
                         f"Failed to search Cognito user by sub: {message}"
                     ) from None
@@ -882,7 +886,9 @@ class Context:
                 resolved_username = response.get("Username")
             except ClientError as exc:
                 message = exc.response.get("Error", {}).get("Message", "Unknown error")
-                raise AlertError(f"Failed to retrieve Cognito user: {message}") from None
+                raise AlertError(
+                    f"Failed to retrieve Cognito user: {message}"
+                ) from None
 
             attributes = {
                 attr["Name"]: attr.get("Value")

{velocity_python-0.1.110 → velocity_python-0.1.112}/src/velocity/aws/handlers/mixins/data_service.py

@@ -558,7 +558,8 @@ class DataServiceMixin:
         requested_cols = _extract_requested_column_names(params.get("columns"))
         if requested_cols:
             available_cols = {
-                c.lower() for c in tx.table(obj).sys_columns(cache_ttl=SCHEMA_CACHE_SECONDS)
+                c.lower()
+                for c in tx.table(obj).sys_columns(cache_ttl=SCHEMA_CACHE_SECONDS)
             }
             missing = sorted(
                 {c for c in requested_cols if c.lower() not in available_cols},

{velocity_python-0.1.110 → velocity_python-0.1.112}/src/velocity/aws/mirror.py

@@ -40,9 +40,13 @@ def alternate_engine(alternate_db: str | None = None):
     """Engine for the alternate database — same instance, other ``database``.
 
     Host/port/user/password are inherited from the running config, so no
-    separate credentials are needed; only the database name changes (the
-    ``DBAlternate`` config key by default). Cached per database name so
-    repeated mirrors in one process reuse the connection.
+    separate credentials are needed; only the database name changes: the
+    ``DBAlternate`` config key, which the site factory's
+    ``provision-capabilities`` derives on every deploy-backend from the
+    sibling environment's ``DBDatabase`` (``/{Project}/{env}/DBAlternate`` =
+    the other stage's database) — nothing to declare by hand inside the
+    factory. Cached per database name so repeated mirrors in one process
+    reuse the connection.
     """
     from velocity.aws import ssm_config
     from velocity.db.servers import postgres
@@ -176,7 +180,9 @@ def _run(operation, context, what, verbs, notify):
     except MirrorSkip as skip:
         log.warning("%s not %s the alternate database: %s", what, done, skip)
         if notify:
-            toast(context, f"{what} {undone} the alternate database: {skip}.", "warning")
+            toast(
+                context, f"{what} {undone} the alternate database: {skip}.", "warning"
+            )
         return None
     except Exception as exc:  # never block the local write
         log.warning("%s: mirror to the alternate database failed", what, exc_info=True)

{velocity_python-0.1.110 → velocity_python-0.1.112}/src/velocity/cache.py

@@ -87,7 +87,9 @@ class ReadCache:
             self.hits += 1
             return True, value
 
-    def put(self, key, value, ttl: float, *, tables=(), now: float | None = None) -> None:
+    def put(
+        self, key, value, ttl: float, *, tables=(), now: float | None = None
+    ) -> None:
         if not self.enabled() or ttl is None or ttl <= 0:
             return
         now = time.monotonic() if now is None else now
@@ -102,7 +104,9 @@ class ReadCache:
                 oldest, _ = self._entries.popitem(last=False)
                 self._untag(oldest)
 
-    def memo(self, key, loader, ttl: float, *, tables=(), cache_none: bool = True, now=None):
+    def memo(
+        self, key, loader, ttl: float, *, tables=(), cache_none: bool = True, now=None
+    ):
         """``loader()`` once per ``ttl`` for ``key``. ``cache_none=False`` never
         stores a ``None`` result (a "not found" that may exist a moment later)."""
         hit, value = self.get(key, now=now)
@@ -166,7 +170,9 @@ read_cache = ReadCache()
 
 def memo(key, loader, ttl: float, *, tables=(), cache_none: bool = True, now=None):
     """Module-level shortcut for :meth:`ReadCache.memo` on the shared store."""
-    return read_cache.memo(key, loader, ttl, tables=tables, cache_none=cache_none, now=now)
+    return read_cache.memo(
+        key, loader, ttl, tables=tables, cache_none=cache_none, now=now
+    )
 
 
 def db_identity(engine) -> str:

{velocity_python-0.1.110 → velocity_python-0.1.112}/src/velocity/db/core/table.py

@@ -1852,7 +1852,9 @@ class Table:
             return None
         if len(rows) > 1:
             sql = self.select("sys_id", sql_only=True, where=lookup, lock=lock)
-            raise exceptions.DuplicateRowsFoundError(f"More than one entry found. {sql}")
+            raise exceptions.DuplicateRowsFoundError(
+                f"More than one entry found. {sql}"
+            )
         # a copy: the Row's write-through cache must never alias a cached entry
         return Row._from_data(self, dict(rows[0]))
 

{velocity_python-0.1.110 → velocity_python-0.1.112/src/velocity_python.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: velocity-python
-Version: 0.1.110
+Version: 0.1.112
 Summary: A rapid application development library for interfacing with data storage
 Author-email: Velocity Team <info@codeclubs.org>
 License-Expression: MIT

{velocity_python-0.1.110 → velocity_python-0.1.112}/tests/test_capability_catalog.py

@@ -169,7 +169,12 @@ def test_summarize_policy_reads_the_same_for_every_capability():
     summary = caps.summarize_policy(
         "security",
         {
-            "ip_denylist": ["203.0.113.0/24", "198.51.100.1", "198.51.100.2", "198.51.100.3"],
+            "ip_denylist": [
+                "203.0.113.0/24",
+                "198.51.100.1",
+                "198.51.100.2",
+                "198.51.100.3",
+            ],
             "blocked_countries": ["KP"],
             "legacy": 1,
         },

{velocity_python-0.1.110 → velocity_python-0.1.112}/tests/test_cognito_user_cache.py

@@ -22,9 +22,16 @@ RESPONSE = {
 
 
 def _ctx():
-    event = {"requestContext": {"identity": {"cognitoAuthenticationProvider": PROVIDER}}}
+    event = {
+        "requestContext": {"identity": {"cognitoAuthenticationProvider": PROVIDER}}
+    }
     ctx = Context(
-        aws_event=event, aws_context=None, args={}, postdata={}, response=None, session={}
+        aws_event=event,
+        aws_context=None,
+        args={},
+        postdata={},
+        response=None,
+        session={},
     )
     return ctx, event
 
@@ -45,7 +52,12 @@ class TestCognitoUserCache(unittest.TestCase):
             mock_cognito.admin_get_user.return_value = RESPONSE
             first = ctx.get_cognito_user(event)
             second = Context(
-                aws_event=event, aws_context=None, args={}, postdata={}, response=None, session={}
+                aws_event=event,
+                aws_context=None,
+                args={},
+                postdata={},
+                response=None,
+                session={},
             ).get_cognito_user(event)
         mock_cognito.admin_get_user.assert_called_once()
         self.assertEqual(first["email"], "user@example.com")

{velocity_python-0.1.110 → velocity_python-0.1.112}/tests/test_mirror.py

@@ -114,12 +114,17 @@ def test_mirror_row_translates_and_upserts_by_natural_key():
     assert pk == {"site_sys_id": 2002, "env_name": "demo", "capability": "retention"}
     data, merged_pk = alt.table("site_capabilities").merges[0]
     assert merged_pk == pk
-    assert data["site_sys_id"] == 2002 and "sys_id" not in data and "sys_dirty" not in data
+    assert (
+        data["site_sys_id"] == 2002 and "sys_id" not in data and "sys_dirty" not in data
+    )
 
 
 def test_mirror_row_skips_keyless_and_translate_none():
     alt = FakeDb()
-    assert mirror.mirror_row({"sys_id": 1, "x": 1}, table="t", key=("k",), engine=alt) is None
+    assert (
+        mirror.mirror_row({"sys_id": 1, "x": 1}, table="t", key=("k",), engine=alt)
+        is None
+    )
     assert (
         mirror.mirror_row(
             {"k": 1}, table="t", key=("k",), engine=alt, translate=lambda *_: None
@@ -134,7 +139,12 @@ def test_mirror_skip_propagates_from_translate():
     alt.tables["sites"].rows.clear()
     with pytest.raises(mirror.MirrorSkip, match="not in the alternate"):
         mirror.mirror_row(
-            ROW, table="site_capabilities", key=KEY, engine=alt, translate=by_slug, tx=src
+            ROW,
+            table="site_capabilities",
+            key=KEY,
+            engine=alt,
+            translate=by_slug,
+            tx=src,
         )
 
 
@@ -149,14 +159,23 @@ def test_delete_mirror_removes_the_counterpart():
 
 def test_reconcile_mirrors_every_row_it_can_and_reports_the_rest():
     src, alt = dbs()
-    src.tables["site_capabilities"].rows.append({"sys_id": 6, "site_sys_id": 1001, "env_name": ""})
-    src.tables["site_capabilities"].rows.append({"sys_id": 7, "site_sys_id": 3, "env_name": "demo", "capability": "jobs"})
+    src.tables["site_capabilities"].rows.append(
+        {"sys_id": 6, "site_sys_id": 1001, "env_name": ""}
+    )
+    src.tables["site_capabilities"].rows.append(
+        {"sys_id": 7, "site_sys_id": 3, "env_name": "demo", "capability": "jobs"}
+    )
     src.tables["sites"].rows.append({"sys_id": 3, "slug": "nowhere"})
     result = mirror.reconcile(
         src, table="site_capabilities", key=KEY, engine=alt, translate=by_slug
     )
-    assert result["synced"] == [{"site_sys_id": 2002, "env_name": "demo", "capability": "retention"}]
-    assert result["skipped"] == [6, 7]  # keyless, and MirrorSkip (site not in alternate)
+    assert result["synced"] == [
+        {"site_sys_id": 2002, "env_name": "demo", "capability": "retention"}
+    ]
+    assert result["skipped"] == [
+        6,
+        7,
+    ]  # keyless, and MirrorSkip (site not in alternate)
 
 
 # ── hooks ────────────────────────────────────────────────────────────────
@@ -171,14 +190,18 @@ def test_write_hook_mirrors_and_toasts_success(monkeypatch):
     ctx = FakeContext()
     hook(src, "site_capabilities", 5, ROW, ctx)
     assert alt.table("site_capabilities").merges[0][1]["site_sys_id"] == 2002
-    assert ctx.toasts == [("Site capability mirrored to the alternate database.", "success")]
+    assert ctx.toasts == [
+        ("Site capability mirrored to the alternate database.", "success")
+    ]
 
 
 def test_write_hook_turns_a_skip_into_a_warning(monkeypatch):
     src, alt = dbs()
     alt.tables["sites"].rows.clear()
     monkeypatch.setattr(mirror, "alternate_engine", lambda *a, **k: alt)
-    hook = mirror.make_mirror_hook(table="site_capabilities", key=KEY, translate=by_slug)
+    hook = mirror.make_mirror_hook(
+        table="site_capabilities", key=KEY, translate=by_slug
+    )
     ctx = FakeContext()
     hook(src, "site_capabilities", 5, ROW, ctx)  # must not raise
     (message, level) = ctx.toasts[0]
@@ -220,7 +243,9 @@ def test_delete_hook_reads_the_row_first_and_removes_the_counterpart(monkeypatch
     assert alt.table("site_capabilities").deletes == [
         {"site_sys_id": 2002, "env_name": "demo", "capability": "retention"}
     ]
-    assert ctx.toasts == [("Site capability removed from the alternate database.", "success")]
+    assert ctx.toasts == [
+        ("Site capability removed from the alternate database.", "success")
+    ]
     # unknown row / no tx: nothing happens, nothing raised
     hook(src, "site_capabilities", 999, ctx)
     hook(None, "site_capabilities", 5, ctx)

{velocity_python-0.1.110 → velocity_python-0.1.112}/tests/test_table_find_cache.py

@@ -117,7 +117,9 @@ def test_find_none_and_duplicates():
     with pytest.raises(LookupError):
         T(FakeTx(), []).find({"email_address": "x@example.com"}, raise_if_missing=True)
     with pytest.raises(exceptions.DuplicateRowsFoundError):
-        T(FakeTx(), [ROW, {**ROW, "sys_id": 8}]).find({"email_address": "a@example.com"})
+        T(FakeTx(), [ROW, {**ROW, "sys_id": 8}]).find(
+            {"email_address": "a@example.com"}
+        )
 
 
 def test_find_with_cache_ttl_is_served_from_the_process_cache():
@@ -140,7 +142,9 @@ def test_find_hands_out_a_copy_not_the_cached_entry():
     t = T(FakeTx(), [ROW])
     where = {"email_address": "a@example.com"}
     first = t.find(where, cache_ttl=60)
-    object.__setattr__(first, "_cache", {**first._cache, "sys_id": 99})  # poke the Row only
+    object.__setattr__(
+        first, "_cache", {**first._cache, "sys_id": 99}
+    )  # poke the Row only
     assert t.find(where, cache_ttl=60)["sys_id"] == 7
 
 
@@ -166,7 +170,9 @@ def test_exists_and_sys_columns_cache_the_schema():
 def test_cache_keys_include_the_database():
     a = T(FakeTx(), [ROW])
     b = T(FakeTx(), [{**ROW, "sys_id": 8}])
-    b.tx.engine = type("E", (), {"config": {"host": "h", "dbname": "other"}, "sql": FakeSQL()})()
+    b.tx.engine = type(
+        "E", (), {"config": {"host": "h", "dbname": "other"}, "sql": FakeSQL()}
+    )()
     where = {"email_address": "a@example.com"}
     assert a.find(where, cache_ttl=60)["sys_id"] == 7
     assert b.find(where, cache_ttl=60)["sys_id"] == 8  # not a's cached row