lex-swarm-github 0.3.2 → 0.3.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: afb02c0714b2eeeb371b7d0890c04b6a5637be7b8584f62872cec6337baa980e
-  data.tar.gz: 794646c0b9897d25e18fb22cf156c3f7ee2a5a07408aa23876b17cef00ece198
+  metadata.gz: 872fde87c10b7bf7e68f2aab5b56d05d9e66e552511061463e75fcabe714ca62
+  data.tar.gz: 72ccbe57f57adabf3ecce969ce4fb721485e9478aabe934e56ed159d6ca31959
 SHA512:
-  metadata.gz: 5c20383d9e86e5db3135a6cca8af1b1b226348f9cada58232425e7ae8064c17c2366f6aab167404b2081a567e95e238cfd37fc23cebf338fedb9078c35d5591c
-  data.tar.gz: 9d494a1404085b965f8bf6d272b9b875fc4bd3c21c291d9d07fdf45452fe356d6e3966f4e5fe6f42c142f5557b18ea80d68092fe33dbc0cd1d5c9628eea9e68e
+  metadata.gz: 4331730e59dafc94477cc02309afd0b04f2c56d5556570fe69169c490b62921e31ac6cc4e49606d5f93082dff93d652cfc68992f7a71ac150b1f764098acdffb
+  data.tar.gz: 39de677ac295aa04c728f47e16433b7a99c6e6baff36353d48702643b34f15d7dd204f70c1a25003342cc48d6352ac3c58d6ac0dab7439948eb4dce6d2dae4d2

data/Gemfile

@@ -6,6 +6,7 @@ gemspec
 
 gem 'rspec', '~> 3.13'
 gem 'rubocop', '~> 1.75', require: false
+gem 'rubocop-legion', '~> 0.1'
 gem 'rubocop-rspec', require: false
 
 if File.exist?(File.expand_path('../../legion-gaia', __dir__))

data/lib/legion/extensions/swarm_github/actors/lifecycle_subscriber.rb

@@ -9,7 +9,7 @@ module Legion
         class LifecycleSubscriber < Legion::Extensions::Actors::Subscription
           def runner_class    = self.class
           def runner_function = 'action'
-          def check_subtask?  = false
+          def check_subtask?  = true
           def generate_task?  = false
 
           def action(payload)
@@ -54,11 +54,7 @@ module Legion
           end
 
           def log
-            return Legion::Logging if defined?(Legion::Logging)
-
-            @log ||= Object.new.tap do |nl|
-              %i[debug info warn error fatal].each { |m| nl.define_singleton_method(m) { |*| nil } }
-            end
+            Legion::Logging
           end
         end
       end

data/lib/legion/extensions/swarm_github/actors/stale_issues.rb

@@ -6,7 +6,7 @@ module Legion
   module Extensions
     module SwarmGithub
       module Actor
-        class StaleIssues < Legion::Extensions::Actors::Every
+        class StaleIssues < Legion::Extensions::Actors::Every # rubocop:disable Legion/Extension/EveryActorRequiresTime
           def runner_class
             Legion::Extensions::SwarmGithub::Runners::GithubSwarm
           end

data/lib/legion/extensions/swarm_github/runners/extension_lifecycle.rb

@@ -7,7 +7,9 @@ module Legion
         module ExtensionLifecycle
           extend self
 
-          def run_lifecycle(generation:, review:, review_k: nil, review_models: nil)
+          def run_lifecycle(generation: nil, review: nil, review_k: nil, review_models: nil, **payload)
+            generation ||= build_generation(payload)
+            review ||= build_review(payload)
             config = github_config
             return { success: false, error: :github_not_enabled } unless config[:enabled]
             return { success: false, error: :target_repo_missing } unless config[:target_repo]
@@ -64,7 +66,7 @@ module Legion
             github_client.commit_files(owner: owner, repo: repo, branch: branch, files: files, message: message)
           end
 
-          def open_pull_request(owner:, repo:, branch:, base:, generation:, review:) # rubocop:disable Metrics/ParameterLists
+          def open_pull_request(owner:, repo:, branch:, base:, generation:, review:)
             return { success: false, error: :github_runner_unavailable } unless github_runner_available?
 
             body = build_pr_body(generation: generation, review: review)
@@ -84,7 +86,7 @@ module Legion
             return { success: true } unless pull_number && labels&.any?
             return { success: false, error: :github_runner_unavailable } unless github_runner_available?
 
-            github_client.add_labels(owner: owner, repo: repo, issue_number: pull_number, labels: labels)
+            github_client.add_labels_to_issue(owner: owner, repo: repo, issue_number: pull_number, labels: labels)
           rescue StandardError => e
             log.warn("Label failed: #{e.message}")
             { success: false, error: e.message }
@@ -172,8 +174,9 @@ module Legion
               Legion::Extensions::SwarmGithub::Runners::PullRequestReviewer.review_pull_request(**kwargs)
             end
 
+            blocker_severities = %w[error critical].freeze
             approvals = reviews.count do |r|
-              r[:status] == 'reviewed' && (r[:comments] || []).none? { |c| %w[error critical].include?(c[:severity]&.to_s) }
+              r[:status] == 'reviewed' && (r[:comments] || []).none? { |c| blocker_severities.include?(c[:severity]&.to_s) }
             end
             rejections = k - approvals
 
@@ -190,7 +193,7 @@ module Legion
             { success: true, skipped: true, reason: :review_error }
           end
 
-          def handle_auto_merge(owner:, repo:, pull_number:, config:, review:, review_result: nil) # rubocop:disable Metrics/ParameterLists
+          def handle_auto_merge(owner:, repo:, pull_number:, config:, review:, review_result: nil)
             return unless config[:auto_merge] && review[:verdict]&.to_sym == :approve
             return if review_result && review_result[:consensus] == :request_changes
 
@@ -275,11 +278,31 @@ module Legion
           end
 
           def log
-            return Legion::Logging if defined?(Legion::Logging)
+            Legion::Logging
+          end
 
-            @log ||= Object.new.tap do |nl|
-              %i[debug info warn error fatal].each { |m| nl.define_singleton_method(m) { |*| nil } }
-            end
+          def build_generation(payload)
+            {
+              name:          payload[:name],
+              generation_id: payload[:generation_id],
+              gap_id:        payload[:gap_id],
+              gap_type:      payload[:gap_type],
+              tier:          payload[:tier],
+              code:          payload[:code],
+              spec_code:     payload[:spec_code],
+              file_path:     payload[:file_path],
+              spec_path:     payload[:spec_path]
+            }.compact
+          end
+
+          def build_review(payload)
+            {
+              verdict:    payload[:verdict],
+              confidence: payload[:confidence],
+              stages:     payload[:stages],
+              issues:     payload[:issues],
+              passed:     payload[:passed]
+            }.compact
           end
         end
       end

data/lib/legion/extensions/swarm_github/runners/github_swarm.rb

@@ -5,8 +5,8 @@ module Legion
     module SwarmGithub
       module Runners
         module GithubSwarm
-          include Legion::Extensions::Helpers::Lex if Legion::Extensions.const_defined?(:Helpers) &&
-                                                      Legion::Extensions::Helpers.const_defined?(:Lex)
+          include Legion::Extensions::Helpers::Lex if Legion::Extensions.const_defined?(:Helpers, false) &&
+                                                      Legion::Extensions::Helpers.const_defined?(:Lex, false)
 
           def ingest_issue(repo:, issue_number:, title:, labels: [], **)
             key = issue_tracker.track(repo: repo, issue_number: issue_number, title: title, labels: labels)

data/lib/legion/extensions/swarm_github/runners/pr_pipeline.rb

@@ -43,7 +43,8 @@ module Legion
             return unless issue_number
 
             review_comments = review_result.dig(:review, :comments) || []
-            approved = review_comments.none? { |c| %w[critical high].include?(c[:severity]&.to_s&.downcase) }
+            blocker_severities = %w[critical high].freeze
+            approved = review_comments.none? { |c| blocker_severities.include?(c[:severity]&.to_s&.downcase) }
             @issue_tracker&.record_validation(
               "#{repo}##{issue_number}",
               validator: :code_review,

data/lib/legion/extensions/swarm_github/runners/pull_request_reviewer.rb

@@ -31,7 +31,7 @@ module Legion
               owner: owner, repo: repo, pull_number: pull_number
             )[:result] || []
           rescue StandardError => e
-            log.warn(e.message) if respond_to?(:log, true)
+            log.warn(e.message) if respond_to?(:log, true) # rubocop:disable Legion/HelperMigration/LoggingGuard
             []
           end
 
@@ -43,18 +43,18 @@ module Legion
               llm_kwargs = { message: prompt, caller: { extension: 'lex-swarm-github' } }
               llm_kwargs[:model] = model if model
               llm_kwargs[:provider] = provider if provider
-              response = Legion::LLM.chat(**llm_kwargs)
+              response = Legion::LLM.chat(**llm_kwargs) # rubocop:disable Legion/HelperMigration/DirectLlm
               parse_review_response(response)
             end
             merge_chunk_reviews(reviews)
           rescue StandardError => e
-            log.warn("Review generation failed: #{e.message}") if respond_to?(:log, true)
+            log.warn("Review generation failed: #{e.message}") if respond_to?(:log, true) # rubocop:disable Legion/HelperMigration/LoggingGuard
             { summary: 'Review generation failed', comments: [] }
           end
 
           def merge_chunk_reviews(reviews)
             merged_comments = reviews.flat_map { |r| r[:comments] || [] }
-            summaries = reviews.map { |r| r[:summary] }.compact
+            summaries = reviews.filter_map { |r| r[:summary] }
             {
               summary:  summaries.join("\n\n"),
               comments: merged_comments
@@ -62,9 +62,9 @@ module Legion
           end
 
           def parse_review_response(response)
-            Legion::JSON.parse(response)
+            json_parse(response)
           rescue StandardError => e
-            log.warn(e.message) if respond_to?(:log, true)
+            log.warn(e.message) if respond_to?(:log, true) # rubocop:disable Legion/HelperMigration/LoggingGuard
             { summary: response.to_s, comments: [] }
           end
 

data/lib/legion/extensions/swarm_github/runners/review_notifier.rb

@@ -17,7 +17,7 @@ module Legion
 
             { notified: result[:ok] == true, channel: channel, ts: result[:ts] }
           rescue StandardError => e
-            log.warn(e.message) if respond_to?(:log, true)
+            log.warn(e.message) if respond_to?(:log, true) # rubocop:disable Legion/HelperMigration/LoggingGuard
             { notified: false, reason: "slack error: #{e.message}" }
           end
 

data/lib/legion/extensions/swarm_github/runners/review_poster.rb

@@ -22,7 +22,7 @@ module Legion
             review_id = result.dig(:result, 'id') || result.dig(:result, :id)
             { posted: true, review_id: review_id, comments_count: inline_comments.size }
           rescue StandardError => e
-            log.warn(e.message) if respond_to?(:log, true)
+            log.warn(e.message) if respond_to?(:log, true) # rubocop:disable Legion/HelperMigration/LoggingGuard
             { posted: false, reason: "post failed: #{e.message}" }
           end
 

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

@@ -3,7 +3,7 @@
 module Legion
   module Extensions
     module SwarmGithub
-      VERSION = '0.3.2'
+      VERSION = '0.3.4'
     end
   end
 end

data/lib/legion/extensions/swarm_github.rb

@@ -11,12 +11,12 @@ require 'legion/extensions/swarm_github/runners/review_poster'
 require 'legion/extensions/swarm_github/runners/review_notifier'
 require 'legion/extensions/swarm_github/runners/pr_pipeline'
 require 'legion/extensions/swarm_github/runners/extension_lifecycle'
-require 'legion/extensions/swarm_github/actors/lifecycle_subscriber' if defined?(Legion::Extensions::Actors::Subscription)
+require 'legion/extensions/swarm_github/actors/lifecycle_subscriber' if defined?(Legion::Extensions::Actors::Subscription) # rubocop:disable Legion/HelperMigration/RequireDefinedGuard
 
 module Legion
   module Extensions
     module SwarmGithub
-      extend Legion::Extensions::Core if Legion::Extensions.const_defined? :Core
+      extend Legion::Extensions::Core if Legion::Extensions.const_defined? :Core, false
     end
   end
 end

data/spec/legion/extensions/swarm_github/actors/lifecycle_subscriber_spec.rb

@@ -22,7 +22,7 @@ RSpec.describe Legion::Extensions::SwarmGithub::Actor::LifecycleSubscriber do
   end
 
   describe '#check_subtask?' do
-    it { expect(actor.check_subtask?).to be false }
+    it { expect(actor.check_subtask?).to be true }
   end
 
   describe '#generate_task?' do

data/spec/legion/extensions/swarm_github/runners/extension_lifecycle_spec.rb

@@ -78,7 +78,7 @@ RSpec.describe Legion::Extensions::SwarmGithub::Runners::ExtensionLifecycle do
         allow(github_client).to receive(:create_pull_request).and_return(
           { result: { 'number' => 42, 'html_url' => 'https://github.com/org/repo/pull/42' } }
         )
-        allow(github_client).to receive(:add_labels).and_return({ success: true })
+        allow(github_client).to receive(:add_labels_to_issue).and_return({ success: true })
         allow(runner).to receive(:pr_reviewer_available?).and_return(false)
       end
 
@@ -109,14 +109,46 @@ RSpec.describe Legion::Extensions::SwarmGithub::Runners::ExtensionLifecycle do
         )
       end
 
-      it 'calls add_labels' do
+      it 'calls add_labels_to_issue' do
         runner.run_lifecycle(generation: generation, review: review)
-        expect(github_client).to have_received(:add_labels).with(
+        expect(github_client).to have_received(:add_labels_to_issue).with(
           hash_including(labels: %w[auto-generated needs-review])
         )
       end
     end
 
+    context 'when called with a flat payload (no generation/review keys)' do
+      let(:github_client) { double('Github::Client') }
+
+      before do
+        runner.instance_variable_set(:@github_client, nil)
+        allow(runner).to receive(:github_config).and_return(
+          { enabled: true, target_repo: 'org/repo', target_branch: 'main',
+            auto_merge: false, pr_labels: [], branch_prefix: 'feature/auto-generated' }
+        )
+        stub_const('Legion::Extensions::Github::Client', Class.new)
+        allow(Legion::Extensions::Github::Client).to receive(:new).and_return(github_client)
+        allow(github_client).to receive(:create_branch).and_return({ success: true })
+        allow(github_client).to receive(:commit_files).and_return({ success: true })
+        allow(github_client).to receive(:create_pull_request).and_return(
+          { result: { 'number' => 99, 'html_url' => 'https://github.com/org/repo/pull/99' } }
+        )
+        allow(runner).to receive(:pr_reviewer_available?).and_return(false)
+      end
+
+      it 'builds generation and review from flat keys and returns success' do
+        result = runner.run_lifecycle(
+          name: 'lex-bar', generation_id: 'gen-flat', gap_id: 'gap-002',
+          gap_type: 'missing_runner', tier: 'utility',
+          code: '# code', spec_code: '# spec', file_path: 'lib/foo.rb', spec_path: 'spec/foo_spec.rb',
+          verdict: 'approve', stages: {}
+        )
+        expect(result[:success]).to be true
+        expect(result[:pull_number]).to eq(99)
+        expect(result[:generation_id]).to eq('gen-flat')
+      end
+    end
+
     context 'when branch creation fails' do
       before do
         runner.instance_variable_set(:@github_client, nil)

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: lex-swarm-github
 version: !ruby/object:Gem::Version
-  version: 0.3.2
+  version: 0.3.4
 platform: ruby
 authors:
 - Esity