skm-cli 0.2.1__tar.gz → 0.2.2__tar.gz

{skm_cli-0.2.1 → skm_cli-0.2.2}/AGENTS.md

@@ -81,6 +81,9 @@ packages:
     agents:                  # optional: further filter agents for this package
       excludes:
         - standard
+  - local_path: ~/Code/devblocks
+    skills_excludes:         # optional: install all skills EXCEPT these (mutually exclusive with skills)
+      - team-squad
   - repo: https://github.com/blader/humanizer   # installs all detected skills to default agents
 ```
 

{skm_cli-0.2.1 → skm_cli-0.2.2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: skm-cli
-Version: 0.2.1
+Version: 0.2.2
 Summary: Skill Manager - manage agent skills from GitHub repos
 Author-email: Reorx <novoreorx@gmail.com>
 Requires-Python: >=3.12
@@ -96,6 +96,8 @@ agents:
 
 packages:
   - repo: https://github.com/vercel-labs/agent-skills
+    clone_strategy: shallow       # optional: use git clone --filter=blob:none --depth 1
+    skills_dir: skills/.curated   # optional: search only this repo-relative directory
     skills:                  # optional: install only these skills (omit = all)
       - vercel-react-best-practices
     agents:                  # optional: further filter agents for this package
@@ -107,10 +109,21 @@ packages:
   - local_path: ~/Code/my-custom-skills         # use a local directory as package source
     skills:
       - my-skill
+
+  - local_path: ~/Code/devblocks
+    skills_excludes:         # optional: install all skills EXCEPT these (mutually exclusive with skills)
+      - team-squad
+      - team-council
 ```
 
 Each package must specify exactly one of `repo` or `local_path`. Local path packages use the directory directly (no cloning) and are skipped by `check-updates` and `update`.
 
+`clone_strategy: shallow` is optional for repo packages. When set, the first clone uses `git clone --filter=blob:none --depth 1`; omitted or other values keep the default clone command, `git clone --filter=blob:none`. Existing cached repos are reused and pulled, so changing this option does not re-clone an already-present repo.
+
+`skills_dir` is optional and must be a relative path inside the package. When set, skm searches only that directory for `SKILL.md` files instead of using the default detection order.
+
+`skills` and `skills_excludes` are mutually exclusive. `skills` installs only the listed skills; `skills_excludes` installs all detected skills except the listed ones. Omitting both installs all detected skills.
+
 ## Install Sync Behavior
 
 `skm install` treats `skills.yaml` as a declarative state file. Each run syncs the agent directories to match the config:
@@ -130,6 +143,8 @@ A skill is a directory containing a `SKILL.md` file with YAML frontmatter (`name
 3. Otherwise — walk all subdirectories from repo root
 4. Stop descending once `SKILL.md` is found (no nested skills)
 
+If a package sets `skills_dir`, this default order is bypassed and only that directory is scanned. If `skills_dir` itself contains `SKILL.md`, it is treated as a singleton skill.
+
 ## Known Agents
 
 Skills are symlinked into these directories by default:

{skm_cli-0.2.1 → skm_cli-0.2.2}/README.md

@@ -85,6 +85,8 @@ agents:
 
 packages:
   - repo: https://github.com/vercel-labs/agent-skills
+    clone_strategy: shallow       # optional: use git clone --filter=blob:none --depth 1
+    skills_dir: skills/.curated   # optional: search only this repo-relative directory
     skills:                  # optional: install only these skills (omit = all)
       - vercel-react-best-practices
     agents:                  # optional: further filter agents for this package
@@ -96,10 +98,21 @@ packages:
   - local_path: ~/Code/my-custom-skills         # use a local directory as package source
     skills:
       - my-skill
+
+  - local_path: ~/Code/devblocks
+    skills_excludes:         # optional: install all skills EXCEPT these (mutually exclusive with skills)
+      - team-squad
+      - team-council
 ```
 
 Each package must specify exactly one of `repo` or `local_path`. Local path packages use the directory directly (no cloning) and are skipped by `check-updates` and `update`.
 
+`clone_strategy: shallow` is optional for repo packages. When set, the first clone uses `git clone --filter=blob:none --depth 1`; omitted or other values keep the default clone command, `git clone --filter=blob:none`. Existing cached repos are reused and pulled, so changing this option does not re-clone an already-present repo.
+
+`skills_dir` is optional and must be a relative path inside the package. When set, skm searches only that directory for `SKILL.md` files instead of using the default detection order.
+
+`skills` and `skills_excludes` are mutually exclusive. `skills` installs only the listed skills; `skills_excludes` installs all detected skills except the listed ones. Omitting both installs all detected skills.
+
 ## Install Sync Behavior
 
 `skm install` treats `skills.yaml` as a declarative state file. Each run syncs the agent directories to match the config:
@@ -119,6 +132,8 @@ A skill is a directory containing a `SKILL.md` file with YAML frontmatter (`name
 3. Otherwise — walk all subdirectories from repo root
 4. Stop descending once `SKILL.md` is found (no nested skills)
 
+If a package sets `skills_dir`, this default order is bypassed and only that directory is scanned. If `skills_dir` itself contains `SKILL.md`, it is treated as a singleton skill.
+
 ## Known Agents
 
 Skills are symlinked into these directories by default:

{skm_cli-0.2.1 → skm_cli-0.2.2}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "skm-cli"
-version = "0.2.1"
+version = "0.2.2"
 description = "Skill Manager - manage agent skills from GitHub repos"
 readme = "README.md"
 authors = [

skm_cli-0.2.2/skills.example.yaml

@@ -0,0 +1,76 @@
+# skills.example.yaml — copy to ~/.config/skm/skills.yaml and edit.
+#
+# skm treats this file as declarative state: it clones each package, detects
+# skills (directories containing a SKILL.md), and symlinks them into your agent
+# skill dirs. Skills you remove from here are unlinked on the next `skm install`.
+
+# Which agents from KNOWN_AGENTS receive skills by default.
+# Omit this whole block to target every known agent.
+# Known agents: standard, claude, codex, openclaw, pi
+agents:
+  default:
+    - claude
+    - codex
+    - standard
+
+packages:
+  # --- Local package --------------------------------------------------------
+  # Point at a directory on disk instead of a git repo — ideal for skills you
+  # author yourself. It's symlinked in place, so your edits apply immediately.
+  - local_path: ~/Code/my-skills
+
+  # --- Whole repo -----------------------------------------------------------
+  # No `skills` filter → every skill detected in the repo is installed.
+  - repo: https://github.com/blader/humanizer
+
+  # --- Pick specific skills -------------------------------------------------
+  # A repo often ships many skills; list only the ones you want.
+  - repo: https://github.com/vercel-labs/agent-skills
+    skills:
+      - vercel-react-best-practices
+      - web-design-guidelines
+
+  # --- Per-package agent filtering ------------------------------------------
+  # `includes` narrows this package to a subset of the default agents.
+  # Use `excludes` for the inverse. (The two are mutually exclusive.)
+  - repo: https://github.com/vercel-labs/agent-browser
+    skills:
+      - agent-browser   # drive a real browser: navigate, fill forms, screenshot
+      - dogfood         # exploratory-test a web app and report bugs
+      - electron        # automate Electron desktop apps (VS Code, Slack, ...)
+    agents:
+      includes:
+        - claude
+        - codex
+
+  # --- Shallow clone --------------------------------------------------------
+  # For big repos where history doesn't matter: clone with --depth 1.
+  - repo: https://github.com/getpaseo/paseo
+    clone_strategy: shallow
+
+  # --- Custom skills directory ----------------------------------------------
+  # When a repo keeps skills somewhere other than its root or ./skills/,
+  # point skm at the right subdirectory with `skills_dir`.
+  - repo: https://github.com/pbakaus/impeccable
+    skills_dir: source
+
+  # --- A grab bag of useful real-world skills -------------------------------
+  - repo: https://github.com/firecrawl/cli
+    skills:
+      - firecrawl-scrape   # extract clean markdown from any URL (JS-rendered too)
+      - firecrawl-search   # web search returning full-page content, not snippets
+  - repo: http://github.com/better-auth/skills
+    skills:
+      - better-auth-best-practices
+      - create-auth-skill
+  - repo: https://github.com/PsiACE/skills
+    skills:
+      - friendly-python    # write Pythonic, readable code
+      - piglet             # deeper Python craftsmanship review
+  - repo: https://github.com/cloudflare/skills
+    skills:
+      - cloudflare         # Workers, KV, D1, R2, AI, and more
+      - wrangler           # the Cloudflare Workers CLI
+  - repo: https://github.com/getsentry/skills
+    skills:
+      - django-perf-review # find N+1 queries and ORM bottlenecks

{skm_cli-0.2.1 → skm_cli-0.2.2}/src/skm/commands/install.py

@@ -179,7 +179,10 @@ def _install_local(repo_config, new_lock_skills, configured_skill_keys, known_ag
     if verbose:
         click.echo(click.style(f'Using local path {source_label}', fg='blue', bold=True))
 
-    detected = detect_skills(local_path)
+    try:
+        detected = detect_skills(local_path, repo_config.skills_dir)
+    except ValueError as e:
+        raise click.ClickException(str(e))
     if verbose:
         click.echo(click.style(f'  Found skills: {", ".join(s.name for s in detected) or "(none)"}', dim=True))
 
@@ -191,6 +194,9 @@ def _install_local(repo_config, new_lock_skills, configured_skill_keys, known_ag
         missing = requested - {s.name for s in skills_to_install}
         if missing:
             click.echo(click.style(f'  Warning: skills not found: {missing}', fg='red'))
+    elif repo_config.skills_excludes is not None:
+        excluded = set(repo_config.skills_excludes)
+        skills_to_install = [s for s in detected if s.name not in excluded]
     else:
         skills_to_install = detected
 
@@ -259,7 +265,8 @@ def _install_repo(
     repo_dir_name = repo_url_to_dirname(repo_config.repo)
     repo_path = store_dir / repo_dir_name
 
-    if repo_path.exists() and (repo_path / '.git').exists():
+    was_existing = repo_path.exists() and (repo_path / '.git').exists()
+    if was_existing:
         if verbose:
             click.echo(click.style(f'Using existing {repo_config.repo}', fg='blue', bold=True))
     else:
@@ -267,10 +274,13 @@ def _install_repo(
             _progress(f'  Cloning {repo_config.repo}...')
         else:
             click.echo(click.style(f'Cloning {repo_config.repo}...', fg='blue', bold=True))
-        clone_or_pull(repo_config.repo, repo_path)
+        clone_or_pull(repo_config.repo, repo_path, clone_strategy=repo_config.clone_strategy)
 
     commit = get_head_commit(repo_path)
-    detected = detect_skills(repo_path)
+    try:
+        detected = detect_skills(repo_path, repo_config.skills_dir)
+    except ValueError as e:
+        raise click.ClickException(str(e))
     if verbose:
         click.echo(click.style(f'  Found skills: {", ".join(s.name for s in detected) or "(none)"}', dim=True))
 
@@ -280,8 +290,29 @@ def _install_repo(
         requested = set(repo_config.skills)
         skills_to_install = [s for s in detected if s.name in requested]
         missing = requested - {s.name for s in skills_to_install}
-        if missing:
+        if missing and was_existing:
+            # Repo was already cloned but requested skills are missing — pull and retry
+            if not verbose:
+                _progress(f'  Pulling {repo_config.repo} (missing skills: {", ".join(sorted(missing))})...')
+            else:
+                click.echo(click.style(f'  Pulling {repo_config.repo} (missing skills: {", ".join(sorted(missing))})...', fg='blue'))
+            clone_or_pull(repo_config.repo, repo_path, clone_strategy=repo_config.clone_strategy)
+            commit = get_head_commit(repo_path)
+            try:
+                detected = detect_skills(repo_path, repo_config.skills_dir)
+            except ValueError as e:
+                raise click.ClickException(str(e))
+            if verbose:
+                click.echo(click.style(f'  Found skills after pull: {", ".join(s.name for s in detected) or "(none)"}', dim=True))
+            skills_to_install = [s for s in detected if s.name in requested]
+            still_missing = requested - {s.name for s in skills_to_install}
+            if still_missing:
+                click.echo(click.style(f'  Warning: skills not found in repo: {still_missing}', fg='red'))
+        elif missing:
             click.echo(click.style(f'  Warning: skills not found in repo: {missing}', fg='red'))
+    elif repo_config.skills_excludes is not None:
+        excluded = set(repo_config.skills_excludes)
+        skills_to_install = [s for s in detected if s.name not in excluded]
     else:
         skills_to_install = detected
 

{skm_cli-0.2.1 → skm_cli-0.2.2}/src/skm/commands/update.py

@@ -66,6 +66,16 @@ def _update_repo(
     repo_dir_name = repo_url_to_dirname(repo_url)
     repo_path = store_dir / repo_dir_name
 
+    repo_config = None
+    for c in config.packages:
+        if c.repo == repo_url:
+            repo_config = c
+            break
+
+    if repo_config is None:
+        click.echo(f"  Error: repo '{repo_url}' not found in config. Cannot determine target agents.")
+        raise SystemExit(1)
+
     # Get old commit from any skill in this repo
     old_commit = None
     for s in lock.skills:
@@ -75,7 +85,7 @@ def _update_repo(
 
     # Pull latest
     click.echo(f'Pulling {click.style(repo_url, fg="cyan")}...')
-    clone_or_pull(repo_url, repo_path)
+    clone_or_pull(repo_url, repo_path, clone_strategy=repo_config.clone_strategy)
     new_commit = get_head_commit(repo_path)
 
     if old_commit == new_commit:
@@ -90,16 +100,10 @@ def _update_repo(
             click.echo(click.style(f'    {line}', dim=True))
 
     # Re-detect and re-link
-    detected = detect_skills(repo_path)
-    repo_config = None
-    for c in config.packages:
-        if c.repo == repo_url:
-            repo_config = c
-            break
-
-    if repo_config is None:
-        click.echo(f"  Error: repo '{repo_url}' not found in config. Cannot determine target agents.")
-        raise SystemExit(1)
+    try:
+        detected = detect_skills(repo_path, repo_config.skills_dir)
+    except ValueError as e:
+        raise click.ClickException(str(e))
 
     target_agents = resolve_target_agents(
         repo_config.agents,

{skm_cli-0.2.1 → skm_cli-0.2.2}/src/skm/detect.py

@@ -16,8 +16,20 @@ def parse_skill_name(skill_md_path: Path) -> str:
     raise ValueError(f"No 'name' field in frontmatter of {skill_md_path}")
 
 
-def detect_skills(repo_path: Path) -> list[DetectedSkill]:
+def detect_skills(repo_path: Path, skills_dir: str | None = None) -> list[DetectedSkill]:
     """Detect skills in a cloned repo by walking for SKILL.md files."""
+    if skills_dir is not None:
+        walk_root = repo_path / skills_dir
+        if not walk_root.is_dir():
+            raise ValueError(f"skills_dir does not exist or is not a directory: {skills_dir}")
+
+        root_skill = walk_root / 'SKILL.md'
+        if root_skill.exists():
+            name = parse_skill_name(root_skill)
+            return [DetectedSkill(name=name, path=walk_root, relative_path=str(walk_root.relative_to(repo_path)))]
+
+        return _walk_for_skills(walk_root, repo_path)
+
     # Case 1: Root has SKILL.md → singleton skill
     root_skill = repo_path / 'SKILL.md'
     if root_skill.exists():

{skm_cli-0.2.1 → skm_cli-0.2.2}/src/skm/git.py

@@ -44,14 +44,18 @@ def _validate_sha(sha: str) -> None:
         raise ValueError(f"Invalid commit SHA: {sha!r}")
 
 
-def clone_or_pull(repo_url: str, dest: Path) -> None:
+def clone_or_pull(repo_url: str, dest: Path, clone_strategy: str | None = None) -> None:
     """Clone repo if not present, otherwise pull latest."""
     if dest.exists() and (dest / ".git").exists():
         run_cmd(["git", "pull", "--ff-only"], cwd=dest)
     else:
         _validate_repo_url(repo_url)
         dest.parent.mkdir(parents=True, exist_ok=True)
-        run_cmd(["git", "clone", "--filter=blob:none", repo_url, str(dest)])
+        cmd = ["git", "clone", "--filter=blob:none"]
+        if clone_strategy == "shallow":
+            cmd.extend(["--depth", "1"])
+        cmd.extend([repo_url, str(dest)])
+        run_cmd(cmd)
 
 
 def get_head_commit(repo_path: Path) -> str:

{skm_cli-0.2.1 → skm_cli-0.2.2}/src/skm/types.py

@@ -1,5 +1,5 @@
 import os
-from pathlib import Path
+from pathlib import Path, PurePosixPath
 from pydantic import BaseModel, field_validator, model_validator
 
 
@@ -20,7 +20,10 @@ class AgentsConfig(BaseModel):
 class SkillRepoConfig(BaseModel):
     repo: str | None = None
     local_path: str | None = None
+    clone_strategy: str | None = None
+    skills_dir: str | None = None
     skills: list[str] | None = None
+    skills_excludes: list[str] | None = None
     agents: AgentsConfig | None = None
 
     @model_validator(mode='after')
@@ -31,6 +34,24 @@ class SkillRepoConfig(BaseModel):
             raise ValueError("Must specify exactly one of 'repo' or 'local_path'")
         return self
 
+    @model_validator(mode='after')
+    def check_skills_mutual_exclusion(self):
+        if self.skills is not None and self.skills_excludes is not None:
+            raise ValueError("Cannot specify both 'skills' and 'skills_excludes' in a package")
+        return self
+
+    @field_validator('skills_dir')
+    @classmethod
+    def check_skills_dir(cls, v: str | None) -> str | None:
+        if v is None:
+            return v
+        if not v.strip():
+            raise ValueError("'skills_dir' cannot be empty")
+        path = PurePosixPath(v)
+        if path.is_absolute() or '..' in path.parts:
+            raise ValueError("'skills_dir' must be a relative path within the package")
+        return v.rstrip('/')
+
     @property
     def is_local(self) -> bool:
         return self.local_path is not None

{skm_cli-0.2.1 → skm_cli-0.2.2}/tests/test_cli_e2e.py

@@ -170,6 +170,63 @@ class TestInstall:
         assert len(lock['skills']) == 1
         assert lock['skills'][0]['name'] == 'wanted'
 
+    def test_install_with_skills_excludes(self, tmp_path):
+        repo = _make_skill_repo(
+            tmp_path,
+            'repo-skill-excl',
+            [
+                {'name': 'keep-a'},
+                {'name': 'keep-b'},
+                {'name': 'dropped'},
+            ],
+        )
+        _write_config(tmp_path, [{'repo': str(repo), 'skills_excludes': ['dropped']}])
+
+        runner = CliRunner()
+        result = runner.invoke(cli, [*_cli_args(tmp_path), 'install'])
+
+        assert result.exit_code == 0, result.output
+        lock = _load_lock(tmp_path)
+        installed = sorted(s['name'] for s in lock['skills'])
+        assert installed == ['keep-a', 'keep-b']
+        assert (tmp_path / 'agents' / 'claude' / 'keep-a').is_symlink()
+        assert (tmp_path / 'agents' / 'claude' / 'keep-b').is_symlink()
+        assert not (tmp_path / 'agents' / 'claude' / 'dropped').exists()
+
+    def test_install_with_skills_excludes_local_path(self, tmp_path):
+        repo = _make_skill_repo(
+            tmp_path,
+            'local-skill-excl',
+            [
+                {'name': 'keep-me'},
+                {'name': 'drop-me'},
+            ],
+        )
+        _write_config(tmp_path, [{'local_path': str(repo), 'skills_excludes': ['drop-me']}])
+
+        runner = CliRunner()
+        result = runner.invoke(cli, [*_cli_args(tmp_path), 'install'])
+
+        assert result.exit_code == 0, result.output
+        lock = _load_lock(tmp_path)
+        installed = sorted(s['name'] for s in lock['skills'])
+        assert installed == ['keep-me']
+        assert (tmp_path / 'agents' / 'claude' / 'keep-me').is_symlink()
+        assert not (tmp_path / 'agents' / 'claude' / 'drop-me').exists()
+
+    def test_install_rejects_skills_and_skills_excludes(self, tmp_path):
+        repo = _make_skill_repo(tmp_path, 'repo-conflict', [{'name': 'a-skill'}])
+        _write_config(
+            tmp_path,
+            [{'repo': str(repo), 'skills': ['a-skill'], 'skills_excludes': ['other']}],
+        )
+
+        runner = CliRunner()
+        result = runner.invoke(cli, [*_cli_args(tmp_path), 'install'])
+
+        assert result.exit_code != 0
+        assert 'skills_excludes' in str(result.exception)
+
     def test_install_with_agents_excludes(self, tmp_path):
         repo = _make_skill_repo(tmp_path, 'repo-excl', [{'name': 'my-skill'}])
         _write_config(
@@ -264,6 +321,40 @@ class TestInstall:
         dup_skills = [s for s in lock['skills'] if s['name'] == 'dup-skill']
         assert len(dup_skills) == 1
 
+    def test_install_pulls_when_requested_skills_missing(self, tmp_path):
+        """When requested skills aren't found in an existing repo, pull and retry."""
+        # Create repo with only one skill initially
+        repo = _make_skill_repo(tmp_path, 'repo-pull', [{'name': 'old-skill'}])
+
+        # First install: only request old-skill
+        _write_config(tmp_path, [{'repo': str(repo), 'skills': ['old-skill']}])
+        runner = CliRunner()
+        result = runner.invoke(cli, [*_cli_args(tmp_path), 'install'])
+        assert result.exit_code == 0, result.output
+        assert (tmp_path / 'agents' / 'claude' / 'old-skill').is_symlink()
+
+        # Add a new skill to the source repo
+        new_skill_dir = repo / 'skills' / 'new-skill'
+        new_skill_dir.mkdir(parents=True, exist_ok=True)
+        (new_skill_dir / 'SKILL.md').write_text(
+            '---\nname: new-skill\ndescription: a new skill\n---\n# new-skill\n'
+        )
+        subprocess.run(['git', 'add', '.'], cwd=repo, capture_output=True, check=True)
+        subprocess.run(['git', 'commit', '-m', 'add new-skill'], cwd=repo, capture_output=True, check=True)
+
+        # Now request both skills — new-skill won't be in the stale clone
+        _write_config(tmp_path, [{'repo': str(repo), 'skills': ['old-skill', 'new-skill']}])
+        result = runner.invoke(cli, [*_cli_args(tmp_path), 'install'])
+        assert result.exit_code == 0, result.output
+
+        # new-skill should be found after auto-pull, no warning
+        assert 'Warning: skills not found' not in result.output
+        assert (tmp_path / 'agents' / 'claude' / 'new-skill').is_symlink()
+
+        lock = _load_lock(tmp_path)
+        names = {s['name'] for s in lock['skills']}
+        assert names == {'old-skill', 'new-skill'}
+
 
 class TestList:
     def test_list_empty(self, tmp_path):
@@ -360,6 +451,26 @@ class TestUpdate:
         assert result.exit_code == 0
         assert 'Already up to date' in result.output
 
+    def test_update_passes_clone_strategy_to_git_helper(self, tmp_path, monkeypatch):
+        repo = _make_skill_repo(tmp_path, 'repo-upd-shallow', [{'name': 'upd-skill'}])
+        _write_config(tmp_path, [{'repo': str(repo), 'clone_strategy': 'shallow'}])
+
+        runner = CliRunner()
+        install_result = runner.invoke(cli, [*_cli_args(tmp_path), 'install'])
+        assert install_result.exit_code == 0, install_result.output
+
+        clone_strategies = []
+
+        def fake_clone_or_pull(repo_url, dest, clone_strategy=None):
+            clone_strategies.append(clone_strategy)
+            subprocess.run(['git', 'pull', '--ff-only'], cwd=dest, capture_output=True, check=True)
+
+        monkeypatch.setattr('skm.commands.update.clone_or_pull', fake_clone_or_pull)
+
+        result = runner.invoke(cli, [*_cli_args(tmp_path), 'update', 'upd-skill'])
+        assert result.exit_code == 0, result.output
+        assert clone_strategies == ['shallow']
+
     def test_update_with_new_commit(self, tmp_path):
         repo = _make_skill_repo(tmp_path, 'repo-upd2', [{'name': 'upd-skill'}])
         _write_config(tmp_path, [{'repo': str(repo)}])

{skm_cli-0.2.1 → skm_cli-0.2.2}/tests/test_config.py

@@ -7,6 +7,7 @@ from skm.types import SkillRepoConfig
 EXAMPLE_YAML = """\
 packages:
   - repo: https://github.com/vercel-labs/agent-skills
+    clone_strategy: shallow
     skills:
       - react-best-practices
       - web-design-guidelines
@@ -33,6 +34,7 @@ def test_load_config(tmp_path):
     config = load_config(config_file)
     assert len(config.packages) == 2
     assert config.packages[0].repo == 'https://github.com/vercel-labs/agent-skills'
+    assert config.packages[0].clone_strategy == 'shallow'
     assert config.packages[0].skills == ['react-best-practices', 'web-design-guidelines']
     assert config.packages[0].agents.excludes == ['openclaw']
     assert config.packages[1].repo == 'https://github.com/blader/humanizer'

{skm_cli-0.2.1 → skm_cli-0.2.2}/tests/test_detect.py

@@ -80,6 +80,36 @@ def test_detect_skills_skips_git_dir(tmp_path):
     assert skills[0].name == 'real-skill'
 
 
+def test_detect_skills_in_explicit_skills_dir(tmp_path):
+    """Explicit skills_dir limits discovery to that directory."""
+    _write_skill_md(tmp_path / 'skills' / 'ignored', 'ignored')
+    _write_skill_md(tmp_path / 'certain' / 'relative' / 'path' / 'skill-a', 'skill-a')
+    _write_skill_md(tmp_path / 'certain' / 'relative' / 'path' / 'nested' / 'skill-b', 'skill-b')
+
+    skills = detect_skills(tmp_path, skills_dir='certain/relative/path')
+    names = {s.name for s in skills}
+    assert names == {'skill-a', 'skill-b'}
+    assert {s.relative_path for s in skills} == {
+        'certain/relative/path/skill-a',
+        'certain/relative/path/nested/skill-b',
+    }
+
+
+def test_detect_explicit_skills_dir_can_be_singleton(tmp_path):
+    """skills_dir may point directly at one skill directory."""
+    _write_skill_md(tmp_path / 'packages' / 'one-skill', 'one-skill')
+
+    skills = detect_skills(tmp_path, skills_dir='packages/one-skill')
+    assert len(skills) == 1
+    assert skills[0].name == 'one-skill'
+    assert skills[0].relative_path == 'packages/one-skill'
+
+
+def test_detect_missing_explicit_skills_dir_raises(tmp_path):
+    with pytest.raises(ValueError, match='skills_dir does not exist'):
+        detect_skills(tmp_path, skills_dir='missing')
+
+
 def test_parse_skill_name_strips_quotes(tmp_path):
     """Quoted name values in frontmatter should have quotes stripped."""
     from skm.detect import parse_skill_name

{skm_cli-0.2.1 → skm_cli-0.2.2}/tests/test_git.py

@@ -34,6 +34,54 @@ def test_clone_and_get_commit(tmp_path):
     clone_or_pull(str(src), dest)
 
 
+def test_clone_uses_partial_clone_by_default(tmp_path, monkeypatch):
+    calls = []
+
+    def fake_run_cmd(cmd, **kwargs):
+        calls.append((cmd, kwargs))
+        return subprocess.CompletedProcess(cmd, 0, stdout="")
+
+    monkeypatch.setattr("skm.git.run_cmd", fake_run_cmd)
+
+    dest = tmp_path / "store" / "repo"
+    clone_or_pull("https://github.com/example/repo", dest)
+
+    assert calls == [
+        (
+            ["git", "clone", "--filter=blob:none", "https://github.com/example/repo", str(dest)],
+            {},
+        )
+    ]
+
+
+def test_clone_uses_depth_for_shallow_strategy(tmp_path, monkeypatch):
+    calls = []
+
+    def fake_run_cmd(cmd, **kwargs):
+        calls.append((cmd, kwargs))
+        return subprocess.CompletedProcess(cmd, 0, stdout="")
+
+    monkeypatch.setattr("skm.git.run_cmd", fake_run_cmd)
+
+    dest = tmp_path / "store" / "repo"
+    clone_or_pull("https://github.com/example/repo", dest, clone_strategy="shallow")
+
+    assert calls == [
+        (
+            [
+                "git",
+                "clone",
+                "--filter=blob:none",
+                "--depth",
+                "1",
+                "https://github.com/example/repo",
+                str(dest),
+            ],
+            {},
+        )
+    ]
+
+
 @pytest.mark.network
 def test_clone_real_repo(tmp_path):
     """Clone a real GitHub repo successfully."""

{skm_cli-0.2.1 → skm_cli-0.2.2}/tests/test_install.py

@@ -63,6 +63,73 @@ def test_install_basic(tmp_path):
     assert lock_path.exists()
 
 
+def test_install_passes_clone_strategy_to_git_helper(tmp_path, monkeypatch):
+    repo = _make_skill_repo(tmp_path, 'shallow-skill')
+
+    config_path = tmp_path / 'config' / 'skills.yaml'
+    config_path.parent.mkdir(parents=True)
+    config_path.write_text(
+        f'packages:\n  - repo: {repo}\n    clone_strategy: shallow\n    skills:\n      - shallow-skill\n'
+    )
+
+    lock_path = tmp_path / 'config' / 'skills-lock.yaml'
+    store_dir = tmp_path / 'store'
+    agents = {'claude': str(tmp_path / 'agents' / 'claude' / 'skills')}
+    clone_strategies = []
+
+    def fake_clone_or_pull(repo_url, dest, clone_strategy=None):
+        clone_strategies.append(clone_strategy)
+        subprocess.run(['git', 'clone', repo_url, str(dest)], capture_output=True, check=True)
+
+    monkeypatch.setattr('skm.commands.install.clone_or_pull', fake_clone_or_pull)
+
+    config = load_config(config_path)
+    run_install(config=config, lock_path=lock_path, store_dir=store_dir, known_agents=agents)
+
+    assert clone_strategies == ['shallow']
+    assert (tmp_path / 'agents' / 'claude' / 'skills' / 'shallow-skill').is_symlink()
+
+
+def test_install_with_skills_dir_limits_detection(tmp_path):
+    """Install only skills below a package's explicit skills_dir."""
+    repo = tmp_path / 'repo-skills-dir'
+    repo.mkdir()
+    subprocess.run(['git', 'init'], cwd=repo, capture_output=True)
+    subprocess.run(['git', 'config', 'user.email', 't@t.com'], cwd=repo, capture_output=True)
+    subprocess.run(['git', 'config', 'user.name', 'T'], cwd=repo, capture_output=True)
+
+    wanted = repo / 'certain' / 'relative' / 'path' / 'wanted'
+    wanted.mkdir(parents=True)
+    (wanted / 'SKILL.md').write_text('---\nname: wanted\ndescription: test\n---\nContent\n')
+
+    ignored = repo / 'skills' / 'ignored'
+    ignored.mkdir(parents=True)
+    (ignored / 'SKILL.md').write_text('---\nname: ignored\ndescription: test\n---\nContent\n')
+
+    subprocess.run(['git', 'add', '.'], cwd=repo, capture_output=True)
+    subprocess.run(['git', 'commit', '-m', 'init'], cwd=repo, capture_output=True)
+
+    config_path = tmp_path / 'config' / 'skills.yaml'
+    config_path.parent.mkdir(parents=True)
+    config_path.write_text(
+        f'packages:\n  - repo: {repo}\n    skills_dir: certain/relative/path\n'
+    )
+
+    lock_path = tmp_path / 'config' / 'skills-lock.yaml'
+    store_dir = tmp_path / 'store'
+    agents = {'claude': str(tmp_path / 'agents' / 'claude' / 'skills')}
+
+    config = load_config(config_path)
+    run_install(config=config, lock_path=lock_path, store_dir=store_dir, known_agents=agents)
+
+    assert (tmp_path / 'agents' / 'claude' / 'skills' / 'wanted').is_symlink()
+    assert not (tmp_path / 'agents' / 'claude' / 'skills' / 'ignored').exists()
+
+    lock = load_lock(lock_path)
+    assert [s.name for s in lock.skills] == ['wanted']
+    assert lock.skills[0].skill_path == 'certain/relative/path/wanted'
+
+
 def test_install_singleton_skill(tmp_path):
     """Install a singleton skill (SKILL.md at repo root)."""
     repo = _make_skill_repo(tmp_path, 'singleton', skills_subdir=False)

skm_cli-0.2.2/tests/test_types.py

@@ -0,0 +1,83 @@
+import pytest
+from skm.types import SkillRepoConfig, AgentsConfig
+
+
+def test_skill_repo_config_minimal():
+    """Repo with no skills listed and no agents config."""
+    cfg = SkillRepoConfig(repo="https://github.com/blader/humanizer")
+    assert cfg.repo == "https://github.com/blader/humanizer"
+    assert cfg.skills is None
+    assert cfg.agents is None
+
+
+def test_skill_repo_config_with_skills():
+    cfg = SkillRepoConfig(
+        repo="https://github.com/vercel-labs/agent-skills",
+        skills=["react-best-practices", "react-native-skills"],
+    )
+    assert cfg.skills == ["react-best-practices", "react-native-skills"]
+
+
+def test_skill_repo_config_with_skills_excludes():
+    cfg = SkillRepoConfig(
+        local_path="~/Code/devblocks",
+        skills_excludes=["team-squad", "team-council"],
+    )
+    assert cfg.skills_excludes == ["team-squad", "team-council"]
+    assert cfg.skills is None
+
+
+def test_skill_repo_config_rejects_skills_and_skills_excludes():
+    with pytest.raises(ValueError):
+        SkillRepoConfig(
+            repo="https://github.com/example/repo",
+            skills=["a"],
+            skills_excludes=["b"],
+        )
+
+
+def test_skill_repo_config_with_skills_dir():
+    cfg = SkillRepoConfig(
+        repo="https://github.com/example/repo",
+        skills_dir="certain/relative/path",
+    )
+    assert cfg.skills_dir == "certain/relative/path"
+
+
+def test_skill_repo_config_with_clone_strategy():
+    cfg = SkillRepoConfig(
+        repo="https://github.com/example/repo",
+        clone_strategy="shallow",
+    )
+    assert cfg.clone_strategy == "shallow"
+
+
+def test_skill_repo_config_rejects_absolute_skills_dir():
+    with pytest.raises(ValueError):
+        SkillRepoConfig(repo="https://github.com/example/repo", skills_dir="/tmp/skills")
+
+
+def test_skill_repo_config_rejects_parent_traversal_skills_dir():
+    with pytest.raises(ValueError):
+        SkillRepoConfig(repo="https://github.com/example/repo", skills_dir="../skills")
+
+
+def test_agents_config_excludes():
+    agents = AgentsConfig(excludes=["openclaw"])
+    assert agents.excludes == ["openclaw"]
+    assert agents.includes is None
+
+
+def test_agents_config_includes():
+    agents = AgentsConfig(includes=["claude", "codex"])
+    assert agents.includes == ["claude", "codex"]
+    assert agents.excludes is None
+
+
+def test_skill_repo_config_with_agents():
+    cfg = SkillRepoConfig(
+        repo="https://github.com/vercel-labs/agent-skills",
+        skills=["react-best-practices"],
+        agents=AgentsConfig(excludes=["openclaw"]),
+    )
+    assert cfg.agents.excludes == ["openclaw"]

{skm_cli-0.2.1 → skm_cli-0.2.2}/uv.lock

@@ -181,7 +181,7 @@ wheels = [
 
 [[package]]
 name = "skm-cli"
-version = "0.2.0"
+version = "0.2.2"
 source = { editable = "." }
 dependencies = [
     { name = "click" },

skm_cli-0.2.1/skills.example.yaml

@@ -1,38 +0,0 @@
-agents:
-  default:
-    - claude
-    - standard
-    - codex
-    - openclaw
-
-packages:
-  - repo: https://github.com/vercel-labs/agent-skills
-    skills:
-      - vercel-react-best-practices
-      - web-design-guidelines
-    agents:
-      excludes:
-       - openclaw
-  - repo: https://github.com/ibelick/ui-skills
-    skills:
-      - baseline-ui
-  - repo: https://github.com/OthmanAdi/planning-with-files
-    skills:
-      - planning-with-files
-    agents:
-      excludes:
-       - openclaw
-  - repo: https://github.com/blader/humanizer
-  - repo: http://github.com/better-auth/skills
-    skills:
-      - better-auth-best-practices
-      - create-auth-skill
-  - repo: https://github.com/PsiACE/skills
-    skills:
-      - friendly-python
-  - repo: https://github.com/SawyerHood/dev-browser
-    skills:
-      - dev-browser
-    agents:
-      excludes:
-        - openclaw

skm_cli-0.2.1/tests/test_types.py

@@ -1,39 +0,0 @@
-import pytest
-from skm.types import SkillRepoConfig, AgentsConfig
-
-
-def test_skill_repo_config_minimal():
-    """Repo with no skills listed and no agents config."""
-    cfg = SkillRepoConfig(repo="https://github.com/blader/humanizer")
-    assert cfg.repo == "https://github.com/blader/humanizer"
-    assert cfg.skills is None
-    assert cfg.agents is None
-
-
-def test_skill_repo_config_with_skills():
-    cfg = SkillRepoConfig(
-        repo="https://github.com/vercel-labs/agent-skills",
-        skills=["react-best-practices", "react-native-skills"],
-    )
-    assert cfg.skills == ["react-best-practices", "react-native-skills"]
-
-
-def test_agents_config_excludes():
-    agents = AgentsConfig(excludes=["openclaw"])
-    assert agents.excludes == ["openclaw"]
-    assert agents.includes is None
-
-
-def test_agents_config_includes():
-    agents = AgentsConfig(includes=["claude", "codex"])
-    assert agents.includes == ["claude", "codex"]
-    assert agents.excludes is None
-
-
-def test_skill_repo_config_with_agents():
-    cfg = SkillRepoConfig(
-        repo="https://github.com/vercel-labs/agent-skills",
-        skills=["react-best-practices"],
-        agents=AgentsConfig(excludes=["openclaw"]),
-    )
-    assert cfg.agents.excludes == ["openclaw"]