robot_lab-audit 0.2.7 → 0.2.8

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: bd6e47211bfc31f854f89128e97ee43b5df24dd1e72a1845145c5592f3d6678c
-  data.tar.gz: 5f23a6f3dd37b8453e98e2d1e637e4697c1d6d56eacc338f0ac1642b312ebf9c
+  metadata.gz: ef178bcab0fa1ddd03a137fb4c2bb97032d66e3e78120a171970c87bcc5ca6af
+  data.tar.gz: 0eeb18834c2ef6a1355e528308ef31805d86c48e1557359b75d1372d8ef5e52f
 SHA512:
-  metadata.gz: af6ab9e243f5537c1a06d61ad04956240aa1a7cc4e07b3d4c3262121d63f8b6191a6e6a6f9d3e535c17bcfbea7c21bbc9ee572646f5657301a84fae96a2d7bd8
-  data.tar.gz: d565c5353d618bc546e6e37e4ebfc3473011eb15f34334d9803ce1f92cba00b5274a20f045393920fea827a3b1e8db5624a9b321d589067d48393d336f414200
+  metadata.gz: 77e6579549b5b4d05cc34162cdae304cd1fa1c38ce6d5257cb3c4344f62bef4fa6adaaa6bd2afddbecaf6dd7b953048c44229e8b3841ff6bb601c7c3fbb11097
+  data.tar.gz: 0d1b28b9d41dc6d312f2777752664b91f5f197d765c26bf17d3ca84a42f09cb25e049e4ddad141ced0f6ae2d18e9c687d5e4160a0945ad08f9f83ead419520f2

data/.envrc

@@ -1,3 +1,6 @@
+# robot_lab_project/robot_lab-audit/.envrc
+
 source_up
+
 export RR=`pwd`
 export BUNDLE_GEMFILE=Gemfile

data/CHANGELOG.md

@@ -7,6 +7,10 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+## [0.2.8] - 2026-09-09
+
+Released in lockstep with `robot_lab` core v0.2.8: this gem now resolves the released core gem from RubyGems instead of the local sibling checkout (local-path development remains available via `BUNDLE_GEMFILE=Gemfile.local`). Also in this release: gem lifecycle tasks (build/install/release) moved from the Rakefile to the asgard task runner.
+
 ### Added
 - `.loki` Asgard task file: `test`, `rubocop`, `rubocop_fix`, `flog`, `flay`, `quality`, `build`, `install`, `release`, and `console` tasks via the Asgard task runner
 - `flay_check` Rake task: structural code duplication gate (mass threshold 50); integrated into the `quality` Rake task

data/Rakefile

@@ -1,6 +1,5 @@
 # frozen_string_literal: true
 
-require 'bundler/gem_tasks'
 require 'rake/testtask'
 
 Rake::TestTask.new(:test) do |t|
@@ -24,101 +23,6 @@ task :test_file, [:file] do |_t, args|
   ruby "test/#{args[:file]}"
 end
 
-desc 'Check code complexity with Flog (warn >=20, fail >=50)'
-task :flog_check do
-  require 'flog'
-
-  method_warn = 20.0
-  method_fail = 50.0
-
-  flogger = Flog.new(all: true)
-  flogger.flog(*Dir.glob('lib/**/*.rb'))
-
-  warnings = []
-  failures = []
-
-  flogger.each_by_score do |method, score|
-    next if method.end_with?('#none')
-
-    if score > method_fail
-      failures << "#{format('%.1f', score)}: #{method}"
-    elsif score > method_warn
-      warnings << "#{format('%.1f', score)}: #{method}"
-    end
-  end
-
-  unless warnings.empty?
-    puts "\nFlog warnings (#{method_warn}–#{method_fail}) — target for future refactoring:"
-    warnings.each { |v| puts "  #{v}" }
-  end
-
-  if failures.empty?
-    puts "\nFlog: no methods exceed the failure threshold (>=#{method_fail})"
-  else
-    puts "\nFlog failures (>=#{method_fail}) — must be refactored:"
-    failures.each { |v| puts "  #{v}" }
-    abort "\nFlog quality gate failed: #{failures.size} method(s) exceed #{method_fail}"
-  end
-end
-
-desc 'Check for structural code duplication with Flay (mass >= 50)'
-task :flay_check do
-  require 'flay'
-
-  mass_threshold = 50
-
-  flay = Flay.new({ mass: mass_threshold, diff: false, verbose: false, summary: false, timeout: 60 })
-  flay.process(*Dir.glob('lib/**/*.rb'))
-  flay.analyze
-
-  if flay.hashes.empty?
-    puts "\nFlay: no structural duplication detected (mass >= #{mass_threshold})"
-  else
-    puts "\nFlay found structural duplication (mass >= #{mass_threshold}):"
-    flay.report
-    abort "\nFlay quality gate failed: #{flay.hashes.length} pattern(s) detected"
-  end
-end
-
-desc 'Run all quality checks: tests (with coverage), RuboCop, Flog, and Flay'
-task :quality do
-  gates = [
-    ['Tests + Coverage', 'bundle exec rake test'],
-    ['RuboCop',          'bundle exec rubocop'],
-    ['Flog Complexity',  'bundle exec rake flog_check'],
-    ['Flay Duplication', 'bundle exec rake flay_check']
-  ]
-
-  results = gates.map do |label, command|
-    puts "\n#{'=' * 60}"
-    puts "Quality Gate: #{label}"
-    puts '=' * 60
-    [label, system(command) ? :pass : :fail]
-  end
-
-  green = ->(s) { "\e[32m#{s}\e[0m" }
-  red   = ->(s) { "\e[31m#{s}\e[0m" }
-  width = results.map { |label, _| label.length }.max
-
-  puts "\n#{'=' * 60}"
-  puts 'Quality Gate Summary'
-  puts '=' * 60
-  results.each do |label, status|
-    badge = status == :pass ? green.call('PASS') : red.call('FAIL')
-    puts "  [#{badge}] #{label.ljust(width)}"
-  end
-  puts '-' * 60
-
-  passed = results.count { |_, s| s == :pass }
-  failed = results.count { |_, s| s == :fail }
-  tally  = "#{passed} passed, #{failed} failed"
-  puts "  #{failed.zero? ? green.call(tally) : red.call(tally)}"
-  puts '=' * 60
-
-  abort "\n#{red.call('Quality gate failed.')}" unless failed.zero?
-  puts "\n#{green.call('All quality gates passed.')}"
-end
-
 namespace :docs do
   desc 'Build MkDocs documentation'
   task :build do

data/lib/robot_lab/audit/event_log.rb

@@ -44,6 +44,9 @@ module RobotLab
         @db.execute_batch(SCHEMA)
       end
 
+      # :reek:LongParameterList -- one required keyword per SCHEMA column, on
+      # purpose: a caller that forgets a column raises ArgumentError instead
+      # of silently inserting NULL.
       def record_network_run(run_id:, network_name:, input:, result:, error_class:,
                              error_message:, started_at:, finished_at:, duration_ms:)
         @db.execute(
@@ -56,6 +59,9 @@ module RobotLab
         )
       end
 
+      # :reek:LongParameterList -- one required keyword per SCHEMA column, on
+      # purpose: a caller that forgets a column raises ArgumentError instead
+      # of silently inserting NULL.
       def record_event(run_id:, event_type:, robot_name:, tool_name:, input:, output:,
                        error_class:, error_message:, started_at:, finished_at:, duration_ms:)
         @db.execute(
@@ -70,12 +76,12 @@ module RobotLab
 
       def network_runs(limit: 100)
         rows = @db.execute('SELECT * FROM network_runs ORDER BY started_at DESC LIMIT ?', [limit])
-        rows.map { |r| row_to_hash(:network_runs, r) }
+        rows.map { |r| row_to_hash(NETWORK_RUN_COLUMNS, r) }
       end
 
       def events_for(run_id)
         rows = @db.execute('SELECT * FROM audit_events WHERE run_id = ? ORDER BY started_at', [run_id])
-        rows.map { |r| row_to_hash(:audit_events, r) }
+        rows.map { |r| row_to_hash(AUDIT_EVENT_COLUMNS, r) }
       end
 
       def recent_errors(limit: 50)
@@ -83,7 +89,7 @@ module RobotLab
           'SELECT * FROM audit_events WHERE error_class IS NOT NULL ORDER BY started_at DESC LIMIT ?',
           [limit]
         )
-        rows.map { |r| row_to_hash(:audit_events, r) }
+        rows.map { |r| row_to_hash(AUDIT_EVENT_COLUMNS, r) }
       end
 
       NETWORK_RUN_COLUMNS = %i[id run_id network_name input result error_class
@@ -93,9 +99,8 @@ module RobotLab
 
       private
 
-      def row_to_hash(table, row)
-        cols = table == :network_runs ? NETWORK_RUN_COLUMNS : AUDIT_EVENT_COLUMNS
-        cols.zip(row).to_h
+      def row_to_hash(columns, row)
+        columns.zip(row).to_h
       end
     end
   end

data/lib/robot_lab/audit/hook.rb

@@ -5,6 +5,12 @@ require 'json'
 
 module RobotLab
   module Audit
+    # :reek:RepeatedConditional -- each callback independently no-ops when
+    # event_log is nil, so the hook can be registered globally (RobotLab.on)
+    # before Audit.enable ever runs, or scoped to one robot/network without
+    # ever calling enable at all. A Null Object would remove the repeated
+    # check but couples every caller to a different `Hook.event_log` contract
+    # (an object always present vs. nil-means-disabled); not worth it here.
     class Hook < RobotLab::Hook
       self.namespace = :audit
 
@@ -62,6 +68,9 @@ module RobotLab
 
         private
 
+        # :reek:LongParameterList -- private helper with exactly two call
+        # sites, each passing every argument explicitly; a wrapper object
+        # would add indirection without making either call site clearer.
         def record_audit_event(event_type, ctx, started, tool_name, input, output, error)
           event_log.record_event(
             run_id:     current_run_id,

data/lib/robot_lab/audit/version.rb

@@ -2,6 +2,6 @@
 
 module RobotLab
   module Audit
-    VERSION = '0.2.7'
+    VERSION = '0.2.8'
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: robot_lab-audit
 version: !ruby/object:Gem::Version
-  version: 0.2.7
+  version: 0.2.8
 platform: ruby
 authors:
 - Dewayne VanHoozer
@@ -90,7 +90,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 4.0.19
+rubygems_version: 4.0.20
 specification_version: 4
 summary: SQLite-backed execution audit log for RobotLab via the Hook system.
 test_files: []