cleanup

Author: swaroopAkkineniWorkos

src/workos/authorization.py

@@ -34,17 +34,12 @@
 
 
 class ParentResourceById(TypedDict):
-    """Identify a parent resource by its WorkOS resource ID."""
-
-    resource_id: str
+    parent_resource_id: str
 
 
 class ParentResourceByExternalId(TypedDict):
-    """Identify a parent resource by organization, type, and external ID."""
-
-    organization_id: str
-    resource_type: str
-    external_id: str
+    parent_resource_external_id: str
+    parent_resource_type_slug: str
 
 
 ParentResource = Union[ParentResourceById, ParentResourceByExternalId]
@@ -188,18 +183,20 @@ def get_resource(self, resource_id: str) -> SyncOrAsync[Resource]: ...
     def create_resource(
         self,
         *,
-        resource_type: str,
+        resource_type_slug: str,
         organization_id: str,
-        external_id: Optional[str] = None,
-        meta: Optional[Dict[str, Any]] = None,
-        parent: Optional[ParentResource] = None,
+        external_id: str,
+        name: str,
+        parent: ParentResource,
+        description: Optional[str] = None,
     ) -> SyncOrAsync[Resource]: ...
 
     def update_resource(
         self,
         resource_id: str,
         *,
-        meta: Optional[Dict[str, Any]] = None,
+        name: Optional[str] = None,
+        description: Optional[str] = None,
     ) -> SyncOrAsync[Resource]: ...
 
     def delete_resource(
@@ -498,22 +495,22 @@ def get_resource(self, resource_id: str) -> Resource:
     def create_resource(
         self,
         *,
-        resource_type: str,
+        resource_type_slug: str,
         organization_id: str,
-        external_id: Optional[str] = None,
-        meta: Optional[Dict[str, Any]] = None,
-        parent: Optional[ParentResource] = None,
+        external_id: str,
+        name: str,
+        parent: ParentResource,
+        description: Optional[str] = None,
     ) -> Resource:
         json: Dict[str, Any] = {
-            "resource_type": resource_type,
+            "resource_type_slug": resource_type_slug,
             "organization_id": organization_id,
+            "external_id": external_id,
+            "name": name,
+            **parent,
         }
-        if external_id is not None:
-            json["external_id"] = external_id
-        if meta is not None:
-            json["meta"] = meta
-        if parent is not None:
-            json["parent"] = parent
+        if description is not None:
+            json["description"] = description
 
         response = self._http_client.request(
             AUTHORIZATION_RESOURCES_PATH,
@@ -525,13 +522,16 @@ def create_resource(
 
     def update_resource(
         self,
-        resource_id: str,
         *,
-        meta: Optional[Dict[str, Any]] = None,
+        resource_id: str,
+        name: Optional[str] = None,
+        description: Optional[str] = None,
     ) -> Resource:
         json: Dict[str, Any] = {}
-        if meta is not None:
-            json["meta"] = meta
+        if name is not None:
+            json["name"] = name
+        if description is not None:
+            json["description"] = description
 
         response = self._http_client.request(
             f"{AUTHORIZATION_RESOURCES_PATH}/{resource_id}",
@@ -543,8 +543,8 @@ def update_resource(
 
     def delete_resource(
         self,
-        resource_id: str,
         *,
+        resource_id: str,
         cascade_delete: Optional[bool] = None,
     ) -> None:
         if cascade_delete is not None:
@@ -847,22 +847,22 @@ async def get_resource(self, resource_id: str) -> Resource:
     async def create_resource(
         self,
         *,
-        resource_type: str,
+        resource_type_slug: str,
         organization_id: str,
-        external_id: Optional[str] = None,
-        meta: Optional[Dict[str, Any]] = None,
-        parent: Optional[ParentResource] = None,
+        external_id: str,
+        name: str,
+        parent: ParentResource,
+        description: Optional[str] = None,
     ) -> Resource:
         json: Dict[str, Any] = {
-            "resource_type": resource_type,
+            "resource_type_slug": resource_type_slug,
             "organization_id": organization_id,
+            "external_id": external_id,
+            "name": name,
+            **parent,
         }
-        if external_id is not None:
-            json["external_id"] = external_id
-        if meta is not None:
-            json["meta"] = meta
-        if parent is not None:
-            json["parent"] = parent
+        if description is not None:
+            json["description"] = description
 
         response = await self._http_client.request(
             AUTHORIZATION_RESOURCES_PATH,
@@ -876,11 +876,14 @@ async def update_resource(
         self,
         resource_id: str,
         *,
-        meta: Optional[Dict[str, Any]] = None,
+        name: Optional[str] = None,
+        description: Optional[str] = None,
     ) -> Resource:
         json: Dict[str, Any] = {}
-        if meta is not None:
-            json["meta"] = meta
+        if name is not None:
+            json["name"] = name
+        if description is not None:
+            json["description"] = description
 
         response = await self._http_client.request(
             f"{AUTHORIZATION_RESOURCES_PATH}/{resource_id}",

src/workos/types/authorization/resource.py

@@ -14,8 +14,5 @@ class Resource(WorkOSModel):
     resource_type_slug: str
     organization_id: str
     parent_resource_id: Optional[str] = None
-    # The API returns meta when set via create_resource / update_resource.
-    # Without this field the model would silently discard that data.
-    meta: Optional[Mapping[str, Any]] = None
     created_at: str
     updated_at: str