selective-ruby-core 0.1.3-arm64-darwin → 0.1.4-arm64-darwin

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 118cff5d299b401144a3813ee39d638a594ebfa985c2be9cd67d9a659224f9a1
-  data.tar.gz: 0541f558c3b705031bfd824ca4e5549518ddfc51f0fc9572d36350fb1ec1c908
+  metadata.gz: 8367e139f147f3a3d385b6559a7abf7cc233b55dce6a9efdd7fb3ad560e2bea8
+  data.tar.gz: f08c207e0e7269048b4aa4a277fb0d930412b1f01115de6059c0a42adfcb6ff3
 SHA512:
-  metadata.gz: f36eb6cdd508b8dd4152224d472025a0933fdf57bf1ab250fbcdc06539138300474346fff50eaa7ae771223912e2341c904fb9957e17fa53fe4d8413f61b80d6
-  data.tar.gz: d1430b790d2356671632eb764856f7d3a56bd05b32f891f78c2047b5d01bca9eb0cc55508ee9798344f3f0b6e2b6b393937eda3ec93c1f5b1a8d960349257feb
+  metadata.gz: 86b82b6d7aa53161760d4b3472543f23dc547f555914c212f29f7230f8d694c0bcbbe433afa8f481191c740e02d31612e0a98348ba829e2a2f4257e90df33c17
+  data.tar.gz: 85b1d0dee79c85655963fd744134c3bf655b3d283e93c098e3637b94ce6f1a2af27d6bb8acc88805927ab9cb58af7ce6603983d2e822a08e9113b7d17870704f

data/lib/bin/build_env.sh

@@ -4,13 +4,13 @@
 if [ -n "$GITHUB_ACTIONS" ]; then
   # Get environment variables
   platform=github_actions
-  branch=${GITHUB_HEAD_REF:-$GITHUB_REF_NAME}
+  branch=${SELECTIVE_BRANCH:-${GITHUB_HEAD_REF:-$GITHUB_REF_NAME}}
   pr_title=$SELECTIVE_PR_TITLE
-  target_branch=${GITHUB_BASE_REF}
+  target_branch=${SELECTIVE_TARGET_BRANCH:-$GITHUB_BASE_REF}
   actor=$GITHUB_ACTOR
-  sha=$GITHUB_SHA
-  run_id=$GITHUB_RUN_ID
-  run_attempt=$GITHUB_RUN_ATTEMPT
+  sha=${SELECTIVE_SHA:-$GITHUB_SHA}
+  run_id=${SELECTIVE_RUN_ID:-$GITHUB_RUN_ID}
+  run_attempt=${SELECTIVE_RUN_ATTEMPT:-$GITHUB_RUN_ATTEMPT}
   commit_message=$(git log --format=%s -n 1 $sha)
 else
   platform=$SELECTIVE_PLATFORM

data/lib/selective/ruby/core/controller.rb

@@ -14,7 +14,7 @@ module Selective
           @debug = debug
           @runner = runner
           @retries = 0
-          @runner_id = ENV.fetch("SELECTIVE_RUNNER_ID", generate_runner_id)
+          @runner_id = safe_filename(ENV.fetch("SELECTIVE_RUNNER_ID", generate_runner_id))
           @logger = init_logger(log)
         end
 
@@ -23,6 +23,7 @@ module Selective
           @transport_pid = spawn_transport_process(reconnect ? transport_url + "&reconnect=true" : transport_url)
 
           handle_termination_signals(transport_pid)
+          wait_for_connectivity
           run_main_loop
         rescue NamedPipe::PipeClosedError
           retry!
@@ -66,8 +67,6 @@ module Selective
         def run_main_loop
           loop do
             message = pipe.read
-            next sleep(0.1) if message.nil? || message.empty?
-
             response = JSON.parse(message, symbolize_names: true)
 
             @logger.info("Received Command: #{response}")
@@ -153,6 +152,31 @@ module Selective
           end
         end
 
+        def wait_for_connectivity
+          @connectivity = false
+
+          Thread.new do
+            sleep(5)
+            unless @connectivity
+              puts "Transport process failed to start. Exiting..."
+              kill_transport
+              exit(1)
+            end
+          end
+
+          loop do
+            message = pipe.read
+
+            # The message is nil until the transport opens the pipe
+            # for writing. So, we must handle that here.
+            next sleep(0.1) if message.nil?
+            
+            response = JSON.parse(message, symbolize_names: true)
+            @connectivity = true if response[:command] == "connected"
+            break
+          end
+        end
+
         def handle_termination_signals(pid)
           ["INT", "TERM"].each do |signal|
             Signal.trap(signal) do
@@ -175,7 +199,7 @@ module Selective
 
               sleep(1)
             end
-          rescue NamedPipe::PipeClosedError
+          rescue NamedPipe::PipeClosedError, IOError
             # If the pipe is close, move straight to killing
             # it forcefully.
           end
@@ -272,6 +296,13 @@ module Selective
           @debug
         end
 
+        def safe_filename(filename)
+          filename
+            .gsub(/[\/\\:*?"<>|\n\r]+/, '_')
+            .gsub(/^\.+|\.+$/, '')
+            .strip[0, 255]
+        end
+
         def with_error_handling(include_header: true)
           yield
         rescue => e

data/lib/selective/ruby/core/version.rb

@@ -3,7 +3,7 @@
 module Selective
   module Ruby
     module Core
-      VERSION = "0.1.3"
+      VERSION = "0.1.4"
     end
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: selective-ruby-core
 version: !ruby/object:Gem::Version
-  version: 0.1.3
+  version: 0.1.4
 platform: arm64-darwin
 authors:
 - Benjamin Wood
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2023-11-21 00:00:00.000000000 Z
+date: 2023-12-06 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: zeitwerk