From 91e08bfda0c8c8a760a2fd890af968914b4ceae6 Mon Sep 17 00:00:00 2001
From: Saravana Achu Mac <saravanakumardb1@gmail.com>
Date: Mon, 4 May 2026 16:40:03 -0700
Subject: [PATCH] chore(diagnostics-client): document console capture

What changed:
- Added a scoped no-console justification around the console capture wrapper.

Warning impact:
- @bytelyst/diagnostics-client scoped lint: 8 warnings -> 0.
- Workspace lint: 101 -> 93 warnings.

Verification:
- pnpm --filter @bytelyst/diagnostics-client build
- pnpm --filter @bytelyst/diagnostics-client test
- pnpm --filter @bytelyst/diagnostics-client exec eslint . --ext .ts,.tsx
- pnpm lint
---
 packages/diagnostics-client/src/client.ts | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/packages/diagnostics-client/src/client.ts b/packages/diagnostics-client/src/client.ts
index d18d6c56..0118e8b3 100644
--- a/packages/diagnostics-client/src/client.ts
+++ b/packages/diagnostics-client/src/client.ts
@@ -492,6 +492,7 @@ export class DiagnosticsClient {
   /**
    * Setup console capture
    */
+  /* eslint-disable no-console -- This method intentionally wraps console APIs to capture diagnostics, then forwards to the originals. */
   private setupConsoleCapture(): void {
     const originalConsole = {
       log: console.log.bind(console),
@@ -526,6 +527,7 @@ export class DiagnosticsClient {
 
     this.breadcrumbs.add('diagnostics', 'Console capture enabled');
   }
+  /* eslint-enable no-console */
 
   /**
    * Setup error capture