opencode-pilot 0.1.0 → 0.2.1

package/test/test_plugin.bash

@@ -1,952 +0,0 @@
-#!/usr/bin/env bash
-#
-# Tests for opencode-ntfy plugin
-#
-# These tests verify plugin file structure and JavaScript syntax.
-# Pure function tests will be added as modules are implemented.
-#
-
-set -euo pipefail
-
-SCRIPT_DIR="$(cd "$(dirname "${BASH_SOURCE[0]}")" && pwd)"
-source "$SCRIPT_DIR/test_helper.bash"
-
-PLUGIN_DIR="$(dirname "$SCRIPT_DIR")/plugin"
-
-echo "Testing opencode-ntfy plugin..."
-echo ""
-
-# =============================================================================
-# Plugin File Structure Tests
-# =============================================================================
-
-test_plugin_index_exists() {
-  assert_file_exists "$PLUGIN_DIR/index.js"
-}
-
-test_plugin_notifier_exists() {
-  assert_file_exists "$PLUGIN_DIR/notifier.js"
-}
-
-# =============================================================================
-# JavaScript Syntax Validation Tests
-# =============================================================================
-
-test_index_js_syntax() {
-  if ! command -v node &>/dev/null; then
-    echo "SKIP: node not available"
-    return 0
-  fi
-  node --check "$PLUGIN_DIR/index.js" 2>&1 || {
-    echo "index.js has syntax errors"
-    return 1
-  }
-}
-
-test_notifier_js_syntax() {
-  if ! command -v node &>/dev/null; then
-    echo "SKIP: node not available"
-    return 0
-  fi
-  node --check "$PLUGIN_DIR/notifier.js" 2>&1 || {
-    echo "notifier.js has syntax errors"
-    return 1
-  }
-}
-
-# =============================================================================
-# Configuration Tests
-# =============================================================================
-
-test_index_imports_config() {
-  # Verify index.js imports config from config.js
-  grep -q "import.*loadConfig.*from.*config" "$PLUGIN_DIR/index.js" || {
-    echo "loadConfig import not found in index.js"
-    return 1
-  }
-}
-
-test_index_uses_load_config() {
-  # Verify index.js calls loadConfig()
-  grep -q "loadConfig()" "$PLUGIN_DIR/index.js" || {
-    echo "loadConfig() call not found in index.js"
-    return 1
-  }
-}
-
-# =============================================================================
-# Idle Notification Behavior Tests
-# =============================================================================
-
-test_handles_session_status_events() {
-  # Verify the event handler checks for session.status events
-  grep -q "session.status" "$PLUGIN_DIR/index.js" || {
-    echo "session.status event handling not found"
-    return 1
-  }
-}
-
-test_handles_idle_status() {
-  # Verify idle status triggers timer
-  grep -q "idle" "$PLUGIN_DIR/index.js" || {
-    echo "idle status handling not found"
-    return 1
-  }
-}
-
-test_handles_busy_status() {
-  # Verify busy status cancels timer
-  grep -q "busy" "$PLUGIN_DIR/index.js" || {
-    echo "busy status handling not found"
-    return 1
-  }
-}
-
-test_uses_settimeout_for_idle() {
-  # Verify setTimeout is used for idle delay
-  grep -q "setTimeout" "$PLUGIN_DIR/index.js" || {
-    echo "setTimeout not found for idle delay"
-    return 1
-  }
-}
-
-test_uses_cleartimeout_for_busy() {
-  # Verify clearTimeout is used when busy
-  grep -q "clearTimeout" "$PLUGIN_DIR/index.js" || {
-    echo "clearTimeout not found for busy cancel"
-    return 1
-  }
-}
-
-test_imports_send_notification_from_notifier() {
-  # Verify index.js imports sendNotification from notifier.js
-  grep -q "sendNotification" "$PLUGIN_DIR/index.js" || {
-    echo "sendNotification not imported/used in index.js"
-    return 1
-  }
-}
-
-test_uses_configured_server_and_topic() {
-  # Verify notification uses config.server and config.topic
-  grep -q 'config\.server' "$PLUGIN_DIR/index.js" && \
-  grep -q 'config\.topic' "$PLUGIN_DIR/index.js" || {
-    echo "Notification should use config.server and config.topic"
-    return 1
-  }
-}
-
-test_uses_configured_idle_delay() {
-  # Verify timeout uses config.idleDelayMs
-  grep -q 'config\.idleDelayMs' "$PLUGIN_DIR/index.js" || {
-    echo "Timer should use config.idleDelayMs"
-    return 1
-  }
-}
-
-test_uses_directory_param_not_process_cwd() {
-  # Should use the directory parameter from OpenCode, not process.cwd()
-  # This ensures devcontainer temp dirs don't show up in notifications
-  grep -q 'directory' "$PLUGIN_DIR/index.js" || {
-    echo "directory parameter not used in index.js"
-    return 1
-  }
-  # Should NOT use process.cwd() for directory name in code (comments are ok)
-  # Look for actual usage like: basename(process.cwd()) or = process.cwd()
-  if grep -E 'basename\(process\.cwd\(\)\)|=\s*process\.cwd\(\)' "$PLUGIN_DIR/index.js"; then
-    echo "Should not use process.cwd() for directory - use directory param instead"
-    return 1
-  fi
-}
-
-test_idle_notification_shows_repo_context() {
-  # Idle notification should include repo name for context
-  # Title should be "Idle (repo)" not just "OpenCode"
-  grep -q 'Idle' "$PLUGIN_DIR/index.js" || {
-    echo "Idle notification title should include 'Idle'"
-    return 1
-  }
-}
-
-# =============================================================================
-# Logging Tests
-# =============================================================================
-# NOTE: Logging tests removed - all console output is now suppressed to avoid TUI interference
-
-# =============================================================================
-# Plugin Export Structure Tests
-# =============================================================================
-
-test_index_exports_notify() {
-  # Plugin should only use default export to prevent double-loading by OpenCode
-  # (Issue #34: Having both named and default export caused plugin to load twice)
-  grep -q "export default Notify" "$PLUGIN_DIR/index.js" || {
-    echo "Default Notify export not found in index.js"
-    return 1
-  }
-  # Ensure named export is NOT present (would cause double-loading)
-  if grep -q "export const Notify" "$PLUGIN_DIR/index.js"; then
-    echo "Named export 'export const Notify' found - should only use default export"
-    return 1
-  fi
-}
-
-test_index_has_default_export() {
-  grep -q "export default" "$PLUGIN_DIR/index.js" || {
-    echo "Default export not found in index.js"
-    return 1
-  }
-}
-
-test_notifier_exports_send_notification() {
-  grep -q "export.*sendNotification" "$PLUGIN_DIR/notifier.js" || {
-    echo "sendNotification export not found in notifier.js"
-    return 1
-  }
-}
-
-# =============================================================================
-# Retry Event Handling Tests (Issue #7)
-# =============================================================================
-
-test_handles_retry_status() {
-  # Verify retry status is handled within session.status events
-  grep -q "retry" "$PLUGIN_DIR/index.js" || {
-    echo "retry status handling not found"
-    return 1
-  }
-}
-
-test_tracks_retry_count() {
-  # Verify retry counter is tracked
-  grep -q "retryCount" "$PLUGIN_DIR/index.js" || {
-    echo "retryCount variable not found"
-    return 1
-  }
-}
-
-test_uses_retry_notify_first_config() {
-  # Verify retryNotifyFirst config is used
-  grep -q "retryNotifyFirst" "$PLUGIN_DIR/index.js" || {
-    echo "retryNotifyFirst config not used"
-    return 1
-  }
-}
-
-test_uses_retry_notify_after_config() {
-  # Verify retryNotifyAfter config is used
-  grep -q "retryNotifyAfter" "$PLUGIN_DIR/index.js" || {
-    echo "retryNotifyAfter config not used"
-    return 1
-  }
-}
-
-test_sends_retry_notification_with_priority() {
-  # Verify retry notifications use high priority (4)
-  grep -q "priority.*4\|4.*priority" "$PLUGIN_DIR/index.js" || {
-    echo "High priority (4) not found for retry notifications"
-    return 1
-  }
-}
-
-# =============================================================================
-# Error Event Handling Tests (Issue #7)
-# =============================================================================
-
-test_handles_session_error() {
-  # Verify session.error events are handled
-  grep -q "session\.error" "$PLUGIN_DIR/index.js" || {
-    echo "session.error event handling not found"
-    return 1
-  }
-}
-
-test_uses_error_notify_config() {
-  # Verify errorNotify config is used
-  grep -q "errorNotify" "$PLUGIN_DIR/index.js" || {
-    echo "errorNotify config not used"
-    return 1
-  }
-}
-
-test_uses_error_debounce_config() {
-  # Verify errorDebounceMs config is used
-  grep -q "errorDebounceMs" "$PLUGIN_DIR/index.js" || {
-    echo "errorDebounceMs config not used"
-    return 1
-  }
-}
-
-test_tracks_last_error_time() {
-  # Verify last error timestamp is tracked for debouncing
-  grep -q "lastErrorTime" "$PLUGIN_DIR/index.js" || {
-    echo "lastErrorTime variable not found"
-    return 1
-  }
-}
-
-test_sends_error_notification_with_urgent_priority() {
-  # Verify error notifications use urgent priority (5)
-  grep -q "priority.*5\|5.*priority" "$PLUGIN_DIR/index.js" || {
-    echo "Urgent priority (5) not found for error notifications"
-    return 1
-  }
-}
-
-
-
-# =============================================================================
-# Counter Reset Tests (Issue #7)
-# =============================================================================
-
-test_resets_retry_counter_on_status_change() {
-  # Verify retry counter is reset when status changes
-  grep -q "retryCount.*=.*0\|retryCount = 0" "$PLUGIN_DIR/index.js" || {
-    echo "Retry counter reset not found"
-    return 1
-  }
-}
-
-# =============================================================================
-# Cancellation Handling Tests
-# =============================================================================
-
-test_handles_canceled_status() {
-  # Verify canceled status is handled (no notification on cancel)
-  grep -q "canceled" "$PLUGIN_DIR/index.js" || {
-    echo "canceled status handling not found"
-    return 1
-  }
-}
-
-test_sets_canceled_flag_on_canceled_status() {
-  # Verify a flag is set when session is canceled
-  grep -q "wasCanceled\|sessionCanceled\|canceled.*true\|isCanceled" "$PLUGIN_DIR/index.js" || {
-    echo "canceled flag not found"
-    return 1
-  }
-}
-
-test_shutdown_checks_canceled_before_notification() {
-  # Verify shutdown handler respects canceled state
-  # The idle timer should be cancelled without sending notification
-  grep -q "shutdown" "$PLUGIN_DIR/index.js" || {
-    echo "shutdown handler not found"
-    return 1
-  }
-}
-
-# =============================================================================
-# Console Output Suppression Tests
-# =============================================================================
-
-test_no_console_log_calls() {
-  # Plugin should not use console.log (interferes with TUI)
-  if grep -q 'console\.log' "$PLUGIN_DIR/index.js"; then
-    echo "console.log found - should use silent or file-based logging"
-    return 1
-  fi
-}
-
-test_no_console_warn_calls() {
-  # Plugin should not use console.warn (interferes with TUI)
-  if grep -q 'console\.warn' "$PLUGIN_DIR/index.js"; then
-    echo "console.warn found - should use silent or file-based logging"
-    return 1
-  fi
-}
-
-test_no_console_error_calls() {
-  # Plugin should not use console.error (interferes with TUI)
-  if grep -q 'console\.error' "$PLUGIN_DIR/index.js"; then
-    echo "console.error found - should use silent or file-based logging"
-    return 1
-  fi
-}
-
-# =============================================================================
-# Debug Logging Tests
-# =============================================================================
-
-test_index_imports_logger() {
-  grep -q "import.*logger\|from.*logger" "$PLUGIN_DIR/index.js" || {
-    echo "logger import not found in index.js"
-    return 1
-  }
-}
-
-test_index_initializes_logger() {
-  grep -q "initLogger" "$PLUGIN_DIR/index.js" || {
-    echo "initLogger call not found in index.js"
-    return 1
-  }
-}
-
-test_index_uses_debug_for_events() {
-  # Plugin should log events received
-  grep -q "debug.*[Ee]vent" "$PLUGIN_DIR/index.js" || {
-    echo "debug logging for events not found in index.js"
-    return 1
-  }
-}
-
-test_index_uses_debug_for_idle_timer() {
-  # Plugin should log idle timer start/cancel
-  grep -q "debug.*[Ii]dle\|debug.*timer" "$PLUGIN_DIR/index.js" || {
-    echo "debug logging for idle timer not found in index.js"
-    return 1
-  }
-}
-
-test_index_uses_debug_for_initialization() {
-  # Plugin should log initialization with key config
-  grep -q "debug.*[Ii]nitial\|debug.*[Pp]lugin" "$PLUGIN_DIR/index.js" || {
-    echo "debug logging for initialization not found in index.js"
-    return 1
-  }
-}
-
-# =============================================================================
-# Notification Suppression Logging Tests (Issue #7)
-# =============================================================================
-# NOTE: Console output suppression tests removed - all logging disabled to avoid TUI interference
-
-# =============================================================================
-# OpenCode Runtime Integration Tests
-# =============================================================================
-# These tests verify the plugin doesn't hang opencode on startup and
-# works correctly in the real OpenCode runtime.
-#
-# Tests run if opencode is installed and plugin is configured.
-# Skipped in CI unless explicitly enabled.
-
-# Helper to check if we can run integration tests
-can_run_integration_tests() {
-  # Check opencode is installed
-  if ! command -v opencode &>/dev/null; then
-    return 1
-  fi
-  
-  # Check plugin is installed (use REAL_HOME since we may have changed HOME)
-  local plugin_path="${REAL_HOME:-$HOME}/.config/opencode/plugins/opencode-ntfy/index.js"
-  if [[ ! -f "$plugin_path" ]]; then
-    return 1
-  fi
-  
-  return 0
-}
-
-# Setup isolated environment for integration tests
-# Uses temp directory for opencode data while symlinking config from real HOME
-setup_integration_env() {
-  REAL_HOME="$HOME"
-  INTEGRATION_TEST_DIR=$(mktemp -d)
-  export HOME="$INTEGRATION_TEST_DIR"
-  
-  # Create necessary directories
-  mkdir -p "$HOME/.config"
-  mkdir -p "$HOME/.local/share"
-  
-  # Symlink opencode config (contains plugin registrations and auth)
-  ln -s "$REAL_HOME/.config/opencode" "$HOME/.config/opencode"
-  
-  # Symlink opencode-ntfy config
-  if [[ -d "$REAL_HOME/.config/opencode-ntfy" ]]; then
-    ln -s "$REAL_HOME/.config/opencode-ntfy" "$HOME/.config/opencode-ntfy"
-  fi
-}
-
-# Cleanup isolated environment after integration tests
-cleanup_integration_env() {
-  if [[ -n "${INTEGRATION_TEST_DIR:-}" ]] && [[ -d "$INTEGRATION_TEST_DIR" ]]; then
-    rm -rf "$INTEGRATION_TEST_DIR"
-  fi
-  if [[ -n "${REAL_HOME:-}" ]]; then
-    export HOME="$REAL_HOME"
-  fi
-}
-
-# Cross-platform timeout wrapper
-# Uses perl alarm which works on macOS and Linux
-run_with_timeout() {
-  local timeout_secs="$1"
-  shift
-  perl -e "alarm $timeout_secs; exec @ARGV" "$@" 2>&1
-}
-
-# Run opencode and capture output (with timeout)
-run_opencode() {
-  local prompt="$1"
-  local timeout="${2:-60}"
-  
-  run_with_timeout "$timeout" opencode run --format json "$prompt"
-}
-
-test_opencode_starts_within_timeout() {
-  if ! can_run_integration_tests; then
-    echo "SKIP: opencode not installed or plugin not configured"
-    return 0
-  fi
-  
-  setup_integration_env
-  
-  # CRITICAL: Verify opencode starts within 10 seconds
-  # This catches plugin initialization hangs that would block startup indefinitely.
-  
-  local start_time end_time elapsed output
-  start_time=$(date +%s)
-  
-  # Use a short timeout - if plugin hangs, this will fail
-  output=$(run_with_timeout 10 opencode run --format json "Say hi" 2>&1)
-  local exit_code=$?
-  
-  end_time=$(date +%s)
-  elapsed=$((end_time - start_time))
-  
-  cleanup_integration_env
-  
-  if [[ $exit_code -ne 0 ]]; then
-    # Check if it was a timeout (exit code 142 = SIGALRM)
-    if [[ $exit_code -eq 142 ]] || [[ "$output" == *"Alarm clock"* ]]; then
-      echo "FAIL: opencode startup timed out after ${elapsed}s (plugin may be hanging)"
-      echo "Output: $output"
-      return 1
-    fi
-    # Skip on model configuration errors (not a plugin issue)
-    if [[ "$output" == *"ModelNotFoundError"* ]] || [[ "$output" == *"ProviderModelNotFoundError"* ]]; then
-      echo "SKIP: model configuration error (not a plugin issue)"
-      return 0
-    fi
-    echo "opencode run failed (exit $exit_code): $output"
-    return 1
-  fi
-  
-  # Verify we got a response
-  if ! echo "$output" | grep -q '"type"'; then
-    # Skip on model configuration errors (not a plugin issue)
-    if [[ "$output" == *"ModelNotFoundError"* ]] || [[ "$output" == *"ProviderModelNotFoundError"* ]]; then
-      echo "SKIP: model configuration error (not a plugin issue)"
-      return 0
-    fi
-    echo "No valid JSON output from opencode"
-    echo "Output: $output"
-    return 1
-  fi
-  
-  return 0
-}
-
-test_opencode_plugin_loads() {
-  if ! can_run_integration_tests; then
-    echo "SKIP: opencode not installed or plugin not configured"
-    return 0
-  fi
-  
-  setup_integration_env
-  
-  # Plugin should load without errors - check opencode doesn't report plugin failures
-  local output
-  output=$(run_opencode "Say hello" 30) || {
-    cleanup_integration_env
-    echo "opencode run failed: $output"
-    return 1
-  }
-  
-  cleanup_integration_env
-  
-  # Check that output doesn't contain plugin error messages
-  if echo "$output" | grep -qi "plugin.*error\|failed to load"; then
-    echo "Plugin load error detected"
-    echo "Output: $output"
-    return 1
-  fi
-  
-  return 0
-}
-
-test_opencode_ntfy_disabled_without_topic() {
-  if ! can_run_integration_tests; then
-    echo "SKIP: opencode not installed or plugin not configured"
-    return 0
-  fi
-  
-  setup_integration_env
-  
-  # Without NTFY_TOPIC, plugin should disable gracefully (not crash)
-  # Unset NTFY_TOPIC temporarily
-  local old_topic="${NTFY_TOPIC:-}"
-  unset NTFY_TOPIC
-  
-  local output
-  output=$(run_opencode "Say hi" 30)
-  local exit_code=$?
-  
-  # Restore
-  if [[ -n "$old_topic" ]]; then
-    export NTFY_TOPIC="$old_topic"
-  fi
-  
-  cleanup_integration_env
-  
-  if [[ $exit_code -ne 0 ]]; then
-    echo "opencode failed without NTFY_TOPIC: $output"
-    return 1
-  fi
-  
-  return 0
-}
-
-# =============================================================================
-# Run Tests
-# =============================================================================
-
-echo "Plugin File Structure Tests:"
-
-for test_func in \
-  test_plugin_index_exists \
-  test_plugin_notifier_exists
-do
-  run_test "${test_func#test_}" "$test_func"
-done
-
-echo ""
-echo "JavaScript Syntax Validation Tests:"
-
-for test_func in \
-  test_index_js_syntax \
-  test_notifier_js_syntax
-do
-  run_test "${test_func#test_}" "$test_func"
-done
-
-echo ""
-echo "Configuration Tests:"
-
-for test_func in \
-  test_index_imports_config \
-  test_index_uses_load_config
-do
-  run_test "${test_func#test_}" "$test_func"
-done
-
-echo ""
-echo "Idle Notification Behavior Tests:"
-
-for test_func in \
-  test_handles_session_status_events \
-  test_handles_idle_status \
-  test_handles_busy_status \
-  test_uses_settimeout_for_idle \
-  test_uses_cleartimeout_for_busy \
-  test_imports_send_notification_from_notifier \
-  test_uses_configured_server_and_topic \
-  test_uses_configured_idle_delay \
-  test_uses_directory_param_not_process_cwd \
-  test_idle_notification_shows_repo_context
-do
-  run_test "${test_func#test_}" "$test_func"
-done
-
-# Logging tests removed - console output is now fully suppressed
-
-echo ""
-echo "Plugin Export Structure Tests:"
-
-for test_func in \
-  test_index_exports_notify \
-  test_index_has_default_export \
-  test_notifier_exports_send_notification
-do
-  run_test "${test_func#test_}" "$test_func"
-done
-
-echo ""
-echo "Retry Event Handling Tests (Issue #7):"
-
-for test_func in \
-  test_handles_retry_status \
-  test_tracks_retry_count \
-  test_uses_retry_notify_first_config \
-  test_uses_retry_notify_after_config \
-  test_sends_retry_notification_with_priority
-do
-  run_test "${test_func#test_}" "$test_func"
-done
-
-echo ""
-echo "Error Event Handling Tests (Issue #7):"
-
-for test_func in \
-  test_handles_session_error \
-  test_uses_error_notify_config \
-  test_uses_error_debounce_config \
-  test_tracks_last_error_time \
-  test_sends_error_notification_with_urgent_priority
-do
-  run_test "${test_func#test_}" "$test_func"
-done
-
-echo ""
-echo "Counter Reset Tests (Issue #7):"
-
-for test_func in \
-  test_resets_retry_counter_on_status_change
-do
-  run_test "${test_func#test_}" "$test_func"
-done
-
-echo ""
-echo "Cancellation Handling Tests:"
-
-for test_func in \
-  test_handles_canceled_status \
-  test_sets_canceled_flag_on_canceled_status \
-  test_shutdown_checks_canceled_before_notification
-do
-  run_test "${test_func#test_}" "$test_func"
-done
-
-echo ""
-echo "Console Output Suppression Tests:"
-
-for test_func in \
-  test_no_console_log_calls \
-  test_no_console_warn_calls \
-  test_no_console_error_calls
-do
-  run_test "${test_func#test_}" "$test_func"
-done
-
-echo ""
-echo "Debug Logging Tests:"
-
-for test_func in \
-  test_index_imports_logger \
-  test_index_initializes_logger \
-  test_index_uses_debug_for_events \
-  test_index_uses_debug_for_idle_timer \
-  test_index_uses_debug_for_initialization
-do
-  run_test "${test_func#test_}" "$test_func"
-done
-
-# Removed notification suppression logging tests - console output is now fully suppressed
-
-# =============================================================================
-# Per-Conversation State Tracking Tests (Issue #34)
-# =============================================================================
-
-test_uses_conversations_map_for_state() {
-  # Plugin should track state per-conversation using a Map, not global variables
-  grep -q "conversations\|Map()" "$PLUGIN_DIR/index.js" || {
-    echo "Conversations Map not found in index.js"
-    return 1
-  }
-}
-
-test_extracts_session_id_from_event() {
-  # Plugin should extract session ID from event properties for routing
-  grep -q "event\.properties.*id\|properties.*info.*id" "$PLUGIN_DIR/index.js" || {
-    echo "Session ID extraction from event not found"
-    return 1
-  }
-}
-
-test_stores_idle_timer_per_conversation() {
-  # Each conversation should have its own idle timer
-  # Look for pattern like conversations.get(sessionId).idleTimer or similar
-  grep -q "idleTimer" "$PLUGIN_DIR/index.js" || {
-    echo "idleTimer not found in index.js"
-    return 1
-  }
-}
-
-test_captures_session_id_in_idle_timer_closure() {
-  # The sessionId should be captured when setting the timer, not read later
-  # This ensures the notification URL points to the correct conversation
-  # Pattern: use local variable in setTimeout callback, not external reference
-  grep -A 30 "setTimeout" "$PLUGIN_DIR/index.js" | grep -q "session" || {
-    echo "Session ID not captured in idle timer"
-    return 1
-  }
-}
-
-test_clears_conversation_state_on_cancel() {
-  # When a conversation is canceled, its state should be cleaned up
-  grep -q "canceled" "$PLUGIN_DIR/index.js" && \
-  grep -q "delete\|clear" "$PLUGIN_DIR/index.js" || {
-    echo "Conversation cleanup on cancel not found"
-    return 1
-  }
-}
-
-# =============================================================================
-# Session Tracking and Open Session Action Tests (Issue #27)
-# =============================================================================
-
-test_tracks_current_session_id() {
-  # Plugin should track current session ID from session events
-  grep -q "currentSessionId\|sessionId" "$PLUGIN_DIR/index.js" || {
-    echo "session ID tracking not found in index.js"
-    return 1
-  }
-}
-
-test_extracts_session_id_from_status_event() {
-  # Plugin should extract session ID from session.status events for per-conversation tracking
-  grep -q "session\.status\|properties.*info.*id" "$PLUGIN_DIR/index.js" || {
-    echo "Session ID extraction from status event not found"
-    return 1
-  }
-}
-
-
-
-# =============================================================================
-# Devcontainer Path Parsing Tests
-# =============================================================================
-
-test_parses_devcontainer_clone_path() {
-  # When directory is a devcontainer clone, should extract repo and branch
-  # Path format: /Users/foo/.cache/devcontainer-clones/{repo}/{branch}
-  grep -q "devcontainer-clones" "$PLUGIN_DIR/index.js" || {
-    echo "devcontainer-clones path parsing not found in index.js"
-    return 1
-  }
-}
-
-test_notification_shows_repo_and_branch() {
-  # Notification title should show both repo name and branch when in devcontainer
-  # e.g., "Idle (opencode-ntfy/fix-something)" instead of just "Idle (fix-something)"
-  grep -q "parseRepoInfo\|getRepoName" "$PLUGIN_DIR/index.js" || {
-    echo "Repo info parsing function not found in index.js"
-    return 1
-  }
-}
-
-test_regular_directory_shows_basename_only() {
-  # For regular directories (not devcontainer clones), should show just basename
-  # e.g., "/Users/foo/code/myrepo" -> "myrepo"
-  grep -q "basename" "$PLUGIN_DIR/index.js" || {
-    echo "basename fallback not found in index.js"
-    return 1
-  }
-}
-
-# =============================================================================
-# Session Ownership Verification Tests (Issue #50)
-# =============================================================================
-# These tests verify the plugin only processes events for sessions that
-# belong to its OpenCode instance, preventing duplicate notifications when
-# multiple OpenCode instances are running.
-
-test_verifies_session_ownership_before_processing() {
-  # Plugin should verify session exists in its OpenCode instance before processing events
-  # This prevents duplicate notifications from multiple plugin instances
-  grep -q "verifySessionOwnership\|session\.get\|verifiedSessions" "$PLUGIN_DIR/index.js" || {
-    echo "Session ownership verification not found in index.js"
-    return 1
-  }
-}
-
-test_caches_verified_sessions() {
-  # Plugin should cache verified sessions to avoid repeated API calls
-  grep -q "verifiedSessions\|ownedSessions\|sessionCache" "$PLUGIN_DIR/index.js" || {
-    echo "Verified sessions cache not found in index.js"
-    return 1
-  }
-}
-
-test_tracks_rejected_sessions() {
-  # Plugin should track sessions that don't belong to it to avoid repeated lookups
-  grep -q "rejectedSessions\|foreignSessions\|notOurs" "$PLUGIN_DIR/index.js" || {
-    echo "Rejected sessions tracking not found in index.js"
-    return 1
-  }
-}
-
-test_uses_client_session_get_for_verification() {
-  # Plugin should use client.session.get() to verify session ownership
-  grep -q "client\.session\.get\|session\.get" "$PLUGIN_DIR/index.js" || {
-    echo "client.session.get() call not found in index.js"
-    return 1
-  }
-}
-
-test_skips_events_for_foreign_sessions() {
-  # Events for sessions that don't belong to this instance should be skipped
-  # Look for early return when session is not owned
-  grep -q "rejectedSessions\|foreignSessions\|not.*ours\|skip.*event" "$PLUGIN_DIR/index.js" || {
-    echo "Foreign session skip logic not found in index.js"
-    return 1
-  }
-}
-
-test_cleans_up_session_caches_on_shutdown() {
-  # Shutdown handler should clear session ownership caches to prevent memory leaks
-  grep -A 20 "shutdown:" "$PLUGIN_DIR/index.js" | grep -q "verifiedSessions.clear\|rejectedSessions.clear" || {
-    echo "Session cache cleanup not found in shutdown handler"
-    return 1
-  }
-}
-
-echo ""
-echo "Session Ownership Verification Tests (Issue #50):"
-
-for test_func in \
-  test_verifies_session_ownership_before_processing \
-  test_caches_verified_sessions \
-  test_tracks_rejected_sessions \
-  test_uses_client_session_get_for_verification \
-  test_skips_events_for_foreign_sessions \
-  test_cleans_up_session_caches_on_shutdown
-do
-  run_test "${test_func#test_}" "$test_func"
-done
-
-echo ""
-echo "Per-Conversation State Tracking Tests (Issue #34):"
-
-for test_func in \
-  test_uses_conversations_map_for_state \
-  test_extracts_session_id_from_event \
-  test_stores_idle_timer_per_conversation \
-  test_captures_session_id_in_idle_timer_closure \
-  test_clears_conversation_state_on_cancel
-do
-  run_test "${test_func#test_}" "$test_func"
-done
-
-echo ""
-echo "Session Tracking Tests:"
-
-for test_func in \
-  test_tracks_current_session_id \
-  test_extracts_session_id_from_status_event
-do
-  run_test "${test_func#test_}" "$test_func"
-done
-
-echo ""
-echo "Devcontainer Path Parsing Tests:"
-
-for test_func in \
-  test_parses_devcontainer_clone_path \
-  test_notification_shows_repo_and_branch \
-  test_regular_directory_shows_basename_only
-do
-  run_test "${test_func#test_}" "$test_func"
-done
-
-echo ""
-echo "OpenCode Runtime Integration Tests:"
-
-for test_func in \
-  test_opencode_starts_within_timeout \
-  test_opencode_plugin_loads \
-  test_opencode_ntfy_disabled_without_topic
-do
-  # Don't use setup/teardown for integration tests - use real HOME
-  run_test "${test_func#test_}" "$test_func"
-done
-
-print_summary