skill-router 0.0.1-snapshot → 0.0.2-snapshot

package/README.md

@@ -11,9 +11,16 @@ npm install -g .
 The package runs this postinstall hook:
 
 ```bash
-python3 -m pip install fastembed numpy --user
+node scripts/postinstall.js
 ```
 
+It creates an isolated venv in `~/.cache/skill-router/venv` and installs:
+
+- `fastembed`
+- `numpy`
+
+This avoids PEP 668 / externally-managed-environment failures on Ubuntu.
+
 ## Usage
 
 ```bash
@@ -38,6 +45,11 @@ If the provided `--skills-dir` does not exist or has no `.md` files, it returns:
 {"skill_id":null,"score":0.0}
 ```
 
+Lookup order:
+
+1. `--skills-dir` (default: `~/.cache/skills-router`)
+2. Fallback: `~/.cache/gemini-skills`
+
 ## Cache
 
 Cache is stored in:

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "skill-router",
-  "version": "0.0.1-snapshot",
+  "version": "0.0.2-snapshot",
   "description": "Local semantic skill router for gemini-cli-pro",
   "license": "MIT",
   "type": "commonjs",
@@ -8,10 +8,11 @@
     "skill-router": "./router.py"
   },
   "scripts": {
-    "postinstall": "python3 -m pip install fastembed numpy --user"
+    "postinstall": "node scripts/postinstall.js"
   },
   "files": [
     "router.py",
-    "README.md"
+    "README.md",
+    "scripts"
   ]
 }

package/router.py

@@ -7,10 +7,26 @@ import argparse
 import hashlib
 import json
 import os
+import site
 import sys
 from pathlib import Path
 from typing import Iterable
 
+
+def _cache_root_dir() -> Path:
+    # Keep ~/.cache across Linux and macOS for compatibility with gemini-cli-pro integration.
+    return Path.home() / ".cache" / "skill-router"
+
+
+def _inject_cached_venv_site_packages() -> None:
+    py_tag = f"python{sys.version_info.major}.{sys.version_info.minor}"
+    candidate = _cache_root_dir() / "venv" / "lib" / py_tag / "site-packages"
+    if candidate.exists() and candidate.is_dir():
+        site.addsitedir(str(candidate))
+
+
+_inject_cached_venv_site_packages()
+
 import numpy as np
 
 try:
@@ -20,9 +36,11 @@ except Exception:  # pragma: no cover - runtime dependency may be unavailable
 
 
 MODEL_NAME = "BAAI/bge-small-en-v1.5"
-CACHE_ROOT = Path.home() / ".cache" / "skill-router"
+CACHE_ROOT = _cache_root_dir()
 DATA_FILE = CACHE_ROOT / "data.npy"
 SKILLS_CACHE_FILE = CACHE_ROOT / ".skills_cache.npy"
+DEFAULT_SKILLS_DIR = Path.home() / ".cache" / "skills-router"
+GLOBAL_GEMINI_SKILLS_DIR = Path.home() / ".cache" / "gemini-skills"
 
 _EMBEDDER = None
 
@@ -77,9 +95,14 @@ def _has_skill_markdown(skills_dir: Path) -> bool:
 
 
 def _resolve_available_skills_dir(preferred_skills_dir: Path) -> Path | None:
-    candidate = preferred_skills_dir.resolve()
-    if _has_skill_markdown(candidate):
-        return candidate
+    primary = preferred_skills_dir.resolve()
+    if _has_skill_markdown(primary):
+        return primary
+
+    fallback = GLOBAL_GEMINI_SKILLS_DIR.resolve()
+    if _has_skill_markdown(fallback):
+        return fallback
+
     return None
 
 
@@ -155,7 +178,9 @@ def _get_embedder() -> TextEmbedding:
     global _EMBEDDER
     if _EMBEDDER is None:
         if TextEmbedding is None:
-            raise RuntimeError("missing_dependency: fastembed")
+            raise RuntimeError(
+                f"missing_dependency: fastembed (expected venv at {CACHE_ROOT / 'venv'})"
+            )
         _EMBEDDER = TextEmbedding(model_name=MODEL_NAME)
     return _EMBEDDER
 
@@ -306,7 +331,7 @@ def main() -> int:
     parser.add_argument("--prompt", default="", help="User text prompt")
     parser.add_argument(
         "--skills-dir",
-        default=".skills",
+        default=str(DEFAULT_SKILLS_DIR),
         help="Path to skills directory",
     )
     parser.add_argument(

package/scripts/postinstall.js

@@ -0,0 +1,46 @@
+#!/usr/bin/env node
+
+const { spawnSync } = require("child_process");
+const fs = require("fs");
+const os = require("os");
+const path = require("path");
+
+function run(cmd, args, options = {}) {
+  const result = spawnSync(cmd, args, {
+    stdio: "inherit",
+    env: process.env,
+    ...options,
+  });
+  return result.status === 0;
+}
+
+function main() {
+  const cacheRoot = path.join(os.homedir(), ".cache", "skill-router");
+  const venvDir = path.join(cacheRoot, "venv");
+  const venvPython = path.join(venvDir, "bin", "python");
+
+  try {
+    fs.mkdirSync(cacheRoot, { recursive: true });
+  } catch (err) {
+    console.warn("[skill-router] warning: failed to create cache dir:", err.message);
+    process.exit(0);
+  }
+
+  if (!fs.existsSync(venvPython)) {
+    const created = run("python3", ["-m", "venv", venvDir]);
+    if (!created) {
+      console.warn("[skill-router] warning: failed to create Python venv. Install will continue without blocking.");
+      process.exit(0);
+    }
+  }
+
+  // Best-effort dependency setup. We avoid failing npm install if pip/network is unavailable.
+  run(venvPython, ["-m", "pip", "install", "--upgrade", "pip", "setuptools", "wheel"]);
+  const depsOk = run(venvPython, ["-m", "pip", "install", "--upgrade", "fastembed", "numpy"]);
+
+  if (!depsOk) {
+    console.warn("[skill-router] warning: could not install Python deps in venv. The CLI will report missing_dependency at runtime.");
+  }
+}
+
+main();