Be sure to retrieve data before init'ing

Author: schloerke

src/posit/connect/content.py

@@ -52,13 +52,16 @@ def _assert_guid(guid: str):
 
 # ContentItem Repository uses a PATCH method, not a PUT for updating.
 class _ContentItemRepository(_Resource):
-    def update(self, **attributes):  # type: ignore[reportIncompatibleMethodOverride]
+    def update(self, **attributes):
         response = self._ctx.client.patch(self._path, json=attributes)
         result = response.json()
-        super().update(**result)
+        # # Calling this method will call `_Resource.update` which will try to PUT to the path.
+        # super().update(**result)
+        # Instead, update the dict directly.
+        dict.update(self, **result)
 
 
-class ContentItemRepository(ResourceP):
+class ContentItemRepository(ResourceP, Protocol):
     """
     Content items GitHub repository information.
 
@@ -217,9 +220,13 @@ def oauth(self) -> ContentItemOAuth:
     @property
     def repository(self) -> ContentItemRepository | None:
         try:
+            path = f"v1/content/{self['guid']}/repository"
+            response = self._ctx.client.get(path)
+            result = response.json()
             return _ContentItemRepository(
                 self._ctx,
-                f"v1/content/{self['guid']}/repository",
+                path,
+                **result,
             )
         except ClientError:
             return None

src/posit/connect/resources.py

@@ -7,7 +7,6 @@
     TYPE_CHECKING,
     Any,
     Iterable,
-    Protocol,
     Sequence,
 )