forgexa-cli 1.13.5__tar.gz → 1.14.0__tar.gz

{forgexa_cli-1.13.5 → forgexa_cli-1.14.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: forgexa-cli
-Version: 1.13.5
+Version: 1.14.0
 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.5 → forgexa_cli-1.14.0}/forgexa_cli/__init__.py

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

{forgexa_cli-1.13.5 → forgexa_cli-1.14.0}/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.5"
+DAEMON_VERSION = "1.14.0"
 
 
 def _detect_client_type() -> str:
@@ -1585,6 +1585,14 @@ class WorkspaceManager:
             "connection closed",
             "connection refused",
             "network is unreachable",
+            # DNS resolution failures — transient (network reconnect, DHCP renewal,
+            # macOS resolver flap).  macOS/BSD SSH prints "nodename nor servname
+            # provided"; Linux SSH/git prints "name or service not known" or
+            # "temporary failure in name resolution".
+            "nodename nor servname",
+            "name or service not known",
+            "could not resolve host",
+            "temporary failure in name resolution",
         )
         return any(marker in err for marker in retry_markers)
 
@@ -2899,9 +2907,12 @@ class ProcessManager:
         "connection reset",
         "connection timed out",
         "connection error",
+        "failed to get response from the ai model",
         "name or service not known",
         "no such host",
         "network is unreachable",
+        "websockettransporterror",
+        "websocket receive failed",
         # "api error" removed: too broad — matches agent-generated code/output
         # discussing API errors. Real API transport errors are covered by the
         # connection patterns above (refused, reset, timed out, etc.).
@@ -3032,6 +3043,21 @@ class ProcessManager:
                         else:
                             has_result = True
                             has_meaningful_content = True
+            elif ev_type == "session.error":
+                err = data.get("data") or {}
+                if isinstance(err, dict):
+                    msg = (
+                        err.get("message")
+                        or err.get("error")
+                        or err.get("errorType")
+                        or ""
+                    )
+                    if msg:
+                        error_messages.append(str(msg))
+                elif isinstance(err, str) and err.strip():
+                    error_messages.append(err)
+                elif isinstance(data.get("message"), str) and data.get("message", "").strip():
+                    error_messages.append(data["message"])
             elif ev_type == "error":
                 msg = data.get("message", "")
                 if msg:
@@ -4198,7 +4224,27 @@ 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)
+            signals = self._extract_output_signals(stdout)
+            completed_without_result = 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"]
+            )
+            structured_errors = [
+                str(msg).strip()
+                for msg in (signals.get("error_messages") or [])
+                if str(msg).strip()
+            ]
+            copilot_specific_error = ""
+            for msg in reversed(structured_errors):
+                prefix = "Copilot exited with code "
+                suffix = msg[len(prefix):].strip() if msg.startswith(prefix) else ""
+                if not (msg.startswith(prefix) and suffix.isdigit()):
+                    copilot_specific_error = msg
+                    break
             effective_rc = copilot_exit if copilot_exit is not None else returncode
 
             if effective_rc == 0 and returncode == 0:
@@ -4236,7 +4282,13 @@ class ProcessManager:
                 # Retry 2: if still failing AND we haven't already tried both drops,
                 #          the error is likely auth/quota — log actionable hint.
                 _no_tools = not self._copilot_called_any_tools(stdout)
-                if effective_rc == 1 and _no_tools and reasoning and not _retry_without_effort:
+                if (
+                    effective_rc == 1
+                    and _no_tools
+                    and reasoning
+                    and not _retry_without_effort
+                    and not copilot_specific_error
+                ):
                     logger.warning(
                         "Copilot exitCode=1 with no tool calls for task %s — "
                         "retrying without --effort (flag may be unsupported by this CLI version). "
@@ -4248,7 +4300,7 @@ class ProcessManager:
                         _retry_without_effort=True,
                     )
                 # Exhausted retries — emit an actionable diagnostic hint
-                if effective_rc == 1 and _no_tools:
+                if effective_rc == 1 and _no_tools and not copilot_specific_error:
                     logger.error(
                         "Copilot exitCode=1 with no tool calls for task %s after retries. "
                         "Likely causes: (1) GitHub auth expired — run `gh auth status` and "
@@ -4257,12 +4309,18 @@ class ProcessManager:
                         "To reproduce manually: %s",
                         task_id, agent.version, _cmd_display,
                     )
+                if copilot_specific_error:
+                    failure_error = copilot_specific_error[:1500]
+                elif stderr.strip():
+                    failure_error = f"Copilot exited with code {effective_rc}: {stderr[-500:].strip()}"
+                else:
+                    failure_error = f"Copilot exited with code {effective_rc}"
                 return TaskResult(
                     status="failed",
                     exit_code=effective_rc,
                     stdout=stdout[-settings.AGENT_MAX_OUTPUT_SIZE:],
                     stderr=stderr[-10000:],
-                    error=f"Copilot exited with code {effective_rc}: {stderr[-500:]}",
+                    error=failure_error,
                     metrics=metrics,
                 )
         except asyncio.TimeoutError as exc:
@@ -6553,6 +6611,39 @@ class RuntimeDaemon:
             if result.status == "success" and task.node_type == "fix":
                 await self._collect_bugfix_artifacts(workspace_path, task, result)
 
+            # 4.9 For review/coding/testing nodes: attach updated analysis.json inline
+            # so the backend always receives the latest snapshot even if push fails.
+            # This closes the mirror-sync gap: review agents update analysis.json
+            # (open_risks, phase_handoffs) and this step ensures the content reaches
+            # runtimes.py as an inline artifact regardless of push success.
+            if result.status == "success" and task.node_type in ("review", "coding", "testing"):
+                analysis_dir = (
+                    (task.input_data or {}).get("analysis_output_dir", "")
+                    or ""
+                ).replace("\\", "/").lstrip("./").rstrip("/")
+                if analysis_dir:
+                    _aj_path = workspace_path / analysis_dir / "analysis.json"
+                    if _aj_path.exists() and _aj_path.stat().st_size > 0:
+                        try:
+                            _aj_rel = str(_aj_path.relative_to(workspace_path)).replace("\\", "/")
+                            _existing_paths = {a.get("path", "").replace("\\", "/") for a in result.artifacts}
+                            if _aj_rel not in _existing_paths:
+                                _aj_content = _aj_path.read_text(encoding="utf-8", errors="replace")
+                                result.artifacts.append({
+                                    "path": _aj_rel,
+                                    "content": _aj_content,
+                                    "type": "application/json",
+                                })
+                                logger.debug(
+                                    "Task %s (%s): attached updated analysis.json inline (%d bytes)",
+                                    task.task_id, task.node_type, len(_aj_content),
+                                )
+                        except Exception as _aj_err:
+                            logger.debug(
+                                "Task %s: could not attach analysis.json artifact: %s",
+                                task.task_id, _aj_err,
+                            )
+
             # 5. Auto-commit and push if changes exist
             if result.status == "success":
                 commit_result = await self._auto_commit(workspace_path, task)
@@ -6733,9 +6824,31 @@ class RuntimeDaemon:
                 json_path = base / "analysis.json"
                 if json_path.exists() and json_path.stat().st_size > 0:
                     try:
-                        _json.loads(json_path.read_text(encoding="utf-8"))
+                        _analysis_data = _json.loads(json_path.read_text(encoding="utf-8"))
                     except _json.JSONDecodeError as e:
                         issues.append(f"analysis.json is not valid JSON: {e}")
+                        _analysis_data = None
+                    # Task 6: validate phase_handoffs and open_risks fields
+                    if _analysis_data is not None:
+                        if "open_risks" not in _analysis_data:
+                            issues.append(
+                                "analysis.json missing 'open_risks' field "
+                                "(expected empty array [] at minimum)"
+                            )
+                        handoffs = _analysis_data.get("phase_handoffs")
+                        if not handoffs or not isinstance(handoffs, list):
+                            issues.append(
+                                "analysis.json missing 'phase_handoffs' array "
+                                "(must contain at least one entry with phase='analysis')"
+                            )
+                        elif not any(
+                            isinstance(h, dict) and h.get("phase") == "analysis"
+                            for h in handoffs
+                        ):
+                            issues.append(
+                                "analysis.json phase_handoffs has no entry "
+                                "with phase='analysis'"
+                            )
 
             # Validate test-intent.json if required by this type
             if "test-intent.json" in required_files:
@@ -6790,7 +6903,7 @@ class RuntimeDaemon:
                 elif design_path.stat().st_size == 0:
                     issues.append(f"Design document is empty: {doc_dir}/design.md")
 
-        elif node_type in ("coding", "testing", "fix"):
+        elif node_type in ("coding", "testing", "fix", "review"):
             # Syntax-check modified Python and JS/TS files
             for f in result.files_changed:
                 fpath = workspace_path / f
@@ -6813,6 +6926,29 @@ class RuntimeDaemon:
                             f"bracket imbalance (open={opens}, close={closes})"
                         )
 
+            # Task 6 (P6 fix): warning-level phase_handoffs check for non-analysis
+            # nodes. These nodes SHOULD append a phase_handoffs entry to analysis.json
+            # after completion. Missing entry is NOT blocking (agent may have skipped
+            # WRITEBACK) but is logged for observability and instructions iteration.
+            _ao_dir = (task.input_data or {}).get("analysis_output_dir", "")
+            if _ao_dir:
+                _aj_p = workspace_path / _ao_dir.lstrip("./").rstrip("/").replace("\\", "/") / "analysis.json"
+                if _aj_p.exists() and _aj_p.stat().st_size > 0:
+                    try:
+                        _aj_d = _json.loads(_aj_p.read_text(encoding="utf-8"))
+                        _handoffs = _aj_d.get("phase_handoffs") or []
+                        if not any(
+                            isinstance(h, dict) and h.get("phase") == node_type
+                            for h in _handoffs
+                        ):
+                            logger.info(
+                                "Node %s (type=%s) did not append phase_handoffs "
+                                "entry to analysis.json — agent may have skipped WRITEBACK",
+                                task.task_id, node_type,
+                            )
+                    except (_json.JSONDecodeError, UnicodeDecodeError):
+                        pass  # analysis.json corrupt — already flagged elsewhere
+
             # Testing-specific: validate structured test assets
             if node_type == "testing":
                 # Determine which checks to run for this requirement type.

{forgexa_cli-1.13.5 → forgexa_cli-1.14.0}/forgexa_cli.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: forgexa-cli
-Version: 1.13.5
+Version: 1.14.0
 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.5 → forgexa_cli-1.14.0}/pyproject.toml

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