forgexa-cli 1.18.2__tar.gz → 1.18.3__tar.gz

{forgexa_cli-1.18.2 → forgexa_cli-1.18.3}/PKG-INFO

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

{forgexa_cli-1.18.2 → forgexa_cli-1.18.3}/forgexa_cli/__init__.py

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

{forgexa_cli-1.18.2 → forgexa_cli-1.18.3}/forgexa_cli/daemon.py

@@ -527,7 +527,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.18.2"
+DAEMON_VERSION = "1.18.3"
 
 
 def _detect_client_type() -> str:
@@ -2087,6 +2087,7 @@ class WorkspaceManager:
                         f"({sync_error or 'unknown sync error'}). Refusing to continue on a "
                         "workspace that may be based on the default branch."
                     )
+            await self._git("config", "core.filemode", "false", cwd=ws_path)
             return ws_path
         else:
             # No repo — create a directory with git init for change tracking
@@ -2109,6 +2110,7 @@ class WorkspaceManager:
                 )
                 # Create a working branch (V5.4: use human-readable name)
                 await self._git("checkout", "-b", branch_name, cwd=ws_path)
+            await self._git("config", "core.filemode", "false", cwd=ws_path)
             return ws_path
 
     async def _is_healthy_worktree(self, ws_path: Path, main_repo: Path | None = None) -> bool:
@@ -7110,6 +7112,44 @@ class RuntimeDaemon:
 
         return had_work
 
+    async def _workspace_checkout_health_error(self, workspace_path: Path) -> str | None:
+        """Return an error when a tracked worktree has been catastrophically emptied."""
+        try:
+            tracked_output = await self.workspace_manager._git(
+                "ls-files", "-z", "--cached", "--", ".",
+                cwd=workspace_path,
+                timeout=30,
+            )
+            tracked_count = len([path for path in tracked_output.split("\0") if path])
+            if tracked_count <= 500:
+                return None
+
+            physical_count = sum(
+                1
+                for path in workspace_path.rglob("*")
+                if path.is_file() and ".git" not in path.parts
+            )
+        except Exception as exc:
+            logger.warning("Workspace health check failed to inspect %s: %s", workspace_path, exc)
+            return None
+
+        if physical_count / tracked_count >= 0.20:
+            return None
+
+        longpath_hint = (
+            " Enable Windows long-path support: run "
+            "`git config --global core.longpaths true` and enable "
+            "LongPathsEnabled in Windows Group Policy / Registry "
+            "(HKLM\\SYSTEM\\CurrentControlSet\\Control\\FileSystem\\LongPathsEnabled=1)."
+            if sys.platform == "win32" else ""
+        )
+        return (
+            f"Workspace health check failed: only {physical_count}/{tracked_count} "
+            f"tracked files exist on disk ({physical_count / tracked_count:.0%}). "
+            "The git checkout was likely damaged during execution."
+            f"{longpath_hint}"
+        )
+
     def _recover_timeout_from_workspace_changes(
         self,
         task: TaskInfo,
@@ -7797,7 +7837,24 @@ class RuntimeDaemon:
 
             # 5. Auto-commit and push if changes exist
             if result.status == "success":
-                commit_result = await self._auto_commit(workspace_path, task)
+                workspace_health_error = await self._workspace_checkout_health_error(
+                    workspace_path
+                )
+                if workspace_health_error:
+                    result.status = "failed"
+                    result.failure_code = "workspace_checkout_failed"
+                    result.error = workspace_health_error
+                    result.files_changed = []
+                    result.lines_added = 0
+                    result.lines_removed = 0
+                    result.git = {}
+
+            if result.status == "success":
+                commit_result = await self._auto_commit(
+                    workspace_path,
+                    task,
+                    before_sha=node_before_sha,
+                )
                 if commit_result:
                     # Propagate push/commit errors in metrics so they're visible
                     result.metrics.update(commit_result)
@@ -7841,21 +7898,15 @@ class RuntimeDaemon:
                     default_branch=default_branch,
                     before_sha=node_before_sha,
                 )
-                # Merge: use the pre-commit file list if post-commit is empty
                 result.git = post_commit_git
-                if not result.git.get("files_changed") and pre_commit_git.get("files_changed"):
-                    result.git["files_changed"] = pre_commit_git["files_changed"]
-                    result.git["lines_added"] = pre_commit_git.get("lines_added", 0)
-                    result.git["lines_removed"] = pre_commit_git.get("lines_removed", 0)
                 result.files_changed = result.git.get("files_changed", [])
                 result.lines_added = result.git.get("lines_added", 0)
                 result.lines_removed = result.git.get("lines_removed", 0)
             else:
-                result.git = pre_commit_git
-                if not result.files_changed and pre_commit_git.get("files_changed"):
-                    result.files_changed = pre_commit_git["files_changed"]
-                    result.lines_added = pre_commit_git.get("lines_added", 0)
-                    result.lines_removed = pre_commit_git.get("lines_removed", 0)
+                result.git = {}
+                result.files_changed = []
+                result.lines_added = 0
+                result.lines_removed = 0
 
             # 6. Report completion (include actual agent used if different from requested)
             result.metrics["actual_agent"] = agent.agent_id
@@ -8922,7 +8973,57 @@ class RuntimeDaemon:
                 ", ".join(removed),
             )
 
-    async def _auto_commit(self, workspace_path: Path, task: TaskInfo) -> dict:
+    async def _pure_file_mode_change_error(
+        self,
+        workspace_path: Path,
+        *diff_args: str,
+    ) -> str | None:
+        """Reject Git changes that alter only file modes, not file contents."""
+        proc = await asyncio.create_subprocess_exec(
+            "git", "diff", "--raw", "--no-abbrev", "--no-renames", "-z", *diff_args,
+            cwd=str(workspace_path),
+            stdout=asyncio.subprocess.PIPE,
+            stderr=asyncio.subprocess.PIPE,
+        )
+        stdout, stderr = await proc.communicate()
+        if proc.returncode != 0:
+            detail = stderr.decode(errors="replace").strip()
+            return (
+                "Auto-commit blocked: could not validate file mode changes"
+                f"{f': {detail}' if detail else ''}"
+            )
+
+        mode_only_paths: list[str] = []
+        entries = stdout.decode(errors="surrogateescape").split("\0")
+        for index in range(0, len(entries) - 1, 2):
+            metadata, path = entries[index], entries[index + 1]
+            fields = metadata.split()
+            if len(fields) < 4 or not fields[0].startswith(":"):
+                continue
+            old_mode = fields[0][1:]
+            new_mode, old_object, new_object = fields[1:4]
+            if old_object == new_object and old_mode != new_mode:
+                mode_only_paths.append(path)
+
+        if not mode_only_paths:
+            return None
+
+        sample = ", ".join(mode_only_paths[:5])
+        remainder = "" if len(mode_only_paths) <= 5 else ", ..."
+        return (
+            "Auto-commit blocked: detected "
+            f"{len(mode_only_paths)} permission-only file change(s) with unchanged content "
+            f"({sample}{remainder}). Restore the original modes before retrying; "
+            "Forgexa does not auto-commit permission-only changes."
+        )
+
+    async def _auto_commit(
+        self,
+        workspace_path: Path,
+        task: TaskInfo,
+        *,
+        before_sha: str = "",
+    ) -> dict:
         """Auto-commit and push agent changes.
 
         Some agents (e.g. claude) commit changes internally, so we must
@@ -8943,6 +9044,13 @@ class RuntimeDaemon:
         try:
             has_new_commit = False
 
+            if before_sha:
+                mode_change_error = await self._pure_file_mode_change_error(
+                    workspace_path, before_sha, "HEAD"
+                )
+                if mode_change_error:
+                    return {"commit_error": mode_change_error}
+
             # Check for uncommitted changes
             proc = await asyncio.create_subprocess_exec(
                 "git", "status", "--porcelain",
@@ -8968,7 +9076,16 @@ class RuntimeDaemon:
                     stdout=asyncio.subprocess.PIPE,
                     stderr=asyncio.subprocess.PIPE,
                 )
-                await proc.communicate()
+                add_stdout, add_stderr = await proc.communicate()
+                if proc.returncode != 0:
+                    detail = (add_stderr or add_stdout).decode(errors="replace").strip()
+                    return {"commit_error": f"git add failed: {detail or 'unknown error'}"}
+
+                mode_change_error = await self._pure_file_mode_change_error(
+                    workspace_path, "--cached"
+                )
+                if mode_change_error:
+                    return {"commit_error": mode_change_error}
 
                 # Collect staged diff stats for rich commit message
                 change_summary = await self._collect_staged_diff_stats(workspace_path)
@@ -9010,7 +9127,10 @@ class RuntimeDaemon:
                          "GIT_AUTHOR_NAME": _git_name, "GIT_AUTHOR_EMAIL": _git_email,
                          "GIT_COMMITTER_NAME": _git_name, "GIT_COMMITTER_EMAIL": _git_email},
                 )
-                await proc.communicate()
+                commit_stdout, commit_stderr = await proc.communicate()
+                if proc.returncode != 0:
+                    detail = (commit_stderr or commit_stdout).decode(errors="replace").strip()
+                    return {"commit_error": f"git commit failed: {detail or 'unknown error'}"}
                 has_new_commit = True
                 logger.info("Auto-committed changes in %s", workspace_path)
             else:

{forgexa_cli-1.18.2 → forgexa_cli-1.18.3}/forgexa_cli.egg-info/PKG-INFO

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

{forgexa_cli-1.18.2 → forgexa_cli-1.18.3}/pyproject.toml

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