activemodel 0.10.0__py3-none-any.whl → 0.11.0__py3-none-any.whl

activemodel/base_model.py

@@ -196,12 +196,13 @@ class BaseModel(SQLModel):
         "convenience method to avoid having to write .select().where() in order to add conditions"
         return cls.select().where(*args)
 
+    # TODO we should add an instance method for this as well
     @classmethod
     def upsert(
         cls,
         data: dict[str, t.Any],
         unique_by: str | list[str],
-    ) -> None:
+    ) -> t.Self:
         """
         This method will insert a new record if it doesn't exist, or update the existing record if it does.
 

activemodel/mixins/typeid.py

@@ -1,6 +1,7 @@
 from sqlmodel import Column, Field
 from typeid import TypeID
 
+from activemodel.types import typeid_patch  # noqa: F401
 from activemodel.types.typeid import TypeIDType
 
 # global list of prefixes to ensure uniqueness
@@ -8,6 +9,10 @@ _prefixes: list[str] = []
 
 
 def TypeIDMixin(prefix: str):
+    """
+    Mixin that adds a TypeID primary key field to a SQLModel. Specify the prefix to use for the TypeID.
+    """
+
     # make sure duplicate prefixes are not used!
     # NOTE this will cause issues on code reloads
     assert prefix
@@ -23,9 +28,12 @@ def TypeIDMixin(prefix: str):
                 TypeIDType(prefix),
                 primary_key=True,
                 nullable=False,
+                # default on the sa_column level ensures that an ID is generated when creating a new record, even when
+                # raw SQLAlchemy operations are used instead of activemodel operations
                 default=lambda: TypeID(prefix),
             ),
-            # default_factory=lambda: TypeID(prefix),
+            # add a database comment to document the prefix, since it's not stored in the DB otherwise
+            description=f"TypeID with prefix: {prefix}",
         )
 
     _prefixes.append(prefix)

activemodel/session_manager.py

@@ -47,7 +47,7 @@ class SessionManager:
     "singleton instance of SessionManager"
 
     session_connection: Connection | None
-    "optionally specify a specific session connection to use for all get_session() calls, useful for testing"
+    "optionally specify a specific session connection to use for all get_session() calls, useful for testing and migrations"
 
     @classmethod
     def get_instance(cls, database_url: str | None = None) -> "SessionManager":
@@ -82,6 +82,8 @@ class SessionManager:
         return self._engine
 
     def get_session(self):
+        "get a new database session, respecting any globally set sessions"
+
         if gsession := _session_context.get():
 
             @contextlib.contextmanager
@@ -103,23 +105,37 @@ def init(database_url: str):
 
 
 def get_engine():
+    "alias to get the database engine without importing SessionManager"
     return SessionManager.get_instance().get_engine()
 
 
 def get_session():
+    "alias to get a database session without importing SessionManager"
     return SessionManager.get_instance().get_session()
 
 
-# contextvars must be at the top-level of a module! You will not get a warning if you don't do this.
-# ContextVar is implemented in C, so it's very special and is both thread-safe and asyncio safe. This variable gives us
-# a place to persist a session to use globally across the application.
 _session_context = contextvars.ContextVar[Session | None](
     "session_context", default=None
 )
+"""
+This is a VERY important ContextVar, it sets a global session to be used across all ActiveModel operations by default
+and ensures get_session() uses this session as well.
+
+contextvars must be at the top-level of a module! You will not get a warning if you don't do this.
+ContextVar is implemented in C, so it's very special and is both thread-safe and asyncio safe. This variable gives us
+a place to persist a session to use globally across the application.
+"""
 
 
 @contextlib.contextmanager
 def global_session():
+    """
+    Generate a session shared across all activemodel calls.
+
+    Alternatively, you can pass a session to use globally into the context manager, which is helpful for migrations
+    and testing.
+    """
+
     if _session_context.get() is not None:
         raise RuntimeError("global session already set")
 

activemodel/types/typeid.py

@@ -2,7 +2,6 @@
 Lifted from: https://github.com/akhundMurad/typeid-python/blob/main/examples/sqlalchemy.py
 """
 
-from typing import Optional
 from uuid import UUID
 
 from pydantic import (
@@ -19,25 +18,32 @@ from activemodel.errors import TypeIDValidationError
 class TypeIDType(types.TypeDecorator):
     """
     A SQLAlchemy TypeDecorator that allows storing TypeIDs in the database.
-    The prefix will not be persisted, instead the database-native UUID field will be used.
-    At retrieval time a TypeID will be constructed based on the configured prefix and the
-    UUID value from the database.
-
-    Usage:
-        # will result in TypeIDs such as "user_01h45ytscbebyvny4gc8cr8ma2"
-        id = mapped_column(
-            TypeIDType("user"),
-            primary_key=True,
-            default=lambda: TypeID("user")
-        )
+
+    The prefix will not be persisted to the database, instead the database-native UUID field will be used.
+    At retrieval time a TypeID will be constructed (in python) based on the configured prefix and the UUID
+    value from the database.
+
+    For example:
+
+    >>> id = mapped_column(
+    >>>     TypeIDType("user"),
+    >>>     primary_key=True,
+    >>>     default=lambda: TypeID("user")
+    >>> )
+
+    Will result in TypeIDs such as "user_01h45ytscbebyvny4gc8cr8ma2". There's a mixin provided to make it easy
+    to add a `id` pk field to your model with a specific prefix.
     """
 
+    # TODO are we sure we wouldn't use TypeID here?
     impl = types.Uuid
+    # TODO why the types version?
     # impl = uuid.UUID
+
     cache_ok = True
-    prefix: Optional[str] = None
+    prefix: str
 
-    def __init__(self, prefix: Optional[str], *args, **kwargs):
+    def __init__(self, prefix: str, *args, **kwargs):
         self.prefix = prefix
         super().__init__(*args, **kwargs)
 
@@ -90,6 +96,8 @@ class TypeIDType(types.TypeDecorator):
         raise ValueError("Unexpected input type")
 
     def process_result_value(self, value, dialect):
+        "convert a raw UUID, without a prefix, to a TypeID with the correct prefix"
+
         if value is None:
             return None
 
@@ -123,13 +131,19 @@ class TypeIDType(types.TypeDecorator):
         - https://github.com/alice-biometrics/petisco/blob/b01ef1b84949d156f73919e126ed77aa8e0b48dd/petisco/base/domain/model/uuid.py#L50
         """
 
+        def convert_from_string(value: str | TypeID) -> TypeID:
+            if isinstance(value, TypeID):
+                return value
+
+            return TypeID.from_string(value)
+
         from_uuid_schema = core_schema.chain_schema(
             [
                 # TODO not sure how this is different from the UUID schema, should try it  out.
                 # core_schema.is_instance_schema(TypeID),
                 # core_schema.uuid_schema(),
                 core_schema.no_info_plain_validator_function(
-                    TypeID.from_string,
+                    convert_from_string,
                     json_schema_input_schema=core_schema.str_schema(),
                 ),
             ]
@@ -151,11 +165,10 @@ class TypeIDType(types.TypeDecorator):
             #     )
             # },
             python_schema=core_schema.union_schema([from_uuid_schema]),
-            serialization=core_schema.plain_serializer_function_ser_schema(
-                lambda x: str(x)
-            ),
+            serialization=core_schema.plain_serializer_function_ser_schema(str),
         )
 
+    # TODO I have a feeling that the `serialization` param in the above method solves this for us.
     @classmethod
     def __get_pydantic_json_schema__(
         cls, schema: CoreSchema, handler: GetJsonSchemaHandler
@@ -164,18 +177,18 @@ class TypeIDType(types.TypeDecorator):
         Called when generating the openapi schema. This overrides the `function-plain` type which
         is generated by the `no_info_plain_validator_function`.
 
-        This logis seems to be a hot part of the codebase, so I'd expect this to break as pydantic
+        This logic seems to be a hot part of the codebase, so I'd expect this to break as pydantic
         fastapi continue to evolve.
 
         Note that this method can return multiple types. A return value can be as simple as:
 
-        {"type": "string"}
+        >>> {"type": "string"}
 
         Or, you could return a more specific JSON schema type:
 
-        core_schema.uuid_schema()
+        >>> core_schema.uuid_schema()
 
-        The problem with using something like uuid_schema is the specifi patterns
+        The problem with using something like uuid_schema is the specific patterns
 
         https://github.com/BeanieODM/beanie/blob/2190cd9d1fc047af477d5e6897cc283799f54064/beanie/odm/fields.py#L153
         """

activemodel/types/typeid_patch.py

@@ -0,0 +1,22 @@
+from typing import Any, Type
+
+from pydantic import GetCoreSchemaHandler
+from pydantic_core import CoreSchema, core_schema
+
+from typeid import TypeID
+
+
+@classmethod
+def get_pydantic_core_schema(
+    cls: Type[TypeID], source_type: Any, handler: GetCoreSchemaHandler
+) -> CoreSchema:
+    return core_schema.union_schema(
+        [
+            core_schema.str_schema(),
+            core_schema.is_instance_schema(cls),
+        ],
+        serialization=core_schema.plain_serializer_function_ser_schema(str),
+    )
+
+
+TypeID.__get_pydantic_core_schema__ = get_pydantic_core_schema

{activemodel-0.10.0.dist-info → activemodel-0.11.0.dist-info}/METADATA

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: activemodel
-Version: 0.10.0
+Version: 0.11.0
 Summary: Make SQLModel more like an a real ORM
 Project-URL: Repository, https://github.com/iloveitaly/activemodel
 Author-email: Michael Bianco <iloveitaly@gmail.com>
@@ -110,14 +110,14 @@ index 0d07420..a63631c 100644
  # Use forward slashes (/) also on windows to provide an os agnostic path
 -script_location = .
 +script_location = migrations
- 
+
  # template used to generate migration file names; The default value is %%(rev)s_%%(slug)s
  # Uncomment the line below if you want the files to be prepended with date and time
  # see https://alembic.sqlalchemy.org/en/latest/tutorial.html#editing-the-ini-file
  # for all available tokens
  # file_template = %%(year)d_%%(month).2d_%%(day).2d_%%(hour).2d%%(minute).2d-%%(rev)s_%%(slug)s
 +file_template = %%(year)d_%%(month).2d_%%(day).2d_%%(rev)s_%%(slug)s
- 
+
  # sys.path path, will be prepended to sys.path if present.
  # defaults to the current working directory.
 diff --git i/test/migrations/env.py w/test/migrations/env.py
@@ -129,12 +129,12 @@ index 36112a3..a1e15c2 100644
 +# isort: off
 +
  from logging.config import fileConfig
- 
+
  from sqlalchemy import engine_from_config
 @@ -14,11 +17,17 @@ config = context.config
  if config.config_file_name is not None:
      fileConfig(config.config_file_name)
- 
+
 +from sqlmodel import SQLModel
 +from test.models import *
 +from test.utils import database_url
@@ -147,7 +147,7 @@ index 36112a3..a1e15c2 100644
  # target_metadata = mymodel.Base.metadata
 -target_metadata = None
 +target_metadata = SQLModel.metadata
- 
+
  # other values from the config, defined by the needs of env.py,
  # can be acquired:
 diff --git i/test/migrations/script.py.mako w/test/migrations/script.py.mako
@@ -155,13 +155,13 @@ index fbc4b07..9dc78bb 100644
 --- i/test/migrations/script.py.mako
 +++ w/test/migrations/script.py.mako
 @@ -9,6 +9,8 @@ from typing import Sequence, Union
- 
+
  from alembic import op
  import sqlalchemy as sa
 +import sqlmodel
 +import activemodel
  ${imports if imports else ""}
- 
+
  # revision identifiers, used by Alembic.
 ```
 
@@ -178,7 +178,7 @@ This tool is added to all `BaseModel`s and makes it easy to write SQL queries. S
 
 ### Easy Database Sessions
 
-I hate the idea f 
+I hate the idea f
 
 * Behavior should be intuitive and easy to understand. If you run `save()`, it should save, not stick the save in a transaction.
 * Don't worry about dead sessions. This makes it easy to lazy-load computed properties and largely eliminates the need to think about database sessions.
@@ -186,7 +186,7 @@ I hate the idea f
 There are a couple of thorny problems we need to solve for here:
 
 * In-memory fastapi servers are not the same as a uvicorn server, which is threaded *and* uses some sort of threadpool model for handling async requests. I don't claim to understand the entire implementation. For global DB session state (a) we can't use global variables (b) we can't use thread-local variables.
-* 
+*
 
 https://github.com/tomwojcik/starlette-context
 
@@ -202,8 +202,11 @@ https://github.com/tomwojcik/starlette-context
 SQLModel & SQLAlchemy are tricky. Here are some useful internal tricks:
 
 * `__sqlmodel_relationships__` is where any `RelationshipInfo` objects are stored. This is used to generate relationship fields on the object.
-* `ModelClass.relationship_name.property.local_columns` 
+* `ModelClass.relationship_name.property.local_columns`
 * Get cached fields from a model `object_state(instance).dict.get(field_name)`
+* Set the value on a field, without marking it as dirty `attributes.set_committed_value(instance, field_name, val)`
+* Is a model dirty `instance_state(instance).modified`
+* `select(Table).outerjoin??` won't work in a ipython session, but `Table.__table__.outerjoin??` will. `__table__` is a reference to the underlying SQLAlchemy table record.
 
 ### TypeID
 

{activemodel-0.10.0.dist-info → activemodel-0.11.0.dist-info}/RECORD

@@ -1,24 +1,25 @@
 activemodel/__init__.py,sha256=q_lHQyIM70ApvjduTo9GtenQjJXsfYZsAAquD_51kF4,137
-activemodel/base_model.py,sha256=VSItKKMxP-g-en_v16VnR9W6ueSLlWqmxn7I2-TgpGk,16646
+activemodel/base_model.py,sha256=2lUcmOxS1i1K9qdKEVjI9vLpbFltUlh-cAfvGPYbXlI,16709
 activemodel/celery.py,sha256=L1vKcO_HoPA5ZCfsXjxgPpDUMYDuoQMakGA9rppN7Lo,897
 activemodel/errors.py,sha256=wycWYmk9ws4TZpxvTdtXVy2SFESb8NqKgzdivBoF0vw,115
 activemodel/get_column_from_field_patch.py,sha256=wAEDm_ZvSqyJwfgkXVpxsevw11hd-7VLy7zuJG8Ak7Y,4986
 activemodel/logger.py,sha256=vU7QiGSy_AJuJFmClUocqIJ-Ltku_8C24ZU8L6fLJR0,53
 activemodel/query_wrapper.py,sha256=rNdvueppMse2MIi-RafTEC34GPGRal_wqH2CzhmlWS8,2520
-activemodel/session_manager.py,sha256=ltmUyBsYCNNddoilLWrh3HX9QY9eQSZiRsyFf0awevs,4835
+activemodel/session_manager.py,sha256=9Yb5sPOUginIC7M0oB3dvxkhaidX2iKGFpV3lpXkKzw,5454
 activemodel/utils.py,sha256=g17UqkphzTmb6YdpmYwT1TM00eDiXXuWn39-xNiu0AA,2112
 activemodel/mixins/__init__.py,sha256=05EQl2u_Wgf_wkly-GTaTsR7zWpmpKcb96Js7r_rZTw,160
 activemodel/mixins/pydantic_json.py,sha256=0pprGZA95BGZL4WOh--NJcvxLWey4YW85lLk4GGTjFM,3530
 activemodel/mixins/soft_delete.py,sha256=Ax4mGsQI7AVTE8c4GiWxpyB_W179-dDct79GtjP0owU,461
 activemodel/mixins/timestamps.py,sha256=Q-IFljeVVJQqw3XHdOi7dkqzefiVg1zhJvq_bldpmjg,992
-activemodel/mixins/typeid.py,sha256=WBZwnryF2QkI1ki0fW-jEbE8cIqMIldwkaeJdGT01S4,841
+activemodel/mixins/typeid.py,sha256=VjhORJ-wf3HT43DMmez6MmZTWjH7fb5c-7Qcdwgdiqg,1331
 activemodel/pytest/__init__.py,sha256=W9KKQHbPkyq0jrMXaiL8hG2Nsbjy_LN9HhvgGm8W_7g,98
 activemodel/pytest/transaction.py,sha256=ln-3N5tXHT0fqy6a8m_NIYg5AXAeA2hDuftQtFxNqi4,2600
 activemodel/pytest/truncate.py,sha256=IGiPLkUm2yyOKww6c6CKcVbwi2xAAFBopx9q2ABfu8w,1582
 activemodel/types/__init__.py,sha256=y5fiGVtPJxGEhuf-TvyrkhM2yaKRcIWo6XAx-CFFjM8,31
-activemodel/types/typeid.py,sha256=1xB79DGIC5-P-PcLpeZW9Ed_WjFOmmVW1yl2Q3pPJis,7250
-activemodel-0.10.0.dist-info/METADATA,sha256=rin3Edbj6CFW8-cj6fcL6S6GU8a1Qn0Sl9tGOLM-_rw,9652
-activemodel-0.10.0.dist-info/WHEEL,sha256=qtCwoSJWgHk21S1Kb4ihdzI2rlJ1ZKaIurTj_ngOhyQ,87
-activemodel-0.10.0.dist-info/entry_points.txt,sha256=YLX62TP_hR-n3HMBkdBex4W7XRiyOtIPkwy22puIjjQ,61
-activemodel-0.10.0.dist-info/licenses/LICENSE,sha256=L8mmpX47rB-xtJ_HsK0zpfO6viEjxbLYGn70BMp8os4,1071
-activemodel-0.10.0.dist-info/RECORD,,
+activemodel/types/typeid.py,sha256=Vqzete8IvZ5SHKf3DW2eKIWxweIZvUN2kjhLNuOl3Cc,7753
+activemodel/types/typeid_patch.py,sha256=y6kiCJQ_NzeKfuI4UtRAs7QW_nEog5RIA_-k4HUBMkU,575
+activemodel-0.11.0.dist-info/METADATA,sha256=tNJjq1XJNFXkua5Ui7XCr6DR_qtSQvbK_f_UPtCwHTE,9986
+activemodel-0.11.0.dist-info/WHEEL,sha256=qtCwoSJWgHk21S1Kb4ihdzI2rlJ1ZKaIurTj_ngOhyQ,87
+activemodel-0.11.0.dist-info/entry_points.txt,sha256=YLX62TP_hR-n3HMBkdBex4W7XRiyOtIPkwy22puIjjQ,61
+activemodel-0.11.0.dist-info/licenses/LICENSE,sha256=L8mmpX47rB-xtJ_HsK0zpfO6viEjxbLYGn70BMp8os4,1071
+activemodel-0.11.0.dist-info/RECORD,,