@probelabs/probe 0.6.0-rc263 → 0.6.0-rc265

package/build/agent/index.js

@@ -9553,6 +9553,11 @@ Example: <extract><targets>${displayPath}</targets></extract>`;
             if (!replace_all && occurrences > 1) {
               return `Error editing file: Multiple occurrences found - the old_string appears ${occurrences} times in ${file_path}. To fix: (1) Set replace_all=true to replace all occurrences, or (2) Include more surrounding lines in old_string to make the match unique (add the full line or adjacent lines for context).`;
             }
+            const oldLines = matchTarget.split("\n").length;
+            const newLines = new_string.split("\n").length;
+            if (oldLines >= 20 && newLines < oldLines * 0.5) {
+              return `Error editing file: Edit scope too large \u2014 replacing ${oldLines} lines with ${newLines} lines risks accidental content deletion. To fix: (1) Use line-targeted editing (start_line/end_line) instead to constrain scope, or (2) Split into smaller, focused edits that each target only the lines you intend to change.`;
+            }
             let newContent;
             if (replace_all) {
               newContent = content.replaceAll(matchTarget, new_string);
@@ -12935,6 +12940,122 @@ var init_bashPermissions = __esm({
 import { spawn as spawn2 } from "child_process";
 import { resolve as resolve3, join } from "path";
 import { existsSync as existsSync2 } from "fs";
+function splitCommandComponents(command) {
+  const parts = [];
+  let current2 = "";
+  let inQuote = false;
+  let quoteChar = "";
+  for (let i = 0; i < command.length; i++) {
+    const c = command[i];
+    const next = command[i + 1] || "";
+    if (c === "\\" && !inQuote) {
+      current2 += c + next;
+      i++;
+      continue;
+    }
+    if (inQuote && quoteChar === '"' && c === "\\" && next) {
+      current2 += c + next;
+      i++;
+      continue;
+    }
+    if (!inQuote && (c === '"' || c === "'")) {
+      inQuote = true;
+      quoteChar = c;
+      current2 += c;
+      continue;
+    }
+    if (inQuote && c === quoteChar) {
+      inQuote = false;
+      current2 += c;
+      continue;
+    }
+    if (!inQuote) {
+      if (c === "&" && next === "&" || c === "|" && next === "|") {
+        if (current2.trim()) parts.push(current2.trim());
+        current2 = "";
+        i++;
+        continue;
+      }
+      if (c === "|" || c === ";") {
+        if (current2.trim()) parts.push(current2.trim());
+        current2 = "";
+        continue;
+      }
+    }
+    current2 += c;
+  }
+  if (current2.trim()) parts.push(current2.trim());
+  return parts;
+}
+function checkSingleCommandInteractive(command) {
+  let effective = command.trim();
+  while (/^\w+=\S*\s/.test(effective)) {
+    effective = effective.replace(/^\w+=\S*\s+/, "");
+  }
+  const parts = effective.split(/\s+/);
+  const base2 = parts[0];
+  const args = parts.slice(1);
+  if (["vi", "vim", "nvim", "nano", "emacs", "pico", "joe", "mcedit"].includes(base2)) {
+    return `'${base2}' is an interactive editor and cannot run without a terminal. Use non-interactive file manipulation commands instead.`;
+  }
+  if (["less", "more"].includes(base2)) {
+    return `'${base2}' is an interactive pager. Use 'cat', 'head', or 'tail' instead.`;
+  }
+  if (base2 === "git") {
+    const sub = args[0];
+    if (sub === "commit") {
+      const hasNonInteractiveFlag = args.some(
+        (a) => a === "-m" || a.startsWith("--message") || a === "-C" || a === "-c" || a.startsWith("--fixup") || a.startsWith("--squash") || a === "--allow-empty-message" || a === "--no-edit"
+      );
+      if (!hasNonInteractiveFlag) {
+        return `Interactive command: 'git commit' opens an editor for the commit message. Use 'git commit -m "your message"' instead.`;
+      }
+    }
+    if (sub === "rebase" && (args.includes("--continue") || args.includes("--skip"))) {
+      return "Interactive command: 'git rebase --continue' opens an editor. Set environment variable GIT_EDITOR=true to accept default messages, e.g. pass env: {GIT_EDITOR: 'true'} or prepend GIT_EDITOR=true to the command.";
+    }
+    if (sub === "rebase" && (args.includes("-i") || args.includes("--interactive"))) {
+      return "Interactive command: 'git rebase -i' requires an interactive editor. Interactive rebase cannot run without a terminal.";
+    }
+    if (sub === "merge" && !args.includes("--no-edit") && !args.includes("--no-commit") && !args.includes("--ff-only")) {
+      return "Interactive command: 'git merge' may open an editor for the merge commit message. Add '--no-edit' to accept the default message.";
+    }
+    if (sub === "cherry-pick" && !args.includes("--no-edit")) {
+      return "Interactive command: 'git cherry-pick' may open an editor. Add '--no-edit' to accept the default message.";
+    }
+    if (sub === "revert" && !args.includes("--no-edit")) {
+      return "Interactive command: 'git revert' opens an editor. Add '--no-edit' to accept the default message.";
+    }
+    if (sub === "tag" && args.includes("-a") && !args.some((a) => a === "-m" || a.startsWith("--message"))) {
+      return `Interactive command: 'git tag -a' opens an editor for the tag message. Use 'git tag -a <name> -m "message"' instead.`;
+    }
+    if (sub === "add" && (args.includes("-i") || args.includes("--interactive") || args.includes("-p") || args.includes("--patch"))) {
+      return "Interactive command: 'git add -i/-p' requires interactive input. Use 'git add <files>' to stage specific files instead.";
+    }
+  }
+  if (["python", "python3", "node", "irb", "ghci", "lua", "R", "ruby"].includes(base2) && args.length === 0) {
+    return `Interactive command: '${base2}' without arguments starts an interactive REPL. Provide a script file or use '-c'/'--eval' for inline code.`;
+  }
+  if (base2 === "mysql" && !args.some((a) => a === "-e" || a.startsWith("--execute"))) {
+    return `Interactive command: 'mysql' without -e flag starts an interactive session. Use 'mysql -e "SQL QUERY"' instead.`;
+  }
+  if (base2 === "psql" && !args.some((a) => a === "-c" || a.startsWith("--command") || a === "-f" || a.startsWith("--file"))) {
+    return `Interactive command: 'psql' without -c flag starts an interactive session. Use 'psql -c "SQL QUERY"' instead.`;
+  }
+  if (["top", "htop", "btop", "nmon"].includes(base2)) {
+    return `Interactive command: '${base2}' is an interactive TUI tool. Use 'ps aux' or 'top -b -n 1' for non-interactive process listing.`;
+  }
+  return null;
+}
+function checkInteractiveCommand(command) {
+  if (!command || typeof command !== "string") return null;
+  const components = splitCommandComponents(command.trim());
+  for (const component of components) {
+    const result = checkSingleCommandInteractive(component);
+    if (result) return result;
+  }
+  return null;
+}
 async function executeBashCommand(command, options = {}) {
   const {
     workingDirectory = process.cwd(),
@@ -12964,6 +13085,24 @@ async function executeBashCommand(command, options = {}) {
     };
   }
   const startTime = Date.now();
+  const interactiveError = checkInteractiveCommand(command);
+  if (interactiveError) {
+    if (debug) {
+      console.log(`[BashExecutor] Blocked interactive command: "${command}"`);
+      console.log(`[BashExecutor] Reason: ${interactiveError}`);
+    }
+    return {
+      success: false,
+      error: interactiveError,
+      stdout: "",
+      stderr: interactiveError,
+      exitCode: 1,
+      command,
+      workingDirectory: cwd,
+      duration: 0,
+      interactive: true
+    };
+  }
   if (debug) {
     console.log(`[BashExecutor] Executing command: "${command}"`);
     console.log(`[BashExecutor] Working directory: "${cwd}"`);
@@ -12974,6 +13113,8 @@ async function executeBashCommand(command, options = {}) {
       ...process.env,
       ...env
     };
+    if (!processEnv.GIT_EDITOR) processEnv.GIT_EDITOR = "true";
+    if (!processEnv.GIT_TERMINAL_PROMPT) processEnv.GIT_TERMINAL_PROMPT = "0";
     const isComplex = isComplexCommand(command);
     let cmd, cmdArgs, useShell;
     if (isComplex) {
@@ -13008,20 +13149,32 @@ async function executeBashCommand(command, options = {}) {
       // stdin ignored, capture stdout/stderr
       shell: useShell,
       // false for security
+      detached: true,
+      // new session — no controlling terminal
       windowsHide: true
     });
     let stdout = "";
     let stderr = "";
     let killed = false;
     let timeoutHandle;
+    const killProcessGroup = (signal) => {
+      try {
+        if (child.pid) process.kill(-child.pid, signal);
+      } catch {
+        try {
+          child.kill(signal);
+        } catch {
+        }
+      }
+    };
     if (timeout > 0) {
       timeoutHandle = setTimeout(() => {
         if (!killed) {
           killed = true;
-          child.kill("SIGTERM");
+          killProcessGroup("SIGTERM");
           setTimeout(() => {
             if (child.exitCode === null) {
-              child.kill("SIGKILL");
+              killProcessGroup("SIGKILL");
             }
           }, 5e3);
         }
@@ -13034,7 +13187,7 @@ async function executeBashCommand(command, options = {}) {
       } else {
         if (!killed) {
           killed = true;
-          child.kill("SIGTERM");
+          killProcessGroup("SIGTERM");
         }
       }
     });
@@ -13045,7 +13198,7 @@ async function executeBashCommand(command, options = {}) {
       } else {
         if (!killed) {
           killed = true;
-          child.kill("SIGTERM");
+          killProcessGroup("SIGTERM");
         }
       }
     });
@@ -39381,7 +39534,6 @@ var init_reg_exp = __esm({
 // node_modules/chevrotain/lib/src/scan/lexer.js
 function analyzeTokenTypes(tokenTypes, options) {
   options = defaults_default(options, {
-    useSticky: SUPPORT_STICKY,
     debug: false,
     safeMode: false,
     positionTracking: "full",
@@ -39430,7 +39582,7 @@ function analyzeTokenTypes(tokenTypes, options) {
         ], regExpSource[1])) {
           return regExpSource[1];
         } else {
-          return options.useSticky ? addStickyFlag(currPattern) : addStartOfInput(currPattern);
+          return addStickyFlag(currPattern);
         }
       } else if (isFunction_default(currPattern)) {
         hasCustom = true;
@@ -39444,7 +39596,7 @@ function analyzeTokenTypes(tokenTypes, options) {
         } else {
           const escapedRegExpString = currPattern.replace(/[\\^$.*+?()[\]{}|]/g, "\\$&");
           const wrappedRegExp = new RegExp(escapedRegExpString);
-          return options.useSticky ? addStickyFlag(wrappedRegExp) : addStartOfInput(wrappedRegExp);
+          return addStickyFlag(wrappedRegExp);
         }
       } else {
         throw Error("non exhaustive match");
@@ -39848,10 +40000,6 @@ function noMetaChar(regExp) {
 function usesLookAheadOrBehind(regExp) {
   return /(\(\?=)|(\(\?!)|(\(\?<=)|(\(\?<!)/.test(regExp.source);
 }
-function addStartOfInput(pattern) {
-  const flags = pattern.ignoreCase ? "i" : "";
-  return new RegExp(`^(?:${pattern.source})`, flags);
-}
 function addStickyFlag(pattern) {
   const flags = pattern.ignoreCase ? "iy" : "y";
   return new RegExp(`${pattern.source}`, flags);
@@ -40040,7 +40188,7 @@ function initCharCodeToOptimizedIndexMap() {
     }
   }
 }
-var PATTERN, DEFAULT_MODE, MODES, SUPPORT_STICKY, end_of_input, start_of_input, LineTerminatorOptimizedTester, minOptimizationVal, charCodeToOptimizedIdxMap;
+var PATTERN, DEFAULT_MODE, MODES, end_of_input, start_of_input, LineTerminatorOptimizedTester, minOptimizationVal, charCodeToOptimizedIdxMap;
 var init_lexer = __esm({
   "node_modules/chevrotain/lib/src/scan/lexer.js"() {
     init_api3();
@@ -40052,7 +40200,6 @@ var init_lexer = __esm({
     PATTERN = "PATTERN";
     DEFAULT_MODE = "defaultMode";
     MODES = "modes";
-    SUPPORT_STICKY = typeof new RegExp("(?:)").sticky === "boolean";
     end_of_input = /[^\\][$]/;
     start_of_input = /[^\\[][\^]|^\^/;
     LineTerminatorOptimizedTester = {
@@ -40368,13 +40515,6 @@ var init_lexer_public = __esm({
             PRINT_WARNING(warningDescriptor.message);
           });
           this.TRACE_INIT("Choosing sub-methods implementations", () => {
-            if (SUPPORT_STICKY) {
-              this.chopInput = identity_default;
-              this.match = this.matchWithTest;
-            } else {
-              this.updateLastIndex = noop_default;
-              this.match = this.matchWithExec;
-            }
             if (hasOnlySingleMode) {
               this.handleModes = noop_default;
             }
@@ -40437,7 +40577,7 @@ var init_lexer_public = __esm({
       // this method also used quite a bit of `!` none null assertions because it is too optimized
       // for `tsc` to always understand it is "safe"
       tokenizeInternal(text, initialMode) {
-        let i, j, k, matchAltImage, longerAlt, matchedImage, payload, altPayload, imageLength, group, tokType, newToken, errLength, droppedChar, msg, match2;
+        let i, j, k, matchAltImage, longerAlt, matchedImage, payload, altPayload, imageLength, group, tokType, newToken, errLength, msg, match2;
         const orgText = text;
         const orgLength = orgText.length;
         let offset2 = 0;
@@ -40456,19 +40596,7 @@ var init_lexer_public = __esm({
         const modeStack = [];
         const emptyArray = [];
         Object.freeze(emptyArray);
-        let getPossiblePatterns;
-        function getPossiblePatternsSlow() {
-          return patternIdxToConfig;
-        }
-        function getPossiblePatternsOptimized(charCode) {
-          const optimizedCharIdx = charCodeToOptimizedIndex(charCode);
-          const possiblePatterns = currCharCodeToPatternIdxToConfig[optimizedCharIdx];
-          if (possiblePatterns === void 0) {
-            return emptyArray;
-          } else {
-            return possiblePatterns;
-          }
-        }
+        let isOptimizedMode = false;
         const pop_mode = (popToken) => {
           if (modeStack.length === 1 && // if we have both a POP_MODE and a PUSH_MODE this is in-fact a "transition"
           // So no error should occur.
@@ -40489,9 +40617,9 @@ var init_lexer_public = __esm({
             currModePatternsLength = patternIdxToConfig.length;
             const modeCanBeOptimized = this.canModeBeOptimized[newMode] && this.config.safeMode === false;
             if (currCharCodeToPatternIdxToConfig && modeCanBeOptimized) {
-              getPossiblePatterns = getPossiblePatternsOptimized;
+              isOptimizedMode = true;
             } else {
-              getPossiblePatterns = getPossiblePatternsSlow;
+              isOptimizedMode = false;
             }
           }
         };
@@ -40503,9 +40631,9 @@ var init_lexer_public = __esm({
           currModePatternsLength = patternIdxToConfig.length;
           const modeCanBeOptimized = this.canModeBeOptimized[newMode] && this.config.safeMode === false;
           if (currCharCodeToPatternIdxToConfig && modeCanBeOptimized) {
-            getPossiblePatterns = getPossiblePatternsOptimized;
+            isOptimizedMode = true;
           } else {
-            getPossiblePatterns = getPossiblePatternsSlow;
+            isOptimizedMode = false;
           }
         }
         push_mode.call(this, initialMode);
@@ -40513,8 +40641,16 @@ var init_lexer_public = __esm({
         const recoveryEnabled = this.config.recoveryEnabled;
         while (offset2 < orgLength) {
           matchedImage = null;
+          imageLength = -1;
           const nextCharCode = orgText.charCodeAt(offset2);
-          const chosenPatternIdxToConfig = getPossiblePatterns(nextCharCode);
+          let chosenPatternIdxToConfig;
+          if (isOptimizedMode) {
+            const optimizedCharIdx = charCodeToOptimizedIndex(nextCharCode);
+            const possiblePatterns = currCharCodeToPatternIdxToConfig[optimizedCharIdx];
+            chosenPatternIdxToConfig = possiblePatterns !== void 0 ? possiblePatterns : emptyArray;
+          } else {
+            chosenPatternIdxToConfig = patternIdxToConfig;
+          }
           const chosenPatternsLength = chosenPatternIdxToConfig.length;
           for (i = 0; i < chosenPatternsLength; i++) {
             currConfig = chosenPatternIdxToConfig[i];
@@ -40523,12 +40659,14 @@ var init_lexer_public = __esm({
             const singleCharCode = currConfig.short;
             if (singleCharCode !== false) {
               if (nextCharCode === singleCharCode) {
+                imageLength = 1;
                 matchedImage = currPattern;
               }
             } else if (currConfig.isCustom === true) {
               match2 = currPattern.exec(orgText, offset2, matchedTokens, groups);
               if (match2 !== null) {
                 matchedImage = match2[0];
+                imageLength = matchedImage.length;
                 if (match2.payload !== void 0) {
                   payload = match2.payload;
                 }
@@ -40536,12 +40674,13 @@ var init_lexer_public = __esm({
                 matchedImage = null;
               }
             } else {
-              this.updateLastIndex(currPattern, offset2);
-              matchedImage = this.match(currPattern, text, offset2);
+              currPattern.lastIndex = offset2;
+              imageLength = this.matchLength(currPattern, text, offset2);
             }
-            if (matchedImage !== null) {
+            if (imageLength !== -1) {
               longerAlt = currConfig.longerAlt;
               if (longerAlt !== void 0) {
+                matchedImage = text.substring(offset2, offset2 + imageLength);
                 const longerAltLength = longerAlt.length;
                 for (k = 0; k < longerAltLength; k++) {
                   const longerAltConfig = patternIdxToConfig[longerAlt[k]];
@@ -40558,11 +40697,12 @@ var init_lexer_public = __esm({
                       matchAltImage = null;
                     }
                   } else {
-                    this.updateLastIndex(longerAltPattern, offset2);
+                    longerAltPattern.lastIndex = offset2;
                     matchAltImage = this.match(longerAltPattern, text, offset2);
                   }
                   if (matchAltImage && matchAltImage.length > matchedImage.length) {
                     matchedImage = matchAltImage;
+                    imageLength = matchAltImage.length;
                     payload = altPayload;
                     currConfig = longerAltConfig;
                     break;
@@ -40572,10 +40712,10 @@ var init_lexer_public = __esm({
               break;
             }
           }
-          if (matchedImage !== null) {
-            imageLength = matchedImage.length;
+          if (imageLength !== -1) {
             group = currConfig.group;
             if (group !== void 0) {
+              matchedImage = matchedImage !== null ? matchedImage : text.substring(offset2, offset2 + imageLength);
               tokType = currConfig.tokenTypeIdx;
               newToken = this.createTokenInstance(matchedImage, offset2, tokType, currConfig.tokenType, line, column, imageLength);
               this.handlePayload(newToken, payload);
@@ -40585,15 +40725,13 @@ var init_lexer_public = __esm({
                 groups[group].push(newToken);
               }
             }
-            text = this.chopInput(text, imageLength);
-            offset2 = offset2 + imageLength;
-            column = this.computeNewColumn(column, imageLength);
             if (trackLines === true && currConfig.canLineTerminator === true) {
               let numOfLTsInMatch = 0;
               let foundTerminator;
               let lastLTEndOffset;
               lineTerminatorPattern.lastIndex = 0;
               do {
+                matchedImage = matchedImage !== null ? matchedImage : text.substring(offset2, offset2 + imageLength);
                 foundTerminator = lineTerminatorPattern.test(matchedImage);
                 if (foundTerminator === true) {
                   lastLTEndOffset = lineTerminatorPattern.lastIndex - 1;
@@ -40604,8 +40742,13 @@ var init_lexer_public = __esm({
                 line = line + numOfLTsInMatch;
                 column = imageLength - lastLTEndOffset;
                 this.updateTokenEndLineColumnLocation(newToken, group, lastLTEndOffset, numOfLTsInMatch, line, column, imageLength);
+              } else {
+                column = this.computeNewColumn(column, imageLength);
               }
+            } else {
+              column = this.computeNewColumn(column, imageLength);
             }
+            offset2 = offset2 + imageLength;
             this.handleModes(currConfig, pop_mode, push_mode, newToken);
           } else {
             const errorStartOffset = offset2;
@@ -40613,7 +40756,6 @@ var init_lexer_public = __esm({
             const errorColumn = column;
             let foundResyncPoint = recoveryEnabled === false;
             while (foundResyncPoint === false && offset2 < orgLength) {
-              text = this.chopInput(text, 1);
               offset2++;
               for (j = 0; j < currModePatternsLength; j++) {
                 const currConfig2 = patternIdxToConfig[j];
@@ -40626,7 +40768,7 @@ var init_lexer_public = __esm({
                 } else if (currConfig2.isCustom === true) {
                   foundResyncPoint = currPattern.exec(orgText, offset2, matchedTokens, groups) !== null;
                 } else {
-                  this.updateLastIndex(currPattern, offset2);
+                  currPattern.lastIndex = offset2;
                   foundResyncPoint = currPattern.exec(text) !== null;
                 }
                 if (foundResyncPoint === true) {
@@ -40669,12 +40811,6 @@ var init_lexer_public = __esm({
           push_mode.call(this, config.push);
         }
       }
-      chopInput(text, length) {
-        return text.substring(length);
-      }
-      updateLastIndex(regExp, newLastIndex) {
-        regExp.lastIndex = newLastIndex;
-      }
       // TODO: decrease this under 600 characters? inspect stripping comments option in TSC compiler
       updateTokenEndLineColumnLocation(newToken, group, lastLTIdx, numOfLTsInMatch, line, column, imageLength) {
         let lastCharIsLT, fixForEndingInLT;
@@ -40737,16 +40873,19 @@ var init_lexer_public = __esm({
           token.payload = payload;
         }
       }
-      matchWithTest(pattern, text, offset2) {
+      match(pattern, text, offset2) {
         const found = pattern.test(text);
         if (found === true) {
           return text.substring(offset2, pattern.lastIndex);
         }
         return null;
       }
-      matchWithExec(pattern, text) {
-        const regExpArray = pattern.exec(text);
-        return regExpArray !== null ? regExpArray[0] : null;
+      matchLength(pattern, text, offset2) {
+        const found = pattern.test(text);
+        if (found === true) {
+          return pattern.lastIndex - offset2;
+        }
+        return -1;
       }
     };
     Lexer.SKIPPED = "This marks a skipped Token pattern, this means each token identified by it will be consumed and then thrown into oblivion, this can be used to for example to completely ignore whitespace.";
@@ -40938,12 +41077,20 @@ For Further details.`;
         return errMsg;
       },
       buildAlternationAmbiguityError(options) {
-        const pathMsg = map_default(options.prefixPath, (currtok) => tokenLabel2(currtok)).join(", ");
         const occurrence = options.alternation.idx === 0 ? "" : options.alternation.idx;
+        const isEmptyPath = options.prefixPath.length === 0;
         let currMessage = `Ambiguous Alternatives Detected: <${options.ambiguityIndices.join(" ,")}> in <OR${occurrence}> inside <${options.topLevelRule.name}> Rule,
-<${pathMsg}> may appears as a prefix path in all these alternatives.
 `;
-        currMessage = currMessage + `See: https://chevrotain.io/docs/guide/resolving_grammar_errors.html#AMBIGUOUS_ALTERNATIVES
+        if (isEmptyPath) {
+          currMessage += `These alternatives are all empty (match no tokens), making them indistinguishable.
+Only the last alternative may be empty.
+`;
+        } else {
+          const pathMsg = map_default(options.prefixPath, (currtok) => tokenLabel2(currtok)).join(", ");
+          currMessage += `<${pathMsg}> may appears as a prefix path in all these alternatives.
+`;
+        }
+        currMessage += `See: https://chevrotain.io/docs/guide/resolving_grammar_errors.html#AMBIGUOUS_ALTERNATIVES
 For Further details.`;
         return currMessage;
       },
@@ -47446,7 +47593,7 @@ function validateFlowchart(text, options = {}) {
         const byLine = /* @__PURE__ */ new Map();
         const collect = (arr) => {
           for (const e of arr || []) {
-            if (e && (e.code === "FL-LABEL-PARENS-UNQUOTED" || e.code === "FL-LABEL-AT-IN-UNQUOTED" || e.code === "FL-LABEL-QUOTE-IN-UNQUOTED")) {
+            if (e && (e.code === "FL-LABEL-PARENS-UNQUOTED" || e.code === "FL-LABEL-AT-IN-UNQUOTED" || e.code === "FL-LABEL-QUOTE-IN-UNQUOTED" || e.code === "FL-LABEL-SLASH-UNQUOTED")) {
               const ln = e.line ?? 0;
               const col = e.column ?? 1;
               const list = byLine.get(ln) || [];
@@ -47529,6 +47676,7 @@ function validateFlowchart(text, options = {}) {
                 const hasAt = seg.includes("@");
                 const hasQuote = seg.includes('"');
                 const isSingleQuoted = /^'[^]*'$/.test(trimmed);
+                const hasLeadingSlash = lsp === "/" || lsp === "\\";
                 if (!covered && !isQuoted && !isParenWrapped && hasParens) {
                   errs.push({ line: ln, column: startCol, severity: "error", code: "FL-LABEL-PARENS-UNQUOTED", message: "Parentheses inside an unquoted label are not supported by Mermaid.", hint: 'Wrap the label in quotes, e.g., A["Mark (X)"] \u2014 or replace ( and ) with HTML entities: &#40; and &#41;.' });
                   existing.push({ start: startCol, end: endCol });
@@ -47539,6 +47687,11 @@ function validateFlowchart(text, options = {}) {
                   existing.push({ start: startCol, end: endCol });
                   byLine.set(ln, existing);
                 }
+                if (!covered && !isQuoted && !isSlashPair && hasLeadingSlash) {
+                  errs.push({ line: ln, column: startCol, severity: "error", code: "FL-LABEL-SLASH-UNQUOTED", message: "Leading / or \\ inside an unquoted label is treated as a shape marker by Mermaid.", hint: 'Wrap the label in quotes, e.g., F["/dev/tty unavailable"], or use &#47; / &#92; for literal slashes.' });
+                  existing.push({ start: startCol, end: endCol });
+                  byLine.set(ln, existing);
+                }
                 if (!covered && !isQuoted && !isSlashPair && hasQuote && !isSingleQuoted) {
                   errs.push({ line: ln, column: startCol, severity: "error", code: "FL-LABEL-QUOTE-IN-UNQUOTED", message: "Quotes are not allowed inside unquoted node labels. Use &quot; for quotes or wrap the entire label in quotes.", hint: 'Example: C["HTML Output: data-trigger-visibility=&quot;true&quot;"]' });
                   existing.push({ start: startCol, end: endCol });
@@ -50547,7 +50700,7 @@ function computeFixes(text, errors, level = "safe") {
       }
       continue;
     }
-    if (is("FL-LABEL-PARENS-UNQUOTED", e) || is("FL-LABEL-AT-IN-UNQUOTED", e)) {
+    if (is("FL-LABEL-PARENS-UNQUOTED", e) || is("FL-LABEL-AT-IN-UNQUOTED", e) || is("FL-LABEL-SLASH-UNQUOTED", e)) {
       if (level === "safe" || level === "all") {
         if (patchedLines.has(e.line))
           continue;
@@ -71842,6 +71995,7 @@ If the solution is clear, you can jump to implementation right away. If not, ask
 - Avoid implementing special cases when a general approach works
 - Never expose secrets, API keys, or credentials in generated code. Never log sensitive information.
 - Do not surprise the user with unrequested changes. Do what was asked, including reasonable follow-up actions, but do not refactor surrounding code or add features that were not requested.
+- When editing files, keep edits focused and minimal. For changes spanning more than a few lines, prefer line-targeted editing (start_line/end_line) over text replacement (old_string) \u2014 it constrains scope and prevents accidental removal of adjacent content. Never include unrelated sections in an edit operation.
 - After every significant change, verify the project still builds and passes linting. Do not wait until the end to discover breakage.
 
 # After Implementation
@@ -71852,11 +72006,33 @@ If the solution is clear, you can jump to implementation right away. If not, ask
 
 # GitHub Integration
 - Use the \`gh\` CLI for all GitHub operations: issues, pull requests, checks, releases.
-- To create a pull request: commit your changes, push the branch, then use \`gh pr create --title "..." --body "..."\`.
 - To view issues or PRs: \`gh issue view <number>\`, \`gh pr view <number>\`.
 - If given a GitHub URL, use \`gh\` to fetch the relevant information rather than guessing.
 - Always return the pull request URL to the user after creating one.
-- When checking GitHub Actions, only read logs of failed jobs \u2014 do not waste time on successful ones. Use \`gh run view <run-id> --log-failed\` to fetch only the relevant output.`,
+- When checking GitHub Actions, only read logs of failed jobs \u2014 do not waste time on successful ones. Use \`gh run view <run-id> --log-failed\` to fetch only the relevant output.
+
+# Pull Request Creation
+- Commit your changes, push the branch, then use \`gh pr create --title "..." --body "..."\`.
+- **PR title**: Keep it short (under 72 characters). Use imperative mood describing the change (e.g. "Add retry logic for API calls", "Fix race condition in cache invalidation"). Prefix with the type of change when useful: \`fix:\`, \`feat:\`, \`refactor:\`, \`docs:\`, \`test:\`, \`chore:\`.
+- **PR body**: MUST follow this structure:
+
+\`\`\`
+## Problem / Task
+<What problem is being solved or what task was requested. If there is a linked issue, reference it with #number. Be specific about the root cause or motivation.>
+
+## Changes
+<Concise list of what was actually changed. Describe each meaningful change \u2014 files modified, logic added/removed, and why. Do NOT just list filenames; explain what each change does.>
+
+## Testing
+<What tests were added, modified, or run. Include:
+- New test names and what they verify
+- Whether existing tests still pass
+- Manual verification steps if applicable
+- Commands used to validate (e.g. \`make test\`, \`npm test\`)>
+\`\`\`
+
+- If the task originated from a GitHub issue, always reference it in the PR body (e.g. "Fixes #123" or "Closes #123") so the issue is automatically closed on merge. If it originated from an external ticket system (Jira, Linear, etc.), include the ticket ID and link in the Problem / Task section (e.g. "Resolves PROJ-456").
+- Do not leave the PR body empty or vague. Every PR must clearly communicate what was done and why so reviewers can understand the change without reading every line of diff.`,
       "support": `You are ProbeChat Support, a specialized AI assistant focused on helping developers troubleshoot issues and solve problems. Your primary function is to help users diagnose errors, understand unexpected behaviors, and find solutions using the provided code analysis tools.
 
 When troubleshooting:
@@ -85122,6 +85298,7 @@ Follow these instructions carefully:
       * For rewriting entire functions/classes/methods, use the symbol parameter instead (no exact text matching needed).
       * For editing specific lines from search/extract output, use start_line (and optionally end_line) with the line numbers shown in the output.${this.hashLines ? ' Line references include content hashes (e.g. "42:ab") for integrity verification.' : ""}
       * For editing inside large functions: first use extract with the symbol target (e.g. "file.js#myFunction") to see the function with line numbers${this.hashLines ? " and hashes" : ""}, then use start_line/end_line to surgically edit specific lines within it.
+      * IMPORTANT: Keep old_string as small as possible \u2014 include only the lines you need to change plus minimal context for uniqueness. For replacing large blocks (10+ lines), prefer line-targeted editing with start_line/end_line to constrain scope.
     - Use 'create' for new files or complete file rewrites.
     - If an edit fails, read the error message \u2014 it tells you exactly how to fix the call and retry.
     - The system tracks which files you've seen via search/extract. If you try to edit a file you haven't read, or one that changed since you last read it, the edit will fail with instructions to re-read first. Always use extract before editing to ensure you have current file content.` : ""}

package/build/agent/shared/prompts.js

@@ -87,6 +87,7 @@ If the solution is clear, you can jump to implementation right away. If not, ask
 - Avoid implementing special cases when a general approach works
 - Never expose secrets, API keys, or credentials in generated code. Never log sensitive information.
 - Do not surprise the user with unrequested changes. Do what was asked, including reasonable follow-up actions, but do not refactor surrounding code or add features that were not requested.
+- When editing files, keep edits focused and minimal. For changes spanning more than a few lines, prefer line-targeted editing (start_line/end_line) over text replacement (old_string) — it constrains scope and prevents accidental removal of adjacent content. Never include unrelated sections in an edit operation.
 - After every significant change, verify the project still builds and passes linting. Do not wait until the end to discover breakage.
 
 # After Implementation
@@ -97,11 +98,33 @@ If the solution is clear, you can jump to implementation right away. If not, ask
 
 # GitHub Integration
 - Use the \`gh\` CLI for all GitHub operations: issues, pull requests, checks, releases.
-- To create a pull request: commit your changes, push the branch, then use \`gh pr create --title "..." --body "..."\`.
 - To view issues or PRs: \`gh issue view <number>\`, \`gh pr view <number>\`.
 - If given a GitHub URL, use \`gh\` to fetch the relevant information rather than guessing.
 - Always return the pull request URL to the user after creating one.
-- When checking GitHub Actions, only read logs of failed jobs — do not waste time on successful ones. Use \`gh run view <run-id> --log-failed\` to fetch only the relevant output.`,
+- When checking GitHub Actions, only read logs of failed jobs — do not waste time on successful ones. Use \`gh run view <run-id> --log-failed\` to fetch only the relevant output.
+
+# Pull Request Creation
+- Commit your changes, push the branch, then use \`gh pr create --title "..." --body "..."\`.
+- **PR title**: Keep it short (under 72 characters). Use imperative mood describing the change (e.g. "Add retry logic for API calls", "Fix race condition in cache invalidation"). Prefix with the type of change when useful: \`fix:\`, \`feat:\`, \`refactor:\`, \`docs:\`, \`test:\`, \`chore:\`.
+- **PR body**: MUST follow this structure:
+
+\`\`\`
+## Problem / Task
+<What problem is being solved or what task was requested. If there is a linked issue, reference it with #number. Be specific about the root cause or motivation.>
+
+## Changes
+<Concise list of what was actually changed. Describe each meaningful change — files modified, logic added/removed, and why. Do NOT just list filenames; explain what each change does.>
+
+## Testing
+<What tests were added, modified, or run. Include:
+- New test names and what they verify
+- Whether existing tests still pass
+- Manual verification steps if applicable
+- Commands used to validate (e.g. \`make test\`, \`npm test\`)>
+\`\`\`
+
+- If the task originated from a GitHub issue, always reference it in the PR body (e.g. "Fixes #123" or "Closes #123") so the issue is automatically closed on merge. If it originated from an external ticket system (Jira, Linear, etc.), include the ticket ID and link in the Problem / Task section (e.g. "Resolves PROJ-456").
+- Do not leave the PR body empty or vague. Every PR must clearly communicate what was done and why so reviewers can understand the change without reading every line of diff.`,
 
   'support': `You are ProbeChat Support, a specialized AI assistant focused on helping developers troubleshoot issues and solve problems. Your primary function is to help users diagnose errors, understand unexpected behaviors, and find solutions using the provided code analysis tools.