chore(lint): ban console in runtime code with Biome noConsole

Runtime code logs through createLogger from @sim/logger. Scripts, CLIs,
script-migrations, the logger itself, SDK examples, and tests keep console
as their interface. Autofix is disabled so lint --unsafe never silently
deletes a console call.

Author: waleedlatif1

apps/docs/app/api/search/route.ts

@@ -1,8 +1,11 @@
+import { createLogger } from '@sim/logger'
 import { sql } from 'drizzle-orm'
 import { type NextRequest, NextResponse } from 'next/server'
 import { db, docsEmbeddings } from '@/lib/db'
 import { generateSearchEmbedding } from '@/lib/embeddings'
 
+const logger = createLogger('DocsSearchAPI')
+
 export const runtime = 'nodejs'
 export const revalidate = 0
 
@@ -196,7 +199,7 @@ export async function GET(request: NextRequest) {
 
     return NextResponse.json(searchResults)
   } catch (error) {
-    console.error('Semantic search error:', error)
+    logger.error('Semantic search error:', error)
 
     return NextResponse.json([])
   }

apps/docs/app/llms-full.txt/route.ts

@@ -1,6 +1,9 @@
+import { createLogger } from '@sim/logger'
 import { getLLMText } from '@/lib/llms'
 import { source } from '@/lib/source'
 
+const logger = createLogger('DocsLlmsFullText')
+
 export const revalidate = false
 
 export async function GET() {
@@ -18,7 +21,7 @@ export async function GET() {
       },
     })
   } catch (error) {
-    console.error('Error generating LLM full text:', error)
+    logger.error('Error generating LLM full text:', error)
     return new Response('Error generating full documentation text', { status: 500 })
   }
 }

apps/docs/app/llms.txt/route.ts

@@ -1,6 +1,9 @@
+import { createLogger } from '@sim/logger'
 import { source } from '@/lib/source'
 import { DOCS_BASE_URL } from '@/lib/urls'
 
+const logger = createLogger('DocsLlmsManifest')
+
 export const revalidate = false
 
 export async function GET() {
@@ -70,7 +73,7 @@ See: https://llmstxt.org for specification`
       },
     })
   } catch (error) {
-    console.error('Error generating LLM manifest:', error)
+    logger.error('Error generating LLM manifest:', error)
     return new Response('Error generating documentation manifest', { status: 500 })
   }
 }

apps/docs/package.json

@@ -20,6 +20,7 @@
   "dependencies": {
     "@sim/db": "workspace:*",
     "@sim/emcn": "workspace:*",
+    "@sim/logger": "workspace:*",
     "@sim/utils": "workspace:*",
     "@sim/workflow-renderer": "workspace:*",
     "@xyflow/react": "12.11.3",

apps/sim/app/_shell/hydration-error-handler.tsx

@@ -33,6 +33,7 @@ function isRadixIdMismatch(args: unknown[]): boolean {
  */
 export function HydrationErrorHandler() {
   useEffect(() => {
+    // biome-ignore lint/suspicious/noConsole: wraps console.error itself to filter extension hydration noise
     const originalError = console.error
     console.error = (...args) => {
       if (typeof args[0] === 'string' && args[0].includes('Hydration')) {

apps/sim/instrumentation-node.ts

@@ -259,8 +259,7 @@ async function initializeOpenTelemetry() {
     exporter.export = (spans, resultCallback) => {
       origExport(spans, (result) => {
         if (result?.code !== 0) {
-          // eslint-disable-next-line no-console
-          console.error('[OTEL] exporter export failed', {
+          logger.error('Exporter export failed', {
             endpoint: telemetryConfig.endpoint,
             resultCode: result?.code,
             error: result?.error?.message,

biome.json

@@ -100,7 +100,8 @@
         "noControlCharactersInRegex": "off",
         "noThenProperty": "off",
         "noAssignInExpressions": "off",
-        "noDocumentCookie": "off"
+        "noDocumentCookie": "off",
+        "noConsole": { "level": "error", "fix": "none" }
       },
       "correctness": {
         "useExhaustiveDependencies": "off",
@@ -155,6 +156,21 @@
       "includes": ["apps/sim/app/_styles/tailwind.css"],
       "linter": { "enabled": false },
       "formatter": { "enabled": false }
+    },
+    {
+      "includes": [
+        "**/scripts/**",
+        "**/*.test.ts",
+        "**/*.test.tsx",
+        "apps/sim/vitest.setup.ts",
+        "packages/cli/**",
+        "packages/sim-cli/**",
+        "packages/sim-setup/**",
+        "packages/db/script-migrations/**",
+        "packages/logger/src/**",
+        "packages/ts-sdk/examples/**"
+      ],
+      "linter": { "rules": { "suspicious": { "noConsole": "off" } } }
     }
   ],
   "javascript": {

bun.lock

@@ -31,6 +31,7 @@ function toolGuard(label: string, detect: () => { ok: boolean; hint: string }):
     }
     if (!warned) {
       warned = true
+      // biome-ignore lint/suspicious/noConsole: a skipped suite must tell the test runner's console why
       console.warn(`[@sim/testing] Skipping tests that require ${label}. ${result.hint}`)
     }
     return false