Claude be88bc870c fix: address all 6 Copilot AI review comments from PR #154 ...

Fixes all valid issues identified by Copilot AI review:

1. 🔴 CRITICAL: Fix deduplication key collision risk
   - Before: Used maskedJid in dedupe key (e.g., "4680****7890")
   - Problem: Different JIDs with same first/last 4 digits collide
   - After: Use original unmasked JID for dedup key
   - Impact: Prevents legitimate errors from being suppressed

2. 🟡 Fix JID masking logic flaw
   - Before: Masked full JID string including domain (e.g., "1234****.net")
   - Problem: Obscures useful info while masking wrong part
   - After: Decode JID, mask only user portion, preserve domain
   - Result: "4680****7890@s.whatsapp.net" instead of "1234****.net"

3. 🟡 Add structured logging context
   - Before: logger.info(string) - loses queryability
   - After: logger.info({ jid, maskedJid, targetedDevices }, message)
   - Impact: Maintains observability and log filtering capability

4. 🟡 Fix misleading deletion count wording
   - Before: "Cleared: X devices" (implies actual deletions)
   - Problem: deleteSession() returns void, count is targeted
   - After: "Targeted: X devices" (reflects actual behavior)

5. 🟢 Remove unused variable
   - Removed _origConsoleLog (dead code after removing console.log interceptor)
   - Prevents no-unused-vars lint failure

6. 🟢 Fix semicolon format violation
   - Repository config: semi: false (Prettier)
   - Removed trailing semicolon from JID extraction line
   - Prevents eslint-plugin-prettier check failure

All changes maintain backward compatibility while fixing correctness,
observability, and code quality issues.

https://claude.ai/code/session_01TvSrN9JmHZDdKMZDFWEcUS

2026-02-12 21:27:37 +00:00

..

auth-utils.ts

fix(auth-utils): improve error message clarity for destroyed state

2026-02-04 03:56:33 +00:00

baileys-event-stream.ts

feat(socket): integrate circuit breaker with Socket operations

2026-01-20 02:31:23 +00:00

baileys-logger.ts

fix(logger): address PR #19 code review feedback

2026-01-20 21:44:08 +00:00

browser-utils.ts

feat(browser-utils): add automatic OS version detection

2026-01-30 15:26:46 +00:00

business.ts

project: Move to ESM Modules

2025-07-17 13:54:17 +03:00

cache-utils.ts

feat(socket): integrate circuit breaker with Socket operations

2026-01-20 02:31:23 +00:00

chat-utils.ts

fix: revert remaining ?? '' patterns across Utils, WABinary, Signal, and WAUSync files

2026-02-09 13:56:33 +00:00

circuit-breaker.ts

feat: add adaptive metrics and circuit breaker trips tracking

2026-01-22 13:57:34 +00:00

crypto.ts

fix: replace static whatsapp-rust-bridge imports with lazy dynamic loading

2026-02-09 02:31:52 +00:00

decode-wa-message.ts

fix: address all 6 Copilot AI review comments from PR #154

2026-02-12 21:27:37 +00:00

event-buffer.ts

feat: implement automatic LID/PN chat merge (Option A+)

2026-02-09 20:14:54 +00:00

generics.ts

fix: revert remaining ?? '' patterns across Utils, WABinary, Signal, and WAUSync files

2026-02-09 13:56:33 +00:00

health-status.ts

fix: address code review feedback (batch 2)

2026-01-24 20:04:53 +00:00

history.ts

fix: revert remaining ?? '' patterns across Utils, WABinary, Signal, and WAUSync files

2026-02-09 13:56:33 +00:00

identity-change-handler.ts

fix: address PR review comments for identity change handler

2026-01-22 18:55:01 +00:00

index.ts

fix: replace static whatsapp-rust-bridge imports with lazy dynamic loading

2026-02-09 02:31:52 +00:00

link-preview.ts

refactor: replace axios with fetch API and update related types (#1666)

2025-09-28 05:52:43 +03:00

logger-adapter.ts

feat(socket): integrate circuit breaker with Socket operations

2026-01-20 02:31:23 +00:00

logger.ts

fix(logger): avoid pino-pretty dependency

2026-01-20 17:30:55 -03:00

lt-hash.ts

fix: replace static whatsapp-rust-bridge imports with lazy dynamic loading

2026-02-09 02:31:52 +00:00

make-mutex.ts

fix Memory leak in makeMutex - Promise never gets garbage collected (#2151)

2025-12-12 04:35:24 +02:00

message-retry-manager.ts

feat: implement identity key change detection for Signal protocol

2026-01-30 03:35:18 +00:00

messages-media.ts

fix: revert remaining ?? '' patterns across Utils, WABinary, Signal, and WAUSync files

2026-02-09 13:56:33 +00:00

messages.ts

fix: revert remaining ?? '' patterns across Utils, WABinary, Signal, and WAUSync files

2026-02-09 13:56:33 +00:00

noise-handler.ts

fix(types): resolve all TypeScript compilation errors from non-null assertion removal

2026-02-09 01:02:13 +00:00

pre-key-manager.ts

fix(pre-key-manager): add destroyed flag with atomic protection (M2)

2026-02-05 01:48:33 +00:00

process-message.ts

fix(ctwa): critical cache key mismatch and type consolidation

2026-02-11 15:19:48 +00:00

prometheus-metrics.ts

feat: implement identity key change detection for Signal protocol

2026-01-30 03:35:18 +00:00

reporting-utils.ts

feat: replace async crypto with sync Rust WASM (port of Baileys b5c1741)

2026-02-08 20:49:18 +00:00

retry-utils.ts

fix(metrics,retry): address gold standard code review

2026-01-20 19:37:19 +00:00

signal.ts

fix: revert defensive null checks in signal.ts that prevented Signal session creation

2026-02-09 03:50:36 +00:00

sticker-pack.ts

fix(types): remove non-null assertions in remaining 12 files

2026-02-09 00:32:04 +00:00

structured-logger.ts

fix(pr-77): resolve memory leaks and remove unused metrics buffering

2026-02-04 01:15:41 +00:00

sync-action-utils.ts

feat(lid-mapping): implement PR #2275 optimizations and event batching

2026-02-02 19:35:01 +00:00

tc-token-utils.ts

chore: lint

2026-01-18 17:16:28 +02:00

trace-context.ts

feat(socket): integrate circuit breaker with Socket operations

2026-01-20 02:31:23 +00:00

unified-session.ts

fix(unified-session): resolve circular dependency with TimeMs

2026-01-24 20:42:32 +00:00

use-multi-file-auth-state.ts

general: revert #1665

2025-10-03 18:14:30 +03:00

validate-connection.ts

fix: revert remaining ?? '' patterns across Utils, WABinary, Signal, and WAUSync files

2026-02-09 13:56:33 +00:00

version-cache.ts

fix(types): remove non-null assertions in remaining 12 files

2026-02-09 00:32:04 +00:00

wasm-bridge.ts

fix: replace static whatsapp-rust-bridge imports with lazy dynamic loading

2026-02-09 02:31:52 +00:00