velocity-python 0.1.61__tar.gz → 0.1.63__tar.gz

{velocity_python-0.1.61 → velocity_python-0.1.63}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: velocity-python
-Version: 0.1.61
+Version: 0.1.63
 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.61 → velocity_python-0.1.63}/pyproject.toml

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

{velocity_python-0.1.61 → velocity_python-0.1.63}/src/velocity/__init__.py

@@ -1,4 +1,4 @@
-__version__ = version = "0.1.61"
+__version__ = version = "0.1.63"
 
 import importlib as _importlib
 

velocity_python-0.1.63/src/velocity/db/core/jsonproxy.py

@@ -0,0 +1,175 @@
+"""
+Live list/dict proxies for JSON column values.
+
+A ``list`` or ``dict`` stored in a row is just a JSON column (JSONB on
+PostgreSQL).  ``Row.__getitem__`` wraps such values in :class:`BoundList` /
+:class:`BoundDict` so in-place mutation writes the whole value back through
+the owning row::
+
+    row["tags"].append("vip")          # UPDATE ... SET tags = '["a","vip"]'
+    row["meta"]["verified"] = True     # whole "meta" value written back
+
+There are no relation declarations and no diffing: every mutation persists
+the complete top-level value.  Nested containers share the root's flush, so
+a deep mutation also writes the root value back.
+"""
+
+
+class _Controller:
+    """Shared link between a root proxy and all of its nested proxies."""
+
+    __slots__ = ("flush", "root")
+
+    def __init__(self, flush):
+        self.flush = flush
+        self.root = None
+
+    def notify(self):
+        self.flush(unwrap(self.root))
+
+
+def bind(value, flush):
+    """
+    Wrap ``value`` (a dict or list) in a bound proxy tree.
+
+    ``flush(plain_value)`` is called with the plain (unwrapped) top-level
+    value after every mutation anywhere in the tree.  Non-container values
+    are returned unchanged.
+    """
+    if not isinstance(value, (dict, list)):
+        return value
+    ctrl = _Controller(flush)
+    root = _wrap(value, ctrl)
+    ctrl.root = root
+    return root
+
+
+def unwrap(value):
+    """Return a plain dict/list copy of a (possibly bound) value tree."""
+    if isinstance(value, dict):
+        return {k: unwrap(v) for k, v in value.items()}
+    if isinstance(value, list):
+        return [unwrap(v) for v in value]
+    return value
+
+
+def _wrap(value, ctrl):
+    if isinstance(value, dict):
+        return BoundDict(value, ctrl)
+    if isinstance(value, list):
+        return BoundList(value, ctrl)
+    return value
+
+
+class BoundList(list):
+    """A list whose mutations flush the root value back to its column."""
+
+    def __init__(self, iterable, ctrl):
+        super().__init__(_wrap(v, ctrl) for v in iterable)
+        self._ctrl = ctrl
+
+    # -- mutators ------------------------------------------------------
+    def __setitem__(self, index, value):
+        if isinstance(index, slice):
+            value = [_wrap(v, self._ctrl) for v in value]
+        else:
+            value = _wrap(value, self._ctrl)
+        super().__setitem__(index, value)
+        self._ctrl.notify()
+
+    def __delitem__(self, index):
+        super().__delitem__(index)
+        self._ctrl.notify()
+
+    def __iadd__(self, other):
+        self.extend(other)
+        return self
+
+    def __imul__(self, n):
+        result = super().__imul__(n)
+        self._ctrl.notify()
+        return result
+
+    def append(self, value):
+        super().append(_wrap(value, self._ctrl))
+        self._ctrl.notify()
+
+    def extend(self, iterable):
+        super().extend(_wrap(v, self._ctrl) for v in iterable)
+        self._ctrl.notify()
+
+    def insert(self, index, value):
+        super().insert(index, _wrap(value, self._ctrl))
+        self._ctrl.notify()
+
+    def remove(self, value):
+        super().remove(value)
+        self._ctrl.notify()
+
+    def pop(self, index=-1):
+        result = super().pop(index)
+        self._ctrl.notify()
+        return result
+
+    def clear(self):
+        super().clear()
+        self._ctrl.notify()
+
+    def sort(self, **kwds):
+        super().sort(**kwds)
+        self._ctrl.notify()
+
+    def reverse(self):
+        super().reverse()
+        self._ctrl.notify()
+
+
+class BoundDict(dict):
+    """A dict whose mutations flush the root value back to its column."""
+
+    def __init__(self, mapping, ctrl):
+        super().__init__({k: _wrap(v, ctrl) for k, v in mapping.items()})
+        self._ctrl = ctrl
+
+    # -- mutators ------------------------------------------------------
+    def __setitem__(self, key, value):
+        super().__setitem__(key, _wrap(value, self._ctrl))
+        self._ctrl.notify()
+
+    def __delitem__(self, key):
+        super().__delitem__(key)
+        self._ctrl.notify()
+
+    def update(self, dict_=None, **kwds):
+        data = {}
+        if dict_:
+            data.update(dict_)
+        data.update(kwds)
+        for k, v in data.items():
+            super().__setitem__(k, _wrap(v, self._ctrl))
+        if data:
+            self._ctrl.notify()
+
+    def pop(self, key, *args):
+        existed = key in self
+        result = super().pop(key, *args)
+        if existed:
+            self._ctrl.notify()
+        return result
+
+    def popitem(self):
+        result = super().popitem()
+        self._ctrl.notify()
+        return result
+
+    def clear(self):
+        super().clear()
+        self._ctrl.notify()
+
+    def setdefault(self, key, default=None):
+        if key in self:
+            return self[key]
+        wrapped = _wrap(default, self._ctrl)
+        super().__setitem__(key, wrapped)
+        self._ctrl.notify()
+        return wrapped

{velocity_python-0.1.61 → velocity_python-0.1.63}/src/velocity/db/core/row.py

@@ -2,6 +2,7 @@ import pprint
 import time as _time
 from collections.abc import MutableMapping
 from velocity.db.exceptions import DbColumnMissingError
+from velocity.db.core import jsonproxy
 
 
 # Attributes that live on the Row instance itself and must never be
@@ -133,9 +134,24 @@ class Row(MutableMapping):
             return self.pk[key]
         cache = self._ensure_cache()
         if key in cache:
-            return cache[key]
+            val = cache[key]
+            # JSON columns come back as plain dict/list — wrap them in live
+            # proxies so in-place mutation writes the whole value back.
+            if isinstance(val, (dict, list)) and not isinstance(
+                val, (jsonproxy.BoundDict, jsonproxy.BoundList)
+            ):
+                val = jsonproxy.bind(
+                    val, lambda plain, _k=key: self.__setitem__(_k, plain)
+                )
+                cache[key] = val
+            return val
         # Fall back to a direct DB fetch for columns not in the initial SELECT
-        return self.table.get_value(key, self.pk)
+        val = self.table.get_value(key, self.pk)
+        if isinstance(val, (dict, list)):
+            val = jsonproxy.bind(
+                val, lambda plain, _k=key: self.__setitem__(_k, plain)
+            )
+        return val
 
     def __setitem__(self, key, val):
         if key in self.pk:
@@ -327,7 +343,7 @@ class Row(MutableMapping):
         """
         Returns the row as a dictionary (from cache or via a SELECT on self.pk).
         """
-        return dict(self._ensure_cache())
+        return {k: jsonproxy.unwrap(v) for k, v in self._ensure_cache().items()}
 
     def extract(self, *args):
         """

{velocity_python-0.1.61 → velocity_python-0.1.63}/src/velocity/db/core/table.py

@@ -1984,12 +1984,21 @@ class Table:
         _ddl_logger.warning("DDL ALTER COLUMN TYPE on %s column=%s", self.name, column)
         self.tx.execute(sql, vals, cursor=self.cursor())
 
+    def _adapt_data(self, data):
+        """
+        Apply the dialect's data-value adaptation (e.g. dict/list -> JSONB on
+        PostgreSQL, JSON text elsewhere) to a column->value mapping.  Used for
+        INSERT/UPDATE data only — WHERE values keep native driver adaptation.
+        """
+        adapt = self.sql.adapt_data_value
+        return {k: adapt(v) for k, v in data.items()}
+
     @create_missing
     def update(self, data, where=None, pk=None, **kwds):
         """
         Performs an UPDATE of rows matching `where` or `pk` with `data`.
         """
-        sql, vals = self.sql.update(self.tx, self.name, data, where, pk)
+        sql, vals = self.sql.update(self.tx, self.name, self._adapt_data(data), where, pk)
         if kwds.get("sql_only", False):
             return sql, vals
         self.tx.invalidate_cache(self.name)
@@ -2002,7 +2011,7 @@ class Table:
         """
         Performs an INSERT of the given data into this table. Resets sys_id on duplicate keys if needed.
         """
-        sql, vals = self.sql.insert(self.name, data)
+        sql, vals = self.sql.insert(self.name, self._adapt_data(data))
         if kwds.get("sql_only", False):
             return sql, vals
         self.tx.invalidate_cache(self.name)
@@ -2018,7 +2027,7 @@ class Table:
         sql, vals = self.sql.merge(
             self.tx,
             self.name,
-            data,
+            self._adapt_data(data),
             pk,
             on_conflict_do_nothing=False,
             on_conflict_update=True,
@@ -2047,7 +2056,9 @@ class Table:
         """
         if not rows:
             return 0
-        sql, args_list, template = self.sql.insert_many(self.name, rows)
+        sql, args_list, template = self.sql.insert_many(
+            self.name, [self._adapt_data(row) for row in rows]
+        )
         page_size = kwds.get("page_size", 1000)
         self.tx.invalidate_cache(self.name)
         return self.tx._execute_values(sql, args_list, template=template, page_size=page_size)
@@ -2068,7 +2079,9 @@ class Table:
         """
         if not rows:
             return 0
-        sql, args_list, template = self.sql.merge_many(self.tx, self.name, rows, pk=pk)
+        sql, args_list, template = self.sql.merge_many(
+            self.tx, self.name, [self._adapt_data(row) for row in rows], pk=pk
+        )
         page_size = kwds.get("page_size", 1000)
         self.tx.invalidate_cache(self.name)
         return self.tx._execute_values(sql, args_list, template=template, page_size=page_size)
@@ -2172,7 +2185,9 @@ class Table:
                 "Current SQL dialect does not support insert-if-not-exists operations."
             )
 
-        sql, vals = ins_builder(self.tx, self.name, insert_payload, exists_where)
+        sql, vals = ins_builder(
+            self.tx, self.name, self._adapt_data(insert_payload), exists_where
+        )
         if sql_only:
             return {"update": update_stmt, "insert": (sql, vals)}
         result = self.tx.execute(sql, vals, cursor=self.cursor())
@@ -2200,7 +2215,9 @@ class Table:
                       must be present in `data`.
         :return: rowcount (0 or 1) or (sql, params) when sql_only=True
         """
-        sql, vals = self.sql.insert_if_not_exists(self.tx, self.name, data, where)
+        sql, vals = self.sql.insert_if_not_exists(
+            self.tx, self.name, self._adapt_data(data), where
+        )
         if kwds.get("sql_only", False):
             return sql, vals
         result = self.tx.execute(sql, vals, cursor=self.cursor())

{velocity_python-0.1.61 → velocity_python-0.1.63}/src/velocity/db/servers/base/sql.py

@@ -2,6 +2,7 @@
 Abstract base class for SQL dialect implementations.
 """
 
+import json
 from abc import ABC, abstractmethod
 from typing import Any, Dict, List, Optional, Tuple, Union
 
@@ -38,6 +39,21 @@ class BaseSQLDialect(ABC):
     DatabaseObjectExistsErrorCodes: List[str] = []
     DataIntegrityErrorCodes: List[str] = []
 
+    @classmethod
+    def adapt_data_value(cls, val: Any) -> Any:
+        """
+        Adapt a Python value being written to a column (INSERT/UPDATE data
+        values only — never WHERE parameters, which keep their native driver
+        adaptation, e.g. list -> ANY(...) on PostgreSQL).
+
+        Default: serialize dict/list to a JSON string for backends without a
+        native JSON parameter adapter (SQLite, SQL Server, MySQL).  PostgreSQL
+        overrides this to bind JSONB natively.
+        """
+        if isinstance(val, (dict, list)):
+            return json.dumps(val)
+        return val
+
     @classmethod
     def quote_identifier(cls, name: str) -> str:
         """Always-quote a single SQL identifier to prevent injection.

{velocity_python-0.1.61 → velocity_python-0.1.63}/src/velocity/db/servers/mysql/types.py

@@ -23,6 +23,7 @@ class TYPES(BaseTypes):
     LONGTEXT = "LONGTEXT"
     MEDIUMTEXT = "MEDIUMTEXT"
     VARCHAR = "VARCHAR"
+    JSON = "JSON"
 
     @classmethod
     def get_type(cls, v):
@@ -51,6 +52,8 @@ class TYPES(BaseTypes):
             return cls.TIME
         if isinstance(v, bytes) or v is bytes:
             return cls.BINARY
+        if isinstance(v, (dict, list)) or v is dict or v is list:
+            return cls.JSON
         return cls.TEXT
 
     @classmethod
@@ -80,6 +83,8 @@ class TYPES(BaseTypes):
             return cls.TIME
         if isinstance(v, bytes) or v is bytes:
             return cls.BINARY
+        if isinstance(v, (dict, list)) or v is dict or v is list:
+            return cls.JSON
         return cls.TEXT
 
     @classmethod
@@ -108,4 +113,6 @@ class TYPES(BaseTypes):
             return datetime.datetime
         if v == cls.BINARY or "BLOB" in v:
             return bytes
+        if v == cls.JSON:
+            return dict
         raise Exception(f"Unmapped MySQL type {v}")

{velocity_python-0.1.61 → velocity_python-0.1.63}/src/velocity/db/servers/postgres/sql.py

@@ -71,6 +71,20 @@ class SQL(BaseSQLDialect):
     DatabaseObjectExistsErrorCodes = ["42710", "42P07", "42P04"]
     DataIntegrityErrorCodes = ["23503", "23502", "23514", "23P01", "22003"]
 
+    @classmethod
+    def adapt_data_value(cls, val):
+        """
+        Bind dict/list data values as JSONB.  Lists must be wrapped here
+        because psycopg's native list adaptation targets PostgreSQL arrays
+        (kept for WHERE params, e.g. ``= ANY(%s)``); bare dicts also work via
+        the engine-registered dumper, but wrapping both keeps it uniform.
+        """
+        if isinstance(val, (dict, list)):
+            from psycopg.types.json import Jsonb
+
+            return Jsonb(val)
+        return val
+
     @classmethod
     def get_error(cls, e):
         # psycopg v3 uses 'sqlstate'; psycopg2 used 'pgcode'.

{velocity_python-0.1.61 → velocity_python-0.1.63}/src/velocity/db/servers/postgres/types.py

@@ -23,6 +23,8 @@ class TYPES(BaseTypes):
     BOOLEAN = "BOOLEAN"
     BINARY = "BYTEA"
     INTERVAL = "INTERVAL"
+    JSONB = "JSONB"
+    JSON = "JSON"
 
     @classmethod
     def get_type(cls, v):
@@ -52,6 +54,8 @@ class TYPES(BaseTypes):
             return cls.INTERVAL
         if isinstance(v, bytes) or v is bytes:
             return cls.BINARY
+        if isinstance(v, (dict, list)) or v is dict or v is list:
+            return cls.JSONB
         return cls.TEXT
 
     @classmethod
@@ -81,6 +85,8 @@ class TYPES(BaseTypes):
             return cls.INTERVAL
         if isinstance(v, bytes) or v is bytes:
             return cls.BINARY
+        if isinstance(v, (dict, list)) or v is dict or v is list:
+            return cls.JSONB
         return cls.TEXT
 
     @classmethod
@@ -107,4 +113,8 @@ class TYPES(BaseTypes):
             return datetime.timedelta
         if v == cls.DATETIME_TZ or v == cls.TIMESTAMP_TZ:
             return datetime.datetime
+        if v == cls.JSONB or v == cls.JSON:
+            # JSON columns hold either a dict or a list; dict is the
+            # canonical marker type for schema purposes.
+            return dict
         raise Exception(f"Unmapped type {v}")

{velocity_python-0.1.61 → velocity_python-0.1.63}/src/velocity/db/servers/sqlite/sql.py

@@ -660,15 +660,15 @@ END;
 
     @classmethod
     def create_savepoint(cls, sp):
-        return f"SAVEPOINT {sp}"
+        return f"SAVEPOINT {sp}", tuple()
 
     @classmethod
     def release_savepoint(cls, sp):
-        return f"RELEASE SAVEPOINT {sp}"
+        return f"RELEASE SAVEPOINT {sp}", tuple()
 
     @classmethod
     def rollback_savepoint(cls, sp):
-        return f"ROLLBACK TO SAVEPOINT {sp}"
+        return f"ROLLBACK TO SAVEPOINT {sp}", tuple()
 
     @classmethod
     def create_view(cls, name, query, temp=False, silent=True):

{velocity_python-0.1.61 → velocity_python-0.1.63}/src/velocity/payment/stripe_adapter.py

@@ -469,6 +469,14 @@ class StripeAdapter(PaymentProcessorAdapter):
             if self.platform:
                 payment_metadata["platform"] = str(self.platform)
 
+            # Optional idempotency key: when the caller supplies one, Stripe
+            # collapses retries of the *same* request (engine auto-retry, donor
+            # double-submit) into a single charge instead of charging again.
+            request_options = {}
+            idempotency_key = payment_data.get("idempotency_key")
+            if idempotency_key:
+                request_options["idempotency_key"] = str(idempotency_key)
+
             payment_intent = stripe.PaymentIntent.create(
                 amount=amount_cents,
                 currency=currency,
@@ -482,6 +490,7 @@ class StripeAdapter(PaymentProcessorAdapter):
                 },
                 metadata=payment_metadata,
                 api_key=self._api_key,
+                **request_options,
             )
             charge = self._mirror_payment_intent_artifacts(
                 tx,
@@ -575,6 +584,11 @@ class StripeAdapter(PaymentProcessorAdapter):
             if self.platform:
                 payment_metadata["platform"] = str(self.platform)
 
+            request_options = {}
+            idempotency_key = payment_data.get("idempotency_key")
+            if idempotency_key:
+                request_options["idempotency_key"] = str(idempotency_key)
+
             payment_intent = stripe.PaymentIntent.create(
                 amount=amount_cents,
                 currency=currency,
@@ -587,6 +601,7 @@ class StripeAdapter(PaymentProcessorAdapter):
                 description=payment_data.get("description"),
                 metadata=payment_metadata,
                 api_key=self._api_key,
+                **request_options,
             )
             charge = self._mirror_payment_intent_artifacts(
                 tx,

{velocity_python-0.1.61 → velocity_python-0.1.63}/src/velocity_python.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: velocity-python
-Version: 0.1.61
+Version: 0.1.63
 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.61 → velocity_python-0.1.63}/src/velocity_python.egg-info/SOURCES.txt

@@ -41,6 +41,7 @@ src/velocity/db/core/column.py
 src/velocity/db/core/database.py
 src/velocity/db/core/decorators.py
 src/velocity/db/core/engine.py
+src/velocity/db/core/jsonproxy.py
 src/velocity/db/core/result.py
 src/velocity/db/core/row.py
 src/velocity/db/core/sequence.py
@@ -165,6 +166,7 @@ tests/test_email_processing.py
 tests/test_http_handler_rollback.py
 tests/test_iconv_money_to_cents.py
 tests/test_identifier_injection_guard.py
+tests/test_json_columns.py
 tests/test_jsonb_dict_adapter.py
 tests/test_lambda_handler.py
 tests/test_lambda_handler_auth.py