opencode-gitlab-dap 1.15.1 → 1.15.3

package/dist/index.js

@@ -3946,83 +3946,6 @@ async function removeIndexEntry(instanceUrl, token, scope, id, indexSlug, name)
     await writeIndex(instanceUrl, token, scope, id, indexSlug, filtered);
   }
 }
-function extractSkillNames(pages, prefix) {
-  const skillSuffix = "/SKILL";
-  const names = /* @__PURE__ */ new Set();
-  for (const p of pages) {
-    if (p.slug.startsWith(prefix + "/") && p.slug.endsWith(skillSuffix)) {
-      const middle = p.slug.slice(prefix.length + 1, -skillSuffix.length);
-      if (middle && !middle.includes("/")) {
-        names.add(middle);
-      }
-    }
-  }
-  return [...names];
-}
-async function rebuildIndex(instanceUrl, token, scope, id, prefix, indexSlug) {
-  const pages = await listWikiPages(instanceUrl, token, scope, id);
-  const actualNames = extractSkillNames(pages, prefix);
-  const currentEntries = await readIndex(instanceUrl, token, scope, id, indexSlug);
-  const indexed = new Set(currentEntries.map((e) => e.name));
-  const actual = new Set(actualNames);
-  let dirty = false;
-  const removed = currentEntries.filter((e) => !actual.has(e.name));
-  if (removed.length > 0) dirty = true;
-  const added = [];
-  for (const name of actualNames) {
-    if (!indexed.has(name)) {
-      added.push(name);
-      dirty = true;
-    }
-  }
-  const staleDescriptions = currentEntries.filter(
-    (e) => actual.has(e.name) && (e.description.startsWith("(auto-indexed") || e.description.startsWith("(no description"))
-  );
-  if (staleDescriptions.length > 0) dirty = true;
-  if (!dirty && added.length === 0) return currentEntries;
-  const kept = [];
-  for (const e of currentEntries) {
-    if (!actual.has(e.name)) continue;
-    if (e.description.startsWith("(auto-indexed") || e.description.startsWith("(no description")) {
-      let description = "";
-      try {
-        const page = await getWikiPage(instanceUrl, token, scope, id, `${prefix}/${e.name}/SKILL`);
-        const content = page.content ?? "";
-        const fmMatch = content.match(/^---\s*\n[\s\S]*?description:\s*(.+)\n[\s\S]*?---/);
-        if (fmMatch) {
-          description = fmMatch[1].trim();
-        } else {
-          description = content.replace(/^---[\s\S]*?---\s*\n/, "").replace(/^#[^\n]*\n+/, "").split("\n\n")[0].replace(/\n/g, " ").trim().slice(0, 200);
-        }
-      } catch {
-      }
-      kept.push({ ...e, description: description || e.description });
-    } else {
-      kept.push(e);
-    }
-  }
-  for (const name of added) {
-    let description = "";
-    try {
-      const page = await getWikiPage(instanceUrl, token, scope, id, `${prefix}/${name}/SKILL`);
-      const content = page.content ?? "";
-      const fmMatch = content.match(/^---\s*\n[\s\S]*?description:\s*(.+)\n[\s\S]*?---/);
-      if (fmMatch) {
-        description = fmMatch[1].trim();
-      } else {
-        description = content.replace(/^---[\s\S]*?---\s*\n/, "").replace(/^#[^\n]*\n+/, "").split("\n\n")[0].replace(/\n/g, " ").trim().slice(0, 200);
-      }
-    } catch {
-    }
-    kept.push({
-      name,
-      description: description || "(no description \u2014 update with gitlab_skill_save)",
-      draft: prefix === DRAFTS_PREFIX
-    });
-  }
-  await writeIndex(instanceUrl, token, scope, id, indexSlug, kept);
-  return kept;
-}
 async function upsertPage(instanceUrl, token, scope, id, slug, content) {
   try {
     await updateWikiPage(instanceUrl, token, scope, id, slug, content);
@@ -4120,7 +4043,10 @@ function downloadSkillFromSkillsSh(identifier) {
         if (statSync(full).isDirectory()) {
           walkStack.push({ dir: full, prefix: rel });
         } else if (entry !== "SKILL.md") {
-          files.push({ path: rel, content: readFileSync2(full, "utf-8") });
+          try {
+            files.push({ path: rel, content: readFileSync2(full, "utf-8") });
+          } catch {
+          }
         }
       }
     }
@@ -4144,7 +4070,7 @@ function makeSkillTools(ctx) {
   }
   return {
     gitlab_skill_list: tool6({
-      description: "List available project skills and optionally draft skills.\nSkills define step-by-step procedures for common tasks (e.g., incident retros, debugging, deployments).\nAuto-rebuilds the skill index if it is out of sync with actual skill pages.",
+      description: "List available project skills and optionally draft skills.\nSkills define step-by-step procedures for common tasks (e.g., incident retros, debugging, deployments).",
       args: {
         project_id: z6.string().describe(PROJECT_ID_DESC2),
         include_drafts: z6.boolean().optional().describe("Also list draft skills (default: false)"),
@@ -4155,25 +4081,12 @@ function makeSkillTools(ctx) {
         const auth = authAndValidate(args.project_id);
         const { scope, id } = resolveScope2(args);
         try {
-          const published = await rebuildIndex(
-            auth.instanceUrl,
-            auth.token,
-            scope,
-            id,
-            SKILLS_PREFIX,
-            SKILLS_INDEX
-          );
+          const published = await readIndex(auth.instanceUrl, auth.token, scope, id, SKILLS_INDEX);
           let drafts = [];
           if (args.include_drafts) {
-            drafts = await rebuildIndex(
-              auth.instanceUrl,
-              auth.token,
-              scope,
-              id,
-              DRAFTS_PREFIX,
-              DRAFTS_INDEX
+            drafts = (await readIndex(auth.instanceUrl, auth.token, scope, id, DRAFTS_INDEX)).map(
+              (e) => ({ ...e, draft: true })
             );
-            drafts = drafts.map((e) => ({ ...e, draft: true }));
           }
           const all = [...published, ...drafts];
           if (all.length === 0) return "No skills found. Use gitlab_skill_save to create one.";
@@ -4422,7 +4335,7 @@ Install: \`gitlab_skill_install(name="${r.identifier}", source="skills.sh")\``
                 args.project_id,
                 `skill:${downloaded.name}`,
                 `Scripts for skill "${downloaded.name}" (installed from skills.sh:${args.name})`,
-                scriptFiles.map((f) => ({ file_path: f.path, content: f.content })),
+                scriptFiles.map((f) => ({ file_path: f.path, content: f.content || "\n" })),
                 "private"
               );
               snippetId = snippet.id;