Skip to content

fix: upgrade diagnostic logs to info level for production visibility - #141

Merged
lishuceo merged 1 commit into
mainfrom
diag/history-dedup-logging
Mar 11, 2026
Merged

fix: upgrade diagnostic logs to info level for production visibility#141
lishuceo merged 1 commit into
mainfrom
diag/history-dedup-logging

Conversation

@lishuceo

Copy link
Copy Markdown
Owner

Summary

影响范围

三处诊断日志:fetchRecentMessages、buildDirectTaskHistory、executeDirectTask

🤖 Generated with Claude Code

Production LOG_LEVEL is info, debug logs were invisible.
Temporary change to capture history dedup pipeline data.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@claude

claude Bot commented Mar 11, 2026

Copy link
Copy Markdown

Claude finished @lishuceo's task in 48s —— View job


Review: PR #141 — Upgrade diagnostic logs to info level

  • Gather context and check previous comments
  • Review changed files
  • Post review summary

Approved — Clean, minimal change. No issues found.

This PR correctly upgrades all 5 diagnostic log statements added in PR #140 from logger.debug to logger.info. Since production runs with LOG_LEVEL=info, the debug-level logs were invisible in production — this fix ensures the diagnostic data is actually observable where it matters.

Files reviewed:

  • src/feishu/client.ts — 1 log in fetchRecentMessages
  • src/feishu/event-handler.ts — 4 logs across executeDirectTask and buildDirectTaskHistory

Notes:

  • These diagnostic logs include potentially verbose data (message ID arrays, content lengths). This is fine for temporary debugging, but consider reverting to debug level or removing them once the history dedup pipeline is confirmed stable in production.

@lishuceo
lishuceo merged commit 3ae095c into main Mar 11, 2026
4 checks passed
@lishuceo
lishuceo deleted the diag/history-dedup-logging branch March 11, 2026 13:28
lishuceo added a commit that referenced this pull request Apr 8, 2026
Production LOG_LEVEL is info, debug logs were invisible.
Temporary change to capture history dedup pipeline data.

Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
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.

1 participant