opencode-pilot 0.1.0 → 0.2.0

package/test/run_tests.bash

@@ -8,7 +8,7 @@ set -euo pipefail
 SCRIPT_DIR="$(cd "$(dirname "${BASH_SOURCE[0]}")" && pwd)"
 
 echo "========================================"
-echo "  opencode-ntfy test suite"
+echo "  opencode-pilot test suite"
 echo "========================================"
 echo ""
 

package/test/test_actions.bash

@@ -62,7 +62,7 @@ test_actions_uses_opencode_run() {
 }
 
 test_actions_prompt_template() {
-  grep -q "prompt_template\|promptTemplate" "$SERVICE_DIR/actions.js" || {
+  grep -q "prompt_template\|promptTemplate\|buildPromptFromTemplate" "$SERVICE_DIR/actions.js" || {
     echo "Prompt template support not found"
     return 1
   }
@@ -104,7 +104,7 @@ test_actions_build_local_command() {
     
     const config = {
       repo_path: '~/code/myrepo',
-      session: { name_template: 'issue-{number}' }
+      session: { name: 'issue-{number}' }
     };
     
     const cmd = buildCommand(item, config);
@@ -130,7 +130,7 @@ test_actions_build_local_command() {
   fi
 }
 
-test_actions_prompt_template_expansion() {
+test_actions_session_name_template() {
   if ! command -v node &>/dev/null; then
     echo "SKIP: node not available"
     return 0
@@ -138,33 +138,19 @@ test_actions_prompt_template_expansion() {
   
   local result
   result=$(node --experimental-vm-modules -e "
-    import { buildCommand } from './service/actions.js';
+    import { buildSessionName } from './service/actions.js';
     
     const item = {
-      title: 'Fix bug',
-      body: 'Details here',
-      number: 456
-    };
-    
-    // Config with custom prompt template (e.g., for devcontainer)
-    const config = {
-      repo_path: '~/code/myrepo',
-      session: { 
-        name_template: 'issue-{number}',
-        prompt_template: '/devcontainer issue-{number}\n\n{title}\n\n{body}'
-      }
+      number: 42,
+      repo_key: 'myorg/backend',
+      repo_short: 'backend'
     };
     
-    const cmd = buildCommand(item, config);
+    const template = 'issue-{repo_short}-{number}';
+    const name = buildSessionName(template, item);
     
-    // Should include the expanded template
-    if (!cmd.includes('/devcontainer issue-456')) {
-      console.log('FAIL: Command should include expanded prompt template');
-      console.log('Got: ' + cmd);
-      process.exit(1);
-    }
-    if (!cmd.includes('Fix bug')) {
-      console.log('FAIL: Command should include title from template');
+    if (name !== 'issue-backend-42') {
+      console.log('FAIL: Expected issue-backend-42, got ' + name);
       process.exit(1);
     }
     console.log('PASS');
@@ -179,7 +165,7 @@ test_actions_prompt_template_expansion() {
   fi
 }
 
-test_actions_session_name_template() {
+test_actions_expand_template() {
   if ! command -v node &>/dev/null; then
     echo "SKIP: node not available"
     return 0
@@ -187,19 +173,18 @@ test_actions_session_name_template() {
   
   local result
   result=$(node --experimental-vm-modules -e "
-    import { buildSessionName } from './service/actions.js';
+    import { expandTemplate } from './service/actions.js';
     
     const item = {
-      number: 42,
-      repo_key: 'myorg/backend',
-      repo_short: 'backend'
+      title: 'Fix bug',
+      number: 456
     };
     
-    const template = 'issue-{repo_short}-{number}';
-    const name = buildSessionName(template, item);
+    const template = 'Issue #{number}: {title}';
+    const expanded = expandTemplate(template, item);
     
-    if (name !== 'issue-backend-42') {
-      console.log('FAIL: Expected issue-backend-42, got ' + name);
+    if (expanded !== 'Issue #456: Fix bug') {
+      console.log('FAIL: Expected \"Issue #456: Fix bug\", got \"' + expanded + '\"');
       process.exit(1);
     }
     console.log('PASS');
@@ -254,8 +239,8 @@ echo "Functional Tests:"
 
 for test_func in \
   test_actions_build_local_command \
-  test_actions_prompt_template_expansion \
-  test_actions_session_name_template
+  test_actions_session_name_template \
+  test_actions_expand_template
 do
   run_test "${test_func#test_}" "$test_func"
 done

package/test/test_cli.bash

@@ -51,9 +51,9 @@ test_cli_help_shows_usage() {
   }
 }
 
-test_cli_help_shows_setup_command() {
-  "$CLI_PATH" help 2>&1 | grep -q "setup" || {
-    echo "help command should show setup command"
+test_cli_help_shows_start_command() {
+  "$CLI_PATH" help 2>&1 | grep -q "start" || {
+    echo "help command should show start command"
     return 1
   }
 }
@@ -65,6 +65,13 @@ test_cli_help_shows_status_command() {
   }
 }
 
+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"
@@ -87,34 +94,23 @@ test_cli_unknown_command_shows_error() {
 # Status Command Tests
 # =============================================================================
 
-test_cli_status_shows_plugin_info() {
-  local output
-  output=$("$CLI_PATH" status 2>&1) || true
-  
-  echo "$output" | grep -qi "plugin" || {
-    echo "status should show plugin info"
-    echo "Output: $output"
-    return 1
-  }
-}
-
-test_cli_status_shows_notification_config() {
+test_cli_status_shows_service_info() {
   local output
   output=$("$CLI_PATH" status 2>&1) || true
   
-  echo "$output" | grep -qi "notification\|topic" || {
-    echo "status should show notification config"
+  echo "$output" | grep -qi "service" || {
+    echo "status should show service info"
     echo "Output: $output"
     return 1
   }
 }
 
-test_cli_status_shows_polling_config() {
+test_cli_status_shows_config_info() {
   local output
   output=$("$CLI_PATH" status 2>&1) || true
   
-  echo "$output" | grep -qi "polling\|repos.yaml" || {
-    echo "status should show polling config"
+  echo "$output" | grep -qi "config" || {
+    echo "status should show config info"
     echo "Output: $output"
     return 1
   }
@@ -139,8 +135,9 @@ echo "Help Command Tests:"
 
 for test_func in \
   test_cli_help_shows_usage \
-  test_cli_help_shows_setup_command \
+  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
@@ -151,9 +148,8 @@ echo ""
 echo "Status Command Tests:"
 
 for test_func in \
-  test_cli_status_shows_plugin_info \
-  test_cli_status_shows_notification_config \
-  test_cli_status_shows_polling_config
+  test_cli_status_shows_service_info \
+  test_cli_status_shows_config_info
 do
   run_test "${test_func#test_}" "$test_func"
 done

package/test/test_plist.bash

@@ -1,7 +1,6 @@
 #!/usr/bin/env bash
 #
-# Tests for service/io.opencode.ntfy.plist - LaunchAgent plist for brew services
-# Issue #13: Separate callback server as brew service
+# Tests for service/io.opencode.pilot.plist - LaunchAgent plist for brew services
 #
 
 set -euo pipefail
@@ -19,7 +18,7 @@ echo ""
 # =============================================================================
 
 test_plist_file_exists() {
-  assert_file_exists "$SERVICE_DIR/io.opencode.ntfy.plist"
+  assert_file_exists "$SERVICE_DIR/io.opencode.pilot.plist"
 }
 
 test_plist_is_valid_xml() {
@@ -27,7 +26,7 @@ test_plist_is_valid_xml() {
     echo "SKIP: plutil not available (macOS only)"
     return 0
   fi
-  plutil -lint "$SERVICE_DIR/io.opencode.ntfy.plist" 2>&1 || {
+  plutil -lint "$SERVICE_DIR/io.opencode.pilot.plist" 2>&1 || {
     echo "plist is not valid XML"
     return 1
   }
@@ -38,56 +37,56 @@ test_plist_is_valid_xml() {
 # =============================================================================
 
 test_plist_has_label() {
-  grep -q "<string>io.opencode.ntfy</string>" "$SERVICE_DIR/io.opencode.ntfy.plist" || {
+  grep -q "<string>io.opencode.pilot</string>" "$SERVICE_DIR/io.opencode.pilot.plist" || {
     echo "Label not found in plist"
     return 1
   }
 }
 
 test_plist_has_program_arguments() {
-  grep -q "<key>ProgramArguments</key>" "$SERVICE_DIR/io.opencode.ntfy.plist" || {
+  grep -q "<key>ProgramArguments</key>" "$SERVICE_DIR/io.opencode.pilot.plist" || {
     echo "ProgramArguments not found in plist"
     return 1
   }
 }
 
 test_plist_runs_node() {
-  grep -q "node" "$SERVICE_DIR/io.opencode.ntfy.plist" || {
+  grep -q "node" "$SERVICE_DIR/io.opencode.pilot.plist" || {
     echo "node command not found in plist"
     return 1
   }
 }
 
 test_plist_runs_server_js() {
-  grep -q "server.js" "$SERVICE_DIR/io.opencode.ntfy.plist" || {
+  grep -q "server.js" "$SERVICE_DIR/io.opencode.pilot.plist" || {
     echo "server.js not found in plist"
     return 1
   }
 }
 
 test_plist_has_keep_alive() {
-  grep -q "<key>KeepAlive</key>" "$SERVICE_DIR/io.opencode.ntfy.plist" || {
+  grep -q "<key>KeepAlive</key>" "$SERVICE_DIR/io.opencode.pilot.plist" || {
     echo "KeepAlive not found in plist"
     return 1
   }
 }
 
 test_plist_has_run_at_load() {
-  grep -q "<key>RunAtLoad</key>" "$SERVICE_DIR/io.opencode.ntfy.plist" || {
+  grep -q "<key>RunAtLoad</key>" "$SERVICE_DIR/io.opencode.pilot.plist" || {
     echo "RunAtLoad not found in plist"
     return 1
   }
 }
 
 test_plist_has_stdout_log() {
-  grep -q "stdout\|StandardOutPath" "$SERVICE_DIR/io.opencode.ntfy.plist" || {
+  grep -q "stdout\|StandardOutPath" "$SERVICE_DIR/io.opencode.pilot.plist" || {
     echo "Stdout logging not found in plist"
     return 1
   }
 }
 
 test_plist_has_stderr_log() {
-  grep -q "stderr\|StandardErrorPath" "$SERVICE_DIR/io.opencode.ntfy.plist" || {
+  grep -q "stderr\|StandardErrorPath" "$SERVICE_DIR/io.opencode.pilot.plist" || {
     echo "Stderr logging not found in plist"
     return 1
   }

package/test/test_poller.bash

@@ -43,30 +43,23 @@ test_poller_exports_create_poller() {
   }
 }
 
-test_poller_exports_poll_source() {
-  grep -q "export.*function pollSource\|export.*pollSource" "$SERVICE_DIR/poller.js" || {
-    echo "pollSource export not found in poller.js"
+test_poller_exports_poll_generic_source() {
+  grep -q "export.*function pollGenericSource\|export.*pollGenericSource" "$SERVICE_DIR/poller.js" || {
+    echo "pollGenericSource export not found in poller.js"
     return 1
   }
 }
 
-# =============================================================================
-# Implementation Tests
-# =============================================================================
-
-test_poller_supports_github_issues() {
-  grep -q "github_issue\|github-issue" "$SERVICE_DIR/poller.js" || {
-    echo "GitHub issue source type not found in poller.js"
+test_poller_exports_apply_mappings() {
+  grep -q "export.*function applyMappings\|export.*applyMappings" "$SERVICE_DIR/poller.js" || {
+    echo "applyMappings export not found in poller.js"
     return 1
   }
 }
 
-test_poller_supports_linear_issues() {
-  grep -q "linear_issue\|linear-issue" "$SERVICE_DIR/poller.js" || {
-    echo "Linear issue source type not found in poller.js"
-    return 1
-  }
-}
+# =============================================================================
+# Implementation Tests
+# =============================================================================
 
 test_poller_uses_mcp_client() {
   grep -q "@modelcontextprotocol/sdk" "$SERVICE_DIR/poller.js" || {
@@ -82,6 +75,13 @@ test_poller_tracks_processed_items() {
   }
 }
 
+test_poller_supports_generic_tools() {
+  grep -q "tool" "$SERVICE_DIR/poller.js" || {
+    echo "Tool-based polling not found in poller.js"
+    return 1
+  }
+}
+
 # =============================================================================
 # Run Tests
 # =============================================================================
@@ -100,7 +100,8 @@ echo "Export Tests:"
 
 for test_func in \
   test_poller_exports_create_poller \
-  test_poller_exports_poll_source
+  test_poller_exports_poll_generic_source \
+  test_poller_exports_apply_mappings
 do
   run_test "${test_func#test_}" "$test_func"
 done
@@ -109,10 +110,9 @@ echo ""
 echo "Implementation Tests:"
 
 for test_func in \
-  test_poller_supports_github_issues \
-  test_poller_supports_linear_issues \
   test_poller_uses_mcp_client \
-  test_poller_tracks_processed_items
+  test_poller_tracks_processed_items \
+  test_poller_supports_generic_tools
 do
   run_test "${test_func#test_}" "$test_func"
 done

package/test/test_repo_config.bash

@@ -68,23 +68,16 @@ test_repo_config_supports_yaml() {
   }
 }
 
-test_repo_config_supports_prefix_matching() {
-  grep -q "prefix\|endsWith.*/" "$SERVICE_DIR/repo-config.js" || {
-    echo "Prefix matching not found"
+test_repo_config_supports_source_config() {
+  grep -q "sources" "$SERVICE_DIR/repo-config.js" || {
+    echo "Sources support not found"
     return 1
   }
 }
 
-test_repo_config_merges_configs() {
-  grep -q "merge\|deep.*merge" "$SERVICE_DIR/repo-config.js" || {
-    echo "Config merging not found"
-    return 1
-  }
-}
-
-test_repo_config_expands_placeholders() {
-  grep -q "{repo}\|placeholder" "$SERVICE_DIR/repo-config.js" || {
-    echo "Placeholder expansion not found"
+test_repo_config_supports_tool_mappings() {
+  grep -q "mappings\|tools" "$SERVICE_DIR/repo-config.js" || {
+    echo "Tool mappings support not found"
     return 1
   }
 }
@@ -93,7 +86,7 @@ test_repo_config_expands_placeholders() {
 # Functional Tests
 # =============================================================================
 
-test_repo_config_defaults() {
+test_repo_config_returns_empty_for_unknown_repo() {
   if ! command -v node &>/dev/null; then
     echo "SKIP: node not available"
     return 0
@@ -103,15 +96,11 @@ test_repo_config_defaults() {
   result=$(node --experimental-vm-modules -e "
     import { getRepoConfig } from './service/repo-config.js';
     
-    // Get config for non-existent repo should return defaults
+    // Get config for non-existent repo should return empty object
     const config = getRepoConfig('nonexistent/repo');
     
-    if (!config.readiness) {
-      console.log('FAIL: Missing readiness in defaults');
-      process.exit(1);
-    }
-    if (!config.readiness.labels) {
-      console.log('FAIL: Missing readiness.labels in defaults');
+    if (typeof config !== 'object') {
+      console.log('FAIL: Expected object for unknown repo');
       process.exit(1);
     }
     console.log('PASS');
@@ -126,7 +115,7 @@ test_repo_config_defaults() {
   fi
 }
 
-test_repo_config_prefix_matching() {
+test_repo_config_gets_sources() {
   if ! command -v node &>/dev/null; then
     echo "SKIP: node not available"
     return 0
@@ -134,206 +123,15 @@ test_repo_config_prefix_matching() {
   
   local result
   result=$(node --experimental-vm-modules -e "
-    import { loadRepoConfig, getRepoConfig } from './service/repo-config.js';
-    
-    // Load test config
-    const testConfig = {
-      repos: {
-        'myorg/': {
-          repo_path: '~/code/{repo}',
-          readiness: { labels: { any_of: ['ready'] } }
-        },
-        'myorg/backend': {
-          readiness: { labels: { any_of: ['backend-ready'] } }
-        }
-      }
-    };
-    
-    loadRepoConfig(testConfig);
+    import { getAllSources } from './service/repo-config.js';
     
-    // Get config for myorg/backend - should merge prefix + exact
-    const config = getRepoConfig('myorg/backend');
-    
-    // Should have readiness from exact match
-    if (!config.readiness.labels.any_of.includes('backend-ready')) {
-      console.log('FAIL: Expected any_of to include backend-ready from exact match');
-      process.exit(1);
-    }
-    
-    // Get config for myorg/frontend - should get prefix config
-    const frontendConfig = getRepoConfig('myorg/frontend');
-    if (!frontendConfig.readiness.labels.any_of.includes('ready')) {
-      console.log('FAIL: Expected any_of to include ready from prefix');
-      process.exit(1);
-    }
-    
-    // repo_path should have {repo} expanded
-    if (frontendConfig.repo_path !== '~/code/frontend') {
-      console.log('FAIL: Expected expanded repo_path, got ' + frontendConfig.repo_path);
-      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
-}
-
-# =============================================================================
-# Default Sources Tests
-# =============================================================================
-
-test_repo_config_default_sources() {
-  if ! command -v node &>/dev/null; then
-    echo "SKIP: node not available"
-    return 0
-  fi
-  
-  local result
-  result=$(node --experimental-vm-modules -e "
-    import { loadRepoConfig, getRepoConfig, getAllSources } from './service/repo-config.js';
-    
-    // Load config with repo that has no sources specified
-    const testConfig = {
-      repos: {
-        'myorg/': {
-          repo_path: '~/code/{repo}'
-        },
-        'myorg/backend': {}  // No sources specified
-      }
-    };
-    
-    loadRepoConfig(testConfig);
-    
-    // Get config - should have default github_issue source
-    const config = getRepoConfig('myorg/backend');
-    
-    if (!config.sources || config.sources.length === 0) {
-      console.log('FAIL: Expected default sources, got empty array');
-      process.exit(1);
-    }
-    
-    const defaultSource = config.sources[0];
-    if (defaultSource.type !== 'github_issue') {
-      console.log('FAIL: Expected default source type github_issue, got ' + defaultSource.type);
-      process.exit(1);
-    }
-    
-    if (!defaultSource.fetch || defaultSource.fetch.assignee !== '@me') {
-      console.log('FAIL: Expected default fetch.assignee=@me, got ' + JSON.stringify(defaultSource.fetch));
-      process.exit(1);
-    }
-    
-    if (!defaultSource.fetch || defaultSource.fetch.state !== 'open') {
-      console.log('FAIL: Expected default fetch.state=open, got ' + JSON.stringify(defaultSource.fetch));
-      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_repo_config_explicit_sources_override_defaults() {
-  if ! command -v node &>/dev/null; then
-    echo "SKIP: node not available"
-    return 0
-  fi
-  
-  local result
-  result=$(node --experimental-vm-modules -e "
-    import { loadRepoConfig, getRepoConfig } from './service/repo-config.js';
-    
-    // Load config with explicit sources
-    const testConfig = {
-      repos: {
-        'myorg/backend': {
-          sources: [
-            { type: 'github_pr', fetch: { state: 'open' } }
-          ]
-        }
-      }
-    };
-    
-    loadRepoConfig(testConfig);
-    
-    // Get config - should have explicit sources, not defaults
-    const config = getRepoConfig('myorg/backend');
-    
-    if (config.sources.length !== 1) {
-      console.log('FAIL: Expected 1 source, got ' + config.sources.length);
-      process.exit(1);
-    }
-    
-    if (config.sources[0].type !== 'github_pr') {
-      console.log('FAIL: Expected explicit github_pr source, got ' + config.sources[0].type);
-      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_repo_config_get_all_sources_includes_defaults() {
-  if ! command -v node &>/dev/null; then
-    echo "SKIP: node not available"
-    return 0
-  fi
-  
-  local result
-  result=$(node --experimental-vm-modules -e "
-    import { loadRepoConfig, getAllSources } from './service/repo-config.js';
-    
-    // Load config with repo that has no sources specified
-    const testConfig = {
-      repos: {
-        'myorg/backend': {
-          repo_path: '~/code/backend'
-        }
-      }
-    };
-    
-    loadRepoConfig(testConfig);
-    
-    // getAllSources should include the default source
+    // getAllSources should return an array
     const sources = getAllSources();
     
-    if (sources.length !== 1) {
-      console.log('FAIL: Expected 1 source from getAllSources, got ' + sources.length);
-      process.exit(1);
-    }
-    
-    if (sources[0].type !== 'github_issue') {
-      console.log('FAIL: Expected github_issue source, got ' + sources[0].type);
+    if (!Array.isArray(sources)) {
+      console.log('FAIL: Expected array from getAllSources');
       process.exit(1);
     }
-    
-    if (sources[0].repo_key !== 'myorg/backend') {
-      console.log('FAIL: Expected repo_key myorg/backend, got ' + sources[0].repo_key);
-      process.exit(1);
-    }
-    
     console.log('PASS');
   " 2>&1) || {
     echo "Functional test failed: $result"
@@ -375,9 +173,8 @@ echo "Implementation Tests:"
 
 for test_func in \
   test_repo_config_supports_yaml \
-  test_repo_config_supports_prefix_matching \
-  test_repo_config_merges_configs \
-  test_repo_config_expands_placeholders
+  test_repo_config_supports_source_config \
+  test_repo_config_supports_tool_mappings
 do
   run_test "${test_func#test_}" "$test_func"
 done
@@ -386,19 +183,8 @@ echo ""
 echo "Functional Tests:"
 
 for test_func in \
-  test_repo_config_defaults \
-  test_repo_config_prefix_matching
-do
-  run_test "${test_func#test_}" "$test_func"
-done
-
-echo ""
-echo "Default Sources Tests:"
-
-for test_func in \
-  test_repo_config_default_sources \
-  test_repo_config_explicit_sources_override_defaults \
-  test_repo_config_get_all_sources_includes_defaults
+  test_repo_config_returns_empty_for_unknown_repo \
+  test_repo_config_gets_sources
 do
   run_test "${test_func#test_}" "$test_func"
 done