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

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

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

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

{forgexa_cli-1.13.6 → 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.6"
+DAEMON_VERSION = "1.14.0"
 
 
 def _detect_client_type() -> str:
@@ -6611,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)
@@ -6791,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:
@@ -6848,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
@@ -6871,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.6 → 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.6
+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.6 → forgexa_cli-1.14.0}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "forgexa-cli"
-version = "1.13.6"
+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" }