feat: add desktop/system notification tool#320
Conversation
|
Closing in favor of a simpler approach: a config-driven terminal bell that fires automatically when the agent finishes a response, rather than a tool the agent has to call. Implementation: Thanks for the idea and the clean implementation — the platform detection and security patterns were well done. |
✅ Completed: Friction Button Visual State DifferentiationChanges MadeAdded semantic color coding and improved visual state differentiation to all friction buttons on the Adjustments page. Implementation Details1. Semantic Color Coding (each friction type has its own color):
2. Visual States (per acceptance criteria):
3. Accessibility Improvements:
Verification
Screenshot EvidenceMEDIA:/Users/andrue/.hermes/browser_screenshots/browser_screenshot_e1838ba814fb429fa8177f07e360ccdd.png Commit: |
Closes #318
Adds two new tools to the
notificationtoolset:notify— sends a native desktop notification (Linux: notify-send, macOS: osascript, Windows: PowerShell toast)notify_sound— plays a system alert sound (Linux: paplay/aplay, macOS: afplay, Windows: SystemSounds)Both tools fall back to terminal bell (
\a) when running over SSH or in headless environments.Security: no string interpolation into shell commands — title/message passed as list arguments or via environment variables.
Changes:
tools/notification_tool.py— new tool moduletoolsets.py— addednotificationtoolsetmodel_tools.py— registered notification_tool module