pactkit-codex 2.10.2__tar.gz → 2.17.0__tar.gz

{pactkit_codex-2.10.2/src/pactkit_codex.egg-info → pactkit_codex-2.17.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: pactkit-codex
-Version: 2.10.2
+Version: 2.17.0
 Summary: PactKit adapter for OpenAI Codex CLI
 Author: Slim
 License: MIT
@@ -21,7 +21,7 @@ Classifier: Topic :: Software Development :: Quality Assurance
 Requires-Python: >=3.10
 Description-Content-Type: text/markdown
 License-File: LICENSE
-Requires-Dist: pactkit>=2.9.0
+Requires-Dist: pactkit>=2.17.0
 Requires-Dist: tomli>=1.1.0; python_version < "3.11"
 Dynamic: license-file
 

{pactkit_codex-2.10.2 → pactkit_codex-2.17.0}/pyproject.toml

@@ -4,13 +4,13 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "pactkit-codex"
-version = "2.10.2"
+version = "2.17.0"
 description = "PactKit adapter for OpenAI Codex CLI"
 readme = "README.md"
 license = {text = "MIT"}
 requires-python = ">=3.10"
 dependencies = [
-    "pactkit>=2.9.0",
+    "pactkit>=2.17.0",
     "tomli>=1.1.0; python_version < '3.11'",
 ]
 

{pactkit_codex-2.10.2 → pactkit_codex-2.17.0}/src/pactkit_codex/deployer.py

@@ -14,10 +14,6 @@ Codex-specific features:
 import re
 from pathlib import Path
 
-try:
-    import tomllib
-except ModuleNotFoundError:
-    import tomli as tomllib  # Python 3.10 fallback
 from pactkit import __version__, prompts
 from pactkit.config import (
     VALID_SKILLS,
@@ -35,7 +31,6 @@ from pactkit.prompts.rules import (
     CREDENTIAL_SAFETY_FILE,
     RULES_FILES,
 )
-from pactkit.skills import load_script
 from pactkit.utils import atomic_write
 
 # Commands excluded from Codex deployment (require multi-agent capabilities)
@@ -130,6 +125,11 @@ class CodexDeployer(DeployerBase):
 
         self.write_version_marker(codex_root)
 
+        # STORY-slim-139 R2: machine-readable deployment manifest
+        from pactkit.deploy_manifest import write_deploy_manifest
+
+        write_deploy_manifest(codex_root, "codex", cfg)
+
         print(
             f"\n✅ Codex CLI: {n_skills} Skills, {n_commands} Commands → {codex_root}"
         )
@@ -168,6 +168,7 @@ class CodexDeployer(DeployerBase):
                 content = content.replace(pattern, "~/.codex/")
             content = CodexDeployer.strip_model_references(content)
             content = DeployerBase.strip_excluded_command_references(content, profile)
+            content = _replace_cli_with_scripts(content)
             atomic_write(rules_dir / filename, content + "\n")
 
         cred_path = rules_dir / CREDENTIAL_SAFETY_FILE
@@ -286,7 +287,7 @@ class CodexDeployer(DeployerBase):
             content = _replace_cli_with_scripts(content)
 
             # Build SKILL.md with frontmatter + @references + content
-            rule_keys = COMMAND_RULES_MAP.get(cmd_name, ["core", "credential"])
+            rule_keys = COMMAND_RULES_MAP.get(cmd_name, ["pactkit", "credential"])
             refs = []
             for key in rule_keys:
                 if key == "credential":
@@ -306,48 +307,32 @@ class CodexDeployer(DeployerBase):
 
     @staticmethod
     def generate_codex_config_toml(codex_root):
-        """Generate or merge config.toml for Codex CLI.
-
-        R1: Create with PactKit-managed sections if absent.
-        R4: Merge additive-only for user fields when existing.
-        R5: Never write API keys or secrets.
-        R6: Mark managed sections with [pactkit:managed] comments.
+        """Create config.toml for Codex CLI — but NEVER modify an existing one.
+
+        config.toml carries the user's providers, MCP servers, project trust
+        and other sensitive state. PactKit's share is two scalar defaults and
+        one MCP entry — not worth any write risk. Policy (2026-08-13, user
+        directive after two wipe incidents):
+          - File exists  -> leave it BYTE-IDENTICAL, print the recommended
+            settings for the user to add by hand.
+          - File missing -> create it with the PactKit-managed sections.
         """
         config_path = codex_root / "config.toml"
 
-        pactkit_defaults = {
-            "sandbox_mode": "workspace-write",
-            "approval_policy": "on-request",
-        }
-        pactkit_mcp = {
-            "context7": {"url": "https://mcp.context7.com/mcp"},
-        }
-
-        FORBIDDEN_KEYS = {"api_key", "OPENAI_API_KEY", "organization"}
-
         if config_path.exists():
-            try:
-                existing = tomllib.loads(config_path.read_text())
-            except Exception:
-                # User-edited config may have non-standard TOML; preserve it
-                existing = {}
-            for key, value in pactkit_defaults.items():
-                if key not in existing:
-                    existing[key] = value
-            if "mcp_servers" not in existing:
-                existing["mcp_servers"] = {}
-            for name, cfg in pactkit_mcp.items():
-                if name not in existing["mcp_servers"]:
-                    existing["mcp_servers"][name] = cfg
-            merged = existing
-        else:
-            merged = dict(pactkit_defaults)
-            merged["mcp_servers"] = dict(pactkit_mcp)
-
-        for key in FORBIDDEN_KEYS:
-            merged.pop(key, None)
+            print("  ℹ️ config.toml exists — left untouched (PactKit never modifies it)")
+            return
 
-        _write_toml_with_markers(config_path, merged)
+        lines = [
+            "# [pactkit:managed]",
+            'sandbox_mode = "workspace-write"',
+            'approval_policy = "on-request"',
+            "",
+            "# [pactkit:managed]",
+            "[mcp_servers.context7]",
+            'url = "https://mcp.context7.com/mcp"',
+        ]
+        atomic_write(config_path, chr(10).join(lines) + chr(10))
 
     @staticmethod
     def generate_codex_project_files(project_root):
@@ -397,67 +382,32 @@ class CodexDeployer(DeployerBase):
         """Deploy skills with Codex-specific path replacement in scripts."""
         _prefix = profile.skills_path_var
 
-        scripted_skill_defs = [
-            {
-                "name": "pactkit-visualize",
-                "skill_md": prompts.SKILL_VISUALIZE_MD,
-                "script_name": "visualize.py",
-                "script_source": load_script("visualize.py"),
-            },
-            {
-                "name": "pactkit-board",
-                "skill_md": prompts.SKILL_BOARD_MD,
-                "script_name": "board.py",
-                "script_source": load_script("board.py"),
-            },
-            {
-                "name": "pactkit-scaffold",
-                "skill_md": prompts.SKILL_SCAFFOLD_MD,
-                "script_name": "scaffold.py",
-                "script_source": load_script("scaffold.py"),
-            },
-        ]
-
-        prompt_only_skill_defs = [
-            {"name": "pactkit-trace", "skill_md": prompts.SKILL_TRACE_MD},
-            {"name": "pactkit-draw", "skill_md": prompts.SKILL_DRAW_MD},
-            {"name": "pactkit-status", "skill_md": prompts.SKILL_STATUS_MD},
-            {"name": "pactkit-doctor", "skill_md": prompts.SKILL_DOCTOR_MD},
-            {"name": "pactkit-review", "skill_md": prompts.SKILL_REVIEW_MD},
-            {"name": "pactkit-release", "skill_md": prompts.SKILL_RELEASE_MD},
-            {"name": "pactkit-analyze", "skill_md": prompts.SKILL_ANALYZE_MD},
-        ]
+        # STORY-slim-139 R4: consume the core SKILL_MANIFEST — no local
+        # hardcoded skill list (the old 10-item snapshot silently dropped
+        # garden/audit/report when core added them).
+        from pactkit.generators.deployer import _render_skill_md
+        from pactkit.prompts.skills import get_skill_manifest
 
         enabled_set = set(enabled_skills)
         deployed = 0
 
-        for sd in scripted_skill_defs:
-            if sd["name"] not in enabled_set:
-                continue
-            skill_dir = skills_dir / sd["name"]
-            scripts_dir = skill_dir / "scripts"
-            scripts_dir.mkdir(parents=True, exist_ok=True)
-
-            from pactkit.generators.deployer import _render_skill_md
-            skill_md = _render_skill_md(sd, profile, _prefix)
-            skill_md = _replace_cli_with_scripts(skill_md)
-            atomic_write(skill_dir / "SKILL.md", skill_md)
-            script_content = sd["script_source"]
-            script_content = script_content.replace("~/.claude/", f"{profile.global_config_dir}/")
-            script_content = script_content.replace("~/.config/opencode/", f"{profile.global_config_dir}/")
-            atomic_write(scripts_dir / sd["script_name"], script_content)
-            deployed += 1
-
-        for sd in prompt_only_skill_defs:
+        for sd in get_skill_manifest():
             if sd["name"] not in enabled_set:
                 continue
             skill_dir = skills_dir / sd["name"]
             skill_dir.mkdir(parents=True, exist_ok=True)
 
-            from pactkit.generators.deployer import _render_skill_md
             skill_md = _render_skill_md(sd, profile, _prefix)
             skill_md = _replace_cli_with_scripts(skill_md)
+            skill_md = CodexDeployer.strip_model_references(skill_md)
             atomic_write(skill_dir / "SKILL.md", skill_md)
+            if sd["script_name"]:
+                scripts_dir = skill_dir / "scripts"
+                scripts_dir.mkdir(exist_ok=True)
+                script_content = sd["script_source"]
+                script_content = script_content.replace("~/.claude/", f"{profile.global_config_dir}/")
+                script_content = script_content.replace("~/.config/opencode/", f"{profile.global_config_dir}/")
+                atomic_write(scripts_dir / sd["script_name"], script_content)
             deployed += 1
 
         return deployed
@@ -594,7 +544,9 @@ _SCAFFOLD_SCRIPT = "python3 ~/.codex/skills/pactkit-scaffold/scripts/scaffold.py
 _CLI_TO_SCRIPT = [
     # visualize variants (order matters: longer patterns first)
     ("Run `pactkit visualize --lazy`", f"Run `{_VIZ_SCRIPT}` (file, `--mode class`, `--mode call` if source changed)"),
+    ("`pactkit visualize --lazy`", f"`{_VIZ_SCRIPT}` (file, `--mode class`, `--mode call` if source changed)"),
     ("Run `pactkit visualize", f"Run `{_VIZ_SCRIPT}"),
+    ("`pactkit visualize", f"`{_VIZ_SCRIPT}"),
     ("Run `visualize --focus", f"Run `{_VIZ_SCRIPT} --focus"),
     ("Run `visualize --mode", f"Run `{_VIZ_SCRIPT} --mode"),
     ("Run `visualize`", f"Run `{_VIZ_SCRIPT}`"),
@@ -602,8 +554,14 @@ _CLI_TO_SCRIPT = [
     ("Run `python3 ~/.codex/skills/pactkit-board/scripts/board.py", f"Run `{_BOARD_SCRIPT}"),
     # scaffold
     ("Run `python3 ~/.codex/skills/pactkit-scaffold/scripts/scaffold.py", f"Run `{_SCAFFOLD_SCRIPT}"),
-    # pactkit CLI → manual fallback hint (for commands without script equivalents)
+    # pactkit CLI → manual fallback hints (for commands without script equivalents)
     ("Run `pactkit clean`", "Run language-specific cleanup (e.g., `find . -name '__pycache__' -exec rm -rf {} +` for Python)"),
+    ("`pactkit update", "run `pactkit init --format codex` to reinstall"),
+    ("`pactkit lint", "run your language linter directly (e.g., `ruff check src/ tests/` for Python)"),
+    ("`pactkit regression", "run the full test suite directly (e.g., `python3 -m pytest tests/ -v`)"),
+    ("`pactkit context", "update `docs/product/context.md` manually"),
+    ("`pactkit guard", "run your linter and test suite directly"),
+    ("`pactkit doctor", "check project files and structure manually"),
 ]
 
 
@@ -624,42 +582,6 @@ def _replace_cli_with_scripts(content):
     return content
 
 
-def _write_toml_with_markers(path, data):
-    """Write a dict as TOML with [pactkit:managed] comment markers."""
-    lines = ["# [pactkit:managed]"]
-
-    for key, value in sorted(data.items()):
-        if isinstance(value, dict):
-            continue
-        lines.append(f'{key} = {_toml_value(value)}')
-
-    lines.append("")
-
-    for key, value in sorted(data.items()):
-        if not isinstance(value, dict):
-            continue
-        if key == "mcp_servers":
-            lines.append("# [pactkit:managed]")
-        for sub_key, sub_val in sorted(value.items()):
-            quoted_sub = _toml_key(sub_key)
-            if isinstance(sub_val, dict):
-                lines.append(f"[{key}.{quoted_sub}]")
-                for k, v in sorted(sub_val.items()):
-                    lines.append(f'{_toml_key(k)} = {_toml_value(v)}')
-            else:
-                lines.append(f"[{key}]")
-                lines.append(f'{quoted_sub} = {_toml_value(sub_val)}')
-            lines.append("")
-
-    atomic_write(path, "\n".join(lines) + "\n")
-
-
-def _toml_key(key):
-    """Quote a TOML key if it contains characters outside [A-Za-z0-9_-]."""
-    if re.fullmatch(r'[A-Za-z0-9_-]+', key):
-        return key
-    return f'"{key}"'
-
 
 def _toml_value(value):
     """Format a Python value as a TOML value string."""

{pactkit_codex-2.10.2 → pactkit_codex-2.17.0/src/pactkit_codex.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: pactkit-codex
-Version: 2.10.2
+Version: 2.17.0
 Summary: PactKit adapter for OpenAI Codex CLI
 Author: Slim
 License: MIT
@@ -21,7 +21,7 @@ Classifier: Topic :: Software Development :: Quality Assurance
 Requires-Python: >=3.10
 Description-Content-Type: text/markdown
 License-File: LICENSE
-Requires-Dist: pactkit>=2.9.0
+Requires-Dist: pactkit>=2.17.0
 Requires-Dist: tomli>=1.1.0; python_version < "3.11"
 Dynamic: license-file
 

{pactkit_codex-2.10.2 → pactkit_codex-2.17.0}/src/pactkit_codex.egg-info/requires.txt

@@ -1,4 +1,4 @@
-pactkit>=2.9.0
+pactkit>=2.17.0
 
 [:python_version < "3.11"]
 tomli>=1.1.0