fix: don't log tool args

[nirinchev]

No description provided.

[Copilot]

Pull Request Overview

Removes logging of tool arguments to prevent sensitive data exposure.

• Eliminates JSON.stringify of arguments in debug logs.
• Updates log message to a generic format without args.

Comments suppressed due to low confidence (2)

src/tools/tool.ts:74

• Add a unit test to verify that the debug log no longer includes tool arguments and aligns with the updated message format.

logger.debug(LogId.toolExecute, "tool", `Executing tool ${this.name}`);

src/tools/tool.ts:74

• [nitpick] Consider adopting structured logging by passing this.name and, if needed, redacted args as separate properties to maintain queryable context without exposing raw data.

logger.debug(LogId.toolExecute, "tool", `Executing tool ${this.name}`);

[fmenezes]

great catch