fix few api path

Author: iam404

src/pytfe/models/__init__.py

@@ -48,6 +48,7 @@
     TerraformVersion,
     TerraformVersionCreateOptions,
     TerraformVersionUpdateOptions,
+    ToolVersionArchitecture,
 )
 
 # ── TFE admin workspaces ──────────────────────────────────────────────────────
@@ -606,6 +607,7 @@
 # ── Public surface ────────────────────────────────────────────────────────────
 __all__ = [
     # TFE admin versions
+    "ToolVersionArchitecture",
     "TerraformVersion",
     "TerraformVersionCreateOptions",
     "TerraformVersionUpdateOptions",

src/pytfe/models/admin_run.py

@@ -20,7 +20,9 @@ class AdminRun(BaseModel):
     model_config = ConfigDict(populate_by_name=True, validate_by_name=True)
     id: str | None = None
     status: RunStatus | None = None
+    has_changes: bool | None = Field(default=None, alias="has-changes")
     plan_only: bool | None = Field(default=None, alias="plan-only")
-    organization_name: str | None = None
-    workspace_name: str | None = None
+    # Populated from the `workspace` relationship (always present).
     workspace_id: str | None = None
+    # Populated only when `?include=workspace.organization` is passed.
+    organization_name: str | None = None

src/pytfe/models/admin_version.py

@@ -6,6 +6,14 @@
 from pydantic import BaseModel, ConfigDict, Field
 
 
+class ToolVersionArchitecture(BaseModel):
+    model_config = ConfigDict(populate_by_name=True, validate_by_name=True)
+    url: str | None = None
+    sha: str | None = None
+    os: str | None = None
+    arch: str | None = None
+
+
 class TerraformVersion(BaseModel):
     model_config = ConfigDict(populate_by_name=True, validate_by_name=True)
     id: str | None = None
@@ -19,6 +27,7 @@ class TerraformVersion(BaseModel):
     deprecated_reason: str | None = Field(default=None, alias="deprecated-reason")
     usage: int | None = None
     created_at: str | None = Field(default=None, alias="created-at")
+    archs: list[ToolVersionArchitecture] | None = None
 
 
 class TerraformVersionCreateOptions(BaseModel):
@@ -31,6 +40,7 @@ class TerraformVersionCreateOptions(BaseModel):
     beta: bool | None = None
     deprecated: bool | None = None
     deprecated_reason: str | None = Field(default=None, alias="deprecated-reason")
+    archs: list[ToolVersionArchitecture] | None = None
 
 
 class TerraformVersionUpdateOptions(BaseModel):
@@ -43,6 +53,7 @@ class TerraformVersionUpdateOptions(BaseModel):
     beta: bool | None = None
     deprecated: bool | None = None
     deprecated_reason: str | None = Field(default=None, alias="deprecated-reason")
+    archs: list[ToolVersionArchitecture] | None = None
 
 
 class OpaVersion(BaseModel):
@@ -58,6 +69,7 @@ class OpaVersion(BaseModel):
     deprecated_reason: str | None = Field(default=None, alias="deprecated-reason")
     usage: int | None = None
     created_at: str | None = Field(default=None, alias="created-at")
+    archs: list[ToolVersionArchitecture] | None = None
 
 
 class OpaVersionCreateOptions(BaseModel):
@@ -70,6 +82,7 @@ class OpaVersionCreateOptions(BaseModel):
     beta: bool | None = None
     deprecated: bool | None = None
     deprecated_reason: str | None = Field(default=None, alias="deprecated-reason")
+    archs: list[ToolVersionArchitecture] | None = None
 
 
 class OpaVersionUpdateOptions(BaseModel):
@@ -82,6 +95,7 @@ class OpaVersionUpdateOptions(BaseModel):
     beta: bool | None = None
     deprecated: bool | None = None
     deprecated_reason: str | None = Field(default=None, alias="deprecated-reason")
+    archs: list[ToolVersionArchitecture] | None = None
 
 
 class SentinelVersion(BaseModel):
@@ -97,6 +111,7 @@ class SentinelVersion(BaseModel):
     deprecated_reason: str | None = Field(default=None, alias="deprecated-reason")
     usage: int | None = None
     created_at: str | None = Field(default=None, alias="created-at")
+    archs: list[ToolVersionArchitecture] | None = None
 
 
 class SentinelVersionCreateOptions(BaseModel):
@@ -109,6 +124,7 @@ class SentinelVersionCreateOptions(BaseModel):
     beta: bool | None = None
     deprecated: bool | None = None
     deprecated_reason: str | None = Field(default=None, alias="deprecated-reason")
+    archs: list[ToolVersionArchitecture] | None = None
 
 
 class SentinelVersionUpdateOptions(BaseModel):
@@ -121,3 +137,4 @@ class SentinelVersionUpdateOptions(BaseModel):
     beta: bool | None = None
     deprecated: bool | None = None
     deprecated_reason: str | None = Field(default=None, alias="deprecated-reason")
+    archs: list[ToolVersionArchitecture] | None = None

src/pytfe/models/admin_workspace.py

@@ -18,6 +18,9 @@ class AdminWorkspace(BaseModel):
     id: str | None = None
     name: str | None = None
     locked: bool | None = None
-    execution_mode: str | None = Field(default=None, alias="execution-mode")
+    # vcs-repo is an attribute object; only the identifier is surfaced here.
+    vcs_repo_identifier: str | None = None
+    # Lifted from the `organization` relationship at parse time.
     organization_name: str | None = None
+    # Lifted from the `current-run` relationship at parse time.
     current_run_id: str | None = None

src/pytfe/resources/admin/_runs.py

@@ -15,12 +15,13 @@
 def _parse_admin_run(data: dict[str, Any]) -> AdminRun:
     attrs = data.get("attributes") or {}
     rels = data.get("relationships") or {}
-    org_data = (rels.get("organization") or {}).get("data") or {}
     ws_data = (rels.get("workspace") or {}).get("data") or {}
+    # organization is a compound include (workspace.organization) — only
+    # present when the caller passes ?include=workspace.organization.
+    # We don't surface that parameter yet, so organization_name stays None.
     return AdminRun.model_validate(
         {
             "id": data.get("id"),
-            "organization_name": org_data.get("id"),
             "workspace_id": ws_data.get("id"),
             **attrs,
         }

src/pytfe/resources/admin/_users.py

@@ -60,21 +60,21 @@ def unsuspend(self, user_id: str) -> AdminUser:
     def grant_admin(self, user_id: str) -> AdminUser:
         if not valid_string_id(user_id):
             raise ValueError(ERR_INVALID_NAME)
-        r = self.t.request("POST", f"/api/v2/admin/users/{user_id}/actions/grant-admin")
+        r = self.t.request("POST", f"/api/v2/admin/users/{user_id}/actions/grant_admin")
         return _parse_admin_user(r.json()["data"])
 
     def revoke_admin(self, user_id: str) -> AdminUser:
         if not valid_string_id(user_id):
             raise ValueError(ERR_INVALID_NAME)
         r = self.t.request(
-            "POST", f"/api/v2/admin/users/{user_id}/actions/revoke-admin"
+            "POST", f"/api/v2/admin/users/{user_id}/actions/revoke_admin"
         )
         return _parse_admin_user(r.json()["data"])
 
     def disable_two_factor(self, user_id: str) -> AdminUser:
         if not valid_string_id(user_id):
             raise ValueError(ERR_INVALID_NAME)
         r = self.t.request(
-            "POST", f"/api/v2/admin/users/{user_id}/actions/disable-two-factor"
+            "POST", f"/api/v2/admin/users/{user_id}/actions/disable_two_factor"
         )
         return _parse_admin_user(r.json()["data"])

src/pytfe/resources/admin/_workspaces.py

@@ -17,11 +17,13 @@ def _parse_admin_workspace(data: dict[str, Any]) -> AdminWorkspace:
     rels = data.get("relationships") or {}
     org_data = (rels.get("organization") or {}).get("data") or {}
     run_data = (rels.get("current-run") or {}).get("data") or {}
+    vcs_repo = attrs.pop("vcs-repo", None) or {}
     return AdminWorkspace.model_validate(
         {
             "id": data.get("id"),
             "organization_name": org_data.get("id"),
             "current_run_id": run_data.get("id"),
+            "vcs_repo_identifier": vcs_repo.get("identifier") if vcs_repo else None,
             **attrs,
         }
     )