agentbundle 0.8.0__tar.gz → 0.9.0__tar.gz

{agentbundle-0.8.0 → agentbundle-0.9.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: agentbundle
-Version: 0.8.0
+Version: 0.9.0
 Summary: npm for your coding agent. Install packs of skills, subagents, and hooks into any repo, for every major agent.
 Author-email: eugenelim <eugenelim@users.noreply.github.com>
 License: Apache-2.0 OR MIT
@@ -46,7 +46,7 @@ agentbundle install --pack research --scope user
 
 User-scope packs land in your home directory, not the repo — they're yours, not the team's, and they're there in every project you open.
 
-The install auto-detects your agent (`--adapter` overrides). To install from a **different** catalogue, pass it as a trailing argument — a git URL or a local path (`agentbundle install --pack core <catalogue>`); a `config set source <catalogue>` makes that the default, and an editable clone (`pip install -e`) defaults to itself.
+The install auto-detects your agent (`--adapter` overrides). Multi-IDE? Install the same pack for each agent at the same scope — they coexist, and the agents that read `.agents/skills/` (codex, cursor, gemini, copilot) share one skill copy instead of fighting over it. To install from a **different** catalogue, pass it as a trailing argument — a git URL or a local path (`agentbundle install --pack core <catalogue>`); a `config set source <catalogue>` makes that the default, and an editable clone (`pip install -e`) defaults to itself.
 
 ## More commands
 

{agentbundle-0.8.0 → agentbundle-0.9.0}/README.md

@@ -28,7 +28,7 @@ agentbundle install --pack research --scope user
 
 User-scope packs land in your home directory, not the repo — they're yours, not the team's, and they're there in every project you open.
 
-The install auto-detects your agent (`--adapter` overrides). To install from a **different** catalogue, pass it as a trailing argument — a git URL or a local path (`agentbundle install --pack core <catalogue>`); a `config set source <catalogue>` makes that the default, and an editable clone (`pip install -e`) defaults to itself.
+The install auto-detects your agent (`--adapter` overrides). Multi-IDE? Install the same pack for each agent at the same scope — they coexist, and the agents that read `.agents/skills/` (codex, cursor, gemini, copilot) share one skill copy instead of fighting over it. To install from a **different** catalogue, pass it as a trailing argument — a git URL or a local path (`agentbundle install --pack core <catalogue>`); a `config set source <catalogue>` makes that the default, and an editable clone (`pip install -e`) defaults to itself.
 
 ## More commands
 

{agentbundle-0.8.0 → agentbundle-0.9.0}/agentbundle/_data/adapter.toml

@@ -84,7 +84,13 @@
 # `_append_layout_section` step (append-if-exists / never-create / never-
 # overwrite) into an adopter-owned `agentbundle-layout.toml`. Additive +
 # optional — packs pinned below v0.16 build and validate unchanged.
-version = "0.16"
+# RFC-0052 / ADR-0039+0040 (v0.17): the `[contract.shared-prefixes]` registry
+# (below) classifies each allowed-prefix as `shared` (a path more than one
+# adapter reads) or — by absence — `private`, and a shared prefix declares its
+# reader cohort. cursor/gemini/copilot route the `skill` primitive to the shared
+# `.agents/skills/` home (joining codex) and gain it in `allowed-prefixes` at
+# both scopes; their agent/hook/command projection stays native.
+version = "0.17"
 
 # Sibling schemas this contract references — pack metadata and the
 # per-pack Claude-plugin manifest. Defined by spec AC #3 + #4.
@@ -92,6 +98,16 @@ version = "0.16"
 pack = "pack.schema.json"
 plugin-manifest = "plugin-manifest.schema.json"
 
+# Shared-prefix registry (RFC-0052 / ADR-0039). Each key is a `shared` prefix —
+# a path more than one shipped adapter reads — mapped to its reader cohort (the
+# shipped adapters that read it). Every prefix NOT listed here is `private`
+# (adapter-exclusive). The install-time cross-adapter disclosure names a shared
+# prefix's *other* shipped cohort members; the footprint gate co-owns a shared
+# path across same-pack rows. A future adapter joins a cohort by one line here.
+[contract.shared-prefixes]
+".agents/skills/" = ["codex", "cursor", "gemini", "copilot"]
+".kiro/skills/" = ["kiro-ide", "kiro-cli"]
+
 # ---------------------------------------------------------------------------
 # Primitive types — source paths within a pack's .apm/ directory
 # ---------------------------------------------------------------------------
@@ -477,7 +493,10 @@ allowed-prefixes.user = [".kiro/", ".agentbundle/"]
 [[adapter.copilot.projection]]
 primitive = "skill"
 mode = "direct-directory"
-target-path = ".github/skills/"
+# RFC-0052 / ADR-0040: cohort skill home. Copilot reads `.agents/skills/`
+# (project) and `~/.agents/skills/` (personal); routing skills there lets the
+# whole cohort (codex, cursor, gemini, copilot) share one co-owned copy.
+target-path = ".agents/skills/"
 on-conflict = "prompt-then-preserve"
 
 [[adapter.copilot.projection]]
@@ -517,12 +536,14 @@ repo = "."
 user = "~"
 # RFC-0024 + copilot-skills-and-web: repo-scope skills at `.github/skills/`,
 # agents at `.github/agents/`, hook wiring + bodies at `.github/hooks/`.
-allowed-prefixes.repo = [".github/skills/", ".github/agents/", ".github/hooks/"]
-# User-scope home is `~/.copilot/` — the prefix the install rewrite produces
-# from the adapter's repo-relpaths. `.agentbundle/` carries the install
-# state file (`~/.agentbundle/state.toml`), same as every other user-scope-
-# capable adapter (claude-code, codex).
-allowed-prefixes.user = [".copilot/skills/", ".copilot/agents/", ".copilot/hooks/", ".agentbundle/"]
+# RFC-0052: skills route to the shared `.agents/skills/` home; agents + hooks
+# stay native under `.github/` (ADR-0040 moves only the skill primitive).
+allowed-prefixes.repo = [".agents/skills/", ".github/skills/", ".github/agents/", ".github/hooks/"]
+# User-scope home is `~/.copilot/` for agents/hooks (the prefix the install
+# rewrite produces) and `~/.agents/skills/` for skills (scope-agnostic, no
+# rewrite). `.agentbundle/` carries the install state file
+# (`~/.agentbundle/state.toml`), same as every other user-scope-capable adapter.
+allowed-prefixes.user = [".agents/skills/", ".copilot/skills/", ".copilot/agents/", ".copilot/hooks/", ".agentbundle/"]
 
 # ---------------------------------------------------------------------------
 # Codex adapter — 5 projections (one per primitive)
@@ -596,7 +617,10 @@ allowed-prefixes.user = [".agents/skills/", ".codex/", ".agentbundle/"]
 [[adapter.cursor.projection]]
 primitive = "skill"
 mode = "direct-directory"
-target-path = ".cursor/skills/"
+# RFC-0052 / ADR-0040: cohort skill home. Cursor reads `.agents/skills/` and
+# `~/.agents/skills/` alongside `.cursor/skills/`; route skills there to share
+# one co-owned copy across the cohort.
+target-path = ".agents/skills/"
 on-conflict = "prompt-then-preserve"
 
 [[adapter.cursor.projection]]
@@ -650,8 +674,10 @@ mode = "dropped"
 [adapter.cursor.scope]
 repo = "."
 user = "~"
-allowed-prefixes.repo = [".cursor/", ".agentbundle/"]
-allowed-prefixes.user = [".cursor/", ".agentbundle/"]
+# RFC-0052: skills route to the shared `.agents/skills/` home (both scopes);
+# agents/hooks stay under `.cursor/` (ADR-0040 moves only the skill primitive).
+allowed-prefixes.repo = [".agents/skills/", ".cursor/", ".agentbundle/"]
+allowed-prefixes.user = [".agents/skills/", ".cursor/", ".agentbundle/"]
 
 # ---------------------------------------------------------------------------
 # Gemini adapter — full-parity native adapter (RFC-0027 / ADR-0016 /
@@ -685,7 +711,10 @@ allowed-prefixes.user = [".cursor/", ".agentbundle/"]
 [[adapter.gemini.projection]]
 primitive = "skill"
 mode = "direct-directory"
-target-path = ".gemini/skills/"
+# RFC-0052 / ADR-0040: cohort skill home. Gemini CLI reads `.agents/skills/`
+# and `~/.agents/skills/`, and its docs now make the alias precedence-winning
+# over `.gemini/skills/`; route skills there to share one co-owned copy.
+target-path = ".agents/skills/"
 on-conflict = "prompt-then-preserve"
 
 [[adapter.gemini.projection]]
@@ -744,8 +773,10 @@ mode = "dropped"
 [adapter.gemini.scope]
 repo = "."
 user = "~"
-allowed-prefixes.repo = [".gemini/", ".agentbundle/"]
-allowed-prefixes.user = [".gemini/", ".agentbundle/"]
+# RFC-0052: skills route to the shared `.agents/skills/` home (both scopes);
+# agents/hooks stay under `.gemini/` (ADR-0040 moves only the skill primitive).
+allowed-prefixes.repo = [".agents/skills/", ".gemini/", ".agentbundle/"]
+allowed-prefixes.user = [".agents/skills/", ".gemini/", ".agentbundle/"]
 
 # ---------------------------------------------------------------------------
 # Frontmatter mappings — rewrite rules applied when projecting a primitive

{agentbundle-0.8.0 → agentbundle-0.9.0}/agentbundle/build/tests/test_adapter_copilot.py

@@ -42,11 +42,9 @@ class CopilotAdapterTests(unittest.TestCase):
         cls.contract = load_contract(CONTRACT_PATH)
 
     def test_skill_projects_as_first_class_skill_md(self) -> None:
-        # v0.11 (docs/specs/copilot-skills-and-web): `skill` flips
-        # `instruction-file` → `direct-directory`. Copilot reads
-        # `.github/skills/<name>/SKILL.md` and accepts our canonical Claude
-        # SKILL.md verbatim, so the source tree is copied byte-for-byte (no
-        # `.instructions.md` wrapper, no injected `applyTo` frontmatter).
+        # RFC-0052 / ADR-0040: copilot `skill` routes to the shared cohort home
+        # `.agents/skills/<name>/SKILL.md` (joining codex, cursor, gemini).
+        # Agents/hooks/hook-wiring remain under `.github/`.
         with tempfile.TemporaryDirectory() as tmp:
             tmp_path = Path(tmp)
             pack = _seed_pack(tmp_path)
@@ -55,7 +53,7 @@ class CopilotAdapterTests(unittest.TestCase):
             source_body = (pack / ".apm" / "skills" / "foo" / "SKILL.md").read_text(
                 encoding="utf-8"
             )
-            output_path = out / ".github" / "skills" / "foo" / "SKILL.md"
+            output_path = out / ".agents" / "skills" / "foo" / "SKILL.md"
             self.assertTrue(output_path.exists())
             self.assertEqual(output_path.read_text(encoding="utf-8"), source_body)
             # No legacy instruction-file output.
@@ -65,19 +63,20 @@ class CopilotAdapterTests(unittest.TestCase):
         # AC4: re-projection sweeps a stale skill dir, and the sweep is bounded
         # to the skill target — it never touches sibling `.github/agents/` or
         # `.github/hooks/` content.
+        # RFC-0052: skill target is now `.agents/skills/` (shared cohort home).
         with tempfile.TemporaryDirectory() as tmp:
             tmp_path = Path(tmp)
             pack = _seed_pack(tmp_path)
             out = tmp_path / "out"
             project(pack, self.contract, out)
-            self.assertTrue((out / ".github" / "skills" / "foo").exists())
+            self.assertTrue((out / ".agents" / "skills" / "foo").exists())
             agents_before = sorted((out / ".github" / "agents").iterdir())
             hooks_before = sorted((out / ".github" / "hooks").iterdir())
             # Rename the source skill; re-project. The old name is now orphaned.
             (pack / ".apm" / "skills" / "foo").rename(pack / ".apm" / "skills" / "renamed")
             project(pack, self.contract, out)
-            self.assertFalse((out / ".github" / "skills" / "foo").exists())
-            self.assertTrue((out / ".github" / "skills" / "renamed").exists())
+            self.assertFalse((out / ".agents" / "skills" / "foo").exists())
+            self.assertTrue((out / ".agents" / "skills" / "renamed").exists())
             # Sibling agent/hook trees are untouched by the skill sweep.
             self.assertEqual(sorted((out / ".github" / "agents").iterdir()), agents_before)
             self.assertEqual(sorted((out / ".github" / "hooks").iterdir()), hooks_before)

{agentbundle-0.8.0 → agentbundle-0.9.0}/agentbundle/build/tests/test_adapter_cursor.py

@@ -54,10 +54,10 @@ class CursorContractTests(unittest.TestCase):
         """AC1 — contract bumped to 0.11 by cursor-full-parity; subsequently
         0.12 (copilot-skills-and-web), 0.13 (docs/specs/gemini-full-parity),
         0.14 (docs/specs/enriched-pack-manifest), 0.15
-        (docs/specs/kiro-cli-agent-skill-resources), then 0.16
+        (docs/specs/kiro-cli-agent-skill-resources), then 0.17
         (docs/specs/consolidated-pack-layout).
         Name preserved to keep the diff small."""
-        self.assertEqual(self.contract["contract"]["version"], "0.16")
+        self.assertEqual(self.contract["contract"]["version"], "0.17")
 
     def test_cursor_block_projects_five_primitives(self) -> None:
         """AC2 — the five standard primitives are in the projection array."""
@@ -68,7 +68,7 @@ class CursorContractTests(unittest.TestCase):
             {"skill", "agent", "hook-body", "hook-wiring", "command"},
         )
         self.assertEqual(prims["skill"]["mode"], "direct-directory")
-        self.assertEqual(prims["skill"]["target-path"], ".cursor/skills/")
+        self.assertEqual(prims["skill"]["target-path"], ".agents/skills/")
         self.assertEqual(prims["agent"]["mode"], "direct-file")
         self.assertEqual(prims["agent"]["target-path"], ".cursor/agents/")
         self.assertEqual(
@@ -109,7 +109,7 @@ class CursorContractTests(unittest.TestCase):
         scope = self.contract["adapter"]["cursor"]["scope"]
         self.assertEqual(scope["repo"], ".")
         self.assertEqual(scope["user"], "~")
-        self.assertEqual(scope["allowed-prefixes"]["repo"], [".cursor/", ".agentbundle/"])
+        self.assertEqual(scope["allowed-prefixes"]["repo"], [".agents/skills/", ".cursor/", ".agentbundle/"])
         self.assertEqual(
             scope["allowed-prefixes"]["repo"], scope["allowed-prefixes"]["user"]
         )
@@ -147,7 +147,7 @@ class CursorProjectionTests(unittest.TestCase):
             (pack / ".apm" / "hooks" / "on-start.py").write_text("#!py", encoding="utf-8")
             out = tmp_path / "out"
             cursor.project(pack, self.contract, out)
-            self.assertTrue((out / ".cursor" / "skills" / "my-skill" / "SKILL.md").exists())
+            self.assertTrue((out / ".agents" / "skills" / "my-skill" / "SKILL.md").exists())
             self.assertTrue((out / ".cursor" / "commands" / "do-thing.md").exists())
             self.assertTrue((out / ".cursor" / "hooks" / "on-start.py").exists())
 
@@ -332,10 +332,10 @@ class CursorProjectionTests(unittest.TestCase):
             os.symlink(secret, skill / "leak.txt")
             out = tmp_path / "out"
             cursor.project(pack, self.contract, out)
-            leaked = out / ".cursor" / "skills" / "s" / "leak.txt"
+            leaked = out / ".agents" / "skills" / "s" / "leak.txt"
             self.assertFalse(leaked.is_symlink(), "nested symlink must not be reproduced")
             self.assertFalse(leaked.exists(), "nested symlink target must not be copied")
-            self.assertTrue((out / ".cursor" / "skills" / "s" / "SKILL.md").exists())
+            self.assertTrue((out / ".agents" / "skills" / "s" / "SKILL.md").exists())
 
 
 class CursorInstallDispatchTests(unittest.TestCase):

{agentbundle-0.8.0 → agentbundle-0.9.0}/agentbundle/build/tests/test_adapter_gemini.py

@@ -151,9 +151,9 @@ class GeminiContractTests(unittest.TestCase):
         cls.contract = load_contract(CONTRACT_PATH)
 
     def test_contract_version_is_0_14(self) -> None:
-        """Contract version is 0.16 (docs/specs/consolidated-pack-layout
+        """Contract version is 0.17 (docs/specs/consolidated-pack-layout
         bumped it from kiro-cli-agent-skill-resources' 0.15). Name preserved."""
-        self.assertEqual(self.contract["contract"]["version"], "0.16")
+        self.assertEqual(self.contract["contract"]["version"], "0.17")
 
     def test_gemini_block_projects_five_primitives(self) -> None:
         """AC2 — five standard primitives with their gemini targets."""
@@ -161,7 +161,7 @@ class GeminiContractTests(unittest.TestCase):
         prims = {e["primitive"]: e for e in block["projection"]}
         self.assertEqual(set(prims), {"skill", "agent", "hook-body", "hook-wiring", "command"})
         self.assertEqual(prims["skill"]["mode"], "direct-directory")
-        self.assertEqual(prims["skill"]["target-path"], ".gemini/skills/")
+        self.assertEqual(prims["skill"]["target-path"], ".agents/skills/")
         self.assertEqual(prims["agent"]["mode"], "direct-file")
         self.assertEqual(prims["agent"]["target-path"], ".gemini/agents/")
         self.assertEqual(prims["agent"]["frontmatter-mapping"], "gemini-agent-frontmatter")
@@ -210,7 +210,7 @@ class GeminiContractTests(unittest.TestCase):
         scope = self.contract["adapter"]["gemini"]["scope"]
         self.assertEqual(scope["repo"], ".")
         self.assertEqual(scope["user"], "~")
-        self.assertEqual(scope["allowed-prefixes"]["repo"], [".gemini/", ".agentbundle/"])
+        self.assertEqual(scope["allowed-prefixes"]["repo"], [".agents/skills/", ".gemini/", ".agentbundle/"])
         self.assertEqual(scope["allowed-prefixes"]["repo"], scope["allowed-prefixes"]["user"])
 
     def test_frontmatter_mapping_keeps_tools_and_model(self) -> None:
@@ -254,7 +254,7 @@ class GeminiProjectionTests(unittest.TestCase):
             (pack / ".apm" / "hooks" / "on-start.py").write_text("#!py", encoding="utf-8")
             out = tmp_path / "out"
             gemini.project(pack, self.contract, out)
-            self.assertTrue((out / ".gemini" / "skills" / "my-skill" / "SKILL.md").exists())
+            self.assertTrue((out / ".agents" / "skills" / "my-skill" / "SKILL.md").exists())
             self.assertTrue((out / ".gemini" / "commands" / "do-thing.toml").exists())
             self.assertTrue((out / ".gemini" / "hooks" / "on-start.py").exists())
 
@@ -361,9 +361,9 @@ class GeminiProjectionTests(unittest.TestCase):
             os.symlink(secret, skill / "leak.txt")
             out = tmp_path / "out"
             gemini.project(pack, self.contract, out)
-            leaked = out / ".gemini" / "skills" / "s" / "leak.txt"
+            leaked = out / ".agents" / "skills" / "s" / "leak.txt"
             self.assertFalse(leaked.exists())
-            self.assertTrue((out / ".gemini" / "skills" / "s" / "SKILL.md").exists())
+            self.assertTrue((out / ".agents" / "skills" / "s" / "SKILL.md").exists())
 
 
 # ===========================================================================

{agentbundle-0.8.0 → agentbundle-0.9.0}/agentbundle/build/tests/test_adapter_kiro_ide.py

@@ -172,13 +172,13 @@ class KiroIdeAdapterTests(unittest.TestCase):
         self.assertIn(".kiro.hook", target_repo)
 
     def test_contract_version_is_0_9(self) -> None:
-        """Contract version is 0.16 (docs/specs/consolidated-pack-layout,
+        """Contract version is 0.17 (docs/specs/consolidated-pack-layout,
         atop kiro-cli-agent-skill-resources' 0.15 and enriched-pack-manifest's 0.14).
         Name preserved to keep the diff small."""
         self.assertEqual(
             self.contract["contract"]["version"],
-            "0.16",
-            "adapter.toml [contract] version must be '0.16' after consolidated-pack-layout",
+            "0.17",
+            "adapter.toml [contract] version must be '0.17' after consolidated-pack-layout",
         )
 
     def test_kiro_ide_hook_projects_with_flat_prefix_path(self) -> None:

{agentbundle-0.8.0 → agentbundle-0.9.0}/agentbundle/build/tests/test_contract.py

@@ -393,7 +393,7 @@ class FrontmatterTableTests(unittest.TestCase):
             if p["primitive"] == "skill"
         )
         self.assertEqual(skill["mode"], "direct-directory")
-        self.assertEqual(skill["target-path"], ".github/skills/")
+        self.assertEqual(skill["target-path"], ".agents/skills/")
         self.assertNotIn("frontmatter-default", skill)
 
     def test_frontmatter_mapping_and_default_are_structurally_distinct(self) -> None:
@@ -452,7 +452,7 @@ class ContractV05Tests(unittest.TestCase):
         self.schema = _load_schema()
 
     def test_contract_version_is_v05(self) -> None:
-        """tomllib.loads of adapter.toml returns contract.version == "0.16"
+        """tomllib.loads of adapter.toml returns contract.version == "0.17"
         (bumped from kiro-cli-agent-skill-resources' "0.15" by
         docs/specs/consolidated-pack-layout: pack.toml gains an optional
         scope-keyed [pack.layout] table). Class/method names preserved to
@@ -460,8 +460,8 @@ class ContractV05Tests(unittest.TestCase):
         """
         self.assertEqual(
             self.contract["contract"]["version"],
-            "0.16",
-            "adapter.toml [contract] version must be '0.16' after consolidated-pack-layout",
+            "0.17",
+            "adapter.toml [contract] version must be '0.17' after consolidated-pack-layout",
         )
 
     def test_claude_code_install_routes_includes_apm(self) -> None:

{agentbundle-0.8.0 → agentbundle-0.9.0}/agentbundle/build/tests/test_contract_scope.py

@@ -89,20 +89,21 @@ class OtherAdaptersOmitScopeTests(unittest.TestCase):
         scope = contract["adapter"]["copilot"].get("scope")
         self.assertIsNotNone(scope, "copilot [scope] block missing")
         self.assertEqual(scope["repo"], ".")
-        # v0.11 (docs/specs/copilot-skills-and-web): copilot `skill` projects as
-        # first-class `direct-directory` SKILL.md under `.github/skills/`, so the
-        # repo skill prefix is `.github/skills/` (was `.github/instructions/`).
-        # Agent + hook homes under `.github/` are unchanged.
+        # RFC-0052 / ADR-0040: copilot skill routes to the shared `.agents/skills/`
+        # cohort home (prepended to the list); agents + hooks stay under `.github/`.
+        # The `.github/skills/` prefix is retained to allow the path-jail to admit
+        # any existing `.github/skills/` files written by the previous contract.
         self.assertEqual(
             scope["allowed-prefixes"]["repo"],
-            [".github/skills/", ".github/agents/", ".github/hooks/"],
+            [".agents/skills/", ".github/skills/", ".github/agents/", ".github/hooks/"],
         )
-        # User scope: `~/.copilot/{skills,agents,hooks}/` + `.agentbundle/`
-        # (the install state-file home, same as every other user-capable adapter).
+        # User scope: RFC-0052 prepends `.agents/skills/` to the user list.
+        # `.copilot/skills/` is kept for path-jail compat; `.agentbundle/` stays.
         self.assertEqual(scope["user"], "~")
         self.assertEqual(
             scope["allowed-prefixes"]["user"],
             [
+                ".agents/skills/",
                 ".copilot/skills/",
                 ".copilot/agents/",
                 ".copilot/hooks/",

{agentbundle-0.8.0 → agentbundle-0.9.0}/agentbundle/build/tests/test_contract_v07.py

@@ -83,17 +83,20 @@ class TestContractV07(unittest.TestCase):
         copilot_scope = self.contract["adapter"]["copilot"].get("scope")
         self.assertIsNotNone(copilot_scope, "copilot scope table missing")
         self.assertEqual(copilot_scope["repo"], ".")
-        # v0.10 (copilot-full-parity) made copilot user-scope-capable; v0.11
-        # (copilot-skills-and-web) flips `skill` to `.github/skills/` SKILL.md, so
-        # the repo skill prefix is `.github/skills/` (was `.github/instructions/`).
+        # RFC-0052 / ADR-0040: copilot skill now routes to `.agents/skills/`
+        # (shared cohort home, prepended); agents + hooks stay under `.github/`.
+        # `.github/skills/` is kept for path-jail compat with existing files.
         self.assertEqual(
             copilot_scope["allowed-prefixes"]["repo"],
-            [".github/skills/", ".github/agents/", ".github/hooks/"],
+            [".agents/skills/", ".github/skills/", ".github/agents/", ".github/hooks/"],
         )
         self.assertEqual(copilot_scope["user"], "~")
+        # RFC-0052: `.agents/skills/` prepended to user list too.
+        # `.copilot/skills/` retained for path-jail compat.
         self.assertEqual(
             copilot_scope["allowed-prefixes"]["user"],
             [
+                ".agents/skills/",
                 ".copilot/skills/",
                 ".copilot/agents/",
                 ".copilot/hooks/",

agentbundle-0.9.0/agentbundle/build/tests/test_shared_prefix_contract.py

@@ -0,0 +1,111 @@
+"""T5 (RFC-0052 / ADR-0039+0040): the `shared` prefix class + cohort routing.
+
+The adapter contract gains a `[contract.shared-prefixes]` registry mapping each
+shared prefix to its reader cohort (shipped adapters); every other prefix is
+`private` by derivation. cursor/gemini/copilot route the `skill` primitive to
+the shared `.agents/skills/` home (joining codex) and include it in
+`allowed-prefixes` at both scopes. The two contract copies agree byte-for-byte.
+"""
+
+from __future__ import annotations
+
+import tomllib
+import unittest
+from pathlib import Path
+
+
+_REPO_ROOT = Path(__file__).resolve().parents[5]
+_DATA_COPY = (
+    _REPO_ROOT
+    / "packages/agentbundle/agentbundle/_data/adapter.toml"
+)
+_DOCS_COPY = _REPO_ROOT / "docs/contracts/adapter.toml"
+
+
+def _contract() -> dict:
+    return tomllib.loads(_DATA_COPY.read_text(encoding="utf-8"))
+
+
+class SharedPrefixRegistryTests(unittest.TestCase):
+    def setUp(self) -> None:
+        self.contract = _contract()
+        self.shared = self.contract["contract"]["shared-prefixes"]
+
+    def test_contract_version_bumped(self) -> None:
+        self.assertEqual(self.contract["contract"]["version"], "0.17")
+
+    def test_agents_skills_shared_with_full_cohort(self) -> None:
+        self.assertEqual(
+            sorted(self.shared[".agents/skills/"]),
+            ["codex", "copilot", "cursor", "gemini"],
+        )
+
+    def test_kiro_skills_shared_with_kiro_cohort(self) -> None:
+        self.assertEqual(
+            sorted(self.shared[".kiro/skills/"]),
+            ["kiro-cli", "kiro-ide"],
+        )
+
+    def test_only_known_shared_prefixes_declared(self) -> None:
+        # Everything not in the registry is `private` by derivation; the
+        # registry itself holds exactly the two shared prefixes.
+        self.assertEqual(
+            set(self.shared), {".agents/skills/", ".kiro/skills/"}
+        )
+
+    def test_private_prefix_not_in_registry(self) -> None:
+        # A representative private prefix (claude-code's `.claude/`) is absent.
+        self.assertNotIn(".claude/", self.shared)
+
+
+class CohortSkillRoutingTests(unittest.TestCase):
+    def setUp(self) -> None:
+        self.contract = _contract()
+
+    def _skill_target(self, adapter: str) -> str:
+        for p in self.contract["adapter"][adapter]["projection"]:
+            if p.get("primitive") == "skill":
+                return p["target-path"]
+        raise AssertionError(f"no skill projection for {adapter}")
+
+    def test_cohort_skill_targets_shared_home(self) -> None:
+        for adapter in ("codex", "cursor", "gemini", "copilot"):
+            self.assertEqual(
+                self._skill_target(adapter), ".agents/skills/",
+                f"{adapter} skill must route to the shared .agents/skills/ home",
+            )
+
+    def test_cohort_allowed_prefixes_include_shared_home_both_scopes(self) -> None:
+        for adapter in ("cursor", "gemini", "copilot"):
+            scope = self.contract["adapter"][adapter]["scope"]
+            for which in ("repo", "user"):
+                self.assertIn(
+                    ".agents/skills/", scope["allowed-prefixes"][which],
+                    f"{adapter}.{which} must admit the shared skill prefix",
+                )
+
+    def test_cohort_agents_stay_native(self) -> None:
+        # ADR-0040 moves only the skill primitive; agents stay native.
+        for adapter, native in (
+            ("cursor", ".cursor/agents/"),
+            ("gemini", ".gemini/agents/"),
+            ("copilot", ".github/agents/"),
+        ):
+            agent = next(
+                p for p in self.contract["adapter"][adapter]["projection"]
+                if p.get("primitive") == "agent"
+            )
+            self.assertEqual(agent["target-path"], native)
+
+
+class ContractMirrorTests(unittest.TestCase):
+    def test_data_and_docs_agree_byte_for_byte(self) -> None:
+        self.assertEqual(
+            _DATA_COPY.read_bytes(),
+            _DOCS_COPY.read_bytes(),
+            "the _data/ and docs/contracts/ adapter.toml copies must be identical",
+        )
+
+
+if __name__ == "__main__":
+    unittest.main()

{agentbundle-0.8.0 → agentbundle-0.9.0}/agentbundle/cli.py

@@ -406,6 +406,15 @@ def _build_parser() -> argparse.ArgumentParser:
     sp.add_argument("pack_path", help="Path to the pack to diff against.")
     sp.add_argument("--root", default=".")
     sp.add_argument("--scope", choices=("repo", "user"))
+    sp.add_argument(
+        "--adapter",
+        choices=_shipped_adapters_choices(),
+        help=(
+            "Disambiguate when the pack is installed for multiple adapters at "
+            "the resolved scope (RFC-0052). Inferred when the pack has a single "
+            "adapter row; required when it has more than one."
+        ),
+    )
     sp.set_defaults(func=_lazy("diff"))
 
     # --- upgrade --- (--scope disambiguator)
@@ -436,6 +445,15 @@ def _build_parser() -> argparse.ArgumentParser:
     )
     sp.add_argument("--root", default=".")
     sp.add_argument("--scope", choices=("repo", "user"))
+    sp.add_argument(
+        "--adapter",
+        choices=_shipped_adapters_choices(),
+        help=(
+            "Disambiguate when the pack is installed for multiple adapters at "
+            "the resolved scope (RFC-0052). Inferred when the pack has a single "
+            "adapter row; required when it has more than one."
+        ),
+    )
     sp.add_argument(
         "--yes",
         action="store_true",
@@ -464,6 +482,17 @@ def _build_parser() -> argparse.ArgumentParser:
     sp.add_argument("--pack", required=True)
     sp.add_argument("--root", default=".")
     sp.add_argument("--scope", choices=("repo", "user"))
+    sp.add_argument(
+        "--adapter",
+        choices=_shipped_adapters_choices(),
+        help=(
+            "Disambiguate when the pack is installed for multiple adapters at "
+            "the resolved scope (RFC-0052). Inferred when the pack has a single "
+            "adapter row; required when it has more than one. Only the named "
+            "adapter row is uninstalled; shared files survive while a sibling "
+            "row still owns them."
+        ),
+    )
     sp.add_argument(
         "--yes",
         action="store_true",