dondon-notify 0.1.2 → 0.1.3

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dondon-notify",
-  "version": "0.1.2",
+  "version": "0.1.3",
   "description": "Native OS notifications for OpenCode - know when tasks complete",
   "type": "module",
   "main": "src/notify.ts",

package/src/notify.ts

@@ -258,7 +258,12 @@ async function getFrontmostApp(): Promise<string | null> {
 
 async function detectTerminalInfo(config: NotifyConfig): Promise<TerminalInfo> {
 	// Use config override if provided
-	const terminalName = config.terminal || detectTerminal() || null
+	let terminalName = config.terminal || detectTerminal() || null
+
+	// Normalize Kitty name (often xterm-kitty)
+	if (terminalName?.toLowerCase().includes("kitty")) {
+		terminalName = "kitty"
+	}
 
 	if (!terminalName) {
 		return { name: null, bundleId: null, processName: null, windowId: null }
@@ -353,8 +358,8 @@ interface NotificationOptions {
 function sendNotification(options: NotificationOptions): void {
 	const { title, message, sound, terminalInfo } = options
 
-	// Use Kitty notifications if available and on supported platform
-	if (terminalInfo.name?.toLowerCase() === "kitty" && process.platform === "linux") {
+	// Use Kitty notifications if available
+	if (terminalInfo.name?.toLowerCase() === "kitty") {
 		// Use default Kitty settings (simpler approach)
 		sendKittyNotification({
 			title,
@@ -366,10 +371,12 @@ function sendNotification(options: NotificationOptions): void {
 	}
 
 	// Fall back to system notifications
-	const notifyOptions: Record<string, unknown> = {
+	const notifyOptions: any = {
 		title,
 		message,
 		sound,
+		appName: "OpenCode",
+		id: "opencode-notify",
 	}
 
 	// macOS-specific: click notification to focus terminal

package/test/test-kitty.js

@@ -5,7 +5,8 @@
  * This script will test the Kitty-specific features we've added
  */
 
-const { execSync } = require('child_process')
+import { execSync } from 'node:child_process'
+import process from 'node:process'
 
 console.log('Testing Kitty notification improvements...\n')
 
@@ -40,8 +41,8 @@ console.log('\n4. Platform Detection:')
 console.log(`   Platform: ${process.platform}`)
 console.log(`   Architecture: ${process.arch}`)
 
-// Test 5: Test if we can detect focus (simplified)
-console.log('\n5. Focus Detection Test:')
+// Test 6: Test if we can detect focus (simplified)
+console.log('\n6. Focus Detection Test:')
 if (process.platform === 'linux' && windowId) {
   console.log(`   Would test focus detection for Kitty window ${windowId}`)
   console.log(`   (This would use xprop to check _NET_ACTIVE_WINDOW)`)
@@ -51,23 +52,27 @@ if (process.platform === 'linux' && windowId) {
   console.log(`   Focus detection not available on this platform`)
 }
 
-console.log('\n6. Kitty Notification Test:')
-if (isKitty && process.platform === 'linux') {
+console.log('\n7. Kitty Notification Test:')
+if (isKitty) {
   console.log('   Sending test notification via Kitty OSC 99...')
   
   try {
     // Send a test Kitty notification
     const title = 'OpenCode Notify Test'
     const message = 'Kitty notification integration working!'
-    const escapeSequence = `\x1b]99;i=1:d=1;p=title;${title}\x1b\\\x1b]99;i=1:d=1;p=body;${message}\x1b\\`
     
-    process.stdout.write(escapeSequence)
+    // Protocol: \x1b]99;<metadata>;<payload>\x1b\\
+    // Metadata: d=0 means more follows, d=1 means done
+    const titleSeq = `\x1b]99;i=1:d=0:p=title;${title}\x1b\\`
+    const bodySeq = `\x1b]99;i=1:d=1:p=body;${message}\x1b\\`
+    
+    process.stdout.write(titleSeq + bodySeq)
     console.log('   ✓ Test notification sent!')
   } catch (error) {
     console.log(`   ✗ Failed to send test notification: ${error.message}`)
   }
 } else {
-  console.log('   Skipping - not in Kitty on Linux')
+  console.log('   Skipping - not in Kitty')
 }
 
 console.log('\nTest completed!')