Skip to content

feat: improve notification content for clarity and context - #17

Merged
athal7 merged 3 commits into
mainfrom
improve-notification-content
Jan 2, 2026
Merged

feat: improve notification content for clarity and context#17
athal7 merged 3 commits into
mainfrom
improve-notification-content

Conversation

@athal7

@athal7 athal7 commented Jan 2, 2026

Copy link
Copy Markdown
Owner

Summary

  • Change permission notification title from 'OpenCode: Permission' to 'Approve? (repoName)' for clearer call-to-action
  • Show actual command/pattern in permission notifications instead of cryptic description
  • Add truncation for commands over 100 characters
  • Change idle notification to 'Idle (repoName)' with descriptive message
  • Use OpenCode's directory parameter instead of process.cwd() to show actual repo name (not devcontainer temp dirs)
  • Handle edge cases: empty commands return empty string, empty directory falls back to 'unknown'

athal7 added 3 commits January 1, 2026 22:11
@athal7
feat: improve permission notification format for clarity
93fd08b 
- Change title from 'OpenCode: Permission' to 'Approve? (repoName)'
- Show actual command/pattern instead of just description
- Add truncation for commands over 100 characters
- Update function signature: description -> command, add repoName
@athal7
feat: improve idle notification format and use directory param
2a5beb0 
- Change idle title from 'OpenCode' to 'Idle (repoName)'
- Add descriptive message 'Session waiting for input'
- Use directory parameter from OpenCode instead of process.cwd()
- This ensures the actual repo name shows, not devcontainer temp dirs
@athal7
fix: handle edge cases in notification content
47fd28a 
- truncate() now returns empty string for undefined/null input
- repoName falls back to 'unknown' if directory is empty/root
@athal7
athal7 merged commit 17bcc65 into main Jan 2, 2026
1 check passed
athal7 added a commit that referenced this pull request Jan 2, 2026
@athal7
fix: update retry/error notification format to match PR #17 style
b10e0ce 
- Use repoName instead of undefined dir variable
- Change title format to "Retry (repoName)" / "Error (repoName)"
- Simplify message format for consistency
athal7 added a commit that referenced this pull request Jan 2, 2026
@athal7
feat(#7): add error and retry notification handlers
2e4a432 
* feat(#7): add error and retry notification handlers

- Add retry event handling with configurable first/Nth notification
- Add error event handling with debounce window
- Reset retry counter on non-retry status changes
- Log suppressed notifications for debugging
- Add debug logging mode via NTFY_DEBUG env var

* fix: use separate config file to avoid breaking OpenCode

OpenCode validates its config schema and rejects unknown keys like 'ntfy'.
Move config to ~/.config/opencode-ntfy/config.json instead of embedding
in opencode.json. Environment variables continue to work as before.

* fix: update retry/error notification format to match PR #17 style

- Use repoName instead of undefined dir variable
- Change title format to "Retry (repoName)" / "Error (repoName)"
- Simplify message format for consistency

* fix: update stale comment referencing old config location
github-actions Bot pushed a commit that referenced this pull request Jan 6, 2026
@semantic-release-bot
chore(release): 1.0.0 [skip ci]
f28fa77 
# 1.0.0 (2026-01-06)

### Bug Fixes

* **#30:** add proxy to enable Open Session button ([ff4957a](ff4957a)), closes [#30](#30) [#30](#30) [#27](#27) [#30](#30)
* **#34:** remove named export to prevent plugin double-loading ([#44](#44)) ([c57c28b](c57c28b)), closes [#34](#34)
* **#36:** use correct OpenCode API endpoint for sending messages ([1d89536](1d89536)), closes [#36](#36)
* **#37:** remove new session page - OpenCode API limitation ([f62bf3e](f62bf3e)), closes [#37](#37)
* **#40:** use dvh units and visualViewport API for mobile keyboard handling ([3df7f8e](3df7f8e)), closes [#40](#40) [#40](#40)
* **#41:** add reconnection mechanism for permission notifications ([#51](#51)) ([810a640](810a640)), closes [#41](#41)
* **#45:** add missing API proxy routes for agent and model selection ([038af90](038af90)), closes [#45](#45)
* **#48:** limit to 20 messages for performance, fix horizontal scroll ([e2b2484](e2b2484))
* **#48:** pin header, session title, and input box while messages scroll ([ac242de](ac242de)), closes [#48](#48)
* **#48:** revert to simple layout with scroll to bottom ([d332bbc](d332bbc)), closes [#48](#48)
* **#48:** skip tool-only messages and fix scroll timing ([311fe15](311fe15))
* **#48:** use column-reverse layout so newest message appears at bottom without scrolling ([cb47ecc](cb47ecc)), closes [#48](#48)
* **#48:** use longer delay for scroll to ensure DOM is rendered ([8308925](8308925)), closes [#48](#48)
* **#50:** add notification deduplication to prevent duplicate notifications ([9c09871](9c09871)), closes [#50](#50)
* **#50:** verify session ownership to prevent duplicate notifications ([#58](#58)) ([b89cdcc](b89cdcc)), closes [#50](#50)
* add npm install to CI test job for httpxy dependency ([3565f13](3565f13))
* add push-to and create-branch params for direct push ([#23](#23)) ([1c1b190](1c1b190))
* **ci:** remove registry-url from setup-node to avoid conflicting .npmrc ([81e3aa2](81e3aa2))
* **ci:** use npx semantic-release instead of action for OIDC support ([2bb747e](2bb747e))
* **ci:** use semantic-release-action to capture release outputs ([57074fd](57074fd))
* correct bats installation in devcontainer ([1180a74](1180a74))
* default max_concurrent to 1 for local execution ([48cb89b](48cb89b))
* escape backticks in markdown regex to avoid template literal syntax error ([9d54e45](9d54e45))
* filter issues by repo when polling ([a9788a6](a9788a6))
* find service dir in both dev and homebrew installs ([3b9675b](3b9675b))
* handle slow OpenCode API response in mobile UI ([5255698](5255698))
* improve mobile UI message display ([cb81b32](cb81b32))
* remove bash integration tests from CI, fix hostname.js ([4896ddc](4896ddc)), closes [#63](#63)
* run ocdc from repo directory instead of --repo flag ([98a8d32](98a8d32))
* show repo name with branch in devcontainer clone notifications ([67409b5](67409b5))
* spawn opencode run detached so polling doesn't block ([3e6c0b8](3e6c0b8))
* status reads config file and integration tests use temp HOME ([#24](#24)) ([9db9db3](9db9db3))
* suppress notifications on cancel and remove console output ([#20](#20)) ([3ebdad1](3ebdad1))
* **test:** relax timing assertion for CI environments ([d570a6e](d570a6e))
* **test:** update service test for opencode-pilot rename ([869f768](869f768))
* treat breaking changes as minor bumps in 0.x ([8f82c73](8f82c73))
* **ui:** add padding to bottom of messages list ([079808c](079808c))
* **ui:** fix scroll jumping and tool call status display ([82aafad](82aafad))
* **ui:** group consecutive messages by role to reduce visual clutter ([6e8c71a](6e8c71a))
* **ui:** include 'all' mode agents in mobile UI selector ([5ae2092](5ae2092))
* **ui:** prevent double-send by checking input after send completes ([3ff164e](3ff164e))
* **ui:** properly hide header on keyboard open and adjust spacing ([701f1ee](701f1ee))
* **ui:** reduce vertical spacing in markdown lists ([2e27722](2e27722))
* **ui:** strip trailing colon when tool calls are hidden ([fdde8a2](fdde8a2))
* **ui:** synchronize input clear, button text, and button state ([e6eb12e](e6eb12e))
* update config path from opencode-ntfy to opencode-pilot ([5109238](5109238))
* update homebrew formula name in CI workflow ([bdb553b](bdb553b))
* update remaining opencode-ntfy references to opencode-pilot ([2fce9fe](2fce9fe))
* use cwd instead of -d flag for opencode run ([e80eb25](e80eb25))
* use extra_plugins to install @semantic-release/npm v13 for trusted publishing ([a30c29a](a30c29a))
* use Node.js 22 (required by semantic-release) ([f23f249](f23f249))
* use npm install instead of npm ci (no package-lock.json) ([#26](#26)) ([c2c4908](c2c4908))
* use opencode run for local action (non-interactive) ([f7883a1](f7883a1))
* use opencode run for non-interactive execution ([693fe06](693fe06))
* use realpath for main module detection in service ([b9d7f3f](b9d7f3f))
* use sessionID (capital ID) from OpenCode events ([2f3847a](2f3847a))

### Features

* **#13:** separate callback server as brew service ([e8dcfcb](e8dcfcb)), closes [#13](#13) [#13](#13) [#13](#13) [#13](#13) [#13](#13) [#13](#13) [#13](#13) [#13](#13) [#13](#13)
* **#1:** add project scaffolding and CI workflows ([1afcebd](1afcebd)), closes [#1](#1)
* **#27:** add Open Session action button to idle notifications ([#28](#28)) ([5948aa4](5948aa4)), closes [#27](#27) [#27](#27) [#27](#27)
* **#29:** add optional debug logging to plugin ([#68](#68)) ([a0c4056](a0c4056)), closes [#29](#29)
* **#2:** add config parsing and idle notifications ([2b19d94](2b19d94)), closes [#2](#2)
* **#30:** mobile session UI with new session page and HTTPS ([#33](#33)) ([f0428f1](f0428f1)), closes [#30](#30) [#30](#30) [#30](#30)
* **#3:** implement ntfy HTTP client with all notification types ([#10](#10)) ([e81d087](e81d087)), closes [#3](#3) [#3](#3) [#3](#3) [#3](#3)
* **#45:** add agent and model selection to conversation UI ([3616559](3616559)), closes [#45](#45)
* **#45:** add agent and model selection to conversation UI ([#59](#59)) ([3cae588](3cae588)), closes [#45](#45) [#45](#45) [#45](#45)
* **#46:** track idle state per conversation, not globally ([#47](#47)) ([d4b707a](d4b707a)), closes [#46](#46)
* **#48:** add session title and conversation history to mobile UI ([#49](#49)) ([16e2711](16e2711)), closes [#48](#48) [#48](#48) [#48](#48)
* **#4:** implement HTTP callback server for permission responses ([#11](#11)) ([a9325e2](a9325e2)), closes [#4](#4) [#4](#4) [#4](#4)
* **#53:** backup opencode.json before modification in setup ([#57](#57)) ([e49defd](e49defd)), closes [#53](#53)
* **#54:** add polling system for automated issue/PR processing ([999e24c](999e24c)), closes [#54](#54)
* **#66:** add default sources when none specified ([84490db](84490db)), closes [#66](#66)
* **#6:** add config file support and callback host configuration ([#12](#12)) ([0b459b4](0b459b4)), closes [#6](#6) [#6](#6) [#6](#6) [#6](#6) [#6](#6)
* **#7:** add error and retry notification handlers ([2e4a432](2e4a432)), closes [#7](#7) [#7](#7) [#17](#17)
* add CLI start command and plugin auto-start ([38d91f4](38d91f4))
* add CLI with setup and status commands ([#19](#19)) ([369eeb5](369eeb5))
* add opt-in debug logging to poll-service (PILOT_DEBUG=true) ([1a11bc0](1a11bc0))
* add polling for real-time updates in mobile UI ([49a8793](49a8793))
* add session link to error notifications ([9d4634a](9d4634a))
* add workflow to update Homebrew formula on release ([#22](#22)) ([4fdb01c](4fdb01c))
* **config:** add idleNotify option to disable idle notifications ([db74c1d](db74c1d))
* improve notification content for clarity and context ([#17](#17)) ([17bcc65](17bcc65))
* install opencode CLI in devcontainer ([dca4355](dca4355))
* migrate from Homebrew to npm distribution ([71b6c0a](71b6c0a))
* start opencode session inside devcontainer ([c0c45a8](c0c45a8))
* **ui:** display tool calls with status indicators ([5ea883d](5ea883d))
* **ui:** hide header on scroll and allow message queuing ([b6a5f6d](b6a5f6d))
* use view actions for private callbacks over Tailscale ([#21](#21)) ([eeaf4d2](eeaf4d2))

### Performance Improvements

* make debug logger async to avoid blocking render thread ([f5c0867](f5c0867))
@github-actions github-actions Bot mentioned this pull request Aug 10, 2026
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@athal7