mkt-skills 1.2.1 → 1.3.1

package/README.md

@@ -58,6 +58,10 @@ Override the API URL for testing against a local WordPress instance:
 MKT_API_URL=http://localhost:8080/wp-json/mkt/v1 npx mkt-skills list
 ```
 
+## Source
+
+https://github.com/sandeepkelvadi/mkt-skills
+
 ## License
 
 MIT

package/bin/mkt-skills.mjs

@@ -37,7 +37,7 @@ if (!command || command === '--help' || command === '-h') {
 }
 
 if (command === '--version' || command === '-v') {
-  console.log('mkt-skills 1.2.1');
+  console.log('mkt-skills 1.3.1');
   process.exit(0);
 }
 
@@ -47,7 +47,8 @@ if (!commandMap[command]) {
 }
 
 try {
-  await commandMap[command](args);
+  // subCommand is argv[3]; reassemble the full positional list (it was dropped before)
+  await commandMap[command]([subCommand, ...args].filter(Boolean));
 } catch (err) {
   if (err.name === 'AbortError') {
     console.error('Request timed out. Check your internet connection.');
@@ -69,7 +70,7 @@ Usage:
 Commands:
   install <slug>        Install a skill to ~/.claude/skills/
   list                  List all available skills
-  search <keyword>      Search skills by keyword
+  search "<query>"      Find skills by what you want to do (ranked)
   info <slug>           Show details about a skill
   uninstall <slug>      Remove an installed skill
   sets                  List available skill sets

package/lib/api.mjs

@@ -8,7 +8,7 @@ async function request(path) {
   try {
     const res = await fetch(`${BASE_URL}${path}`, {
       signal: controller.signal,
-      headers: { 'User-Agent': 'mkt-skills-cli/1.1.0' },
+      headers: { 'User-Agent': 'mkt-skills-cli/1.3.1' },
     });
     return res;
   } finally {
@@ -40,6 +40,13 @@ export async function fetchSet(slug) {
   return res.json();
 }
 
+export async function fetchSearch(query, limit = 5) {
+  const params = new URLSearchParams({ q: query, limit: String(limit) });
+  const res = await request(`/search?${params.toString()}`);
+  if (!res.ok) throw new Error(`Failed to fetch search results (HTTP ${res.status})`);
+  return res.json();
+}
+
 export async function fetchInstall(slug, code) {
   const path = `/install/${encodeURIComponent(slug)}` + (code ? `?code=${encodeURIComponent(code)}` : '');
   const res = await request(path);

package/lib/commands.mjs

@@ -1,6 +1,6 @@
 import { mkdir, rm, access, writeFile } from 'fs/promises';
 import { createInterface } from 'readline';
-import { fetchCatalog, fetchInstall, fetchSets, fetchSet } from './api.mjs';
+import { fetchCatalog, fetchInstall, fetchSets, fetchSet, fetchSearch } from './api.mjs';
 import { skillDir, skillFile, installedSlugs } from './paths.mjs';
 import { bold, dim, green, red, yellow, cyan } from './ui.mjs';
 
@@ -112,39 +112,43 @@ export async function list(args) {
 
 // ── search ──────────────────────────────────────────────────
 
-export async function search(args) {
-  const query = args.filter(a => !a.startsWith('-')).join(' ').toLowerCase();
+export function renderSearchResults(payload) {
+  const { query, results = [] } = payload;
+  const out = [];
 
-  if (!query) {
-    console.error('Usage: mkt-skills search <keyword>\n');
-    process.exit(1);
-  }
-
-  const catalog = await fetchCatalog();
-  const results = catalog.filter(s =>
-    s.slug.includes(query) ||
-    s.title.toLowerCase().includes(query) ||
-    s.description.toLowerCase().includes(query) ||
-    s.categories.some(c => c.toLowerCase().includes(query))
-  );
-
-  if (results.length === 0) {
-    console.log(`\nNo skills found matching "${query}".`);
-    console.log(`${dim('->')} Browse all: ${cyan('npx mkt-skills list')}\n`);
-    return;
+  if (!results.length) {
+    out.push(`\nNo close matches for ${bold(`"${query}"`)}.`);
+    out.push(`${dim('->')} Browse all: ${cyan('npx mkt-skills list')}\n`);
+    return out.join('\n');
   }
 
-  console.log(`\n${bold(`Search: "${query}"`)} (${results.length} results)\n`);
+  out.push(`\n${bold(`Best matches for "${query}"`)}\n`);
+  results.forEach((r, i) => {
+    const price = r.price === 'paid' ? yellow(' [premium]') : '';
+    const cats  = r.categories?.length ? dim(` (${r.categories.join(', ')})`) : '';
+    out.push(`  ${bold(`${i + 1}. ${r.slug}`)}${price}${cats}`);
+    out.push(`     ${r.description}`);
+    if (r.matched && r.matched.length) {
+      out.push(`     ${dim(`↳ matched: ${r.matched.join(', ')}`)}`);
+    }
+    out.push(`     ${dim('Install:')} ${cyan(`npx mkt-skills install ${r.slug}`)}`);
+    out.push('');
+  });
+  return out.join('\n');
+}
 
-  for (const s of results) {
-    const price = s.price === 'paid' ? yellow(' [premium]') : '';
-    const cats  = s.categories.length ? dim(` (${s.categories.join(', ')})`) : '';
-    console.log(`  ${bold(s.slug)}${price}${cats}`);
-    console.log(`  ${s.description}`);
-    console.log('');
+export async function search(args) {
+  const query = args.filter(a => !a.startsWith('-')).join(' ');
+  if (!query) {
+    console.error('Usage: mkt-skills search "<what you want to do>"\n');
+    process.exit(1);
   }
+  const limitFlag = args.find(a => a.startsWith('--limit='));
+  const parsedLimit = limitFlag ? parseInt(limitFlag.split('=')[1], 10) : NaN;
+  const limit = Number.isInteger(parsedLimit) && parsedLimit > 0 ? parsedLimit : 5;
 
-  console.log(`${dim('Install:')} npx mkt-skills install <slug>\n`);
+  const payload = await fetchSearch(query, limit);
+  console.log(renderSearchResults(payload));
 }
 
 // ── info ────────────────────────────────────────────────────

package/lib/paths.mjs

@@ -4,7 +4,13 @@ import { readdir } from 'fs/promises';
 
 const SKILLS_ROOT = join(homedir(), '.claude', 'skills');
 
+const SLUG_RE = /^[a-z0-9][a-z0-9-]*$/;
+
+// Slugs come from the user and from the server. Never let one escape SKILLS_ROOT.
 export function skillDir(slug) {
+  if (typeof slug !== 'string' || !SLUG_RE.test(slug)) {
+    throw new Error(`Invalid skill slug: ${JSON.stringify(slug)}`);
+  }
   return join(SKILLS_ROOT, slug);
 }
 

package/lib/video.mjs

@@ -1,13 +1,14 @@
 import { createInterface } from "readline";
-import { mkdir, rm, writeFile, readFile } from "fs/promises";
+import { mkdir, mkdtemp, rm, writeFile, readFile } from "fs/promises";
 import { existsSync } from "fs";
 import { join, dirname } from "path";
-import { execSync } from "child_process";
+import { execSync, execFileSync } from "child_process";
+import { tmpdir } from "os";
 import { bold, dim, green, red, cyan } from "./ui.mjs";
 
 const TEMPLATE_REPO = "https://github.com/sandeepkelvadi/tm-videogen-template.git";
 const TEMPLATE_BRANCH = "main";
-const TEMPLATE_DIR = "/tmp/.tm-videogen-template";
+let TEMPLATE_DIR; // private temp dir, created per run in cloneTemplate()
 
 async function question(query) {
   const rl = createInterface({ input: process.stdin, output: process.stdout });
@@ -105,10 +106,8 @@ function exec(cmd, cwd) {
 }
 
 async function cloneTemplate() {
-  if (existsSync(TEMPLATE_DIR)) {
-    await rm(TEMPLATE_DIR, { recursive: true, force: true });
-  }
-  execSync(`git clone --depth=1 --branch ${TEMPLATE_BRANCH} ${TEMPLATE_REPO} "${TEMPLATE_DIR}"`, { stdio: "pipe" });
+  TEMPLATE_DIR = await mkdtemp(join(tmpdir(), "tm-videogen-"));
+  execFileSync("git", ["clone", "--depth=1", "--branch", TEMPLATE_BRANCH, TEMPLATE_REPO, TEMPLATE_DIR], { stdio: "pipe" });
 }
 
 export async function video(args) {

package/package.json

@@ -1,11 +1,14 @@
 {
   "name": "mkt-skills",
-  "version": "1.2.1",
+  "version": "1.3.1",
   "description": "Install Claude Code marketing skills from marketingskills.directory",
   "type": "module",
   "bin": {
     "mkt-skills": "bin/mkt-skills.mjs"
   },
+  "scripts": {
+    "test": "node --test"
+  },
   "engines": {
     "node": ">=18.0.0"
   },
@@ -30,7 +33,11 @@
   "license": "MIT",
   "repository": {
     "type": "git",
-    "url": "git+https://github.com/sandeepkelvadi/mkt-skills-dir.git"
+    "url": "git+https://github.com/sandeepkelvadi/mkt-skills.git",
+    "directory": "cli"
   },
-  "homepage": "https://marketingskills.directory"
+  "homepage": "https://marketingskills.directory",
+  "bugs": {
+    "url": "https://github.com/sandeepkelvadi/mkt-skills/issues"
+  }
 }