devhelm 1.3.0__tar.gz → 1.5.0__tar.gz

{devhelm-1.3.0 → devhelm-1.5.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: devhelm
-Version: 1.3.0
+Version: 1.5.0
 Summary: DevHelm SDK for Python — typed client for monitors, incidents, alerting, and more
 Project-URL: Homepage, https://github.com/devhelmhq/sdk-python
 Project-URL: Repository, https://github.com/devhelmhq/sdk-python.git

{devhelm-1.3.0 → devhelm-1.5.0}/docs/openapi/monitoring-api.json

@@ -2274,6 +2274,18 @@
         ],
         "summary": "List categories with service counts",
         "operationId": "listCategories",
+        "parameters": [
+          {
+            "name": "publishedOnly",
+            "in": "query",
+            "description": "Count only published services (curated public pSEO set); default false",
+            "required": false,
+            "schema": {
+              "type": "boolean",
+              "default": false
+            }
+          }
+        ],
         "responses": {
           "200": {
             "description": "OK",
@@ -13529,6 +13541,15 @@
               "type": "string"
             }
           },
+          {
+            "name": "sort",
+            "in": "query",
+            "description": "Result ordering: 'recent' (default, newest first) or 'curated' (curated/recognizable first)",
+            "required": false,
+            "schema": {
+              "type": "string"
+            }
+          },
           {
             "name": "cursor",
             "in": "query",
@@ -13670,6 +13691,16 @@
               "type": "boolean",
               "default": false
             }
+          },
+          {
+            "name": "publishedOnly",
+            "in": "query",
+            "description": "Resolve only published services (curated public pSEO set); 404 otherwise. Default false",
+            "required": false,
+            "schema": {
+              "type": "boolean",
+              "default": false
+            }
           }
         ],
         "responses": {
@@ -15367,6 +15398,18 @@
         "summary": "Global status summary across all services",
         "description": "Returns aggregate counts of services by status and a list of services currently experiencing issues.",
         "operationId": "getGlobalStatusSummary",
+        "parameters": [
+          {
+            "name": "publishedOnly",
+            "in": "query",
+            "description": "Aggregate only published services (curated public pSEO set); default false",
+            "required": false,
+            "schema": {
+              "type": "boolean",
+              "default": false
+            }
+          }
+        ],
         "responses": {
           "200": {
             "description": "OK",

{devhelm-1.3.0 → devhelm-1.5.0}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "devhelm"
-version = "1.3.0"
+version = "1.5.0"
 description = "DevHelm SDK for Python — typed client for monitors, incidents, alerting, and more"
 authors = [{ name = "DevHelm", email = "hello@devhelm.io" }]
 license = "MIT"

{devhelm-1.3.0 → devhelm-1.5.0}/scripts/inject_strict_config.py

@@ -50,7 +50,24 @@ from pathlib import Path
 # `root-model-extra`), so skip them. Their behavior is governed by the
 # inner type, which on its own enforces strict validation.
 CLASS_RE = re.compile(r"^class\s+([A-Za-z_][\w]*)\s*\(\s*(BaseModel)\s*\)\s*:\s*$")
-CONFIG_LINE = "    model_config = ConfigDict(extra='forbid', populate_by_name=True)"
+CONFIG_LINE_STRICT = "    model_config = ConfigDict(extra='forbid', populate_by_name=True)"
+CONFIG_LINE_TOLERANT = "    model_config = ConfigDict(extra='ignore', populate_by_name=True)"
+
+
+def _is_response_shape(class_name: str) -> bool:
+    """Response-shape classes tolerate unknown fields (Postel's Law)."""
+    if class_name[0].islower():
+        return False
+    if class_name.endswith(("Request", "Params")):
+        return False
+    return bool(
+        class_name.endswith(("Dto", "Response"))
+        or class_name.startswith(("SingleValueResponse", "TableValueResult", "CursorPage"))
+    )
+
+
+# Keep the old name for backward compat in case anything imports it
+CONFIG_LINE = CONFIG_LINE_STRICT
 
 # Doc-banner injections keyed by class name. Inserted as a leading docstring
 # inside the target class so the note shows up in IDE hovers and stays put
@@ -130,6 +147,7 @@ def inject(source: str) -> tuple[str, int]:
             i += 1
             continue
         class_name = m.group(1)
+        config_line = CONFIG_LINE_TOLERANT if _is_response_shape(class_name) else CONFIG_LINE_STRICT
         # Look at the very next line. If it's already model_config or pass,
         # leave the class alone (idempotency / empty class).
         next_idx = i + 1
@@ -141,10 +159,10 @@ def inject(source: str) -> tuple[str, int]:
             out.append(f'    """{banner}"""\n')
             modified += 1
         if "model_config" in next_line:
-            # Upgrade the existing config line to include populate_by_name=True
-            # if it isn't already there. Idempotent across re-runs.
-            if "populate_by_name" not in next_line:
-                out.append(CONFIG_LINE + "\n")
+            # Replace the existing config line if it doesn't match the
+            # desired strictness or is missing populate_by_name.
+            if next_line.strip() != config_line.strip():
+                out.append(config_line + "\n")
                 i += 2  # replace the existing model_config line
                 modified += 1
                 continue
@@ -154,11 +172,11 @@ def inject(source: str) -> tuple[str, int]:
         # exact match (NOT startswith) — fields like `passed: Annotated[...]`
         # also start with "pass" but are not empty class markers.
         if next_line.strip() in ("pass", "pass\n"):
-            out.append(CONFIG_LINE + "\n")
+            out.append(config_line + "\n")
             i += 2  # skip the pass
             modified += 1
             continue
-        out.append(CONFIG_LINE + "\n")
+        out.append(config_line + "\n")
         modified += 1
         i += 1
     return "".join(out), modified

{devhelm-1.3.0 → devhelm-1.5.0}/src/devhelm/_errors.py

@@ -82,6 +82,11 @@ class DevhelmApiError(DevhelmError):
     The optional `request_id` field is the per-request id emitted by the
     API as the `X-Request-Id` response header and embedded in the JSON
     error body. Always include it in support tickets.
+
+    The optional `retry_after` field is the parsed value of the
+    `Retry-After` response header in whole seconds. It's populated on
+    rate-limit (429) responses that include the header so callers can back
+    off for exactly as long as the server asked; ``None`` otherwise.
     """
 
     status: int
@@ -93,6 +98,7 @@ class DevhelmApiError(DevhelmError):
     # narrowing. (Subclasses still inherit the same `str` type.)
     code: str
     request_id: str | None
+    retry_after: int | None
 
     def __init__(
         self,
@@ -103,6 +109,7 @@ class DevhelmApiError(DevhelmError):
         body: dict[str, Any] | str | None = None,
         code: str | None = None,
         request_id: str | None = None,
+        retry_after: int | None = None,
     ) -> None:
         super().__init__(message)
         self.status = status
@@ -113,6 +120,9 @@ class DevhelmApiError(DevhelmError):
         # `err.code` is never ``None`` for callers switching on category.
         self.code = code or "API_ERROR"
         self.request_id = request_id
+        # Parsed from the `Retry-After` response header (seconds). Populated
+        # on 429 / 503 responses that include it; ``None`` otherwise.
+        self.retry_after = retry_after
 
 
 class DevhelmAuthError(DevhelmApiError):
@@ -152,8 +162,28 @@ class DevhelmTransportError(DevhelmError):
             self.__cause__ = cause
 
 
+def _parse_retry_after(value: str | None) -> int | None:
+    """Parse a ``Retry-After`` header value into whole seconds.
+
+    The API emits ``Retry-After`` as an integer number of seconds. We parse
+    defensively: any non-integer value (an HTTP-date form, or garbage from a
+    misbehaving proxy) yields ``None`` rather than raising, so a malformed
+    header can never break error construction.
+    """
+    if value is None:
+        return None
+    try:
+        return int(value)
+    except (TypeError, ValueError):
+        return None
+
+
 def error_from_response(
-    status: int, body: str, *, request_id: str | None = None
+    status: int,
+    body: str,
+    *,
+    request_id: str | None = None,
+    retry_after: str | None = None,
 ) -> DevhelmApiError:
     """Map an HTTP error response to a typed DevhelmApiError subclass.
 
@@ -161,6 +191,11 @@ def error_from_response(
     pulled out at the call site (rather than re-parsed from the body) so the
     SDK still surfaces the id even when the server returns a non-JSON body
     (e.g. an HTML error page from a misconfigured proxy).
+
+    `retry_after` is the raw value of the `Retry-After` response header,
+    pulled out at the call site for the same reason. It's parsed into whole
+    seconds and surfaced as ``err.retry_after`` (e.g. on 429 responses) so
+    callers can back off for exactly as long as the server asked.
     """
     message = f"HTTP {status}"
     detail: str | None = None
@@ -195,6 +230,7 @@ def error_from_response(
         "body": parsed_body,
         "code": code,
         "request_id": resolved_request_id,
+        "retry_after": _parse_retry_after(retry_after),
     }
 
     if status in (401, 403):