refactor(chapter-26): bring chat tools into structured logging

↗ view on GitHub · Amal · 2026-05-24 · ca69fd8c

Chapter: 26 - Sensitive-path observability.

Plain-English map:
Replace remaining console logging in `chatTools.ts` with the same structured
Pino logger used elsewhere.

Why it matters:
`chatTools.ts` assembles prompts and handles model tool calls. It should have
the same traceability and privacy discipline as the rest of the API.

Principle:
Observability conventions should be consistent, especially in sensitive code.

Precedent borrowed:
Upstream PR #156 and the structured logging baseline from Chapter 05.

Upstream base: willchen96/mike@d39f580.
Original local commit: 921ef5b.
Repository amal66/mike
Author Amal <mamalanand3@gmail.com>
Authored
Parents b358c585
Stats 1 file changed , +34 , -63
Part of Structured logging and PII minimization

Capture this commit into my fork

Download a Markdown prompt that tells Claude how to port this exact commit into your working tree. Run it via claude -p < capture-commit-ca69fd8c.md from inside the repo you want the change in.

⬇ Download capture-commit-ca69fd8c.md