forgexa-cli 1.13.1__tar.gz → 1.13.2__tar.gz

{forgexa_cli-1.13.1 → forgexa_cli-1.13.2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: forgexa-cli
-Version: 1.13.1
+Version: 1.13.2
 Summary: Forgexa CLI — command-line client and AI agent runtime for the Forgexa platform
 Author-email: Jason Sun <dev.winds@gmail.com>
 License: MIT

{forgexa_cli-1.13.1 → forgexa_cli-1.13.2}/forgexa_cli/__init__.py

@@ -1,2 +1,2 @@
 """forgexa-cli — Forgexa command-line client."""
-__version__ = "1.13.1"
+__version__ = "1.13.2"

{forgexa_cli-1.13.1 → forgexa_cli-1.13.2}/forgexa_cli/daemon.py

@@ -523,7 +523,7 @@ except (ImportError, ModuleNotFoundError):
 # DAEMON_VERSION is the protocol/logic version of the daemon code.
 # Kept in sync with pyproject.toml version via bump-version.sh.
 # CLIENT_TYPE identifies which packaging/distribution this daemon runs in.
-DAEMON_VERSION = "1.13.1"
+DAEMON_VERSION = "1.13.2"
 
 
 def _detect_client_type() -> str:
@@ -2792,8 +2792,11 @@ class ProcessManager:
                 msg_data = data.get("data") or {}
                 if isinstance(msg_data.get("content"), str) and msg_data["content"].strip():
                     has_result = True
+            elif ev_type == "assistant.message_start":
+                has_assistant_events = True
             elif ev_type == "assistant.message_delta":
                 has_meaningful_content = True
+                has_assistant_events = True
             # ── Generic / Claude "result" event ────────────────────────────
             elif ev_type == "result":
                 # Copilot result format: {"type":"result","exitCode":0,"usage":{...}}
@@ -2916,6 +2919,19 @@ class ProcessManager:
             or signals["has_meaningful_content"]
         )
 
+    @staticmethod
+    def _copilot_completed_without_result(stdout: str) -> bool:
+        """Detect Copilot runs that completed a turn but omitted the final result event."""
+        signals = ProcessManager._extract_output_signals(stdout)
+        return bool(
+            signals["has_turn_completed"]
+            and signals["has_assistant_events"]
+            and signals["has_meaningful_content"]
+            and not signals["has_result"]
+            and not signals["has_turn_failed"]
+            and not signals["error_messages"]
+        )
+
     @staticmethod
     def is_rate_limited(result: "TaskResult") -> bool:
         """Check if an agent failure warrants trying a different agent.
@@ -3930,6 +3946,7 @@ class ProcessManager:
             # Copilot always exits 0 on normal completion; check result.exitCode
             # from the JSONL "result" event for a true success signal.
             copilot_exit = self._extract_copilot_exit_code(stdout)
+            completed_without_result = self._copilot_completed_without_result(stdout)
             effective_rc = copilot_exit if copilot_exit is not None else returncode
 
             if effective_rc == 0 and returncode == 0:
@@ -3940,6 +3957,21 @@ class ProcessManager:
                     stderr=stderr[-10000:],
                     metrics=metrics,
                 )
+            elif copilot_exit is None and returncode != 0 and completed_without_result:
+                logger.warning(
+                    "Copilot exited with return code %s for task %s despite a completed assistant turn and no result event; recovering to validation path",
+                    returncode,
+                    task_id,
+                )
+                metrics["recovered_missing_result_event"] = True
+                metrics["recovered_process_returncode"] = returncode
+                return TaskResult(
+                    status="success",
+                    exit_code=0,
+                    stdout=stdout[-settings.AGENT_MAX_OUTPUT_SIZE:],
+                    stderr=stderr[-10000:],
+                    metrics=metrics,
+                )
             else:
                 return TaskResult(
                     status="failed",

{forgexa_cli-1.13.1 → forgexa_cli-1.13.2}/forgexa_cli/main.py

@@ -247,6 +247,31 @@ def _is_virtualenv() -> bool:
     )
 
 
+def _is_pep668_environment() -> bool:
+    """Return True if the active Python is governed by PEP 668 (externally-managed).
+
+    Debian 12, Ubuntu 22.10+/24.04+, Fedora 38+ and other modern distros place
+    an EXTERNALLY-MANAGED marker in their stdlib directory.  pip refuses both
+    user-site and system-wide installs on these environments unless
+    --break-system-packages is explicitly passed — even ``pip install --user``
+    is blocked.  We detect the marker up-front so ``forgexa upgrade`` can add
+    the flag automatically rather than failing and asking the user to do it.
+    """
+    try:
+        import sysconfig as _sysconfig
+        stdlib = _sysconfig.get_path("stdlib")
+        if stdlib and (Path(stdlib) / "EXTERNALLY-MANAGED").exists():
+            return True
+    except Exception:
+        pass
+    try:
+        if (Path(sys.prefix) / "EXTERNALLY-MANAGED").exists():
+            return True
+    except Exception:
+        pass
+    return False
+
+
 def _is_user_site_install(dist_root: Path | None) -> bool:
     if dist_root is None:
         return False
@@ -355,6 +380,12 @@ def _build_upgrade_plan(target_version: str | None) -> UpgradePlan:
     command = [python, "-m", "pip", "install", "--upgrade"]
     if _is_user_site_install(dist_root) and not _is_virtualenv():
         command.append("--user")
+    if _is_pep668_environment():
+        # PEP 668 (Ubuntu 24.04+, Debian 12+, Fedora 38+, …) blocks pip
+        # installs — including --user — unless this flag is passed.  The
+        # original installation on this machine must have used it too, so
+        # adding it here is consistent and safe.
+        command.append("--break-system-packages")
     command.append(_package_spec_for_version(target_version))
     return UpgradePlan(
         installer="pip",

{forgexa_cli-1.13.1 → forgexa_cli-1.13.2}/forgexa_cli.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: forgexa-cli
-Version: 1.13.1
+Version: 1.13.2
 Summary: Forgexa CLI — command-line client and AI agent runtime for the Forgexa platform
 Author-email: Jason Sun <dev.winds@gmail.com>
 License: MIT

{forgexa_cli-1.13.1 → forgexa_cli-1.13.2}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "forgexa-cli"
-version = "1.13.1"
+version = "1.13.2"
 description = "Forgexa CLI — command-line client and AI agent runtime for the Forgexa platform"
 requires-python = ">=3.9"
 license = { text = "MIT" }

{forgexa_cli-1.13.1 → forgexa_cli-1.13.2}/tests/test_auth_and_runtime_commands.py

@@ -244,6 +244,7 @@ def test_build_upgrade_plan_for_pip_user_site(monkeypatch: pytest.MonkeyPatch) -
     monkeypatch.setattr(main.importlib.util, "find_spec", lambda name: object() if name == "pip" else None)
     monkeypatch.setattr(main, "_is_user_site_install", lambda root: True)
     monkeypatch.setattr(main, "_is_virtualenv", lambda: False)
+    monkeypatch.setattr(main, "_is_pep668_environment", lambda: False)
     monkeypatch.setattr(main.sys, "executable", "/usr/bin/python3")
 
     plan = main._build_upgrade_plan("1.12.3")
@@ -260,6 +261,41 @@ def test_build_upgrade_plan_for_pip_user_site(monkeypatch: pytest.MonkeyPatch) -
     ]
 
 
+def test_build_upgrade_plan_adds_break_system_packages_on_pep668(monkeypatch: pytest.MonkeyPatch) -> None:
+    monkeypatch.setattr(main, "_cli_distribution", lambda: SimpleNamespace(version="1.13.1"))
+    monkeypatch.setattr(main, "_distribution_root", lambda dist: Path("/usr/lib/python3/dist-packages"))
+    monkeypatch.setattr(main, "_running_from_distribution_root", lambda root: True)
+    monkeypatch.setattr(main, "_direct_url_metadata", lambda dist: None)
+    monkeypatch.setattr(main, "_looks_like_pipx_environment", lambda python=None: False)
+    monkeypatch.setattr(main.importlib.util, "find_spec", lambda name: object() if name == "pip" else None)
+    monkeypatch.setattr(main, "_is_user_site_install", lambda root: True)
+    monkeypatch.setattr(main, "_is_virtualenv", lambda: False)
+    monkeypatch.setattr(main, "_is_pep668_environment", lambda: True)
+    monkeypatch.setattr(main.sys, "executable", "/usr/bin/python3")
+
+    plan = main._build_upgrade_plan(None)
+
+    assert "--break-system-packages" in plan.command
+    assert "--user" in plan.command
+    assert plan.command[-1] == "forgexa-cli"
+
+
+def test_pep668_detection_returns_false_in_normal_venv(tmp_path: Path, monkeypatch: pytest.MonkeyPatch) -> None:
+    monkeypatch.setattr(main.sys, "prefix", str(tmp_path))
+    import sysconfig as _sc
+    monkeypatch.setattr(_sc, "get_path", lambda name, **kw: str(tmp_path))
+
+    assert main._is_pep668_environment() is False
+
+
+def test_pep668_detection_returns_true_when_marker_present(tmp_path: Path, monkeypatch: pytest.MonkeyPatch) -> None:
+    (tmp_path / "EXTERNALLY-MANAGED").write_text("[externally-managed]\n")
+    import sysconfig as _sc
+    monkeypatch.setattr(_sc, "get_path", lambda name, **kw: str(tmp_path))
+
+    assert main._is_pep668_environment() is True
+
+
 def test_build_upgrade_plan_rejects_editable_install(monkeypatch: pytest.MonkeyPatch) -> None:
     monkeypatch.setattr(main, "_cli_distribution", lambda: SimpleNamespace(version="1.12.2"))
     monkeypatch.setattr(main, "_distribution_root", lambda dist: Path("/installed/site-packages"))