pycharter 0.0.20__py3-none-any.whl → 0.0.22__py3-none-any.whl

pycharter/metadata_store/postgres.py

@@ -889,25 +889,10 @@ class PostgresMetadataStore(MetadataStoreClient):
             existing = cur.fetchone()
 
             if existing:
-                # Update existing metadata_record
-                metadata_id = existing[0]
-                cur.execute(
-                    f"""
-                    UPDATE {self._table_name("metadata_records")}
-                    SET title = %s,
-                        status = %s,
-                        description = %s,
-                        governance_rules = %s,
-                        updated_at = CURRENT_TIMESTAMP
-                    WHERE id = %s
-                    """,
-                    (
-                        title,
-                        status,
-                        description,
-                        json.dumps(governance_rules) if governance_rules else None,
-                        metadata_id,
-                    ),
+                # Prevent overwriting existing metadata - raise error instead
+                raise ValueError(
+                    f"Metadata with title '{title}' and version '{version}' already exists. "
+                    f"Cannot create duplicate artifacts. Use a different version number."
                 )
             else:
                 # Insert new metadata_record
@@ -1103,23 +1088,10 @@ class PostgresMetadataStore(MetadataStoreClient):
             existing = cur.fetchone()
 
             if existing:
-                # Update existing coercion_rules
-                rule_id = existing[0]
-                cur.execute(
-                    f"""
-                    UPDATE {self._table_name("coercion_rules")}
-                    SET rules = %s,
-                        title = %s,
-                        schema_id = %s,
-                        updated_at = CURRENT_TIMESTAMP
-                    WHERE id = %s
-                    """,
-                    (
-                        json.dumps(coercion_rules),
-                        title,
-                        schema_id,
-                        rule_id,
-                    ),
+                # Prevent overwriting existing coercion rules - raise error instead
+                raise ValueError(
+                    f"Coercion rules with title '{title}' and version '{version}' already exists. "
+                    f"Cannot create duplicate artifacts. Use a different version number."
                 )
             else:
                 # Insert new coercion_rules
@@ -1141,7 +1113,7 @@ class PostgresMetadataStore(MetadataStoreClient):
                 )
                 rule_id = cur.fetchone()[0]
 
-            # Update data_contract with coercion_rules_id (no version field - UUID is sufficient)
+            # Update data_contract with coercion_rules_id
             cur.execute(
                 f"""
                 UPDATE {self._table_name("data_contracts")}
@@ -1151,9 +1123,7 @@ class PostgresMetadataStore(MetadataStoreClient):
                 (rule_id, data_contract_id),
             )
             conn.commit()
-
-            conn.commit()
-            return f"coercion:{schema_id}" + (f":{version}" if version else "")
+            return str(rule_id)
 
     def get_coercion_rules(
         self, schema_id: str, version: Optional[str] = None
@@ -1214,136 +1184,6 @@ class PostgresMetadataStore(MetadataStoreClient):
             conn.rollback()
             raise
 
-    def get_coercion_rules_by_id(
-        self, coercion_rules_id: str
-    ) -> Optional[Dict[str, Any]]:
-        """
-        Retrieve coercion rules directly by their UUID.
-
-        Args:
-            coercion_rules_id: Coercion rules UUID
-
-        Returns:
-            Coercion rules dictionary or None if not found
-        """
-        self._require_connection()
-        conn = self._get_connection()
-
-        try:
-            with conn.cursor(cursor_factory=RealDictCursor) as cur:
-                cur.execute(
-                    f"""
-                    SELECT rules
-                    FROM {self._table_name("coercion_rules")}
-                    WHERE id = %s
-                    """,
-                    (coercion_rules_id,),
-                )
-
-                row = cur.fetchone()
-                if not row:
-                    return None
-
-                return self._parse_jsonb(row["rules"])
-        except Exception as e:
-            conn.rollback()
-            raise
-
-    def get_validation_rules_by_id(
-        self, validation_rules_id: str
-    ) -> Optional[Dict[str, Any]]:
-        """
-        Retrieve validation rules directly by their UUID.
-
-        Args:
-            validation_rules_id: Validation rules UUID
-
-        Returns:
-            Validation rules dictionary or None if not found
-        """
-        self._require_connection()
-        conn = self._get_connection()
-
-        try:
-            with conn.cursor(cursor_factory=RealDictCursor) as cur:
-                cur.execute(
-                    f"""
-                    SELECT rules
-                    FROM {self._table_name("validation_rules")}
-                    WHERE id = %s
-                    """,
-                    (validation_rules_id,),
-                )
-
-                row = cur.fetchone()
-                if not row:
-                    return None
-
-                return self._parse_jsonb(row["rules"])
-        except Exception as e:
-            conn.rollback()
-            raise
-
-    def get_metadata_by_id(
-        self, metadata_id: str
-    ) -> Optional[Dict[str, Any]]:
-        """
-        Retrieve metadata directly by its UUID.
-
-        Args:
-            metadata_id: Metadata record UUID
-
-        Returns:
-            Metadata dictionary or None if not found
-        """
-        self._require_connection()
-        conn = self._get_connection()
-
-        try:
-            with conn.cursor(cursor_factory=RealDictCursor) as cur:
-                cur.execute(
-                    f"""
-                    SELECT *
-                    FROM {self._table_name("metadata_records")}
-                    WHERE id = %s
-                    """,
-                    (metadata_id,),
-                )
-
-                row = cur.fetchone()
-                if not row:
-                    return None
-
-                metadata_record_id = row["id"]
-
-                # Reconstruct metadata dictionary with all basic fields
-                metadata = {
-                    "title": row.get("title"),
-                    "status": row.get("status"),
-                    "type": row.get("type"),
-                    "description": row.get("description"),
-                    "version": row.get("version"),
-                    "created_at": row.get("created_at").isoformat() if row.get("created_at") else None,
-                    "updated_at": row.get("updated_at").isoformat() if row.get("updated_at") else None,
-                    "created_by": row.get("created_by"),
-                    "updated_by": row.get("updated_by"),
-                }
-
-                # Add JSON fields
-                if row.get("governance_rules"):
-                    metadata["governance_rules"] = self._parse_jsonb(
-                        row["governance_rules"]
-                    )
-
-                # Get all relationships
-                relationships = self._get_metadata_relationships(cur, metadata_record_id)
-                metadata.update(relationships)
-
-                return metadata
-        except Exception as e:
-            conn.rollback()
-            raise
-
     # ============================================================================
     # Validation Rules Operations
     # ============================================================================
@@ -1418,23 +1258,10 @@ class PostgresMetadataStore(MetadataStoreClient):
             existing = cur.fetchone()
 
             if existing:
-                # Update existing validation_rules
-                rule_id = existing[0]
-                cur.execute(
-                    f"""
-                    UPDATE {self._table_name("validation_rules")}
-                    SET rules = %s,
-                        title = %s,
-                        schema_id = %s,
-                        updated_at = CURRENT_TIMESTAMP
-                    WHERE id = %s
-                    """,
-                    (
-                        json.dumps(validation_rules),
-                        title,
-                        schema_id,
-                        rule_id,
-                    ),
+                # Prevent overwriting existing validation rules - raise error instead
+                raise ValueError(
+                    f"Validation rules with title '{title}' and version '{version}' already exists. "
+                    f"Cannot create duplicate artifacts. Use a different version number."
                 )
             else:
                 # Insert new validation_rules
@@ -1456,7 +1283,7 @@ class PostgresMetadataStore(MetadataStoreClient):
                 )
                 rule_id = cur.fetchone()[0]
 
-            # Update data_contract with validation_rules_id (no version field - UUID is sufficient)
+            # Update data_contract with validation_rules_id
             cur.execute(
                 f"""
                 UPDATE {self._table_name("data_contracts")}
@@ -1466,9 +1293,7 @@ class PostgresMetadataStore(MetadataStoreClient):
                 (rule_id, data_contract_id),
             )
             conn.commit()
-
-            conn.commit()
-            return f"validation:{schema_id}" + (f":{version}" if version else "")
+            return str(rule_id)
 
     def get_validation_rules(
         self, schema_id: str, version: Optional[str] = None

pycharter/metadata_store/sqlite.py

@@ -562,25 +562,10 @@ class SQLiteMetadataStore(MetadataStoreClient):
         existing = cursor.fetchone()
 
         if existing:
-            # Update existing metadata_record
-            metadata_id = existing[0]
-            cursor.execute(
-                """
-                UPDATE metadata_records
-                SET title = ?,
-                    status = ?,
-                    description = ?,
-                    governance_rules = ?,
-                    updated_at = CURRENT_TIMESTAMP
-                WHERE id = ?
-                """,
-                (
-                    title,
-                    status,
-                    description,
-                    json.dumps(governance_rules) if governance_rules else None,
-                    metadata_id,
-                ),
+            # Prevent overwriting existing metadata - raise error instead
+            raise ValueError(
+                f"Metadata with title '{title}' and version '{version}' already exists. "
+                f"Cannot create duplicate artifacts. Use a different version number."
             )
         else:
             # Generate UUID for new metadata_record
@@ -755,17 +740,10 @@ class SQLiteMetadataStore(MetadataStoreClient):
         existing = cursor.fetchone()
 
         if existing:
-            # Update existing coercion_rules
-            rule_id = existing[0]
-            cursor.execute(
-                """
-                UPDATE coercion_rules
-                SET rules = ?,
-                    title = ?,
-                    schema_id = ?
-                WHERE id = ?
-                """,
-                (json.dumps(coercion_rules), title, schema_id, rule_id),
+            # Prevent overwriting existing coercion rules - raise error instead
+            raise ValueError(
+                f"Coercion rules with title '{title}' and version '{version}' already exists. "
+                f"Cannot create duplicate artifacts. Use a different version number."
             )
         else:
             # Generate UUID for new coercion_rules
@@ -1027,17 +1005,10 @@ class SQLiteMetadataStore(MetadataStoreClient):
         existing = cursor.fetchone()
 
         if existing:
-            # Update existing validation_rules
-            rule_id = existing[0]
-            cursor.execute(
-                """
-                UPDATE validation_rules
-                SET rules = ?,
-                    title = ?,
-                    schema_id = ?
-                WHERE id = ?
-                """,
-                (json.dumps(validation_rules), title, schema_id, rule_id),
+            # Prevent overwriting existing validation rules - raise error instead
+            raise ValueError(
+                f"Validation rules with title '{title}' and version '{version}' already exists. "
+                f"Cannot create duplicate artifacts. Use a different version number."
             )
         else:
             # Generate UUID for new validation_rules