process_bot 0.1.27 → 0.1.28

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 016c72db98675857edb0e34b274af5cc835d0cd5496d4bb6fccb7ebdaf05c555
-  data.tar.gz: 51a18efde6ff957df0dfe6f8e07e9df2f341a1519b3f17356de1ef7422f21e85
+  metadata.gz: e123dbde56775d5539bb91d48be68c1f83a09a2b001c6a61a1334aa3dba2b1ab
+  data.tar.gz: 34f9ff90352cad06e742ba69f7d4402d095678359caded93b20204f89b804a44
 SHA512:
-  metadata.gz: f2387a26b91b32675f6679af84f4e4096bde512f62f344a962a0312abea804395e6013b88f268c20094a04724f27c71680c421f63893c785f47586b472f68ff8
-  data.tar.gz: '0491239c4e11851cfef5704cbbc9e79611c356595192f5187199fd3a7c898f7e61e5885a302ed84cdedb53692a4bf7e89369791fe69a169663273d95b5f4ebd8'
+  metadata.gz: 8cbc29f75c5ffa3358f8614c11698fc4ac6548efa5a00a4d63a1004300707c80b395eb9229b877b06cc93c1d3b7e10987fa99d02f1942c67a133db277d9af49b
+  data.tar.gz: 8042fcbf747ba1162120b87e54d64bd1ac972615eaca4288865864ea76747e27d94b697b8de97d0a38ede453abbc9c30e5a54fdc3af32052f9ff466c82b9d311

data/AGENTS.md

@@ -9,7 +9,7 @@
 - Made graceful shutdown waiting optional and defaulted Capistrano to not wait.
 - Kept graceful handling synchronous and verified `bundle exec rspec`.
 - Enabled ProcessBot logging by default for Capistrano hooks (configurable via `process_bot_log`).
-- Always run RuboCop against changed or created Ruby files.
+- Always run RuboCop against changed or created Ruby files before pushing or opening a PR.
 - Added `graceful_no_wait` command and Capistrano task for non-blocking graceful shutdowns.
 - Always add or update tests for new/changed functionality, and run them.
 - Added coverage for graceful_no_wait and Capistrano wait defaults.

data/CHANGELOG.md

@@ -8,8 +8,8 @@
 - Add optional Sidekiq restart overlap and a new ProcessBot restart command.
 - Guard stop-related process scanning when subprocess PID/PGID is unavailable and fail stop loudly.
 - Wait briefly for subprocess PID assignment during stop; raise if PID is still missing so stop cannot silently succeed.
-
 - Require an active runner for custom stop commands to avoid constructing a fresh runner with no PID.
+- Buffer subprocess output by line before broadcasting it to control clients so Capistrano does not receive one-character log chunks.
 
 ## [0.1.0] - 2022-04-03
 

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    process_bot (0.1.27)
+    process_bot (0.1.28)
       knjrbfw (>= 0.0.116)
       pry
       rake
@@ -58,7 +58,7 @@ GEM
     rspec-expectations (3.13.5)
       diff-lcs (>= 1.2.0, < 2.0)
       rspec-support (~> 3.13.0)
-    rspec-mocks (3.13.7)
+    rspec-mocks (3.13.8)
       diff-lcs (>= 1.2.0, < 2.0)
       rspec-support (~> 3.13.0)
     rspec-support (3.13.7)

data/Rakefile

@@ -7,6 +7,6 @@ require "rubocop/rake_task"
 RSpec::Core::RakeTask.new(:spec)
 RuboCop::RakeTask.new
 
-Dir[File.expand_path("../lib/tasks/**/*.rake", __FILE__)].each { |f| load f }
+Dir[File.expand_path("lib/tasks/**/*.rake", __dir__)].each { |f| load f }
 
 task default: %i[spec rubocop]

data/lib/process_bot/process/runner.rb

@@ -3,6 +3,8 @@ require "knjrbfw"
 class ProcessBot::Process::Runner
   attr_reader :command, :exit_status, :handler_instance, :handler_name, :logger, :monitor, :options, :pid, :stop_time, :subprocess_pid
 
+  READ_CHUNK_SIZE = 4096
+
   def initialize(command:, handler_instance:, handler_name:, logger:, options:)
     @command = command
     @handler_instance = handler_instance
@@ -16,6 +18,17 @@ class ProcessBot::Process::Runner
     logger.log(output, type: type)
   end
 
+  def stream_output(io, type:)
+    buffer = +""
+
+    loop do
+      buffer << io.readpartial(READ_CHUNK_SIZE)
+      flush_complete_lines(type: type, buffer: buffer)
+    end
+  rescue EOFError, Errno::EIO
+    flush_remaining_output(type: type, buffer: buffer)
+  end
+
   def running?
     !stop_time
   end
@@ -31,13 +44,7 @@ class ProcessBot::Process::Runner
       logger.logs "Command running with PID #{pid}: #{command}"
 
       stdout_reader_thread = Thread.new do
-        stdout.each_char do |chunk|
-          monitor.synchronize do
-            output(type: :stdout, output: chunk)
-          end
-        end
-      rescue Errno::EIO
-        # Process done
+        stream_output(stdout, type: :stdout)
       ensure
         status = Process::Status.wait(subprocess_pid, 0)
 
@@ -46,11 +53,7 @@ class ProcessBot::Process::Runner
       end
 
       stderr_reader_thread = Thread.new do
-        stderr_reader.each_char do |chunk|
-          monitor.synchronize do
-            output(type: :stderr, output: chunk)
-          end
-        end
+        stream_output(stderr_reader, type: :stderr)
       end
 
       find_sidekiq_pid if handler_name == "sidekiq"
@@ -182,4 +185,37 @@ class ProcessBot::Process::Runner
       break
     end
   end
+
+  def flush_complete_lines(type:, buffer:)
+    loop do
+      separator_index = next_separator_index(buffer)
+      break unless separator_index
+
+      monitor.synchronize do
+        output(type: type, output: buffer.slice!(0, separator_index + 1))
+      end
+    end
+
+    return if buffer.bytesize < READ_CHUNK_SIZE
+
+    monitor.synchronize do
+      output(type: type, output: buffer.dup)
+    end
+
+    buffer.clear
+  end
+
+  def flush_remaining_output(type:, buffer:)
+    return if buffer.empty?
+
+    monitor.synchronize do
+      output(type: type, output: buffer.dup)
+    end
+
+    buffer.clear
+  end
+
+  def next_separator_index(buffer)
+    buffer.index(/[\r\n]/)
+  end
 end

data/lib/process_bot/version.rb

@@ -1,3 +1,3 @@
 module ProcessBot
-  VERSION = "0.1.27".freeze
+  VERSION = "0.1.28".freeze
 end

data/lib/tasks/release.rake

@@ -4,7 +4,7 @@ require "pathname"
 require "rubygems/version"
 require "shellwords"
 
-class ProcessBotRubygemsRelease # rubocop:disable Lint/ConstantDefinitionInBlock
+class ProcessBotRubygemsRelease
   VERSION_FILE = Pathname.new(File.expand_path("../process_bot/version.rb", __dir__)) unless const_defined?(:VERSION_FILE)
 
   def call
@@ -60,13 +60,15 @@ private
   end
 
   def bumped_version
+    major, minor, patch = version_segments
+
     case bump_type
     when "major"
-      [version_segments[0] + 1, 0, 0].join(".")
+      format_version(major + 1, 0, 0)
     when "minor"
-      [version_segments[0], version_segments[1] + 1, 0].join(".")
+      format_version(major, minor + 1, 0)
     when "patch"
-      [version_segments[0], version_segments[1], version_segments[2] + 1].join(".")
+      format_version(major, minor, patch + 1)
     else
       raise "Unsupported BUMP=#{bump_type.inspect}. Use patch, minor, major, or VERSION=x.y.z."
     end
@@ -84,6 +86,10 @@ private
     @current_version ||= VERSION_FILE.read[/VERSION = "([^"]+)"\.freeze/, 1] || raise("Could not find current version")
   end
 
+  def format_version(major, minor, patch)
+    [major, minor, patch].join(".")
+  end
+
   def bump_version!(next_version)
     raise "Next version must differ from current version" if next_version == current_version
 

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: process_bot
 version: !ruby/object:Gem::Version
-  version: 0.1.27
+  version: 0.1.28
 platform: ruby
 authors:
 - kaspernj
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2026-04-04 00:00:00.000000000 Z
+date: 2026-04-09 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: knjrbfw