process_bot 0.1.30 → 0.1.31

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 5e091db49a7e39df21edb300dc0ebb57cb5b98a73be033b220d24f72a7f72c36
-  data.tar.gz: 59e9442735c2a07cc568e8f9b0f1ed3c3660fa407d1f85ed7326348722ed7dbc
+  metadata.gz: c2183ea1e8ec1a90a39ec3d3d7d2753150f99e164f31e3650d83a40a784d1a67
+  data.tar.gz: 0b009604dcd29677832a88cd814b29e7259cef2709d22f5ef140673d298a0981
 SHA512:
-  metadata.gz: 3459f12d21d2d221c152ed047cd51dc4c9134e285cfc52847997099ace48a1ddd60fc8c7c54d5a6b6ca8535b2054e47f9ae5b72da010188e56eca304194b30e6
-  data.tar.gz: 8b7bd9482806698b84f3ea356b00c2f71d58850249fb0e2aabb28591495094e727324eb6bad5f0e5ee4740adef932c58f91f232d90e8ea8488f7f453a9bde138
+  metadata.gz: f6239960b599dc220ab6e8519f7edd7061556b69e0918d48cd655a25bd1239307b6f7b836599811cc3b69dd72ad40a830988aef00d3cb92fdfbbd9ebf02428c2
+  data.tar.gz: 4566c50a189d90b5c638369db712d63355700894a6643f17b44e4b0544bee9fdb949e13046c9f24c21652742e4352d8139bf80a5c79612d007a8114f06a4fe9c

data/.ruby-version

@@ -1 +1 @@
-2.7.8
+3.2.5

data/CHANGELOG.md

@@ -1,6 +1,7 @@
 ## [Unreleased]
 - Drop Ruby 2.x support. Minimum Ruby is now 3.2; CI covers 3.2/3.3/3.4 and RuboCop's `TargetRubyVersion` follows. Shorthand anonymous forwarding (`*, **, &`) is applied where it replaces redundant `*args, **opts, &block` pass-throughs.
 - Fail `start_tcp_server` when another ProcessBot with the same `--id` is already running instead of silently drifting to a new port. Port drift across unrelated services is still supported; drift for a duplicate id was the root cause of Capistrano deploys leaving a stale previous-release ProcessBot alive on a drifted port while every subsequent `stop --port X` hit the wrong instance.
+- Treat legacy ProcessBot titles without `application_basename` as duplicates when they share the same `--id` and control port, and include the same legacy match in forced stop diagnostics.
 - Stop accepting new control commands during shutdown so in-flight responses complete reliably.
 - Stream ProcessBot logs to connected control clients for Capistrano output.
 - Sanitize broadcast log output to keep JSON encoding safe.

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    process_bot (0.1.30)
+    process_bot (0.1.31)
       knjrbfw (>= 0.0.116)
       pry
       rake
@@ -18,7 +18,7 @@ GEM
     http2 (0.0.36)
       string-cases (~> 0)
     io-console (0.8.2)
-    json (2.19.3)
+    json (2.19.5)
     knjrbfw (0.0.116)
       datet
       http2
@@ -62,7 +62,7 @@ GEM
       diff-lcs (>= 1.2.0, < 2.0)
       rspec-support (~> 3.13.0)
     rspec-support (3.13.7)
-    rubocop (1.86.1)
+    rubocop (1.86.2)
       json (~> 2.3)
       language_server-protocol (~> 3.17.0.2)
       lint_roller (~> 1.1.0)

data/lib/process_bot/control_socket.rb

@@ -74,10 +74,23 @@ class ProcessBot::ControlSocket
 
   def find_duplicate_id_entries(id, basename)
     running_process_bot_entries.select do |entry|
-      entry[:id] == id && entry[:application_basename] == basename
+      duplicate_id_entry_matches?(entry, id, basename)
     end
   end
 
+  def duplicate_id_entry_matches?(entry, id, basename)
+    return false unless entry[:id] == id
+    return true if basename && entry[:application_basename] == basename
+
+    entry[:application_basename].nil? && same_control_port?(entry)
+  end
+
+  def same_control_port?(entry)
+    return false unless entry[:port] && options[:port]
+
+    entry[:port].to_i == options[:port].to_i
+  end
+
   def duplicate_id_error_message(id, duplicates)
     details = duplicates.map { |entry| "PID #{entry[:pid]} on port #{entry[:port]}" }.join(", ")
     example_port = duplicates.first[:port]
@@ -235,15 +248,19 @@ class ProcessBot::ControlSocket
         next
       end
 
-      pid = process.data["pid"] || process.pid
-      entries << {
-        application_basename: process_data["application_basename"],
-        id: process_data["id"]&.to_s,
-        pid: pid,
-        port: process_data["port"]&.to_i
-      }
+      entries << process_bot_entry_from_process_data(process, process_data)
     end
 
     entries
   end
+
+  def process_bot_entry_from_process_data(process, process_data)
+    {
+      application: process_data["application"],
+      application_basename: process_data["application_basename"],
+      id: process_data["id"]&.to_s,
+      pid: process.data["pid"] || process.pid,
+      port: process_data["port"]&.to_i
+    }
+  end
 end

data/lib/process_bot/options.rb

@@ -36,17 +36,19 @@ class ProcessBot::Options
   end
 
   def application_basename
-    @application_basename ||= begin
-      app_path_parts = release_path.split("/")
+    @application_basename ||= application_basename_from_release_path
+  end
 
-      if release_path.include?("/releases/")
-        app_path_parts.pop(2)
-      elsif release_path.end_with?("/current")
-        app_path_parts.pop
-      end
+  def application_basename_from_release_path
+    app_path_parts = release_path.split("/")
+    deployment_marker_index = [
+      app_path_parts.rindex("releases"),
+      app_path_parts.rindex("current")
+    ].compact.max
 
-      app_path_parts.last
-    end
+    return app_path_parts[0...deployment_marker_index].last if deployment_marker_index
+
+    app_path_parts.last
   end
 
   def possible_process_titles

data/lib/process_bot/process.rb

@@ -334,9 +334,35 @@ class ProcessBot::Process # rubocop:disable Metrics/ClassLength
   end
 
   def process_bot_process_line_matches?(line)
-    line.include?("ProcessBot {") &&
-      line.include?("\"application\":\"#{options[:application]}\"") &&
-      line.include?("\"id\":\"#{options[:id]}\"")
+    process_data = process_bot_data_from_line(line)
+    return false unless process_data
+    return false unless process_data["id"]&.to_s == options[:id].to_s
+
+    process_bot_application_matches?(process_data) || legacy_process_bot_port_matches?(process_data)
+  end
+
+  def process_bot_data_from_line(line)
+    match = line.match(/ProcessBot (\{.+\})/)
+    return unless match
+
+    JSON.parse(match[1])
+  rescue JSON::ParserError
+    nil
+  end
+
+  def process_bot_application_matches?(process_data)
+    application = options[:application]
+    return true if application && process_data["application"] == application
+
+    basename = safe_application_basename
+    basename && process_data["application_basename"] == basename
+  end
+
+  def legacy_process_bot_port_matches?(process_data)
+    return false unless process_data["port"] && options[:port]
+
+    process_data["application_basename"].nil? &&
+      process_data["port"].to_i == options[:port].to_i
   end
 
   def force_stop_process_bot_if_configured(matching_processes)

data/lib/process_bot/version.rb

@@ -1,3 +1,3 @@
 module ProcessBot
-  VERSION = "0.1.30".freeze
+  VERSION = "0.1.31".freeze
 end

metadata

@@ -1,13 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: process_bot
 version: !ruby/object:Gem::Version
-  version: 0.1.30
+  version: 0.1.31
 platform: ruby
 authors:
 - kaspernj
+autorequire:
 bindir: exe
 cert_chain: []
-date: 1980-01-02 00:00:00.000000000 Z
+date: 2026-05-20 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: knjrbfw
@@ -186,6 +187,7 @@ metadata:
   source_code_uri: https://github.com/kaspernj/process_bot
   changelog_uri: https://github.com/kaspernj/process_bot/blob/master/CHANGELOG.md
   rubygems_mfa_required: 'true'
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -200,7 +202,8 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.6.9
+rubygems_version: 3.4.19
+signing_key:
 specification_version: 4
 summary: Run and control processes.
 test_files: []