command_proposal 1.0.15 → 1.0.16

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 321b3627d2af7cf183da1cd1071775e7f13907b0fda6101acce253f9649f8a42
-  data.tar.gz: c9a30e58a53bb372807e05e765a7b92368433ced38325e8d7800e9f6b983eeb8
+  metadata.gz: 43e714b8d84f486d67fabcf4d171cc38f6fe3e97c1e4cd776b558ef9feef2bb0
+  data.tar.gz: 5107629d432ed32fb26bd3fbfe56094287b1a9a0eb2feaf72b1e9f493b3d1dfd
 SHA512:
-  metadata.gz: ddcd08648f107fb5e50c57ae46a7ff93ddb1f7e79ea900e7a052afd6fb31574fd85fae1bf4a2214313a6fae8b46d79e4406d2be6756714319d928c259b612557
-  data.tar.gz: 629c1bc1f60e363396a7672122dc0a945c042baed8c43339fbc5ae7d80cb71aa645de0d3c199709d923cc1c1376e3d305b273ef3f71275ae53215899e59e3ef1
+  metadata.gz: 283e95ed280a541f6e607e2802068664244d532186448cafe42850763bb318159335b33b64e5411c729a65cbd427de94b6e672154efe0231487a71ba8023e659
+  data.tar.gz: bbece28324234cfee33481dbdeef5fdadd0b90ff982bb3eeabeaa76e6732c558298ed8dd7a0e313f9304b574dc7559e13187c4bdc3e7cec0a3ed7c0c85102e9c

data/app/models/command_proposal/iteration.rb

@@ -65,6 +65,10 @@ class ::CommandProposal::Iteration < ApplicationRecord
     task.primary_iteration == self
   end
 
+  def approved?
+    super || (session_type == "function" && approved_at?)
+  end
+
   def complete?
     success? || failed? || cancelled? || terminated?
   end

data/app/models/command_proposal/service/proposal_presenter.rb

@@ -13,6 +13,7 @@ module CommandProposal
       delegate :description, to: :iteration
       delegate :args, to: :iteration
       delegate :code, to: :iteration
+      delegate :result, to: :iteration
       delegate :status, to: :iteration
       delegate :approved_at, to: :iteration
       delegate :started_at, to: :iteration

data/app/views/command_proposal/tasks/_lines.html.erb

@@ -1,5 +1,5 @@
 <% if lines.blank? && !(skip_empty ||= false) -%><div class="line"></div><% end
--%><% lines&.split("\n").each do |line|
+-%><% lines&.split("\n")&.each do |line|
   truncate = ::CommandProposal::Iteration::TRUNCATE_COUNT
   -%><div class="line"><%= line.truncate(truncate) -%></div><%=
   if line.length > truncate

data/lib/command_proposal/services/runner.rb

@@ -65,7 +65,7 @@ module CommandProposal
       end
 
       def quick_run(friendly_id)
-        task = ::CommandProposal::Task.module.find_by!(friendly_id: friendly_id)
+        task = ::CommandProposal::Task.find_by!(friendly_id: friendly_id)
         iteration = task&.primary_iteration
 
         raise CommandProposal::Error, ":#{friendly_id} does not have approval to run." unless iteration&.approved?
@@ -98,12 +98,12 @@ module CommandProposal
           return @iteration.result = results_from_exception(e)
         end
 
-        stored_stdout = $stdout
-        $stdout = StringIO.new
+        stored_stdout = $stdout # quiet
+        $stdout = StringIO.new # quiet
         result = nil # Init var for scope
         status = nil
 
-        running_thread = Thread.new do
+        runner_proc = proc {
           begin
             # Run `bring` functions in here so we can capture any string outputs
             # OR! Run the full runner and instead of saving to an iteration, return the string for prepending here
@@ -115,28 +115,36 @@ module CommandProposal
 
             result = results_from_exception(e)
           end
-        end
-
-        while running_thread.status.present?
-          @iteration.reload
+        }
 
-          if $stdout.try(:string) != @iteration.result
-            @iteration.update(result: $stdout.try(:string).dup)
+        if Rails.application.config.active_job&.queue_adapter == :inline
+          runner_proc.call
+        else
+          running_thread = Thread.new do
+            runner_proc.call
           end
 
-          if @iteration.cancelling?
-            running_thread.exit
-            status = :cancelled
-          end
+          while running_thread.status.present?
+            @iteration.reload
 
-          sleep 0.4
+            if $stdout.try(:string) != @iteration.result
+              @iteration.update(result: $stdout.try(:string).dup)
+            end
+
+            if @iteration.cancelling?
+              running_thread.exit
+              status = :cancelled
+            end
+
+            sleep 0.4
+          end
         end
 
         output = $stdout.try(:string)
         output = nil if output == ""
         # Not using presence because we want to maintain other empty objects such as [] and {}
 
-        $stdout = stored_stdout
+        $stdout = stored_stdout # quiet
         @iteration.status = status
         @iteration.result = [output, result].compact.join("\n")
       end

data/lib/command_proposal/services/shut_down.rb

@@ -13,7 +13,7 @@ module CommandProposal
       def terminate(iteration)
         return unless iteration.running?
 
-        terminated_result = iteration.result + "\n\n~~~~~ TERMINATED ~~~~~"
+        terminated_result = "#{iteration&.result}\n\n~~~~~ TERMINATED ~~~~~"
         iteration.update(
           status: :terminated,
           result: terminated_result,

data/lib/command_proposal/version.rb

@@ -1,3 +1,3 @@
 module CommandProposal
-  VERSION = "1.0.15"
+  VERSION = "1.0.16"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: command_proposal
 version: !ruby/object:Gem::Version
-  version: 1.0.15
+  version: 1.0.16
 platform: ruby
 authors:
 - Rocco Nicholls
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2022-03-05 00:00:00.000000000 Z
+date: 2022-03-11 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -162,7 +162,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.2.22
+rubygems_version: 3.2.3
 signing_key:
 specification_version: 4
 summary: Gives the ability to run approved commands through a UI in your browser