agent-toggle 0.7.0__tar.gz → 0.7.3__tar.gz

{agent_toggle-0.7.0 → agent_toggle-0.7.3}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: agent-toggle
-Version: 0.7.0
+Version: 0.7.3
 Summary: Temporarily disable / re-enable AI-agent skills, agents, commands, rules, plugins and MCP servers across harnesses (Claude Code, Codex, Grok, OpenCode, OpenClaw, Copilot, Vibe, Devin, Antigravity)
 License-Expression: MIT
 Project-URL: Homepage, https://github.com/weskao/agent-toggle
@@ -855,6 +855,10 @@ ruff check .
 stdlib `unittest`, no fixtures, no network. Every test runs against a
 throwaway temp `HOME` and a stubbed `claude` CLI.
 
+`python3 tools/wheel_smoke.py` builds the wheel, installs it without extras into a
+fresh venv (what a PyPI user gets) and runs the CLI against a temp `HOME`; CI runs
+it on macOS, Linux and Windows.
+
 ## CI notifications
 
 `.github/workflows/ci.yml` can send a Telegram message when the test matrix fails on a

{agent_toggle-0.7.0 → agent_toggle-0.7.3}/README.md

@@ -823,6 +823,10 @@ ruff check .
 stdlib `unittest`, no fixtures, no network. Every test runs against a
 throwaway temp `HOME` and a stubbed `claude` CLI.
 
+`python3 tools/wheel_smoke.py` builds the wheel, installs it without extras into a
+fresh venv (what a PyPI user gets) and runs the CLI against a temp `HOME`; CI runs
+it on macOS, Linux and Windows.
+
 ## CI notifications
 
 `.github/workflows/ci.yml` can send a Telegram message when the test matrix fails on a

{agent_toggle-0.7.0 → agent_toggle-0.7.3}/agent_toggle/__init__.py

@@ -1,3 +1,3 @@
 """agent-toggle: temporarily disable / re-enable AI-agent resources."""
 
-__version__ = "0.7.0"
+__version__ = "0.7.3"

{agent_toggle-0.7.0 → agent_toggle-0.7.3}/agent_toggle/backends/plugin_cli.py

@@ -28,7 +28,8 @@ def claude_bin() -> str | None:
     if found:
         return found
     home = fs.home()
-    for cand in (home / ".local/bin/claude", home / ".claude/local/claude"):
+    for cand in (home / ".local/bin/claude", home / ".local/bin/claude.exe",   # .exe: Windows
+                 home / ".claude/local/claude"):
         if cand.is_file() and os.access(cand, os.X_OK):
             return str(cand)
     return None

{agent_toggle-0.7.0 → agent_toggle-0.7.3}/agent_toggle/fs.py

@@ -416,6 +416,9 @@ def safe_move(src: Path, dest_dir: Path) -> Path:
     if target.exists() or target.is_symlink():
         raise FileExistsError(f"{target} already exists -- refusing to overwrite")
 
+    if src.is_symlink() and link_text(src, dest_dir) != os.readlink(src):
+        _copy_move(src, target)         # a rename keeps the relative text, now dangling
+        return target
     try:
         os.rename(src, target)                      # same filesystem: one atomic step
     except OSError as e:
@@ -425,6 +428,17 @@ def safe_move(src: Path, dest_dir: Path) -> Path:
     return target
 
 
+def link_text(src: Path, dest_dir: Path) -> str:
+    """The symlink `src`'s text as seen from `dest_dir`: a relative link keeps its target,
+    not its text, since parks sit at another depth (skills/ vs parked/user/<h>/skills/)."""
+    link = os.readlink(src)
+    if os.path.isabs(link):
+        return link
+    # the kernel resolves a relative link from the link's physical dir
+    target = os.path.normpath(os.path.join(os.path.realpath(src.parent), link))
+    return os.path.relpath(target, os.path.realpath(dest_dir))
+
+
 def move_leftovers(src: Path, target: Path) -> tuple[Path, Path]:
     """(half copy, trash) a killed cross-fs move can leave beside `target` / `src`."""
     return (target.with_name(f".{target.name}.agent-toggle-tmp"),
@@ -477,7 +491,7 @@ def _copy_move(src: Path, target: Path) -> None:
     remove_leftover(tmp)                 # a killed earlier attempt's half copy
     try:
         if src.is_symlink():
-            os.symlink(os.readlink(src), tmp)
+            os.symlink(link_text(src, target.parent), tmp)
         elif src.is_dir():
             shutil.copytree(src, tmp, symlinks=True)
         else:

{agent_toggle-0.7.0 → agent_toggle-0.7.3}/agent_toggle/mechanisms.py

@@ -360,6 +360,31 @@ def migrate_parks(state: dict, table: dict | None = None) -> list[str]:
         lines.append(f"moved {len(moved)} item(s) {old} -> {v.parked}"
                      + (f", repointed {repointed} state entry(ies)" if repointed else "")
                      + ("" if _exists(old) else f", removed {old}"))
+    return lines + repair_parked_links(table)
+
+
+def repair_parked_links(table: dict) -> list[str]:
+    """0.7.0's migrate moved relative symlinks with their text unchanged, so they dangle
+    from the deeper park dir; re-aim each one whose text still resolves from its live dir."""
+    lines, seen = [], set()
+    for hn, h in table.items():
+        for type_, subs in h.dirs.items():
+            for sub in subs:
+                v = dir_view(table, hn, type_, h.home, sub)
+                if v.parked in seen or not v.parked.is_dir() or v.parked.is_symlink():
+                    continue
+                seen.add(v.parked)
+                for p in sorted(v.parked.iterdir()):
+                    if not p.is_symlink() or p.exists() or os.path.isabs(link := os.readlink(p)):
+                        continue
+                    target = os.path.normpath(os.path.join(os.path.realpath(v.live), link))
+                    if not os.path.exists(target):
+                        continue                 # dangles from its live dir too: not ours
+                    tmp = fs.move_leftovers(p, p)[0]
+                    fs.remove_leftover(tmp)
+                    os.symlink(os.path.relpath(target, os.path.realpath(v.parked)), tmp)
+                    fs.replace(tmp, p)           # atomic: the link is never missing
+                    lines.append(f"re-aimed dangling link {p} -> {target}")
     return lines
 
 

{agent_toggle-0.7.0 → agent_toggle-0.7.3}/agent_toggle/ui/picker.py

@@ -201,22 +201,18 @@ def heading(type_: str, count: int, width: int, g: theme.Glyphs) -> list[theme.S
     return [(label, "title"), (str(count), "muted"), (" " + g.h * max(0, rest), "muted")]
 
 
-def detail_lines(r: Row, width: int, g: theme.Glyphs) -> list[list[theme.Segment]]:
-    """The detail pane for the cursor row: one segment list per screen line."""
+def detail_lines(r: Row, width: int, g: theme.Glyphs,
+                 height: int | None = None) -> list[list[theme.Segment]]:
+    """The detail pane for the cursor row: one segment list per screen line. With `height`,
+    the description is cut (with an ellipsis) so the fields below it still fit."""
     lab = 12                                 # "description" is 11 cells; leave a gap
     val = max(1, width - lab)
     out = [[(theme.truncate(r.name, width, g.ellipsis), "title")], []]
+    rest: list[list[theme.Segment]] = []
 
     def field(label: str, text: str, role: str = "text") -> None:
         for n in range(0, max(1, len(text)), val):          # a long path wraps
-            out.append([(pad(label if n == 0 else "", lab), "muted"), (text[n:n + val], role)])
-
-    if r.description:                        # frontmatter prose, not a path
-        chunks = wrap_cells(r.description, val)
-        label = t("picker.d.description", "description")
-        for n, chunk in enumerate(chunks):
-            out.append([(pad(label if n == 0 else "", lab), "muted"), (chunk, "text")])
-        out.append([])
+            rest.append([(pad(label if n == 0 else "", lab), "muted"), (text[n:n + val], role)])
 
     state = t("picker.state.live", "live") if r.enabled else t("picker.state.parked", "parked")
     field(t("picker.d.harness", "harness"), r.harness, theme.harness_role(r.harness))
@@ -242,7 +238,20 @@ def detail_lines(r: Row, width: int, g: theme.Glyphs) -> list[list[theme.Segment
         field(t("picker.d.since", "since"), r.since[:19].replace("T", " "))
     if r.mechanism:
         field(t("picker.d.mechanism", "mechanism"), r.mechanism)
-    return out
+
+    if r.description:                        # frontmatter prose, not a path
+        chunks = wrap_cells(r.description, val)
+        # the "staged" line comes and goes with Space; reserving it keeps the text from jumping
+        spare = len(rest) + (not r.changed)
+        room = len(chunks) if height is None else max(1, height - len(out) - spare - 1)
+        if len(chunks) > room:
+            chunks = chunks[:room - 1] + [theme.truncate(" ".join(chunks[room - 1:]), val,
+                                                         g.ellipsis)]
+        label = t("picker.d.description", "description")
+        for n, chunk in enumerate(chunks):
+            out.append([(pad(label if n == 0 else "", lab), "muted"), (chunk, "text")])
+        out.append([])
+    return out + rest
 
 
 def chips(v: View, g: theme.Glyphs) -> list[tuple[str, str]]:
@@ -322,7 +331,7 @@ def draw(win, v: View, shown: list[Row], pal: theme.Palette, g: theme.Glyphs,
         else:
             put(y, row_segments(line[1], lw, v, g, line[1] is cursor_row, any_shared))
     if dw:
-        pane = detail_lines(cursor_row, dw - 2, g) if cursor_row else []
+        pane = detail_lines(cursor_row, dw - 2, g, body) if cursor_row else []
         for i in range(body):
             put(BODY_TOP + i, [(g.v, "muted"), (" ", "text"), *(pane[i] if i < len(pane) else [])],
                 lw)

{agent_toggle-0.7.0 → agent_toggle-0.7.3}/agent_toggle.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: agent-toggle
-Version: 0.7.0
+Version: 0.7.3
 Summary: Temporarily disable / re-enable AI-agent skills, agents, commands, rules, plugins and MCP servers across harnesses (Claude Code, Codex, Grok, OpenCode, OpenClaw, Copilot, Vibe, Devin, Antigravity)
 License-Expression: MIT
 Project-URL: Homepage, https://github.com/weskao/agent-toggle
@@ -855,6 +855,10 @@ ruff check .
 stdlib `unittest`, no fixtures, no network. Every test runs against a
 throwaway temp `HOME` and a stubbed `claude` CLI.
 
+`python3 tools/wheel_smoke.py` builds the wheel, installs it without extras into a
+fresh venv (what a PyPI user gets) and runs the CLI against a temp `HOME`; CI runs
+it on macOS, Linux and Windows.
+
 ## CI notifications
 
 `.github/workflows/ci.yml` can send a Telegram message when the test matrix fails on a

{agent_toggle-0.7.0 → agent_toggle-0.7.3}/tests/test_fs.py

@@ -38,6 +38,23 @@ class SafeMoveTest(SandboxCase):
         # existing parked copy untouched
         self.assertEqual((self.home / "skills-disabled/a/SKILL.md").read_text(encoding="utf-8"), "old")
 
+    @unittest.skipUnless(CAN_SYMLINK, "cannot create symlinks here")
+    def test_relative_link_keeps_its_target_across_depths(self) -> None:
+        real = self.write(".repos/r/demo/SKILL.md", "hi").parent
+        link = self.home / "skills" / "demo"
+        link.parent.mkdir(parents=True, exist_ok=True)
+        link.symlink_to("../.repos/r/demo")
+        parked = fs.safe_move(link, fs.user_park("claude", "skills"))
+        self.assertEqual(parked.resolve(), real.resolve())        # not dangling while parked
+        back = fs.safe_move(parked, self.home / "skills")
+        # the round trip is exact (Windows re-aims with native separators)
+        self.assertEqual(os.path.normpath(os.readlink(back)), os.path.normpath("../.repos/r/demo"))
+        absolute = self.home / "skills" / "abs"
+        absolute.symlink_to(real)
+        text = os.readlink(absolute)                              # Windows may add \\?\
+        self.assertEqual(os.readlink(fs.safe_move(absolute, fs.user_park("claude", "skills"))),
+                         text)                                    # absolute: untouched
+
 
 POSIX = os.name != "nt"
 

{agent_toggle-0.7.0 → agent_toggle-0.7.3}/tests/test_picker.py

@@ -465,6 +465,19 @@ class LayoutTest(ScreenCase):
         self.assertIn("alpha beta", body)
         self.assertIn("gamma delta", body)
 
+    def test_long_description_is_cut_so_the_fields_still_show(self) -> None:
+        r = model.Row("claude", "skill", "solo", True, description="word " * 200,
+                      mechanism="move")
+        lines = picker.detail_lines(r, 30, theme.ASCII, height=12)
+        self.assertLessEqual(len(lines), 12)
+        flat = ["".join(s for s, _ in line) for line in lines]
+        self.assertTrue(any(line.rstrip().endswith("...") for line in flat))
+        self.assertTrue(any(line.startswith("mechanism") for line in flat))
+        r.staged = False                     # toggling adds the "staged" line, not a cut
+        toggled = picker.detail_lines(r, 30, theme.ASCII, height=12)
+        self.assertLessEqual(len(toggled), 12)
+        self.assertEqual(lines[:8], toggled[:8])
+
     def test_detail_pane_description_is_translated(self) -> None:
         self.addCleanup(i18n.set_language, i18n.LANGUAGE)
         i18n.set_language("zh-TW")

{agent_toggle-0.7.0 → agent_toggle-0.7.3}/tests/test_platform.py

@@ -11,6 +11,7 @@ from base import SandboxCase
 
 from agent_toggle import fs
 from agent_toggle import mechanisms as mech
+from agent_toggle.backends import plugin_cli
 
 
 class PluginIdTest(SandboxCase):
@@ -103,5 +104,16 @@ class CaseRulesTest(SandboxCase):
         self.assertEqual(mech.live_names(self.home / "commands", "command"), ["group:sub:tool"])
 
 
+class ClaudeBinTest(SandboxCase):
+    def test_off_path_windows_install_is_found_by_its_exe_name(self) -> None:
+        """The native Windows installer writes ~/.local/bin/claude.exe; PATH may lack it."""
+        plugin_cli.which = lambda name: None
+        exe = self.tmp / ".local" / "bin" / "claude.exe"
+        exe.parent.mkdir(parents=True)
+        exe.touch()
+        exe.chmod(0o755)
+        self.assertEqual(plugin_cli.claude_bin(), str(exe))
+
+
 if __name__ == "__main__":
     unittest.main()

{agent_toggle-0.7.0 → agent_toggle-0.7.3}/tests/test_store.py

@@ -152,6 +152,20 @@ class MigrateParksTest(CliCase):
         self.assertEqual(store.load_state()["disabled"]["claude:skill:demo-skill"]["parked_at"],
                          str(self.user_parked("skills", "demo-skill")))
 
+    @unittest.skipUnless(CAN_SYMLINK, "cannot create symlinks here")
+    def test_a_link_left_dangling_by_0_7_0_is_re_aimed(self) -> None:
+        real = self.write(".repos/r/linked/SKILL.md", "hi").parent
+        link = self.user_parked("skills", "linked")
+        link.parent.mkdir(parents=True, exist_ok=True)
+        link.symlink_to("../.repos/r/linked")                       # 0.7.0 kept the text
+        gone = self.user_parked("skills", "gone")
+        gone.symlink_to("../.repos/nowhere")                        # dangles from skills/ too
+        self.run_json("migrate")
+        self.assertEqual(link.resolve(), real.resolve())
+        self.assertEqual(os.readlink(gone), "../.repos/nowhere")    # left alone
+        rc, env = self.run_json("migrate")
+        self.assertNotIn("re-aimed", env["results"][0]["detail"])   # idempotent
+
 
 class LogSchemaTest(SandboxCase):
     FIELDS = {"ts", "harness", "type", "name", "action", "result", "batch",