strands-coder 1.4.0__tar.gz → 1.4.1__tar.gz

{strands_coder-1.4.0 → strands_coder-1.4.1}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: strands-coder
-Version: 1.4.0
+Version: 1.4.1
 Summary: Add an AI assistant to your GitHub repositories that can review code, manage issues, and improve over time.
 Project-URL: Homepage, https://github.com/cagataycali/strands-coder
 Project-URL: Bug Tracker, https://github.com/cagataycali/strands-coder/issues
@@ -261,6 +261,8 @@ from strands import Agent
 from strands_coder import use_github, projects, scheduler, store_in_kb
 
 # Create agent with GitHub tools
+[![Awesome Strands Agents](https://img.shields.io/badge/Awesome-Strands%20Agents-00FF77?style=flat-square&logo=data:image/svg+xml;base64,PHN2ZyB3aWR0aD0iMjkwIiBoZWlnaHQ9IjQ2MyIgdmlld0JveD0iMCAwIDI5MCA0NjMiIGZpbGw9Im5vbmUiIHhtbG5zPSJodHRwOi8vd3d3LnczLm9yZy8yMDAwL3N2ZyI+CjxwYXRoIGQ9Ik05Ny4yOTAyIDUyLjc4ODRDODUuMDY3NCA0OS4xNjY3IDcyLjIyMzQgNTYuMTM4OSA2OC42MDE3IDY4LjM2MTZDNjQuOTgwMSA4MC41ODQzIDcxLjk1MjQgOTMuNDI4MyA4NC4xNzQ5IDk3LjA1MDFMMjM1LjExNyAxMzkuNzc1QzI0NS4yMjMgMTQyLjc2OSAyNDYuMzU3IDE1Ni42MjggMjM2Ljg3NCAxNjEuMjI2TDMyLjU0NiAyNjAuMjkxQy0xNC45NDM5IDI4My4zMTYgLTkuMTYxMDcgMzUyLjc0IDQxLjQ4MzUgMzY3LjU5MUwxODkuNTUxIDQxMS4wMDlMMTkwLjEyNSA0MTEuMTY5QzIwMi4xODMgNDE0LjM3NiAyMTQuNjY1IDQwNy4zOTYgMjE4LjE5NiAzOTUuMzU1QzIyMS43ODQgMzgzLjEyMiAyMTQuNzc0IDM3MC4yOTYgMjAyLjU0MSAzNjYuNzA5TDU0LjQ3MzggMzIzLjI5MUM0NC4zNDQ3IDMyMC4zMjEgNDMuMTg3OSAzMDYuNDM2IDUyLjY4NTcgMzAxLjgzMUwyNTcuMDE0IDIwMi43NjZDMzA0LjQzMiAxNzkuNzc2IDI5OC43NTggMTEwLjQ4MyAyNDguMjMzIDk1LjUxMkw5Ny4yOTAyIDUyLjc4ODRaIiBmaWxsPSIjRkZGRkZGIi8+CjxwYXRoIGQ9Ik0yNTkuMTQ3IDAuOTgxODEyQzI3MS4zODkgLTIuNTc0OTggMjg0LjE5NyA0LjQ2NTcxIDI4Ny43NTQgMTYuNzA3NEMyOTEuMzExIDI4Ljk0OTIgMjg0LjI3IDQxLjc1NyAyNzIuMDI4IDQ1LjMxMzhMNzEuMTcyNyAxMDMuNjcxQzQwLjcxNDIgMTEyLjUyMSAzNy4xOTc2IDE1NC4yNjIgNjUuNzQ1OSAxNjguMDgzTDI0MS4zNDMgMjUzLjA5M0MzMDcuODcyIDI4NS4zMDIgMjk5Ljc5NCAzODIuNTQ2IDIyOC44NjIgNDAzLjMzNkwzMC40MDQxIDQ2MS41MDJDMTguMTcwNyA0NjUuMDg4IDUuMzQ3MDggNDU4LjA3OCAxLjc2MTUzIDQ0NS44NDRDLTEuODIzOSA0MzMuNjExIDUuMTg2MzcgNDIwLjc4NyAxNy40MTk3IDQxNy4yMDJMMjE1Ljg3OCAzNTkuMDM1QzI0Ni4yNzcgMzUwLjEyNSAyNDkuNzM5IDMwOC40NDkgMjIxLjIyNiAyOTQuNjQ1TDQ1LjYyOTcgMjA5LjYzNUMtMjAuOTgzNCAxNzcuMzg2IC0xMi43NzcyIDc5Ljk4OTMgNTguMjkyOCA1OS4zNDAyTDI1OS4xNDcgMC45ODE4MTJaIiBmaWxsPSIjRkZGRkZGIi8+Cjwvc3ZnPgo=&logoColor=white)](https://github.com/cagataycali/awesome-strands-agents)
+
 agent = Agent(
     tools=[use_github, projects, scheduler, store_in_kb],
     system_prompt="You are an autonomous GitHub agent."

{strands_coder-1.4.0 → strands_coder-1.4.1}/README.md

@@ -217,6 +217,8 @@ from strands import Agent
 from strands_coder import use_github, projects, scheduler, store_in_kb
 
 # Create agent with GitHub tools
+[![Awesome Strands Agents](https://img.shields.io/badge/Awesome-Strands%20Agents-00FF77?style=flat-square&logo=data:image/svg+xml;base64,PHN2ZyB3aWR0aD0iMjkwIiBoZWlnaHQ9IjQ2MyIgdmlld0JveD0iMCAwIDI5MCA0NjMiIGZpbGw9Im5vbmUiIHhtbG5zPSJodHRwOi8vd3d3LnczLm9yZy8yMDAwL3N2ZyI+CjxwYXRoIGQ9Ik05Ny4yOTAyIDUyLjc4ODRDODUuMDY3NCA0OS4xNjY3IDcyLjIyMzQgNTYuMTM4OSA2OC42MDE3IDY4LjM2MTZDNjQuOTgwMSA4MC41ODQzIDcxLjk1MjQgOTMuNDI4MyA4NC4xNzQ5IDk3LjA1MDFMMjM1LjExNyAxMzkuNzc1QzI0NS4yMjMgMTQyLjc2OSAyNDYuMzU3IDE1Ni42MjggMjM2Ljg3NCAxNjEuMjI2TDMyLjU0NiAyNjAuMjkxQy0xNC45NDM5IDI4My4zMTYgLTkuMTYxMDcgMzUyLjc0IDQxLjQ4MzUgMzY3LjU5MUwxODkuNTUxIDQxMS4wMDlMMTkwLjEyNSA0MTEuMTY5QzIwMi4xODMgNDE0LjM3NiAyMTQuNjY1IDQwNy4zOTYgMjE4LjE5NiAzOTUuMzU1QzIyMS43ODQgMzgzLjEyMiAyMTQuNzc0IDM3MC4yOTYgMjAyLjU0MSAzNjYuNzA5TDU0LjQ3MzggMzIzLjI5MUM0NC4zNDQ3IDMyMC4zMjEgNDMuMTg3OSAzMDYuNDM2IDUyLjY4NTcgMzAxLjgzMUwyNTcuMDE0IDIwMi43NjZDMzA0LjQzMiAxNzkuNzc2IDI5OC43NTggMTEwLjQ4MyAyNDguMjMzIDk1LjUxMkw5Ny4yOTAyIDUyLjc4ODRaIiBmaWxsPSIjRkZGRkZGIi8+CjxwYXRoIGQ9Ik0yNTkuMTQ3IDAuOTgxODEyQzI3MS4zODkgLTIuNTc0OTggMjg0LjE5NyA0LjQ2NTcxIDI4Ny43NTQgMTYuNzA3NEMyOTEuMzExIDI4Ljk0OTIgMjg0LjI3IDQxLjc1NyAyNzIuMDI4IDQ1LjMxMzhMNzEuMTcyNyAxMDMuNjcxQzQwLjcxNDIgMTEyLjUyMSAzNy4xOTc2IDE1NC4yNjIgNjUuNzQ1OSAxNjguMDgzTDI0MS4zNDMgMjUzLjA5M0MzMDcuODcyIDI4NS4zMDIgMjk5Ljc5NCAzODIuNTQ2IDIyOC44NjIgNDAzLjMzNkwzMC40MDQxIDQ2MS41MDJDMTguMTcwNyA0NjUuMDg4IDUuMzQ3MDggNDU4LjA3OCAxLjc2MTUzIDQ0NS44NDRDLTEuODIzOSA0MzMuNjExIDUuMTg2MzcgNDIwLjc4NyAxNy40MTk3IDQxNy4yMDJMMjE1Ljg3OCAzNTkuMDM1QzI0Ni4yNzcgMzUwLjEyNSAyNDkuNzM5IDMwOC40NDkgMjIxLjIyNiAyOTQuNjQ1TDQ1LjYyOTcgMjA5LjYzNUMtMjAuOTgzNCAxNzcuMzg2IC0xMi43NzcyIDc5Ljk4OTMgNTguMjkyOCA1OS4zNDAyTDI1OS4xNDcgMC45ODE4MTJaIiBmaWxsPSIjRkZGRkZGIi8+Cjwvc3ZnPgo=&logoColor=white)](https://github.com/cagataycali/awesome-strands-agents)
+
 agent = Agent(
     tools=[use_github, projects, scheduler, store_in_kb],
     system_prompt="You are an autonomous GitHub agent."

{strands_coder-1.4.0 → strands_coder-1.4.1}/strands_coder/context.py

@@ -218,18 +218,24 @@ def fetch_github_event_context() -> str:
             if "errors" in data:
                 print(f"⚠ GraphQL errors: {data['errors']}")
             else:
-                issue_data = data.get("data", {}).get("repository", {}).get("issue", {})
+                # GraphQL returns null (not {}) for absent nodes - e.g. when the
+                # number refers to a PR (issue: null) or the repo/data is null on a
+                # partial error. Chained .get(k, {}) does NOT guard against a present
+                # key with a null value, so coalesce every nullable node with `or {}`.
+                issue_data = (
+                    ((data.get("data") or {}).get("repository") or {}).get("issue") or {}
+                )
 
                 # Comments
-                comments = issue_data.get("comments", {}).get("nodes", [])
-                total_comments = issue_data.get("comments", {}).get("totalCount", 0)
+                comments = (issue_data.get("comments") or {}).get("nodes") or []
+                total_comments = (issue_data.get("comments") or {}).get("totalCount", 0)
 
                 if comments:
                     context_parts.append(
                         f"\n### 💬 Comments ({total_comments} total)\n"
                     )
                     for idx, comment in enumerate(comments, 1):
-                        author = comment.get("author", {}).get("login", "unknown")
+                        author = (comment.get("author") or {}).get("login", "unknown")
                         body = comment.get("body", "")
                         created = comment.get("createdAt", "")
                         context_parts.append(f"""
@@ -240,10 +246,10 @@ def fetch_github_event_context() -> str:
 """)
 
                 # Linked PRs/Issues
-                timeline = issue_data.get("timelineItems", {}).get("nodes", [])
+                timeline = (issue_data.get("timelineItems") or {}).get("nodes") or []
                 linked_items = []
                 for item in timeline:
-                    source = item.get("source", {})
+                    source = item.get("source") or {}
                     if source:
                         num = source.get("number")
                         title = source.get("title")
@@ -367,11 +373,11 @@ def fetch_github_event_context() -> str:
                 print(f"⚠ GraphQL errors: {data['errors']}")
             else:
                 pr_data = (
-                    data.get("data", {}).get("repository", {}).get("pullRequest", {})
+                    ((data.get("data") or {}).get("repository") or {}).get("pullRequest") or {}
                 )
 
                 # Reviews
-                reviews = pr_data.get("reviews", {}).get("nodes", [])
+                reviews = (pr_data.get("reviews") or {}).get("nodes") or []
                 total_reviews = pr_data.get("reviews", {}).get("totalCount", 0)
 
                 if reviews:
@@ -397,7 +403,7 @@ def fetch_github_event_context() -> str:
                         f"\n### 💬 Comments ({total_comments} total)\n"
                     )
                     for idx, comment in enumerate(comments, 1):
-                        author = comment.get("author", {}).get("login", "unknown")
+                        author = (comment.get("author") or {}).get("login", "unknown")
                         body = comment.get("body", "")
                         created = comment.get("createdAt", "")
                         context_parts.append(f"""
@@ -526,15 +532,15 @@ def fetch_github_event_context() -> str:
                 print(f"⚠ GraphQL errors: {data['errors']}")
             else:
                 disc_data = (
-                    data.get("data", {}).get("repository", {}).get("discussion", {})
+                    ((data.get("data") or {}).get("repository") or {}).get("discussion") or {}
                 )
-                comments = disc_data.get("comments", {}).get("nodes", [])
-                total_comments = disc_data.get("comments", {}).get("totalCount", 0)
+                comments = (disc_data.get("comments") or {}).get("nodes") or []
+                total_comments = (disc_data.get("comments") or {}).get("totalCount", 0)
 
                 if comments:
                     context_parts.append(f"\n### 💬 Replies ({total_comments} total)\n")
                     for idx, comment in enumerate(comments, 1):
-                        author = comment.get("author", {}).get("login", "unknown")
+                        author = (comment.get("author") or {}).get("login", "unknown")
                         body = comment.get("body", "")
                         created = comment.get("createdAt", "")
                         context_parts.append(f"""

strands_coder-1.4.1/tests/test_context_graphql_null.py

@@ -0,0 +1,133 @@
+"""Regression tests for fetch_github_event_context GraphQL-null handling.
+
+GitHub's GraphQL API returns ``null`` (not ``{}``) for an absent node - e.g.
+``repository.issue`` is ``null`` when the queried number is a PR, or when a
+partial error nulls a field. ``dict.get(key, {})`` does NOT guard against a key
+that is present with a ``None`` value, so the old code crashed with::
+
+    AttributeError: 'NoneType' object has no attribute 'get'
+
+at ``issue_data.get("comments", {})``. These tests pin the fix: a null node must
+degrade gracefully (return context built so far), never raise.
+"""
+
+from __future__ import annotations
+
+import json
+from unittest.mock import patch
+
+import importlib.util as _ilu
+import os as _os
+
+_CTX_PATH = _os.path.join(
+    _os.path.dirname(_os.path.dirname(_os.path.abspath(__file__))),
+    "strands_coder",
+    "context.py",
+)
+_spec = _ilu.spec_from_file_location("strands_coder_context", _CTX_PATH)
+_mod = _ilu.module_from_spec(_spec)
+_spec.loader.exec_module(_mod)
+fetch_github_event_context = _mod.fetch_github_event_context
+
+
+class _FakeResponse:
+    def __init__(self, payload: dict):
+        self._payload = payload
+
+    def raise_for_status(self) -> None:
+        return None
+
+    def json(self) -> dict:
+        return self._payload
+
+
+def _ctx(event_name: str, event: dict) -> str:
+    return json.dumps(
+        {
+            "event_name": event_name,
+            "event": event,
+            "repository": "owner/repo",
+            "actor": "tester",
+        }
+    )
+
+
+def test_issue_node_null_does_not_raise(monkeypatch):
+    """repository.issue == null (number is actually a PR) must not crash."""
+    monkeypatch.setenv("GITHUB_CONTEXT", _ctx("issues", {"issue": {"number": 7}, "action": "opened"}))
+    monkeypatch.setenv("GITHUB_TOKEN", "x")
+    monkeypatch.delenv("PAT_TOKEN", raising=False)
+
+    with patch("requests.post", return_value=_FakeResponse({"data": {"repository": {"issue": None}}})):
+        out = fetch_github_event_context()
+
+    assert isinstance(out, str)
+    assert "owner/repo" in out  # raw context header still rendered
+
+
+def test_repository_null_does_not_raise(monkeypatch):
+    """data.repository == null (partial error) must not crash."""
+    monkeypatch.setenv("GITHUB_CONTEXT", _ctx("issues", {"issue": {"number": 7}, "action": "opened"}))
+    monkeypatch.setenv("GITHUB_TOKEN", "x")
+    monkeypatch.delenv("PAT_TOKEN", raising=False)
+
+    with patch("requests.post", return_value=_FakeResponse({"data": {"repository": None}})):
+        out = fetch_github_event_context()
+
+    assert isinstance(out, str)
+
+
+def test_pull_request_node_null_does_not_raise(monkeypatch):
+    """repository.pullRequest == null must not crash."""
+    monkeypatch.setenv(
+        "GITHUB_CONTEXT",
+        _ctx("pull_request", {"pull_request": {"number": 9}, "action": "opened"}),
+    )
+    monkeypatch.setenv("GITHUB_TOKEN", "x")
+    monkeypatch.delenv("PAT_TOKEN", raising=False)
+
+    with patch("requests.post", return_value=_FakeResponse({"data": {"repository": {"pullRequest": None}}})):
+        out = fetch_github_event_context()
+
+    assert isinstance(out, str)
+
+
+def test_discussion_node_null_does_not_raise(monkeypatch):
+    """repository.discussion == null must not crash."""
+    monkeypatch.setenv(
+        "GITHUB_CONTEXT",
+        _ctx("discussion", {"discussion": {"number": 3}, "action": "created"}),
+    )
+    monkeypatch.setenv("GITHUB_TOKEN", "x")
+    monkeypatch.delenv("PAT_TOKEN", raising=False)
+
+    with patch("requests.post", return_value=_FakeResponse({"data": {"repository": {"discussion": None}}})):
+        out = fetch_github_event_context()
+
+    assert isinstance(out, str)
+
+
+def test_null_comment_author_does_not_raise(monkeypatch):
+    """A comment whose author is null (deleted user) must not crash."""
+    monkeypatch.setenv("GITHUB_CONTEXT", _ctx("issues", {"issue": {"number": 7}, "action": "opened"}))
+    monkeypatch.setenv("GITHUB_TOKEN", "x")
+    monkeypatch.delenv("PAT_TOKEN", raising=False)
+
+    payload = {
+        "data": {
+            "repository": {
+                "issue": {
+                    "comments": {
+                        "totalCount": 1,
+                        "nodes": [{"author": None, "body": "hi", "createdAt": "2026-01-01"}],
+                    },
+                    "timelineItems": {"nodes": [{"source": None}]},
+                }
+            }
+        }
+    }
+    with patch("requests.post", return_value=_FakeResponse(payload)):
+        out = fetch_github_event_context()
+
+    assert isinstance(out, str)
+    assert "@unknown" in out