github-agent 0.30.0__tar.gz → 0.33.0__tar.gz

{github_agent-0.30.0 → github_agent-0.33.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: github-agent
-Version: 0.30.0
+Version: 0.33.0
 Summary: GitHub Agent for MCP
 Author-email: Audel Rouhi <knucklessg1@gmail.com>
 License: MIT
@@ -12,7 +12,7 @@ Classifier: Programming Language :: Python :: 3
 Requires-Python: <3.15,>=3.11
 Description-Content-Type: text/markdown
 License-File: LICENSE
-Requires-Dist: agent-utilities[agent,logfire]>=0.48.0
+Requires-Dist: agent-utilities[agent,logfire]>=0.51.0
 Provides-Extra: test
 Requires-Dist: pytest-xdist>=3.6.0; extra == "test"
 Requires-Dist: pytest; extra == "test"
@@ -41,7 +41,7 @@ Dynamic: license-file
 ![PyPI - Wheel](https://img.shields.io/pypi/wheel/github-agent)
 ![PyPI - Implementation](https://img.shields.io/pypi/implementation/github-agent)
 
-*Version: 0.30.0*
+*Version: 0.33.0*
 
 > **Documentation** — Installation, deployment, and usage across the MCP, API, and
 > CLI interfaces, including the integrated A2A agent server, are maintained in the

{github_agent-0.30.0 → github_agent-0.33.0}/README.md

@@ -20,7 +20,7 @@
 ![PyPI - Wheel](https://img.shields.io/pypi/wheel/github-agent)
 ![PyPI - Implementation](https://img.shields.io/pypi/implementation/github-agent)
 
-*Version: 0.30.0*
+*Version: 0.33.0*
 
 > **Documentation** — Installation, deployment, and usage across the MCP, API, and
 > CLI interfaces, including the integrated A2A agent server, are maintained in the

{github_agent-0.30.0 → github_agent-0.33.0}/github_agent/agent_server.py

@@ -4,7 +4,7 @@ import os
 import sys
 import warnings
 
-__version__ = "0.30.0"
+__version__ = "0.33.0"
 
 logging.basicConfig(
     level=logging.INFO,

{github_agent-0.30.0 → github_agent-0.33.0}/github_agent/api/api_client_base.py

@@ -1,6 +1,8 @@
 #!/usr/bin/env python
 import logging
+import os
 import re
+from collections.abc import Mapping
 from concurrent.futures import ThreadPoolExecutor, as_completed
 from typing import TypeVar
 
@@ -18,6 +20,52 @@ logger = get_logger(__name__)
 T = TypeVar("T", bound=BaseModel)
 
 
+def _default_timeout() -> tuple[float, float]:
+    """(connect, read) timeout for every outbound request, env-overridable.
+
+    A bounded timeout stops a single slow GitHub response from hanging an async
+    handler up to the gateway's 300s and wedging the whole server for every
+    concurrent caller (the failure mode behind stuck in-flight calls and child
+    restarts). Override with GITHUB_HTTP_CONNECT_TIMEOUT / GITHUB_HTTP_READ_TIMEOUT.
+    """
+    connect = float(os.getenv("GITHUB_HTTP_CONNECT_TIMEOUT", "10"))
+    read = float(os.getenv("GITHUB_HTTP_READ_TIMEOUT", "30"))
+    return (connect, read)
+
+
+class _TimeoutAdapter(requests.adapters.HTTPAdapter):
+    """HTTPAdapter that applies a default timeout when a request sets none.
+
+    Mounted on the session rather than subclassing Session, so test suites that
+    replace ``requests.Session`` with a mock are unaffected (``mount`` is a no-op
+    on a mock) while real requests still get a bounded timeout.
+    """
+
+    def __init__(self, timeout: tuple[float, float], *args, **kwargs):
+        self._timeout = timeout
+        super().__init__(*args, **kwargs)
+
+    def send(
+        self,
+        request: requests.PreparedRequest,
+        stream: bool = False,
+        timeout: None | float | tuple[float, float] | tuple[float, None] = None,
+        verify: bool | str = True,
+        cert: None | bytes | str | tuple[bytes | str, bytes | str] = None,
+        proxies: Mapping[str, str] | None = None,
+    ) -> requests.Response:
+        if timeout is None:
+            timeout = self._timeout
+        return super().send(
+            request,
+            stream=stream,
+            timeout=timeout,
+            verify=verify,
+            cert=cert,
+            proxies=proxies,
+        )
+
+
 class BaseApiClient:
     def __init__(
         self,
@@ -36,6 +84,9 @@ class BaseApiClient:
             raise MissingParameterError
 
         self._session = requests.Session()
+        _adapter = _TimeoutAdapter(_default_timeout())
+        self._session.mount("https://", _adapter)
+        self._session.mount("http://", _adapter)
         self.url = url.rstrip("/")
         self.headers = {
             "Accept": "application/vnd.github+json",
@@ -119,10 +170,16 @@ class BaseApiClient:
 
         total_pages = self._get_total_pages(response)
 
-        max_pages = getattr(model, "max_pages", total_pages)
-        if not max_pages or max_pages == 0 or max_pages > total_pages:
+        # Bounded by default: an unset max_pages fetches only the first page so a
+        # single list call can't pull an entire org/run-history (the cause of
+        # multi-hundred-KB responses and gateway timeouts). Callers opt into more
+        # by passing max_pages (use max_pages<=0 to mean "all pages").
+        max_pages = getattr(model, "max_pages", None)
+        if max_pages is None:
+            max_pages = 1
+        elif max_pages <= 0 or max_pages > total_pages:
             max_pages = total_pages
-            model.max_pages = total_pages  # type: ignore[attr-defined]
+        model.max_pages = max_pages  # type: ignore[attr-defined]
 
         if max_pages > 1:
             with ThreadPoolExecutor(max_workers=5) as executor:

{github_agent-0.30.0 → github_agent-0.33.0}/github_agent/github_input_models.py

@@ -14,10 +14,15 @@ class BaseModelWrapper(BaseModel):
     """
 
     max_pages: int | None = Field(
-        description="Max amount of pages to retrieve", default=None
+        description=(
+            "Max number of pages to retrieve. Unset (default) fetches only the "
+            "first page to keep responses small; pass a number for more, or "
+            "max_pages<=0 to fetch all pages."
+        ),
+        default=None,
     )
     page: int | None = Field(description="Pagination page", default=1)
-    per_page: int | None = Field(description="Results per page", default=100)
+    per_page: int | None = Field(description="Results per page (1-100)", default=30)
     api_parameters: dict[str, Any] = Field(
         description="API Parameters", default_factory=dict
     )

{github_agent-0.30.0 → github_agent-0.33.0}/github_agent/mcp/mcp_action.py

@@ -3,12 +3,44 @@
 Auto-generated from mcp_server.py during ecosystem standardization.
 """
 
+from agent_utilities.mcp_utilities import resolve_action, run_blocking
 from fastmcp import Context, FastMCP
 from fastmcp.dependencies import Depends
 from pydantic import Field
 
 from github_agent.auth import get_client
 
+#: Exact keys dropped by _slim (pure hypermedia/noise, never semantic data).
+_SLIM_DROP_EXACT = {"_links", "url", "node_id"}
+
+#: Valid workflow actions for the shared ``resolve_action`` discovery helper.
+WORKFLOW_ACTIONS = (
+    "list_workflows",
+    "list_runs",
+    "get_run",
+    "trigger_dispatch",
+    "rerun",
+    "cancel",
+    "delete_run",
+)
+
+
+def _slim(obj):
+    """Recursively drop hypermedia ``*_url`` hrefs and ``_links`` noise.
+
+    Mirror of ``mcp_server._slim``; see that module for rationale.
+    """
+    if isinstance(obj, list):
+        return [_slim(item) for item in obj]
+    if isinstance(obj, dict):
+        return {
+            k: _slim(v)
+            for k, v in obj.items()
+            if k not in _SLIM_DROP_EXACT
+            and not (k.endswith("_url") and k != "html_url")
+        }
+    return obj
+
 
 def register_action_tools(mcp: FastMCP):
     @mcp.tool(tags={"actions"})
@@ -24,7 +56,15 @@ def register_action_tools(mcp: FastMCP):
             default=None, description="MCP context for progress reporting"
         ),
     ) -> dict:
-        """Manage GitHub actions workflows and runs."""
+        """Manage GitHub actions workflows and runs.
+
+        list_runs params (via params_json): owner, repo, and optional filters
+        applied server-side to keep results small — status (e.g. failure,
+        success, completed, in_progress), branch, per_page (1-100, default 30),
+        max_pages (default 1 page; max_pages<=0 = all pages), slim (default
+        true: drops hypermedia *_url/_links noise, keeps html_url and all data).
+        Prefer status=failure + branch=<default> for a CI-health sweep.
+        """
         if ctx:
             await ctx.info("Executing github_actions action...")
         import json
@@ -35,6 +75,12 @@ def register_action_tools(mcp: FastMCP):
             return {"status": 400, "error": f"Invalid params_json: {e}", "data": None}
 
         kwargs = {k: v for k, v in kwargs.items() if v is not None}
+        slim = kwargs.pop("slim", True)
+
+        resolved = resolve_action(action, WORKFLOW_ACTIONS, service="github-agent")
+        if isinstance(resolved, dict):
+            return resolved
+        action = resolved
 
         try:
             if action == "list_workflows":
@@ -46,18 +92,21 @@ def register_action_tools(mcp: FastMCP):
                         "error": "Missing required 'owner' or 'repo' parameter",
                         "data": None,
                     }
-                response = client.get_workflows(owner=owner, repo=repo)
+                response = await run_blocking(
+                    client.get_workflows, owner=owner, repo=repo
+                )
                 return {
                     "status": 200,
                     "message": "Workflows retrieved successfully",
                     "data": [w.model_dump() for w in response.data],
                 }
             elif action == "list_runs":
-                response = client.get_workflow_runs(**kwargs)
+                response = await run_blocking(client.get_workflow_runs, **kwargs)
+                data = [r.model_dump() for r in response.data]
                 return {
                     "status": 200,
                     "message": "Workflow runs retrieved successfully",
-                    "data": [r.model_dump() for r in response.data],
+                    "data": _slim(data) if slim else data,
                 }
             elif action == "get_run":
                 owner = kwargs.get("owner")
@@ -69,8 +118,8 @@ def register_action_tools(mcp: FastMCP):
                         "error": "Missing required 'owner', 'repo', or 'run_id' parameter",
                         "data": None,
                     }
-                response = client.get_workflow_run(
-                    owner=owner, repo=repo, run_id=int(run_id)
+                response = await run_blocking(
+                    client.get_workflow_run, owner=owner, repo=repo, run_id=int(run_id)
                 )
                 return {
                     "status": 200,
@@ -89,7 +138,8 @@ def register_action_tools(mcp: FastMCP):
                         "error": "Missing 'owner', 'repo', 'workflow_id', or 'ref' parameter",
                         "data": None,
                     }
-                response = client.trigger_workflow_dispatch(
+                response = await run_blocking(
+                    client.trigger_workflow_dispatch,
                     owner=owner,
                     repo=repo,
                     workflow_id=workflow_id,
@@ -111,8 +161,11 @@ def register_action_tools(mcp: FastMCP):
                         "error": "Missing 'owner', 'repo', or 'run_id' parameter",
                         "data": None,
                     }
-                response = client.rerun_workflow_run(
-                    owner=owner, repo=repo, run_id=int(run_id)
+                response = await run_blocking(
+                    client.rerun_workflow_run,
+                    owner=owner,
+                    repo=repo,
+                    run_id=int(run_id),
                 )
                 return {
                     "status": 200,
@@ -129,8 +182,11 @@ def register_action_tools(mcp: FastMCP):
                         "error": "Missing 'owner', 'repo', or 'run_id' parameter",
                         "data": None,
                     }
-                response = client.cancel_workflow_run(
-                    owner=owner, repo=repo, run_id=int(run_id)
+                response = await run_blocking(
+                    client.cancel_workflow_run,
+                    owner=owner,
+                    repo=repo,
+                    run_id=int(run_id),
                 )
                 return {
                     "status": 200,
@@ -147,8 +203,11 @@ def register_action_tools(mcp: FastMCP):
                         "error": "Missing 'owner', 'repo', or 'run_id' parameter",
                         "data": None,
                     }
-                response = client.delete_workflow_run(
-                    owner=owner, repo=repo, run_id=int(run_id)
+                response = await run_blocking(
+                    client.delete_workflow_run,
+                    owner=owner,
+                    repo=repo,
+                    run_id=int(run_id),
                 )
                 return {
                     "status": 200,

{github_agent-0.30.0 → github_agent-0.33.0}/github_agent/mcp/mcp_branch.py

@@ -3,12 +3,24 @@
 Auto-generated from mcp_server.py during ecosystem standardization.
 """
 
+from agent_utilities.mcp_utilities import resolve_action, run_blocking
 from fastmcp import Context, FastMCP
 from fastmcp.dependencies import Depends
 from pydantic import Field
 
 from github_agent.auth import get_client
 
+#: Valid branch actions for the shared ``resolve_action`` discovery helper.
+BRANCH_ACTIONS = (
+    "list",
+    "get",
+    "create",
+    "delete",
+    "get_protection",
+    "update_protection",
+    "delete_protection",
+)
+
 
 def register_branch_tools(mcp: FastMCP):
     @mcp.tool(tags={"branches"})
@@ -36,9 +48,14 @@ def register_branch_tools(mcp: FastMCP):
 
         kwargs = {k: v for k, v in kwargs.items() if v is not None}
 
+        resolved = resolve_action(action, BRANCH_ACTIONS, service="github-agent")
+        if isinstance(resolved, dict):
+            return resolved
+        action = resolved
+
         try:
             if action == "list":
-                response = client.get_branches(**kwargs)
+                response = await run_blocking(client.get_branches, **kwargs)
                 return {
                     "status": 200,
                     "message": "Branches retrieved successfully",
@@ -54,7 +71,9 @@ def register_branch_tools(mcp: FastMCP):
                         "error": "Missing 'owner', 'repo', or 'branch' parameter",
                         "data": None,
                     }
-                response = client.get_branch(owner=owner, repo=repo, branch=branch)
+                response = await run_blocking(
+                    client.get_branch, owner=owner, repo=repo, branch=branch
+                )
                 return {
                     "status": 200,
                     "message": "Branch retrieved successfully",
@@ -71,8 +90,8 @@ def register_branch_tools(mcp: FastMCP):
                         "error": "Missing 'owner', 'repo', 'branch', or 'ref' parameter",
                         "data": None,
                     }
-                response = client.create_branch(
-                    owner=owner, repo=repo, branch=branch, ref=ref
+                response = await run_blocking(
+                    client.create_branch, owner=owner, repo=repo, branch=branch, ref=ref
                 )
                 return {
                     "status": 201,
@@ -89,7 +108,9 @@ def register_branch_tools(mcp: FastMCP):
                         "error": "Missing 'owner', 'repo', or 'branch' parameter",
                         "data": None,
                     }
-                response = client.delete_branch(owner=owner, repo=repo, branch=branch)
+                response = await run_blocking(
+                    client.delete_branch, owner=owner, repo=repo, branch=branch
+                )
                 return {
                     "status": 200,
                     "message": "Branch deleted successfully",

{github_agent-0.30.0 → github_agent-0.33.0}/github_agent/mcp/mcp_collaborator.py

@@ -3,12 +3,16 @@
 Auto-generated from mcp_server.py during ecosystem standardization.
 """
 
+from agent_utilities.mcp_utilities import resolve_action, run_blocking
 from fastmcp import Context, FastMCP
 from fastmcp.dependencies import Depends
 from pydantic import Field
 
 from github_agent.auth import get_client
 
+#: Valid collaborator actions for the shared ``resolve_action`` discovery helper.
+COLLABORATOR_ACTIONS = ("list", "add", "remove")
+
 
 def register_collaborator_tools(mcp: FastMCP):
     @mcp.tool(tags={"collaborators"})
@@ -36,9 +40,14 @@ def register_collaborator_tools(mcp: FastMCP):
 
         kwargs = {k: v for k, v in kwargs.items() if v is not None}
 
+        resolved = resolve_action(action, COLLABORATOR_ACTIONS, service="github-agent")
+        if isinstance(resolved, dict):
+            return resolved
+        action = resolved
+
         try:
             if action == "list":
-                response = client.get_collaborators(**kwargs)
+                response = await run_blocking(client.get_collaborators, **kwargs)
                 return {
                     "status": 200,
                     "message": "Collaborators retrieved successfully",
@@ -55,8 +64,12 @@ def register_collaborator_tools(mcp: FastMCP):
                         "error": "Missing 'owner', 'repo', or 'username' parameter",
                         "data": None,
                     }
-                response = client.add_collaborator(
-                    owner=owner, repo=repo, username=username, permission=permission
+                response = await run_blocking(
+                    client.add_collaborator,
+                    owner=owner,
+                    repo=repo,
+                    username=username,
+                    permission=permission,
                 )
                 return {
                     "status": 200,
@@ -73,8 +86,11 @@ def register_collaborator_tools(mcp: FastMCP):
                         "error": "Missing 'owner', 'repo', or 'username' parameter",
                         "data": None,
                     }
-                response = client.remove_collaborator(
-                    owner=owner, repo=repo, username=username
+                response = await run_blocking(
+                    client.remove_collaborator,
+                    owner=owner,
+                    repo=repo,
+                    username=username,
                 )
                 return {
                     "status": 200,

{github_agent-0.30.0 → github_agent-0.33.0}/github_agent/mcp/mcp_commit.py

@@ -3,12 +3,16 @@
 Auto-generated from mcp_server.py during ecosystem standardization.
 """
 
+from agent_utilities.mcp_utilities import resolve_action, run_blocking
 from fastmcp import Context, FastMCP
 from fastmcp.dependencies import Depends
 from pydantic import Field
 
 from github_agent.auth import get_client
 
+#: Valid commit actions for the shared ``resolve_action`` discovery helper.
+COMMIT_ACTIONS = ("list", "get")
+
 
 def register_commit_tools(mcp: FastMCP):
     @mcp.tool(tags={"commits"})
@@ -36,9 +40,14 @@ def register_commit_tools(mcp: FastMCP):
 
         kwargs = {k: v for k, v in kwargs.items() if v is not None}
 
+        resolved = resolve_action(action, COMMIT_ACTIONS, service="github-agent")
+        if isinstance(resolved, dict):
+            return resolved
+        action = resolved
+
         try:
             if action == "list":
-                response = client.get_commits(**kwargs)
+                response = await run_blocking(client.get_commits, **kwargs)
                 return {
                     "status": 200,
                     "message": "Commits retrieved successfully",
@@ -54,7 +63,9 @@ def register_commit_tools(mcp: FastMCP):
                         "error": "Missing 'owner', 'repo', or 'sha' parameter",
                         "data": None,
                     }
-                response = client.get_commit(owner=owner, repo=repo, sha=sha)
+                response = await run_blocking(
+                    client.get_commit, owner=owner, repo=repo, sha=sha
+                )
                 return {
                     "status": 200,
                     "message": "Commit retrieved successfully",

{github_agent-0.30.0 → github_agent-0.33.0}/github_agent/mcp/mcp_content.py

@@ -3,12 +3,16 @@
 Auto-generated from mcp_server.py during ecosystem standardization.
 """
 
+from agent_utilities.mcp_utilities import resolve_action, run_blocking
 from fastmcp import Context, FastMCP
 from fastmcp.dependencies import Depends
 from pydantic import Field
 
 from github_agent.auth import get_client
 
+#: Valid content actions for the shared ``resolve_action`` discovery helper.
+CONTENT_ACTIONS = ("get", "create", "update", "delete")
+
 
 def register_content_tools(mcp: FastMCP):
     @mcp.tool(tags={"contents"})
@@ -36,9 +40,14 @@ def register_content_tools(mcp: FastMCP):
 
         kwargs = {k: v for k, v in kwargs.items() if v is not None}
 
+        resolved = resolve_action(action, CONTENT_ACTIONS, service="github-agent")
+        if isinstance(resolved, dict):
+            return resolved
+        action = resolved
+
         try:
             if action == "get":
-                response = client.get_contents(**kwargs)
+                response = await run_blocking(client.get_contents, **kwargs)
                 if isinstance(response.data, list):
                     data = [item.model_dump() for item in response.data]
                 else:
@@ -60,7 +69,8 @@ def register_content_tools(mcp: FastMCP):
                         "error": "Missing 'owner', 'repo', 'path', 'message', or 'content' parameter",
                         "data": None,
                     }
-                response = client.create_content(
+                response = await run_blocking(
+                    client.create_content,
                     owner=owner,
                     repo=repo,
                     path=path,
@@ -93,7 +103,8 @@ def register_content_tools(mcp: FastMCP):
                         "error": "Missing 'owner', 'repo', 'path', 'message', 'content', or 'sha' parameter",
                         "data": None,
                     }
-                response = client.update_content(
+                response = await run_blocking(
+                    client.update_content,
                     owner=owner,
                     repo=repo,
                     path=path,
@@ -119,7 +130,8 @@ def register_content_tools(mcp: FastMCP):
                         "error": "Missing 'owner', 'repo', 'path', 'message', or 'sha' parameter",
                         "data": None,
                     }
-                response = client.delete_content(
+                response = await run_blocking(
+                    client.delete_content,
                     owner=owner,
                     repo=repo,
                     path=path,

{github_agent-0.30.0 → github_agent-0.33.0}/github_agent/mcp/mcp_issue.py

@@ -3,12 +3,16 @@
 Auto-generated from mcp_server.py during ecosystem standardization.
 """
 
+from agent_utilities.mcp_utilities import resolve_action, run_blocking
 from fastmcp import Context, FastMCP
 from fastmcp.dependencies import Depends
 from pydantic import Field
 
 from github_agent.auth import get_client
 
+#: Valid issue actions for the shared ``resolve_action`` discovery helper.
+ISSUE_ACTIONS = ("list", "get", "create", "update")
+
 
 def register_issue_tools(mcp: FastMCP):
     @mcp.tool(tags={"issues"})
@@ -24,7 +28,14 @@ def register_issue_tools(mcp: FastMCP):
             default=None, description="MCP context for progress reporting"
         ),
     ) -> dict:
-        """Manage GitHub issues."""
+        """Manage GitHub issues.
+
+        list params (via params_json): owner, repo, and optional filters applied
+        server-side — state (open/closed/all), labels, assignee, since, per_page
+        (1-100, default 30), max_pages (default 1 page; max_pages<=0 = all pages).
+        Note: the list/search APIs return PRs alongside issues — a returned item
+        with a 'pull_request' field is a PR, not an issue.
+        """
         if ctx:
             await ctx.info("Executing github_issues action...")
         import json
@@ -36,9 +47,14 @@ def register_issue_tools(mcp: FastMCP):
 
         kwargs = {k: v for k, v in kwargs.items() if v is not None}
 
+        resolved = resolve_action(action, ISSUE_ACTIONS, service="github-agent")
+        if isinstance(resolved, dict):
+            return resolved
+        action = resolved
+
         try:
             if action == "list":
-                response = client.get_issues(**kwargs)
+                response = await run_blocking(client.get_issues, **kwargs)
                 return {
                     "status": 200,
                     "message": "Issues retrieved successfully",
@@ -54,7 +70,9 @@ def register_issue_tools(mcp: FastMCP):
                         "error": "Missing 'owner', 'repo', or 'number' parameter",
                         "data": None,
                     }
-                response = client.get_issue(owner=owner, repo=repo, number=int(number))
+                response = await run_blocking(
+                    client.get_issue, owner=owner, repo=repo, number=int(number)
+                )
                 return {
                     "status": 200,
                     "message": "Issue retrieved successfully",
@@ -70,8 +88,8 @@ def register_issue_tools(mcp: FastMCP):
                         "error": "Missing 'owner', 'repo', or 'title' parameter",
                         "data": None,
                     }
-                response = client.create_issue(
-                    owner=owner, repo=repo, title=title, **kwargs
+                response = await run_blocking(
+                    client.create_issue, owner=owner, repo=repo, title=title, **kwargs
                 )
                 return {
                     "status": 201,
@@ -88,8 +106,12 @@ def register_issue_tools(mcp: FastMCP):
                         "error": "Missing 'owner', 'repo', or 'number' parameter",
                         "data": None,
                     }
-                response = client.update_issue(
-                    owner=owner, repo=repo, number=int(number), **kwargs
+                response = await run_blocking(
+                    client.update_issue,
+                    owner=owner,
+                    repo=repo,
+                    number=int(number),
+                    **kwargs,
                 )
                 return {
                     "status": 200,