mcp-github-crunchtools 1.3.0__tar.gz → 1.3.1__tar.gz

{mcp_github_crunchtools-1.3.0 → mcp_github_crunchtools-1.3.1}/CHANGELOG.md

@@ -6,6 +6,14 @@ All notable changes to this project are documented here. The format follows
 
 ## [Unreleased]
 
+## [1.3.1] - 2026-10-16
+
+### Fixed
+
+- A primary rate limit reported GitHub's `x-ratelimit-reset` epoch as the
+  wait ("Retry after 1792152901 seconds"). It now reports the seconds left
+  until the reset.
+
 ## [1.3.0] - 2026-10-14
 
 ### Added

{mcp_github_crunchtools-1.3.0 → mcp_github_crunchtools-1.3.1}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.5
 Name: mcp-github-crunchtools
-Version: 1.3.0
+Version: 1.3.1
 Summary: Secure MCP server for GitHub issues, pull requests, Actions, releases, files, and search
 Author: crunchtools.com
 License-Expression: AGPL-3.0-or-later

{mcp_github_crunchtools-1.3.0 → mcp_github_crunchtools-1.3.1}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "mcp-github-crunchtools"
-version = "1.3.0"
+version = "1.3.1"
 description = "Secure MCP server for GitHub issues, pull requests, Actions, releases, files, and search"
 requires-python = ">=3.10"
 license = "AGPL-3.0-or-later"

{mcp_github_crunchtools-1.3.0 → mcp_github_crunchtools-1.3.1}/src/mcp_github_crunchtools/__init__.py

@@ -29,7 +29,7 @@ import argparse
 
 from .server import mcp
 
-__version__ = "1.3.0"
+__version__ = "1.3.1"
 __all__ = ["main", "mcp"]
 
 

{mcp_github_crunchtools-1.3.0 → mcp_github_crunchtools-1.3.1}/src/mcp_github_crunchtools/client.py

@@ -7,6 +7,7 @@ All requests go through this client to ensure consistent security practices.
 import ipaddress
 import logging
 import re
+import time
 from typing import Any
 from urllib.parse import urlparse
 
@@ -25,6 +26,23 @@ from .errors import (
 
 logger = logging.getLogger(__name__)
 
+
+def _retry_after_seconds(headers: httpx.Headers) -> int | None:
+    """Seconds until a rate-limited call may be retried.
+
+    ``retry-after`` is already a duration; ``x-ratelimit-reset`` is the UTC
+    epoch second the window resets, so it is converted to a duration.
+    """
+    try:
+        if (retry_after := headers.get("retry-after")) is not None:
+            return max(0, int(retry_after))
+        if (reset := headers.get("x-ratelimit-reset")) is not None:
+            return max(0, int(reset) - int(time.time()))
+    except ValueError:
+        return None
+    return None
+
+
 MAX_RESPONSE_SIZE = 10 * 1024 * 1024
 # A job log is streamed and only its tail is kept, so it may be larger than a
 # buffered API response; past this the download is abandoned.
@@ -302,10 +320,7 @@ class GitHubClient:
         rate_remaining = response.headers.get("x-ratelimit-remaining")
 
         if status_code == 429 or (status_code == 403 and rate_remaining == "0"):
-            retry_after = response.headers.get("retry-after")
-            if retry_after is None:
-                retry_after = response.headers.get("x-ratelimit-reset")
-            raise RateLimitError(int(retry_after) if retry_after else None)
+            raise RateLimitError(_retry_after_seconds(response.headers))
 
         if status_code == 401:
             raise PermissionDeniedError("Valid token with required scopes")

{mcp_github_crunchtools-1.3.0 → mcp_github_crunchtools-1.3.1}/src/mcp_github_crunchtools/server.py

@@ -44,7 +44,7 @@ READ_ONLY = {"readOnlyHint": True}
 
 mcp = FastMCP(
     name="mcp-github",
-    version="1.3.0",
+    version="1.3.1",
     instructions=(
         "Secure MCP server for GitHub repositories: issues, pull requests "
         "(open, review discussion, diffs, CI checks, merge), Actions runs and "

{mcp_github_crunchtools-1.3.0 → mcp_github_crunchtools-1.3.1}/tests/test_tools.py

@@ -6,6 +6,7 @@ Integration tests with a real GitHub account should be run separately.
 
 import base64
 import os
+import time
 
 import httpx
 import pytest
@@ -1909,6 +1910,28 @@ class TestClientErrorHandling:
         with _patch_client(resp), pytest.raises(RateLimitError):
             await get_issue(owner="o", repo="r", issue_number=1)
 
+    @pytest.mark.asyncio
+    async def test_403_rate_limit_reset_is_a_duration(self) -> None:
+        """x-ratelimit-reset is an epoch second; the error reports the wait,
+        not the timestamp."""
+        from mcp_github_crunchtools.errors import RateLimitError
+        from mcp_github_crunchtools.tools import get_issue
+
+        resp = _mock_response(
+            status_code=403,
+            json_data={"message": "API rate limit exceeded"},
+            headers={
+                "x-ratelimit-remaining": "0",
+                "x-ratelimit-reset": str(int(time.time()) + 600),
+            },
+        )
+
+        with _patch_client(resp), pytest.raises(RateLimitError) as exc_info:
+            await get_issue(owner="o", repo="r", issue_number=1)
+
+        seconds = int(str(exc_info.value).split("Retry after ")[1].split()[0])
+        assert 590 <= seconds <= 600
+
     @pytest.mark.asyncio
     async def test_403_preserves_github_message_when_not_rate_limited(
         self,

{mcp_github_crunchtools-1.3.0 → mcp_github_crunchtools-1.3.1}/uv.lock

@@ -922,7 +922,7 @@ wheels = [
 
 [[package]]
 name = "mcp-github-crunchtools"
-version = "1.3.0"
+version = "1.3.1"
 source = { editable = "." }
 dependencies = [
     { name = "fastmcp" },