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

{skm_cli-0.2.0 → 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.2/CLAUDE.md

@@ -0,0 +1 @@
+AGENTS.md

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

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: skm-cli
-Version: 0.2.0
+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:
@@ -141,6 +156,47 @@ Skills are symlinked into these directories by default:
 | `codex` | `~/.codex/skills/` |
 | `openclaw` | `~/.openclaw/skills/` |
 
+## Copy Strategy
+
+When skm links a skill into an agent directory, it picks a strategy based on the agent config and filesystem:
+
+### 1. Symlink (default)
+
+A symbolic link from `<agent_dir>/skills/<skill_name>` → `<store>/<skill_name>`. This is the default for all agents. Changes in the store are immediately visible.
+
+### 2. Hardlink
+
+When `use_hardlink: true` is set for an agent in `AGENT_OPTIONS`, skm creates hardlinks instead. Each file in the skill directory gets its own hardlink pointing to the same inode as the source. This only works when source and target are on the **same filesystem/device**.
+
+### 3. Reflink (copy-on-write)
+
+When hardlinks can't be used (source and target on **different devices**), skm attempts a reflink/COW clone. This creates an independent copy that shares physical disk blocks with the source until either side is modified — fast and space-efficient.
+
+The reflink backend is platform-specific:
+
+| Platform | Mechanism | Supported filesystems |
+|---|---|---|
+| **Linux** | `FICLONE` ioctl (`fcntl.ioctl`) | Btrfs, XFS, OCFS2, and others with reflink support |
+| **macOS** | `clonefile(2)` syscall (via `ctypes`) | APFS (default since macOS 10.13) |
+
+### 4. Plain copy (fallback)
+
+If reflink is not available (unsupported filesystem, non-Unix platform, etc.), skm falls back to a plain `shutil.copy2` — a full byte copy with metadata preserved.
+
+### Selection flow
+
+```
+use_hardlink enabled?
+├── No  → symlink
+└── Yes → same device?
+    ├── Yes → hardlink
+    └── No  → reflink supported?
+        ├── Yes → reflink (COW clone)
+        └── No  → plain copy
+```
+
+The reflink implementation is isolated in `src/skm/clonefile.py` with dedicated tests in `tests/test_clonefile.py`.
+
 ## CLI Path Overrides
 
 Override default paths for testing or custom setups:

{skm_cli-0.2.0 → 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:
@@ -130,6 +145,47 @@ Skills are symlinked into these directories by default:
 | `codex` | `~/.codex/skills/` |
 | `openclaw` | `~/.openclaw/skills/` |
 
+## Copy Strategy
+
+When skm links a skill into an agent directory, it picks a strategy based on the agent config and filesystem:
+
+### 1. Symlink (default)
+
+A symbolic link from `<agent_dir>/skills/<skill_name>` → `<store>/<skill_name>`. This is the default for all agents. Changes in the store are immediately visible.
+
+### 2. Hardlink
+
+When `use_hardlink: true` is set for an agent in `AGENT_OPTIONS`, skm creates hardlinks instead. Each file in the skill directory gets its own hardlink pointing to the same inode as the source. This only works when source and target are on the **same filesystem/device**.
+
+### 3. Reflink (copy-on-write)
+
+When hardlinks can't be used (source and target on **different devices**), skm attempts a reflink/COW clone. This creates an independent copy that shares physical disk blocks with the source until either side is modified — fast and space-efficient.
+
+The reflink backend is platform-specific:
+
+| Platform | Mechanism | Supported filesystems |
+|---|---|---|
+| **Linux** | `FICLONE` ioctl (`fcntl.ioctl`) | Btrfs, XFS, OCFS2, and others with reflink support |
+| **macOS** | `clonefile(2)` syscall (via `ctypes`) | APFS (default since macOS 10.13) |
+
+### 4. Plain copy (fallback)
+
+If reflink is not available (unsupported filesystem, non-Unix platform, etc.), skm falls back to a plain `shutil.copy2` — a full byte copy with metadata preserved.
+
+### Selection flow
+
+```
+use_hardlink enabled?
+├── No  → symlink
+└── Yes → same device?
+    ├── Yes → hardlink
+    └── No  → reflink supported?
+        ├── Yes → reflink (COW clone)
+        └── No  → plain copy
+```
+
+The reflink implementation is isolated in `src/skm/clonefile.py` with dedicated tests in `tests/test_clonefile.py`.
+
 ## CLI Path Overrides
 
 Override default paths for testing or custom setups:

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

@@ -1,6 +1,6 @@
 [project]
 name = "skm-cli"
-version = "0.2.0"
+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.2/src/skm/clonefile.py

@@ -0,0 +1,106 @@
+"""Platform-aware reflink (copy-on-write) file cloning.
+
+Supports two backends:
+- Linux: FICLONE ioctl (Btrfs, XFS, etc.)
+- macOS: clonefile(2) syscall (APFS)
+
+Both backends raise OSError on failure. Callers should catch OSError and
+check ``is_reflink_unsupported(exc)`` to decide whether to fall back to
+a plain copy.
+"""
+
+import ctypes
+import ctypes.util
+import errno
+import os
+import platform
+import shutil
+from pathlib import Path
+
+# --- errno values that mean "this filesystem doesn't support reflink" -------
+
+REFLINK_UNSUPPORTED_ERRNOS: set[int] = {
+    errno.ENOTSUP,
+    getattr(errno, 'EOPNOTSUPP', errno.ENOTSUP),
+    getattr(errno, 'ENOSYS', errno.ENOTSUP),
+    getattr(errno, 'ENOTTY', errno.ENOTSUP),
+    getattr(errno, 'EXDEV', errno.ENOTSUP),
+}
+
+_SYSTEM = platform.system()
+
+# --- Linux: FICLONE ioctl ---------------------------------------------------
+
+_FICLONE = 0x40049409  # from <linux/fs.h>
+_fcntl = None
+
+if _SYSTEM == 'Linux':
+    try:
+        import fcntl as _fcntl_mod
+
+        _fcntl = _fcntl_mod
+    except ImportError:  # pragma: no cover
+        pass
+
+
+def _clone_file_linux(src: Path, dst: Path) -> None:
+    """Clone via FICLONE ioctl (Linux only)."""
+    assert _fcntl is not None
+    with src.open('rb') as src_f, dst.open('wb') as dst_f:
+        _fcntl.ioctl(dst_f.fileno(), _FICLONE, src_f.fileno())
+    shutil.copystat(src, dst)
+
+
+# --- macOS: clonefile(2) ----------------------------------------------------
+
+_clonefile_func = None
+
+if _SYSTEM == 'Darwin':
+    _libc_path = ctypes.util.find_library('System')
+    if _libc_path:
+        _libc = ctypes.CDLL(_libc_path, use_errno=True)
+        _clonefile_func = _libc.clonefile
+        _clonefile_func.argtypes = [ctypes.c_char_p, ctypes.c_char_p, ctypes.c_int]
+        _clonefile_func.restype = ctypes.c_int
+
+
+def _clone_file_darwin(src: Path, dst: Path) -> None:
+    """Clone via clonefile(2) syscall (macOS only)."""
+    assert _clonefile_func is not None
+    ret = _clonefile_func(os.fsencode(src), os.fsencode(dst), 0)
+    if ret != 0:
+        err = ctypes.get_errno()
+        raise OSError(err, os.strerror(err), str(src))
+    shutil.copystat(src, dst)
+
+
+# --- Public API --------------------------------------------------------------
+
+
+def reflink_supported() -> bool:
+    """Return True if the current platform has a reflink backend available."""
+    if _SYSTEM == 'Linux':
+        return _fcntl is not None
+    if _SYSTEM == 'Darwin':
+        return _clonefile_func is not None
+    return False
+
+
+def clone_file(src: Path, dst: Path) -> None:
+    """Clone a single file using the platform's COW mechanism.
+
+    Raises OSError if the filesystem or platform doesn't support it.
+    Use ``is_reflink_unsupported(exc)`` to distinguish "not supported"
+    from genuine I/O errors.
+    """
+    if _SYSTEM == 'Linux' and _fcntl is not None:
+        _clone_file_linux(src, dst)
+    elif _SYSTEM == 'Darwin' and _clonefile_func is not None:
+        _clone_file_darwin(src, dst)
+    else:
+        raise OSError(errno.ENOTSUP, 'reflink is not supported on this platform')
+
+
+def is_reflink_unsupported(exc: OSError) -> bool:
+    """Return True when the error means reflink isn't available, not a real I/O failure."""
+    return exc.errno in REFLINK_UNSUPPORTED_ERRNOS

{skm_cli-0.2.0 → 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.0 → 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,
@@ -128,7 +132,7 @@ def _update_repo(
         skill = matching[0]
         linked_paths = []
         for agent_name, agent_dir in target_agents.items():
-            link, _status = link_skill(skill.path, skill.name, agent_dir, agent_name=agent_name)
+            link, _status = link_skill(skill.path, skill.name, agent_dir, force=True, agent_name=agent_name)
             linked_paths.append(compact_path(str(link)))
 
         lock.skills[i] = InstalledSkill(

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

@@ -6,26 +6,38 @@ from skm.types import DetectedSkill
 
 def parse_skill_name(skill_md_path: Path) -> str:
     """Extract 'name' from SKILL.md YAML frontmatter."""
-    text = skill_md_path.read_text(encoding="utf-8")
-    match = re.match(r"^---\s*\n(.*?)\n---", text, re.DOTALL)
+    text = skill_md_path.read_text(encoding='utf-8')
+    match = re.match(r'^---\s*\n(.*?)\n---', text, re.DOTALL)
     if not match:
-        raise ValueError(f"No frontmatter found in {skill_md_path}")
+        raise ValueError(f'No frontmatter found in {skill_md_path}')
     for line in match.group(1).splitlines():
-        if line.startswith("name:"):
-            return line.split(":", 1)[1].strip()
+        if line.startswith('name:'):
+            return line.split(':', 1)[1].strip().strip('\'"')
     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"
+    root_skill = repo_path / 'SKILL.md'
     if root_skill.exists():
         name = parse_skill_name(root_skill)
-        return [DetectedSkill(name=name, path=repo_path, relative_path=".")]
+        return [DetectedSkill(name=name, path=repo_path, relative_path='.')]
 
     # Determine walk root
-    skills_dir = repo_path / "skills"
+    skills_dir = repo_path / 'skills'
     walk_root = skills_dir if skills_dir.is_dir() else repo_path
 
     return _walk_for_skills(walk_root, repo_path)
@@ -37,9 +49,9 @@ def _walk_for_skills(walk_root: Path, repo_path: Path) -> list[DetectedSkill]:
     for child in sorted(walk_root.iterdir()):
         if not child.is_dir() or child.is_symlink():
             continue
-        if child.name.startswith("."):
+        if child.name == '.git':
             continue
-        skill_md = child / "SKILL.md"
+        skill_md = child / 'SKILL.md'
         if skill_md.exists():
             name = parse_skill_name(skill_md)
             relative = str(child.relative_to(repo_path))

{skm_cli-0.2.0 → 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: