@probelabs/visor 0.1.87 → 0.1.89

package/dist/sdk/sdk.d.mts

@@ -59,6 +59,8 @@ interface ReviewIssue {
 interface ReviewSummary {
     issues?: ReviewIssue[];
     debug?: AIDebugInfo;
+    /** Session ID created for this check (for cleanup tracking) */
+    sessionId?: string;
 }
 
 /**
@@ -262,6 +264,8 @@ interface CheckConfig {
     if?: string;
     /** Check name to reuse AI session from, or true to use first dependency (only works with depends_on) */
     reuse_ai_session?: string | boolean;
+    /** How to reuse AI session: 'clone' (default, copy history) or 'append' (share history) */
+    session_mode?: 'clone' | 'append';
     /** Simple fail condition - fails check if expression evaluates to true */
     fail_if?: string;
     /** Check-specific failure conditions - optional (deprecated, use fail_if) */

package/dist/sdk/sdk.d.ts

@@ -59,6 +59,8 @@ interface ReviewIssue {
 interface ReviewSummary {
     issues?: ReviewIssue[];
     debug?: AIDebugInfo;
+    /** Session ID created for this check (for cleanup tracking) */
+    sessionId?: string;
 }
 
 /**
@@ -262,6 +264,8 @@ interface CheckConfig {
     if?: string;
     /** Check name to reuse AI session from, or true to use first dependency (only works with depends_on) */
     reuse_ai_session?: string | boolean;
+    /** How to reuse AI session: 'clone' (default, copy history) or 'append' (share history) */
+    session_mode?: 'clone' | 'append';
     /** Simple fail condition - fails check if expression evaluates to true */
     fail_if?: string;
     /** Check-specific failure conditions - optional (deprecated, use fail_if) */

package/dist/sdk/sdk.js

@@ -400,6 +400,47 @@ var init_session_registry = __esm({
       hasSession(sessionId) {
         return this.sessions.has(sessionId);
       }
+      /**
+       * Clone a session with a new session ID
+       * Creates a new ProbeAgent with a copy of the conversation history
+       */
+      async cloneSession(sourceSessionId, newSessionId) {
+        const sourceAgent = this.sessions.get(sourceSessionId);
+        if (!sourceAgent) {
+          console.error(`\u26A0\uFE0F  Cannot clone session: ${sourceSessionId} not found`);
+          return void 0;
+        }
+        try {
+          const sourceHistory = sourceAgent.conversationHistory || [];
+          const sourceOptions = sourceAgent.options || {};
+          const { ProbeAgent: ProbeAgentClass } = await import("@probelabs/probe");
+          const clonedAgent = new ProbeAgentClass({
+            ...sourceOptions,
+            sessionId: newSessionId
+          });
+          if (sourceHistory.length > 0) {
+            try {
+              const deepClonedHistory = JSON.parse(JSON.stringify(sourceHistory));
+              clonedAgent.conversationHistory = deepClonedHistory;
+              console.error(
+                `\u{1F4CB} Cloned session ${sourceSessionId} \u2192 ${newSessionId} (${sourceHistory.length} messages, deep copy)`
+              );
+            } catch (cloneError) {
+              console.error(
+                `\u26A0\uFE0F  Warning: Deep clone failed for session ${sourceSessionId}, using shallow copy: ${cloneError}`
+              );
+              clonedAgent.conversationHistory = [...sourceHistory];
+            }
+          } else {
+            console.error(`\u{1F4CB} Cloned session ${sourceSessionId} \u2192 ${newSessionId} (no history)`);
+          }
+          this.registerSession(newSessionId, clonedAgent);
+          return clonedAgent;
+        } catch (error) {
+          console.error(`\u26A0\uFE0F  Failed to clone session ${sourceSessionId}: ${error}`);
+          return void 0;
+        }
+      }
       /**
        * Register process exit handlers to cleanup sessions on exit
        */
@@ -671,8 +712,9 @@ var init_ai_review_service = __esm({
       }
       /**
        * Execute AI review using session reuse - reuses an existing ProbeAgent session
+       * @param sessionMode - 'clone' (default) clones history, 'append' shares history
        */
-      async executeReviewWithSessionReuse(prInfo, customPrompt, parentSessionId, schema, checkName) {
+      async executeReviewWithSessionReuse(prInfo, customPrompt, parentSessionId, schema, checkName, sessionMode = "clone") {
         const startTime = Date.now();
         const timestamp = (/* @__PURE__ */ new Date()).toISOString();
         const existingAgent = this.sessionRegistry.getSession(parentSessionId);
@@ -682,7 +724,24 @@ var init_ai_review_service = __esm({
           );
         }
         const prompt = await this.buildCustomPrompt(prInfo, customPrompt, schema);
-        log(`\u{1F504} Reusing AI session ${parentSessionId} for review...`);
+        let agentToUse;
+        let currentSessionId;
+        if (sessionMode === "clone") {
+          currentSessionId = `${parentSessionId}-clone-${Date.now()}`;
+          log(`\u{1F4CB} Cloning AI session ${parentSessionId} \u2192 ${currentSessionId}...`);
+          const clonedAgent = await this.sessionRegistry.cloneSession(
+            parentSessionId,
+            currentSessionId
+          );
+          if (!clonedAgent) {
+            throw new Error(`Failed to clone session ${parentSessionId}. Falling back to append mode.`);
+          }
+          agentToUse = clonedAgent;
+        } else {
+          log(`\u{1F504} Appending to AI session ${parentSessionId} (shared history)...`);
+          agentToUse = existingAgent;
+          currentSessionId = parentSessionId;
+        }
         log(`\u{1F527} Debug: Raw schema parameter: ${JSON.stringify(schema)} (type: ${typeof schema})`);
         log(`Schema type: ${schema || "none (no schema)"}`);
         let debugInfo;
@@ -706,7 +765,7 @@ var init_ai_review_service = __esm({
         }
         try {
           const { response, effectiveSchema } = await this.callProbeAgentWithExistingSession(
-            existingAgent,
+            agentToUse,
             prompt,
             schema,
             debugInfo,
@@ -722,6 +781,9 @@ var init_ai_review_service = __esm({
           if (debugInfo) {
             result.debug = debugInfo;
           }
+          if (sessionMode === "clone" && currentSessionId !== parentSessionId) {
+            result.sessionId = currentSessionId;
+          }
           return result;
         } catch (error) {
           if (debugInfo) {
@@ -2714,9 +2776,10 @@ var init_ai_check_provider = __esm({
           }
           let result;
           if (sessionInfo?.reuseSession && sessionInfo.parentSessionId) {
+            const sessionMode = config.session_mode || "clone";
             if (aiConfig.debug) {
               console.error(
-                `\u{1F504} Debug: Using session reuse with parent session: ${sessionInfo.parentSessionId}`
+                `\u{1F504} Debug: Using session reuse with parent session: ${sessionInfo.parentSessionId} (mode: ${sessionMode})`
               );
             }
             result = await service.executeReviewWithSessionReuse(
@@ -2724,7 +2787,8 @@ var init_ai_check_provider = __esm({
               processedPrompt,
               sessionInfo.parentSessionId,
               schema,
-              config.checkName
+              config.checkName,
+              sessionMode
             );
           } else {
             if (aiConfig.debug) {
@@ -5656,9 +5720,11 @@ var init_github_check_service = __esm({
             reviewIssues,
             executionError
           );
-          let filteredIssues = reviewIssues;
+          let filteredIssues = reviewIssues.filter(
+            (issue) => !(issue.file === "system" && issue.line === 0)
+          );
           if (filesChangedInCommit && filesChangedInCommit.length > 0) {
-            filteredIssues = reviewIssues.filter(
+            filteredIssues = filteredIssues.filter(
               (issue) => filesChangedInCommit.some((changedFile) => issue.file === changedFile)
             );
           }
@@ -7749,6 +7815,12 @@ ${expr}
                     `\u{1F527} Debug: Completed check: ${checkName}, issues found: ${(finalResult.issues || []).length}`
                   );
                 }
+                if (finalResult.sessionId) {
+                  sessionIds.set(checkName, finalResult.sessionId);
+                  if (debug) {
+                    log2(`\u{1F527} Debug: Tracked cloned session for cleanup: ${finalResult.sessionId}`);
+                  }
+                }
               }
               const enrichedIssues = (finalResult.issues || []).map((issue) => ({
                 ...issue,
@@ -9586,6 +9658,11 @@ var init_config_schema = __esm({
               type: ["string", "boolean"],
               description: "Check name to reuse AI session from, or true to use first dependency (only works with depends_on)"
             },
+            session_mode: {
+              type: "string",
+              enum: ["clone", "append"],
+              description: "How to reuse AI session: 'clone' (default, copy history) or 'append' (share history)"
+            },
             fail_if: {
               type: "string",
               description: "Simple fail condition - fails check if expression evaluates to true"
@@ -11129,6 +11206,22 @@ var ConfigManager = class {
         }
       }
     }
+    if (checkConfig.session_mode !== void 0) {
+      if (checkConfig.session_mode !== "clone" && checkConfig.session_mode !== "append") {
+        errors.push({
+          field: `checks.${checkName}.session_mode`,
+          message: `Invalid session_mode value for "${checkName}": must be 'clone' or 'append'`,
+          value: checkConfig.session_mode
+        });
+      }
+      if (!checkConfig.reuse_ai_session) {
+        errors.push({
+          field: `checks.${checkName}.session_mode`,
+          message: `Check "${checkName}" has session_mode but no reuse_ai_session. session_mode requires reuse_ai_session to be set.`,
+          value: checkConfig.session_mode
+        });
+      }
+    }
     if (checkConfig.tags !== void 0) {
       if (!Array.isArray(checkConfig.tags)) {
         errors.push({