forgexa-cli 1.10.6__tar.gz → 1.10.8__tar.gz

{forgexa_cli-1.10.6 → forgexa_cli-1.10.8}/PKG-INFO

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

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

{forgexa_cli-1.10.6 → forgexa_cli-1.10.8}/forgexa_cli/daemon.py

@@ -404,7 +404,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.10.6"
+DAEMON_VERSION = "1.10.8"
 
 
 def _detect_client_type() -> str:
@@ -2248,6 +2248,17 @@ class ProcessManager:
         )
         if not has_token_usage and ProcessManager._should_scan_short_success_stdout(stdout, signals):
             error_channels = "\n".join(filter(None, [error_channels, stdout.strip()]))
+        # Always include structured error messages parsed from JSONL stream events
+        # (result events with is_error=True, error events, turn.failed payloads).
+        # These are safe to scan unconditionally because they are structured — NOT
+        # free-form agent work text.  This catches mid-session model overload errors
+        # (e.g. 529 overloaded_error) that appear as structured events AFTER some
+        # assistant turns have already been emitted.  Without this, such errors are
+        # invisible to the stdout-scan guard (has_assistant_events=True → no scan)
+        # yet still signal that the underlying model call failed.
+        structured_errors = signals.get("error_messages", [])
+        if structured_errors:
+            error_channels = "\n".join(filter(None, [error_channels, "\n".join(structured_errors)]))
         return error_channels
 
     @staticmethod
@@ -5793,6 +5804,106 @@ class RuntimeDaemon:
         except Exception as e:
             logger.warning("Failed to read design artifact: %s", e)
 
+    async def _remove_root_scratch_files(
+        self, workspace_path: Path, task: TaskInfo, git_status_output: str
+    ) -> None:
+        """Remove unauthorized scratch/intermediate files created in the repo root.
+
+        Agents sometimes create working notes (findings.md, progress.md,
+        task_plan.md, notes.md, etc.) directly in the repository root instead of
+        inside the designated docs/requirements/<key>/ output directory.  These
+        files:
+          1. Pollute the root directory with unstructured content.
+          2. May survive into subsequent agent runs and cause cross-requirement
+             context contamination.
+          3. Can cause confusion when multiple requirements share a worktree.
+
+        This method removes any NEW (untracked or staged-as-add) markdown file
+        found at the repository root that is not a recognised project-level doc,
+        and logs a warning so the prompt can be improved.
+
+        NOTE: This only catches files that the agent has NOT yet committed.
+        Files the agent self-committed (e.g. via `git add -A && git commit`)
+        are prevented by the prompt-level prohibitions in CRITICAL REQUIREMENT
+        and the type-specific IMPORTANT constraints.
+        """
+        # Well-known root-level markdown files that must never be removed
+        _PROTECTED = frozenset({
+            "README.md", "readme.md",
+            "CONTRIBUTING.md", "contributing.md",
+            "CHANGELOG.md", "changelog.md",
+            "LICENSE.md", "license.md",
+            "SECURITY.md", "security.md",
+            "CODE_OF_CONDUCT.md",
+            "CONVENTIONS.md",
+            "WORKFLOW.md",
+        })
+
+        removed: list[str] = []
+        for raw_line in git_status_output.splitlines():
+            if len(raw_line) < 4:
+                continue
+            index_status = raw_line[0]
+            worktree_status = raw_line[1]
+            file_path = raw_line[3:].strip().strip('"')
+
+            # Only new files: untracked (??) or staged as addition (A)
+            is_new_untracked = (index_status == "?" and worktree_status == "?")
+            is_staged_new = (index_status == "A")
+            if not (is_new_untracked or is_staged_new):
+                continue
+
+            # Only root-level files (no path separator)
+            if "/" in file_path or "\\" in file_path:
+                continue
+
+            # Only markdown files
+            if not file_path.lower().endswith(".md"):
+                continue
+
+            # Skip protected project-level docs
+            if file_path in _PROTECTED:
+                continue
+
+            # Unstage if it was already staged
+            if is_staged_new:
+                try:
+                    proc = await asyncio.create_subprocess_exec(
+                        "git", "restore", "--staged", file_path,
+                        cwd=str(workspace_path),
+                        stdout=asyncio.subprocess.PIPE,
+                        stderr=asyncio.subprocess.PIPE,
+                    )
+                    await proc.communicate()
+                except Exception as exc:
+                    logger.warning(
+                        "Failed to unstage root scratch file '%s': %s", file_path, exc
+                    )
+
+            # Delete the file
+            target = workspace_path / file_path
+            if target.exists():
+                try:
+                    target.unlink()
+                    removed.append(file_path)
+                except Exception as exc:
+                    logger.warning(
+                        "Failed to remove root scratch file '%s': %s", file_path, exc
+                    )
+
+        if removed:
+            logger.warning(
+                "Task %s (req: %s, node_type: %s): removed %d unauthorized root-level "
+                "markdown file(s) before commit: [%s]. "
+                "All documentation must be placed inside docs/requirements/<key>/. "
+                "Check agent prompt for STRICT FILE PLACEMENT RULES compliance.",
+                task.task_id,
+                task.requirement_key or "?",
+                task.node_type,
+                len(removed),
+                ", ".join(removed),
+            )
+
     async def _auto_commit(self, workspace_path: Path, task: TaskInfo) -> dict:
         """Auto-commit and push agent changes.
 
@@ -5824,6 +5935,14 @@ class RuntimeDaemon:
             stdout, _ = await proc.communicate()
 
             if stdout.strip():
+                # Safety net: remove any scratch/intermediate .md files the agent
+                # created directly in the repository root before staging everything.
+                # Agents sometimes write working notes (findings.md, progress.md,
+                # task_plan.md, etc.) to cwd instead of the designated output dir.
+                # These files pollute the root and cause cross-requirement context
+                # contamination on subsequent agent runs.
+                await self._remove_root_scratch_files(workspace_path, task, stdout.decode())
+
                 # Stage all changes
                 proc = await asyncio.create_subprocess_exec(
                     "git", "add", "-A",

{forgexa_cli-1.10.6 → forgexa_cli-1.10.8}/forgexa_cli.egg-info/PKG-INFO

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

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