trackops 2.0.3 → 2.0.5

package/lib/skills.js

@@ -8,6 +8,14 @@ const { t, setLocale } = require("./i18n");
 const { resolveSkillFile } = require("./resources");
 
 const SKILLS_TEMPLATES_DIR = path.join(__dirname, "..", "templates", "skills");
+const INSTALLED_SKILL_PRIORITY = [
+  "opera-skill",
+  "project-starter-skill",
+  "opera-contract-auditor",
+  "opera-policy-guard",
+  "commiter",
+  "changelog-updater",
+];
 
 function copyDirRecursive(src, dest) {
   fs.mkdirSync(dest, { recursive: true });
@@ -22,9 +30,9 @@ function copyDirRecursive(src, dest) {
   }
 }
 
-function parseFrontmatter(content) {
-  const match = content.match(/^---\s*\n([\s\S]*?)\n---/);
-  if (!match) return {};
+function parseFrontmatter(content) {
+  const match = content.match(/^---\s*\n([\s\S]*?)\n---/);
+  if (!match) return {};
   const fm = {};
   for (const line of match[1].split("\n")) {
     const sep = line.indexOf(":");
@@ -34,16 +42,16 @@ function parseFrontmatter(content) {
       fm[key] = val;
     }
   }
-  return fm;
-}
-
-function getSkillsDir(root) {
-  return config.ensureContext(root).paths.skillsDir;
-}
-
-function getRegistryPath(root) {
-  return config.ensureContext(root).paths.registryPath;
-}
+  return fm;
+}
+
+function getSkillsDir(root) {
+  return config.ensureContext(root).paths.skillsDir;
+}
+
+function getRegistryPath(root) {
+  return config.ensureContext(root).paths.registryPath;
+}
 
 function catalogSkills() {
   if (!fs.existsSync(SKILLS_TEMPLATES_DIR)) return [];
@@ -58,28 +66,37 @@ function catalogSkills() {
     .filter(Boolean);
 }
 
-function installedSkills(root) {
-  const skillsDir = getSkillsDir(root);
-  if (!fs.existsSync(skillsDir)) return [];
-  return fs.readdirSync(skillsDir, { withFileTypes: true })
-    .filter((e) => e.isDirectory() && fs.existsSync(path.join(skillsDir, e.name, "SKILL.md")))
-    .map((e) => {
-      const fm = parseFrontmatter(fs.readFileSync(path.join(skillsDir, e.name, "SKILL.md"), "utf8"));
-      return { name: e.name, description: fm.description || "", version: fm.version || "1.0" };
-    })
-    .filter(Boolean);
-}
+function installedSkills(root) {
+  const skillsDir = getSkillsDir(root);
+  if (!fs.existsSync(skillsDir)) return [];
+  const skills = fs.readdirSync(skillsDir, { withFileTypes: true })
+    .filter((e) => e.isDirectory() && fs.existsSync(path.join(skillsDir, e.name, "SKILL.md")))
+    .map((e) => {
+      const fm = parseFrontmatter(fs.readFileSync(path.join(skillsDir, e.name, "SKILL.md"), "utf8"));
+      return { name: e.name, description: fm.description || "", version: fm.version || "1.0" };
+    })
+    .filter(Boolean);
+  skills.sort((a, b) => {
+    const ai = INSTALLED_SKILL_PRIORITY.indexOf(a.name);
+    const bi = INSTALLED_SKILL_PRIORITY.indexOf(b.name);
+    const ap = ai >= 0 ? ai : INSTALLED_SKILL_PRIORITY.length;
+    const bp = bi >= 0 ? bi : INSTALLED_SKILL_PRIORITY.length;
+    if (ap !== bp) return ap - bp;
+    return a.name.localeCompare(b.name);
+  });
+  return skills;
+}
 
 function updateRegistry(root) {
   const registryPath = getRegistryPath(root);
   fs.mkdirSync(path.dirname(registryPath), { recursive: true });
-  let locale = "es";
-  const context = config.ensureContext(root);
-  const controlFile = config.controlFilePath(context);
-  if (fs.existsSync(controlFile)) {
-    try {
-      locale = config.getLocale(config.loadControl(context));
-      setLocale(locale);
+  let locale = "es";
+  const context = config.ensureContext(root);
+  const controlFile = config.controlFilePath(context);
+  if (fs.existsSync(controlFile)) {
+    try {
+      locale = config.getLocale(config.loadControl(context));
+      setLocale(locale);
     } catch (_error) {
       setLocale("es");
     }
@@ -97,29 +114,29 @@ function updateRegistry(root) {
   fs.writeFileSync(registryPath, lines.join("\n") + "\n", "utf8");
 
   // Also update control meta
-  if (fs.existsSync(controlFile)) {
-    try {
-      const control = config.loadControl(context);
-      if (control.meta.opera) {
-        control.meta.opera.skills = skills.map((s) => s.name);
-        config.saveControl(context, control);
-      }
+  if (fs.existsSync(controlFile)) {
+    try {
+      const control = config.loadControl(context);
+      if (control.meta.opera) {
+        control.meta.opera.skills = skills.map((s) => s.name);
+        config.saveControl(context, control);
+      }
     } catch (_e) { /* ignore */ }
   }
 }
 
-function installSkill(root, skillName) {
-  const context = config.ensureContext(root);
-  const control = config.loadControl(context);
-  setLocale(config.getLocale(control));
-  const locale = config.getLocale(control);
+function installSkill(root, skillName) {
+  const context = config.ensureContext(root);
+  const control = config.loadControl(context);
+  setLocale(config.getLocale(control));
+  const locale = config.getLocale(control);
 
   const templateDir = path.join(SKILLS_TEMPLATES_DIR, skillName);
   if (!fs.existsSync(templateDir)) {
     throw new Error(t("skill.notFound", { name: skillName }));
   }
 
-  const targetDir = path.join(getSkillsDir(context), skillName);
+  const targetDir = path.join(getSkillsDir(context), skillName);
   if (fs.existsSync(path.join(targetDir, "SKILL.md"))) {
     console.log(t("skill.alreadyInstalled", { name: skillName }));
     return;
@@ -130,25 +147,25 @@ function installSkill(root, skillName) {
   if (localizedSkill) {
     fs.copyFileSync(localizedSkill, path.join(targetDir, "SKILL.md"));
   }
-  updateRegistry(context);
-  console.log(t("skill.installed", { name: skillName }));
-}
-
-function removeSkill(root, skillName) {
-  const context = config.ensureContext(root);
-  const control = config.loadControl(context);
-  setLocale(config.getLocale(control));
-
-  const targetDir = path.join(getSkillsDir(context), skillName);
+  updateRegistry(context);
+  console.log(t("skill.installed", { name: skillName }));
+}
+
+function removeSkill(root, skillName) {
+  const context = config.ensureContext(root);
+  const control = config.loadControl(context);
+  setLocale(config.getLocale(control));
+
+  const targetDir = path.join(getSkillsDir(context), skillName);
   if (!fs.existsSync(targetDir)) {
     console.log(t("skill.notInstalled", { name: skillName }));
     return;
   }
 
   fs.rmSync(targetDir, { recursive: true, force: true });
-  updateRegistry(context);
-  console.log(t("skill.removed", { name: skillName }));
-}
+  updateRegistry(context);
+  console.log(t("skill.removed", { name: skillName }));
+}
 
 /* ── CLI commands ── */