#18841: fix(feishu): replace console.log with runtime log for typing indicator errors
channel: zalo
channel: feishu
size: XS
trusted-contributor
Cluster:
Hooks and UI Fixes
Typing indicators are non-critical, but errors should still be logged via the runtime logger (verbose only) rather than console.log, which bypasses log levels.
Fix: Updated addTypingIndicator/removeTypingIndicator to accept runtime env and use it for logging.
<!-- greptile_comment -->
<h3>Greptile Summary</h3>
Replaces `console.log`/`console.error` calls with runtime logger methods in the Feishu and Zalo extensions, ensuring error and debug output respects log-level configuration.
- **Feishu `typing.ts`**: Error logging in `addTypingIndicator`/`removeTypingIndicator` catch blocks now uses `runtime?.log?.()` gated by `shouldLogVerbose()`, instead of unconditional `console.log`. Both functions accept a new optional `runtime?: RuntimeEnv` parameter.
- **Feishu `reply-dispatcher.ts`**: Updated call sites for `addTypingIndicator`/`removeTypingIndicator` to pass the existing `params.runtime` through.
- **Zalo `monitor.ts`**: Replaced `console.error` with `runtime.error?.()` for polling and image download errors, and replaced `console.log` with the existing `logVerbose()` helper for sticker/unsupported message logging.
<h3>Confidence Score: 5/5</h3>
- This PR is safe to merge — it only changes logging mechanisms without altering any business logic.
- The changes are straightforward console.log/console.error replacements with runtime logger calls, following patterns already established elsewhere in these extensions (e.g., Zalo's `logVerbose` helper). No logic changes, no new dependencies, no API surface changes. All call sites are updated and no stale console calls remain in the affected files.
- No files require special attention.
<sub>Last reviewed commit: 658392c</sub>
<!-- greptile_other_comments_section -->
<!-- /greptile_comment -->
Most Similar PRs
#18858: fix(nextcloud-talk): replace console.log with runtime logging
by Clawborn · 2026-02-17
86.2%
#18825: fix(zalo): replace console.log/error with runtime logging
by Clawborn · 2026-02-17
83.8%
#19103: fix(voice-call): replace console.log with runtime logging
by Clawborn · 2026-02-17
77.8%
#23669: refactor(logging): migrate node-host and tailscale console calls to...
by kevinWangSheng · 2026-02-22
73.6%
#21804: fix: improve type safety and use project runtime logging
by hobostay · 2026-02-20
72.3%
#10513: feat(feishu): add quota optimization flags
by BigUncle · 2026-02-06
71.3%
#3392: fix(hooks): remove debug console.log statements from session-memory...
by WinJayX · 2026-01-28
71.1%
#9425: refactor(agents): replace console.warn with SubsystemLogger in comp...
by dinakars777 · 2026-02-05
71.0%
#11305: fix(logging): remove redundant subsystem prefix from log output
by janckerchen · 2026-02-07
70.2%
#19611: fix: use local timezone in log file and console timestamps
by tag-assistant · 2026-02-18
70.0%