commiti 1.3.1 → 1.3.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 026566db469248936e47f40faa446650926b7bb893089246a6c23a45dfaa432a
-  data.tar.gz: 251995106b1883d61cb12c00f911a4e3158d6d47e9b9d1b434d2026234dd3c2f
+  metadata.gz: 7853991e7fa65bd7c35642c50b00aead27bc7b3a4c11ffec7bb38be03a9e96ca
+  data.tar.gz: a42e9fbfec33a80cf95befe00c14f3e2bd23e7d334606589e21d2e1b6ab31a6f
 SHA512:
-  metadata.gz: c0d857c40f016686ee37b6b7f2d1109c5d7f79a7037c0ed41d0e49e1d3556ff57ce0db9f1da3f0f26e4956840e5a7ab0376a39799f359673068542e89a11480c
-  data.tar.gz: 677c08e9d0d5ae62723f7bb3fdea97c3c3799dd5a3775b133ca5520af32a102d924863c5f733794e005b3ff82c79a22d30a7aaee578985118ec0af31cd13c8ce
+  metadata.gz: 82ed71618c42e1fbb745bfc04974a754b43396f232d26923c214a34738860d0a7566c90c22caa8e60dee383d953dcce0937e908a3bd81c1d1519e4cb84b3790a
+  data.tar.gz: b45f9669306182469b502bc98a2a1754c9ad3e9320b8801ac2a72bd3f412d0d01f348e8ac75fd5a82b002f84356b9e0ebb99df9c941ac6cc4607b8f53912c2ce

data/README.md

@@ -1,5 +1,7 @@
 # Commiti
 
+[![Coverage](https://img.shields.io/badge/coverage-90%25-brightgreen)](coverage/index.html) [![RSpec](https://img.shields.io/badge/rspec-passing-brightgreen)](#)
+
 AI-powered commit message and pull request description generator for Git repositories, using Google AI models.
 
 ## What It Does
@@ -39,25 +41,49 @@ commiti [options]
 
 ## Configuration
 
-Commiti uses a single configuration approach: environment variables.
+Commiti uses environment variables for secrets and a checked-in project config file for text-generation styling.
 
 Set variables in your shell, CI secret manager, or local `.env` file (in your project):
 
 ```dotenv
 GOOGLE_API_KEY=your_google_ai_key
 
+# Optional: provider API tokens for API-first PR/MR creation
+# COMMITI_GITHUB_TOKEN=your_github_token
+# COMMITI_GITLAB_TOKEN=your_gitlab_token
+# COMMITI_GITBUCKET_TOKEN=your_gitbucket_token
+
 # Optional overrides:
 # COMMITI_MODEL=gemma-4-31b-it
 # COMMITI_CANDIDATES=1
 # COMMITI_BASE_BRANCH=main
 # COMMITI_NO_COPY=false
 # COMMITI_AUTO_SPLIT=false
+
+# Optional per-project prompt styling (safe YAML, no code execution):
+# COMMITI_CONFIG=.commiti.yml
 ```
 
 `GEMINI_API_KEY` is also accepted as an alias for `GOOGLE_API_KEY`.
 
 You can copy `.env.example` as a starting point.
 
+For project-specific wording and structure, add a `.commiti.yml` file at the repo root:
+
+```yaml
+text_generation:
+  commit:
+    subject_case: uppercase # uppercase, lowercase, or preserve
+  pr:
+    sections:
+      - name: Overview
+        guidance: Summarize the change in one paragraph.
+      - name: Validation
+        guidance: Describe the checks or tests that were run.
+```
+
+The file is parsed with safe YAML loading, and Commiti only accepts declarative styling settings from it.
+
 Your API key is sent directly from your local process to Google's API.
 Commiti does not store it and does not proxy requests through any Commiti server.
 Never commit `.env` to git.
@@ -76,6 +102,8 @@ Never commit `.env` to git.
 By default, Commiti creates a single commit from staged changes.
 Use `--auto-split` to let Commiti group connected file changes into multiple atomic commits.
 
+When auto-split detects multiple groups, Commiti can optionally open a quick group editor so you can move files between groups before committing.
+
 1. Shows `git status --short`.
 2. Asks for confirmation before staging (`git add -A`).
 3. Ensures there are staged changes.
@@ -113,14 +141,52 @@ Commit edit mode uses:
    - `## Motivation`
    - `## Changes Made`
    - `## Testing Notes`
-3. Builds provider compare/MR URL with prefilled title/body using query params.
-  - GitHub: compare URL with `quick_pull=1` (opens the PR form directly)
-  - GitBucket: compare URL with `expand=1`
-  - GitLab: new merge request URL
-   - If the URL would exceed safe browser/provider limits, Commiti drops description prefill automatically and keeps the shortest usable URL.
+3. Attempts to create and open PR/MR:
+   - **API-first path** (when token is configured):
+     - GitHub/GitBucket: creates PR via provider API and opens the created PR URL.
+     - GitLab: creates MR via provider API and opens the created MR URL.
+   - **Fallback path** (when no token, provider unsupported, or API call fails):
+     - Opens browser with prefilled PR/MR form using query parameters.
+     - If the URL would exceed safe browser/provider limits (~1800 characters), Commiti keeps the title and intelligently truncates the description to the longest text that still fits.
 4. Asks before opening browser.
 
-The tool opens a browser URL only. It does not call provider APIs.
+Commiti can create PRs/MRs via provider APIs when tokens are configured, and always opens the resulting page in your browser.
+
+### Provider API Logic
+
+When you set a provider token in your configuration, Commiti uses an **API-first strategy**:
+
+**Supported Providers:**
+- **GitHub** (github.com and GitHub Enterprise): Uses GitHub REST API v3
+- **GitLab** (gitlab.com and self-hosted): Uses GitLab API v4
+- **GitBucket**: Uses GitHub-compatible API
+
+**Token Configuration:**
+```dotenv
+COMMITI_GITHUB_TOKEN=ghp_xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx
+COMMITI_GITLAB_TOKEN=glpat-xxxxxxxxxxxxxxxxxxxxxxxxxxxx
+COMMITI_GITBUCKET_TOKEN=xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx
+```
+
+**API Request Flow:**
+1. Parses the Git remote URL to extract provider, host, namespace, and repository.
+2. Constructs provider-specific API endpoint and authentication headers.
+3. Sends HTTP POST request with generated PR title and description.
+4. On success (HTTP 2xx): Returns the created PR/MR URL directly.
+5. On failure: Falls back to browser prefill with a user-friendly error message explaining why.
+
+**Error Handling:**
+- **Missing token**: Falls back to browser prefill. (Info message)
+- **Unsupported provider**: Falls back to browser prefill. (Warning message)
+- **API error**: Falls back to browser prefill with error details. (Warning message)
+- **Redirect handling**: Automatically follows HTTP redirects (301, 302, 307, 308) but aborts if redirected to a different host.
+- **Network errors**: Caught and reported with fallback to browser prefill.
+
+**Advantages of API-First:**
+- Creates PR/MR immediately without manual form interaction.
+- Preserves full description text (no URL length constraints).
+- Seamlessly opens the created PR/MR for immediate review and collaboration.
+- Gracefully degrades to browser prefill if API is unavailable.
 
 ### Diff Context Protocol
 
@@ -180,7 +246,7 @@ Core services:
 - `lib/services/diff_summarization/diff_summarizer.rb`: Orchestrates large-diff summarization and summary combine.
   - `lib/services/diff_summarization/batch_runner.rb`: Runs asynchronous, batched per-file summarization jobs.
   - `lib/services/diff_summarization/fallback_builder.rb`: Builds deterministic summaries when model summarization fails or times out.
-- `lib/services/helpers/config_loader.rb`: Loads configuration from environment variables.
+- `lib/services/helpers/config_loader.rb`: Loads environment config plus secure project-level text-generation styling.
   - `lib/services/helpers/prompt_builder.rb`: Builds strict system/user prompts for commit and PR modes.
   - `lib/services/helpers/interactive_prompt.rb`: Handles confirmation prompts, candidate selection, editor loop, and commit message validation.
   - `lib/services/helpers/clipboard.rb`: Provides cross-platform clipboard support.

data/bin/commiti

@@ -47,6 +47,12 @@ OptionParser.new do |opts|
 end.parse!
 
 begin
+  model_label = ENV.fetch('COMMITI_MODEL', Commiti::GoogleClient::DEFAULT_MODEL)
+  flow_label = options[:type] == :pr ? 'PR flow' : 'Commit flow'
+  base_label = options[:type] == :pr ? "Base: #{options[:base_branch] || 'main'}" : nil
+  meta = [flow_label, "Model: #{model_label}", base_label].compact.join(' • ')
+  puts Commiti::TerminalUI.banner(title: 'Commiti', subtitle: 'AI commit & PR generator', meta: meta)
+
   flow = if options[:type] == :pr
            Commiti::Flows::PrFlow.new(options: options)
          else

data/lib/commiti.rb

@@ -2,6 +2,7 @@
 
 require_relative 'services/git/git_reader'
 require_relative 'services/git/git_writer'
+require_relative 'services/text_generation_style'
 require_relative 'services/google_client'
 require_relative 'services/helpers/config_loader'
 require_relative 'services/git/diff_parser'
@@ -9,6 +10,7 @@ require_relative 'services/diff_summarization/diff_summarizer'
 require_relative 'services/helpers/prompt_builder'
 require_relative 'services/helpers/interactive_prompt'
 require_relative 'services/git/pr/pr_opener'
+require_relative 'services/git/pr/pr_creator'
 require_relative 'services/helpers/clipboard'
 require_relative 'services/helpers/terminal_ui'
 require_relative 'services/helpers/spinner'
@@ -18,6 +20,7 @@ require_relative 'services/message_presenter'
 require_relative 'services/git/commit/commit_staging'
 require_relative 'services/git/commit/commit_execution'
 require_relative 'services/git/commit/change_grouping'
+require_relative 'services/git/commit/group_editor'
 require_relative 'flows/base_flow'
 require_relative 'flows/commit_flow'
 require_relative 'flows/pr_flow'

data/lib/flows/base_flow.rb

@@ -18,7 +18,8 @@ module Commiti
           diff: diff,
           client: client,
           run_stage: method(:run_stage),
-          model: selected_model
+          model: selected_model,
+          text_generation_config: options[:text_generation]
         )
         Commiti::MessagePresenter.print_summarization_notice(context[:summarized_result])
 
@@ -91,7 +92,11 @@ module Commiti
       end
 
       def message_generator
-        @message_generator ||= Commiti::MessageGenerator.new(flow_type: flow_type, run_stage: method(:run_stage))
+        @message_generator ||= Commiti::MessageGenerator.new(
+          flow_type: flow_type,
+          run_stage: method(:run_stage),
+          text_generation_config: options[:text_generation]
+        )
       end
     end
   end

data/lib/flows/commit_flow.rb

@@ -61,7 +61,12 @@ module Commiti
       end
 
       def run_grouped_context(context:, client:, model:)
-        groups = context[:change_groups]
+        groups = Commiti::GroupEditor.edit(context[:change_groups])
+        if groups.length <= 1
+          single_context = groups.first ? build_context(diff: group_diff(groups.first), client:, model:) : context
+          return run_single_group_context(context: single_context, client:, model:)
+        end
+
         run_stage('Unstaging current index for grouped commit execution') { Commiti::GitWriter.unstage_all! }
 
         puts "\n#{Commiti::TerminalUI.status(:info, "Auto-split detected #{groups.length} connected change groups.")}"
@@ -75,8 +80,7 @@ module Commiti
         run_stage("Staging files for group #{index + 1}/#{total}") { Commiti::GitWriter.stage_files!(group[:files]) }
         return :continue unless run_stage('Checking staged changes') { Commiti::GitWriter.staged_changes? }
 
-        puts "\n#{Commiti::TerminalUI.header("Group #{index + 1}/#{total} files")}:"
-        group[:files].each { |path| puts "- #{path}" }
+        puts "\n#{Commiti::TerminalUI.panel("Group #{index + 1}/#{total} files", Commiti::TerminalUI.bullets(group[:files]))}\n"
 
         group_context = build_context(diff: group_diff(group), client:, model:)
         Commiti::MessagePresenter.print_summarization_notice(group_context[:summarized_result])

data/lib/flows/pr_flow.rb

@@ -20,25 +20,53 @@ module Commiti
       end
 
       def maybe_open_pr_page(description, base_branch)
-        pr_url = run_stage('Preparing prefilled PR URL') do
-          head_branch = Commiti::GitWriter.current_branch
-          origin_url = Commiti::GitWriter.origin_url
-          title = Commiti::PrOpener.suggest_title(description, head_branch: head_branch)
-          Commiti::PrOpener.compare_url(
+        head_branch = Commiti::GitWriter.current_branch
+        origin_url = Commiti::GitWriter.origin_url
+        title = Commiti::PrOpener.suggest_title(description, head_branch: head_branch)
+
+        prompt_text = 'Create PR and open it in browser now?'
+
+        unless Commiti::InteractivePrompt.ask_yes_no(prompt_text, default: :no)
+          puts "\n#{Commiti::TerminalUI.status(:warn, 'PR creation skipped.')}\n\n"
+          return
+        end
+
+        api_result = run_stage('Creating PR/MR via provider API (if token configured)') do
+          Commiti::PrCreator.create(
             origin_url: origin_url,
             base_branch: base_branch,
             head_branch: head_branch,
             title: title,
-            body: description
+            body: description,
+            config: options
           )
         end
 
-        if Commiti::InteractivePrompt.ask_yes_no('Open prefilled PR page in browser now?', default: :no)
-          run_stage('Opening browser') { Commiti::PrOpener.open_in_browser(pr_url) }
-          puts "\nOpened PR page:\n#{pr_url}\n\n"
-        else
-          puts "\nPR URL:\n#{pr_url}\n\n"
+        pr_url = api_result[:url]
+
+        if pr_url.nil?
+          case api_result[:reason]
+          when :missing_token
+            puts "\n#{Commiti::TerminalUI.status(:info, "No #{api_result[:provider]} token configured; using browser prefill fallback.")}"
+          when :unsupported_provider
+            puts "\n#{Commiti::TerminalUI.status(:warn, 'Provider API is unsupported; using browser prefill fallback.')}"
+          when :api_error
+            puts "\n#{Commiti::TerminalUI.status(:warn, "PR API create failed: #{api_result[:error]}. Using browser prefill fallback.")}"
+          end
+
+          pr_url = run_stage('Preparing prefilled PR URL') do
+            Commiti::PrOpener.compare_url(
+              origin_url: origin_url,
+              base_branch: base_branch,
+              head_branch: head_branch,
+              title: title,
+              body: description
+            )
+          end
         end
+
+        run_stage('Opening browser') { Commiti::PrOpener.open_in_browser(pr_url) }
+        puts "\n#{Commiti::TerminalUI.panel('Opened PR page', pr_url)}\n\n"
       end
     end
   end

data/lib/services/diff_summarization/fallback_builder.rb

@@ -12,49 +12,88 @@ module Commiti
 
       def fallback_summary(diff, chunks: nil)
         parsed_chunks = chunks || Commiti::DiffParser.split_by_file(diff)
-        files = []
+        files = file_stats_for(parsed_chunks)
+        return diff.to_s[0, FALLBACK_BYTES] if files.empty?
 
-        parsed_chunks.each do |chunk|
-          current = {
-            path: chunk[:path].to_s,
-            additions: 0,
-            deletions: 0,
-            status: 'modified'
-          }
+        render_fallback_summary(files)
+      end
 
-          chunk[:diff].to_s.each_line do |line|
-            stripped = line.strip
-            current[:status] = 'added' if stripped.start_with?('new file mode')
-            current[:status] = 'deleted' if stripped.start_with?('deleted file mode')
-            current[:status] = 'renamed' if stripped.start_with?('rename from ') || stripped.start_with?('rename to ')
+      private
 
-            next if line.start_with?('diff --git ', '+++', '---', '@@')
+      def file_stats_for(chunks)
+        chunks.map { |chunk| file_stats_for_chunk(chunk) }
+      end
+
+      def file_stats_for_chunk(chunk)
+        status, additions, deletions = file_status_and_counts(chunk[:diff])
+        {
+          path: chunk[:path].to_s,
+          additions: additions,
+          deletions: deletions,
+          status: status
+        }
+      end
 
-            current[:additions] += 1 if line.start_with?('+')
-            current[:deletions] += 1 if line.start_with?('-')
-          end
+      def file_status_and_counts(diff_text)
+        status = 'modified'
+        additions = 0
+        deletions = 0
 
-          files << current
+        diff_text.to_s.each_line do |line|
+          status = detect_status(line, current: status)
+          next if metadata_line?(line)
+
+          additions += 1 if line.start_with?('+')
+          deletions += 1 if line.start_with?('-')
         end
 
-        return diff.to_s[0, FALLBACK_BYTES] if files.empty?
+        [status, additions, deletions]
+      end
+
+      def detect_status(line, current:)
+        stripped = line.strip
+        return 'added' if stripped.start_with?('new file mode')
+        return 'deleted' if stripped.start_with?('deleted file mode')
+        return 'renamed' if stripped.start_with?('rename from ') || stripped.start_with?('rename to ')
 
-        lines = []
-        lines << '### Diff Overview'
-        lines << "- Total files changed: #{files.length}"
-        lines << ''
+        current
+      end
+
+      def metadata_line?(line)
+        line.start_with?('diff --git ', '+++', '---', '@@')
+      end
+
+      def render_fallback_summary(files)
+        summary_lines = [
+          '### Diff Overview',
+          "- Total files changed: #{files.length}",
+          ''
+        ]
+
+        append_file_sections(summary_lines, files)
+        append_truncation_notice(summary_lines, files)
 
+        summary_lines.join("\n").strip
+      end
+
+      def append_file_sections(summary_lines, files)
         files.first(MAX_FILES_IN_SUMMARY).each do |file|
-          lines << "### #{file[:path]}"
-          lines << "- Status: #{file[:status]}"
-          lines << "- Added lines: #{file[:additions]}"
-          lines << "- Removed lines: #{file[:deletions]}"
-          lines << ''
+          summary_lines.concat(render_file_section(file))
         end
+      end
 
-        lines << "...and #{files.length - MAX_FILES_IN_SUMMARY} more files" if files.length > MAX_FILES_IN_SUMMARY
+      def append_truncation_notice(summary_lines, files)
+        summary_lines << "...and #{files.length - MAX_FILES_IN_SUMMARY} more files" if files.length > MAX_FILES_IN_SUMMARY
+      end
 
-        lines.join("\n").strip
+      def render_file_section(file)
+        [
+          "### #{file[:path]}",
+          "- Status: #{file[:status]}",
+          "- Added lines: #{file[:additions]}",
+          "- Removed lines: #{file[:deletions]}",
+          ''
+        ]
       end
     end
   end

data/lib/services/flow_context_builder.rb

@@ -2,7 +2,7 @@
 
 module Commiti
   module FlowContextBuilder
-    def self.build(flow_type:, diff:, client:, run_stage:, model:)
+    def self.build(flow_type:, diff:, client:, run_stage:, model:, text_generation_config: nil)
       line_chunks = Commiti::DiffParser.split_by_file_lines(diff)
       diff_metadata = Commiti::DiffParser.metadata_from_line_chunks(line_chunks)
       change_groups = Commiti::ChangeGrouping.group(line_chunks)
@@ -21,7 +21,8 @@ module Commiti
         diff: summarized_result[:content],
         summarized: summarized_result[:summarized],
         raw_diff: diff,
-        diff_metadata: diff_metadata
+        diff_metadata: diff_metadata,
+        style_config: text_generation_config
       )
 
       {

data/lib/services/git/commit/change_grouping.rb

@@ -18,6 +18,12 @@ module Commiti
       end
     end
 
+    def self.related?(left, right)
+      return false if left.to_s.strip.empty? || right.to_s.strip.empty?
+
+      connected?(left, right)
+    end
+
     def self.connected_components(paths)
       visited = {}
       ordered_components = []

data/lib/services/git/commit/commit_execution.rb

@@ -10,31 +10,14 @@ module Commiti
 
         case action
         when :yes
-          errors = Commiti::InteractivePrompt.commit_message_errors(working_message)
-          unless errors.empty?
-            puts "\n#{Commiti::TerminalUI.status(:warn, 'Current message needs fixes before commit:')}"
-            errors.each { |error| puts "- #{error}" }
-
-            if Commiti::InteractivePrompt.ask_yes_no('Open editor to fix now?', default: :yes)
-              edited = edit_message_until_valid(working_message)
-              if edited.nil?
-                puts "\n#{Commiti::TerminalUI.status(:fail, 'Editor did not exit successfully. Commit skipped.')}\n\n"
-                return :skipped
-              end
-
-              working_message = edited
-              print_message.call(working_message)
-              next
-            end
-
-            puts "\n#{Commiti::TerminalUI.status(:warn, 'Commit skipped.')}\n\n"
-            return :skipped
-          end
+          result, next_message = handle_yes_action(
+            working_message,
+            run_stage: run_stage,
+            print_message: print_message
+          )
+          return result if result
 
-          output = run_stage.call('Writing commit') { Commiti::GitWriter.commit_with_message_file(working_message) }
-          puts output unless output.to_s.strip.empty?
-          puts "\n#{Commiti::TerminalUI.status(:success, 'Commit created.')}\n\n"
-          return :committed
+          working_message = next_message
         when :edit
           edited = edit_message_until_valid(working_message)
           if edited.nil?
@@ -45,12 +28,48 @@ module Commiti
           working_message = edited
           print_message.call(working_message)
         else
-          puts "\n#{Commiti::TerminalUI.status(:warn, 'Commit skipped.')}\n\n"
+          print_skip_message
           return :skipped
         end
       end
     end
 
+    def self.handle_yes_action(working_message, run_stage:, print_message:)
+      errors = Commiti::InteractivePrompt.commit_message_errors(working_message)
+      return commit_message(working_message, run_stage: run_stage) if errors.empty?
+
+      puts "\n#{Commiti::TerminalUI.status(:warn, 'Current message needs fixes before commit:')}"
+      errors.each { |error| puts Commiti::TerminalUI.bullet(error) }
+
+      unless Commiti::InteractivePrompt.ask_yes_no('Open editor to fix now?', default: :yes)
+        print_skip_message
+        return [:skipped, nil]
+      end
+
+      edited = edit_message_until_valid(working_message)
+      if edited.nil?
+        puts "\n#{Commiti::TerminalUI.status(:fail, 'Editor did not exit successfully. Commit skipped.')}\n\n"
+        return [:skipped, nil]
+      end
+
+      print_message.call(edited)
+      [nil, edited]
+    end
+    private_class_method :handle_yes_action
+
+    def self.commit_message(message, run_stage:)
+      output = run_stage.call('Writing commit') { Commiti::GitWriter.commit_with_message_file(message) }
+      puts output unless output.to_s.strip.empty?
+      puts "\n#{Commiti::TerminalUI.status(:success, 'Commit created.')}\n\n"
+      [:committed, nil]
+    end
+    private_class_method :commit_message
+
+    def self.print_skip_message
+      puts "\n#{Commiti::TerminalUI.status(:warn, 'Commit skipped.')}\n\n"
+    end
+    private_class_method :print_skip_message
+
     def self.edit_message_until_valid(initial_message)
       working = initial_message
 
@@ -69,7 +88,7 @@ module Commiti
         return edited if errors.empty?
 
         puts "\n#{Commiti::TerminalUI.status(:warn, 'Edited message needs fixes:')}"
-        errors.each { |error| puts "- #{error}" }
+        errors.each { |error| puts Commiti::TerminalUI.bullet(error) }
         return edited unless Commiti::InteractivePrompt.ask_yes_no('Re-open editor now?', default: :yes)
 
         working = edited

data/lib/services/git/commit/commit_staging.rb

@@ -11,7 +11,7 @@ module Commiti
       status = run_stage.call('Reading git status') { Commiti::GitWriter.status_short }
       raise 'No changes found in working tree.' if status.strip.empty?
 
-      puts "\n#{Commiti::TerminalUI.header('Current git status')}\n\n#{status}"
+      puts "\n#{Commiti::TerminalUI.panel('Current git status', status)}\n"
       return unless Commiti::InteractivePrompt.ask_yes_no('Run git add -A now?', default: :no)
 
       run_stage.call('Staging changes (git add -A)') { Commiti::GitWriter.stage_all! }