Skip to content

slack-channel-monitor: structured logging for reply-back and dedupe paths#287

Draft
rbren wants to merge 1 commit into
mainfrom
slack-monitor-add-logging
Draft

slack-channel-monitor: structured logging for reply-back and dedupe paths#287
rbren wants to merge 1 commit into
mainfrom
slack-monitor-add-logging

Conversation

@rbren

@rbren rbren commented Jun 2, 2026

Copy link
Copy Markdown
Member

Adds tagged log lines ([slack-post], [reply-back], [dedupe], [state], [poll], [match], [forward]) throughout scripts/main.py so operators can trace, per cron run:

  • every chat.postMessage call (channel, thread_ts presence, length, preview, resulting ts, failure detail)
  • the reply-back lifecycle (debounce decision with idle-for seconds, status fetch, final-response length, success/error branch, conv close)
  • duplicate-message tracking (per-skip reason in _is_human_message, every append to bot_message_ts with running total, MAX_BOT_TS trim events)
  • state load/save with row-count summary
  • per-poll counters: candidates fetched, conversations spawned, replies forwarded

No behavioural changes — logging only. Verified python3 -m py_compile passes.

This PR was created by an AI agent (OpenHands) on behalf of @rbren.

…pe paths

Adds tagged log lines ([slack-post], [reply-back], [dedupe], [state],
[poll], [match], [forward]) throughout scripts/main.py so operators can
trace, per cron run:

* every chat.postMessage call (channel, thread_ts presence, length,
  preview, resulting ts, failure detail)
* the reply-back lifecycle (debounce decision with idle-for seconds,
  status fetch, final-response length, success/error branch, conv close)
* duplicate-message tracking (per-skip reason in _is_human_message,
  every append to bot_message_ts with running total, MAX_BOT_TS trim
  events)
* state load/save with row-count summary
* per-poll counters: candidates fetched, conversations spawned, replies
  forwarded

No behavioural changes -- logging only.

Co-authored-by: openhands <openhands@all-hands.dev>
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