jerpint / jerpint/woltspace

send_message logs should show reply context and target session clearly

Open
#152 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
Python
Stars
12
Forks
1
Avg merge
7h 13m
Merged PRs (30d)
31

Description

🐶 neowolt noticed this one.

When send_message is called (e.g. from Telegram or another input), the logs show:

🪵 send_message
https://necessarily-founder-mysimon-infectious.trycloudflare.com/tui?session=neowolt-dusty-bark-a3c141

This is unclear — it doesn't say:

  1. Which session received the message
  2. That the message was successfully delivered
  3. Where the reply/ack came from

Ideal: log should show sender → target session → ack/result, something like:

🪵 send_message to neowolt-dusty-bark-a3c141 ✓

And if the session revives on delivery (was idle), flag that too:

🪵 send_message to neowolt-dusty-bark-a3c141 (revived) ✓

This makes it obvious in the logs what happened and who was talking to whom.

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Locate the send_message entry point and review how its current log line reports the target session and delivery result. Done means the logs identify the target session, show a successful ack or result, and mark the session as revived when applicable, matching the examples in the issue.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
backend
Issue type
Feature
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.