Skip to content

docs: explain /session-monitor is required for the pet's Alerts tab - #3

Merged
mingeonho1 merged 2 commits into
mainfrom
docs/session-monitor-connect
Oct 27, 2026
Merged

mingeonho1 merged 2 commits into
mainfrom
docs/session-monitor-connect

Conversation

@mingeonho1

Copy link
Copy Markdown
Owner

Closes #2

Summary

  • README.md / README.ko.md — new "Connect the pet to your sessions" subsection under Session monitoring: the Alerts tab stays empty until a monitor session runs (hooks only log to events.jsonl; the monitor sends 🟡/🔴 via notify.py), where to run it (CLI or desktop Code tab, one dedicated session, excluded from its own alerts), re-run after session/app restarts, hooks apply only to sessions opened after install, recommended /model opus + effort high.
  • Troubleshooting — new row "Pet is running but the Alerts tab is empty" → check monitor-session marker and heartbeat mtime.
  • Install-via-link flow — most users just hand Claude the repo link, so:
    • README (both languages) has a short note telling Claude to follow INSTALL.md end to end, including the wrap-up.
    • INSTALL.md §6 Wrap-up is now a checklist Claude must always tell the user after install.
  • CHANGELOG — Unreleased entry.

Test plan

  • python3 -m unittest discover -s tests (43 tests OK)
  • Render check on GitHub (README tables/links)

🤖 Generated with Claude Code

- README/README.ko: add "Connect the pet to your sessions" subsection and an
  "Alerts tab is empty" troubleshooting row; add a note for Claude to follow
  INSTALL.md end to end when handed the repo link
- INSTALL.md: wrap-up is now a checklist Claude must always tell the user
- CHANGELOG: Unreleased entry

Closes #2

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@mingeonho1 mingeonho1 added the documentation Improvements or additions to documentation label Oct 27, 2026
@mingeonho1 mingeonho1 self-assigned this Oct 27, 2026
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@mingeonho1
mingeonho1 merged commit 40fc3f7 into main Oct 27, 2026
10 checks passed
@mingeonho1
mingeonho1 deleted the docs/session-monitor-connect branch October 27, 2026 01:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Docs: explain that /session-monitor must be running for the pet/Alerts tab to receive alerts

1 participant