opencode-gitlab-dap 1.8.1 → 1.8.3

package/dist/index.js

@@ -3468,14 +3468,51 @@ async function safeRead(instanceUrl, token, scope, id, slug) {
     return null;
   }
 }
+var MAX_RETRIES = 3;
+var RETRY_DELAY_MS = 1e3;
+async function sleep2(ms) {
+  return new Promise((resolve) => setTimeout(resolve, ms));
+}
 async function appendToPage(instanceUrl, token, scope, id, slug, newContent) {
+  for (let attempt = 0; attempt < MAX_RETRIES; attempt++) {
+    const existing2 = await safeRead(instanceUrl, token, scope, id, slug);
+    if (existing2 !== null) {
+      try {
+        await updateWikiPage(instanceUrl, token, scope, id, slug, existing2 + "\n\n" + newContent);
+        return;
+      } catch (err) {
+        if (attempt < MAX_RETRIES - 1 && err.message?.includes("reference update")) {
+          await sleep2(RETRY_DELAY_MS * (attempt + 1));
+          continue;
+        }
+        throw err;
+      }
+    } else {
+      try {
+        await createWikiPage(instanceUrl, token, scope, id, slug, newContent);
+        return;
+      } catch (err) {
+        if (err.message?.includes("Duplicate page") || err.message?.includes("reference update")) {
+          await sleep2(RETRY_DELAY_MS * (attempt + 1));
+          continue;
+        }
+        throw err;
+      }
+    }
+  }
   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);
+    throw new Error(`Failed to append to ${slug} after ${MAX_RETRIES} retries`);
   }
 }
+function validateProjectId(projectId) {
+  if (!projectId.includes("/")) {
+    return `Invalid project_id "${projectId}". Must be the full project path containing at least one slash (e.g., "my-group/my-project"), not just the project name.`;
+  }
+  return null;
+}
 function resolveScope(args) {
   if (args.scope === "groups" && args.group_id) {
     return { scope: "groups", id: args.group_id };
@@ -3483,18 +3520,26 @@ function resolveScope(args) {
   return { scope: "projects", id: args.project_id };
 }
 function makeMemoryTools(ctx) {
+  function authAndValidate(projectId) {
+    const auth = ctx.ensureAuth();
+    if (!auth) throw new Error("GitLab authentication not available");
+    const err = validateProjectId(projectId);
+    if (err) throw new Error(err);
+    return auth;
+  }
   return {
     gitlab_memory_load: tool5({
       description: "Load project memory to understand context, known facts, past decisions, and observed patterns.\nUse this at the start of complex tasks to check what is already known about the project.\nReturns accumulated knowledge from previous sessions.",
       args: {
-        project_id: z5.string().describe('Project path (e.g., "gitlab-org/gitlab"). Use the same value consistently.'),
+        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.'
+        ),
         type: z5.enum(["all", "facts", "decisions", "patterns"]).optional().describe('Which memory to load: "all" (default), "facts", "decisions", or "patterns"'),
         scope: z5.enum(["projects", "groups"]).optional().describe("Scope (default: projects)"),
         group_id: z5.string().optional().describe("Group path (required when scope is groups)")
       },
       execute: async (args) => {
-        const auth = ctx.ensureAuth();
-        if (!auth) throw new Error("GitLab authentication not available");
+        const auth = authAndValidate(args.project_id);
         const { scope, id } = resolveScope(args);
         const memType = args.type ?? "all";
         const slugs = MEMORY_SLUGS[memType];
@@ -3517,15 +3562,16 @@ ${content}`);
     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.",
       args: {
-        project_id: z5.string().describe('Project path (e.g., "gitlab-org/gitlab"). Use the same value consistently.'),
+        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.'
+        ),
         type: z5.enum(["fact", "decision", "pattern"]).describe("Type of knowledge to record"),
         content: z5.string().describe("The knowledge to record (markdown)"),
         scope: z5.enum(["projects", "groups"]).optional().describe("Scope (default: projects)"),
         group_id: z5.string().optional().describe("Group path (required when scope is groups)")
       },
       execute: async (args) => {
-        const auth = ctx.ensureAuth();
-        if (!auth) throw new Error("GitLab authentication not available");
+        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}`;
@@ -3553,14 +3599,15 @@ ${args.content}`;
     gitlab_memory_recall: tool5({
       description: "Search project knowledge for relevant information.\nSearches across all memory pages, session logs, and skills.\nUse this to check if something is already known before investigating.",
       args: {
-        project_id: z5.string().describe('Project path (e.g., "gitlab-org/gitlab"). Use the same value consistently.'),
+        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.'
+        ),
         query: z5.string().describe("What to search for"),
         scope: z5.enum(["projects", "groups"]).optional().describe("Scope (default: projects)"),
         group_id: z5.string().optional().describe("Group path (required when scope is groups)")
       },
       execute: async (args) => {
-        const auth = ctx.ensureAuth();
-        if (!auth) throw new Error("GitLab authentication not available");
+        const auth = authAndValidate(args.project_id);
         const { scope, id } = resolveScope(args);
         try {
           const results = await searchWikiPages(
@@ -3584,7 +3631,9 @@ ${args.content}`;
     gitlab_memory_log_session: tool5({
       description: "Log a session summary including what was accomplished, what was learned, and any suggestions.\nUse this at the end of significant work sessions to preserve context for future sessions.",
       args: {
-        project_id: z5.string().describe('Project path (e.g., "gitlab-org/gitlab"). Use the same value consistently.'),
+        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.'
+        ),
         title: z5.string().describe('Brief session title (e.g., "fix-ai-gateway-healthcheck")'),
         summary: z5.string().describe(
           "Session summary in markdown (what happened, what was learned, what went wrong)"
@@ -3593,25 +3642,36 @@ ${args.content}`;
         group_id: z5.string().optional().describe("Group path (required when scope is groups)")
       },
       execute: async (args) => {
-        const auth = ctx.ensureAuth();
-        if (!auth) throw new Error("GitLab authentication not available");
+        const auth = authAndValidate(args.project_id);
         const { scope, id } = resolveScope(args);
         const date = today();
         const slug = `${PREFIX}/memory/sessions/${date}-${slugify(args.title)}`;
-        try {
-          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}`;
+        for (let attempt = 0; attempt < MAX_RETRIES; attempt++) {
+          try {
+            await createWikiPage(auth.instanceUrl, auth.token, scope, id, slug, args.summary);
+            return `Session logged: ${slug}`;
+          } catch (err) {
+            const msg = err.message ?? "";
+            if (msg.includes("Duplicate page") || msg.includes("already exists") || msg.includes("422")) {
+              try {
+                await updateWikiPage(auth.instanceUrl, auth.token, scope, id, slug, args.summary);
+                return `Session log updated: ${slug}`;
+              } catch (err2) {
+                if (attempt < MAX_RETRIES - 1 && err2.message?.includes("reference update")) {
+                  await sleep2(RETRY_DELAY_MS * (attempt + 1));
+                  continue;
+                }
+                return `Error logging session: ${err2.message}`;
+              }
+            }
+            if (attempt < MAX_RETRIES - 1 && msg.includes("reference update")) {
+              await sleep2(RETRY_DELAY_MS * (attempt + 1));
+              continue;
             }
+            return `Error logging session: ${msg}`;
           }
-          return `Error logging session: ${err.message}`;
         }
+        return `Error logging session: failed after ${MAX_RETRIES} retries`;
       }
     })
   };
@@ -3629,19 +3689,33 @@ function resolveScope2(args) {
   }
   return { scope: "projects", id: args.project_id };
 }
+function validateProjectId2(projectId) {
+  if (!projectId.includes("/")) {
+    return `Invalid project_id "${projectId}". Must be the full project path containing at least one slash (e.g., "my-group/my-project"), not just the project name.`;
+  }
+  return null;
+}
 function makeSkillTools(ctx) {
+  function authAndValidate(projectId) {
+    const auth = ctx.ensureAuth();
+    if (!auth) throw new Error("GitLab authentication not available");
+    const err = validateProjectId2(projectId);
+    if (err) throw new Error(err);
+    return auth;
+  }
   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).",
       args: {
-        project_id: z6.string().describe('Project path (e.g., "gitlab-org/gitlab"). Use the same value consistently.'),
+        project_id: z6.string().describe(
+          'FULL project path with namespace (e.g., "gitlab-org/gitlab"). Must contain a slash. Never use just the project name.'
+        ),
         include_drafts: z6.boolean().optional().describe("Also list draft skills (default: false)"),
         scope: z6.enum(["projects", "groups"]).optional().describe("Scope (default: projects)"),
         group_id: z6.string().optional().describe("Group path (required when scope is groups)")
       },
       execute: async (args) => {
-        const auth = ctx.ensureAuth();
-        if (!auth) throw new Error("GitLab authentication not available");
+        const auth = authAndValidate(args.project_id);
         const { scope, id } = resolveScope2(args);
         try {
           const pages = await listWikiPages(auth.instanceUrl, auth.token, scope, id);
@@ -3671,14 +3745,15 @@ function makeSkillTools(ctx) {
     gitlab_skill_load: tool6({
       description: "Load a specific skill by name.\nSkills contain step-by-step instructions for common tasks.\nChecks published skills first, then falls back to draft skills.",
       args: {
-        project_id: z6.string().describe('Project path (e.g., "gitlab-org/gitlab"). Use the same value consistently.'),
+        project_id: z6.string().describe(
+          'FULL project path with namespace (e.g., "gitlab-org/gitlab"). Must contain a slash. Never use just the project name.'
+        ),
         name: z6.string().describe('Skill name (e.g., "incident-retro", "helm-rollback")'),
         scope: z6.enum(["projects", "groups"]).optional().describe("Scope (default: projects)"),
         group_id: z6.string().optional().describe("Group path (required when scope is groups)")
       },
       execute: async (args) => {
-        const auth = ctx.ensureAuth();
-        if (!auth) throw new Error("GitLab authentication not available");
+        const auth = authAndValidate(args.project_id);
         const { scope, id } = resolveScope2(args);
         try {
           const page = await getWikiPage(
@@ -3710,7 +3785,9 @@ ${draft.content}`;
     gitlab_skill_save: tool6({
       description: "Create or update a skill.\nSkills define step-by-step procedures for common tasks.\nUse draft=true for skills that haven't been proven yet.",
       args: {
-        project_id: z6.string().describe('Project path (e.g., "gitlab-org/gitlab"). Use the same value consistently.'),
+        project_id: z6.string().describe(
+          'FULL project path with namespace (e.g., "gitlab-org/gitlab"). Must contain a slash. Never use just the project name.'
+        ),
         name: z6.string().describe('Skill name (e.g., "incident-retro")'),
         content: z6.string().describe("Skill content in markdown"),
         draft: z6.boolean().optional().describe("Save as draft skill (default: false)"),
@@ -3718,35 +3795,44 @@ ${draft.content}`;
         group_id: z6.string().optional().describe("Group path (required when scope is groups)")
       },
       execute: async (args) => {
-        const auth = ctx.ensureAuth();
-        if (!auth) throw new Error("GitLab authentication not available");
+        const auth = authAndValidate(args.project_id);
         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({
       description: "Promote a draft skill to published.\nMoves the skill from the drafts directory to the published skills directory.",
       args: {
-        project_id: z6.string().describe('Project path (e.g., "gitlab-org/gitlab"). Use the same value consistently.'),
+        project_id: z6.string().describe(
+          'FULL project path with namespace (e.g., "gitlab-org/gitlab"). Must contain a slash. Never use just the project name.'
+        ),
         name: z6.string().describe("Skill name to promote"),
         scope: z6.enum(["projects", "groups"]).optional().describe("Scope (default: projects)"),
         group_id: z6.string().optional().describe("Group path (required when scope is groups)")
       },
       execute: async (args) => {
-        const auth = ctx.ensureAuth();
-        if (!auth) throw new Error("GitLab authentication not available");
+        const auth = authAndValidate(args.project_id);
         const { scope, id } = resolveScope2(args);
         const draftSlug = `${DRAFTS_PREFIX}/${args.name}`;
         const publishedSlug = `${SKILLS_PREFIX}/${args.name}`;