opencode-pilot 0.2.0 → 0.2.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "opencode-pilot",
-  "version": "0.2.0",
+  "version": "0.2.1",
   "type": "module",
   "main": "plugin/index.js",
   "description": "Automation daemon for OpenCode - polls for work and spawns sessions",

package/plugin/index.js

@@ -48,7 +48,7 @@ export const PilotPlugin = async ({ $ }) => {
   } catch {
     // Not running, start it
     try {
-      await $`opencode-pilot start &`.quiet()
+      await $`npx opencode-pilot start &`.quiet()
     } catch {
       // Ignore start errors (maybe already starting)
     }

package/test/unit/paths.test.js

@@ -32,19 +32,4 @@ describe('Path naming consistency', () => {
         'server.js should not reference old opencode-ntfy name');
     });
   });
-  
-  describe('plist file', () => {
-    const plistPath = join(SERVICE_DIR, 'io.opencode.pilot.plist');
-    const content = readFileSync(plistPath, 'utf8');
-    
-    test('uses opencode-pilot label', () => {
-      assert.match(content, /io\.opencode\.pilot/,
-        'plist should use io.opencode.pilot label');
-    });
-    
-    test('does not reference old opencode-ntfy name', () => {
-      assert.doesNotMatch(content, /opencode-ntfy/,
-        'plist should not reference old opencode-ntfy name');
-    });
-  });
 });

package/service/io.opencode.pilot.plist

@@ -1,29 +0,0 @@
-<?xml version="1.0" encoding="UTF-8"?>
-<!DOCTYPE plist PUBLIC "-//Apple//DTD PLIST 1.0//EN" "http://www.apple.com/DTDs/PropertyList-1.0.dtd">
-<plist version="1.0">
-<dict>
-    <key>Label</key>
-    <string>io.opencode.pilot</string>
-    
-    <key>ProgramArguments</key>
-    <array>
-        <string>/usr/local/bin/node</string>
-        <string>/usr/local/opt/opencode-pilot/libexec/server.js</string>
-    </array>
-    
-    <key>RunAtLoad</key>
-    <true/>
-    
-    <key>KeepAlive</key>
-    <true/>
-    
-    <key>StandardOutPath</key>
-    <string>/usr/local/var/log/opencode-pilot.log</string>
-    
-    <key>StandardErrorPath</key>
-    <string>/usr/local/var/log/opencode-pilot.log</string>
-    
-    <key>WorkingDirectory</key>
-    <string>/usr/local/opt/opencode-pilot/libexec</string>
-</dict>
-</plist>

package/test/run_tests.bash

@@ -1,34 +0,0 @@
-#!/usr/bin/env bash
-#
-# Run all tests for opencode-ntfy
-#
-
-set -euo pipefail
-
-SCRIPT_DIR="$(cd "$(dirname "${BASH_SOURCE[0]}")" && pwd)"
-
-echo "========================================"
-echo "  opencode-pilot test suite"
-echo "========================================"
-echo ""
-
-FAILED=0
-
-for test_file in "$SCRIPT_DIR"/test_*.bash; do
-  if [[ -f "$test_file" ]] && [[ "$test_file" != *"test_helper.bash"* ]]; then
-    echo "----------------------------------------"
-    if ! bash "$test_file"; then
-      FAILED=1
-    fi
-    echo ""
-  fi
-done
-
-echo "========================================"
-if [[ $FAILED -eq 0 ]]; then
-  echo "  All test suites passed!"
-else
-  echo "  Some tests failed!"
-  exit 1
-fi
-echo "========================================"

package/test/test_actions.bash

@@ -1,248 +0,0 @@
-#!/usr/bin/env bash
-#
-# Tests for actions.js - Action system for starting sessions
-#
-
-set -euo pipefail
-
-SCRIPT_DIR="$(cd "$(dirname "${BASH_SOURCE[0]}")" && pwd)"
-source "$SCRIPT_DIR/test_helper.bash"
-
-SERVICE_DIR="$(dirname "$SCRIPT_DIR")/service"
-
-echo "Testing actions.js module..."
-echo ""
-
-# =============================================================================
-# File Structure Tests
-# =============================================================================
-
-test_actions_file_exists() {
-  assert_file_exists "$SERVICE_DIR/actions.js"
-}
-
-test_actions_js_syntax() {
-  if ! command -v node &>/dev/null; then
-    echo "SKIP: node not available"
-    return 0
-  fi
-  node --check "$SERVICE_DIR/actions.js" 2>&1 || {
-    echo "actions.js has syntax errors"
-    return 1
-  }
-}
-
-# =============================================================================
-# Export Tests
-# =============================================================================
-
-test_actions_exports_execute_action() {
-  grep -q "export.*function executeAction\|export.*executeAction" "$SERVICE_DIR/actions.js" || {
-    echo "executeAction export not found"
-    return 1
-  }
-}
-
-test_actions_exports_build_command() {
-  grep -q "export.*function buildCommand\|export.*buildCommand" "$SERVICE_DIR/actions.js" || {
-    echo "buildCommand export not found"
-    return 1
-  }
-}
-
-# =============================================================================
-# Implementation Tests
-# =============================================================================
-
-test_actions_uses_opencode_run() {
-  grep -q "opencode.*run" "$SERVICE_DIR/actions.js" || {
-    echo "opencode run command not found"
-    return 1
-  }
-}
-
-test_actions_prompt_template() {
-  grep -q "prompt_template\|promptTemplate\|buildPromptFromTemplate" "$SERVICE_DIR/actions.js" || {
-    echo "Prompt template support not found"
-    return 1
-  }
-}
-
-test_actions_calls_opencode() {
-  grep -q "opencode" "$SERVICE_DIR/actions.js" || {
-    echo "OpenCode invocation not found"
-    return 1
-  }
-}
-
-test_actions_builds_prompt_from_item() {
-  grep -q "title\|body" "$SERVICE_DIR/actions.js" || {
-    echo "Prompt building from item not found"
-    return 1
-  }
-}
-
-# =============================================================================
-# Functional Tests
-# =============================================================================
-
-test_actions_build_local_command() {
-  if ! command -v node &>/dev/null; then
-    echo "SKIP: node not available"
-    return 0
-  fi
-  
-  local result
-  result=$(node --experimental-vm-modules -e "
-    import { buildCommand } from './service/actions.js';
-    
-    const item = {
-      title: 'Fix bug #123',
-      html_url: 'https://github.com/org/repo/issues/123',
-      number: 123
-    };
-    
-    const config = {
-      repo_path: '~/code/myrepo',
-      session: { name: 'issue-{number}' }
-    };
-    
-    const cmd = buildCommand(item, config);
-    
-    if (!cmd.includes('opencode run')) {
-      console.log('FAIL: Local command should include opencode run');
-      process.exit(1);
-    }
-    // Check for cwd pattern (cd ~/code/myrepo && ...)
-    if (!cmd.includes('cd') || !cmd.includes('myrepo')) {
-      console.log('FAIL: Command should include cd to repo path, got: ' + cmd);
-      process.exit(1);
-    }
-    console.log('PASS');
-  " 2>&1) || {
-    echo "Functional test failed: $result"
-    return 1
-  }
-  
-  if ! echo "$result" | grep -q "PASS"; then
-    echo "$result"
-    return 1
-  fi
-}
-
-test_actions_session_name_template() {
-  if ! command -v node &>/dev/null; then
-    echo "SKIP: node not available"
-    return 0
-  fi
-  
-  local result
-  result=$(node --experimental-vm-modules -e "
-    import { buildSessionName } from './service/actions.js';
-    
-    const item = {
-      number: 42,
-      repo_key: 'myorg/backend',
-      repo_short: 'backend'
-    };
-    
-    const template = 'issue-{repo_short}-{number}';
-    const name = buildSessionName(template, item);
-    
-    if (name !== 'issue-backend-42') {
-      console.log('FAIL: Expected issue-backend-42, got ' + name);
-      process.exit(1);
-    }
-    console.log('PASS');
-  " 2>&1) || {
-    echo "Functional test failed: $result"
-    return 1
-  }
-  
-  if ! echo "$result" | grep -q "PASS"; then
-    echo "$result"
-    return 1
-  fi
-}
-
-test_actions_expand_template() {
-  if ! command -v node &>/dev/null; then
-    echo "SKIP: node not available"
-    return 0
-  fi
-  
-  local result
-  result=$(node --experimental-vm-modules -e "
-    import { expandTemplate } from './service/actions.js';
-    
-    const item = {
-      title: 'Fix bug',
-      number: 456
-    };
-    
-    const template = 'Issue #{number}: {title}';
-    const expanded = expandTemplate(template, item);
-    
-    if (expanded !== 'Issue #456: Fix bug') {
-      console.log('FAIL: Expected \"Issue #456: Fix bug\", got \"' + expanded + '\"');
-      process.exit(1);
-    }
-    console.log('PASS');
-  " 2>&1) || {
-    echo "Functional test failed: $result"
-    return 1
-  }
-  
-  if ! echo "$result" | grep -q "PASS"; then
-    echo "$result"
-    return 1
-  fi
-}
-
-# =============================================================================
-# Run Tests
-# =============================================================================
-
-echo "File Structure Tests:"
-
-for test_func in \
-  test_actions_file_exists \
-  test_actions_js_syntax
-do
-  run_test "${test_func#test_}" "$test_func"
-done
-
-echo ""
-echo "Export Tests:"
-
-for test_func in \
-  test_actions_exports_execute_action \
-  test_actions_exports_build_command
-do
-  run_test "${test_func#test_}" "$test_func"
-done
-
-echo ""
-echo "Implementation Tests:"
-
-for test_func in \
-  test_actions_uses_opencode_run \
-  test_actions_prompt_template \
-  test_actions_calls_opencode \
-  test_actions_builds_prompt_from_item
-do
-  run_test "${test_func#test_}" "$test_func"
-done
-
-echo ""
-echo "Functional Tests:"
-
-for test_func in \
-  test_actions_build_local_command \
-  test_actions_session_name_template \
-  test_actions_expand_template
-do
-  run_test "${test_func#test_}" "$test_func"
-done
-
-print_summary

package/test/test_cli.bash

@@ -1,157 +0,0 @@
-#!/usr/bin/env bash
-#
-# Tests for bin/opencode-pilot CLI
-#
-
-set -euo pipefail
-
-SCRIPT_DIR="$(cd "$(dirname "${BASH_SOURCE[0]}")" && pwd)"
-source "$SCRIPT_DIR/test_helper.bash"
-
-PROJECT_DIR="$(dirname "$SCRIPT_DIR")"
-CLI_PATH="$PROJECT_DIR/bin/opencode-pilot"
-
-echo "Testing opencode-pilot CLI..."
-echo ""
-
-# =============================================================================
-# File Structure Tests
-# =============================================================================
-
-test_cli_file_exists() {
-  assert_file_exists "$CLI_PATH"
-}
-
-test_cli_is_executable() {
-  [[ -x "$CLI_PATH" ]] || {
-    echo "CLI is not executable"
-    return 1
-  }
-}
-
-test_cli_js_syntax() {
-  if ! command -v node &>/dev/null; then
-    echo "SKIP: node not available"
-    return 0
-  fi
-  node --check "$CLI_PATH" 2>&1 || {
-    echo "CLI has syntax errors"
-    return 1
-  }
-}
-
-# =============================================================================
-# Help Command Tests
-# =============================================================================
-
-test_cli_help_shows_usage() {
-  "$CLI_PATH" help 2>&1 | grep -q "Usage:" || {
-    echo "help command should show Usage"
-    return 1
-  }
-}
-
-test_cli_help_shows_start_command() {
-  "$CLI_PATH" help 2>&1 | grep -q "start" || {
-    echo "help command should show start command"
-    return 1
-  }
-}
-
-test_cli_help_shows_status_command() {
-  "$CLI_PATH" help 2>&1 | grep -q "status" || {
-    echo "help command should show status command"
-    return 1
-  }
-}
-
-test_cli_help_shows_config_command() {
-  "$CLI_PATH" help 2>&1 | grep -q "config" || {
-    echo "help command should show config command"
-    return 1
-  }
-}
-
-test_cli_default_shows_help() {
-  "$CLI_PATH" 2>&1 | grep -q "Usage:" || {
-    echo "default should show usage"
-    return 1
-  }
-}
-
-test_cli_unknown_command_shows_error() {
-  local output
-  output=$("$CLI_PATH" unknowncommand 2>&1) || true
-  
-  echo "$output" | grep -q "Unknown command" || {
-    echo "unknown command should show error"
-    echo "Output: $output"
-    return 1
-  }
-}
-
-# =============================================================================
-# Status Command Tests
-# =============================================================================
-
-test_cli_status_shows_service_info() {
-  local output
-  output=$("$CLI_PATH" status 2>&1) || true
-  
-  echo "$output" | grep -qi "service" || {
-    echo "status should show service info"
-    echo "Output: $output"
-    return 1
-  }
-}
-
-test_cli_status_shows_config_info() {
-  local output
-  output=$("$CLI_PATH" status 2>&1) || true
-  
-  echo "$output" | grep -qi "config" || {
-    echo "status should show config info"
-    echo "Output: $output"
-    return 1
-  }
-}
-
-# =============================================================================
-# Run Tests
-# =============================================================================
-
-echo "File Structure Tests:"
-
-for test_func in \
-  test_cli_file_exists \
-  test_cli_is_executable \
-  test_cli_js_syntax
-do
-  run_test "${test_func#test_}" "$test_func"
-done
-
-echo ""
-echo "Help Command Tests:"
-
-for test_func in \
-  test_cli_help_shows_usage \
-  test_cli_help_shows_start_command \
-  test_cli_help_shows_status_command \
-  test_cli_help_shows_config_command \
-  test_cli_default_shows_help \
-  test_cli_unknown_command_shows_error
-do
-  run_test "${test_func#test_}" "$test_func"
-done
-
-echo ""
-echo "Status Command Tests:"
-
-for test_func in \
-  test_cli_status_shows_service_info \
-  test_cli_status_shows_config_info
-do
-  run_test "${test_func#test_}" "$test_func"
-done
-
-print_summary

package/test/test_helper.bash

@@ -1,140 +0,0 @@
-#!/usr/bin/env bash
-#
-# Test helper functions for opencode-ntfy tests
-#
-
-# Test counters
-TESTS_RUN=0
-TESTS_PASSED=0
-TESTS_FAILED=0
-
-# Colors
-RED='\033[0;31m'
-GREEN='\033[0;32m'
-YELLOW='\033[0;33m'
-NC='\033[0m' # No Color
-
-# Setup test environment
-setup_test_env() {
-  export TEST_DIR=$(mktemp -d)
-  export HOME="$TEST_DIR/home"
-  mkdir -p "$HOME"
-}
-
-# Cleanup test environment
-cleanup_test_env() {
-  if [[ -n "$TEST_DIR" ]] && [[ -d "$TEST_DIR" ]]; then
-    rm -rf "$TEST_DIR"
-  fi
-}
-
-# Assert equality
-assert_equals() {
-  local expected="$1"
-  local actual="$2"
-  local message="${3:-Values should be equal}"
-  
-  if [[ "$expected" == "$actual" ]]; then
-    return 0
-  else
-    echo "  Expected: $expected"
-    echo "  Actual:   $actual"
-    return 1
-  fi
-}
-
-# Assert string contains
-assert_contains() {
-  local haystack="$1"
-  local needle="$2"
-  local message="${3:-String should contain substring}"
-  
-  if [[ "$haystack" == *"$needle"* ]]; then
-    return 0
-  else
-    echo "  String: $haystack"
-    echo "  Should contain: $needle"
-    return 1
-  fi
-}
-
-# Assert command succeeds
-assert_success() {
-  local cmd="$1"
-  local message="${2:-Command should succeed}"
-  
-  if eval "$cmd" >/dev/null 2>&1; then
-    return 0
-  else
-    echo "  Command failed: $cmd"
-    return 1
-  fi
-}
-
-# Assert command fails
-assert_failure() {
-  local cmd="$1"
-  local message="${2:-Command should fail}"
-  
-  if eval "$cmd" >/dev/null 2>&1; then
-    echo "  Command should have failed: $cmd"
-    return 1
-  else
-    return 0
-  fi
-}
-
-# Assert file exists
-assert_file_exists() {
-  local file="$1"
-  local message="${2:-File should exist}"
-  
-  if [[ -f "$file" ]]; then
-    return 0
-  else
-    echo "  File does not exist: $file"
-    return 1
-  fi
-}
-
-# Run a single test
-run_test() {
-  local test_name="$1"
-  local test_func="$2"
-  
-  TESTS_RUN=$((TESTS_RUN + 1))
-  
-  printf "  %-50s " "$test_name"
-  
-  # Run test in subshell to isolate failures
-  local output
-  if output=$($test_func 2>&1); then
-    echo -e "${GREEN}PASS${NC}"
-    TESTS_PASSED=$((TESTS_PASSED + 1))
-  else
-    echo -e "${RED}FAIL${NC}"
-    if [[ -n "$output" ]]; then
-      echo "$output" | sed 's/^/    /'
-    fi
-    TESTS_FAILED=$((TESTS_FAILED + 1))
-  fi
-}
-
-# Print test summary
-print_summary() {
-  echo ""
-  echo "========================================"
-  echo "Tests run:    $TESTS_RUN"
-  echo -e "Tests passed: ${GREEN}$TESTS_PASSED${NC}"
-  if [[ $TESTS_FAILED -gt 0 ]]; then
-    echo -e "Tests failed: ${RED}$TESTS_FAILED${NC}"
-  else
-    echo "Tests failed: $TESTS_FAILED"
-  fi
-  echo "========================================"
-  
-  if [[ $TESTS_FAILED -gt 0 ]]; then
-    return 1
-  fi
-  return 0
-}