fix: make failed-post retry work — enum status mismatch (zernio-claude-plugin#1) (#33)

The MCP retry path never recognized a failed post. Two bugs conspired:

1. posts_retry compared the post's status — a generated plain Enum with an
   unstable positional name (Status10 in the released build, renumbered on
   every spec regen) — against the handwritten PostStatus str-enum. Two
   different enum classes never compare equal, so every failed post was
   rejected with 'is not in failed status (current: Status10.FAILED)'.

2. posts_retry_all_failed / posts_list_failed passed PostStatus.FAILED into
   the query string, where httpx serialized it via str() as
   'status=PostStatus.FAILED' instead of 'status=failed', so the API
   matched nothing and the tools reported no failed posts.

Fixes:
- codegen: add --use-subclass-enum so generated enums are (str, Enum) and
  compare equal to their plain value and to the PostStatus twin
  (models regenerated).
- resources: unwrap Enum members to .value in _build_params/_build_payload —
  in the generator template (each generated resource carries its own copy
  that shadows BaseResource) and in the handwritten BaseResource
  (resources regenerated).
- mcp: posts_retry guard compares plain status values and reports the clean
  value ('current: scheduled') in the warning.
- tests: unit tests pin enum unwrapping and value-compatibility;
  test_mcp_retry_regression.py drives the real MCP tools against a strict
  in-memory fake API (httpx.MockTransport, no network) that only matches
  the literal status=failed — both bugs reproduce exactly against the
  unfixed code with the same error messages as the issue.

Co-authored-by: Claude Fable 5 <noreply@anthropic.com>

Author: Ooscaar

scripts/generate_models.py

@@ -56,6 +56,7 @@ def main() -> int:
         "--field-constraints",
         "--use-field-description",
         "--capitalise-enum-members",
+        "--use-subclass-enum",
         "--use-default-kwarg",
         "--collapse-root-models",
         "--use-union-operator",

scripts/generate_resources.py

@@ -394,20 +394,39 @@ def generate_resource_class(
         "        default for optional string args, and the API rejects empty query",
         "        values (e.g. ``platform=``) with a 400. Filtering here keeps both direct",
         "        SDK callers and MCP tool callers safe.",
+        "",
+        "        Enum members are unwrapped to their value: httpx serializes params",
+        '        via str(), which yields "ClassName.MEMBER" for Enum members, so',
+        "        e.g. ``status=PostStatus.FAILED`` would otherwise reach the API as",
+        "        ``status=PostStatus.FAILED`` instead of ``status=failed``.",
         '        """',
+        "        from enum import Enum",
         "        def to_camel(s: str) -> str:",
         '            parts = s.split("_")',
         '            return parts[0] + "".join(p.title() for p in parts[1:])',
-        '        return {to_camel(k): v for k, v in kwargs.items() if v is not None and v != ""}',
+        "        result: dict[str, Any] = {}",
+        "        for k, v in kwargs.items():",
+        "            if isinstance(v, Enum):",
+        "                v = v.value",
+        '            if v is None or v == "":',
+        "                continue",
+        "            result[to_camel(k)] = v",
+        "        return result",
         "",
         "    def _build_payload(self, **kwargs: Any) -> dict[str, Any]:",
-        '        """Build request payload, filtering None values."""',
+        '        """Build request payload, filtering None values. Enum members are',
+        '        unwrapped to their value so JSON bodies carry e.g. "failed" rather',
+        "        than a raw Enum member (plain Enum members are not JSON-serializable).",
+        '        """',
         "        from datetime import datetime",
+        "        from enum import Enum",
         "        def to_camel(s: str) -> str:",
         '            parts = s.split("_")',
         '            return parts[0] + "".join(p.title() for p in parts[1:])',
         "        result: dict[str, Any] = {}",
         "        for k, v in kwargs.items():",
+        "            if isinstance(v, Enum):",
+        "                v = v.value",
         "            if v is None:",
         "                continue",
         "            if isinstance(v, datetime):",

src/late/client/late_client.py

@@ -17,16 +17,16 @@
     AdCampaignsResource,
     AdCreativesResource,
     AdInsightsResource,
-    AdTargetingResource,
     AdsResource,
+    AdTargetingResource,
     AnalyticsResource,
     ApiKeysResource,
     BroadcastsResource,
     CallsResource,
     CommentAutomationsResource,
     CommentsResource,
-    ConnectResource,
     ConnectedAppsResource,
+    ConnectResource,
     ContactsResource,
     ConversionsResource,
     CustomFieldsResource,

src/late/mcp/server.py

@@ -756,8 +756,9 @@ def posts_retry(post_id: str) -> str:
         post = post_response.post
         if not post:
             return f"\u274c Post {post_id} not found"
-        if post.status != PostStatus.FAILED:
-            return f"\u26a0\ufe0f Post {post_id} is not in failed status (current: {post.status})"
+        status_value = post.status.value if post.status else "unknown"
+        if status_value != PostStatus.FAILED.value:
+            return f"\u26a0\ufe0f Post {post_id} is not in failed status (current: {status_value})"
     except Exception as e:
         return f"\u274c Could not find post {post_id}: {e}"