fix: remove debug console.logs and use logger in library modules

[ashsolei]

Summary

Remove leftover debug console.log statements and replace bare console.log/warn/error calls with the project's logger utility in library modules.

Problem

• config-validator.ts contained 3 leftover console.log('DEBUG: ...') statements inside a conditional block. In stdio mode, any direct console.log output corrupts the JSON-RPC transport stream.
• docs-mapper.ts and node-loader.ts used bare console.log/warn/error for operational logging instead of the project's logger utility, which properly suppresses output in stdio mode via the MCP_MODE environment variable check.

Changes

File	Change
src/services/config-validator.ts	Removed 3 console.log('DEBUG: ...') statements and their containing conditional block
src/mappers/docs-mapper.ts	Replaced 4 console.log calls with logger.warn/logger.debug
src/loaders/node-loader.ts	Replaced 7 console.log/warn/error calls with logger.info/debug/warn/error

Testing

• The logger utility is already well-tested and used throughout the codebase (18+ existing imports)
• In stdio mode: logger suppresses all output, preventing JSON-RPC corruption
• In HTTP mode: logger suppresses output during active MCP requests
• In rebuild/validate scripts: logger outputs normally since MCP_MODE is not set

[Copilot]

Pull request overview

This PR removes leftover direct console.* output from library modules and routes operational/debug logging through the project logger utility to avoid corrupting stdio JSON-RPC streams (and to respect MCP_MODE-based suppression).

Changes:

• Removed a debug-only console.log block from ConfigValidator’s Python return-format checks.
• Replaced console.log usage in DocsMapper with logger.warn/logger.debug.
• Replaced console.log/warn/error usage in N8nNodeLoader with logger.info/debug/warn/error.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File	Description
src/services/config-validator.ts	Removes debug console.log statements that could interfere with stdio transports.
src/mappers/docs-mapper.ts	Uses the shared logger for docs lookup logging (warn/debug).
src/loaders/node-loader.ts	Uses the shared logger for package/node loading logs; updates error/warn/info/debug paths.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

loadAllNodes() catch block interpolates the caught error into the message (${error}), which can degrade diagnostics (e.g., [object Object]) and drop stack traces. Since the project logger supports variadic args, pass the error as a separate argument (or log error instanceof Error ? error.message/stack : String(error)) so stack/metadata is preserved (see similar logger.error(msg, error) usage in src/http-server.ts).

Suggested change

	logger.error(`Failed to load ${pkg.name}: ${error}`);
	logger.error(`Failed to load ${pkg.name}`, error);

[ashsolei]

Closing — will maintain fixes in a private mirror instead. Thank you for the review.