mongodb-js / mongodb-mcp-server / 28594888219

83%

push

github

chore: Adds logging support to mongodb-atlas-mcp-remote MCP-537 (#1297)

## Proposed changes

- Replaces the temporary SimpleLogger stub with a proper composite
logger (LoggerBase → ConsoleLogger → CompositeLogger)
- Migrates all logging call sites in cli.ts and tokenManager.ts to
structured LogPayload with log IDs
- Registers clientId and clientSecret as redacted secrets after config
loads, so they are scrubbed from all log output
- Removes MDB_MCP_LOG_LEVEL / logLevel config — console output is
unfiltered, consistent with how the main server's ConsoleLogger works


### Relationship to the main server's logger
LoggerBase, CompositeLogger, and ConsoleLogger in src/logging/ are
direct ports of their counterparts in src/common/logging/ — identical
output format, noRedaction payload logic, attribute serialization, and
EventEmitter generics.

**One intentional deviation:** Keychain → getSecrets?: () => Secret[]

The main server's LoggerBase takes a Keychain object (from
src/common/keychain.ts) to source secrets for redaction. The wrapper
cannot import Keychain without coupling itself to the main package's
internals. The replacement is a lazy getSecrets callback — semantically
identical (Keychain.allSecrets is Secret[]) and called at the same point
in the redaction logic. This difference goes away when PR #1122 moves
Keychain to packages/core.

###  Why the logger was not extracted

- PR #1122 already does this properly — it creates packages/core (for
LoggerBase, CompositeLogger, Keychain) and packages/logging (for
concrete loggers)
- A partial extraction now would conflict with or duplicate that
606-file restructuring
  - Extraction also requires changes out of scope for this ticket:
- Creating and publishing new workspace packages with their own
package.json, tsconfig, build scripts, and CI integration
- Updating the main server's imports across all files that depend on
src/common/logging/
- Decoupling Keychain from LoggerBase in the main server (or co-l... (continued)

2201 of 2894 branches covered (76.05%)

Branch coverage included in aggregate %.

40 of 61 new or added lines in 6 files covered. (65.57%)

3935 of 4522 relevant lines covered (87.02%)

218.52 hits per line