askcii 0.3.0 → 0.4.0

data/lib/askcii/commands/help_command.rb

@@ -0,0 +1,52 @@
+# frozen_string_literal: true
+
+module Askcii
+  module Commands
+    # Displays help information
+    class HelpCommand < BaseCommand
+      def execute
+        puts <<~HELP
+          askcii - Command-line interface for multiple LLM providers
+
+          Usage:
+            askcii [options] "Your prompt here"
+            echo "input" | askcii [options] "Your prompt"
+
+          Options:
+            -p, --private              Create a private session (no history saved)
+            -r, --last-response        Retrieve the last assistant response from current session
+            -c, --configure            Open configuration management interface
+            -m, --model ID             Use specific configuration by ID
+            -v, --verbose              Show detailed information during execution
+            --session NAME             Use specific session name (alternative to ASKCII_SESSION)
+            --list-sessions            List all available sessions
+            --history                  Show current session history
+            --clear-history            Clear current session history
+            -h, --help                 Show this help message
+
+          Environment Variables:
+            ASKCII_SESSION             Session identifier for conversation persistence
+            ASKCII_API_KEY             API key (legacy, use -c to configure)
+            ASKCII_API_ENDPOINT        API endpoint (legacy)
+            ASKCII_MODEL_ID            Model ID (legacy)
+
+          Examples:
+            askcii "Explain what a Ruby block is"
+            askcii -p "What is the capital of France?"
+            echo "def hello\nputs 'hi'\nend" | askcii "Explain this Ruby code"
+            askcii -m 2 "Use configuration #2"
+            askcii --session my-project "Continue conversation in my-project session"
+            ASKCII_SESSION=work askcii "Use work session"
+
+          Configuration:
+            Run 'askcii -c' to configure providers interactively.
+            Supports: OpenAI, Anthropic, Gemini, DeepSeek, OpenRouter, Ollama
+
+          For more information, visit: https://github.com/yourusername/askcii
+        HELP
+
+        exit 0
+      end
+    end
+  end
+end

data/lib/askcii/commands/history_command.rb

@@ -0,0 +1,46 @@
+# frozen_string_literal: true
+
+module Askcii
+  module Commands
+    # Displays conversation history for the current session
+    class HistoryCommand < BaseCommand
+      def execute
+        verbose "Retrieving history for session: #{session_context}"
+
+        chat = Chat.find(context: session_context)
+
+        unless chat
+          puts "No history found for this session."
+          exit 0
+        end
+
+        messages = chat.messages_dataset.order(:created_at).all
+
+        if messages.empty?
+          puts "No messages in this session."
+          exit 0
+        end
+
+        puts "Session: #{chat.context}"
+        puts "Model: #{chat.model_id}"
+        puts "Messages: #{messages.count}"
+        puts
+        puts "=" * 80
+        puts
+
+        messages.each do |msg|
+          role_label = msg.role.to_s.capitalize
+          timestamp = msg.created_at.strftime('%Y-%m-%d %H:%M:%S')
+
+          puts "#{role_label} (#{timestamp}):"
+          puts msg.content
+          puts
+          puts "-" * 80
+          puts
+        end
+
+        exit 0
+      end
+    end
+  end
+end

data/lib/askcii/commands/last_response_command.rb

@@ -0,0 +1,31 @@
+# frozen_string_literal: true
+
+module Askcii
+  module Commands
+    # Retrieves and displays the last assistant response from the current session
+    class LastResponseCommand < BaseCommand
+      def execute
+        verbose "Retrieving last response from session: #{session_context}"
+
+        chat = Chat.find(context: session_context)
+
+        unless chat
+          error! "No chat session found for context: #{session_context}"
+        end
+
+        last_message = chat.messages_dataset
+                          .where(role: 'assistant')
+                          .order(Sequel.desc(:created_at))
+                          .first
+
+        unless last_message
+          error! "No assistant messages found in this session"
+        end
+
+        puts last_message.content
+
+        exit 0
+      end
+    end
+  end
+end

data/lib/askcii/commands/list_sessions_command.rb

@@ -0,0 +1,36 @@
+# frozen_string_literal: true
+
+module Askcii
+  module Commands
+    # Lists all available chat sessions
+    class ListSessionsCommand < BaseCommand
+      def execute
+        verbose "Listing all sessions..."
+
+        chats = Chat.order(Sequel.desc(:created_at)).all
+
+        if chats.empty?
+          puts "No sessions found."
+          exit 0
+        end
+
+        puts "Available sessions:"
+        puts
+
+        chats.each do |chat|
+          message_count = chat.messages.count
+          last_message = chat.messages_dataset.order(Sequel.desc(:created_at)).first
+          last_updated = last_message&.created_at || chat.created_at
+
+          puts "Session: #{chat.context}"
+          puts "  Model: #{chat.model_id}"
+          puts "  Messages: #{message_count}"
+          puts "  Last updated: #{last_updated}"
+          puts
+        end
+
+        exit 0
+      end
+    end
+  end
+end

data/lib/askcii/config_validator.rb

@@ -0,0 +1,90 @@
+# frozen_string_literal: true
+
+module Askcii
+  # Validates configuration settings before use
+  class ConfigValidator
+    VALID_PROVIDERS = %w[openai anthropic gemini deepseek openrouter ollama].freeze
+
+    class << self
+      # Validates a configuration hash
+      # @param config [Hash] Configuration to validate
+      # @raise [ConfigurationError] if configuration is invalid
+      # @return [Hash] The validated configuration
+      def validate!(config)
+        validate_presence!(config)
+        validate_provider!(config)
+        validate_api_key!(config)
+        validate_model_id!(config)
+        validate_api_endpoint!(config)
+        config
+      end
+
+      # Checks if a provider name is valid
+      # @param name [String] Provider name
+      # @return [Boolean]
+      def valid_provider?(name)
+        VALID_PROVIDERS.include?(name.to_s.downcase)
+      end
+
+      # Lists all valid provider names
+      # @return [Array<String>]
+      def valid_providers
+        VALID_PROVIDERS
+      end
+
+      private
+
+      def validate_presence!(config)
+        raise ConfigurationError, "Configuration cannot be nil" if config.nil?
+        raise ConfigurationError, "Configuration cannot be empty" if config.empty?
+      end
+
+      def validate_provider!(config)
+        provider = config[:provider].to_s.strip
+        if provider.empty?
+          raise ConfigurationError, "Provider is required"
+        end
+
+        unless valid_provider?(provider)
+          raise ConfigurationError,
+                "Invalid provider '#{provider}'. Must be one of: #{VALID_PROVIDERS.join(', ')}"
+        end
+      end
+
+      def validate_api_key!(config)
+        provider = config[:provider].to_s.downcase
+
+        # Ollama doesn't require an API key
+        return if provider == 'ollama'
+
+        api_key = config[:api_key].to_s.strip
+        if api_key.empty?
+          raise ConfigurationError, "API key is required for #{provider}"
+        end
+
+        if api_key == 'blank'
+          raise ConfigurationError, "API key cannot be 'blank'"
+        end
+      end
+
+      def validate_model_id!(config)
+        model_id = config[:model_id].to_s.strip
+        if model_id.empty?
+          raise ConfigurationError, "Model ID is required"
+        end
+      end
+
+      def validate_api_endpoint!(config)
+        endpoint = config[:api_endpoint].to_s.strip
+        if endpoint.empty?
+          raise ConfigurationError, "API endpoint is required"
+        end
+
+        # Basic URL format validation
+        unless endpoint.start_with?('http://', 'https://')
+          raise ConfigurationError, "API endpoint must start with http:// or https://"
+        end
+      end
+    end
+  end
+end

data/lib/askcii/configuration_manager.rb

@@ -1,8 +1,10 @@
 # frozen_string_literal: true
 
 module Askcii
+  # Interactive terminal UI for managing provider configurations
   class ConfigurationManager
-    PROVIDER_MAP = {
+    # Maps user selection numbers to provider identifiers
+    PROVIDER_MENU = {
       '1' => 'openai',
       '2' => 'anthropic',
       '3' => 'gemini',
@@ -11,102 +13,50 @@ module Askcii
       '6' => 'ollama'
     }.freeze
 
-    DEFAULT_ENDPOINTS = {
-      'openai' => 'https://api.openai.com/v1',
-      'anthropic' => 'https://api.anthropic.com',
-      'gemini' => 'https://generativelanguage.googleapis.com/v1',
-      'deepseek' => 'https://api.deepseek.com/v1',
-      'openrouter' => 'https://openrouter.ai/api/v1',
-      'ollama' => 'http://localhost:11434/v1'
-    }.freeze
-
-    PROVIDER_MODELS = {
-      'openai' => {
-        default: 'gpt-4o',
-        models: [
-          'gpt-4o',
-          'gpt-4o-mini',
-          'gpt-4-turbo',
-          'gpt-4',
-          'gpt-3.5-turbo'
-        ]
-      },
-      'anthropic' => {
-        default: 'claude-3-5-sonnet-20241022',
-        models: [
-          'claude-3-5-sonnet-20241022',
-          'claude-3-5-haiku-20241022',
-          'claude-3-opus-20240229',
-          'claude-3-sonnet-20240229',
-          'claude-3-haiku-20240307'
-        ]
-      },
-      'gemini' => {
-        default: 'gemini-pro',
-        models: [
-          'gemini-pro',
-          'gemini-pro-vision',
-          'gemini-1.5-pro',
-          'gemini-1.5-flash'
-        ]
-      },
-      'deepseek' => {
-        default: 'deepseek-chat',
-        models: %w[
-          deepseek-chat
-          deepseek-coder
-        ]
-      },
-      'openrouter' => {
-        default: 'anthropic/claude-3.5-sonnet',
-        models: [
-          'anthropic/claude-3.5-sonnet',
-          'openai/gpt-4o',
-          'google/gemini-pro',
-          'meta-llama/llama-3.1-405b-instruct',
-          'anthropic/claude-3-opus',
-          'openai/gpt-4-turbo'
-        ]
-      },
-      'ollama' => {
-        default: 'llama3.2',
-        models: [
-          'llama3.2',
-          'llama3.1',
-          'mistral',
-          'codellama',
-          'phi3',
-          'gemma2'
-        ]
-      }
-    }.freeze
-
     def run
-      show_current_configurations
-      show_menu
-      handle_user_choice
+      loop do
+        show_current_configurations
+        show_menu
+        choice = handle_user_choice
+        break if choice == :exit
+      end
     end
 
     private
 
     def show_current_configurations
+      puts
       puts 'Configuration Management'
-      puts '======================'
+      puts '=' * 50
+      puts
 
       configs = Askcii::Config.configurations
       default_id = Askcii::Config.default_configuration_id
 
       if configs.empty?
         puts 'No configurations found.'
+        puts 'Add a configuration to get started.'
       else
         puts 'Current configurations:'
+        puts
         configs.each do |config|
-          marker = config['id'] == default_id ? ' (default)' : ''
-          provider_info = config['provider'] ? " [#{config['provider']}]" : ''
-          puts "  #{config['id']}. #{config['name']}#{provider_info}#{marker}"
+          show_configuration_summary(config, default_id)
         end
-        puts
       end
+      puts
+    end
+
+    def show_configuration_summary(config, default_id)
+      is_default = config['id'] == default_id
+      marker = is_default ? ' ★ (default)' : ''
+      provider_name = ProviderConfig.display_name(config['provider'])
+
+      puts "  #{config['id']}. #{config['name']}"
+      puts "     Provider: #{provider_name}"
+      puts "     Model: #{config['model_id']}"
+      puts "     Endpoint: #{config['api_endpoint']}"
+      puts "#{marker}"
+      puts
     end
 
     def show_menu
@@ -114,8 +64,9 @@ module Askcii
       puts '  1. Add new configuration'
       puts '  2. Set default configuration'
       puts '  3. Delete configuration'
-      puts '  4. Exit'
-      print 'Select option (1-4): '
+      puts '  4. Test configuration'
+      puts '  5. Exit'
+      print 'Select option (1-5): '
     end
 
     def handle_user_choice
@@ -124,20 +75,32 @@ module Askcii
       case choice
       when '1'
         add_new_configuration
+        :continue
       when '2'
         set_default_configuration
+        :continue
       when '3'
         delete_configuration
+        :continue
       when '4'
-        puts 'Exiting.'
+        test_configuration
+        :continue
+      when '5'
+        puts 'Exiting configuration manager.'
+        :exit
       else
-        puts 'Invalid option.'
+        puts 'Invalid option. Please select 1-5.'
+        :continue
       end
     end
 
     def add_new_configuration
+      puts
+      puts 'Add New Configuration'
+      puts '-' * 50
+
       print 'Enter configuration name: '
-      name = $stdin.gets.chomp
+      name = $stdin.gets.chomp.strip
 
       provider = select_provider
       return unless provider
@@ -147,28 +110,65 @@ module Askcii
 
       endpoint = get_api_endpoint(provider)
       model_id = get_model_id(provider)
-
       return unless model_id
 
+      # Use model_id as name if no name provided
       name = model_id if name.empty?
-      Askcii::Config.add_configuration(name, api_key || '', endpoint, model_id, provider)
-      puts 'Configuration added successfully!'
+
+      # Build configuration hash
+      config = {
+        name: name,
+        api_key: api_key || '',
+        api_endpoint: endpoint,
+        model_id: model_id,
+        provider: provider
+      }
+
+      # Validate configuration
+      begin
+        ConfigValidator.validate!(config)
+      rescue ConfigurationError => e
+        puts "Validation failed: #{e.message}"
+        return
+      end
+
+      # Save configuration
+      Askcii::Config.add_configuration(
+        name: name,
+        api_key: api_key || '',
+        api_endpoint: endpoint,
+        model_id: model_id,
+        provider: provider
+      )
+
+      puts
+      puts '✓ Configuration added successfully!'
+      puts
+
+      # Offer to test the configuration
+      print 'Test this configuration now? (y/n): '
+      if $stdin.gets.chomp.downcase == 'y'
+        # Get the newly added configuration ID
+        configs = Askcii::Config.configurations
+        new_config = configs.last
+        test_specific_configuration(new_config) if new_config
+      end
     end
 
     def select_provider
+      puts
       puts 'Select provider:'
-      puts '  1. OpenAI'
-      puts '  2. Anthropic'
-      puts '  3. Gemini'
-      puts '  4. DeepSeek'
-      puts '  5. OpenRouter'
-      puts '  6. Ollama (no API key needed)'
-      print 'Provider (1-6): '
-
-      provider_choice = $stdin.gets.chomp
-      provider = PROVIDER_MAP[provider_choice]
-
-      if provider.nil?
+      ProviderConfig.list.each_with_index do |provider_id, index|
+        provider_meta = ProviderConfig.get(provider_id)
+        key_info = provider_meta[:requires_key] ? '' : ' (no API key needed)'
+        puts "  #{index + 1}. #{provider_meta[:name]}#{key_info}"
+      end
+      print "Provider (1-#{ProviderConfig.list.size}): "
+
+      choice = $stdin.gets.chomp
+      provider = PROVIDER_MENU[choice]
+
+      if provider.nil? || !ProviderConfig.valid?(provider)
         puts 'Invalid provider selection.'
         return nil
       end
@@ -179,11 +179,12 @@ module Askcii
     def get_api_key(provider)
       return '' if provider == 'ollama'
 
-      print "Enter #{provider.capitalize} API key: "
-      api_key = $stdin.gets.chomp
+      provider_name = ProviderConfig.display_name(provider)
+      print "Enter #{provider_name} API key: "
+      api_key = $stdin.gets.chomp.strip
 
-      if api_key.empty?
-        puts 'API key is required for this provider.'
+      if api_key.empty? && ProviderConfig.requires_key?(provider)
+        puts "API key is required for #{provider_name}."
         return nil
       end
 
@@ -191,54 +192,24 @@ module Askcii
     end
 
     def get_api_endpoint(provider)
-      default_endpoint = DEFAULT_ENDPOINTS[provider]
+      default_endpoint = ProviderConfig.default_endpoint(provider)
       print "Enter API endpoint (default: #{default_endpoint}): "
-      api_endpoint = $stdin.gets.chomp
+      api_endpoint = $stdin.gets.chomp.strip
       api_endpoint.empty? ? default_endpoint : api_endpoint
     end
 
     def get_model_id(provider)
-      provider_config = PROVIDER_MODELS[provider]
-
-      if provider_config
-        default_model = provider_config[:default]
-        available_models = provider_config[:models]
-
-        puts "\nAvailable models for #{provider.capitalize}:"
-        available_models.each_with_index do |model, index|
-          marker = model == default_model ? ' (recommended)' : ''
-          puts "  #{index + 1}. #{model}#{marker}"
-        end
-
-        puts "  #{available_models.length + 1}. Enter custom model ID"
-        print "\nSelect model (1-#{available_models.length + 1}) or press Enter for default [#{default_model}]: "
+      examples = ProviderConfig.example_models(provider)
+      puts "Example models for #{ProviderConfig.display_name(provider)}: #{examples.join(', ')}"
+      print 'Enter model ID: '
+      model_id = $stdin.gets.chomp.strip
 
-        choice = $stdin.gets.chomp
-
-        if choice.empty?
-          default_model
-        elsif choice.to_i.between?(1, available_models.length)
-          available_models[choice.to_i - 1]
-        elsif choice.to_i == available_models.length + 1
-          print 'Enter custom model ID: '
-          custom_model = $stdin.gets.chomp
-          custom_model.empty? ? nil : custom_model
-        else
-          puts 'Invalid selection.'
-          nil
-        end
-      else
-        # Fallback for unknown providers
-        print 'Enter model ID: '
-        model_id = $stdin.gets.chomp
-
-        if model_id.empty?
-          puts 'Model ID is required.'
-          return nil
-        end
-
-        model_id
+      if model_id.empty?
+        puts 'Model ID is required.'
+        return nil
       end
+
+      model_id
     end
 
     def set_default_configuration
@@ -249,14 +220,15 @@ module Askcii
         return
       end
 
+      puts
       print 'Enter configuration ID to set as default: '
-      new_default = $stdin.gets.chomp
+      new_default = $stdin.gets.chomp.strip
 
       if configs.any? { |c| c['id'] == new_default }
         Askcii::Config.set_default_configuration(new_default)
-        puts "Configuration #{new_default} set as default."
+        puts "✓ Configuration #{new_default} set as default."
       else
-        puts 'Invalid configuration ID.'
+        puts '✗ Invalid configuration ID.'
       end
     end
 
@@ -268,18 +240,65 @@ module Askcii
         return
       end
 
+      puts
       print 'Enter configuration ID to delete: '
-      delete_id = $stdin.gets.chomp
+      delete_id = $stdin.gets.chomp.strip
 
       if configs.any? { |c| c['id'] == delete_id }
+        print "Are you sure you want to delete configuration #{delete_id}? (y/n): "
+        return unless $stdin.gets.chomp.downcase == 'y'
+
         if Askcii::Config.delete_configuration(delete_id)
-          puts "Configuration #{delete_id} deleted successfully."
+          puts "✓ Configuration #{delete_id} deleted successfully."
         else
-          puts 'Failed to delete configuration.'
+          puts '✗ Failed to delete configuration.'
         end
       else
-        puts 'Invalid configuration ID.'
+        puts '✗ Invalid configuration ID.'
       end
     end
+
+    def test_configuration
+      configs = Askcii::Config.configurations
+
+      if configs.empty?
+        puts 'No configurations available to test.'
+        return
+      end
+
+      puts
+      print 'Enter configuration ID to test: '
+      config_id = $stdin.gets.chomp.strip
+
+      config = configs.find { |c| c['id'] == config_id }
+
+      if config
+        test_specific_configuration(config)
+      else
+        puts '✗ Invalid configuration ID.'
+      end
+    end
+
+    def test_specific_configuration(config)
+      puts
+      puts "Testing configuration: #{config['name']}"
+      puts '-' * 50
+
+      # Validate first
+      begin
+        symbolized_config = config.transform_keys(&:to_sym)
+        ConfigValidator.validate!(symbolized_config)
+        puts '✓ Configuration validation passed'
+      rescue ConfigurationError => e
+        puts "✗ Validation failed: #{e.message}"
+        return
+      end
+
+      # TODO: Add actual connectivity test with minimal LLM call
+      # For now, just validate the configuration structure
+      puts '✓ Configuration appears valid'
+      puts
+      puts 'Note: Actual connectivity testing will be added in a future update.'
+    end
   end
 end