pactkit-opencode 2.9.0__tar.gz → 2.9.2__tar.gz

{pactkit_opencode-2.9.0 → pactkit_opencode-2.9.2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: pactkit-opencode
-Version: 2.9.0
+Version: 2.9.2
 Summary: PactKit adapter for OpenCode IDE
 License-Expression: MIT
 Requires-Python: >=3.10

{pactkit_opencode-2.9.0 → pactkit_opencode-2.9.2}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "pactkit-opencode"
-version = "2.9.0"
+version = "2.9.2"
 description = "PactKit adapter for OpenCode IDE"
 requires-python = ">=3.10"
 license = "MIT"

{pactkit_opencode-2.9.0 → pactkit_opencode-2.9.2}/src/pactkit_opencode/deployer.py

@@ -113,7 +113,7 @@ class OpenCodeDeployer(DeployerBase):
             if replaced != content:
                 atomic_write(cmd_md, replaced)
 
-        n_rules = _deploy_rules(opencode_root, enabled_rules)
+        n_rules = _deploy_rules(opencode_root, enabled_rules, profile=self.profile)
         self._deploy_agents_md_inline(opencode_root)
 
         providers = self._load_opencode_providers(opencode_root)
@@ -297,33 +297,19 @@ class OpenCodeDeployer(DeployerBase):
         config["instructions"] = existing
 
         # OpenCode auto-discovers commands from commands/*.md directory.
-        # opencode.json "command" entries are only needed for model routing overrides.
-        # NOTE: "template" in JSON is inline prompt text, NOT a file path — do not write it.
+        # opencode.json "command" section requires BOTH "model" AND "template" fields.
+        # Since we don't control template content (it's inline prompt text, not a file path),
+        # we must NOT write model-only entries — they cause OpenCode validation errors.
+        # Clean up any PactKit-managed command entries that lack template.
         cmd_config = config.get("command", {})
-        if enabled_commands:
-            # Remove stale command entries not in enabled set
-            stale_keys = [k for k in cmd_config if k.startswith("project-") and k not in enabled_commands]
-            for k in stale_keys:
-                del cmd_config[k]
-            # Remove legacy "template" fields (was incorrectly treated as file path)
-            for cmd_name in list(cmd_config):
-                if cmd_name.startswith("project-") and "template" in cmd_config[cmd_name]:
-                    del cmd_config[cmd_name]["template"]
-                    # Remove entry entirely if only template was present
-                    if not cmd_config[cmd_name]:
-                        del cmd_config[cmd_name]
-
-        # Apply model routing
-        if command_models and providers:
-            for cmd_name, model_short in command_models.items():
-                model_id = OpenCodeDeployer._resolve_opencode_model_id(model_short, providers)
-                if model_id:
-                    if cmd_name not in cmd_config:
-                        cmd_config[cmd_name] = {}
-                    cmd_config[cmd_name]["model"] = model_id
+        for cmd_name in list(cmd_config):
+            if cmd_name.startswith("project-") and "template" not in cmd_config[cmd_name]:
+                del cmd_config[cmd_name]
 
         if cmd_config:
             config["command"] = cmd_config
+        else:
+            config.pop("command", None)
 
         content = json.dumps(config, indent=2, ensure_ascii=False) + "\n"
         atomic_write(json_path, content)

{pactkit_opencode-2.9.0 → pactkit_opencode-2.9.2}/src/pactkit_opencode.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: pactkit-opencode
-Version: 2.9.0
+Version: 2.9.2
 Summary: PactKit adapter for OpenCode IDE
 License-Expression: MIT
 Requires-Python: >=3.10

{pactkit_opencode-2.9.0 → pactkit_opencode-2.9.2}/tests/test_bug035_opencode_dual_layer.py

@@ -60,18 +60,35 @@ class TestR3ProjectInitOpencodeDetection:
     """R3: /project-init playbook must detect OpenCode environment."""
 
     def test_project_init_contains_opencode_detection(self):
-        """project-init.md contains OpenCode environment detection instructions."""
+        """project-init.md rendered for OpenCode contains 'OpenCode' and 'opencode'.
+
+        After STORY-slim-074 DIP refactor, the raw template uses {DISPLAY_NAME} /
+        {FORMAT_NAME} placeholders. The rendered output (via _render_prompt) is
+        what gets deployed and must contain the format-specific strings.
+        """
+        from pactkit.generators.deployer import _render_prompt
+        from pactkit.profiles import get_profile
         from pactkit.prompts.commands import COMMANDS_CONTENT
 
-        content = COMMANDS_CONTENT["project-init.md"]
+        profile = get_profile("opencode")
+        content = _render_prompt(COMMANDS_CONTENT["project-init.md"], profile)
         assert "opencode" in content.lower() or "OpenCode" in content
 
     def test_project_init_contains_opencode_json_generation(self):
-        """project-init.md contains opencode.json generation instructions."""
+        """project-init.md template references format config via {PACTKIT_YAML} placeholder.
+
+        After STORY-slim-074 DIP refactor, opencode-specific paths are expressed as
+        template variables ({PACTKIT_YAML}, {FORMAT_NAME}) rather than hardcoded strings.
+        The rendered output for opencode will contain '.opencode/pactkit.yaml'.
+        """
+        from pactkit.generators.deployer import _render_prompt
+        from pactkit.profiles import get_profile
         from pactkit.prompts.commands import COMMANDS_CONTENT
 
-        content = COMMANDS_CONTENT["project-init.md"]
-        assert "opencode.json" in content
+        profile = get_profile("opencode")
+        content = _render_prompt(COMMANDS_CONTENT["project-init.md"], profile)
+        # {PACTKIT_YAML} renders to '.opencode/pactkit.yaml' for opencode format
+        assert ".opencode/pactkit.yaml" in content or "pactkit.yaml" in content
 
 
 # ===========================================================================

{pactkit_opencode-2.9.0 → pactkit_opencode-2.9.2}/tests/test_story071_opencode_config_parity.py

@@ -91,16 +91,25 @@ class TestAC3PreserveFields:
 
 
 class TestAC4PlaybookPactkitYaml:
-    """AC4: project-init playbook explains pactkit.yaml stays in .claude/."""
+    """AC4: project-init playbook explains pactkit.yaml stays in .claude/ (classic) or .opencode/ (OpenCode)."""
 
     def test_playbook_mentions_pactkit_yaml_location(self):
-        """project-init playbook references pactkit.yaml in .claude/ context."""
+        """project-init playbook rendered for classic format references pactkit.yaml in .claude/.
+
+        After STORY-slim-074 DIP refactor, config paths are expressed as {PACTKIT_YAML} and
+        {PROJECT_CONFIG_DIR} template variables. The raw template always contains 'pactkit.yaml'
+        (as part of the {PACTKIT_YAML} placeholder), and the rendered classic output contains '.claude'.
+        """
+        from pactkit.generators.deployer import _render_prompt
+        from pactkit.profiles import get_profile
         from pactkit.prompts import COMMANDS_CONTENT
 
-        init_content = COMMANDS_CONTENT["project-init.md"]
-        assert "pactkit.yaml" in init_content
-        # Should mention it stays in .claude/ not .opencode/
-        assert ".claude" in init_content
+        raw_content = COMMANDS_CONTENT["project-init.md"]
+        # Raw template always references pactkit.yaml via the {PACTKIT_YAML} variable name
+        assert "pactkit.yaml" in raw_content
+        # Rendered for classic: {PROJECT_CONFIG_DIR} → '.claude', {PACTKIT_YAML} → '.claude/pactkit.yaml'
+        classic_content = _render_prompt(raw_content, get_profile("classic"))
+        assert ".claude" in classic_content
 
 
 # ===========================================================================

{pactkit_opencode-2.9.0 → pactkit_opencode-2.9.2}/tests/test_story073_command_model_routing.py

@@ -40,26 +40,16 @@ class TestAC1CommandModel:
         deploy(format="opencode", target=str(out))
         return out
 
-    def test_act_model_in_opencode_json(self, tmp_path):
-        """project-act model routing is in opencode.json command section."""
+    def test_no_model_only_entries_in_opencode_json(self, tmp_path):
+        """Model-only command entries must NOT be written — OpenCode requires both model AND template."""
         out = self._deploy_with_providers(tmp_path)
         data = json.loads((out / "opencode.json").read_text())
         cmd_config = data.get("command", {})
-        assert "project-act" in cmd_config
-        assert "model" in cmd_config["project-act"]
-        assert "sonnet" in cmd_config["project-act"]["model"]
-
-    def test_done_model_in_opencode_json(self, tmp_path):
-        """project-done model routing is in opencode.json command section."""
-        out = self._deploy_with_providers(tmp_path)
-        data = json.loads((out / "opencode.json").read_text())
-        assert "project-done" in data.get("command", {})
-
-    def test_check_model_in_opencode_json(self, tmp_path):
-        """project-check model routing is in opencode.json command section."""
-        out = self._deploy_with_providers(tmp_path)
-        data = json.loads((out / "opencode.json").read_text())
-        assert "project-check" in data.get("command", {})
+        for cmd_name, cmd_val in cmd_config.items():
+            if cmd_name.startswith("project-"):
+                assert "template" in cmd_val, (
+                    f"command.{cmd_name} has model but no template — OpenCode validation will fail"
+                )
 
     def test_frontmatter_has_no_model(self, tmp_path):
         """Command frontmatter should NOT contain model: (routing is in opencode.json)."""
@@ -137,16 +127,26 @@ class TestAC3ClassicNoModel:
 
 
 class TestAC4InitConditional:
-    """AC4: project-init has conditional CLAUDE.md vs AGENTS.md."""
+    """AC4: project-init rendered per-format references the correct instructions file."""
 
     def test_init_has_conditional(self):
-        """project-init playbook has OpenCode conditional for CLAUDE.md."""
+        """project-init playbook rendered for OpenCode contains 'OpenCode' and 'AGENTS.md'.
+
+        After STORY-slim-074 DIP refactor, the raw template uses {DISPLAY_NAME} and
+        {INSTRUCTIONS_FILE} placeholders instead of hardcoded format-specific strings.
+        _render_prompt() injects the correct values at deploy time.
+        """
+        from pactkit.generators.deployer import _render_prompt
+        from pactkit.profiles import get_profile
         from pactkit.prompts import COMMANDS_CONTENT
 
-        init_content = COMMANDS_CONTENT["project-init.md"]
+        profile = get_profile("opencode")
+        init_content = _render_prompt(COMMANDS_CONTENT["project-init.md"], profile)
         assert "OpenCode" in init_content
         assert "AGENTS.md" in init_content
-        assert "Claude Code" in init_content or ".claude/" in init_content
+        # Rendered for classic: {DISPLAY_NAME} → 'Claude Code', {PROJECT_CONFIG_DIR} → '.claude'
+        classic_content = _render_prompt(COMMANDS_CONTENT["project-init.md"], get_profile("classic"))
+        assert "Claude Code" in classic_content or ".claude" in classic_content
 
 
 # ===========================================================================