hermes - 💡(How to fix) Fix 🔧 gateway/platforms: Replace 129 print() calls with proper logging

Official PRs (…)
ON THIS PAGE

Recommended Tools

×6

Utilities matched from this issue’s tags and category — try them while you read without losing context.

GitHub issue graph ai analysis

Paste a GitHub issue URL. We fetch that issue, discover linked issues from bodies/comments/timeline, collect linked pull requests, and produce a structured English report.

The report is written in English Markdown for sharing and archival.

Helpful · Quick feedback

Loading…
RAW_BUFFERClick to expand / collapse

Severity: Medium Component: Gateway Priority: P2

Problem

Gateway platform adapters use print() statements instead of proper logging. 129 print() calls across 20+ platform files pollute stdout and bypass log configuration.

Evidence

  • 129 print() calls in gateway/platforms/
  • 284 TODO/FIXME comments across platforms
  • Multiple deprecated patterns: json.dumps() without ensure_ascii=False for non-ASCII content

Impact

  • No log level filtering → debug prints leak to production
  • Inconsistent with codebase logging standards
  • Hard to debug platform-specific issues
  • Potential Unicode encoding issues in some platforms

Suggested Fix

  1. Replace all print() with logger.debug() or logger.info()
  2. Add ensure_ascii=False to json.dumps() where non-ASCII content is expected (telegram, feishu, etc.)
  3. Clean up 284 TODO/FIXME comments (either fix or create tracked issues)

Files

  • gateway/platforms/telegram.py
  • gateway/platforms/discord.py
  • gateway/platforms/feishu.py
  • gateway/platforms/slack.py
  • All other platform adapters

Auto-generated by Юка (Hermes Agent) bug scanner

Vote matrix · Quick signals

Works
Did the solution work? Tap to confirm.
Easy Fix
Was it a quick fix?
Time Saver
Did it save you time?
Blocking
Was it severely blocking?
Common Issue
Are others likely hitting this too?
Flaky / Intermittent
Is it intermittent?
Verified / Reproducible
Can you reproduce it reliably?
Loading…

Still need to ship something?

×6

Another batch ranked right after the header list — different links, same matching logic.

Back to top recommendations

TRENDING

hermes - 💡(How to fix) Fix 🔧 gateway/platforms: Replace 129 print() calls with proper logging