better-notion 2.1.2__tar.gz → 2.1.4__tar.gz

{better_notion-2.1.2 → better_notion-2.1.4}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: better-notion
-Version: 2.1.2
+Version: 2.1.4
 Summary: A high-level Python SDK for the Notion API with developer experience in mind.
 Project-URL: Homepage, https://github.com/nesalia-inc/better-notion
 Project-URL: Documentation, https://github.com/nesalia-inc/better-notion#readme

{better_notion-2.1.2 → better_notion-2.1.4}/better_notion/utils/agents/schemas.py

@@ -421,7 +421,13 @@ class TaskSchema:
 
     @staticmethod
     def get_schema() -> Dict[str, Dict[str, Any]]:
-        """Return Notion database schema for Tasks."""
+        """Return Notion database schema for Tasks.
+
+        Note: The following properties are NOT included here and will be added
+        after database creation via database.update():
+        - "Dependencies": Self-referential relation (requires database's own ID)
+        - "Related Work Issue": Relation to Work Issues (Work Issues created after Tasks)
+        """
         return {
             "Title": PropertyBuilder.title("Title"),
             "Version": PropertyBuilder.relation("Version"),
@@ -458,10 +464,6 @@ class TaskSchema:
                     SelectOption.option("Low", "blue"),
                 ],
             ),
-            # Dependencies: Self-referential relation (will be updated after database creation)
-            "Dependencies": PropertyBuilder.relation("Dependencies", dual_property=False),
-            # Related Work Issue: Relation to Work Issues (will be updated after database creation)
-            "Related Work Issue": PropertyBuilder.relation("Related Work Issue", dual_property=False),
             "Estimated Hours": PropertyBuilder.number("Estimated Hours"),
             "Actual Hours": PropertyBuilder.number("Actual Hours"),
             "Assignee": PropertyBuilder.people("Assignee"),

{better_notion-2.1.2 → better_notion-2.1.4}/better_notion/utils/agents/workspace.py

@@ -141,11 +141,21 @@ class WorkspaceInitializer:
                     logger.info(f"✓ {display_name} database created: {self._database_ids[key]}")
                 except Exception as e:
                     # Clean up databases created so far
+                    import traceback
                     logger.error(f"Failed to create {display_name} database: {str(e)}")
+                    logger.error(f"Full traceback:\n{traceback.format_exc()}")
+
+                    # Try to get more error details if available
+                    error_details = str(e)
+                    if hasattr(e, 'response') and hasattr(e.response, 'text'):
+                        logger.error(f"Notion API response: {e.response.text}")
+                        error_details += f"\nNotion API response: {e.response.text}"
+
                     logger.warning(f"Cleaning up {len(self._database_ids)} databases that were created...")
                     await self._delete_existing_databases(parent, self._database_ids)
+
                     error_msg = (
-                        f"Failed to create {display_name} database: {str(e)}\n"
+                        f"Failed to create {display_name} database: {error_details}\n"
                         f"All partially created databases have been deleted.\n"
                         f"Parent page: {parent_page_id}\n"
                         f"Workspace name: {workspace_name}"
@@ -159,6 +169,14 @@ class WorkspaceInitializer:
         # Update cross-database relations that require both databases to exist
         await self._update_cross_database_relations()
 
+        # Add self-referential Dependencies relation to Tasks database
+        if "tasks" in self._database_ids:
+            await self._update_relation(
+                self._database_ids["tasks"],
+                "Dependencies",
+                self._database_ids["tasks"]  # Self-reference
+            )
+
         # Save workspace metadata
         self.save_database_ids()
 
@@ -239,7 +257,7 @@ class WorkspaceInitializer:
         )
 
     async def _create_tasks_db(self, parent: Page) -> None:
-        """Create Tasks database with relations to Versions (self-referencing for dependencies)."""
+        """Create Tasks database with relations to Versions."""
         schema = TaskSchema.get_schema()
 
         # Update relations with Versions database ID
@@ -249,8 +267,6 @@ class WorkspaceInitializer:
                 "versions"
             ]
 
-        # Tasks reference themselves for dependencies
-        # Will be updated after database creation
         db = await self._client.databases.create(
             parent=parent,
             title="Tasks",
@@ -259,9 +275,6 @@ class WorkspaceInitializer:
 
         self._database_ids["tasks"] = db.id
 
-        # Update self-referential relations
-        await self._update_self_relations(db.id)
-
         # Update Versions database with reverse relation
         await self._add_reverse_relation(
             self._database_ids["versions"],
@@ -387,29 +400,6 @@ class WorkspaceInitializer:
             logger.warning(f"Failed to delete database {database_id}: {str(e)}")
             # Continue with other deletions even if this one fails
 
-    async def _update_self_relations(self, database_id: str) -> None:
-        """Update self-referential relations in Tasks database.
-
-        Note: The database was created with placeholder relations.
-        This method updates them to point to themselves.
-        """
-        # Get the current database schema
-        db = await self._client.databases.get(database_id)
-
-        # Update the Dependencies relation to point to itself
-        schema = db.schema
-        if "Dependencies" in schema:
-            schema["Dependencies"]["relation"]["database_id"] = database_id
-
-        # Update the database schema via API
-        await self._client._api._request(
-            "PATCH",
-            f"/databases/{database_id}",
-            json={"properties": schema}
-        )
-
-        logger.info(f"Updated self-referential relations for Tasks database {database_id}")
-
     async def _update_cross_database_relations(self) -> None:
         """Update cross-database relations after all databases are created.
 
@@ -453,7 +443,7 @@ class WorkspaceInitializer:
         property_name: str,
         target_database_id: Optional[str],
     ) -> None:
-        """Update a relation property to point to the target database.
+        """Add or update a relation property to point to the target database.
 
         Args:
             database_id: Database to update
@@ -468,20 +458,56 @@ class WorkspaceInitializer:
         db = await self._client.databases.get(database_id)
         schema = db.schema
 
-        # Update the relation property
+        # Add or update the relation property
+        # Some properties (like Dependencies, Related Work Issue) are added AFTER
+        # database creation, so they may not exist in the initial schema
         if property_name in schema:
+            # Property exists, just update the database_id
             schema[property_name]["relation"]["database_id"] = target_database_id
+        else:
+            # Property doesn't exist, ADD it
+            # This handles cases where the property couldn't be in the initial schema
+            if property_name == "Dependencies":
+                # Self-referential relation
+                schema[property_name] = {
+                    "relation": {
+                        "database_id": database_id,  # Self-reference
+                        "type": "dual_property",
+                        "dual_property": {
+                            "synced_property_name": "Dependent Tasks",
+                            "synced_property_type": "relation"
+                        }
+                    }
+                }
+            elif property_name == "Related Work Issue":
+                # Relation to Work Issues database
+                schema[property_name] = {
+                    "relation": {
+                        "database_id": target_database_id,
+                        "type": "dual_property",
+                        "dual_property": {
+                            "synced_property_name": "Related Tasks",
+                            "synced_property_type": "relation"
+                        }
+                    }
+                }
+            else:
+                # Generic relation
+                schema[property_name] = {
+                    "relation": {
+                        "database_id": target_database_id,
+                        "dual_property": {}
+                    }
+                }
 
-            # Update the database schema via API
-            await self._client._api._request(
-                "PATCH",
-                f"/databases/{database_id}",
-                json={"properties": schema}
-            )
+        # Update the database schema via API
+        await self._client._api._request(
+            "PATCH",
+            f"/databases/{database_id}",
+            json={"properties": schema}
+        )
 
-            logger.info(f"Updated {property_name} relation to {target_database_id}")
-        else:
-            logger.warning(f"Property {property_name} not found in database {database_id}")
+        logger.info(f"Added/Updated {property_name} relation to {target_database_id}")
 
     def save_database_ids(self, path: Optional[Path] = None) -> None:
         """Save workspace metadata (database IDs and workspace info) to config file.

{better_notion-2.1.2 → better_notion-2.1.4}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "better-notion"
-version = "2.1.2"
+version = "2.1.4"
 description = "A high-level Python SDK for the Notion API with developer experience in mind."
 readme = "README.md"
 requires-python = ">=3.10"