forgexa-cli 1.26.0__tar.gz → 1.26.2__tar.gz

{forgexa_cli-1.26.0 → forgexa_cli-1.26.2}/PKG-INFO

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

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

{forgexa_cli-1.26.0 → forgexa_cli-1.26.2}/forgexa_cli/daemon.py

@@ -909,7 +909,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.26.0"
+DAEMON_VERSION = "1.26.2"
 
 
 def _detect_client_type() -> str:
@@ -971,6 +971,14 @@ logger = logging.getLogger("daemon")
 logger.info("Daemon log: %s", DAEMON_LOG_PATH)
 
 
+def _format_exc(exc: BaseException) -> str:
+    """Render an exception for logging, always including its type name so a
+    str()-less exception (e.g. an empty httpx/asyncio timeout) never logs as
+    a blank, undiagnosable "... error: " line."""
+    detail = str(exc)
+    return f"{type(exc).__name__}: {detail}" if detail else type(exc).__name__
+
+
 # ── Hardware ID — stable cross-IP machine fingerprint ──
 
 
@@ -6776,7 +6784,7 @@ class HeartbeatService:
                 else:
                     logger.warning("Heartbeat failed: %s %s", resp.status_code, resp.text)
             except Exception as e:
-                logger.warning("Heartbeat error: %s", e)
+                logger.warning("Heartbeat error: %s", _format_exc(e))
 
             await asyncio.sleep(self.interval)
 
@@ -6940,7 +6948,7 @@ class TaskPoller:
                 ))
             return tasks
         except Exception as e:
-            logger.warning("Task poll error: %s", e)
+            logger.warning("Task poll error: %s", _format_exc(e))
             return []
 
     async def poll_ai_jobs(self) -> list[dict]:
@@ -6964,7 +6972,7 @@ class TaskPoller:
             self.cancelled_ai_job_ids = []
             return []
         except Exception as e:
-            logger.debug("AIJob poll error: %s", e)
+            logger.debug("AIJob poll error: %s", _format_exc(e))
             self.cancelled_ai_job_ids = []
             return []
 

{forgexa_cli-1.26.0 → forgexa_cli-1.26.2}/forgexa_cli/main.py

@@ -2859,6 +2859,20 @@ def cmd_project_bind_local(args: argparse.Namespace) -> None:
         sys.exit(1)
 
     mapping = load_local_workspaces_mapping()
+    local_path_str = str(local_path)
+    # A folder can only ever be validly bound to one binding_id at a time -
+    # drop any other (necessarily stale) entry already pointing at this same
+    # path first. Without this, a stale entry left behind by an earlier
+    # attempt (e.g. a re-registered runtime after a reinstall, or a
+    # verification that timed out client-side) can linger alongside the new
+    # one, and agent_host.py's `_match_local_binding_id` (a plain path scan
+    # with no tie-breaking) can resolve to the wrong, orphaned binding_id
+    # instead of the one just verified.
+    mapping = {
+        key: entry
+        for key, entry in mapping.items()
+        if key == binding_id or entry.get("local_path") != local_path_str
+    }
     mapping[binding_id] = {
         "project_id": args.project,
         "user_id": user_id,

{forgexa_cli-1.26.0 → forgexa_cli-1.26.2}/forgexa_cli.egg-info/PKG-INFO

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

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

{forgexa_cli-1.26.0 → forgexa_cli-1.26.2}/tests/test_local_bind_commands.py

@@ -70,6 +70,64 @@ def test_bind_local_happy_path(tmp_path: Path, monkeypatch: pytest.MonkeyPatch)
     }
 
 
+def test_bind_local_replaces_stale_entry_for_same_local_path(tmp_path: Path, monkeypatch: pytest.MonkeyPatch) -> None:
+    """A stale entry from an earlier attempt (e.g. before a daemon/runtime
+    reinstall) must not linger alongside a freshly-verified one for the same
+    folder - agent_host.py's _match_local_binding_id does a plain path scan
+    with no tie-breaking, so a leftover duplicate can resolve to the wrong,
+    orphaned binding_id and fail turn.start with binding_verification_required
+    even though the IDE just linked successfully."""
+    monkeypatch.setenv("HOME", str(tmp_path))
+    repo = tmp_path / "my-real-project"
+    _init_git_repo_with_remote(repo)
+
+    mapping_path = tmp_path / ".forgexa" / "local-workspaces.json"
+    mapping_path.parent.mkdir(parents=True, exist_ok=True)
+    mapping_path.write_text(json.dumps({
+        "binding-old": {
+            "project_id": "project-1",
+            "user_id": "user-1",
+            "runtime_id": "runtime-old",
+            "repo_identity": "github.com/acme/widgets",
+            "local_path": str(repo.resolve()),
+        },
+    }))
+
+    monkeypatch.setattr(main, "_get_runtimes", lambda: [
+        {"id": "runtime-new", "pool_type": "personal", "daemon_id": "d2", "device_name": "laptop"},
+    ])
+
+    def fake_get(path: str):
+        if path == "/auth/me":
+            return {"id": "user-1"}
+        assert path == "/projects/project-1/local-workspace-bindings/binding-new"
+        return {"id": "binding-new", "status": "active"}
+
+    def fake_post(path: str, data=None):
+        if path.endswith("/local-workspace-bindings"):
+            return {"id": "binding-new", "status": "pending"}
+        if path.endswith("/verify"):
+            return {"id": "binding-new", "status": "active"}
+        raise AssertionError(f"unexpected POST {path}")
+
+    monkeypatch.setattr(main, "_get", fake_get)
+    monkeypatch.setattr(main, "_post", fake_post)
+
+    args = SimpleNamespace(project="project-1", path=str(repo), runtime=None)
+    main.cmd_project_bind_local(args)
+
+    mapping = json.loads(mapping_path.read_text())
+    assert mapping == {
+        "binding-new": {
+            "project_id": "project-1",
+            "user_id": "user-1",
+            "runtime_id": "runtime-new",
+            "repo_identity": "github.com/acme/widgets",
+            "local_path": str(repo.resolve()),
+        },
+    }
+
+
 def test_bind_local_rejects_non_git_directory(tmp_path: Path, monkeypatch: pytest.MonkeyPatch) -> None:
     monkeypatch.setenv("HOME", str(tmp_path))
     plain_dir = tmp_path / "not-a-repo"