opencode-pilot 0.1.0 → 0.2.0

package/test/unit/paths.test.js

@@ -1,7 +1,7 @@
 /**
  * Tests for consistent path naming across the codebase.
  * 
- * These tests ensure all config/socket paths use "opencode-pilot" 
+ * These tests ensure all config paths use "opencode-pilot" 
  * and not the old "opencode-ntfy" name.
  */
 
@@ -14,64 +14,37 @@ import { fileURLToPath } from 'url';
 const __filename = fileURLToPath(import.meta.url);
 const __dirname = dirname(__filename);
 const ROOT_DIR = join(__dirname, '..', '..');
-const PLUGIN_DIR = join(ROOT_DIR, 'plugin');
 const SERVICE_DIR = join(ROOT_DIR, 'service');
 
 describe('Path naming consistency', () => {
   
-  describe('config.js', () => {
-    const configPath = join(PLUGIN_DIR, 'config.js');
-    const content = readFileSync(configPath, 'utf8');
+  describe('server.js', () => {
+    const serverPath = join(SERVICE_DIR, 'server.js');
+    const content = readFileSync(serverPath, 'utf8');
     
     test('uses opencode-pilot config path', () => {
-      assert.match(content, /opencode-pilot.*config\.yaml/, 
-        'config.js should reference opencode-pilot config path');
-    });
-    
-    test('does not reference old opencode-ntfy path', () => {
-      assert.doesNotMatch(content, /opencode-ntfy/,
-        'config.js should not reference old opencode-ntfy name');
-    });
-  });
-  
-  describe('logger.js', () => {
-    const loggerPath = join(PLUGIN_DIR, 'logger.js');
-    const content = readFileSync(loggerPath, 'utf8');
-    
-    test('uses opencode-pilot debug log path', () => {
-      assert.match(content, /opencode-pilot.*debug\.log/,
-        'logger.js should reference opencode-pilot debug log path');
+      assert.match(content, /opencode-pilot.*config\.yaml|opencode-pilot/,
+        'server.js should reference opencode-pilot paths');
     });
     
-    test('does not reference old opencode-ntfy path', () => {
+    test('does not reference old opencode-ntfy name', () => {
       assert.doesNotMatch(content, /opencode-ntfy/,
-        'logger.js should not reference old opencode-ntfy name');
+        'server.js should not reference old opencode-ntfy name');
     });
   });
   
-  describe('server.js', () => {
-    const serverPath = join(SERVICE_DIR, 'server.js');
-    const content = readFileSync(serverPath, 'utf8');
-    
-    test('uses opencode-pilot socket path', () => {
-      assert.match(content, /opencode-pilot\.sock/,
-        'server.js should reference opencode-pilot socket path');
-    });
+  describe('plist file', () => {
+    const plistPath = join(SERVICE_DIR, 'io.opencode.pilot.plist');
+    const content = readFileSync(plistPath, 'utf8');
     
-    test('uses opencode-pilot config path', () => {
-      assert.match(content, /opencode-pilot.*config\.json|opencode-pilot/,
-        'server.js should reference opencode-pilot paths');
+    test('uses opencode-pilot label', () => {
+      assert.match(content, /io\.opencode\.pilot/,
+        'plist should use io.opencode.pilot label');
     });
-  });
-  
-  describe('index.js (plugin entry)', () => {
-    const indexPath = join(PLUGIN_DIR, 'index.js');
-    const content = readFileSync(indexPath, 'utf8');
     
-    test('does not reference old opencode-ntfy name in comments', () => {
-      // Allow "ntfy" alone (the service name) but not "opencode-ntfy"
+    test('does not reference old opencode-ntfy name', () => {
       assert.doesNotMatch(content, /opencode-ntfy/,
-        'index.js should not reference old opencode-ntfy name');
+        'plist should not reference old opencode-ntfy name');
     });
   });
 });

package/test/unit/plugin.test.js

@@ -0,0 +1,46 @@
+/**
+ * Tests for plugin/index.js - Auto-start plugin for OpenCode
+ */
+
+import { describe, it, mock } from "node:test";
+import assert from "node:assert";
+
+describe("plugin/index.js", () => {
+  describe("exports", () => {
+    it("exports PilotPlugin as named export", async () => {
+      const plugin = await import("../../plugin/index.js");
+      assert.strictEqual(typeof plugin.PilotPlugin, "function");
+    });
+
+    it("exports PilotPlugin as default export", async () => {
+      const plugin = await import("../../plugin/index.js");
+      assert.strictEqual(plugin.default, plugin.PilotPlugin);
+    });
+  });
+
+  describe("PilotPlugin", () => {
+    it("returns empty hooks object", async () => {
+      const plugin = await import("../../plugin/index.js");
+      
+      // Mock context with $ shell function
+      const mockShell = mock.fn(() => Promise.resolve());
+      mockShell.quiet = mock.fn(() => Promise.resolve());
+      const ctx = { $: mockShell };
+      
+      // Call plugin - it will try to fetch health endpoint which will fail
+      // in test environment, so it will try to start daemon
+      const hooks = await plugin.PilotPlugin(ctx);
+      
+      // Should return empty hooks object (no event handlers)
+      assert.deepStrictEqual(hooks, {});
+    });
+
+    it("is an async function", async () => {
+      const plugin = await import("../../plugin/index.js");
+      assert.strictEqual(
+        plugin.PilotPlugin.constructor.name,
+        "AsyncFunction"
+      );
+    });
+  });
+});

package/plugin/config.js

@@ -1,76 +0,0 @@
-// Configuration management for opencode-pilot
-// Reads from ~/.config/opencode-pilot/config.yaml
-//
-// Example config file (~/.config/opencode-pilot/config.yaml):
-// notifications:
-//   topic: my-secret-topic
-//   server: https://ntfy.sh
-//   idle_delay_ms: 300000
-//   debug: true
-
-import { readFileSync, existsSync } from 'fs'
-import { join } from 'path'
-import { homedir } from 'os'
-import YAML from 'yaml'
-
-const DEFAULT_CONFIG_PATH = join(homedir(), '.config', 'opencode-pilot', 'config.yaml')
-
-/**
- * Load configuration from config file
- * @param {string} [configPath] - Optional path to config file (for testing)
- */
-export function loadConfig(configPath) {
-  const actualPath = configPath || DEFAULT_CONFIG_PATH
-  
-  // Load config.yaml if it exists
-  let fileConfig = {}
-  if (existsSync(actualPath)) {
-    try {
-      const content = readFileSync(actualPath, 'utf8')
-      const parsed = YAML.parse(content)
-      // Extract notifications section
-      fileConfig = parsed?.notifications || {}
-    } catch (err) {
-      // Silently ignore parse errors
-    }
-  }
-
-  // Helper to get value with default
-  const get = (key, defaultValue) => {
-    if (fileConfig[key] !== undefined && fileConfig[key] !== '') {
-      return fileConfig[key]
-    }
-    return defaultValue
-  }
-
-  // Helper to parse boolean
-  const getBool = (key, defaultValue) => {
-    const value = get(key, undefined)
-    if (value === undefined) return defaultValue
-    if (typeof value === 'boolean') return value
-    return String(value).toLowerCase() !== 'false' && String(value) !== '0'
-  }
-
-  // Helper to parse int
-  const getInt = (key, defaultValue) => {
-    const value = get(key, undefined)
-    if (value === undefined) return defaultValue
-    if (typeof value === 'number') return value
-    const parsed = parseInt(String(value), 10)
-    return isNaN(parsed) ? defaultValue : parsed
-  }
-
-  return {
-    topic: get('topic', null),
-    server: get('server', 'https://ntfy.sh'),
-    authToken: get('token', null),
-    idleDelayMs: getInt('idle_delay_ms', 300000),
-    errorNotify: getBool('error_notify', true),
-    errorDebounceMs: getInt('error_debounce_ms', 60000),
-    retryNotifyFirst: getBool('retry_notify_first', true),
-    retryNotifyAfter: getInt('retry_notify_after', 3),
-    idleNotify: getBool('idle_notify', true),
-    debug: getBool('debug', false),
-    debugPath: get('debug_path', null),
-  }
-}

package/plugin/logger.js

@@ -1,125 +0,0 @@
-// Debug logging module for opencode-pilot plugin
-// Writes to ~/.config/opencode-pilot/debug.log when enabled via NTFY_DEBUG=true or config.debug
-//
-// Usage:
-//   import { initLogger, debug } from './logger.js'
-//   initLogger({ debug: true, debugPath: '/custom/path.log' })
-//   debug('Event received', { type: 'session.status', status: 'idle' })
-
-import { appendFileSync, existsSync, mkdirSync, statSync, unlinkSync } from 'fs'
-import { join, dirname } from 'path'
-import { homedir } from 'os'
-
-// Maximum log file size before rotation (1MB)
-export const MAX_LOG_SIZE = 1024 * 1024
-
-// Default log path
-const DEFAULT_LOG_PATH = join(homedir(), '.config', 'opencode-pilot', 'debug.log')
-
-// Module state
-let enabled = false
-let logPath = DEFAULT_LOG_PATH
-
-/**
- * Initialize the logger with configuration
- * @param {Object} options
- * @param {boolean} [options.debug] - Enable debug logging
- * @param {string} [options.debugPath] - Custom log file path
- */
-export function initLogger(options = {}) {
-  // Check environment variables first, then options
-  const envDebug = process.env.NTFY_DEBUG
-  const envDebugPath = process.env.NTFY_DEBUG_PATH
-  
-  // Enable if NTFY_DEBUG is set to any truthy value (not 'false' or '0')
-  if (envDebug !== undefined && envDebug !== '' && envDebug !== 'false' && envDebug !== '0') {
-    enabled = true
-  } else if (options.debug !== undefined) {
-    enabled = Boolean(options.debug)
-  } else {
-    enabled = false
-  }
-  
-  // Set log path (env var takes precedence)
-  if (envDebugPath) {
-    logPath = envDebugPath
-  } else if (options.debugPath) {
-    logPath = options.debugPath
-  } else {
-    logPath = DEFAULT_LOG_PATH
-  }
-  
-  // Create directory if it doesn't exist
-  if (enabled) {
-    try {
-      const dir = dirname(logPath)
-      if (!existsSync(dir)) {
-        mkdirSync(dir, { recursive: true })
-      }
-    } catch {
-      // Silently ignore directory creation errors
-    }
-  }
-}
-
-/**
- * Write a debug log entry
- * @param {string} message - Log message
- * @param {Object} [data] - Optional data to include
- */
-export function debug(message, data) {
-  if (!enabled) {
-    return
-  }
-  
-  try {
-    // Check file size and rotate if needed
-    rotateIfNeeded()
-    
-    // Format log entry with ISO 8601 timestamp
-    const timestamp = new Date().toISOString()
-    let entry = `[${timestamp}] ${message}`
-    
-    // Append data if provided
-    if (data !== undefined) {
-      if (typeof data === 'object') {
-        entry += ' ' + JSON.stringify(data)
-      } else {
-        entry += ' ' + String(data)
-      }
-    }
-    
-    entry += '\n'
-    
-    // Ensure directory exists
-    const dir = dirname(logPath)
-    if (!existsSync(dir)) {
-      mkdirSync(dir, { recursive: true })
-    }
-    
-    // Append to log file
-    appendFileSync(logPath, entry)
-  } catch {
-    // Silently ignore write errors to avoid affecting the plugin
-  }
-}
-
-/**
- * Rotate log file if it exceeds MAX_LOG_SIZE
- */
-function rotateIfNeeded() {
-  try {
-    if (!existsSync(logPath)) {
-      return
-    }
-    
-    const stats = statSync(logPath)
-    if (stats.size > MAX_LOG_SIZE) {
-      // Simple rotation: just truncate the file
-      // For more sophisticated rotation, could rename to .old first
-      unlinkSync(logPath)
-    }
-  } catch {
-    // Silently ignore rotation errors
-  }
-}

package/plugin/notifier.js

@@ -1,110 +0,0 @@
-// ntfy HTTP client for sending notifications
-
-import { debug } from './logger.js'
-
-// Deduplication cache: track recently sent notifications to prevent duplicates
-// Key: hash of notification content, Value: timestamp
-const recentNotifications = new Map()
-const DEDUPE_WINDOW_MS = 5000 // 5 seconds
-
-/**
- * Generate a simple hash for deduplication
- * @param {string} str - String to hash
- * @returns {string} Simple hash
- */
-function simpleHash(str) {
-  let hash = 0
-  for (let i = 0; i < str.length; i++) {
-    const char = str.charCodeAt(i)
-    hash = ((hash << 5) - hash) + char
-    hash = hash & hash // Convert to 32bit integer
-  }
-  return hash.toString(36)
-}
-
-/**
- * Check if notification was recently sent (for deduplication)
- * @param {string} key - Deduplication key
- * @returns {boolean} True if duplicate
- */
-function isDuplicate(key) {
-  const now = Date.now()
-  
-  // Clean up old entries
-  for (const [k, timestamp] of recentNotifications) {
-    if (now - timestamp > DEDUPE_WINDOW_MS) {
-      recentNotifications.delete(k)
-    }
-  }
-  
-  if (recentNotifications.has(key)) {
-    return true
-  }
-  
-  recentNotifications.set(key, now)
-  return false
-}
-
-/**
- * Build headers for ntfy requests
- * @param {string} [authToken] - Optional ntfy access token for Bearer auth
- * @returns {Object} Headers object
- */
-function buildHeaders(authToken) {
-  const headers = {
-    'Content-Type': 'application/json',
-  }
-  
-  if (authToken) {
-    headers['Authorization'] = `Bearer ${authToken}`
-  }
-  
-  return headers
-}
-
-/**
- * Send a basic notification to ntfy
- * @param {Object} options
- * @param {string} options.server - ntfy server URL
- * @param {string} options.topic - ntfy topic name
- * @param {string} options.title - Notification title
- * @param {string} options.message - Notification message
- * @param {number} [options.priority] - Priority (1-5, default 3)
- * @param {string[]} [options.tags] - Emoji tags
- * @param {string} [options.authToken] - Optional ntfy access token for protected topics
- */
-export async function sendNotification({ server, topic, title, message, priority, tags, authToken }) {
-  // Deduplicate: skip if same notification sent recently
-  const dedupeKey = simpleHash(`${topic}:${title}:${message}`)
-  if (isDuplicate(dedupeKey)) {
-    debug(`Notification skipped (duplicate): ${title}`)
-    return
-  }
-
-  const body = {
-    topic,
-    title,
-    message,
-  }
-
-  // Add optional fields only if provided
-  if (priority !== undefined) {
-    body.priority = priority
-  }
-  if (tags && tags.length > 0) {
-    body.tags = tags
-  }
-
-  try {
-    debug(`Notification sending: ${title}`)
-    const response = await fetch(server, {
-      method: 'POST',
-      headers: buildHeaders(authToken),
-      body: JSON.stringify(body),
-    })
-    debug(`Notification sent: ${title} (status=${response.status})`)
-  } catch (error) {
-    debug(`Notification failed: ${title} (error=${error.message})`)
-    // Silently ignore - errors here shouldn't affect the user
-  }
-}