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

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

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: agent-toggle
-Version: 0.7.0
+Version: 0.7.2
 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

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

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

{agent_toggle-0.7.0 → agent_toggle-0.7.2}/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.2}/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.2}/agent_toggle.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: agent-toggle
-Version: 0.7.0
+Version: 0.7.2
 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

{agent_toggle-0.7.0 → agent_toggle-0.7.2}/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.2}/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",