forgexa-cli 1.14.7__tar.gz → 1.14.8__tar.gz

{forgexa_cli-1.14.7 → forgexa_cli-1.14.8}/PKG-INFO

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

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

{forgexa_cli-1.14.7 → forgexa_cli-1.14.8}/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.14.7"
+DAEMON_VERSION = "1.14.8"
 
 
 def _detect_client_type() -> str:
@@ -1711,17 +1711,27 @@ class WorkspaceManager:
 
     @staticmethod
     def _safe_rmtree(path: Path) -> None:
-        """Remove a directory tree, handling read-only files on Windows.
-
-        On Windows, git object files (.git/objects/**/*) are stored read-only
-        (mode 0444).  Plain shutil.rmtree(path, ignore_errors=True) silently
-        skips those files, leaving the directory intact.  A subsequent
-        git clone then fails with 'destination path already exists and is
-        not an empty directory'.
-
-        This method uses the onerror callback to chmod each entry writable
-        before retrying — the established pattern for handling git repos on
-        Windows.
+        """Remove a directory tree, handling two distinct Windows failure modes.
+
+        Mode 1 — Read-only git objects (chmod 0444):
+          Plain shutil.rmtree(path, ignore_errors=True) silently skips
+          read-only files.  Fixed by the onerror callback that chmod’s each
+          entry writable before retrying.
+
+        Mode 2 — Windows reserved device names (NUL, CON, AUX, PRN,
+          COM1–COM9, LPT1–LPT9):
+          The Win32 API intercepts these names before they reach the NTFS
+          driver, so os.unlink / shutil.rmtree / PowerShell Remove-Item all
+          fail with a permissions or access error.  The solution is the
+          \\?\ extended-length path prefix, which instructs Windows to
+          skip Win32 name parsing and pass the path directly to the NT
+          kernel file-system layer.  'cmd /c rmdir /s /q' reliably handles
+          this when given a \\?\ prefixed path.
+
+        Deletion strategy (each stage only runs if the path still exists):
+          1. shutil.rmtree with onerror chmod  — handles read-only objects.
+          2. Windows only: subprocess cmd /c rmdir /s /q \\\\?\\<path>  —
+             handles reserved names and any entries missed by stage 1.
         """
         import shutil
         import stat as _stat
@@ -1738,6 +1748,39 @@ class WorkspaceManager:
 
         shutil.rmtree(path, onerror=_handle_readonly)
 
+        # Windows fallback: reserved device names survive shutil.rmtree.
+        # cmd /c rmdir /s /q with the \\?\ extended-length prefix bypasses
+        # Win32 name interception and reaches the NTFS layer directly.
+        if path.exists() and sys.platform == "win32":
+            try:
+                import subprocess
+                resolved = str(path.resolve())
+                # Prepend \\?\ extended-length prefix if not already present.
+                # In Python source "\\\\?\\" represents the 4-char string \\?\
+                # which is the Win32 flag to bypass reserved-name interception.
+                if not resolved.startswith("\\\\?\\"):
+                    unc = "\\\\?\\" + resolved
+                else:
+                    unc = resolved
+                result = subprocess.run(
+                    ["cmd", "/c", "rmdir", "/s", "/q", unc],
+                    capture_output=True,
+                    timeout=60,
+                )
+                if result.returncode != 0 and path.exists():
+                    logger.warning(
+                        "_safe_rmtree: cmd rmdir could not fully remove %s "
+                        "(exit %d: %s). Manual deletion may be required.",
+                        path,
+                        result.returncode,
+                        result.stderr.decode(errors="replace").strip()[:200],
+                    )
+            except Exception as exc:
+                logger.warning(
+                    "_safe_rmtree: Windows rmdir fallback failed for %s: %s",
+                    path, exc,
+                )
+
     async def _clone_repo(
         self,
         repo_url: str,
@@ -4616,6 +4659,18 @@ class ProcessManager:
                 *cmd,
                 stdout=asyncio.subprocess.PIPE,
                 stderr=asyncio.subprocess.PIPE,
+                # stdin=DEVNULL is critical on Windows.
+                # Without it, the child process inherits the parent daemon's stdin.
+                # On Windows, the daemon often has a real console handle on stdin
+                # (e.g. from a CMD/PowerShell window or the Desktop app's console).
+                # Copilot CLI detects this via GetConsoleMode(GetStdHandle(STD_INPUT))
+                # and enters interactive/session-check mode, ignoring -p as a task:
+                # it checks the session todo table (empty), reports "no tracked work
+                # items", and waits for user input instead of executing the task.
+                # On Linux/macOS (daemon as service), stdin is typically /dev/null so
+                # the bug doesn't manifest.  Explicit DEVNULL ensures non-interactive
+                # mode on all platforms, matching the intended headless operation.
+                stdin=asyncio.subprocess.DEVNULL,
                 cwd=str(cwd),
                 env=env,
                 limit=100 * 1024 * 1024,

{forgexa_cli-1.14.7 → forgexa_cli-1.14.8}/forgexa_cli.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: forgexa-cli
-Version: 1.14.7
+Version: 1.14.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.14.7 → forgexa_cli-1.14.8}/pyproject.toml

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