lex-autofix 0.1.5 → 0.1.8

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: f46f6aa06b02c6485e2961a8e6c0b7af4f29a23ed37b9892cdc2b9c6cf866753
-  data.tar.gz: 146efbb61abce2ec7e173fd868ebe638ecfad4a00c25882bfc5431bbf91a35eb
+  metadata.gz: 5c27afcb0c60cb91e4200e4a3bb7a32b94396524aafadaa3cc9ef8d927e826ca
+  data.tar.gz: 2ada02009c3c3743a0b1582977440acfea287147e7150816756d8e9875556dad
 SHA512:
-  metadata.gz: baab241f9021c2436e4eaf90bf3e66001c5fa6dcf056e914a2519a00a314772dd8e8f6b190749bd80c7eae17e8092ebc9d17866f9b499c25fb82fe8faf4d87e7
-  data.tar.gz: 6fc0e1c7f2b3d864e6ca3f5e0f571bf70e2388353829683ab56c43c8c3cda23c648a4211f2f0af3b6ca933a4e09e00cf9cfa87405e2995b14e30eb9bc97cabcc
+  metadata.gz: 6796f6a043c8b8af11239b738adcffd1ce6a64da2d14a495c3f789146f4c7849f5b7f96b822992eae59d9752ea2c86d49eb392ad83372b1f77b7fe0504e4c68e
+  data.tar.gz: '08f196e5d2c79792731ba2e103b19f69adec94b9c8b315198a742162496cc7765da36f3b0adbe5e0bb2e8795d2bc1e873bc4b7e9dfd7414b27a83fd4d33d1f42'

data/CHANGELOG.md

@@ -2,6 +2,29 @@
 
 ## [Unreleased]
 
+## [0.1.8] - 2026-03-27
+
+### Fixed
+- Add `=> e` capture and `log.warn` logging to all four bare `rescue StandardError` clauses in `Pipeline` (`resolve_token`, `resolve_org`, `resolve_max_retries`, `resolve_checkout_dir`)
+- Replace `log_info`/`log_warn` wrapper methods with direct `log.info`/`log.warn` calls at call sites and remove the now-unused wrappers
+- Replace direct `Legion::Cache.get` calls with `cache_get` helper (private method defined on `Pipeline` using the full qualified key, avoiding namespace double-prefixing)
+
+## [0.1.7] - 2026-03-27
+
+### Changed
+- Update `Transport.additional_e_to_q` to emit three targeted bindings (`legion.logging.exception.warn.#`, `legion.logging.exception.error.#`, `legion.logging.exception.fatal.#`) instead of the broad `legion.#` catch-all
+- Update `BatchBuffer#build_key` to use `error_fingerprint` as the group key when present, falling back to `lex:exception_class`
+- Update `Pipeline#handle_log_event` to check fingerprint cache (`autofix:wip:` and `autofix:fixed:` keys via `Legion::Cache`) and skip buffering for in-progress or already-fixed errors
+- Update `Fix#extract_file_paths` to strip `gem_path` prefix from `caller_file` and backtrace paths using new `strip_gem_prefix` helper
+- All keys consumed from events are now flat (`caller_file`, `caller_line`, `gem_path`, `error_fingerprint`, `exception_class`) — no nested `:caller` or `:exception` sub-hashes
+
+## [0.1.6] - 2026-03-25
+
+### Added
+- Add repo governance files: CODEOWNERS, dependabot.yml, and reusable CI workflows
+
+## [0.1.5] - 2026-03-24
+
 ### Fixed
 - Fix actor discovery: rename `module Actors` to `module Actor` (singular) to match framework convention
 - Add explicit `runner_class` override to LogConsumer pointing to `Runners::Pipeline` where `handle_log_event` lives

data/README.md

@@ -1,6 +1,55 @@
 # lex-autofix
 
-Autonomous error fix agent for LegionIO. Subscribes to the `legion.logging` RabbitMQ exchange, batches and triages errors via LLM, manages GitHub issues, and opens PRs with automated code fixes.
+Autonomous error fix agent for LegionIO. Subscribes to structured exception events from the `legion.logging` exchange, batches and deduplicates errors by fingerprint, triages them via LLM, manages GitHub issues, and opens PRs with automated code fixes.
+
+## Architecture
+
+```
+legion.logging exchange
+  legion.logging.exception.warn.#  ─┐
+  legion.logging.exception.error.# ─┼─> autofix.ingest queue --> [LogConsumer] --> [BatchBuffer]
+  legion.logging.exception.fatal.# ─┘                                                    │
+                                                                                          ▼
+                                                                               [Pipeline Runner]
+                                                                          triage -> diagnose -> fix -> ship
+```
+
+### Event Payload
+
+Each consumed message is a structured exception payload with the following fields:
+
+| Field | Description |
+|---|---|
+| `exception_class` | Ruby exception class name (e.g. `NoMethodError`) |
+| `message` | Exception message string |
+| `backtrace` | Array of backtrace lines |
+| `caller_file` | Source file where the exception originated |
+| `caller_line` | Line number within `caller_file` |
+| `caller_function` | Method name at the call site |
+| `gem_name` | Name of the gem that raised the exception |
+| `gem_version` | Version of that gem |
+| `component_type` | Component type (runner, actor, helper, etc.) |
+| `error_fingerprint` | Stable hash for deduplication across occurrences |
+| `handled` | Whether the exception was caught and handled |
+| `lex` | Extension name (used as suggested repo target) |
+| `user` | Identity of the user or caller at the time of the error |
+| `task_id` | Task ID if the error occurred within a task |
+| `conversation_id` | Conversation ID if the error occurred during a chat session |
+
+### BatchBuffer
+
+Events are grouped by `error_fingerprint` (falling back to `lex:exception_class` when no fingerprint is present). A flush is triggered when any group reaches `batch.count_threshold` events or the `batch.window_seconds` time window elapses.
+
+### Pipeline
+
+`handle_log_event` checks the fingerprint cache before buffering:
+
+- `autofix:wip:<fingerprint>` — a fix is in progress; skip
+- `autofix:fixed:<fingerprint>` — already resolved; skip
+
+On flush, the pipeline runs: **triage** (LLM clusters actionable vs transient errors) → **diagnose** (search or create a GitHub issue) → **fix** (`Fix` runner extracts file paths from `caller_file` and `backtrace`, applies LLM-generated edits, validates with rspec + rubocop) → **ship** (commit, push branch, open PR).
+
+Only active when `Legion::LLM.started?` returns true.
 
 ## Installation
 
@@ -13,13 +62,13 @@ gem 'lex-autofix'
 ## Configuration
 
 | Setting | Default | Purpose |
-|---------|---------|---------|
-| `autofix.batch.window_seconds` | 300 | Time window before batch flush |
-| `autofix.batch.count_threshold` | 3 | Error count per group to trigger flush |
-| `autofix.llm.max_retries` | 3 | Max LLM fix attempts before issue-only |
-| `autofix.github.token` | nil | GitHub PAT (supports `vault://`) |
-| `autofix.github.org` | LegionIO | GitHub org for operations |
-| `autofix.checkout_dir` | ~/.legionio/autofix/ | Temp directory for checkouts |
+|---|---|---|
+| `autofix.batch.window_seconds` | `300` | Time window before batch flush |
+| `autofix.batch.count_threshold` | `3` | Error count per group to trigger flush |
+| `autofix.llm.max_retries` | `3` | Max LLM fix attempts before issue-only |
+| `autofix.github.token` | `nil` | GitHub PAT (supports `vault://`) |
+| `autofix.github.org` | `LegionIO` | GitHub org for issue and PR operations |
+| `autofix.checkout_dir` | `~/.legionio/autofix/` | Temp directory for repo checkouts |
 
 ## License
 

data/lib/legion/extensions/autofix/helpers/batch_buffer.rb

@@ -56,6 +56,8 @@ module Legion
           private
 
           def build_key(event)
+            return event[:error_fingerprint] if event[:error_fingerprint]
+
             lex             = event[:lex] || 'core'
             exception_class = event[:exception_class] || 'unknown'
             "#{lex}:#{exception_class}"

data/lib/legion/extensions/autofix/runners/fix.rb

@@ -53,6 +53,7 @@ module Legion
             tc.cleanup(checkout_path)
             { success: false, reason: "fix failed after max retries (#{max_retries})" }
           rescue StandardError => e
+            log.log_exception(e, context: 'autofix: attempt_fix failed')
             { success: false, reason: e.message }
           end
 
@@ -75,19 +76,28 @@ module Legion
           private
 
           def extract_file_paths(error_details)
+            gem_base = error_details[:gem_path].to_s
             paths = []
-            paths << error_details[:caller_file] if error_details[:caller_file]
+            paths << strip_gem_prefix(error_details[:caller_file].to_s, gem_base)
 
             Array(error_details[:backtrace]).each do |line|
               file = line.to_s.split(':').first
-              paths << file if file && !file.empty?
+              next if file.nil? || file.empty?
+
+              paths << strip_gem_prefix(file, gem_base)
             end
 
-            paths = paths.uniq
+            paths = paths.uniq.reject(&:empty?)
             spec_paths = paths.map { |p| p.sub('lib/', 'spec/').sub(/\.rb$/, '_spec.rb') }
             (paths + spec_paths).uniq
           end
 
+          def strip_gem_prefix(path, gem_base)
+            return path if gem_base.empty?
+
+            path.delete_prefix("#{gem_base}/")
+          end
+
           def build_messages(attempt:, error_details:, files:, test_output:)
             prompt = if attempt.zero?
                        Helpers::Prompts.fix(error_details: error_details, files: files)

data/lib/legion/extensions/autofix/runners/pipeline.rb

@@ -29,6 +29,12 @@ module Legion
           end
 
           def handle_log_event(**event)
+            fingerprint = event[:error_fingerprint]
+            if fingerprint
+              return { success: true, action: :skip_wip } unless cache_get("autofix:wip:#{fingerprint}").nil?
+              return { success: true, action: :skip_fixed } unless cache_get("autofix:fixed:#{fingerprint}").nil?
+            end
+
             Pipeline.buffer.add(event)
             run_pipeline if Pipeline.buffer.flush_ready?
             { success: true }
@@ -40,7 +46,7 @@ module Legion
             return triage_result unless triage_result[:success]
 
             triage_result[:non_actionable].each do |cluster|
-              log_info("autofix: skipping non-actionable cluster: #{cluster[:summary]}")
+              log.info("autofix: skipping non-actionable cluster: #{cluster[:summary]}")
             end
 
             triage_result[:actionable].each do |cluster|
@@ -79,7 +85,7 @@ module Legion
             )
 
             unless fix_result[:success]
-              log_warn("autofix: fix failed for cluster #{cluster[:summary]}: #{fix_result[:reason]}")
+              log.warn("autofix: fix failed for cluster #{cluster[:summary]}: #{fix_result[:reason]}")
               return fix_result
             end
 
@@ -99,25 +105,29 @@ module Legion
 
           def resolve_token
             Legion::Settings.dig(:autofix, :github, :token)
-          rescue StandardError
+          rescue StandardError => e
+            log.warn("autofix: could not resolve token: #{e.message}")
             nil
           end
 
           def resolve_org
             Legion::Settings.dig(:autofix, :github, :org) || 'LegionIO'
-          rescue StandardError
+          rescue StandardError => e
+            log.warn("autofix: could not resolve org: #{e.message}")
             'LegionIO'
           end
 
           def resolve_max_retries
             Legion::Settings.dig(:autofix, :llm, :max_retries) || 3
-          rescue StandardError
+          rescue StandardError => e
+            log.warn("autofix: could not resolve max_retries: #{e.message}")
             3
           end
 
           def resolve_checkout_dir
             Legion::Settings.dig(:autofix, :checkout_dir)
-          rescue StandardError
+          rescue StandardError => e
+            log.warn("autofix: could not resolve checkout_dir: #{e.message}")
             nil
           end
 
@@ -125,12 +135,18 @@ module Legion
             text.to_s.downcase.gsub(/[^a-z0-9]+/, '-').slice(0, 40).chomp('-')
           end
 
-          def log_info(msg)
-            log.info(msg)
+          def cache_get(key)
+            return unless defined?(Legion::Cache)
+
+            cache = Legion::Cache
+            cache.get(key)
           end
 
-          def log_warn(msg)
-            log.warn(msg)
+          def cache_set(key, value, ttl: 60)
+            return unless defined?(Legion::Cache)
+
+            cache = Legion::Cache
+            cache.set(key, value, ttl)
           end
         end
       end

data/lib/legion/extensions/autofix/transport.rb

@@ -7,7 +7,9 @@ module Legion
         extend Legion::Extensions::Transport if defined?(Legion::Extensions::Transport)
 
         def self.additional_e_to_q
-          [{ from: 'legion.logging', to: 'autofix.ingest', routing_key: 'legion.#' }]
+          %w[warn error fatal].map do |level|
+            { from: 'legion.logging', to: 'autofix.ingest', routing_key: "legion.logging.exception.#{level}.#" }
+          end
         end
       end
     end

data/lib/legion/extensions/autofix/version.rb

@@ -3,7 +3,7 @@
 module Legion
   module Extensions
     module Autofix
-      VERSION = '0.1.5'
+      VERSION = '0.1.8'
     end
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: lex-autofix
 version: !ruby/object:Gem::Version
-  version: 0.1.5
+  version: 0.1.8
 platform: ruby
 authors:
 - LegionIO