omq-cli 0.14.2 → 0.14.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 23eb8d602298e5db1259e6d7d0effe08e8e0dfacba55b8d9a5287800290f2eb9
-  data.tar.gz: b492d78fd4997c82434853bc284f7f98c662d905a450e3545c1df2ae822bf40c
+  metadata.gz: a207b74cefe2b70d8f5910610be8ef955a2d466ae78e3b620860a5425361af8b
+  data.tar.gz: 7a15e05713459c741f2d780c89057bbfee435e63767464245041bcdfe5b31c59
 SHA512:
-  metadata.gz: a3abd80d9856c52e2674c86a3e31ed4f02d6511cad2e3d9f35a2ab5b65faaf07acdc4c442f3746bc5ed0dd56e730df0fabc4930c0cdfe05ad8e289335fc0e8d4
-  data.tar.gz: 98e5cf25c2e267aace41bb958843459ead62da2a338b8a64aa3f6fc3af571dfefaee30b01a0de720b8880a96c6c0c3b55fe7a8439a6b5151a54085233ded3aaa
+  metadata.gz: d38d7f5548147b2b5a4fdfe391180b29dba07975fe8fbd391586eaf580d739f66c05f66e0fbc488e976d47afaabbced2322940d39c1b783ed4c5636e5ce4d194
+  data.tar.gz: 301999bb8145e49e39d28f77dc75a71d22ead3ad3c01ebe85e809a73b2cc8baf3a59e980361dd43319d2c7d5be90ff7f22f74370ad89f7a53ae85e77e09634d9

data/CHANGELOG.md

@@ -1,5 +1,36 @@
 # Changelog
 
+## 0.14.3 — 2026-04-14
+
+### Fixed
+
+- **`omq req` (and other interactive senders) no longer wedge on
+  blank input lines.** `Formatter#decode` used to return `[]` for
+  a blank line, which `BaseRunner#send_msg` then silently dropped
+  — so REQ never sent a request but still blocked in `recv_msg`
+  waiting for a reply. Blank lines now decode to a single empty
+  frame (`[""]`) via the new
+  `Formatter::EMPTY_MSG = [Protocol::ZMTP::Codec::EMPTY_BINARY]`
+  constant, so the request actually goes out. As a side effect,
+  ascii/quoted decoding now uses `split("\t", -1)` and preserves
+  trailing empty frames (`"a\t\n"` → `["a", ""]`).
+- **Cleaner `:disconnected` log lines on plain peer close.**
+  `-vv` used to emit `omq: disconnected tcp://… (Stream finished
+  before reading enough data!)` — the raw io-stream message for
+  what is really just an `EOFError` at the ZMTP framing boundary.
+  `Term.format_event` now routes through a new
+  `Term.format_event_detail` helper that rewrites `EOFError` to
+  `(closed by peer)`, leaving other errors' messages untouched.
+  The underlying `event.detail[:error]` is unchanged.
+
+### Changed
+
+- **Lowercased `Term.format_attach` verbs.** `omq: Bound to …` /
+  `omq: Connecting to …` now render as `omq: bound to …` /
+  `omq: connecting to …`, matching the lowercase style already
+  used by every other `omq:` log line (`disconnected`, `listening`,
+  `handshake_succeeded`, …).
+
 ## 0.14.2 — 2026-04-13
 
 ### Changed

data/lib/omq/cli/formatter.rb

@@ -7,6 +7,10 @@ module OMQ
     # (omq-rfc-zstd) once enabled via +socket.compression=+; the
     # formatter sees plaintext frames in both directions.
     class Formatter
+      # Single empty frame — used as the decoded form of a blank input line.
+      EMPTY_MSG = [::Protocol::ZMTP::Codec::EMPTY_BINARY].freeze
+
+
       # @param format [Symbol] wire format (:ascii, :quoted, :raw, :jsonl, :msgpack, :marshal)
       def initialize(format)
         @format = format
@@ -45,9 +49,11 @@ module OMQ
       def decode(line)
         case @format
         when :ascii, :marshal
-          line.chomp.split("\t")
+          parts = line.chomp.split("\t", -1)
+          parts.empty? ? EMPTY_MSG : parts
         when :quoted
-          line.chomp.split("\t").map { |p| "\"#{p}\"".undump }
+          parts = line.chomp.split("\t", -1).map { |p| "\"#{p}\"".undump }
+          parts.empty? ? EMPTY_MSG : parts
         when :raw
           [line]
         when :jsonl

data/lib/omq/cli/term.rb

@@ -49,27 +49,46 @@ module OMQ
           "#{prefix}omq: << ZDICT (#{event.detail[:size]}B)"
         else
           ep     = event.endpoint ? " #{event.endpoint}" : ""
-          detail =
-            if event.detail.is_a?(Hash) && event.detail[:reason]
-              " (#{event.detail[:reason]})"
-            elsif event.detail
-              " #{event.detail}"
-            else
-              ""
-            end
+          detail = format_event_detail(event.detail)
           "#{prefix}omq: #{event.type}#{ep}#{detail}"
         end
       end
 
 
-      # Formats an "attached endpoint" log line (Bound to / Connecting to).
+      # Renders +MonitorEvent#detail+ as a " (...)" suffix for log lines.
+      # Rewrites plain peer-close exceptions (EOFError) to "closed by
+      # peer" — the io-stream library reports these as "Stream finished
+      # before reading enough data!", which is confusing noise for what
+      # is just a normal disconnect.
+      #
+      # @param detail [Hash, Object, nil]
+      # @return [String]
+      def format_event_detail(detail)
+        return "" if detail.nil?
+        return " #{detail}" unless detail.is_a?(Hash)
+
+        error = detail[:error]
+        reason = detail[:reason]
+
+        case error
+        when nil
+          reason ? " (#{reason})" : ""
+        when EOFError
+          " (closed by peer)"
+        else
+          " (#{reason || error.message})"
+        end
+      end
+
+
+      # Formats an "attached endpoint" log line (bound to / connecting to).
       #
       # @param kind [:bind, :connect]
       # @param url [String]
       # @param timestamps [Symbol, nil]
       # @return [String]
       def format_attach(kind, url, timestamps)
-        verb = kind == :bind ? "Bound to" : "Connecting to"
+        verb = kind == :bind ? "bound to" : "connecting to"
         "#{log_prefix(timestamps)}omq: #{verb} #{url}"
       end
 
@@ -85,7 +104,7 @@ module OMQ
       end
 
 
-      # Writes one "Bound to / Connecting to" line to +io+
+      # Writes one "bound to / connecting to" line to +io+
       # (default $stderr).
       #
       # @param kind [:bind, :connect]

data/lib/omq/cli/version.rb

@@ -2,6 +2,6 @@
 
 module OMQ
   module CLI
-    VERSION = "0.14.2"
+    VERSION = "0.14.3"
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: omq-cli
 version: !ruby/object:Gem::Version
-  version: 0.14.2
+  version: 0.14.3
 platform: ruby
 authors:
 - Patrik Wenger