Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@oscargavin
Copy link

Summary

  • Add diagnostic.ts with webhook stats, session state tracking, and heartbeat logging
  • Enhance telegram message processing with request timing and detailed logging
  • Add lane enqueue/dequeue logging to command queue
  • Track active embedded runs with state changes
  • Improve onboarding flow logging for iMessage and Signal
  • Add batch-gemini logging for memory operations

Why

This improves observability for debugging:

  • Message flow issues (where did a message get stuck?)
  • Stuck sessions (which session has been processing for too long?)
  • Performance problems (which webhooks are slow?)
  • Queue depth monitoring (is there a backlog building up?)

Test Plan

  • Service starts and runs without errors
  • Heartbeat logs appear every 30 seconds
  • Message processing logs timing information
  • Stuck session warnings appear after 2 minutes

🤖 Generated with Claude Code

- Add diagnostic.ts with webhook stats, session state tracking, heartbeat
- Enhance telegram bot-message.ts with request timing and logging
- Add lane logging to command-queue.ts
- Track active runs in pi-embedded-runner/runs.ts
- Improve onboarding flow logging for iMessage and Signal
- Add batch-gemini logging improvements

This improves observability for debugging message flow issues,
stuck sessions, and performance problems.
@steipete
Copy link
Contributor

Summary of what I changed in this workset:

  • Moved diagnostics emission into the shared dispatch path so all channels get message queued/processed + session state transitions (gated by diagnostics config).
  • Removed Telegram-only diagnostic logging; webhook + heartbeat are now diagnostics-gated.
  • Start/stop diagnostic heartbeat with gateway lifecycle; unref heartbeat interval.
  • Tightened diagnostic event typing; updated dispatch + Telegram tests.

Full gate: pnpm lint && pnpm build && pnpm test (passed).

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.

2 participants