cinchdb 0.1.7__tar.gz → 0.1.8__tar.gz

{cinchdb-0.1.7 → cinchdb-0.1.8}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: cinchdb
-Version: 0.1.7
+Version: 0.1.8
 Summary: A Git-like SQLite database management system with branching and multi-tenancy
 Project-URL: Homepage, https://github.com/russellromney/cinchdb
 Project-URL: Documentation, https://russellromney.github.io/cinchdb

{cinchdb-0.1.7 → cinchdb-0.1.8}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "cinchdb"
-version = "0.1.7"
+version = "0.1.8"
 description = "A Git-like SQLite database management system with branching and multi-tenancy"
 readme = "README.md"
 requires-python = ">=3.10"

{cinchdb-0.1.7 → cinchdb-0.1.8}/src/cinchdb/core/database.py

@@ -507,8 +507,12 @@ class CinchDB:
             # Named index
             db.create_index("products", ["category", "price"], name="idx_category_price")
         """
+        # Convert parameters to Index model for validation
+        from cinchdb.models import Index
+        index = Index(columns=columns, name=name, unique=unique)
+        
         if self.is_local:
-            return self.indexes.create_index(table, columns, name, unique)
+            return self.indexes.create_index(table, index.columns, index.name, index.unique)
         else:
             # Remote index creation
             result = self._make_request(
@@ -516,9 +520,9 @@ class CinchDB:
                 "/indexes",
                 json={
                     "table": table,
-                    "columns": columns,
-                    "name": name,
-                    "unique": unique,
+                    "columns": index.columns,
+                    "name": index.name,
+                    "unique": index.unique,
                 },
             )
             return result.get("name")

{cinchdb-0.1.7 → cinchdb-0.1.8}/src/cinchdb/managers/index.py

@@ -52,14 +52,18 @@ class IndexManager:
         Raises:
             ValueError: If table doesn't exist or columns are invalid
         """
-        if not columns:
+        # Convert parameters to Index model for validation
+        from cinchdb.models import Index
+        index = Index(columns=columns, name=name, unique=unique)
+        
+        if not index.columns:
             raise ValueError("At least one column must be specified for the index")
 
         # Generate index name if not provided
-        if not name:
-            column_str = "_".join(columns)
-            unique_prefix = "uniq_" if unique else "idx_"
-            name = f"{unique_prefix}{table}_{column_str}"
+        if not index.name:
+            column_str = "_".join(index.columns)
+            unique_prefix = "uniq_" if index.unique else "idx_"
+            index.name = f"{unique_prefix}{table}_{column_str}"
 
         # Get connection to main tenant database (indexes are branch-level)
         db_path = get_tenant_db_path(
@@ -79,18 +83,18 @@ class IndexManager:
             result = conn.execute(f"PRAGMA table_info({table})")
             existing_columns = {row[1] for row in result.fetchall()}
             
-            invalid_columns = set(columns) - existing_columns
+            invalid_columns = set(index.columns) - existing_columns
             if invalid_columns:
                 raise ValueError(
                     f"Columns {invalid_columns} do not exist in table '{table}'"
                 )
             
             # Build and execute CREATE INDEX statement
-            unique_clause = "UNIQUE " if unique else ""
+            unique_clause = "UNIQUE " if index.unique else ""
             if_not_exists_clause = "IF NOT EXISTS " if if_not_exists else ""
-            column_list = ", ".join(columns)
+            column_list = ", ".join(index.columns)
             
-            sql = f"CREATE {unique_clause}INDEX {if_not_exists_clause}{name} ON {table} ({column_list})"
+            sql = f"CREATE {unique_clause}INDEX {if_not_exists_clause}{index.name} ON {table} ({column_list})"
             
             try:
                 result = conn.execute(sql)
@@ -98,18 +102,18 @@ class IndexManager:
             except sqlite3.Error as e:
                 if "already exists" in str(e):
                     if not if_not_exists:
-                        raise ValueError(f"Index '{name}' already exists")
+                        raise ValueError(f"Index '{index.name}' already exists")
                 else:
                     raise
         
         # Track the change
         self._track_change(
             ChangeType.CREATE_INDEX,
-            name,
-            {"table": table, "columns": columns, "unique": unique}
+            index.name,
+            {"table": table, "columns": index.columns, "unique": index.unique}
         )
         
-        return name
+        return index.name
 
     def drop_index(self, name: str, if_exists: bool = True) -> None:
         """Drop an index.

{cinchdb-0.1.7 → cinchdb-0.1.8}/src/cinchdb/managers/table.py

@@ -1,9 +1,12 @@
 """Table management for CinchDB."""
 
 from pathlib import Path
-from typing import List
+from typing import List, Optional, TYPE_CHECKING
 
 from cinchdb.models import Table, Column, Change, ChangeType
+
+if TYPE_CHECKING:
+    from cinchdb.models import Index
 from cinchdb.core.connection import DatabaseConnection
 from cinchdb.core.path_utils import get_tenant_db_path
 from cinchdb.core.maintenance import check_maintenance_mode
@@ -61,12 +64,13 @@ class TableManager:
 
         return tables
 
-    def create_table(self, table_name: str, columns: List[Column]) -> Table:
-        """Create a new table with optional foreign key constraints.
+    def create_table(self, table_name: str, columns: List[Column], indexes: Optional[List["Index"]] = None) -> Table:
+        """Create a new table with optional foreign key constraints and indexes.
 
         Args:
             table_name: Name of the table
             columns: List of Column objects defining the schema
+            indexes: Optional list of Index objects to create on the table
 
         Returns:
             Created Table object
@@ -168,6 +172,20 @@ class TableManager:
         applier = ChangeApplier(self.project_root, self.database, self.branch)
         applier.apply_change(change.id)
 
+        # Create indexes if specified
+        if indexes:
+            from cinchdb.managers.index import IndexManager
+            index_manager = IndexManager(self.project_root, self.database, self.branch)
+            
+            for index in indexes:
+                index_manager.create_index(
+                    table=table_name,
+                    columns=index.columns,
+                    name=index.name,
+                    unique=index.unique,
+                    if_not_exists=True
+                )
+
         # Return the created table
         return Table(
             name=table_name,

{cinchdb-0.1.7 → cinchdb-0.1.8}/src/cinchdb/managers/tenant.py

@@ -104,6 +104,10 @@ class TenantManager:
                 conn.execute(f"DELETE FROM {table}")
 
             conn.commit()
+        
+        # Open a new connection to vacuum the database
+        with DatabaseConnection(new_db_path) as conn:
+            conn.execute("VACUUM")
 
         return Tenant(
             name=tenant_name,

{cinchdb-0.1.7 → cinchdb-0.1.8}/src/cinchdb/models/__init__.py

@@ -5,7 +5,7 @@ from .project import Project
 from .database import Database
 from .branch import Branch
 from .tenant import Tenant
-from .table import Table, Column, ColumnType, ForeignKeyRef, ForeignKeyAction
+from .table import Table, Column, ColumnType, ForeignKeyRef, ForeignKeyAction, Index
 from .view import View
 from .change import Change, ChangeType
 
@@ -21,6 +21,7 @@ __all__ = [
     "ColumnType",
     "ForeignKeyRef",
     "ForeignKeyAction",
+    "Index",
     "View",
     "Change",
     "ChangeType",

{cinchdb-0.1.7 → cinchdb-0.1.8}/src/cinchdb/models/table.py

@@ -27,6 +27,16 @@ class ForeignKeyRef(BaseModel):
     )
 
 
+class Index(BaseModel):
+    """Index specification for table columns."""
+
+    model_config = ConfigDict(extra="forbid")
+
+    columns: List[str] = Field(description="Column names to index")
+    name: Optional[str] = Field(default=None, description="Index name")
+    unique: bool = Field(default=False, description="Create unique index")
+
+
 class Column(BaseModel):
     """Represents a column in a table."""