@probelabs/visor 0.1.176 → 0.1.177

package/dist/sdk/sdk.js

@@ -704,7 +704,7 @@ var require_package = __commonJS({
   "package.json"(exports2, module2) {
     module2.exports = {
       name: "@probelabs/visor",
-      version: "0.1.176",
+      version: "0.1.177",
       main: "dist/index.js",
       bin: {
         visor: "./dist/index.js"
@@ -823,7 +823,7 @@ var require_package = __commonJS({
         "@opentelemetry/sdk-node": "^0.203.0",
         "@opentelemetry/sdk-trace-base": "^1.30.1",
         "@opentelemetry/semantic-conventions": "^1.30.1",
-        "@probelabs/probe": "^0.6.0-rc291",
+        "@probelabs/probe": "^0.6.0-rc292",
         "@types/commander": "^2.12.0",
         "@types/uuid": "^10.0.0",
         acorn: "^8.16.0",
@@ -48791,11 +48791,26 @@ var init_worktree_manager = __esm({
         await this.deleteLocalBranches(worktreePath);
       }
       /**
-       * Delete all local branches in a worktree.
+       * Delete local branches in a worktree that are safe to remove.
        * Worktrees are always used in detached HEAD state, so any local branches
        * were unintentionally created and should be cleaned up.
+       * IMPORTANT: Git worktrees share the branch namespace with the main repo
+       * and all other worktrees. We must NOT delete branches that are checked out
+       * in the main working tree or any other worktree — doing so would destroy
+       * the user's work.
        */
       async deleteLocalBranches(worktreePath) {
+        const worktreeListCmd = `git -C ${this.escapeShellArg(worktreePath)} worktree list --porcelain`;
+        const worktreeListResult = await this.executeGitCommand(worktreeListCmd, { timeout: 1e4 });
+        const protectedBranches = /* @__PURE__ */ new Set();
+        if (worktreeListResult.exitCode === 0) {
+          for (const line of worktreeListResult.stdout.split("\n")) {
+            const match = line.match(/^branch refs\/heads\/(.+)$/);
+            if (match) {
+              protectedBranches.add(match[1]);
+            }
+          }
+        }
         const listCmd = `git -C ${this.escapeShellArg(worktreePath)} branch --list --format='%(refname:short)'`;
         const listResult = await this.executeGitCommand(listCmd, { timeout: 1e4 });
         if (listResult.exitCode !== 0 || !listResult.stdout.trim()) {
@@ -48803,6 +48818,10 @@ var init_worktree_manager = __esm({
         }
         const branches = listResult.stdout.trim().split("\n").map((b) => b.trim()).filter((b) => b.length > 0);
         for (const branch of branches) {
+          if (protectedBranches.has(branch)) {
+            logger.debug(`Skipping branch '${branch}' \u2014 checked out in another worktree`);
+            continue;
+          }
           const deleteCmd = `git -C ${this.escapeShellArg(worktreePath)} branch -D ${this.escapeShellArg(branch)}`;
           const deleteResult = await this.executeGitCommand(deleteCmd, { timeout: 1e4 });
           if (deleteResult.exitCode === 0) {
@@ -50307,7 +50326,7 @@ async function executeCheckWithForEachItems(checkId, forEachParent, forEachItems
         workflowInputs,
         ai: {
           ...checkConfig.ai || {},
-          timeout: checkConfig.ai?.timeout || 18e5,
+          timeout: checkConfig.timeout || checkConfig.ai?.timeout || 18e5,
           debug: !!context2.debug
         }
       };
@@ -50394,7 +50413,7 @@ async function executeCheckWithForEachItems(checkId, forEachParent, forEachItems
             context2,
             prInfo,
             dependencyResults,
-            checkConfig.ai?.timeout || 18e5,
+            checkConfig.timeout || checkConfig.ai?.timeout || 18e5,
             () => provider.execute(prInfo, providerConfig, dependencyResults, executionContext)
           );
           try {
@@ -50810,7 +50829,7 @@ async function executeInvocation(item, context2, scope, prInfo, dependencyResult
       __outputHistory: outputHistory,
       ai: {
         ...stepConfig.ai || {},
-        timeout: stepConfig.ai?.timeout || 18e5,
+        timeout: stepConfig.timeout || stepConfig.ai?.timeout || 18e5,
         debug: !!context2.debug
       }
     };
@@ -51334,7 +51353,7 @@ async function executeSingleCheck(checkId, context2, state, emitEvent, transitio
       workflowInputs,
       ai: {
         ...checkConfig.ai || {},
-        timeout: checkConfig.ai?.timeout || 18e5,
+        timeout: checkConfig.timeout || checkConfig.ai?.timeout || 18e5,
         debug: !!context2.debug
       }
     };
@@ -51435,7 +51454,7 @@ async function executeSingleCheck(checkId, context2, state, emitEvent, transitio
           context2,
           prInfo,
           dependencyResults,
-          checkConfig.ai?.timeout || 18e5,
+          checkConfig.timeout || checkConfig.ai?.timeout || 18e5,
           () => provider.execute(prInfo, providerConfig, dependencyResults, executionContext)
         );
         try {
@@ -52207,7 +52226,7 @@ async function executeCheckWithForEachItems2(checkId, forEachParent, forEachItem
         workflowInputs,
         ai: {
           ...checkConfig.ai || {},
-          timeout: checkConfig.ai?.timeout || 18e5,
+          timeout: checkConfig.timeout || checkConfig.ai?.timeout || 18e5,
           debug: !!context2.debug
         }
       };
@@ -52242,7 +52261,10 @@ async function executeCheckWithForEachItems2(checkId, forEachParent, forEachItem
                   `[LevelDispatch] Conversation extracted (${conv?.transport || "unknown"}): ${messageCount} messages`
                 );
               }
-              const transportCtx = slackConv ? { slack: { event: event || {}, conversation: slackConv } } : { telegram: { event: event || {}, conversation: telegramConv }, webhook: payload };
+              const transportCtx = slackConv ? { slack: { event: event || {}, conversation: slackConv } } : {
+                telegram: { event: event || {}, conversation: telegramConv },
+                webhook: payload
+              };
               providerConfig.eventContext = {
                 ...providerConfig.eventContext,
                 ...transportCtx,
@@ -52406,7 +52428,7 @@ async function executeCheckWithForEachItems2(checkId, forEachParent, forEachItem
             context2,
             prInfo,
             dependencyResults,
-            checkConfig.ai?.timeout || 18e5,
+            checkConfig.timeout || checkConfig.ai?.timeout || 18e5,
             () => provider.execute(prInfo, providerConfig, dependencyResults, executionContext)
           );
           try {
@@ -53335,7 +53357,7 @@ async function executeSingleCheck2(checkId, context2, state, emitEvent, transiti
       workflowInputs,
       ai: {
         ...checkConfig2.ai || {},
-        timeout: checkConfig2.ai?.timeout || 18e5,
+        timeout: checkConfig2.timeout || checkConfig2.ai?.timeout || 18e5,
         debug: !!context2.debug
       }
     };
@@ -53366,7 +53388,9 @@ async function executeSingleCheck2(checkId, context2, state, emitEvent, transiti
             const event = payload?.event;
             const messageCount = Array.isArray(conv?.messages) ? conv.messages.length : 0;
             if (context2.debug) {
-              logger.info(`[LevelDispatch] Conversation extracted (${conv?.transport || "unknown"}): ${messageCount} messages`);
+              logger.info(
+                `[LevelDispatch] Conversation extracted (${conv?.transport || "unknown"}): ${messageCount} messages`
+              );
             }
             const transportCtx = slackConv ? { slack: { event: event || {}, conversation: slackConv } } : { telegram: { event: event || {}, conversation: telegramConv }, webhook: payload };
             providerConfig.eventContext = {
@@ -53504,7 +53528,7 @@ async function executeSingleCheck2(checkId, context2, state, emitEvent, transiti
           context2,
           prInfo,
           dependencyResults,
-          checkConfig2.ai?.timeout || 18e5,
+          checkConfig2.timeout || checkConfig2.ai?.timeout || 18e5,
           () => provider.execute(prInfo, providerConfig, dependencyResults, executionContext)
         );
         try {
@@ -56663,10 +56687,27 @@ var init_workspace_manager = __esm({
         await this.deleteLocalBranches(worktreePath);
       }
       /**
-       * Delete all local branches in a worktree.
+       * Delete local branches in a worktree that are safe to remove.
+       * IMPORTANT: Git worktrees share the branch namespace with the main repo
+       * and all other worktrees. We must NOT delete branches that are checked out
+       * in the main working tree or any other worktree — doing so would destroy
+       * the user's work.
        */
       async deleteLocalBranches(worktreePath) {
         const escapedPath = shellEscape(worktreePath);
+        const worktreeListResult = await commandExecutor.execute(
+          `git -C ${escapedPath} worktree list --porcelain`,
+          { timeout: 1e4 }
+        );
+        const protectedBranches = /* @__PURE__ */ new Set();
+        if (worktreeListResult.exitCode === 0) {
+          for (const line of worktreeListResult.stdout.split("\n")) {
+            const match = line.match(/^branch refs\/heads\/(.+)$/);
+            if (match) {
+              protectedBranches.add(match[1]);
+            }
+          }
+        }
         const listResult = await commandExecutor.execute(
           `git -C ${escapedPath} branch --list --format='%(refname:short)'`,
           { timeout: 1e4 }
@@ -56676,6 +56717,10 @@ var init_workspace_manager = __esm({
         }
         const branches = listResult.stdout.trim().split("\n").map((b) => b.trim()).filter((b) => b.length > 0);
         for (const branch of branches) {
+          if (protectedBranches.has(branch)) {
+            logger.debug(`[Workspace] Skipping branch '${branch}' \u2014 checked out in another worktree`);
+            continue;
+          }
           const deleteResult = await commandExecutor.execute(
             `git -C ${escapedPath} branch -D ${shellEscape(branch)}`,
             { timeout: 1e4 }
@@ -59435,6 +59480,21 @@ ${message}`;
           if (out && typeof out._rawOutput === "string" && out._rawOutput.trim().length > 0) {
             text = (text || "") + "\n\n" + out._rawOutput.trim();
           }
+          if (!text) {
+            const issues = result?.issues || [];
+            const errorIssues = issues.filter(
+              (i) => i.severity === "error" && (i.ruleId?.startsWith("system/") || i.ruleId?.endsWith("/error"))
+            );
+            if (errorIssues.length > 0) {
+              const errorMessages = errorIssues.map((i) => i.message).join("\n");
+              text = `:warning: Something went wrong while processing your request:
+${errorMessages}`;
+              this.errorNotified = true;
+              ctx.logger.warn(
+                `[slack-frontend] posting error fallback for ${checkId}: ${errorIssues.length} system error(s)`
+              );
+            }
+          }
           if (!text) {
             ctx.logger.info(
               `[slack-frontend] skip posting AI reply for ${checkId}: no renderable text in check output`