agent-toolkit-cli 1.0.5__tar.gz → 1.0.7__tar.gz

{agent_toolkit_cli-1.0.5 → agent_toolkit_cli-1.0.7}/CHANGELOG.md

@@ -41,6 +41,24 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 - loop runner: resolve toolkit root from package data dir before CWD fallback,
   preventing it from picking up the user's ai-workspace loops when run via uvx/pip
 
+## [1.0.6] — 2026-08-04
+
+### Fixed
+- **CRITICAL**: Local `toolkit_root()` in cli/install.py, doctor.py, mcp.py, skills.py,
+  plugin.py was defined *after* the import from `_paths.py`, shadowing it — so the
+  correct resolution logic was never called in wheel/uvx installs
+- Removed module-level `TOOLKIT_DIR: Path = toolkit_root()` (evaluated at import time,
+  crashed before any function ran) — replaced with lazy `toolkit_root()` per call site
+
+## [1.0.7] — 2026-08-04
+
+### Fixed
+- `skills list`: now correctly parses YAML block scalars (>-, |-) in SKILL.md descriptions
+  — all 52 skill descriptions now display properly instead of showing ">-"
+- `install` command: Copilot prompt no longer blocks in non-interactive/dry-run mode
+  (stdin.isatty() check; skips automatically in CI, pipes, --dry-run)
+- `pyproject.toml`: added agent-toolkit-cli script alias so `uvx agent-toolkit-cli` works
+
 ## [Unreleased]
 
 ### Added

{agent_toolkit_cli-1.0.5 → agent_toolkit_cli-1.0.7}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: agent-toolkit-cli
-Version: 1.0.5
+Version: 1.0.7
 Summary: Composable AI agent toolkit — loops, skills, profiles, and MCP for Claude Code, Cursor, OpenCode, Windsurf, and more
 Project-URL: Homepage, https://github.com/ulises-jeremias/agent-toolkit
 Project-URL: Repository, https://github.com/ulises-jeremias/agent-toolkit.git

{agent_toolkit_cli-1.0.5 → agent_toolkit_cli-1.0.7}/pyproject.toml

@@ -35,6 +35,7 @@ all = ["pyyaml>=6.0"]
 
 [project.scripts]
 agent-toolkit = "agent_toolkit.cli.main:main"
+agent-toolkit-cli = "agent_toolkit.cli.main:main"
 
 [project.urls]
 Homepage = "https://github.com/ulises-jeremias/agent-toolkit"

{agent_toolkit_cli-1.0.5 → agent_toolkit_cli-1.0.7}/src/agent_toolkit/__init__.py

@@ -1,4 +1,4 @@
 """agent-toolkit: Composable AI agent capabilities for all major AI coding tools."""
 
-__version__ = "1.0.5"
+__version__ = "1.0.7"
 __author__ = "ulises-jeremias"

{agent_toolkit_cli-1.0.5 → agent_toolkit_cli-1.0.7}/src/agent_toolkit/cli/doctor.py

@@ -56,17 +56,7 @@ class CheckResult:
         }
 
 
-# ---------------------------------------------------------------------------
-# Toolkit directory resolution
-# ---------------------------------------------------------------------------
 
-def toolkit_root() -> Path | None:
-    candidate = Path(__file__).resolve()
-    for _ in range(10):
-        candidate = candidate.parent
-        if (candidate / "profiles").is_dir():
-            return candidate
-    return None
 
 
 # ---------------------------------------------------------------------------

{agent_toolkit_cli-1.0.5 → agent_toolkit_cli-1.0.7}/src/agent_toolkit/cli/install.py

@@ -19,24 +19,9 @@ from pathlib import Path
 from agent_toolkit._paths import toolkit_root
 
 
-# ---------------------------------------------------------------------------
-# Toolkit directory resolution
-# ---------------------------------------------------------------------------
 
-def toolkit_root() -> Path:
-    """Walk up from this file until we find the directory containing profiles/."""
-    candidate = Path(__file__).resolve()
-    for _ in range(10):
-        candidate = candidate.parent
-        if (candidate / "profiles").is_dir():
-            return candidate
-    raise FileNotFoundError(
-        "Cannot locate toolkit root (directory containing profiles/). "
-        "Make sure agent-toolkit is installed from its source tree."
-    )
 
 
-TOOLKIT_DIR: Path = toolkit_root()
 
 # ---------------------------------------------------------------------------
 # Helpers
@@ -181,7 +166,7 @@ def _windsurf_config_dir() -> Path:
 def _install_claude_code(*, dry_run: bool, force: bool) -> bool:
     print()
     _info("Installing: Claude Code")
-    src = TOOLKIT_DIR / "profiles" / "claude-code"
+    src = toolkit_root() / "profiles" / "claude-code"
 
     if not src.is_dir():
         _warn(f"Profile directory not found: {src}")
@@ -202,7 +187,7 @@ def _install_claude_code(*, dry_run: bool, force: bool) -> bool:
 def _install_cursor(*, dry_run: bool, force: bool) -> bool:
     print()
     _info("Installing: Cursor")
-    src = TOOLKIT_DIR / "profiles" / "cursor" / "rules"
+    src = toolkit_root() / "profiles" / "cursor" / "rules"
 
     if not src.is_dir():
         _warn(f"Cursor rules directory not found: {src}")
@@ -215,7 +200,7 @@ def _install_cursor(*, dry_run: bool, force: bool) -> bool:
 def _install_opencode(*, dry_run: bool, force: bool) -> bool:
     print()
     _info("Installing: OpenCode")
-    src = TOOLKIT_DIR / "profiles" / "opencode"
+    src = toolkit_root() / "profiles" / "opencode"
 
     if not src.is_dir():
         _warn(f"OpenCode profile directory not found: {src}")
@@ -236,7 +221,7 @@ def _install_copilot(*, dry_run: bool, force: bool) -> bool:
     _info("Installing: GitHub Copilot")
     _info("Copilot instructions are per-project and committed to the repository.")
 
-    src = TOOLKIT_DIR / "profiles" / "copilot" / "copilot-instructions.md"
+    src = toolkit_root() / "profiles" / "copilot" / "copilot-instructions.md"
     if not src.is_file():
         _warn(f"Copilot instructions not found: {src}")
         return False
@@ -245,6 +230,11 @@ def _install_copilot(*, dry_run: bool, force: bool) -> bool:
         _dry("Would copy copilot-instructions.md to <project>/.github/")
         return True
 
+    import sys as _sys
+    if not _sys.stdin.isatty():
+        _skip("Non-interactive — run 'agent-toolkit install --tools copilot' interactively")
+        return True
+
     try:
         project_dir_raw = input("  Enter project directory path (or press Enter to skip): ").strip()
     except EOFError:
@@ -272,7 +262,7 @@ def _install_copilot(*, dry_run: bool, force: bool) -> bool:
 def _install_windsurf(*, dry_run: bool, force: bool) -> bool:
     print()
     _info("Installing: Windsurf")
-    src = TOOLKIT_DIR / "profiles" / "windsurf"
+    src = toolkit_root() / "profiles" / "windsurf"
 
     if not src.is_dir():
         _warn(f"Windsurf profile directory not found: {src}")
@@ -292,7 +282,7 @@ def _install_windsurf(*, dry_run: bool, force: bool) -> bool:
 def _install_pi(*, dry_run: bool, force: bool) -> bool:
     print()
     _info("Installing: Pi Coding Agent")
-    src = TOOLKIT_DIR / "profiles" / "pi" / "skills"
+    src = toolkit_root() / "profiles" / "pi" / "skills"
 
     if not src.is_dir():
         _warn(f"Pi skills directory not found: {src}")
@@ -359,7 +349,7 @@ def cmd_install(args: list[str]) -> int:
 
     print()
     print("agent-toolkit installer")
-    print(f"Toolkit: {TOOLKIT_DIR}")
+    print(f"Toolkit: {toolkit_root()}")
 
     if dry_run:
         print()
@@ -385,10 +375,12 @@ def cmd_install(args: list[str]) -> int:
             tools.append("pi")
             _info("  Detected: Pi Coding Agent")
 
-        # Copilot is per-project — ask
-        print()
-        if _confirm("Install GitHub Copilot instructions for a project?", force=False):
-            tools.append("copilot")
+        # Copilot is per-project — only ask in interactive sessions
+        import sys as _sys
+        if _sys.stdin.isatty() and not dry_run:
+            print()
+            if _confirm("Install GitHub Copilot instructions for a project?", force=False):
+                tools.append("copilot")
 
         if not tools:
             _warn("No AI tools detected. Install at least one supported tool and re-run,")

{agent_toolkit_cli-1.0.5 → agent_toolkit_cli-1.0.7}/src/agent_toolkit/cli/mcp.py

@@ -28,13 +28,7 @@ from agent_toolkit._paths import toolkit_root
 # Toolkit directory & config paths
 # ---------------------------------------------------------------------------
 
-def toolkit_root() -> Path | None:
-    candidate = Path(__file__).resolve()
-    for _ in range(10):
-        candidate = candidate.parent
-        if (candidate / "profiles").is_dir():
-            return candidate
-    return None
+
 
 
 def _mcp_config_path() -> Path:

{agent_toolkit_cli-1.0.5 → agent_toolkit_cli-1.0.7}/src/agent_toolkit/cli/plugin.py

@@ -20,17 +20,7 @@ from pathlib import Path
 from agent_toolkit._paths import toolkit_root
 
 
-# ---------------------------------------------------------------------------
-# Toolkit directory resolution
-# ---------------------------------------------------------------------------
 
-def toolkit_root() -> Path | None:
-    candidate = Path(__file__).resolve()
-    for _ in range(10):
-        candidate = candidate.parent
-        if (candidate / "profiles").is_dir():
-            return candidate
-    return None
 
 
 # ---------------------------------------------------------------------------

{agent_toolkit_cli-1.0.5 → agent_toolkit_cli-1.0.7}/src/agent_toolkit/cli/skills.py

@@ -24,23 +24,26 @@ import sys
 from pathlib import Path
 from agent_toolkit._paths import toolkit_root
 
+def _parse_skill_description(skill_dir):
+    skill_md = skill_dir / "SKILL.md"
+    if not skill_md.exists():
+        return ""
+    text = skill_md.read_text(errors="replace")
+    fm_match = re.match(r"^---[ \t]*\n(.*?)\n---", text, re.DOTALL)
+    if not fm_match:
+        return ""
+    fm = fm_match.group(1)
+    # Block scalar: description: >-\n  content
+    block = re.search(r"^description:[ \t]*[|>][\-+]?[ \t]*\n((?:[ \t]+.+\n?)+)", fm, re.MULTILINE)
+    if block:
+        lines = [l.strip() for l in block.group(1).splitlines() if l.strip()]
+        return " ".join(lines)[:120]
+    # Inline: description: text
+    inline = re.search(r"^description:[ \t]+(.+)$", fm, re.MULTILINE)
+    if inline:
+        return inline.group(1).strip()[:120]
+    return ""
 
-# ---------------------------------------------------------------------------
-# Toolkit directory resolution
-# ---------------------------------------------------------------------------
-
-def toolkit_root() -> Path | None:
-    candidate = Path(__file__).resolve()
-    for _ in range(10):
-        candidate = candidate.parent
-        if (candidate / "profiles").is_dir():
-            return candidate
-    return None
-
-
-# ---------------------------------------------------------------------------
-# Skills layout loading
-# ---------------------------------------------------------------------------
 
 def _load_layout(toolkit_dir: Path) -> dict | None:
     layout_path = toolkit_dir / "catalogs" / "skills-layout.json"
@@ -63,20 +66,35 @@ _SIMPLE_KV_RE = re.compile(r"^([A-Za-z_][A-Za-z0-9_-]*)\s*:\s*(.*)$")
 
 
 def _parse_frontmatter_simple(content: str) -> dict | None:
-    """Parse simple YAML frontmatter (key: value pairs only, no nesting)."""
+    """Parse YAML frontmatter — handles inline values and block scalars (>-, |-)."""
     match = _FRONTMATTER_RE.match(content)
     if not match:
         return None
     result: dict[str, str] = {}
-    for line in match.group(1).splitlines():
-        line = line.strip()
-        if not line or line.startswith("#"):
+    lines = match.group(1).splitlines()
+    i = 0
+    while i < len(lines):
+        line = lines[i]
+        stripped = line.strip()
+        if not stripped or stripped.startswith("#"):
+            i += 1
             continue
-        m = _SIMPLE_KV_RE.match(line)
+        m = _SIMPLE_KV_RE.match(stripped)
         if m:
             key = m.group(1).strip()
             val = m.group(2).strip().strip('"').strip("'")
+            # YAML block scalar — collect indented lines that follow
+            if val in (">", ">-", "|-", "|", ">+", "|+"):
+                block_lines = []
+                j = i + 1
+                while j < len(lines) and (lines[j].startswith(" ") or lines[j].startswith("\t")):
+                    block_lines.append(lines[j].strip())
+                    j += 1
+                result[key] = " ".join(block_lines)
+                i = j
+                continue
             result[key] = val
+        i += 1
     return result