@booklib/skills 1.6.0 → 1.7.0

package/benchmark/review-report.md

@@ -1,129 +0,0 @@
-# skill-router benchmark: book-based review vs. native PR toolkit
-
-**skill-router** analyzes your code and picks the right book-based skill for the job — so instead of one AI trying to apply four books superficially, you get a deep, focused review from the correct authority. This benchmark runs the same intentionally broken JavaScript file through two parallel pipelines to compare what each finds.
-
-| | Native | skill-router |
-|---|---|---|
-| **Pipeline** | `pr-review-toolkit:code-reviewer` | `skill-router` → `clean-code-reviewer` + `design-patterns` |
-| **Output file** | `order-processing.pr-toolkit.js` | `order-processing.skill-router.js` |
-
-**Bottom line:** Native wins on security depth (catches PCI violation, highest confidence scores). skill-router wins on architectural insight and principle education — finds ~47% more unique issues when both pipelines are combined.
-
----
-
-## The code under test
-
-`order-processing.original.js` — 157-line Node.js order processing module with: one god function (`process`), no error handling, SQL injection on every query, global mutable state shared across all requests, and `eval()` used to return a plain variable.
-
----
-
-## How skill-router picks its skills
-
-The router commits to a routing decision before any review begins:
-
-```
-Primary:    clean-code-reviewer  — god function, cryptic names, magic numbers (Ch.2/3/7)
-Secondary:  design-patterns      — duplicated payment blocks = Strategy pattern
-Don't use:  domain-driven-design — implementation-level code, not a model design problem
-```
-
-This is the core value: rather than spreading one review thin across four books, the router picks the right lens and goes deep. The `Don't use` line matters — it avoids premature or irrelevant advice.
-
----
-
-## Issues found
-
-> **Notation:** `C1`, `I9`, `S4` are clean-code-reviewer's severity tiers (Critical / Important / Suggestion). `G30`, `N1`, `Ch.7` etc. are chapter/guideline references from *Clean Code* by Robert C. Martin. `Strategy(1)`, `Singleton(4)` etc. are pattern references from *Head First Design Patterns*.
-
-### Critical / High severity
-
-| Issue | Native | clean-code | design-patterns |
-|---|---|---|---|
-| SQL injection — every query (7 locations) | ✅ Confidence 100 | ✅ C1 — G30 | — |
-| `eval("stats")` — unnecessary, disables JIT | ✅ Confidence 100 | ✅ C3 — G30 | ✅ Singleton(4) |
-| Global mutable `usr`/`items`/`total` — cross-request leak | ✅ Confidence 98 | ✅ C2 — G18 | ✅ Singleton(4) |
-| `items` unbounded memory leak | ✅ Confidence 95 | ⚠️ implied | — |
-| `refund()` null dereference crash | ✅ Confidence 95 | ✅ C4 — Ch.7 | — |
-| PCI violation — card data logged to stdout | ✅ Confidence 92 | ❌ missed | — |
-| Payment fall-through returns `undefined` not `false` | ✅ Confidence 91 | ✅ I9 | ✅ Strategy(1) |
-| Duplicated card/paypal confirmation block | ✅ Confidence 85 | ✅ C5 — G5 | ✅ Strategy(1) |
-| Wrong-recipient email via global `usr` in cancel/refund | ✅ Confidence 89 | ✅ C4/I7 | — |
-
-### Important / Improvement
-
-| Issue | Native | clean-code | design-patterns |
-|---|---|---|---|
-| `==` instead of `===` throughout | ✅ Confidence 82 | ✅ I5 — G15 | — |
-| `var` instead of `const`/`let` | ✅ Confidence 80 | ✅ I6 — G29 | — |
-| Magic discount numbers (0.1, 0.2, 0.5) | ✅ | ✅ I3 — G25 | ✅ Strategy(3) |
-| `discount` variable declared, never used | ✅ Confidence 88 | ✅ I3 | — |
-| Discount should be data-driven, not if-chain | — | ✅ I4 — G23 | ✅ Strategy(3) |
-| God function — 10 responsibilities | ⚠️ via nesting | ✅ I1 — F1/F2 | ✅ Template(5) |
-| Six-level arrow-head nesting | ✅ Confidence 82 | ✅ I2 | — |
-| No error handling on db/mailer calls | ✅ Confidence 85 | ✅ I8 — Ch.7 | — |
-| `cancel()` never updates stats | ✅ Confidence 80 | — | — |
-| Stats inconsistency — two separate objects | ✅ Confidence 82 | ✅ S4 | ✅ Singleton(4) |
-| Cryptic names (`o`, `u`, `pay`, `s`, `rsn`) | ✅ Confidence 80 | ✅ S1 — N1/N2 | — |
-| Noise/lying comments | — | ✅ S2 — C2/C3 | — |
-| `formatMoney` floating-point rounding bug | — | ✅ S3 | — |
-| Stubs always return `true` — lying | — | ✅ S5 — C3 | — |
-| State machine needed for order lifecycle | — | — | ✅ State(2) |
-| Side effects hardcoded — use Observer | — | — | ✅ Observer(6) |
-| Flat module export — use Facade | — | — | ✅ Facade(7) |
-
-### Totals
-
-| | Native | clean-code | design-patterns | Combined |
-|---|---|---|---|---|
-| Critical/High | 9 | 5 | 4 | **9 unique** |
-| Important/Improvement | 10 | 9 | 3 | **14 unique** |
-| Suggestion/Low | 0 | 5 | 0 | **5** |
-| **Total** | **19** | **19** | **7 patterns** | **~28 unique** |
-
-Overlap: ~89% of native issues were also found by skill-router.
-
----
-
-## Pattern opportunities identified (skill-router only)
-
-The native pipeline finds bugs. The skill-router pipeline additionally surfaces structural opportunities — places where a known pattern *could* reduce complexity — and explains the problem each one solves. Whether to apply a pattern is a judgment call: the skill flags the opportunity and the reasoning; you decide if the trade-off is worth it in your context.
-
-| # | Pattern | Impact | Problem it would solve |
-|---|---|---|---|
-| 1 | Strategy — payments | HIGH | Copy-pasted if-block per payment method |
-| 2 | State — order lifecycle | HIGH | Status strings checked in every function |
-| 3 | Strategy — discounts | HIGH | Magic-number if-chain, no extensibility |
-| 4 | Singleton (broken) | HIGH | Module-scope mutable state crossing requests |
-| 5 | Template Method | MEDIUM | Arrow pyramid duplicated in process+cancel |
-| 6 | Observer | MEDIUM | Email/stats hardcoded into business logic |
-| 7 | Facade | MEDIUM | Unrelated concerns in one flat export |
-
-**Suggested refactor sequence if you choose to act:** 4 → 1 → 3 → 2 → 5 → 6 → 7
-
----
-
-
-## Updates since this benchmark
-
-**skill-router now outputs severity tiers.** One gap this benchmark exposed was that native's confidence scores (≥80 threshold) act as a noise filter — skill-router had no equivalent. The router has since been updated to instruct selected skills to classify every finding as **HIGH** (correctness/security/data loss), **MEDIUM** (design/maintainability), or **LOW** (style/naming), and to skip LOW findings for standard code reviews. This closes the signal-to-noise gap without re-running the benchmark — detection quality hasn't changed, output actionability has.
-
----
-
-## When to use each
-
-| Situation | Use |
-|---|---|
-| Pre-merge PR review, security audit | **Native** — pre-merge gate: fast, confidence-filtered, adapts to CLAUDE.md project conventions |
-| Larger refactor, architecture planning | **skill-router** — patterns, principles, refactor roadmap |
-| Both together | ~95% total issue coverage vs ~80% for either alone |
-
----
-
-## Files in this benchmark
-
-| File | Description |
-|---|---|
-| `order-processing.original.js` | Original bad code — unchanged input |
-| `order-processing.pr-toolkit.js` | Rewritten applying all `pr-review-toolkit` findings |
-| `order-processing.skill-router.js` | Rewritten applying `clean-code-reviewer` + `design-patterns` |
-| `review-report.md` | This file |

package/demo.tape

@@ -1,40 +0,0 @@
-Output demo.gif
-
-Set Shell "bash"
-Set FontSize 15
-Set Width 1100
-Set Height 650
-Set Theme "Dracula"
-Set Padding 24
-Set Framerate 30
-Set PlaybackSpeed 1
-Env PATH "/opt/homebrew/bin:/usr/local/bin:/usr/bin:/bin"
-
-Hide
-Type "cd /Users/fvst/other/fp/skills && clear"
-Enter
-Sleep 500ms
-Show
-
-Sleep 500ms
-
-Type "node bin/skills.js list"
-Sleep 300ms
-Enter
-Sleep 5s
-
-Sleep 1500ms
-
-Type "node bin/skills.js info clean-code-reviewer"
-Sleep 300ms
-Enter
-Sleep 3s
-
-Sleep 1500ms
-
-Type "node bin/skills.js demo clean-code-reviewer"
-Sleep 300ms
-Enter
-Sleep 5s
-
-Sleep 3s

package/docs/index.html

@@ -1,411 +0,0 @@
-<!DOCTYPE html>
-<html lang="en">
-<head>
-  <meta charset="UTF-8"/>
-  <meta name="viewport" content="width=device-width, initial-scale=1.0"/>
-  <title>booklib-ai/skills — Book-grounded AI agent skills</title>
-  <meta name="description" content="22 AI agent skills grounded in canonical programming books. Give your AI coding assistant expert knowledge from Clean Code, Effective Java, DDD, and more."/>
-  <meta property="og:title" content="booklib-ai/skills"/>
-  <meta property="og:description" content="Book-grounded AI agent skills for Claude Code, Cursor, Copilot, and Windsurf."/>
-  <meta property="og:image" content="https://booklib-ai.github.io/skills/og.png"/>
-  <meta property="og:image:width" content="1200"/>
-  <meta property="og:image:height" content="630"/>
-  <meta name="twitter:card" content="summary_large_image"/>
-  <meta name="twitter:image" content="https://booklib-ai.github.io/skills/og.png"/>
-  <style>
-    *, *::before, *::after { box-sizing: border-box; margin: 0; padding: 0; }
-
-    body {
-      background: #0d0d1a;
-      color: #e2e8f0;
-      font-family: -apple-system, BlinkMacSystemFont, "Segoe UI", Roboto, sans-serif;
-      line-height: 1.6;
-    }
-
-    a { color: #818cf8; text-decoration: none; }
-    a:hover { text-decoration: underline; }
-
-    /* Hero */
-    .hero {
-      text-align: center;
-      padding: 80px 24px 64px;
-      max-width: 720px;
-      margin: 0 auto;
-    }
-    .hero img.logo { width: 96px; height: 96px; margin-bottom: 24px; }
-    .hero h1 { font-size: 2.5rem; font-weight: 700; color: #f1f5f9; letter-spacing: -0.03em; }
-    .hero p {
-      font-size: 1.125rem;
-      color: #94a3b8;
-      margin-top: 16px;
-      max-width: 520px;
-      margin-left: auto;
-      margin-right: auto;
-    }
-
-    /* Install */
-    #install {
-      display: flex;
-      align-items: center;
-      justify-content: center;
-      gap: 12px;
-      margin-top: 36px;
-      flex-wrap: wrap;
-    }
-    .install-box {
-      background: #161625;
-      border: 1px solid #2d2d4a;
-      border-radius: 10px;
-      padding: 12px 20px;
-      font-family: "SF Mono", "Fira Code", monospace;
-      font-size: 0.9rem;
-      color: #a5b4fc;
-      cursor: pointer;
-      user-select: all;
-      transition: border-color 0.2s;
-    }
-    .install-box.highlighted {
-      border-color: #6366f1;
-      color: #c7d2fe;
-    }
-    .copy-btn {
-      background: #6366f1;
-      color: white;
-      border: none;
-      border-radius: 8px;
-      padding: 12px 20px;
-      font-size: 0.875rem;
-      font-weight: 600;
-      cursor: pointer;
-      transition: background 0.15s;
-    }
-    .copy-btn:hover { background: #4f46e5; }
-
-    /* Badges */
-    .badges {
-      display: flex;
-      justify-content: center;
-      gap: 8px;
-      margin-top: 28px;
-      flex-wrap: wrap;
-    }
-    .badges img { height: 20px; }
-
-    /* Skills grid */
-    .skills-section {
-      max-width: 1100px;
-      margin: 64px auto;
-      padding: 0 24px;
-    }
-    .skills-section h2 {
-      font-size: 1.5rem;
-      font-weight: 700;
-      color: #f1f5f9;
-      margin-bottom: 8px;
-      text-align: center;
-    }
-    .skills-section .subtitle {
-      text-align: center;
-      color: #64748b;
-      font-size: 0.9rem;
-      margin-bottom: 32px;
-    }
-    .skills-grid {
-      display: grid;
-      grid-template-columns: repeat(auto-fill, minmax(130px, 1fr));
-      gap: 14px;
-    }
-
-    /* Card */
-    .skill-card {
-      background: #11111f;
-      border: 1px solid #1e1e35;
-      border-radius: 12px;
-      overflow: hidden;
-      cursor: pointer;
-      transition: border-color 0.15s, transform 0.15s, box-shadow 0.15s;
-      display: flex;
-      flex-direction: column;
-      text-decoration: none;
-      color: inherit;
-    }
-    .skill-card:hover {
-      border-color: #6366f1;
-      transform: translateY(-3px);
-      box-shadow: 0 8px 24px rgba(99, 102, 241, 0.15);
-      text-decoration: none;
-    }
-    .skill-card.active {
-      border-color: #818cf8;
-      box-shadow: 0 0 0 2px rgba(99, 102, 241, 0.3);
-    }
-
-    /* Book cover area */
-    .cover-wrap {
-      width: 100%;
-      aspect-ratio: 2 / 3;
-      background: #161625;
-      display: flex;
-      align-items: center;
-      justify-content: center;
-      overflow: hidden;
-      position: relative;
-    }
-    .cover-wrap img {
-      width: 100%;
-      height: 100%;
-      object-fit: cover;
-      display: block;
-    }
-    .cover-wrap img.loading {
-      background: linear-gradient(90deg, #161625 25%, #1e1e38 50%, #161625 75%);
-      background-size: 200% 100%;
-      animation: shimmer 1.5s infinite linear;
-    }
-    @keyframes shimmer {
-      0%   { background-position: 200% 0; }
-      100% { background-position: -200% 0; }
-    }
-    .cover-fallback {
-      width: 100%;
-      height: 100%;
-      display: flex;
-      flex-direction: column;
-      align-items: center;
-      justify-content: center;
-      padding: 16px;
-      text-align: center;
-      gap: 8px;
-    }
-    .cover-fallback .initials {
-      font-size: 2rem;
-      font-weight: 700;
-      color: #6366f1;
-      line-height: 1;
-    }
-    .cover-fallback .fallback-title {
-      font-size: 0.75rem;
-      color: #64748b;
-      line-height: 1.3;
-    }
-
-    /* Card body */
-    .card-body {
-      padding: 10px 12px 12px;
-      flex: 1;
-      display: flex;
-      flex-direction: column;
-      gap: 3px;
-    }
-    .card-body .skill-name {
-      font-size: 0.7rem;
-      font-weight: 700;
-      color: #a5b4fc;
-      font-family: "SF Mono", "Fira Code", monospace;
-      word-break: break-all;
-    }
-    .card-body .book-title {
-      font-size: 0.75rem;
-      font-weight: 600;
-      color: #e2e8f0;
-      line-height: 1.3;
-      margin-top: 2px;
-    }
-    .card-body .book-author {
-      font-size: 0.68rem;
-      color: #64748b;
-    }
-    .card-body .card-desc {
-      font-size: 0.68rem;
-      color: #475569;
-      line-height: 1.4;
-      margin-top: 4px;
-    }
-
-    /* Footer */
-    footer {
-      text-align: center;
-      padding: 48px 24px;
-      color: #475569;
-      font-size: 0.875rem;
-      border-top: 1px solid #1e1e35;
-    }
-    footer a { color: #6366f1; }
-  </style>
-</head>
-<body>
-
-<section class="hero">
-  <img class="logo" src="https://raw.githubusercontent.com/booklib-ai/skills/main/assets/logo.svg" alt="booklib-ai skills logo"/>
-  <h1>Skills</h1>
-  <p>Book-grounded AI agent skills — expert knowledge from canonical programming books, packaged for Claude Code, Cursor, Copilot, and Windsurf.</p>
-
-  <div id="install">
-    <div class="install-box" id="cmd">npx skills add booklib-ai/skills --all -g</div>
-    <button class="copy-btn" onclick="copyCmd()">Copy</button>
-  </div>
-
-  <div class="badges">
-    <img src="https://img.shields.io/npm/v/@booklib/skills.svg" alt="npm version"/>
-    <img src="https://img.shields.io/npm/dw/@booklib/skills.svg" alt="downloads"/>
-    <img src="https://img.shields.io/github/stars/booklib-ai/skills?style=flat" alt="stars"/>
-    <img src="https://img.shields.io/badge/License-MIT-yellow.svg" alt="license"/>
-  </div>
-</section>
-
-<section class="skills-section">
-  <h2>22 skills, 22 books</h2>
-  <p class="subtitle">Click any skill to see its install command</p>
-  <div class="skills-grid" id="grid"></div>
-</section>
-
-<footer>
-  <p>MIT License &middot; <a href="https://github.com/booklib-ai/skills">GitHub</a> &middot; <a href="https://github.com/booklib-ai/skills/blob/main/CONTRIBUTING.md">Contributing</a> &middot; <a href="https://github.com/booklib-ai/skills/blob/main/AGENTS.md">Agent Setup</a></p>
-</footer>
-
-<script>
-  const SKILLS = [
-    { name: "animation-at-work",      book: "Animation at Work",                   author: "Rachel Nabors",          isbn: "9781937557942", desc: "Motion perception, 12 animation principles, CSS transitions, and performance." },
-    { name: "clean-code-reviewer",    book: "Clean Code",                           author: "Robert C. Martin",       isbn: "9780132350884", desc: "Naming, functions, comments, classes, and the Boy Scout Rule." },
-    { name: "data-intensive-patterns",book: "Designing Data-Intensive Applications",author: "Martin Kleppmann",       isbn: "9781449373320", desc: "Storage engines, replication, transactions, and distributed systems." },
-    { name: "data-pipelines",         book: "Data Pipelines Pocket Reference",      author: "James Densmore",         isbn: "9781492087601", desc: "Ingestion, streaming, transformation, and orchestration." },
-    { name: "design-patterns",        book: "Head First Design Patterns",           author: "Freeman & Robson",       isbn: "9781492078005", desc: "GoF creational, structural, and behavioral patterns in depth." },
-    { name: "domain-driven-design",   book: "Domain-Driven Design",                author: "Eric Evans",             isbn: "9780321125217", desc: "Aggregates, Value Objects, Bounded Contexts, Ubiquitous Language." },
-    { name: "effective-java",         book: "Effective Java",                       author: "Joshua Bloch",           isbn: "9780134685991", desc: "Object creation, generics, enums, lambdas, and concurrency." },
-    { name: "effective-kotlin",       book: "Effective Kotlin",                     author: "Marcin Moskala",         isbn: "9781803248776", desc: "Safety, readability, reusability, and abstraction." },
-    { name: "effective-python",       book: "Effective Python",                     author: "Brett Slatkin",          isbn: "9780134853987", desc: "Pythonic thinking, functions, classes, concurrency, and testing." },
-    { name: "effective-typescript",   book: "Effective TypeScript",                 author: "Dan Vanderkam",          isbn: "9781492053743", desc: "Type system, type design, avoiding any, declarations, and migration." },
-    { name: "kotlin-in-action",       book: "Kotlin in Action",                     author: "Elizarov & Isakova",     isbn: "9781617293290", desc: "Functions, classes, lambdas, nullability, coroutines, and flows." },
-    { name: "lean-startup",           book: "The Lean Startup",                     author: "Eric Ries",              isbn: "9780307887894", desc: "MVP testing, validated learning, Build-Measure-Learn loop." },
-    { name: "microservices-patterns", book: "Microservices Patterns",               author: "Chris Richardson",       isbn: "9781617294549", desc: "Decomposition, sagas, API gateways, event sourcing, and CQRS." },
-    { name: "programming-with-rust",  book: "Programming with Rust",                author: "Donis Marshall",         isbn: "9780137889655", desc: "Ownership, borrowing, lifetimes, error handling, traits, concurrency." },
-    { name: "refactoring-ui",         book: "Refactoring UI",                       author: "Wathan & Schoger",       isbn: null,            desc: "Visual hierarchy, layout, typography, color, and spacing." },
-    { name: "rust-in-action",         book: "Rust in Action",                       author: "Tim McNamara",           isbn: "9781617294556", desc: "Systems programming — smart pointers, memory, networking, OS." },
-    { name: "skill-router",           book: "Meta-skill",                           author: "booklib-ai",             isbn: null,            desc: "Automatically routes to the most relevant skill for any task." },
-    { name: "spring-boot-in-action",  book: "Spring Boot in Action",                author: "Craig Walls",            isbn: "9781617292545", desc: "Auto-configuration, starters, testing, Actuator, and deployment." },
-    { name: "storytelling-with-data", book: "Storytelling with Data",               author: "Cole Nussbaumer Knaflic",isbn: "9781119002253", desc: "Effective visuals, decluttering, and narrative structure." },
-    { name: "system-design-interview",book: "System Design Interview",              author: "Alex Xu",                isbn: "9798664653403", desc: "Scaling, estimation, load balancing, caching, and sharding." },
-    { name: "using-asyncio-python",   book: "Using Asyncio in Python",              author: "Caleb Hattingh",         isbn: "9781492075325", desc: "Coroutines, event loop, tasks, and signal handling." },
-    { name: "web-scraping-python",    book: "Web Scraping with Python",             author: "Ryan Mitchell",          isbn: "9781491985564", desc: "BeautifulSoup, Scrapy, Selenium, and data storage." },
-  ];
-
-  const ALL_CMD = "npx skills add booklib-ai/skills --all -g";
-  let activeCard = null;
-
-  function initials(name) {
-    return name.split(/[\s-]+/).map(w => w[0]).join("").toUpperCase().slice(0, 3);
-  }
-
-  // Fetch cover URL from Google Books JSON API.
-  // Using the API (not direct hotlinking) avoids false-positive placeholder images
-  // ("No preview available" hatch / "image not available" text) that Google returns
-  // as HTTP 200 responses — the API simply omits imageLinks when no cover exists.
-  function showFb(img) {
-    img.style.display = "none";
-    const fb = img.nextElementSibling;
-    if (fb) fb.style.display = "flex";
-  }
-
-  async function loadCover(img) {
-    const isbn = img.dataset.isbn;
-    try {
-      const url = `https://www.googleapis.com/books/v1/volumes?q=isbn:${isbn}&fields=items(volumeInfo(imageLinks))&maxResults=1`;
-      const d = await fetch(url).then(r => r.json());
-      const links = d.items?.[0]?.volumeInfo?.imageLinks;
-      // thumbnail/smallThumbnail are the only sizes returned by a search query.
-      // Replacing zoom=1 with zoom=0 fetches the largest available version of the same image.
-      const src = (links?.thumbnail || links?.smallThumbnail || "")
-        .replace(/^http:/, "https:")
-        .replace("zoom=1", "zoom=5")
-        .replace("&edge=curl", "");
-      if (src) {
-        img.onload = () => img.classList.remove("loading");
-        img.onerror = () => showFb(img);
-        img.src = src;
-      } else {
-        showFb(img);
-      }
-    } catch {
-      showFb(img);
-    }
-  }
-
-  function buildCard(skill) {
-    const card = document.createElement("a");
-    card.className = "skill-card";
-    card.href = `https://github.com/booklib-ai/skills/tree/main/skills/${skill.name}`;
-    card.target = "_blank";
-    card.rel = "noopener";
-
-    const coverHtml = skill.isbn
-      ? `<img
-           class="loading"
-           data-isbn="${skill.isbn}"
-           alt="${skill.book} cover"
-         />
-         <div class="cover-fallback" style="display:none">
-           <div class="initials">${initials(skill.book)}</div>
-           <div class="fallback-title">${skill.book}</div>
-         </div>`
-      : `<div class="cover-fallback">
-           <div class="initials">${initials(skill.book)}</div>
-           <div class="fallback-title">${skill.book}</div>
-         </div>`;
-
-    card.innerHTML = `
-      <div class="cover-wrap">${coverHtml}</div>
-      <div class="card-body">
-        <div class="skill-name">${skill.name}</div>
-        <div class="book-title">${skill.book}</div>
-        <div class="book-author">${skill.author}</div>
-        <div class="card-desc">${skill.desc}</div>
-      </div>
-    `;
-
-    card.addEventListener("click", (e) => {
-      e.preventDefault();
-      selectSkill(skill, card);
-    });
-
-    return card;
-  }
-
-  function selectSkill(skill, card) {
-    // Deactivate previous
-    if (activeCard && activeCard !== card) {
-      activeCard.classList.remove("active");
-    }
-    const cmdEl = document.getElementById("cmd");
-
-    if (activeCard === card) {
-      // Deselect — back to install all
-      activeCard.classList.remove("active");
-      activeCard = null;
-      cmdEl.textContent = ALL_CMD;
-      cmdEl.classList.remove("highlighted");
-    } else {
-      activeCard = card;
-      card.classList.add("active");
-      cmdEl.textContent = `npx skills add booklib-ai/skills ${skill.name} -g`;
-      cmdEl.classList.add("highlighted");
-      document.getElementById("install").scrollIntoView({ behavior: "smooth", block: "center" });
-    }
-  }
-
-  function copyCmd() {
-    const text = document.getElementById("cmd").textContent;
-    navigator.clipboard.writeText(text);
-    const btn = document.querySelector(".copy-btn");
-    btn.textContent = "Copied!";
-    setTimeout(() => btn.textContent = "Copy", 2000);
-  }
-
-  // Build grid
-  const grid = document.getElementById("grid");
-  SKILLS.forEach(skill => grid.appendChild(buildCard(skill)));
-  // Kick off async cover loading after DOM is populated
-  document.querySelectorAll("img[data-isbn]").forEach(loadCover);
-</script>
-
-</body>
-</html>