Merge commit '35e6a301aa36e43df8458cb7c6ed40f60c642b78' into lsp-server

docs/standards/playbook/tests/cli/test_playbook_cli.py

@@ -126,6 +126,27 @@ langs = ["tsl"]
             self.assertEqual(block[bullet_idx - 1], "")
 
     def test_install_skills(self):
+        with tempfile.TemporaryDirectory() as tmp_dir:
+            target = Path(tmp_dir) / "agents"
+            config_body = f"""
+[playbook]
+project_root = "{tmp_dir}"
+
+[install_skills]
+agents_home = "{target}"
+mode = "list"
+skills = ["brainstorming"]
+"""
+            config_path = Path(tmp_dir) / "playbook.toml"
+            config_path.write_text(config_body, encoding="utf-8")
+
+            result = run_cli("-config", str(config_path))
+
+            skill_file = target / "skills/brainstorming/SKILL.md"
+            self.assertEqual(result.returncode, 0)
+            self.assertTrue(skill_file.is_file())
+
+    def test_install_skills_rejects_codex_home(self):
         with tempfile.TemporaryDirectory() as tmp_dir:
             target = Path(tmp_dir) / "codex"
             config_body = f"""
@@ -142,9 +163,8 @@ skills = ["brainstorming"]
 
             result = run_cli("-config", str(config_path))
 
-            skill_file = target / "skills/brainstorming/SKILL.md"
-            self.assertEqual(result.returncode, 0)
-            self.assertTrue(skill_file.is_file())
+            self.assertNotEqual(result.returncode, 0)
+            self.assertIn("codex_home", result.stdout + result.stderr)
 
 if __name__ == "__main__":
     unittest.main()