rb-portless 0.5.1 → 0.5.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 0b18af4327ab4b82c67d859bac408f7b26488979f5dbde99e576095769a29dc5
-  data.tar.gz: 71be0d1633e12ad74ee1f8ca25859e3675cb135e710b76cb526b7019b644a5f0
+  metadata.gz: 8a45c842e40387472fcb2ea9bb452cdd1015566db6de7ee653824f881a522af1
+  data.tar.gz: 6b3c996607326c31b3abfb1fd8916cf3abe81ead74af7de13be36a8cc07bd119
 SHA512:
-  metadata.gz: 4a22873edc19f846dbdc8937c5b7eaa281e033d49c7d6f5b4228aa668ecbb249bf34d4dd2a81f0a46a671369ffe0a539db18e551a7ae57e9bb5c952703de2168
-  data.tar.gz: a01760f31c79efbf03b8e6b79c260f1764c7fe6851a8d7b5c93bc7558a4956677dfeb064bb16ac05de6c23e2dabfbf1b18ce79d525e790afec6949f803d78484
+  metadata.gz: 593a5feaabae0a2781acd89831eeecb3ac03c99bc0e721c7626eeaf8f3a6c2b2f836d7160f04dffabae9645825753970bfd6545495a730c298a5f6eca9f06a71
+  data.tar.gz: 607dd39cc15026cbddcc0af63d5f6a4f21b536511a34174df10517e1ff0807db3d1bea0df7c5005a9c5aed3c60cddad83df2edb6f6c91aa840d9bb5042da7b94

data/CHANGELOG.md

@@ -4,6 +4,19 @@ All notable changes to this project are documented here. The format follows
 [Keep a Changelog](https://keepachangelog.com/), and the project adheres to
 [Semantic Versioning](https://semver.org/).
 
+## [0.5.2] — 2026-09-08
+
+### Fixed
+
+- **`run --force` killed the run doing the forcing.** Taking over a hostname
+  held by a *static alias* passed its pid straight to `kill`, and an alias
+  records pid 0 — which to `kill(2)` means "every process in my group". So
+  `bin/dev --force` TERMed itself and its shell job, printing `Terminated: 15`
+  before the dev server ever started. An alias has no owner process to displace,
+  so force now just takes the hostname over, and `terminate` refuses any pid
+  that isn't a real process. The conflict message you get *without* `--force`
+  now says "a static alias" instead of the unlookuppable "pid 0".
+
 ## [0.5.1]
 
 ### Security

data/lib/portless/route_store.rb

@@ -45,7 +45,9 @@ module Portless
     end
 
     # Register (or replace) a route. Conflicts with a *live* different owner raise
-    # unless force, which SIGTERMs the incumbent. Alias routes use pid 0. Public
+    # unless force, which SIGTERMs the incumbent. Alias routes use pid 0: they
+    # count as live (never reaped) but have no process, so force just takes the
+    # hostname over. Public
     # share URLs (tailscale/ngrok), when present, are recorded so `list` can show
     # them while the run is active.
     def add(hostname:, port:, pid:, force: false, tailscale: nil, ngrok: nil, lan: false)
@@ -55,7 +57,7 @@ module Portless
         if existing && existing["pid"].to_i != pid.to_i && !dead?(existing["pid"])
           unless force
             raise RouteConflictError,
-                  "#{hostname} is already served by pid #{existing['pid']} — pass --force to take it over"
+                  "#{hostname} is already served by #{owner(existing)} — pass --force to take it over"
           end
           terminate(existing["pid"])
         end
@@ -154,8 +156,21 @@ module Portless
       true # exists, owned by someone else (e.g. root proxy)
     end
 
+    # Who to blame in a conflict message. An alias has no process behind it, so
+    # "pid 0" would be nonsense to read and impossible to go look up.
+    def owner(route)
+      route["pid"].to_i.zero? ? "a static alias" : "pid #{route['pid']}"
+    end
+
+    # Displace the route's current owner. Only a real pid gets signalled: 0 means
+    # "my whole process group" and a negative one means "that group", so passing
+    # an alias route straight through would TERM the very run asking for the
+    # takeover (and its shell job with it) instead of the incumbent.
     def terminate(pid)
-      Process.kill("TERM", pid.to_i)
+      pid = pid.to_i
+      return unless pid.positive?
+
+      Process.kill("TERM", pid)
     rescue StandardError
       nil
     end

data/lib/portless/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Portless
-  VERSION = "0.5.1"
+  VERSION = "0.5.2"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: rb-portless
 version: !ruby/object:Gem::Version
-  version: 0.5.1
+  version: 0.5.2
 platform: ruby
 authors:
 - David Afonso
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2026-08-05 00:00:00.000000000 Z
+date: 2026-09-08 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: async