Skip to content

Improve Kanban notification readability - #31696

Closed
houenyang-momo wants to merge 1 commit into
NousResearch:mainfrom
houenyang-momo:fix/kanban-friendly-notifications
Closed

Improve Kanban notification readability#31696
houenyang-momo wants to merge 1 commit into
NousResearch:mainfrom
houenyang-momo:fix/kanban-friendly-notifications

Conversation

@houenyang-momo

Copy link
Copy Markdown
Contributor

Summary

  • make Kanban terminal notifications lead with friendly board labels instead of task IDs
  • include multi-line completion summaries instead of truncating to the first 200 chars
  • keep the task id as a low-priority Ref line for debugging

Test plan

  • /usr/local/lib/hermes-agent/venv/bin/python -m pytest tests/gateway/test_kanban_notifier.py -q
  • /usr/local/lib/hermes-agent/venv/bin/python -m py_compile gateway/run.py /opt/athena/hermes-home/profiles/athena-master/hooks/athena-topic-kanban-intake/handler.py

@houenyang-momo
houenyang-momo force-pushed the fix/kanban-friendly-notifications branch 2 times, most recently from d0e250f to 7803b70 Compare May 24, 2026 22:16
@teknium1

Copy link
Copy Markdown
Contributor

Thanks for the focused notification-readability improvement. The underlying problem remains on current main: completed notifications still truncate to the first 200 characters and lead with Kanban <task-id> (gateway/kanban_watchers.py:354-366).

Problems

  • The notifier was extracted from gateway/run.py into gateway/kanban_watchers.py by 1c68f6f81; the live formatter is now at gateway/kanban_watchers.py:337-407, so the replacement needs to be applied there.
  • Current main also sends status events (gateway/kanban_watchers.py:167, 394-398). The submitted formatter has no status branch and returns None for unsupported kinds, so porting it directly would silently suppress those notifications.

Suggested changes

  • Move the formatter/board-name collection into gateway/kanban_watchers.py and preserve a human-readable status message.
  • Add a notifier-path test for a named board and a status event; the submitted test exercises only the formatter helper.

Automated hermes-sweeper review.

@teknium1 teknium1 added sweeper:risk-message-delivery Sweeper risk: may drop, duplicate, misroute, or suppress messages sweeper:blast-moderate Sweeper blast radius: moderate — a subsystem or single platform labels Jul 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

comp/gateway Gateway runner, session dispatch, delivery P3 Low — cosmetic, nice to have sweeper:blast-moderate Sweeper blast radius: moderate — a subsystem or single platform sweeper:risk-message-delivery Sweeper risk: may drop, duplicate, misroute, or suppress messages type/feature New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants