Skip to content

116 log tools#117

Closed
mborne wants to merge 3 commits into
masterfrom
116-log-tools
Closed

116 log tools#117
mborne wants to merge 3 commits into
masterfrom
116-log-tools

Conversation

@mborne
Copy link
Copy Markdown
Contributor

@mborne mborne commented May 5, 2026

(closes #116)

Summary

This PR improves logging consistency and observability around tool execution.

It introduces a clear logging split:

  • Tool entry points now log execution at info level with the tool name and input payload.
  • Internal provider/helper calls (GPF and WFS internals) are moved to debug level and use standardized log prefixes.

This helps keep production logs concise while preserving detailed traces for debugging.

What Changed

  • Added logger imports in all MCP tool classes and logged each execute(...) call with:
    • tool name (this.name)
    • input payload
  • Updated internal logs from info to debug in several GPF/WFS modules:
    • adminexpress
    • altitude
    • geocode
    • parcellaire-express
    • urbanisme / assiettes servitudes
    • WFS feature execution helper
  • Standardized debug log prefixes (e.g. [gpf:...]) for easier filtering.

Why

  • Improve operator visibility of actual tool invocations.
  • Reduce noise in default logs by moving low-level internals to debug.
  • Make troubleshooting easier with consistent, grep-friendly log tags.

Impact

  • No functional behavior change to tool outputs or APIs.
  • Logging behavior changes only:
    • More explicit top-level tool execution logs at info.
    • More detailed internals available at debug.

Testing

  • Existing unit tests pass (no schema or business logic changes expected).
  • Change is primarily operational/observability-focused.

@mborne mborne closed this May 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Review logs ensuring that tool call are logged

1 participant