Merge remote-tracking branch 'origin/staging' into dallin/bus-1431-auto-sharing-for-queries-from-slack-channels

apps/server/src/middleware/logger.test.ts

@@ -34,7 +34,6 @@ describe('logger middleware', () => {
   });
 
   it('should use info level by default when LOG_LEVEL is not set', async () => {
-    // biome-ignore lint/performance/noDelete: <explanation>
     delete process.env.LOG_LEVEL;
 
     const { loggerMiddleware } = await import('./logger');
@@ -83,7 +82,6 @@ describe('logger middleware', () => {
   });
 
   it('should not capture console methods when LOG_LEVEL is not set', async () => {
-    // biome-ignore lint/performance/noDelete: <explanation>
     delete process.env.LOG_LEVEL;
 
     // Create mocks

biome.json

@@ -23,7 +23,8 @@
         "noConsoleLog": "warn"
       },
       "complexity": {
-        "noExcessiveCognitiveComplexity": "off"
+        "noExcessiveCognitiveComplexity": "off",
+        "noForEach": "off"
       },
       "performance": {
         "noDelete": "error"
@@ -128,6 +129,9 @@
           "suspicious": {
             "noConsoleLog": "off",
             "noExplicitAny": "off"
+          },
+          "performance": {
+            "noDelete": "off"
           }
         }
       }

packages/ai/src/tools/communication-tools/message-user-clarifying-question.ts

@@ -78,7 +78,7 @@ async function processMessageUserClarifyingQuestion(): Promise<
 // Main message user clarifying question function with tracing
 const executeMessageUserClarifyingQuestion = wrapTraced(
   async (
-    clarifyingQuestion: string
+    _clarifyingQuestion: string
   ): Promise<z.infer<typeof messageUserClarifyingQuestionOutputSchema>> => {
     return await processMessageUserClarifyingQuestion();
   },

packages/ai/src/utils/database/chunk-processor.ts

@@ -13,7 +13,6 @@ import {
 } from '../file-selection';
 import { normalizeEscapedText } from '../streaming/escape-normalizer';
 import { OptimisticJsonParser, getOptimisticValue } from '../streaming/optimistic-json-parser';
-import { extractResponseMessages } from './format-llm-messages-as-reasoning';
 import type {
   AssistantMessageContent,
   GenericToolSet,
@@ -2021,15 +2020,15 @@ export class ChunkProcessor<T extends ToolSet = GenericToolSet> {
 
       // Append results to the existing content
       if (fileObj.file) {
-        fileObj.file.text = currentContent + '\n\n' + resultsYaml;
+        fileObj.file.text = `${currentContent}\n\n${resultsYaml}`;
       }
 
       // Update the entry title and secondary title
       if ('title' in entry) {
-        (entry as any).title = title;
+        (entry as ReasoningEntry & { title: string }).title = title;
       }
       if ('secondary_title' in entry && secondaryTitle) {
-        (entry as any).secondary_title = secondaryTitle;
+        (entry as ReasoningEntry & { secondary_title: string }).secondary_title = secondaryTitle;
       }
     } catch (error) {
       console.error('Error updating SQL file with results:', {

packages/ai/src/utils/memory/message-converters.ts

@@ -128,7 +128,7 @@ export function convertToolCallToMessage(
           message: parsed.message,
         };
         return { type: 'response', message: responseMessage };
-      } catch (error) {
+      } catch (_) {
         return null;
       }
     }
@@ -144,7 +144,7 @@ export function convertToolCallToMessage(
           message: parsed.message,
         };
         return { type: 'response', message: responseMessage };
-      } catch (error) {
+      } catch (_error) {
         return null;
       }
     }
@@ -164,7 +164,7 @@ export function convertToolCallToMessage(
           finished_reasoning: !parsed.nextThoughtNeeded,
         };
         return { type: 'reasoning', message: reasoningMessage };
-      } catch (error) {
+      } catch (_error) {
         return null;
       }
     }
@@ -223,7 +223,7 @@ export function convertToolCallToMessage(
           files,
         };
         return { type: 'reasoning', message: reasoningMessage };
-      } catch (error) {
+      } catch (_error) {
         return null;
       }
     }
@@ -241,7 +241,7 @@ export function convertToolCallToMessage(
           status,
         };
         return { type: 'reasoning', message: reasoningMessage };
-      } catch (error) {
+      } catch (_error) {
         return null;
       }
     }
@@ -300,7 +300,7 @@ export function convertToolCallToMessage(
           files,
         };
         return { type: 'reasoning', message: reasoningMessage };
-      } catch (error) {
+      } catch (_error) {
         return null;
       }
     }
@@ -359,7 +359,7 @@ export function convertToolCallToMessage(
           files,
         };
         return { type: 'reasoning', message: reasoningMessage };
-      } catch (error) {
+      } catch (_error) {
         return null;
       }
     }
@@ -418,7 +418,7 @@ export function convertToolCallToMessage(
           files,
         };
         return { type: 'reasoning', message: reasoningMessage };
-      } catch (error) {
+      } catch (_error) {
         return null;
       }
     }

packages/test-utils/src/envHelpers/env-helpers.ts

@@ -30,6 +30,8 @@ export async function setupTestEnvironment(): Promise<TestEnvironment> {
 export function withTestEnv<T>(testFn: () => Promise<T>): () => Promise<T> {
   return async () => {
     const originalEnv = { ...process.env };
+
+    // biome-ignore lint/correctness/noUnusedVariables:
     const env = await setupTestEnvironment();
     try {
       return await testFn();