@minhpnq1807/contextos 0.5.45 → 0.5.49

package/plugins/ctx/lib/skill-discoverer.js

@@ -2,28 +2,20 @@ import fs from "node:fs";
 import os from "node:os";
 import path from "node:path";
 
-import { enhanceRuleScoresWithEmbeddings, warmRuleEmbeddings } from "./embedding-scorer.js";
+import {
+  enhanceRuleScoresWithEmbeddings,
+  searchIndexedEmbeddings,
+  warmIndexedEmbeddings
+} from "./embedding-scorer.js";
+import { fusedProjectQuery, workspacePackagePaths } from "./project-profiler.js";
 
 const DEFAULT_LIMIT = 3;
 const DEFAULT_MAX_SKILLS = 2000;
 const DEFAULT_EMBEDDING_CANDIDATES = 120;
-const DEFAULT_SEMANTIC_CATALOG_LIMIT = 300;
 const SCAN_CACHE_TTL_MS = 5 * 60 * 1000;
 const MAX_DESCRIPTION_CHARS = 500;
-const GENERIC_SKILL_TOKENS = new Set([
-  "active", "agent", "agents", "code", "config", "configuration", "create", "development",
-  "environment", "file", "files", "graph", "install", "integration", "local", "node", "package",
-  "project", "refresh", "rebuild", "setup", "skill", "skills", "sync", "tool", "tools", "using",
-  "build", "can", "not", "production", "show", "something", "https", "http", "com", "www",
-  "a", "an", "and", "are", "as", "at", "be", "before", "after", "both", "by", "from", "for",
-  "if", "in", "into", "is", "must", "of", "on", "or", "the", "then", "this", "to", "user",
-  "users", "when", "where", "whether", "with"
-]);
-const SPECIALIZED_SKILL_TOKENS = new Set([
-  "android", "authorization", "cicd", "eas", "expo", "frontend", "ios", "next", "nextjs",
-  "mcp", "modelcontextprotocol", "postgres", "postgresql", "react", "react-native", "tailwind",
-  "typescript", "ui"
-]);
+const SKILL_EMBEDDING_THRESHOLD = 0.45;
+const DEFAULT_SKILL_TIMEOUT_MS = 2000;
 
 const scanCache = new Map();
 
@@ -123,8 +115,9 @@ function monotonicNow() {
 }
 
 function cacheAndReturnSkills(cacheKey, skills) {
-  scanCache.set(cacheKey, { createdAt: monotonicNow(), skills });
-  return skills;
+  const deduped = dedupeSkills(skills);
+  scanCache.set(cacheKey, { createdAt: monotonicNow(), skills: deduped });
+  return deduped;
 }
 
 function findSkillFiles(root) {
@@ -170,18 +163,40 @@ export async function suggestSkills({
   dataDir,
   cwd = process.cwd(),
   limit = DEFAULT_LIMIT,
-  timeoutMs = Number(process.env.CONTEXTOS_SKILL_EMBEDDING_TIMEOUT_MS || process.env.CONTEXTOS_EMBEDDING_TIMEOUT_MS || 800)
+  timeoutMs = Number(process.env.CONTEXTOS_SKILL_EMBEDDING_TIMEOUT_MS || process.env.CONTEXTOS_EMBEDDING_TIMEOUT_MS || DEFAULT_SKILL_TIMEOUT_MS),
+  indexedSearcher = searchIndexedEmbeddings,
+  embeddingEnhancer = enhanceRuleScoresWithEmbeddings
 } = {}) {
   if (!String(prompt || "").trim() || !skills.length) return [];
-  const base = scoreSkillsByKeyword({ prompt, skills, projectHints: projectSkillHints({ cwd }) });
-  if (skills.length > DEFAULT_SEMANTIC_CATALOG_LIMIT) {
-    return finalizeSkillScores(base, limit, { minimumKeywordScore: 0.5 });
+  const catalog = dedupeSkills(skills);
+  const query = fusedProjectQuery({ prompt, cwd, dataDir });
+  const byId = new Map(catalog.map((skill) => [skillIndexId(skill), skill]));
+
+  if (dataDir) {
+    const indexed = await indexedSearcher({
+      kind: skillIndexKind(cwd),
+      task: query,
+      dataDir,
+      timeoutMs,
+      allowRemote: false
+    });
+    if (indexed.status === "enabled" && indexed.items.length) {
+      return finalizeSkillScores(indexed.items
+        .map((item) => {
+          const skill = byId.get(item.id);
+          if (!skill) return null;
+          return skillScoreFromEmbedding(skill, item.embeddingScore, [`embedding:${Number(item.embeddingScore || 0).toFixed(2)}`]);
+        })
+        .filter(Boolean), limit);
+    }
   }
 
-  const embeddingCandidates = selectEmbeddingCandidates(base);
+  if (catalog.length > DEFAULT_EMBEDDING_CANDIDATES) return [];
+
+  const embeddingCandidates = catalog.map((skill, index) => skillRule({ skill, index }));
   if (!embeddingCandidates.length) return [];
 
-  const embedding = await enhanceRuleScoresWithEmbeddings(embeddingCandidates, prompt, {
+  const embedding = await embeddingEnhancer(embeddingCandidates, query, {
     dataDir,
     sources: embeddingCandidates.map((skill) => skill.path).filter(Boolean),
     timeoutMs,
@@ -191,26 +206,20 @@ export async function suggestSkills({
   return finalizeSkillScores(embedding.rules, limit);
 }
 
-function finalizeSkillScores(skills, limit, { minimumKeywordScore = 0.35 } = {}) {
+function finalizeSkillScores(skills, limit) {
   const ranked = skills
-    .filter((rule) => rule.domainEligible !== false)
     .map((rule) => ({
       name: rule.name,
       description: rule.description,
       path: rule.path,
       scope: rule.scope,
-      keywordScore: rule.keywordScore,
       score: Math.min(1, Number(rule.score || 0)),
       embeddingScore: rule.embeddingScore,
-      relevancePriority: Number(rule.relevancePriority || 0),
-      rankScore: Math.min(1, Number(rule.score || 0)) + Number(rule.relevancePriority || 0) / 100,
+      rankScore: Math.min(1, Number(rule.score || 0)),
       reasons: rule.reasons || []
     }))
-    .filter((skill) => Number(skill.keywordScore || 0) >= minimumKeywordScore
-      || Number(skill.embeddingScore || 0) >= 0.62
-      || Number(skill.relevancePriority || 0) >= 50)
+    .filter((skill) => Number(skill.embeddingScore || skill.score || 0) >= SKILL_EMBEDDING_THRESHOLD)
     .sort((a, b) => b.rankScore - a.rankScore
-      || b.relevancePriority - a.relevancePriority
       || b.score - a.score
       || scopePriority(b.scope) - scopePriority(a.scope)
       || a.name.localeCompare(b.name));
@@ -229,14 +238,6 @@ function scopePriority(scope) {
   return scope === "project" ? 1 : 0;
 }
 
-function selectEmbeddingCandidates(skills) {
-  if (skills.length <= DEFAULT_EMBEDDING_CANDIDATES) return skills;
-  return [...skills]
-    .filter((skill) => Number(skill.keywordScore || 0) > 0)
-    .sort((a, b) => Number(b.keywordScore || 0) - Number(a.keywordScore || 0) || a.name.localeCompare(b.name))
-    .slice(0, DEFAULT_EMBEDDING_CANDIDATES);
-}
-
 export async function warmSkillEmbeddings({
   cwd = process.cwd(),
   dataDir,
@@ -244,249 +245,81 @@ export async function warmSkillEmbeddings({
   skills = scanSkills({ cwd })
 } = {}) {
   if (!dataDir || !skills.length) return { count: 0, cachePath: null };
-  return warmRuleEmbeddings({
-    rules: skills.map((skill) => ({ content: `${skill.name} ${skill.description}` })),
-    task: "skill discovery semantic retrieval",
+  const catalog = dedupeSkills(skills);
+  return warmIndexedEmbeddings({
+    kind: skillIndexKind(cwd),
+    items: catalog.map((skill) => ({
+      id: skillIndexId(skill),
+      text: skillEmbeddingText(skill)
+    })),
+    task: fusedProjectQuery({ prompt: "skill discovery semantic retrieval", cwd, dataDir }),
     dataDir,
-    sources: skills.map((skill) => skill.path).filter(Boolean),
+    sources: catalog.map((skill) => skill.path).filter(Boolean),
     allowRemote
   });
 }
 
-function scoreSkillsByKeyword({ prompt, skills, projectHints = [] }) {
-  const normalizedPrompt = normalizePrompt(prompt);
-  const promptTokens = new Set(normalizedPrompt.split(/\s+/).filter(Boolean));
-  const projectTokens = new Set(projectHints);
-  return skills.map((skill, index) => {
-    const enriched = skill.searchTokens ? skill : enrichSkill(skill);
-    const name = String(enriched.name || "");
-    const description = truncateDescription(enriched.description || "");
-    const content = `${name} ${description}`;
-    const matches = filterSkillMatches(
-      enriched.searchTokens.filter((token) => promptTokens.has(token) && token.length > 2 && !GENERIC_SKILL_TOKENS.has(token)),
-      { normalizedPrompt, enriched }
-    );
-    const projectMatches = enriched.searchTokens.filter((token) => projectTokens.has(token) && SPECIALIZED_SKILL_TOKENS.has(token));
-    const normalizedName = enriched.normalizedName;
-    const nameTokens = enriched.nameTokens;
-    const nameHit = normalizedPrompt.includes(normalizedName);
-    const nameTokenHit = nameTokens.length > 1 && nameTokens.every((token) => promptTokens.has(token));
-    const scopeBonus = enriched.scope === "project" ? 0.08 : 0;
-    const intentBonus = skillIntentBonus(normalizedPrompt, enriched, projectTokens);
-    const relevancePriority = skillRelevancePriority(normalizedPrompt, enriched, projectTokens);
-    const domainEligible = isSkillDomainEligible(normalizedPrompt, enriched, projectTokens);
-    const matchScore = matches.reduce((sum, token) => sum + (SPECIALIZED_SKILL_TOKENS.has(token) ? 0.2 : 0.08), 0);
-    const projectBonus = intentBonus ? Math.min(0.16, projectMatches.length * 0.04) : 0;
-    const score = Math.min(1, (matches.length ? 0.25 + matchScore : 0) + projectBonus + intentBonus + (nameHit ? 0.2 : 0) + (nameTokenHit ? 0.18 : 0) + scopeBonus);
-    return {
-      id: `skill-${index + 1}`,
-      name,
-      description,
-      path: enriched.path,
-      scope: enriched.scope,
-      content,
-      score,
-      keywordScore: score,
-      relevancePriority,
-      domainEligible,
-      reasons: [
-        ...(matches.length ? [`keyword:${matches.slice(0, 4).join(",")}`] : []),
-        ...(projectBonus ? [`project:${projectMatches.slice(0, 4).join(",")}`] : []),
-        ...(intentBonus ? ["intent-match"] : []),
-        ...(nameHit || nameTokenHit ? ["name-match"] : [])
-      ],
-      originalOrder: index
-    };
-  });
-}
-
-function filterSkillMatches(matches, { normalizedPrompt, enriched }) {
-  if (!/\beas\b/.test(normalizedPrompt)) return matches;
-  const skillText = normalize(`${enriched.name} ${enriched.description}`);
-  if (/\b(eas|expo|cicd)\b/.test(skillText)) return matches;
-  return matches.filter((token) => token !== "android" && token !== "ios");
+function skillRule({ skill, index }) {
+  const enriched = skill.searchTokens ? skill : enrichSkill(skill);
+  return {
+    id: skillIndexId(enriched),
+    name: enriched.name,
+    description: enriched.description,
+    path: enriched.path,
+    scope: enriched.scope,
+    content: skillEmbeddingText(enriched),
+    score: 0,
+    originalOrder: index
+  };
 }
 
-function isSkillDomainEligible(normalizedPrompt, enriched, projectTokens = new Set()) {
-  const skillText = normalize(`${enriched.name} ${enriched.description}`);
-  if (isMcpSkill(skillText) && !isMcpRelevantTask(normalizedPrompt, projectTokens)) return false;
-  if (isOffensiveSecuritySkill(skillText) && !isSecurityTask(normalizedPrompt)) return false;
-  if (isPlatformCommerceSkill(skillText) && !isPlatformCommerceTask(normalizedPrompt, skillText)) return false;
-  if (!/\beas\b/.test(normalizedPrompt)) return true;
-  if (!/\b(android|ios)\b/.test(skillText)) return true;
-  return /\b(eas|expo|cicd)\b/.test(skillText);
+function skillScoreFromEmbedding(skill, embeddingScore, reasons = []) {
+  const score = Math.min(1, Number(embeddingScore || 0));
+  return {
+    name: skill.name,
+    description: skill.description,
+    path: skill.path,
+    scope: skill.scope,
+    score,
+    embeddingScore: score,
+    reasons
+  };
 }
 
-function skillIntentBonus(normalizedPrompt, enriched, projectTokens = new Set()) {
-  const skillText = normalize(`${enriched.name} ${enriched.description}`);
-  if (isMcpRelevantTask(normalizedPrompt, projectTokens)
-    && /\b(mcp|model context protocol|modelcontextprotocol|agent memory|tool developer|tool builder)\b/.test(skillText)) {
-    return 0.48;
-  }
-  if (isCommerceTask(normalizedPrompt)
-    && /\b(payment|payments|checkout|billing|bill|invoice|wallet|balance|stripe|paypal|commerce|monetization)\b/.test(skillText)) {
-    return 0.46;
-  }
-  if (isContentAccessTask(normalizedPrompt)
-    && /\b(api|endpoint|backend|service|services|auth|authorization|permission|permissions|access|rbac|frontend api)\b/.test(skillText)) {
-    return 0.34;
-  }
-  if (isNotificationTask(normalizedPrompt)
-    && /\b(notification|notifications|notify|message|sms|email|event|webhook)\b/.test(skillText)) {
-    return 0.3;
-  }
-  if (isFrontendCheckoutTask(normalizedPrompt)
-    && /\b(frontend|react|next|nextjs|ui|component|modal|api integration)\b/.test(skillText)) {
-    return 0.32;
-  }
-  if (isExpoRuntimeTask(normalizedPrompt, projectTokens)
-    && /\b(expo|eas|nativewind|react native|tailwind)\b/.test(skillText)) {
-    return 0.46;
-  }
-  if (isNextAppRouterTask(normalizedPrompt)
-    && /\b(next|nextjs)\b/.test(skillText)
-    && /\b(app router|router|routing|server components)\b/.test(skillText)) {
-    return 0.5;
-  }
-  if (/\beas\b/.test(normalizedPrompt)
-    && /\b(eas|expo)\b/.test(skillText)
-    && /\b(cicd|workflow|workflows|build|deploy|deployment|pipeline|pipelines)\b/.test(skillText)) {
-    return 0.28;
-  }
-  if (/\b(webapp|frontend|ui|dashboard|button|page|component|app|router)\b/.test(normalizedPrompt)
-    && /\b(frontend|react|next|nextjs|ui|component|tailwind|app router)\b/.test(skillText)) {
-    return 0.36;
-  }
-  if (/\b(role|admin|creator|permission|permissions|authorization|access)\b/.test(normalizedPrompt)
-    && /\b(auth|authentication|authorization|permission|permissions|access|rbac)\b/.test(skillText)) {
-    return 0.32;
+function dedupeSkills(skills) {
+  const byName = new Map();
+  for (const skill of skills || []) {
+    const enriched = skill.searchTokens ? skill : enrichSkill(skill);
+    const key = normalize(enriched.name);
+    if (!key) continue;
+    const existing = byName.get(key);
+    if (!existing || skillSourcePriority(enriched) > skillSourcePriority(existing)) {
+      byName.set(key, enriched);
+    }
   }
-  return 0;
+  return [...byName.values()];
 }
 
-function skillRelevancePriority(normalizedPrompt, enriched, projectTokens = new Set()) {
-  const skillText = normalize(`${enriched.name} ${enriched.description}`);
-  const skillName = normalize(enriched.name);
+function skillSourcePriority(skill) {
   let priority = 0;
-  if (isMcpRelevantTask(normalizedPrompt, projectTokens)) {
-    if (skillName === "mcp builder") priority += 760;
-    if (skillName === "mcp management") priority += 740;
-    if (skillName === "mcp tool developer") priority += 720;
-    if (skillName === "agent memory mcp") priority += 700;
-    if (skillName === "agent tool builder" || skillName === "context agent") priority += 260;
-    if (/\b(mcp|model context protocol|modelcontextprotocol)\b/.test(skillText)) priority += 160;
-  }
-  if (isCommerceTask(normalizedPrompt)) {
-    if (/\b(payment integration|stripe integration|paypal integration)\b/.test(skillText)) priority += 520;
-    if (/\bbilling automation\b/.test(skillText)) priority += 430;
-    if (/\b(payment|payments|checkout|billing|wallet|balance|stripe|paypal|commerce|monetization)\b/.test(skillText)) priority += 160;
-    if (!/\bstripe\b/.test(normalizedPrompt) && /\bstripe\b/.test(skillText)) priority -= 520;
-    if (!/\bpaypal\b/.test(normalizedPrompt) && /\bpaypal\b/.test(skillText)) priority -= 520;
-    if (!/\bsquare\b/.test(normalizedPrompt) && /\bsquare\b/.test(skillText)) priority -= 440;
-    if (/\b(mcp|metasploit|penetration|exploit|bug bounty)\b/.test(skillText)) priority -= 500;
-  }
-  if (isContentAccessTask(normalizedPrompt)) {
-    if (/\b(api endpoint builder|backend development|backend architect|frontend api integration patterns)\b/.test(skillText)) priority += 260;
-    if (/\b(auth implementation patterns|authorization|permission|permissions|access|rbac)\b/.test(skillText)) priority += 120;
-  }
-  if (isNotificationTask(normalizedPrompt)) {
-    if (/\bsendblue notify\b/.test(skillText)) priority += 140;
-    if (/\b(notification|notifications|notify|message|sms|email|event|webhook)\b/.test(skillText)) priority += 90;
-  }
-  if (isFrontendCheckoutTask(normalizedPrompt)) {
-    if (/\bfrontend api integration patterns\b/.test(skillText)) priority += 220;
-    if (/\breact nextjs development|nextjs best practices|nextjs app router patterns|frontend developer\b/.test(skillText)) priority += 90;
-  }
-  if (isExpoRuntimeTask(normalizedPrompt, projectTokens)) {
-    if (/\bexpo deployment\b/.test(skillText)) priority += 900;
-    if (/\bbuilding native ui\b/.test(skillText)) priority += 760;
-    if (/\bexpo tailwind setup\b/.test(skillText)) priority += 620;
-    if (/\bexpo\b/.test(skillText) && /\b(qr|expo go|run|running|start|connect|eas|deployment|build)\b/.test(skillText)) priority += 220;
-    if (/\bnativewind|tailwind\b/.test(skillText) && projectTokens.has("nativewind")) priority += 120;
-    if (/\b(next|nextjs|frontend designer|dark themed|glassmorphism|framer motion)\b/.test(skillText)) priority -= 160;
-  }
-  if (isNextAppRouterTask(normalizedPrompt)) {
-    if (/\bnextjs app router patterns\b/.test(skillText)) priority += 600;
-    if (/\bnextjs best practices\b/.test(skillText)) priority += 560;
-    if (/\breact nextjs development\b/.test(skillText)) priority += 420;
-    if (/\b(next|nextjs)\b/.test(skillText) && /\b(app router|router|routing|server components)\b/.test(skillText)) priority += 100;
-    if (/\b(next|nextjs)\b/.test(skillText) && /\breact\b/.test(skillText)) priority += 70;
-    if (/\b(glassmorphism|dark themed|dark theme|framer motion)\b/.test(skillText)) priority -= 40;
-  }
-  if (/\b(role|admin|creator|permission|permissions|authorization|access)\b/.test(normalizedPrompt)
-    && /\b(auth|authentication|authorization|permission|permissions|access|rbac)\b/.test(skillText)) {
-    priority += 55;
-  }
+  if (skill.scope === "project") priority += 100;
+  const skillPath = String(skill.path || "");
+  if (skillPath.includes(`${path.sep}.codex${path.sep}skills${path.sep}`)) priority += 30;
+  if (skillPath.includes(`${path.sep}.config${path.sep}skillshare${path.sep}skills${path.sep}`)) priority += 20;
+  if (skillPath.includes(`${path.sep}.agents${path.sep}skills${path.sep}`)) priority += 10;
   return priority;
 }
 
-function isNextAppRouterTask(normalizedPrompt) {
-  return /\bwebapp\b.*\bsrc\b.*\bapp\b/.test(normalizedPrompt)
-    || /\b(next|nextjs)\b.*\b(app router|router|routing)\b/.test(normalizedPrompt)
-    || /\bapp router\b/.test(normalizedPrompt);
-}
-
-function isExpoRuntimeTask(normalizedPrompt, projectTokens = new Set()) {
-  const expoProject = projectTokens.has("expo") || projectTokens.has("nativewind") || projectTokens.has("eas");
-  if (!expoProject) return false;
-  return /\b(qr|connect|run|start|expo go|device|metro|tunnel|lan)\b/.test(normalizedPrompt);
-}
-
-function isCommerceTask(normalizedPrompt) {
-  return /\b(purchase|purchased|buy|buyer|seller|payment|pay|checkout|wallet|balance|top up|topup|funded|billing|invoice)\b/.test(normalizedPrompt);
+function skillIndexKind(cwd) {
+  return `skill:${path.resolve(cwd)}`;
 }
 
-function isContentAccessTask(normalizedPrompt) {
-  return /\b(content access service|content access|access permissions|grant access|permissions|library|resources|tutorials|collections)\b/.test(normalizedPrompt);
+function skillIndexId(skill) {
+  return normalize(skill.name);
 }
 
-function isNotificationTask(normalizedPrompt) {
-  return /\b(notification|notifications|notify|buyer|seller)\b/.test(normalizedPrompt);
-}
-
-function isFrontendCheckoutTask(normalizedPrompt) {
-  return /\b(modal|display|show|checkout|library|frontend|webapp|page|button)\b/.test(normalizedPrompt);
-}
-
-function isMcpTask(normalizedPrompt) {
-  return /\b(mcp|model context protocol|tool server|tools server|server tool|bridge|proxy)\b/.test(normalizedPrompt);
-}
-
-function isMcpRelevantTask(normalizedPrompt, projectTokens = new Set()) {
-  return isMcpTask(normalizedPrompt)
-    || (isMcpProject(projectTokens) && isContextRetrievalTask(normalizedPrompt));
-}
-
-function isMcpProject(projectTokens = new Set()) {
-  return projectTokens.has("mcp") || projectTokens.has("modelcontextprotocol");
-}
-
-function isContextRetrievalTask(normalizedPrompt) {
-  return /\b(suggest|suggested|suggestion|skills|files|context|retrieval|retrieve|scorer|scoring|match|matching|prompt|hook|inject|injection)\b/.test(normalizedPrompt);
-}
-
-function isSecurityTask(normalizedPrompt) {
-  return /\b(security|pentest|penetration|exploit|vulnerability|metasploit|bug bounty|owasp|xss|csrf|attack|audit)\b/.test(normalizedPrompt);
-}
-
-function isMcpSkill(skillText) {
-  return /\bmcp\b|\bmodel context protocol\b/.test(skillText);
-}
-
-function isOffensiveSecuritySkill(skillText) {
-  return /\b(metasploit|penetration testing|bug bounty|exploit|exploitation|privilege escalation|ethical hacking|web fuzzing|security assessment)\b/.test(skillText);
-}
-
-function isPlatformCommerceSkill(skillText) {
-  return /\b(wordpress|woocommerce|shopify|odoo)\b/.test(skillText);
-}
-
-function isPlatformCommerceTask(normalizedPrompt, skillText) {
-  if (/\bwordpress\b/.test(skillText)) return /\bwordpress\b/.test(normalizedPrompt);
-  if (/\bwoocommerce\b/.test(skillText)) return /\bwoocommerce\b/.test(normalizedPrompt);
-  if (/\bshopify\b/.test(skillText)) return /\bshopify\b/.test(normalizedPrompt);
-  if (/\bodoo\b/.test(skillText)) return /\bodoo\b/.test(normalizedPrompt);
-  return true;
+function skillEmbeddingText(skill) {
+  return [skill.name, skill.description].filter(Boolean).join("\n");
 }
 
 export function projectSkillHints({ cwd = process.cwd() } = {}) {
@@ -511,48 +344,6 @@ export function projectSkillHints({ cwd = process.cwd() } = {}) {
   return [...hints];
 }
 
-function workspacePackagePaths(cwd) {
-  const rootPackagePath = path.join(cwd, "package.json");
-  const rootPackage = readJson(rootPackagePath);
-  const paths = new Set([rootPackagePath]);
-  for (const workspace of workspacePatterns(rootPackage?.workspaces)) {
-    for (const packagePath of expandWorkspacePattern({ cwd, pattern: workspace })) {
-      paths.add(packagePath);
-    }
-  }
-  return [...paths];
-}
-
-function workspacePatterns(workspaces) {
-  if (Array.isArray(workspaces)) return workspaces.filter((item) => typeof item === "string");
-  if (Array.isArray(workspaces?.packages)) return workspaces.packages.filter((item) => typeof item === "string");
-  return [];
-}
-
-function expandWorkspacePattern({ cwd, pattern }) {
-  const normalized = String(pattern || "").replace(/\\/g, "/").replace(/\/+$/g, "");
-  if (!normalized || normalized.startsWith("..") || path.isAbsolute(normalized)) return [];
-  if (!normalized.includes("*")) {
-    const packagePath = path.join(cwd, normalized, "package.json");
-    return fs.existsSync(packagePath) ? [packagePath] : [];
-  }
-  const parts = normalized.split("/");
-  const starIndex = parts.indexOf("*");
-  if (starIndex < 0 || parts.includes("**")) return [];
-  const baseDir = path.join(cwd, ...parts.slice(0, starIndex));
-  const suffix = parts.slice(starIndex + 1);
-  let entries = [];
-  try {
-    entries = fs.readdirSync(baseDir, { withFileTypes: true });
-  } catch {
-    return [];
-  }
-  return entries
-    .filter((entry) => entry.isDirectory() && !entry.name.startsWith("."))
-    .map((entry) => path.join(baseDir, entry.name, ...suffix, "package.json"))
-    .filter((packagePath) => fs.existsSync(packagePath));
-}
-
 function readJson(filePath) {
   try {
     return JSON.parse(fs.readFileSync(filePath, "utf8"));
@@ -583,10 +374,3 @@ function enrichSkill(skill) {
 function normalize(value) {
   return String(value || "").toLowerCase().replace(/[^a-z0-9]+/g, " ").trim();
 }
-
-function normalizePrompt(value) {
-  return normalize(String(value || "")
-    .replace(/https?:\/\/\S+/gi, " ")
-    .replace(/giao\s+di[eệ]n/gi, "frontend ui")
-    .replace(/phan\s+quyen/gi, "authorization role"));
-}