opencode-gitlab-dap 1.8.2 → 1.8.4

package/dist/index.js

@@ -3443,13 +3443,13 @@ async function searchWikiPages(instanceUrl, token, scope, id, query) {
 // src/tools/memory-tools.ts
 var z5 = tool5.schema;
 var PREFIX = "agents";
-var MEMORY_SLUGS = {
+var MEMORY_DIRS = {
   all: [`${PREFIX}/memory/facts`, `${PREFIX}/memory/decisions`, `${PREFIX}/memory/patterns`],
   facts: [`${PREFIX}/memory/facts`],
   decisions: [`${PREFIX}/memory/decisions`],
   patterns: [`${PREFIX}/memory/patterns`]
 };
-var RECORD_SLUG = {
+var RECORD_DIR = {
   fact: `${PREFIX}/memory/facts`,
   decision: `${PREFIX}/memory/decisions`,
   pattern: `${PREFIX}/memory/patterns`
@@ -3460,21 +3460,9 @@ function today() {
 function slugify(text) {
   return text.toLowerCase().replace(/[^a-z0-9]+/g, "-").replace(/(^-|-$)/g, "").slice(0, 60);
 }
-async function safeRead(instanceUrl, token, scope, id, slug) {
-  try {
-    const page = await getWikiPage(instanceUrl, token, scope, id, slug);
-    return page.content;
-  } catch {
-    return null;
-  }
-}
-async function appendToPage(instanceUrl, token, scope, id, slug, newContent) {
-  const existing = await safeRead(instanceUrl, token, scope, id, slug);
-  if (existing !== null) {
-    await updateWikiPage(instanceUrl, token, scope, id, slug, existing + "\n\n" + newContent);
-  } else {
-    await createWikiPage(instanceUrl, token, scope, id, slug, newContent);
-  }
+function titleFromContent(content) {
+  const first = content.split("\n")[0].replace(/^[#*\->\s]+/, "").trim();
+  return first.slice(0, 80) || "untitled";
 }
 function validateProjectId(projectId) {
   if (!projectId.includes("/")) {
@@ -3511,25 +3499,38 @@ function makeMemoryTools(ctx) {
         const auth = authAndValidate(args.project_id);
         const { scope, id } = resolveScope(args);
         const memType = args.type ?? "all";
-        const slugs = MEMORY_SLUGS[memType];
-        if (!slugs) return `Unknown memory type: ${memType}`;
-        const sections = [];
-        for (const slug of slugs) {
-          const content = await safeRead(auth.instanceUrl, auth.token, scope, id, slug);
-          if (content) {
-            const label = slug.split("/").pop();
-            sections.push(`## ${label.charAt(0).toUpperCase() + label.slice(1)}
-
-${content}`);
+        const dirs = MEMORY_DIRS[memType];
+        if (!dirs) return `Unknown memory type: ${memType}`;
+        try {
+          const allPages = await listWikiPages(
+            auth.instanceUrl,
+            auth.token,
+            scope,
+            id,
+            true
+          );
+          const sections = [];
+          for (const dir of dirs) {
+            const label = dir.split("/").pop();
+            const pages = allPages.filter((p) => p.slug.startsWith(dir + "/") && p.content);
+            if (pages.length === 0) continue;
+            const entries = pages.map((p) => p.content).join("\n\n---\n\n");
+            sections.push(
+              `## ${label.charAt(0).toUpperCase() + label.slice(1)} (${pages.length})
+
+${entries}`
+            );
           }
+          if (sections.length === 0)
+            return "No project memory found. Use gitlab_memory_record to start building project knowledge.";
+          return sections.join("\n\n---\n\n");
+        } catch (err) {
+          return `Error loading memory: ${err.message}`;
         }
-        if (sections.length === 0)
-          return "No project memory found. Use gitlab_memory_record to start building project knowledge.";
-        return sections.join("\n\n---\n\n");
       }
     }),
     gitlab_memory_record: tool5({
-      description: "Record a fact, decision, or pattern in project memory.\nFacts: stable truths about the project (e.g., deploy targets, tech stack, team conventions).\nDecisions: architectural choices with reasoning (why X was chosen over Y).\nPatterns: recurring observations that may evolve into skills over time.\nEntries are automatically timestamped and appended to the appropriate memory page.\nMultiple facts can be recorded in a single call by separating them with newlines.",
+      description: "Record a fact, decision, or pattern in project memory.\nFacts: stable truths about the project (e.g., deploy targets, tech stack, team conventions).\nDecisions: architectural choices with reasoning (why X was chosen over Y).\nPatterns: recurring observations that may evolve into skills over time.\nEach record creates its own page \u2014 safe for parallel writes.",
       args: {
         project_id: z5.string().describe(
           'FULL project path with namespace (e.g., "gitlab-org/gitlab"). Must contain a slash. Never use just the project name.'
@@ -3542,24 +3543,18 @@ ${content}`);
       execute: async (args) => {
         const auth = authAndValidate(args.project_id);
         const { scope, id } = resolveScope(args);
-        const slug = RECORD_SLUG[args.type];
-        if (!slug) return `Unknown memory type: ${args.type}`;
+        const dir = RECORD_DIR[args.type];
+        if (!dir) return `Unknown memory type: ${args.type}`;
         const date = today();
-        let formatted;
-        if (args.type === "decision") {
-          const firstLine = args.content.split("\n")[0];
-          const rest = args.content.split("\n").slice(1).join("\n").trim();
-          formatted = `## ${date}: ${firstLine}${rest ? "\n" + rest : ""}`;
-        } else if (args.type === "fact") {
-          formatted = `## ${date}
-- ${args.content.replace(/\n/g, "\n- ")}`;
-        } else {
-          formatted = `## ${date}
-${args.content}`;
-        }
+        const title = titleFromContent(args.content);
+        const slug = `${dir}/${date}-${slugify(title)}`;
+        const header = `*Recorded: ${date} | Type: ${args.type}*
+
+`;
+        const body = header + args.content;
         try {
-          await appendToPage(auth.instanceUrl, auth.token, scope, id, slug, formatted);
-          return `Recorded ${args.type} in project memory.`;
+          await createWikiPage(auth.instanceUrl, auth.token, scope, id, slug, body);
+          return `Recorded ${args.type} in project memory: ${slug}`;
         } catch (err) {
           return `Error recording ${args.type}: ${err.message}`;
         }
@@ -3619,14 +3614,6 @@ ${args.content}`;
           await createWikiPage(auth.instanceUrl, auth.token, scope, id, slug, args.summary);
           return `Session logged: ${slug}`;
         } catch (err) {
-          if (err.message?.includes("already exists") || err.message?.includes("422")) {
-            try {
-              await updateWikiPage(auth.instanceUrl, auth.token, scope, id, slug, args.summary);
-              return `Session log updated: ${slug}`;
-            } catch (err2) {
-              return `Error logging session: ${err2.message}`;
-            }
-          }
           return `Error logging session: ${err.message}`;
         }
       }
@@ -3756,17 +3743,26 @@ ${draft.content}`;
         const { scope, id } = resolveScope2(args);
         const prefix = args.draft ? DRAFTS_PREFIX : SKILLS_PREFIX;
         const slug = `${prefix}/${args.name}`;
-        try {
-          await updateWikiPage(auth.instanceUrl, auth.token, scope, id, slug, args.content);
-          return `Updated ${args.draft ? "draft " : ""}skill: ${args.name}`;
-        } catch {
+        const label = args.draft ? "draft " : "";
+        for (let attempt = 0; attempt < 3; attempt++) {
           try {
-            await createWikiPage(auth.instanceUrl, auth.token, scope, id, slug, args.content);
-            return `Created ${args.draft ? "draft " : ""}skill: ${args.name}`;
-          } catch (err) {
-            return `Error saving skill: ${err.message}`;
+            await updateWikiPage(auth.instanceUrl, auth.token, scope, id, slug, args.content);
+            return `Updated ${label}skill: ${args.name}`;
+          } catch {
+            try {
+              await createWikiPage(auth.instanceUrl, auth.token, scope, id, slug, args.content);
+              return `Created ${label}skill: ${args.name}`;
+            } catch (err) {
+              const msg = err.message ?? "";
+              if (msg.includes("Duplicate page") || msg.includes("reference update")) {
+                await new Promise((r) => setTimeout(r, 1e3 * (attempt + 1)));
+                continue;
+              }
+              return `Error saving skill: ${msg}`;
+            }
           }
         }
+        return `Error saving skill: failed after 3 retries`;
       }
     }),
     gitlab_skill_promote: tool6({