bytelyst/learning_ai_common_plat
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

chore(diagnostics-client): document console capture

Browse Source 
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
This commit is contained in:
Saravana Achu Mac 2026-05-04 16:40:03 -07:00
parent 2218067ef0
commit 91e08bfda0
1 changed files with 2 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
packages/diagnostics-client/src/client.ts
View File

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