codee-agent 0.5.4__tar.gz → 0.5.6__tar.gz

{codee_agent-0.5.4 → codee_agent-0.5.6}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: codee-agent
-Version: 0.5.4
+Version: 0.5.6
 Summary: A virtual co-worker that picks up Jira and Azure DevOps tasks and solves them with coding agents.
 Keywords: ai,agent,jira,azure-devops,claude-code,github-copilot,automation
 Author: Denis Kibalko

{codee_agent-0.5.4 → codee_agent-0.5.6}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "codee-agent"
-version = "0.5.4"
+version = "0.5.6"
 description = "A virtual co-worker that picks up Jira and Azure DevOps tasks and solves them with coding agents."
 readme = "README.md"
 license = "MIT"

{codee_agent-0.5.4 → codee_agent-0.5.6}/src/codee/test_admin_service.py

@@ -1719,7 +1719,7 @@ class SetupTasksMcpTest(unittest.TestCase):
             config = self._config(root)
             self.assertEqual(config["mcpServers"]["ado"], {
                 "command": "npx",
-                "args": ["-y", "@azure-devops/mcp", "acme",
+                "args": ["-y", "@azure-devops/mcp@2.8.0", "acme",
                          "--authentication", "azcli"],
             })
             self.assertEqual(config["servers"]["ado"]["type"], "stdio")

{codee_agent-0.5.4 → codee_agent-0.5.6}/src/codee_agent_claude_code/provider.py

@@ -72,31 +72,35 @@ class ClaudeCodeAgent(AbstractCodingAgent):
                 cmd,
                 capture_output=True,
                 text=True,
+                encoding="utf-8",
+                errors="replace",
                 timeout=self.TIMEOUT_SECONDS,
                 cwd=self._cwd,
             )
         except subprocess.TimeoutExpired:
             raise RuntimeError("Claude CLI timed out after 2 hours")
 
+        stdout = result.stdout or ""
+        stderr = result.stderr or ""
         log.debug("claude exited %d (%d bytes stdout, %d bytes stderr)",
-                  result.returncode, len(result.stdout), len(result.stderr))
+              result.returncode, len(stdout), len(stderr))
 
         # Raise on any non-success so callers retry. Over-limit exits non-zero;
         # a completed-but-errored run sets is_error in the JSON.
         if result.returncode != 0:
             raise RuntimeError(
-                f"Claude CLI exited {result.returncode}: {result.stderr.strip()[:500]}"
+                f"Claude CLI exited {result.returncode}: {stderr.strip()[:500]}"
             )
 
         try:
-            response = json.loads(result.stdout)
+            response = json.loads(stdout)
         except json.JSONDecodeError:
-            return result.stdout
+            return stdout
         if isinstance(response, dict):
             if response.get("is_error"):
                 raise RuntimeError(
                     f"Claude run errored ({response.get('subtype', 'unknown')}): "
                     f"{str(response.get('result', ''))[:500]}"
                 )
-            return response.get("result", result.stdout)
-        return result.stdout
+            return response.get("result", stdout)
+        return stdout

{codee_agent-0.5.4 → codee_agent-0.5.6}/src/codee_agent_claude_code/test.py

@@ -34,6 +34,20 @@ class ClaudeCodeRunTest(unittest.TestCase):
 
         self.assertEqual(cmd[cmd.index("--model") + 1], "opus")
 
+    def test_cli_output_is_decoded_as_utf8(self) -> None:
+        with patch("subprocess.run", return_value=_completed()) as run:
+            self.agent.run("/do-it CORE-1", SESSION)
+
+        self.assertEqual(run.call_args.kwargs["encoding"], "utf-8")
+        self.assertEqual(run.call_args.kwargs["errors"], "replace")
+
+    def test_missing_captured_streams_do_not_raise_type_error(self) -> None:
+        completed = subprocess.CompletedProcess(
+            args=["claude"], returncode=0, stdout=None, stderr=None)
+
+        with patch("subprocess.run", return_value=completed):
+            self.assertEqual(self.agent.run("/do-it CORE-1", SESSION), "")
+
     def test_the_best_model_is_a_version_free_alias(self) -> None:
         # The alias tracks the newest Opus, so no release needs an edit here.
         self.assertEqual(ClaudeCodeAgent.best_model(), "opus")

{codee_agent-0.5.4 → codee_agent-0.5.6}/src/codee_agent_github_copilot/provider.py

@@ -102,16 +102,20 @@ class GitHubCopilotAgent(AbstractCodingAgent):
                 cmd,
                 capture_output=True,
                 text=True,
+                encoding="utf-8",
+                errors="replace",
                 timeout=self.TIMEOUT_SECONDS,
                 cwd=self._cwd,
             )
         except subprocess.TimeoutExpired:
             raise RuntimeError("Copilot CLI timed out after 2 hours")
 
+        stdout = result.stdout or ""
+        stderr = result.stderr or ""
         log.debug("copilot exited %d (%d bytes stdout, %d bytes stderr)",
-                  result.returncode, len(result.stdout), len(result.stderr))
+                  result.returncode, len(stdout), len(stderr))
 
-        reply, outcome, errors = _parse_events(result.stdout)
+        reply, outcome, errors = _parse_events(stdout)
 
         # Raise on any non-success so callers retry. A run that fails before the
         # session starts (bad model, no auth) exits non-zero with nothing on
@@ -119,22 +123,22 @@ class GitHubCopilotAgent(AbstractCodingAgent):
         if result.returncode != 0:
             raise RuntimeError(
                 f"Copilot CLI exited {result.returncode}: "
-                f"{_detail(errors, result.stderr)}"
+                f"{_detail(errors, stderr)}"
             )
         if outcome is None:
             # Not the JSONL we know how to read — hand back whatever it printed
             # rather than failing a run that the CLI itself called successful.
             log.warning(
                 "copilot produced no result event; returning raw output")
-            return result.stdout
+            return stdout
         if outcome.get("exitCode"):
             raise RuntimeError(
                 f"Copilot run errored (exit code {outcome['exitCode']}): "
-                f"{_detail(errors, result.stderr)}"
+                f"{_detail(errors, stderr)}"
             )
         if not reply:
             raise RuntimeError(
-                f"Copilot run produced no response: {_detail(errors, result.stderr)}"
+                f"Copilot run produced no response: {_detail(errors, stderr)}"
             )
         return reply
 
@@ -152,6 +156,8 @@ def _fetch_acp_models() -> list[AgentModel]:
         stdout=subprocess.PIPE,
         stderr=subprocess.PIPE,
         text=True,
+        encoding="utf-8",
+        errors="replace",
         bufsize=1,
         cwd=Path.cwd(),
     )

{codee_agent-0.5.4 → codee_agent-0.5.6}/src/codee_agent_github_copilot/test.py

@@ -71,6 +71,14 @@ class CopilotRunTest(unittest.TestCase):
         for flag in ("--allow-all", "--no-ask-user", "--output-format"):
             self.assertIn(flag, cmd)
         self.assertEqual(self.captured.call_args.kwargs["cwd"], Path("/repo"))
+        self.assertEqual(self.captured.call_args.kwargs["encoding"], "utf-8")
+        self.assertEqual(self.captured.call_args.kwargs["errors"], "replace")
+
+    def test_missing_captured_streams_do_not_raise_type_error(self) -> None:
+        completed = subprocess.CompletedProcess(
+            args=["copilot"], returncode=0, stdout=None, stderr=None)
+
+        self.assertEqual(self._run(completed), "")
 
     def test_the_skill_model_is_passed_on_the_command_line(self) -> None:
         stdout = _stream(_event("assistant.message", {"content": "ok"}),
@@ -194,7 +202,7 @@ class CopilotModelCatalogTest(unittest.TestCase):
             {"name": "nameless"},            # no id at all: unusable, skipped
         ]}}
 
-        with patch("codee_agent_github_copilot.provider.subprocess.Popen"), \
+        with patch("codee_agent_github_copilot.provider.subprocess.Popen") as popen, \
                 patch("codee_agent_github_copilot.provider._send"), \
                 patch("codee_agent_github_copilot.provider._await_result",
                       return_value=result):
@@ -202,6 +210,8 @@ class CopilotModelCatalogTest(unittest.TestCase):
 
         self.assertEqual([(m.id, m.name) for m in models],
                          [("claude-opus-5", "Claude Opus 5"), ("gpt-5.4", "gpt-5.4")])
+        self.assertEqual(popen.call_args.kwargs["encoding"], "utf-8")
+        self.assertEqual(popen.call_args.kwargs["errors"], "replace")
 
     def test_an_unavailable_cli_yields_no_models_rather_than_raising(self) -> None:
         with patch("codee_agent_github_copilot.provider.subprocess.Popen",

{codee_agent-0.5.4 → codee_agent-0.5.6}/src/codee_tasks_azure_devops/provider.py

@@ -44,8 +44,10 @@ _FIELDS = [
 _PRIORITY_NAMES = {1: "Highest", 2: "High", 3: "Medium", 4: "Low"}
 
 # Microsoft's own Azure DevOps MCP server, run from npm through `npx` so
-# nothing has to be installed alongside it.
-MCP_SERVER_PACKAGE = "@azure-devops/mcp"
+# nothing has to be installed alongside it. Version 2.8.0 keeps work item
+# creation compatible with coding agents that stringify the consolidated
+# write tool's fields parameter introduced in 2.9.0.
+MCP_SERVER_PACKAGE = "@azure-devops/mcp@2.8.0"
 
 # Ceiling the WIQL query is capped at, matching the JIRA provider's page size.
 _MAX_TASKS = 50

{codee_agent-0.5.4 → codee_agent-0.5.6}/src/codee_tasks_azure_devops/test.py

@@ -629,7 +629,7 @@ class AzureDevOpsMcpTest(unittest.TestCase):
 
         self.assertEqual(server.name, "ado")
         self.assertEqual(server.command, "npx")
-        self.assertEqual(server.args, ["-y", "@azure-devops/mcp", "acme",
+        self.assertEqual(server.args, ["-y", "@azure-devops/mcp@2.8.0", "acme",
                                        "--authentication", "azcli"])
         # It signs in through `az login`, so it carries no credentials of ours.
         self.assertEqual(server.env, {})