commitgpt 0.3.9 → 0.3.11

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: d812f02e181a923e72486f7ffc5ac1d2e7614bd5eee7194bca7e3a764df7afda
-  data.tar.gz: 7195e0edac02853ef4d1dcc7179664beffd023dacdb36c1133c5a01463a0175c
+  metadata.gz: b8a2b39ee34a81a322ba56a693b002e1a8b29bc6adc432b4f41992c71ff3bd4a
+  data.tar.gz: cd4289fee106ee2feb1e89c71b29889b6fe0232c542452588fd0564b4b49593f
 SHA512:
-  metadata.gz: 61f5d8a81bc519c7341dde8808656d729c8c3c6c826aa4ecb27779a5b59b3866f8f4fc671424139aa1fa801ce0d1bee14cac903363babafb082f5e3eb354904d
-  data.tar.gz: a32a40428daef8ecfe051e39fc0f4cf4040755190c01f0e92eb747aed232b8cc4acabe9ba88b6a9d27b0b7edf666d2da96a07f7d0e75e80ca47adc33b1eb9227
+  metadata.gz: 77b3f3fe14333b001537c52482b3424f2b5c679434b7ea52bb06e001e64cd5cc0f0c61782ead4d8416bf01473a7099398ba2d4eea84c268f84a14e6a62e5448b
+  data.tar.gz: 073de1a61a918be54e7ca2f2bb750b094aa1f0e1d56e195ad6239a8e4eb0431570ff4dfaedd22255e843d8c994df8c248d74f7cb1d306c6644e145e064c306ca

data/README.md

@@ -17,9 +17,12 @@ The easiest way to install and keep CommitGPT updated.
 
 ```bash
 brew tap ZPVIP/commitgpt https://github.com/ZPVIP/commitgpt
+brew trust zpvip/commitgpt
 brew install commitgpt
 ```
 
+> **Note**: Recent Homebrew versions refuse to load formulae from third-party taps until you trust them, so `brew trust` is required before installing.
+
 **Upgrade:**
 
 ```bash
@@ -194,8 +197,41 @@ docs: update API documentation
 📝 update API documentation
 ```
 
+### Choose Commit Message Detail Level
+Select how much the commit message says:
+```bash
+$ aicm -d
+# or
+$ aicm --detail
+```
+
+- **Concise** - Subject line only (default)
+- **Detailed** - Subject line, an explanatory paragraph, and a bullet list of notable changes
+
+This is independent of the format above: the format decides how the subject line is written, the detail level decides how much follows it. Your selection is saved as `commit_detail` in `~/.config/commitgpt/config.yml`.
+
+#### Detail Examples
+
+**Concise:**
+```
+feat(dfb): store and expand DFB event and speed logs
+```
+
+**Detailed:**
+```
+feat(dfb): store and expand DFB event and speed logs
+
+Persist DFB device logs ingested from the proactive `evt` (event log) and
+`lge` (speed stats) topics, one record per message.
+
+- Add `dfb_event_logs` and `dfb_speed_stats` tables and models, each with an
+  idempotent unique index so re-delivery and overlapping backfills are safe.
+- Associate both log tables on `Product`.
+- Update `README_DFB.md` to document the cloud tables.
+```
+
 ### Check Configuration
-View your current configuration (Provider, Model, Format, Base URL, Diff Len):
+View your current configuration (Provider, Model, Format, Detail, Base URL, Diff Len):
 ```bash
 $ aicm help
 ```
@@ -261,13 +297,17 @@ Get an API key at [https://developer.amd.com.cn/radeon/tokenfactory](https://dev
 
 > **Note**: Registration requires **both an email address and a Chinese (+86) mobile phone number**. Without a Chinese phone number you cannot complete sign-up, so this provider is only usable if you have one.
 
+> **Note**: The gateway in front of this endpoint always answers `GET /models` with 403, so models cannot be discovered automatically. CommitGPT ships the list below and offers it directly during setup — no detection request is made for this provider.
+
 ```
-DeepSeek-V4-Flash
-Qwen3.6-35B-A3B
-MiniCPM5-1B
-MiniCPM-V46
+DeepSeek-V4-Flash    # Max Context Length: 1,000,000 tokens
+Qwen3.6-35B-A3B      # Max Context Length:   262,144 tokens
+MiniCPM-V46          # Max Context Length:   262,144 tokens
+MiniCPM5-1B          # Max Context Length:   131,072 tokens
 ```
 
+Setup reports the selected model's context window and suggests a matching `diff_len`, so you don't have to work the number out yourself.
+
 ## How It Works
 This CLI tool runs a `git diff` command to grab all staged changes, sends this to OpenAI's GPT API (or compatible endpoint), and returns an AI-generated commit message. The tool uses the `/v1/chat/completions` endpoint with optimized prompts/system instructions for generating conventional commit messages.
 

data/lib/commitgpt/cli.rb

@@ -14,6 +14,7 @@ module CommitGpt
     method_option :verbose, aliases: '-v', type: :boolean, desc: 'Show git diff being sent to AI'
     method_option :provider, aliases: '-p', type: :boolean, desc: 'Switch active provider'
     method_option :format, aliases: '-f', type: :boolean, desc: 'Choose commit message format'
+    method_option :detail, aliases: '-d', type: :boolean, desc: 'Choose commit message detail level'
     def generate
       if options[:provider]
         CommitGpt::SetupWizard.new.switch_provider
@@ -21,6 +22,8 @@ module CommitGpt
         CommitGpt::SetupWizard.new.change_model
       elsif options[:format]
         CommitGpt::SetupWizard.new.choose_format
+      elsif options[:detail]
+        CommitGpt::SetupWizard.new.choose_detail
       else
         CommitGpt::CommitAi.new.aicm(verbose: options[:verbose])
       end
@@ -42,6 +45,7 @@ module CommitGpt
       shell.say '  -m, --models             # Interactive model selection'
       shell.say '  -p, --provider           # Switch active provider'
       shell.say '  -f, --format             # Choose commit message format'
+      shell.say '  -d, --detail             # Choose commit message detail level'
       shell.say '  -v, --verbose            # Show git diff being sent to AI'
       shell.say ''
 
@@ -61,6 +65,7 @@ module CommitGpt
           shell.say "  Provider:  #{config['name'].green}"
           shell.say "  Model:     #{config['model'].cyan}"
           shell.say "  Format:    #{format.capitalize.yellow}"
+          shell.say "  Detail:    #{CommitGpt::ConfigManager.get_commit_detail.capitalize.yellow}"
           shell.say "  Base URL:  #{config['base_url']}"
           shell.say "  Diff Len:  #{config['diff_len']}"
           shell.say "  Timeout:   #{CommitGpt::ConfigManager.get_model_fetch_timeout}s"

data/lib/commitgpt/commit_ai.rb

@@ -133,6 +133,51 @@ module CommitGpt
       end
 
       @commit_format = ConfigManager.get_commit_format
+      @commit_detail = ConfigManager.get_commit_detail
+    end
+
+    # True when the message should carry a body, not just a subject line
+    def detailed?
+      @commit_detail == 'detailed'
+    end
+
+    # Streaming cutoff and completion budget, both of which have to grow to fit a body
+    def max_content_chars
+      detailed? ? 2000 : 300
+    end
+
+    def reasoning_disabled_max_tokens
+      detailed? ? 1000 : 300
+    end
+
+    # Header printed before the streamed message starts arriving
+    def content_prefix
+      detailed? ? "✦ Commit message:\n" : '✦ Commit message: git commit -am "'
+    end
+
+    # Closing quote that matches content_prefix; a body has no quote to close
+    def content_suffix
+      detailed? ? '' : '"'
+    end
+
+    # Editor git will actually use, resolved through its own precedence chain
+    def git_editor
+      editor = `git var GIT_EDITOR 2>/dev/null`.strip
+      editor.empty? ? 'your editor' : editor
+    end
+
+    # Trim the model output down to what git should receive
+    def finalize_message(full_content)
+      unless detailed?
+        # Take only the first non-empty line to avoid repetition or multi-line garbage
+        first_line = full_content.split("\n").map(&:strip).reject(&:empty?).first
+        return first_line&.gsub(/\A["']|["']\z/, '') || ''
+      end
+
+      body = full_content.strip.gsub(/\A["']|["']\z/, '').strip
+      # Keep the subject/body/bullets structure but drop trailing spaces and
+      # collapse the extra blank lines models like to emit between sections
+      body.split("\n").map(&:rstrip).join("\n").gsub(/\n{3,}/, "\n\n")
     end
 
     def aicm(verbose: false)
@@ -150,9 +195,10 @@ module CommitGpt
 
         case action
         when :commit
-          commit_command = "git commit -m \"#{ai_commit_message}\""
-          puts "\n→ Executing: #{commit_command}".yellow
-          system(commit_command)
+          # Pass argv directly: a body with quotes, backticks or newlines must
+          # not be re-interpreted by the shell
+          puts "\n→ Executing: git commit -m <message>".yellow
+          system('git', 'commit', '-m', ai_commit_message)
           puts "\n\n"
           puts `git log -1`
           break
@@ -160,15 +206,15 @@ module CommitGpt
           puts "\n"
           next
         when :edit
-          prompt = TTY::Prompt.new
-          new_message = prompt.ask('Enter your commit message:')
-          if new_message && !new_message.strip.empty?
-             commit_command = "git commit -m \"#{new_message}\""
-             system(commit_command)
-             puts "\n"
-             puts `git log -1`
+          # Let git open the message in the user's editor: it honours the
+          # GIT_EDITOR/core.editor/VISUAL/EDITOR chain, handles multi-line
+          # bodies, and aborts by itself if the message is left empty
+          puts "\n→ Opening #{git_editor} with the generated message...".yellow
+          if system('git', 'commit', '-e', '-m', ai_commit_message)
+            puts "\n"
+            puts `git log -1`
           else
-             puts '✖ Commit aborted (empty message).'.red
+            puts '✖ Commit aborted.'.red
           end
           break
         when :exit
@@ -219,7 +265,7 @@ module CommitGpt
         prompt.select('Action:') do |menu|
           menu.choice 'Commit', :commit
           menu.choice 'Regenerate', :regenerate
-          menu.choice 'Edit', :edit
+          menu.choice 'Edit in editor', :edit
           menu.choice 'Exit without commit', :exit
         end
       rescue TTY::Reader::InputInterrupt, Interrupt
@@ -279,7 +325,11 @@ module CommitGpt
     # rubocop:disable Metrics/AbcSize, Metrics/CyclomaticComplexity, Metrics/MethodLength, Metrics/PerceivedComplexity, Metrics/BlockLength
     def generate_commit(diff = '', chunk_label: nil)
       # Build format-specific prompt
-      base_prompt = 'Generate a concise git commit message title in present tense that precisely describes the key changes in the following code diff. Focus on what was changed, not just file names. Provide only the title, no description or body.'
+      base_prompt = if detailed?
+                      'Generate a git commit message in present tense that precisely describes the key changes in the following code diff. Focus on what was changed and why, not just file names.'
+                    else
+                      'Generate a concise git commit message title in present tense that precisely describes the key changes in the following code diff. Focus on what was changed, not just file names. Provide only the title, no description or body.'
+                    end
 
       format_instruction = case @commit_format
                            when 'conventional'
@@ -290,17 +340,38 @@ module CommitGpt
                              ''
                            end
 
-      format_spec = "The output response must be in format:\n#{COMMIT_FORMATS[@commit_format]}"
+      format_spec = if detailed?
+                      "The subject line must be in format:\n#{COMMIT_FORMATS[@commit_format]}"
+                    else
+                      "The output response must be in format:\n#{COMMIT_FORMATS[@commit_format]}"
+                    end
+
+      rules = if detailed?
+                [
+                  '- The first line is the subject: a maximum of 100 characters, no trailing period.',
+                  '- Then one blank line, then a body paragraph of 1-3 sentences explaining what changed and why.',
+                  '- Then one blank line, then a bullet list where each bullet starts with "- " and describes one notable change.',
+                  '- Wrap body and bullet text at 72 characters.',
+                  '- Be specific: include concrete details (file, table, method, package names) rather than generic statements.',
+                  '- Do not use markdown headings, bold, or code fences. Plain text only.',
+                  '- Exclude anything unnecessary such as translation. Your entire response will be passed directly into git commit.',
+                  '- Return ONLY the commit message, nothing else. No explanations, no introductions, no surrounding quotes.'
+                ]
+              else
+                [
+                  '- Commit message must be a maximum of 100 characters.',
+                  '- Exclude anything unnecessary such as translation. Your entire response will be passed directly into git commit.',
+                  '- IMPORTANT: Do not include any explanations, introductions, or additional text. Do not wrap the commit message in quotes or any other formatting. The commit message must not exceed 100 characters. Respond with ONLY the commit message text.',
+                  '- Be specific: include concrete details (package names, versions, functionality) rather than generic statements.',
+                  '- Return ONLY the commit message, nothing else.'
+                ]
+              end
 
       system_content = [
         base_prompt,
         'Message language: English.',
         'Rules:',
-        '- Commit message must be a maximum of 100 characters.',
-        '- Exclude anything unnecessary such as translation. Your entire response will be passed directly into git commit.',
-        '- IMPORTANT: Do not include any explanations, introductions, or additional text. Do not wrap the commit message in quotes or any other formatting. The commit message must not exceed 100 characters. Respond with ONLY the commit message text.',
-        '- Be specific: include concrete details (package names, versions, functionality) rather than generic statements.',
-        '- Return ONLY the commit message, nothing else.',
+        *rules,
         format_instruction,
         format_spec
       ].reject(&:empty?).join("\n")
@@ -331,7 +402,7 @@ module CommitGpt
 
       if can_disable_reasoning
         payload[:disable_reasoning] = true
-        payload[:max_tokens] = 300
+        payload[:max_tokens] = reasoning_disabled_max_tokens
       else
         payload[:max_tokens] = configured_max_tokens
       end
@@ -430,13 +501,13 @@ module CommitGpt
                     if chunk_label
                       print "◆ #{chunk_label}: ".magenta
                     else
-                      print '✦ Commit message: git commit -am "'.green
+                      print content_prefix.green
                     end
                     printed_content_prefix = true
                   end
 
                   # Prevent infinite loops/repetitive garbage
-                  if full_content.length + content_chunk.length > 300
+                  if full_content.length + content_chunk.length > max_content_chars
                     stop_stream = true
                     break
                   end
@@ -460,7 +531,7 @@ module CommitGpt
       end
 
       # Close the quote
-      puts(chunk_label ? '' : '"'.green) if printed_content_prefix
+      puts(chunk_label ? '' : content_suffix.green) if printed_content_prefix
 
       # Post-processing Logic (Retry if empty content)
       if (full_content.nil? || full_content.strip.empty?) && (full_reasoning && !full_reasoning.strip.empty?)
@@ -511,9 +582,7 @@ module CommitGpt
       # Reset retrying flag
       @is_retrying = false
 
-      # Take only the first non-empty line to avoid repetition or multi-line garbage
-      first_line = full_content.split("\n").map(&:strip).reject(&:empty?).first
-      first_line&.gsub(/\A["']|["']\z/, '') || ''
+      finalize_message(full_content)
     end
     # rubocop:enable Metrics/AbcSize, Metrics/CyclomaticComplexity, Metrics/MethodLength, Metrics/PerceivedComplexity, Metrics/BlockLength
 
@@ -530,14 +599,31 @@ module CommitGpt
                              ''
                            end
 
+      rules = if detailed?
+                [
+                  '- The first line is the subject: a maximum of 100 characters, no trailing period.',
+                  '- Then one blank line, then a body paragraph of 1-3 sentences explaining what changed and why.',
+                  '- Then one blank line, then a bullet list where each bullet starts with "- " and describes one notable change.',
+                  '- Wrap body and bullet text at 72 characters.',
+                  '- Present tense.',
+                  '- Be specific: include concrete details rather than generic statements.',
+                  '- Do not use markdown headings, bold, or code fences. Plain text only.',
+                  '- Return ONLY the commit message, nothing else. No quotes, no explanations.'
+                ]
+              else
+                [
+                  '- Maximum 100 characters.',
+                  '- Present tense.',
+                  '- Be specific: include concrete details rather than generic statements.',
+                  '- Return ONLY the commit message, nothing else. No quotes, no explanations.'
+                ]
+              end
+
       system_content = [
         'You are given multiple commit messages generated from different segments of a single large git diff.',
-        'Synthesize them into ONE concise, unified git commit message that captures the overall change.',
+        "Synthesize them into ONE unified git commit message that captures the overall change.#{' Keep it concise.' unless detailed?}",
         'Rules:',
-        '- Maximum 100 characters.',
-        '- Present tense.',
-        '- Be specific: include concrete details rather than generic statements.',
-        '- Return ONLY the commit message, nothing else. No quotes, no explanations.',
+        *rules,
         format_instruction
       ].reject(&:empty?).join("\n")
 
@@ -558,7 +644,7 @@ module CommitGpt
 
       if can_disable_reasoning
         payload[:disable_reasoning] = true
-        payload[:max_tokens] = 300
+        payload[:max_tokens] = reasoning_disabled_max_tokens
       else
         payload[:max_tokens] = provider_config['max_tokens'] || 2000
       end
@@ -606,11 +692,11 @@ module CommitGpt
                 content_chunk = delta['content']
                 if content_chunk && !content_chunk.empty?
                   unless printed_content_prefix
-                    print "\n✦ Commit message: git commit -am \"".green
+                    print "\n#{content_prefix}".green
                     printed_content_prefix = true
                   end
 
-                  if full_content.length + content_chunk.length > 300
+                  if full_content.length + content_chunk.length > max_content_chars
                     stop_stream = true
                     break
                   end
@@ -630,15 +716,14 @@ module CommitGpt
         return nil
       end
 
-      puts '"'.green if printed_content_prefix
+      puts content_suffix.green if printed_content_prefix
 
       if full_content.strip.empty?
         puts '✖ No response from AI during synthesis.'.red
         return nil
       end
 
-      first_line = full_content.split("\n").map(&:strip).reject(&:empty?).first
-      first_line&.gsub(/\A["']|["']\z/, '') || ''
+      finalize_message(full_content)
     end
   end
 end

data/lib/commitgpt/config_manager.rb

@@ -11,6 +11,11 @@ module CommitGpt
     # Seconds to wait when fetching a provider's model list
     DEFAULT_MODEL_FETCH_TIMEOUT = 30
 
+    # How much the commit message says: a subject line only, or a subject
+    # followed by an explanatory body
+    COMMIT_DETAILS = %w[concise detailed].freeze
+    DEFAULT_COMMIT_DETAIL = 'concise'
+
     class << self
       # Get the config directory path
       def config_dir
@@ -168,6 +173,23 @@ module CommitGpt
         save_main_config(main_config)
       end
 
+      # Get commit detail level ('concise' or 'detailed')
+      def get_commit_detail
+        return DEFAULT_COMMIT_DETAIL unless config_exists?
+
+        main_config = YAML.load_file(main_config_path)
+        detail = main_config['commit_detail']
+        COMMIT_DETAILS.include?(detail) ? detail : DEFAULT_COMMIT_DETAIL
+      end
+
+      # Set commit detail level
+      def set_commit_detail(detail)
+        ensure_config_dir
+        main_config = config_exists? ? YAML.load_file(main_config_path) : { 'providers' => [], 'active_provider' => '' }
+        main_config['commit_detail'] = detail
+        save_main_config(main_config)
+      end
+
       # Get the model list fetch timeout (seconds)
       def get_model_fetch_timeout
         return DEFAULT_MODEL_FETCH_TIMEOUT unless config_exists?

data/lib/commitgpt/provider_presets.rb

@@ -3,7 +3,16 @@
 module CommitGpt
   # Provider presets for common AI providers
   PROVIDER_PRESETS = [
-    { label: 'AMD Radeon (China)', value: 'amd', base_url: 'https://developer.amd.com.cn/radeon/api/v1' },
+    # The Azure gateway in front of this endpoint always answers /models with 403,
+    # so the model list is built in and no request is made during setup.
+    # Each value is that model's context window in tokens.
+    { label: 'AMD Radeon (China)', value: 'amd', base_url: 'https://developer.amd.com.cn/radeon/api/v1',
+      models: {
+        'DeepSeek-V4-Flash' => 1_000_000,
+        'MiniCPM-V46' => 262_144,
+        'MiniCPM5-1B' => 131_072,
+        'Qwen3.6-35B-A3B' => 262_144
+      } },
     { label: 'Anthropic Claude', value: 'anthropic', base_url: 'https://api.anthropic.com/v1' },
     { label: 'Cerebras', value: 'cerebras', base_url: 'https://api.cerebras.ai/v1' },
     { label: 'DeepSeek', value: 'deepseek', base_url: 'https://api.deepseek.com' },

data/lib/commitgpt/setup_wizard.rb

@@ -10,6 +10,18 @@ require_relative 'string'
 module CommitGpt
   # Interactive setup wizard for configuring AI providers
   class SetupWizard
+    # Diff bytes per context token. Code and CJK pack more bytes into a token
+    # than English prose, so this stays deliberately conservative.
+    DIFF_BYTES_PER_TOKEN = 2
+
+    # Ceiling for the suggested diff length. Past this a single request gets
+    # slow and one commit message stops describing the change usefully, so
+    # chunked mode is the better answer than a bigger window.
+    MAX_SUGGESTED_DIFF_LEN = 131_072
+
+    # Used when the model's context window is unknown
+    FALLBACK_DIFF_LEN = 32_768
+
     def initialize
       @prompt = TTY::Prompt.new
     end
@@ -45,13 +57,13 @@ module CommitGpt
       provider = config['providers'].find { |p| p['name'] == selected }
 
       # Fetch models and let user select
-      models = fetch_models_with_timeout(provider['base_url'], provider['api_key'])
+      models = fetch_models_with_timeout(provider['base_url'], provider['api_key'], preset_models(selected))
       return if models.nil?
 
       model = select_model(models, provider['model'])
 
       # Prompt for diff length
-      diff_len = prompt_diff_len(provider['diff_len'] || 32_768)
+      diff_len = prompt_diff_len(default_diff_len(selected, model, provider['diff_len']))
 
       # Update config
       ConfigManager.update_provider(selected, { 'model' => model, 'diff_len' => diff_len })
@@ -75,7 +87,8 @@ module CommitGpt
       end
 
       # Fetch models and let user select
-      models = fetch_models_with_timeout(provider_config['base_url'], provider_config['api_key'])
+      models = fetch_models_with_timeout(provider_config['base_url'], provider_config['api_key'],
+                                         preset_models(provider_config['name']))
       return if models.nil?
 
       model = select_model(models, provider_config['model'])
@@ -103,8 +116,26 @@ module CommitGpt
       puts "\n✔ Commit format set to: #{format}".green
     end
 
+    # Choose how much the commit message says
+    def choose_detail
+      puts "\n→ Choose git commit message detail level:\n".green
+
+      detail = @prompt.select('Select detail level:', default: default_detail_index) do |menu|
+        menu.choice 'Concise - Subject line only', 'concise'
+        menu.choice 'Detailed - Subject line, explanatory paragraph, and bullet list of changes', 'detailed'
+      end
+
+      ConfigManager.set_commit_detail(detail)
+      puts "\n✔ Commit detail set to: #{detail}".green
+    end
+
     private
 
+    # Menu position of the currently configured detail level (1-based)
+    def default_detail_index
+      ConfigManager::COMMIT_DETAILS.index(ConfigManager.get_commit_detail).to_i + 1
+    end
+
     # Select provider from list
     def select_provider
       config = ConfigManager.load_config
@@ -148,14 +179,14 @@ module CommitGpt
       return if api_key.nil? # User cancelled
 
       # Fetch models with timeout
-      models = fetch_models_with_timeout(base_url, api_key)
+      models = fetch_models_with_timeout(base_url, api_key, preset_models(provider_name))
       return if models.nil?
 
       # Let user select model
       model = select_model(models, existing_provider&.dig('model'))
 
       # Prompt for diff length
-      diff_len = prompt_diff_len(existing_provider&.dig('diff_len') || 32_768)
+      diff_len = prompt_diff_len(default_diff_len(provider_name, model, existing_provider&.dig('diff_len')))
 
       # Save configuration
       ConfigManager.update_provider(
@@ -181,14 +212,14 @@ module CommitGpt
         q.default 'http://localhost:8080/v1'
       end
 
-      api_key = @prompt.mask('Enter your API key (optional):') { |q| q.echo false }
+      api_key = @prompt.mask('Enter your API key (optional):') { |q| q.echo false }.to_s
 
       # Fetch models
       models = fetch_models_with_timeout(base_url, api_key)
       return if models.nil?
 
       model = select_model(models)
-      diff_len = prompt_diff_len(32_768)
+      diff_len = prompt_diff_len(FALLBACK_DIFF_LEN)
 
       # Add to presets dynamically (just for this session)
       # Save to config
@@ -230,20 +261,19 @@ module CommitGpt
 
     # Prompt for API key
     def prompt_api_key(_provider_name, existing_key)
-      message = if existing_key && !existing_key.empty?
+      has_existing = existing_key && !existing_key.empty?
+
+      message = if has_existing
                   'Enter your API key (press Enter to keep existing):'
                 else
                   'Enter your API key:'
                 end
 
-      key = @prompt.mask(message) { |q| q.echo false }
+      # mask returns nil, not '', when the input is empty
+      key = @prompt.mask(message) { |q| q.echo false }.to_s
 
       # If user pressed Enter and there's an existing key, use it
-      if key.empty? && existing_key && !existing_key.empty?
-        existing_key
-      else
-        key
-      end
+      key.empty? && has_existing ? existing_key : key
     end
 
     # Ask how long to wait when fetching the model list, and persist it
@@ -262,12 +292,56 @@ module CommitGpt
       seconds
     end
 
-    # Fetch models from provider with timeout
-    def fetch_models_with_timeout(base_url, api_key)
+    # Built-in models a preset declares, as { model => context tokens }
+    def preset_model_table(provider_name)
+      models = PROVIDER_PRESETS.find { |p| p[:value] == provider_name }&.dig(:models)
+      models unless models.nil? || models.empty?
+    end
+
+    # Built-in model list for a preset, if it declares one
+    def preset_models(provider_name)
+      preset_model_table(provider_name)&.keys
+    end
+
+    # Context window of a built-in model, in tokens
+    def model_context_tokens(provider_name, model)
+      preset_model_table(provider_name)&.dig(model)
+    end
+
+    # Default diff length for a model, derived from its context window when
+    # known, and reported so the number is not a mystery. Numbers are printed
+    # undelimited because they get typed straight into the next prompt.
+    def default_diff_len(provider_name, model, existing)
+      context_tokens = model_context_tokens(provider_name, model)
+      return existing || FALLBACK_DIFF_LEN if context_tokens.nil?
+
+      fits = context_tokens * DIFF_BYTES_PER_TOKEN
+      suggested = [fits, MAX_SUGGESTED_DIFF_LEN].min
+      capped = ' (capped; larger diffs use chunked mode)' if suggested < fits
+
+      puts "\n#{model} context window: #{context_tokens} tokens (~#{fits} bytes of diff)".gray
+      puts "  Suggested max diff length: #{suggested} bytes#{capped}".gray
+
+      # Only a value the user actually chose outranks the suggestion
+      customized = existing && existing != FALLBACK_DIFF_LEN
+      customized ? existing : suggested
+    end
+
+    # Fetch models from provider with timeout.
+    # A preset that declares :models has no usable /models endpoint, so its
+    # built-in list is offered directly and no request is made.
+    def fetch_models_with_timeout(base_url, api_key, builtin_models = nil)
+      if builtin_models
+        puts 'Using the built-in model list (this provider does not expose /models).'.gray
+        return builtin_models
+      end
+
       timeout = ConfigManager.get_model_fetch_timeout
       puts 'Fetching available models...'.gray
 
       models = nil
+      failure = nil
+
       begin
         Timeout.timeout(timeout) do
           headers = {
@@ -279,28 +353,23 @@ module CommitGpt
           response = HTTParty.get("#{base_url}/models", headers: headers)
 
           if response.code == 200
-            models = response['data'] || []
-            models = models.map { |m| m['id'] }.compact.sort
+            models = (response['data'] || []).map { |m| m['id'] }.compact.sort
           else
-            puts "✖ Failed to fetch models: HTTP #{response.code}".red
-            return nil
+            failure = "Failed to fetch models: HTTP #{response.code}"
           end
         end
       rescue Timeout::Error
-        puts "✖ Connection timeout (#{timeout}s). Please check your network, base_url, and api_key.".red
-        puts "  Run 'aicm setup' again to raise the timeout.".gray
-        exit(0)
+        failure = "Connection timeout (#{timeout}s). Please check your network, base_url, and api_key."
       rescue StandardError => e
-        puts "✖ Error fetching models: #{e.message}".red
-        exit(0)
+        failure = "Error fetching models: #{e.message}"
       end
 
-      if models.nil? || models.empty?
-        puts '✖ No models found. Please check your configuration.'.red
-        exit(0)
-      end
+      failure ||= 'No models found. Please check your configuration.' if models.nil? || models.empty?
+      return models if failure.nil?
 
-      models
+      puts "✖ #{failure}".red
+      puts "  Run 'aicm setup' again to raise the timeout.".gray if failure.start_with?('Connection timeout')
+      exit(0)
     end
 
     # Let user select a model
@@ -328,7 +397,7 @@ module CommitGpt
     end
 
     # Prompt for diff length
-    def prompt_diff_len(default = 32_768)
+    def prompt_diff_len(default = FALLBACK_DIFF_LEN)
       answer = @prompt.ask('Set the maximum diff length (Bytes) for generating commit message:') do |q|
         q.default default.to_s
         q.convert :int

data/lib/commitgpt/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module CommitGpt
-  VERSION = '0.3.9'
+  VERSION = '0.3.11'
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: commitgpt
 version: !ruby/object:Gem::Version
-  version: 0.3.9
+  version: 0.3.11
 platform: ruby
 authors:
 - Peng Zhang