pi-sync-cli 0.2.0__tar.gz → 0.3.0__tar.gz

{pi_sync_cli-0.2.0 → pi_sync_cli-0.3.0}/.gitignore

@@ -216,3 +216,6 @@ __marimo__/
 
 # Streamlit
 .streamlit/secrets.toml
+
+# pi background task artifacts
+.pi/tasks/

{pi_sync_cli-0.2.0 → pi_sync_cli-0.3.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.5
 Name: pi-sync-cli
-Version: 0.2.0
+Version: 0.3.0
 Summary: Sync pi agent config and extensions between hosts over rsync
 Project-URL: Repository, https://github.com/say4n/pi-sync
 Project-URL: Issues, https://github.com/say4n/pi-sync/issues
@@ -63,6 +63,7 @@ Host-local state is deliberately never touched: `sessions/`, `npm/`,
 | `--dry-run` | report changes, copy nothing |
 | `-x, --exclude PATTERN` | skip matching files (repeatable) |
 | `--install` | install pi on hosts that lack it, without prompting |
+| `--uninstall` | remove pi from the host instead of syncing (config is kept) |
 | `--local-dir` | default `$PI_CODING_AGENT_DIR` or `~/.pi/agent` |
 | `--remote-dir` | default `~/.pi/agent` |
 | `-v` / `--verbose` | print each rsync command and its output |
@@ -98,6 +99,21 @@ The probe checks `command -v pi` plus the usual install locations
 `/usr/local/bin`), because a non-interactive ssh session does not source the
 host's shell init — on a linuxbrew host `command -v pi` alone misses it.
 
+## Uninstalling
+
+`--uninstall` removes pi from the host instead of syncing. It runs
+`npm uninstall -g --prefix <prefix> @earendil-works/pi-coding-agent`, deriving the
+prefix from where pi actually lives (as the official installer does), then
+re-probes to confirm the binary is really gone — npm can exit 0 having removed
+nothing. `~/.pi/agent` is deliberately left untouched, since that is your config
+rather than the CLI.
+
+This exists because the official installer can only uninstall through its
+interactive menu: its unattended mode always installs or reinstalls. If the
+uninstall fails, that is usually a managed install
+(`PI_EXPERIMENTAL=1`, under `<agent dir>/install`) — run
+`curl -fsSL https://pi.dev/install.sh | sh` on the host and choose `u`.
+
 ## Shell completions
 
 Host arguments complete from `~/.ssh/config`, following `Include` directives and

{pi_sync_cli-0.2.0 → pi_sync_cli-0.3.0}/README.md

@@ -46,6 +46,7 @@ Host-local state is deliberately never touched: `sessions/`, `npm/`,
 | `--dry-run` | report changes, copy nothing |
 | `-x, --exclude PATTERN` | skip matching files (repeatable) |
 | `--install` | install pi on hosts that lack it, without prompting |
+| `--uninstall` | remove pi from the host instead of syncing (config is kept) |
 | `--local-dir` | default `$PI_CODING_AGENT_DIR` or `~/.pi/agent` |
 | `--remote-dir` | default `~/.pi/agent` |
 | `-v` / `--verbose` | print each rsync command and its output |
@@ -81,6 +82,21 @@ The probe checks `command -v pi` plus the usual install locations
 `/usr/local/bin`), because a non-interactive ssh session does not source the
 host's shell init — on a linuxbrew host `command -v pi` alone misses it.
 
+## Uninstalling
+
+`--uninstall` removes pi from the host instead of syncing. It runs
+`npm uninstall -g --prefix <prefix> @earendil-works/pi-coding-agent`, deriving the
+prefix from where pi actually lives (as the official installer does), then
+re-probes to confirm the binary is really gone — npm can exit 0 having removed
+nothing. `~/.pi/agent` is deliberately left untouched, since that is your config
+rather than the CLI.
+
+This exists because the official installer can only uninstall through its
+interactive menu: its unattended mode always installs or reinstalls. If the
+uninstall fails, that is usually a managed install
+(`PI_EXPERIMENTAL=1`, under `<agent dir>/install`) — run
+`curl -fsSL https://pi.dev/install.sh | sh` on the host and choose `u`.
+
 ## Shell completions
 
 Host arguments complete from `~/.ssh/config`, following `Include` directives and

{pi_sync_cli-0.2.0 → pi_sync_cli-0.3.0}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "pi-sync-cli"
-version = "0.2.0"
+version = "0.3.0"
 description = "Sync pi agent config and extensions between hosts over rsync"
 readme = "README.md"
 requires-python = ">=3.10"

{pi_sync_cli-0.2.0 → pi_sync_cli-0.3.0}/src/pi_sync/cli.py

@@ -24,6 +24,7 @@ from click.shell_completion import CompletionItem
 DEFAULT_AGENT_DIR = "~/.pi/agent"
 SSH_CONFIG = "~/.ssh/config"
 PI_INSTALL_CMD = "curl -fsSL https://pi.dev/install.sh | sh"
+PI_PACKAGE = "@earendil-works/pi-coding-agent"
 # One ssh round trip answers both "is it reachable" and "where is pi".
 PI_PROBE = """\
 found="$(command -v pi 2>/dev/null)"
@@ -195,6 +196,30 @@ def install_pi(target: str) -> str | None:
     return None
 
 
+def uninstall_pi(target: str, pi_path: str) -> str | None:
+    """Remove pi's npm install. Returns an error message, or None on success.
+
+    Mirrors the installer's own npm path — the prefix comes from where pi
+    actually lives — because its unattended mode can only install or reinstall;
+    uninstall is reachable only through its interactive menu. Like the
+    installer, the result is verified rather than trusted, and only the CLI is
+    removed: the agent directory is left alone.
+    """
+    prefix = Path(pi_path).parent.parent
+    cmd = (
+        f"npm uninstall -g --prefix {prefix} --no-fund --no-audit "
+        f"--loglevel=error --progress=false {PI_PACKAGE}"
+    )
+    proc = run_cmd(["ssh", target, cmd])
+    if proc.returncode:
+        detail = ((proc.stderr or proc.stdout) or "").strip().splitlines()
+        return detail[-1] if detail else f"npm uninstall exited {proc.returncode}"
+    _, remaining = probe_host(target)
+    if remaining:
+        return f"npm uninstall finished, but pi is still present at {remaining}"
+    return None
+
+
 def ensure_pi(target: str, assume_yes: bool, remote_dir: str) -> bool:
     """Make sure the host has pi. Returns False when the host must be skipped.
 
@@ -327,6 +352,12 @@ def sync_host(
     is_flag=True,
     help="Install pi on hosts that lack it, without prompting.",
 )
+@click.option(
+    "--uninstall",
+    "uninstall_",
+    is_flag=True,
+    help="Uninstall pi from the host instead of syncing (keeps ~/.pi/agent).",
+)
 @click.option(
     "--local-dir",
     default=None,
@@ -352,6 +383,7 @@ def main(
     dry_run: bool,
     excludes: tuple[str, ...],
     install_: bool,
+    uninstall_: bool,
     local_dir: str | None,
     remote_dir: str,
     verbose: bool,
@@ -397,6 +429,28 @@ def main(
             click.secho(f"→ {target}\n  unreachable: {error}", fg="red")
             continue
         click.secho(f"→ {target}", bold=True)
+        if uninstall_:
+            if pi_path is None:
+                click.secho("  pi is not installed — nothing to uninstall")
+                continue
+            prefix = Path(pi_path).parent.parent
+            if dry_run:
+                click.secho(
+                    f"  would run: npm uninstall -g --prefix {prefix} {PI_PACKAGE}"
+                )
+                continue
+            click.secho(f"  uninstalling pi from {pi_path}...")
+            error = uninstall_pi(target, pi_path)
+            if error:
+                failed = True
+                click.secho(f"  {error}", fg="red")
+                click.secho(
+                    "  for a managed install, run the installer and choose 'u'",
+                    fg="yellow",
+                )
+            else:
+                click.secho("  pi uninstalled — ~/.pi/agent was left alone", fg="green")
+            continue
         if pi_path is None:
             if dry_run:
                 click.secho("  pi not installed — nothing would be synced", fg="yellow")

{pi_sync_cli-0.2.0 → pi_sync_cli-0.3.0}/tests/test_cli.py

@@ -29,6 +29,8 @@ class FakeRun:
         pi_path: str | None = "/usr/bin/pi",
         install_rc: int = 0,
         install_installs: bool = True,
+        uninstall_rc: int = 0,
+        uninstall_removes: bool = True,
     ):
         self.calls: list[list[str]] = []
         self.inputs: list[str | None] = []
@@ -39,6 +41,8 @@ class FakeRun:
         self.pi_path = pi_path
         self.install_rc = install_rc
         self.install_installs = install_installs
+        self.uninstall_rc = uninstall_rc
+        self.uninstall_removes = uninstall_removes
         self.installed = False
 
     def __call__(
@@ -51,6 +55,13 @@ class FakeRun:
             return subprocess.CompletedProcess(
                 argv, self.rsync_rc, self.rsync_stdout, ""
             )
+        if any("npm uninstall" in arg for arg in argv):
+            if self.uninstall_rc == 0 and self.uninstall_removes:
+                self.pi_path = None
+            stderr = "" if self.uninstall_rc == 0 else "npm ERR! code E404"
+            return subprocess.CompletedProcess(
+                argv, self.uninstall_rc, "removed\n", stderr
+            )
         if any("install.sh" in arg for arg in argv):
             if self.install_rc == 0:
                 # a real installer exits 0 even when the user picks "do nothing"
@@ -550,3 +561,72 @@ class TestInstallOffering:
         )
         assert result.exit_code == 1
         assert install_calls(fake) == []
+
+
+class TestUninstall:
+    def uninstall_calls(self, fake: FakeRun) -> list[list[str]]:
+        return [c for c in fake.calls if any("npm uninstall" in a for a in c)]
+
+    def test_uninstalls_with_the_prefix_pi_lives_in(
+        self, monkeypatch: pytest.MonkeyPatch, agent_dir: Path
+    ) -> None:
+        fake = FakeRun(pi_path="/home/linuxbrew/.linuxbrew/bin/pi")
+        monkeypatch.setattr(cli, "run_cmd", fake)
+        result = CliRunner().invoke(cli.main, ["--uninstall", "host"])
+        assert result.exit_code == 0, result.output
+        (call,) = self.uninstall_calls(fake)
+        cmd = call[-1]
+        assert "npm uninstall -g --prefix /home/linuxbrew/.linuxbrew" in cmd
+        assert cli.PI_PACKAGE in cmd
+        assert "~/.pi/agent was left alone" in result.output
+        # uninstalling never syncs
+        assert not any(c[0] == "rsync" for c in fake.calls)
+
+    def test_absent_pi_is_a_noop(
+        self, monkeypatch: pytest.MonkeyPatch, agent_dir: Path
+    ) -> None:
+        fake = FakeRun(pi_path=None)
+        monkeypatch.setattr(cli, "run_cmd", fake)
+        result = CliRunner().invoke(cli.main, ["--uninstall", "host"])
+        assert result.exit_code == 0
+        assert "nothing to uninstall" in result.output
+        assert self.uninstall_calls(fake) == []
+
+    def test_npm_failure_is_reported(
+        self, monkeypatch: pytest.MonkeyPatch, agent_dir: Path
+    ) -> None:
+        fake = FakeRun(pi_path="/usr/bin/pi", uninstall_rc=1)
+        monkeypatch.setattr(cli, "run_cmd", fake)
+        result = CliRunner().invoke(cli.main, ["--uninstall", "host"])
+        assert result.exit_code == 1
+        assert "npm ERR!" in result.output
+
+    def test_still_present_after_uninstall_is_a_failure(
+        self, monkeypatch: pytest.MonkeyPatch, agent_dir: Path
+    ) -> None:
+        """npm can exit 0 having removed nothing; the re-probe has the last word."""
+        fake = FakeRun(pi_path="/usr/bin/pi", uninstall_removes=False)
+        monkeypatch.setattr(cli, "run_cmd", fake)
+        result = CliRunner().invoke(cli.main, ["--uninstall", "host"])
+        assert result.exit_code == 1
+        assert "still present at /usr/bin/pi" in result.output
+
+    def test_dry_run_uninstall_changes_nothing(
+        self, monkeypatch: pytest.MonkeyPatch, agent_dir: Path
+    ) -> None:
+        fake = FakeRun(pi_path="/usr/bin/pi")
+        monkeypatch.setattr(cli, "run_cmd", fake)
+        result = CliRunner().invoke(cli.main, ["--uninstall", "--dry-run", "host"])
+        assert result.exit_code == 0
+        assert "would run: npm uninstall" in result.output
+        assert self.uninstall_calls(fake) == []
+
+    def test_unreachable_host_fails(
+        self, monkeypatch: pytest.MonkeyPatch, agent_dir: Path
+    ) -> None:
+        fake = FakeRun(ssh_rc=255)
+        monkeypatch.setattr(cli, "run_cmd", fake)
+        result = CliRunner().invoke(cli.main, ["--uninstall", "host"])
+        assert result.exit_code == 1
+        assert "unreachable" in result.output
+        assert self.uninstall_calls(fake) == []

{pi_sync_cli-0.2.0 → pi_sync_cli-0.3.0}/uv.lock

@@ -52,7 +52,7 @@ wheels = [
 
 [[package]]
 name = "pi-sync-cli"
-version = "0.2.0"
+version = "0.3.0"
 source = { editable = "." }
 dependencies = [
     { name = "click" },