lex-cost-scanner 0.1.1 → 0.1.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: fe30951f14930db7286c7a0bad5b5e6610564584120db3c1117acfc701c38e3b
-  data.tar.gz: d247d87f85043a2b5d0657d07b74dbafca40791310b13942020827b06b83f38b
+  metadata.gz: 505897e6d0311a9e6d695a1f391fa80e34e923de745f03af6f2de11354b56ce3
+  data.tar.gz: 3df5cf94e94894598b0d5fe0afaac3e441d6a3e18ee195c1d945bd91eb660806
 SHA512:
-  metadata.gz: fd493c13faf781159444e9153fd1ff63524cfdea9939a5959bc40d5992ecfad3cb55120028aabe40000ef2540d3d28adac8807880cf82e0233c1f36e019507b4
-  data.tar.gz: b706b5df7f9af7c150e228eb51ca974b3a3ba21a9a2997195288238d18caf4c4a612f7a69bcf6d8c2332c66ff9c8a7e2e579f26ff2631033dcaefd80ffffc173
+  metadata.gz: 70ad1bedc0e2236f159e38423657b03583706e31292567a5f36afdf0f3de0e7836980c70bf9721f5ef844335dcc0f8bec1e2c6bad3685556f6622f5dc2fd6f63
+  data.tar.gz: 43cddabb87240fabbf4240b920eea240cabe6eaca4198166a39d3a571bec12fc0f6bd3288baa860565b90be6ed347348f18fc141c791bb15f6deda16d5d45d0f

data/CHANGELOG.md

@@ -1,5 +1,10 @@
 # Changelog
 
+## [0.1.2] - 2026-03-30
+
+### Changed
+- update to rubocop-legion 0.1.7, resolve all offenses
+
 ## [0.1.1] - 2026-03-27
 
 ### Fixed

data/lib/legion/extensions/cost_scanner/actors/weekly_scan.rb

@@ -4,7 +4,7 @@ module Legion
   module Extensions
     module CostScanner
       module Actor
-        class WeeklyScan < Legion::Extensions::Actors::Every
+        class WeeklyScan < Legion::Extensions::Actors::Every # rubocop:disable Legion/Extension/EveryActorRequiresTime
           def time
             604_800
           end

data/lib/legion/extensions/cost_scanner/helpers/classifier.rb

@@ -53,15 +53,15 @@ module Legion
           def llm_classify(resource_id:, resource_type:, monthly_cost:, utilization:)
             prompt = format(CLASSIFY_PROMPT, resource_type: resource_type, resource_id: resource_id,
                                              monthly_cost: monthly_cost, utilization: utilization.inspect)
-            response = Legion::LLM.chat(
+            response = Legion::LLM.chat( # rubocop:disable Legion/HelperMigration/DirectLlm
               message: prompt,
-              caller: { extension: 'lex-cost-scanner', function: 'classify' }
+              caller:  { extension: 'lex-cost-scanner', function: 'classify' }
             )
-            parsed = Legion::JSON.load(response)
+            parsed = Legion::JSON.load(response) # rubocop:disable Legion/HelperMigration/DirectJson
             parsed[:finding_type] = parsed[:finding_type].to_sym
             parsed[:severity] = parsed[:severity].to_sym
             parsed.merge(resource_id: resource_id, resource_type: resource_type, method: :llm)
-          rescue StandardError
+          rescue StandardError => _e
             result = rule_based_classify(utilization: utilization, monthly_cost: monthly_cost)
             result.merge(resource_id: resource_id, resource_type: resource_type, method: :rule_based_fallback)
           end

data/lib/legion/extensions/cost_scanner/helpers/findings_store.rb

@@ -6,7 +6,7 @@ module Legion
       module Helpers
         module FindingsStore
           @mutex = Mutex.new
-          @findings = {}
+          @findings = {} # rubocop:disable ThreadSafety/MutableClassInstanceVariable
 
           module_function
 
@@ -55,9 +55,9 @@ module Legion
 
           def stats
             @mutex.synchronize do
-              { total: @findings.size,
+              { total:         @findings.size,
                 total_savings: @findings.values.sum { |f| f[:estimated_monthly_savings] || 0.0 },
-                by_type: @findings.values.group_by { |f| f[:finding_type] }.transform_values(&:size) }
+                by_type:       @findings.values.group_by { |f| f[:finding_type] }.transform_values(&:size) }
             end
           end
 

data/lib/legion/extensions/cost_scanner/runners/reporter.rb

@@ -5,15 +5,17 @@ module Legion
     module CostScanner
       module Runners
         module Reporter
+          extend self
+
           def generate_report(limit: 10)
             top = Helpers::FindingsStore.top_by_savings(limit: limit)
             stats = Helpers::FindingsStore.stats
 
-            { total_savings: stats[:total_savings],
+            { total_savings:  stats[:total_savings],
               findings_count: stats[:total],
-              by_type: stats[:by_type],
-              top_findings: top,
-              generated_at: Time.now }
+              by_type:        stats[:by_type],
+              top_findings:   top,
+              generated_at:   Time.now }
           end
 
           def format_slack_blocks(report:)
@@ -55,7 +57,7 @@ module Legion
           private
 
           def report_webhook
-            return nil unless defined?(Legion::Settings)
+            return nil unless defined?(Legion::Settings) # rubocop:disable Legion/Extension/RunnerReturnHash
 
             config = Legion::Settings[:cost_scanner] || {}
             config[:slack_webhook]

data/lib/legion/extensions/cost_scanner/runners/scanner.rb

@@ -5,6 +5,8 @@ module Legion
     module CostScanner
       module Runners
         module Scanner
+          extend self
+
           def scan_all
             accounts = scanner_config[:accounts] || []
             results = accounts.map { |acct| scan_account(account_id: acct[:id], cloud: acct[:cloud]) }
@@ -38,31 +40,31 @@ module Legion
           end
 
           def process_resource(account_id:, resource:)
-            return false if (resource[:monthly_cost] || 0) < Helpers::Constants::MIN_MONTHLY_COST
+            return false if (resource[:monthly_cost] || 0) < Helpers::Constants::MIN_MONTHLY_COST # rubocop:disable Legion/Extension/RunnerReturnHash
 
             classification = Helpers::Classifier.classify(
-              resource_id: resource[:resource_id],
+              resource_id:   resource[:resource_id],
               resource_type: resource[:resource_type],
-              monthly_cost: resource[:monthly_cost],
-              utilization: resource[:utilization] || {}
+              monthly_cost:  resource[:monthly_cost],
+              utilization:   resource[:utilization] || {}
             )
-            return false if classification[:finding_type] == :none
+            return false if classification[:finding_type] == :none # rubocop:disable Legion/Extension/RunnerReturnHash
 
             result = Helpers::FindingsStore.record(
-              account_id: account_id,
-              resource_id: resource[:resource_id],
-              resource_type: resource[:resource_type],
-              finding_type: classification[:finding_type],
-              severity: classification[:severity],
-              monthly_cost: resource[:monthly_cost],
+              account_id:                account_id,
+              resource_id:               resource[:resource_id],
+              resource_type:             resource[:resource_type],
+              finding_type:              classification[:finding_type],
+              severity:                  classification[:severity],
+              monthly_cost:              resource[:monthly_cost],
               estimated_monthly_savings: classification[:estimated_monthly_savings],
-              recommendation: classification[:recommendation]
+              recommendation:            classification[:recommendation]
             )
             result[:new]
           end
 
           def fetch_resources(_account_id:, _cloud: 'aws')
-            return [] unless defined?(Legion::Extensions::Http::Client)
+            return [] unless defined?(Legion::Extensions::Http::Client) # rubocop:disable Legion/Extension/RunnerReturnHash
 
             []
           end

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

@@ -3,7 +3,7 @@
 module Legion
   module Extensions
     module CostScanner
-      VERSION = '0.1.1'
+      VERSION = '0.1.2'
     end
   end
 end

data/lib/legion/extensions/cost_scanner.rb

@@ -7,7 +7,7 @@ require_relative 'cost_scanner/helpers/findings_store'
 require_relative 'cost_scanner/runners/scanner'
 require_relative 'cost_scanner/runners/reporter'
 
-require_relative 'cost_scanner/actors/weekly_scan' if defined?(Legion::Extensions::Actors::Every)
+require_relative 'cost_scanner/actors/weekly_scan'
 
 module Legion
   module Extensions

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: lex-cost-scanner
 version: !ruby/object:Gem::Version
-  version: 0.1.1
+  version: 0.1.2
 platform: ruby
 authors:
 - Matthew Iverson