jeeves-git-commit 3.0.0 → 3.1.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 9dfc1f28648d8afaf7035592aff4dc5cc2bacda8bb2ae760caa04df038fc2359
-  data.tar.gz: 8d9f768afab1305f0ded8f14192af519f2d0c45cd9bcb7062c876ea113599488
+  metadata.gz: 4bf456f354f172b140a241db27b10459ae17c6b96748f3cab435f5dd7024c6ad
+  data.tar.gz: d8764cc9556d4b8e3d220310c43ec7163fa6195e959445240025d7ab94a9ae4d
 SHA512:
-  metadata.gz: f7bb5569cd7f8b204660bebc3c69f2cf25c96b120b3556a924d6f8bbfe16319a6acaa37550e98d9130966eefb420f1367e4942b1fe4325eb7c1970697c00f4dc
-  data.tar.gz: c1556ab3fa0eac2a91fd73579f4061a1968a6eae4134b875cff5f165ffb2ce0de5f9aa092106708909acfbccdc379f6d1d86d6c03222ba433ed99684370ac650
+  metadata.gz: 36df36dafc54fd9a2c609d238417f2f8740e7090c255590a84aa87da755726e63a5745f7fc0e8e64653c934f3b3beb3b8417e8f92c388cba2bce43c41c689b6a
+  data.tar.gz: 50077793c804ab2d87e6681b3c07f18ee4be560487c628b9fa249d6bce4c66a58a50eb3beb4ffbce267e1303deeb5fc8cf1c818fea2cf782d4986f6e994b3dfe

data/README.md

@@ -17,7 +17,7 @@ never going to make the missing endpoint appear.
 
 ## Install
 
-Use Ruby 3.3 through 4.0 and Git. Development uses the version in `.ruby-version`.
+Use Ruby 3.3 or newer and Git. Development uses Homebrew's current Ruby.
 
 ```sh
 gem install jeeves-git-commit
@@ -50,7 +50,7 @@ Plain `jeeves` then uses local generation; no per-command flag or cloud key is
 needed. See [configuration](docs/configuration.md) for installation, fish
 syntax, all settings, prompt customization, and input limits.
 
-Version 3 requires Ruby 3.3 through 4.0. It validates conventional commit
+Version 3 requires Ruby 3.3 or newer. It validates conventional commit
 subjects by default; use `GIT_COMMIT_MESSAGE_FORMAT=plain` for a custom format.
 Existing personal prompts are preserved. See the [release notes](docs/releases.md).
 
@@ -95,6 +95,11 @@ Git and generation failures stop the command with a nonzero exit status.
 Jeeves checks that staged changes and HEAD still match what it reviewed before
 committing. See [architecture and Git behavior](docs/architecture.md).
 
+Local generation defaults to a 65,536-token context. Oversized diffs are
+automatically shortened, with a warning and omission notices for the model.
+Only the model input is shortened; Git commits the complete staged changes.
+See [input limits](docs/configuration.md#output-checks-and-large-diffs).
+
 ## Develop
 
 ```sh
@@ -108,13 +113,15 @@ bin/doctor
 ```
 
 `bin/check` runs fast foundation checks. `bin/check --documents-only` checks
-Markdown. `bin/check --full` adds the foundation tests and Ruby tests, lint,
+Markdown. `bin/check --full` adds `bin/check-application`: Ruby tests, lint,
 and gem build. Application checks are not part of the two routine modes.
 Built gems go in `gems/`. Publishing with `rake push` is an explicit release
 action; it is never part of normal validation.
 
-- [Development workflow](docs/development-workflow.md): setup, knowledge search,
-  worktrees, hooks, runtime policy, and validation.
+- [Development workflow](docs/development-workflow.md): setup, validation,
+  runtime policy, and releases.
+- [Project Starter foundation](docs/foundation/README.md): knowledge search,
+  worktrees, hooks, task tracking, and shared engineering policy.
 - [Documents](docs/README.md) and [memory](memory/README.md).
 - [GitHub Issues](https://github.com/jubishop/Jeeves/issues): bugs and planned work.
 - [Agent instructions](AGENTS.md).
@@ -124,3 +131,11 @@ action; it is never part of normal validation.
 
 Jeeves uses the [MIT license](LICENSE). The adopted project foundation retains
 its separate [license notice](LICENSE.project-starter).
+
+## Local task tracking
+
+Use `td` for local tasks, progress, blockers, and session handoffs. After cloning,
+install td and run `td init` in the primary checkout. Use `td status` or
+`td monitor` to inspect progress. Follow the [task workflow](docs/foundation/task-tracking.md)
+for setup, review, worktrees, and local data. Keep GitHub Issues for shared scope
+and acceptance criteria, linked from related td tasks.

data/lib/jeeves/diff.rb

@@ -0,0 +1,90 @@
+# frozen_string_literal: true
+
+module Jeeves
+  class Diff
+    NOTICE = "[Jeeves: diff shortened. Excerpts may omit files, hunks, or parts of lines. " \
+             "Hunk ranges are from the original diff; omitted content is unknown.]\n"
+    OMITTED = "\n[... content omitted ...]\n"
+
+    def initialize(text)
+      @text = text
+    end
+
+    def fit(limit)
+      return @text if @text.bytesize <= limit
+
+      budget = limit - NOTICE.bytesize
+      if budget < OMITTED.bytesize + 8
+        raise Error, 'Too little room for a shortened diff. Shorten the prompt or increase ' \
+                     'GIT_COMMIT_LOCAL_CONTEXT and GIT_COMMIT_MAX_DIFF_BYTES.'
+      end
+      compact = without_context
+      return NOTICE + compact if compact.bytesize <= budget
+
+      files = compact.split(/(?=^diff --(?:git |cc |combined ))/)
+      NOTICE + fit_parts(files, budget) { |file, size| fit_file(file, size) }
+    end
+
+    private
+
+    def without_context
+      context_prefix = nil
+      @text.each_line.reject do |line|
+        context_prefix = nil if line.start_with?('diff --')
+        match = line.match(/\A(@{2,}) /)
+        context_prefix = ' ' * (match[1].length - 1) if match
+        context_prefix && line.start_with?(context_prefix)
+      end.join
+    end
+
+    def fit_file(file, budget)
+      return file if file.bytesize <= budget
+
+      header = header_for(file)
+      body = file.delete_prefix(header)
+      hunks = body.split(/(?=^@@)/)
+      header + fit_parts(hunks, budget - header.bytesize) do |hunk, size|
+        hunk_header = header_for(hunk)
+        hunk_header + excerpt(hunk.delete_prefix(hunk_header), size - hunk_header.bytesize)
+      end
+    end
+
+    def header_for(part)
+      lines = part.lines
+      return lines.first.to_s if lines.first&.start_with?('@@')
+
+      lines.take_while { |line| !line.start_with?('@@') }.join
+    end
+
+    def fit_parts(parts, budget)
+      return parts.join if parts.sum(&:bytesize) <= budget
+
+      # Keep file metadata and hunk headers before sharing space for changes.
+      minimums = parts.map { |part| [part.bytesize, header_for(part).bytesize + OMITTED.bytesize + 8].min }
+      return excerpt(parts.join, budget) if minimums.sum > budget
+
+      sizes = minimums.dup
+      remaining = budget - sizes.sum
+      order = parts.each_index.sort_by { |index| parts[index].bytesize - sizes[index] }
+      order.each_with_index do |index, position|
+        extra = [parts[index].bytesize - sizes[index], remaining / (order.length - position)].min
+        sizes[index] += extra
+        remaining -= extra
+      end
+      parts.each_with_index.map { |part, index| yield part, sizes[index] }.join
+    end
+
+    def excerpt(text, budget)
+      return text if text.bytesize <= budget
+
+      available = budget - OMITTED.bytesize
+      head_size = (available + 1) / 2
+      tail_size = available / 2
+      head = text.byteslice(0, head_size).scrub('')
+      tail = text.byteslice(-tail_size, tail_size).scrub('')
+      head = head[0..head.rindex("\n")] if head.include?("\n")
+      tail = tail[(tail.index("\n") + 1)..] if tail.include?("\n")
+      head + OMITTED + tail
+    end
+  end
+end

data/lib/jeeves/prompt.rb

@@ -10,7 +10,7 @@ module Jeeves
       @errors = errors
     end
 
-    def render(diff)
+    def render(diff, settings:)
       path = @repository && File.join(@repository, '.jeeves_prompt')
       unless path && File.file?(path)
         path = @global_path
@@ -19,7 +19,21 @@ module Jeeves
       template = File.read(path, encoding: 'UTF-8')
       raise Error, "Prompt must contain {{DIFF}}: #{path}" unless template.include?('{{DIFF}}')
 
-      template.gsub('{{DIFF}}') { diff }
+      limit = settings.max_diff_bytes
+      if settings.prompt_budget
+        overhead = template.gsub('{{DIFF}}', '').bytesize
+        available = (settings.prompt_budget - overhead) / template.scan('{{DIFF}}').length
+        if available < 1
+          raise Error, 'The prompt leaves no room for a diff. Shorten the prompt or increase GIT_COMMIT_LOCAL_CONTEXT.'
+        end
+        limit = [limit, available].min
+      end
+      shortened = Diff.new(diff).fit(limit)
+      if shortened != diff
+        @errors.puts "Warning: diff shortened from #{diff.bytesize} to #{shortened.bytesize} bytes. " \
+                     'Some content is omitted; the commit message may miss changes.'
+      end
+      template.gsub('{{DIFF}}') { shortened }
     end
 
     private

data/lib/jeeves/providers.rb

@@ -80,9 +80,7 @@ module Jeeves
       end
 
       def generate(prompt)
-        # One byte per token is a conservative bound without a model-specific tokenizer.
-        budget = @settings.context - 1000 - 256
-        if prompt.bytesize > budget
+        if prompt.bytesize > @settings.prompt_budget
           raise Error, 'Prompt exceeds the conservative local context budget. Split the changes or increase GIT_COMMIT_LOCAL_CONTEXT.'
         end
         body = {

data/lib/jeeves/runtime.rb

@@ -4,12 +4,12 @@ require 'rubygems/requirement'
 
 module Jeeves
   module Runtime
-    REQUIREMENT = Gem::Requirement.new('>= 3.3', '< 4.1').freeze
+    REQUIREMENT = Gem::Requirement.new('>= 3.3').freeze
 
-    def self.check!
-      return if REQUIREMENT.satisfied_by?(Gem::Version.new(RUBY_VERSION))
+    def self.check!(version = RUBY_VERSION)
+      return if REQUIREMENT.satisfied_by?(Gem::Version.new(version))
 
-      abort "Jeeves requires Ruby #{REQUIREMENT}; found #{RUBY_VERSION}. Select a supported Ruby with your version manager."
+      abort "Jeeves requires Ruby #{REQUIREMENT}; found #{version}. Select the latest stable Ruby with your version manager."
     end
   end
 end

data/lib/jeeves/settings.rb

@@ -16,7 +16,7 @@ module Jeeves
                                   end
       raise Error, 'The configured model must not be empty' if @model.strip.empty?
 
-      @context = integer('GIT_COMMIT_LOCAL_CONTEXT', 32_768) if @provider == 'ollama'
+      @context = integer('GIT_COMMIT_LOCAL_CONTEXT', 65_536) if @provider == 'ollama'
       @max_diff_bytes = integer('GIT_COMMIT_MAX_DIFF_BYTES', 65_536)
       @message_format = env.fetch('GIT_COMMIT_MESSAGE_FORMAT', 'conventional')
       return if %w[conventional plain].include?(@message_format)
@@ -27,9 +27,11 @@ module Jeeves
     def check_diff!(diff)
       raise Error, 'Diff must be valid UTF-8 text.' unless diff.valid_encoding?
       raise Error, 'No diff content provided.' if diff.strip.empty?
-      return if diff.bytesize <= max_diff_bytes
+    end
 
-      raise Error, "Diff is too large (#{diff.bytesize} bytes; limit #{max_diff_bytes}). Split the changes or increase GIT_COMMIT_MAX_DIFF_BYTES."
+    def prompt_budget
+      # One byte per token, with room for output and chat framing.
+      context - 1000 - 256 if context
     end
 
     private

data/lib/jeeves/version.rb

@@ -1,3 +1,3 @@
 module Jeeves
-  VERSION = '3.0.0'
+  VERSION = '3.1.1'
 end

data/lib/jeeves.rb

@@ -10,6 +10,7 @@ module Jeeves
 end
 
 require 'jeeves/settings'
+require 'jeeves/diff'
 require 'jeeves/prompt'
 require 'jeeves/message'
 require 'jeeves/providers'
@@ -38,7 +39,7 @@ module Jeeves
       provider = settings.provider == 'ollama' ? Providers::Ollama.new(settings) : Providers::OpenRouter.new(settings)
       piped = !@input.tty? && !@input.closed?
       if piped
-        diff = (@input.read(settings.max_diff_bytes + 1) || +'').force_encoding(Encoding::UTF_8)
+        diff = (@input.read || +'').force_encoding(Encoding::UTF_8)
       else
         raise Error, 'Not inside a Git working tree.' unless @git.root
 
@@ -47,7 +48,8 @@ module Jeeves
         diff = options[:all] && options[:dry_run] ? @git.preview_all : @git.diff
       end
       settings.check_diff!(diff)
-      prompt = Prompt.new(home: @env.fetch('HOME') { Dir.home }, repository: @git.root, errors: @errors).render(diff)
+      prompt = Prompt.new(home: @env.fetch('HOME') { Dir.home }, repository: @git.root, errors: @errors)
+                     .render(diff, settings: settings)
       @errors.puts "Using #{settings.provider}: #{settings.model}" unless piped
       message = Message.prepare(provider.generate(prompt), format: settings.message_format)
       return show(message) if piped || options[:dry_run]

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: jeeves-git-commit
 version: !ruby/object:Gem::Version
-  version: 3.0.0
+  version: 3.1.1
 platform: ruby
 authors:
 - Justin Bishop
@@ -79,6 +79,7 @@ files:
 - bin/jeeves
 - config/prompt
 - lib/jeeves.rb
+- lib/jeeves/diff.rb
 - lib/jeeves/git_repository.rb
 - lib/jeeves/message.rb
 - lib/jeeves/prompt.rb
@@ -98,9 +99,6 @@ required_ruby_version: !ruby/object:Gem::Requirement
   - - ">="
     - !ruby/object:Gem::Version
       version: '3.3'
-  - - "<"
-    - !ruby/object:Gem::Version
-      version: '4.1'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="