samagotchi 0.22.0 → 0.23.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 9019626f95da411558ec5350303433de58fd07f9db6f9e22323856e4e2ed31a0
-  data.tar.gz: 015df1a469fb64ff5556ce6f683ee23abf039865a34584fff07e3f3c32e4724e
+  metadata.gz: 236e301c873eb2b96520aeeb3e7668e535f50174b1afb94d28a149cef161d418
+  data.tar.gz: adc3281cace9460e307abe5abc33719041863898e848f10b290d5d2d3e4c93d5
 SHA512:
-  metadata.gz: 1bd652f5d5c2d549dce4eba12cc6577f34cb80ae2a95ea4d63492b4581ed510a5f4eeb8b741f86da7af39198dbfc4981e5d6e7d019dfff7d90f8c0a19722cd6b
-  data.tar.gz: fe6e4cbc412b112d157ad16a1f176fa1308ed094498eb982770b5a6b87859a89c9d544b9c0791d953e0823e51e09396f13e484eccdb240cdd0a0daad8dfe3f8e
+  metadata.gz: 6e97629a0a1459a7c9b403a495aea87807f4b2570421e369a56a01afd140830b8284e7d25a86132f14ebdab95275913f66caa8aea48680d23b507dec9d57b519
+  data.tar.gz: 284540411644a596492ad2d91c36eecb23f8eb435b95e06781e387b4933a6210680e15b6d79f035b1abd029b726aa9489bdee39816a9d509bfb7a1810b92b00e

data/CHANGELOG.md

@@ -8,6 +8,37 @@ and commands may change between minor versions. How releases are made:
 
 ## [Unreleased]
 
+## [0.23.0] - 2026-10-04
+
+### Changed
+
+- chi no longer strips the blank lines Splash before 1.2.0 put at the start of an answer (incoai/splash#254, fixed
+  in Splash 1.2.0): upgrade Splash with `brew upgrade incoai/tap/splash`.
+- Requests to OpenRouter send `max_tokens: 32768` (a request's own limit, as a recap's, still wins). OpenRouter
+  holds each running request's estimated cost against the balance, counting the output `max_tokens` allows (a
+  fixed per-request cap without one), so concurrent requests hit "would exceed your available credits given your
+  current in-flight requests" (402) sooner without a limit. `default.max_tokens` raises or lowers that limit.
+- `default.n_predict` is renamed `default.max_tokens` (env `SAMAGOTCHI_DEFAULT_MAX_TOKENS`, flag
+  `--default-max-tokens`), with no alias: an old `n_predict:` in config.yml warns as an unknown key. It now
+  limits chat turns on OpenAI-compatible hosts too, sent as `max_tokens` on any host; it was used only by native
+  hosts (where it is still sent as llama.cpp's `n_predict`).
+
+### Fixed
+
+- A Ctrl-C (or SIGTERM) while chi loads bundle hooks and plugins stops chi again; before, it was logged as a
+  failed hook and chi carried on. A hook's or plugin's own error, `exit` or syntax error is still only reported.
+- A continue turn that fails before it begins (its session save, say) asks the step-limit question again
+  instead of crashing the session's worker.
+- Native Gemma 4 prompts follow Gemma 4's chat template: turns end with `<turn|>` (not Gemma 3's
+  `<end_of_turn>`, which Gemma 4 reads as plain text), so the stop sequence matches; tool results stay
+  inside the model's turn as `response:NAME{value:…}` blocks, one per call; tools are declared in the
+  template's compact form at the end of the system turn; an answer's thought is dropped once the next
+  user turn starts; and thinking off prefills Gemma's empty thought channel.
+
+Update with `chi update`: btw 0.1.2, check-in 0.2.2, known-names 0.1.5, loop-guard 0.3.2, mcp 0.4.2, skills 0.1.6 and
+source-links 0.3.3 move (code style only, no behaviour change). If your config.yml has `default: n_predict:`, rename it
+to `max_tokens:`. Restart `chi web` and running sessions afterwards (`chi sessions restart ID`).
+
 ## [0.22.0] - 2026-10-04
 
 ### Added
@@ -1273,7 +1304,8 @@ and long-lived sessions.
 - A macOS desktop helper (`chi desktop install`): a "Send to chi" Service and a
   hotkey panel that send selected text or the clipboard to your sessions.
 
-[Unreleased]: https://github.com/dm1try/samagotchi/compare/v0.22.0...HEAD
+[Unreleased]: https://github.com/dm1try/samagotchi/compare/v0.23.0...HEAD
+[0.23.0]: https://github.com/dm1try/samagotchi/compare/v0.22.0...v0.23.0
 [0.22.0]: https://github.com/dm1try/samagotchi/compare/v0.21.0...v0.22.0
 [0.21.0]: https://github.com/dm1try/samagotchi/compare/v0.20.0...v0.21.0
 [0.20.0]: https://github.com/dm1try/samagotchi/compare/v0.19.0...v0.20.0

data/bin/chi

@@ -72,6 +72,7 @@ cli_overrides = {}
 # Strict kebab enforcement: --recap_base_url is unknown (OptionParser would otherwise accept _ as -)
 ARGV.each do |arg|
   next unless arg.start_with?("--") && arg.include?("_")
+
   # Allow --no-interrupt etc. already dash; underscore is always rejected for config flags
   # We check raw ARGV before OptionParser normalizes
   base = arg.split("=", 2).first
@@ -108,7 +109,7 @@ parser = OptionParser.new do |opts|
   opts.on("--[no-]shared", "Run the session in a background worker and attach to it, so `chi web` can share it (the default, config session.shared; with --resume: that session); --no-shared runs a plain in-process REPL") { |v| options[:shared] = v }
   opts.on("--model NAME", "Model for this run, not saved (overrides --resume's; config default.model) [env SAMAGOTCHI_DEFAULT_MODEL]") { |m| options[:model] = m; cli_overrides["default.model"] = m }
   opts.on("--profile NAME", Samagotchi::ModelProfile::NAMES,
-          "Prompt profile for every model in this run (#{Samagotchi::ModelProfile::NAMES.join('|')}) [alias for --model-profile]") { |p| cli_overrides["model.profile"] = p }
+          "Prompt profile for every model in this run (#{Samagotchi::ModelProfile::NAMES.join("|")}) [alias for --model-profile]") { |p| cli_overrides["model.profile"] = p }
   opts.on("--thinking LEVEL", Samagotchi::Thinking::LEVELS.map(&:to_s), "How much the model thinks this run: off|low|medium|high|default (outranks config.yml; " \
                               "sessions already running keep theirs) [env SAMAGOTCHI_THINKING_LEVEL]") { |l| cli_overrides["thinking.level"] = l }
   opts.on("--memory NAME", "Preload a memory entry into the system prompt (repeatable; a comma list too)") { |m| (options[:memories] ||= []) << m }
@@ -133,13 +134,14 @@ parser = OptionParser.new do |opts|
     next if entry.key == "web.port" # already handled as --port
     next if entry.key == "default.model" # alias --model above
     next if entry.key == "web.markdown" # handled below with explicit inverse flags
+
     case entry.type
     when :bool
       # --[no-]: a bool that defaults to on is only useful switched off.
-      opts.on("--[no-]#{entry.cli_flag.delete_prefix('--')}", "Set #{entry.key} (bool, default #{entry.default ? 'on' : 'off'}) [env #{entry.env_key}]") { |v| cli_overrides[entry.key] = v }
+      opts.on("--[no-]#{entry.cli_flag.delete_prefix("--")}", "Set #{entry.key} (bool, default #{entry.default ? "on" : "off"}) [env #{entry.env_key}]") { |v| cli_overrides[entry.key] = v }
     when :integer, :float, :string, :enum
       desc = "Set #{entry.key} [env #{entry.env_key}]"
-      desc += " (#{entry.enum_values.join('|')})" if entry.type == :enum
+      desc += " (#{entry.enum_values.join("|")})" if entry.type == :enum
       desc += " (#{value_hints[entry.key]})" if value_hints.key?(entry.key)
       desc += "; only without a hosts: config" if %w[server.host server.port].include?(entry.key)
       desc += "; hosts: entries without a transport use it" if entry.key == "server.transport"
@@ -257,7 +259,10 @@ if options[:memories] || options[:muted]
   end
   memory_exists = lambda do |raw|
     scope, name = raw.split("/", 2)
-    scope, name = nil, raw unless name && Samagotchi::Tools::VALID_SCOPES.include?(scope)
+    unless name && Samagotchi::Tools::VALID_SCOPES.include?(scope)
+      scope = nil
+      name = raw
+    end
     !Samagotchi::Tools::MemoryRead.call(name, scope: scope).start_with?("Error:")
   end
   cli_memories = (options[:memories] || []).flat_map { |m| m.split(",") }.map(&:strip).reject(&:empty?)

data/docs/configuration.md

@@ -308,8 +308,8 @@ prompt profile (see "Prompt profile" below). How the selector reaches the reques
   (`model: Field required`) without it, so samagotchi forwards the selector
   resolved to the exact id listed in the server's `/v1/models` — matched by exact
   (case-insensitive) first, then substring, then passed through unchanged. That
-  resolved id is usually prefixed (e.g. `mlx-community--gemma-3-4b-it-4bit`), so a
-  short selector such as `gemma-3-4b-it-4bit` is what you set in
+  resolved id is usually prefixed (e.g. `mlx-community--gemma-4-e4b-it-4bit`), so a
+  short selector such as `gemma-4-e4b-it-4bit` is what you set in
   `default.model`. An unknown selector passes through raw and oMLX 404s,
   listing its available models; if `/v1/models` is unreachable, samagotchi falls
   back to the raw selector and lets the server decide (its own 400/404). Either
@@ -535,7 +535,7 @@ With several `hosts:`, `host:model` (or an alias naming a host) pins the host.
 An unqualified model name goes to the default host when its `/models` list has
 that exact id, else to the first host in `hosts:` order whose list has it, else
 to the default host. Only exact ids count, never a substring: `/model gemma` on
-a box that lists `gemma-3-12b` needs `box:gemma` or the exact id. The lists are
+a box that lists `gemma-4-26b` needs `box:gemma` or the exact id. The lists are
 known only after `/models` ran (nothing is fetched before the first turn), so
 until then an unqualified name goes to the default host; use `host:model` to pin
 one. A **remote** host (an `https` url or a public address; see "Remote or local") keeps its model
@@ -837,7 +837,7 @@ described in their own sections.
 |---|---|---|---|
 | `default.model` | (required) | `--model` | The model a new session starts with; `host:model` pins a host. |
 | `default.input` | none | | Text pre-filled at the first prompt (a trailing space is kept); `--no-default-input` skips it. See [CLI](cli.md). |
-| `default.n_predict` | server's | yes | Most tokens one generation may produce (native hosts). |
+| `default.max_tokens` | server's | yes | Most tokens one generation may produce (`n_predict` on a native llama.cpp host; OpenRouter: 32768 when unset). |
 | `model.profile` | none | `--profile` | Prompt profile for every model (`qwen36`, `gemma4`); env and CLI only. See "Prompt profile". |
 | `server.transport` | `llama_cpp` | yes | `llama_cpp`, `mlx` or `omlx`; see "Model Server Transport". |
 | `server.host` | `localhost` | yes | The model server when there is no `hosts:` map. |

data/docs/releasing.md

@@ -52,7 +52,7 @@ as the GitHub release body.
 | `rake bundles:check` | Sha lines match; a bundle changed since the last `v*` tag has a higher version (skipped, with a note, before the first tag). Warns (doesn't fail) when such a bundle's `requires_chi` still admits the tagged chi: raise it if the change uses anything new. |
 | `rake release:draft_changelog` | A draft Unreleased section from the commit subjects since the last tag, grouped. Stdout only. |
 | `rake release:bump[X.Y.Z]` | VERSION, the system manifest, Gemfile.lock, CHANGELOG. No commit. |
-| `rake release:check` | Clean tracked tree, VERSION == system bundle, a CHANGELOG section, `bundles:check`, rspec + npm test, `gem build`, then a clean install into a temp GEM_HOME that runs `chi --version`, `chi self` and `chi bundle list` with a temp HOME. Needs the network (gem dependencies). |
+| `rake release:check` | Clean tracked tree, VERSION == system bundle, a CHANGELOG section, `bundles:check`, `rake lint` (RuboCop), rspec + npm test, `gem build`, then a clean install into a temp GEM_HOME that runs `chi --version`, `chi self` and `chi bundle list` with a temp HOME. Needs the network (gem dependencies). |
 | `rake release:notes[X.Y.Z]` | Prints the CHANGELOG section (default: the current VERSION). |
 
 ## Runbook

data/docs/testing.md

@@ -4,8 +4,14 @@
 bundle exec parallel_rspec -n 8   # Ruby specs, in parallel (or plain: bundle exec rspec)
 npm test                          # web frontend specs
 npm run e2e                       # web UI happy paths in Chromium, fake model (once: npx playwright install chromium)
+bundle exec rake lint             # RuboCop: no offenses (CI runs it)
 ```
 
+Run `bundle exec rubocop -a` on the files you changed before committing: every enabled style cop follows the
+code's own style and has a safe autocorrect, so `-a` fixes them. What it leaves is a Lint finding (a possible
+bug) or a broken spec: fix it, or disable that line with the reason (`# rubocop:disable Cop -- why`).
+`.rubocop.yml` says why each cop that is off is off.
+
 No spec reads your own setup. The suite points `XDG_CONFIG_HOME` and `XDG_STATE_HOME` at temp folders,
 clears your `SAMAGOTCHI_*` environment, and keeps every spec off the network (WebMock).
 

data/lib/samagotchi/answer_command.rb

@@ -84,12 +84,14 @@ module Samagotchi
       options = parsed.options
       return usage_error("give one session id") unless parsed.args.size == 1
       return usage_error("--question QID is required (chi send --wait prints it)") if options[:question].to_s.strip.empty?
+
       if options[:dismiss]
         return usage_error("--dismiss takes no --option or --text") if !options[:options].empty? || options[:text]
       elsif options[:options].empty? && options[:text].nil?
         return usage_error("nothing to answer with: --option, --text or --dismiss")
       end
       return usage_error("--format takes text or json") if options[:format] && !FORMATS.include?(options[:format])
+
       if options[:timeout]
         options[:timeout] = Float(options[:timeout], exception: false)
         return usage_error("--timeout takes seconds") unless options[:timeout]&.positive?
@@ -155,7 +157,7 @@ module Samagotchi
     # (Guardrails::ParentApprovals, with this process's config.yml). A
     # Deny, text alone or a dismissal always passes: each one denies. The
     # worker checks it again with its own config.
-    def approval_refusal(pending, selected, id)
+    def approval_refusal(pending, selected, _id)
       offered = Array(pending[:options]).map(&:to_s)
       indices = selected.map { |label| offered.index(label) }
       reason = Guardrails::ParentApprovals.refusal(pending, indices, setting: Guardrails::ParentApprovals.setting)
@@ -170,7 +172,7 @@ module Samagotchi
 
     # A step-limit question can't be left unanswered: the turn waits on it.
     def not_dismissable(pending)
-      "the step-limit question can't be dismissed: answer #{Array(pending[:options]).join(' or ')} " \
+      "the step-limit question can't be dismissed: answer #{Array(pending[:options]).join(" or ")} " \
         "(--option Stop --text WHY stops it)"
     end
 

data/lib/samagotchi/bootstrap/config_writer.rb

@@ -15,7 +15,8 @@ module Samagotchi
     # comments), after a backup, and is checked by parsing it again: every
     # other key must be unchanged, or the backup goes back.
     class ConfigWriter
-      Error = Class.new(StandardError)
+      class Error < StandardError
+      end
 
       # kind: :new (a fresh file), :appended, :exists (the host is already
       #   there as +existing+; nothing written), :snippet (a file chi can't

data/lib/samagotchi/bootstrap/probe.rb

@@ -18,7 +18,7 @@ module Samagotchi
     # port answers at once instead of after chi's usual backoff.
     class Probe
       # The ports `chi bootstrap` with no target tries on localhost.
-      LOCAL_PORTS = { 8080 => "llama.cpp", 11434 => "Ollama", 1234 => "LM Studio", 8000 => "vLLM" }.freeze
+      LOCAL_PORTS = { 8080 => "llama.cpp", 11_434 => "Ollama", 1234 => "LM Studio", 8000 => "vLLM" }.freeze
       DEFAULT_PORT = 8080
       # Connect and read seconds for /props and /v1/models: a remote https
       # handshake needs more than Client#server_props' 1 s / 2 s.

data/lib/samagotchi/bootstrap_command.rb

@@ -193,7 +193,7 @@ module Samagotchi
     def describe(result)
       if result.native?
         build = result.props["build_info"].to_s
-        @stdout.puts("found: llama.cpp at #{result.candidate.root}#{build.empty? ? "" : " (build #{build})"}")
+        @stdout.puts("found: llama.cpp at #{result.candidate.root}#{" (build #{build})" unless build.empty?}")
       else
         @stdout.puts("found: an OpenAI-compatible API at #{result.candidate.base}")
       end
@@ -406,7 +406,7 @@ module Samagotchi
         shown.first(PICK_SHOWN).each_with_index { |item, i| @stdout.puts(format("  %2d) %s", i + 1, item)) }
         more = shown.size - PICK_SHOWN
         @stdout.puts("  … #{more} more: type part of a name to narrow the list") if more.positive?
-        @stdout.print("#{noun} (number#{items.size > PICK_SHOWN ? " or text" : ""}, empty to stop): ")
+        @stdout.print("#{noun} (number#{" or text" if items.size > PICK_SHOWN}, empty to stop): ")
         answer = @stdin.gets.to_s.strip
         return nil if answer.empty?
 

data/lib/samagotchi/bridge/bounded_queue.rb

@@ -1,7 +1,5 @@
 # frozen_string_literal: true
 
-require "thread"
-
 module Samagotchi
   class Bridge
     # A thread-safe bounded FIFO used to decouple the turn thread (which

data/lib/samagotchi/bridge/sse_writer.rb

@@ -217,8 +217,6 @@ module Samagotchi
           "\r\n"
         )
         io.flush
-      rescue Errno::EPIPE, Errno::ECONNRESET, IOError
-        raise
       end
 
       # Serialise one SSE event: an `id:` line (the replay cursor), an optional
@@ -232,15 +230,11 @@ module Samagotchi
         JSON.generate(data).each_line { |line| lines << "data: #{line.chomp}" }
         io.write(lines.join("\r\n") + "\r\n\r\n")
         io.flush
-      rescue Errno::EPIPE, Errno::ECONNRESET, IOError
-        raise
       end
 
       def write_heartbeat(io)
         io.write(": ping\r\n\r\n")
         io.flush
-      rescue Errno::EPIPE, Errno::ECONNRESET, IOError
-        raise
       end
 
       # Emit a control frame carrying the current snapshot so a client that

data/lib/samagotchi/bridge/turn_accumulator.rb

@@ -230,8 +230,8 @@ module Samagotchi
         if Events::TURN_END.include?(type)
           @turn = nil
           @merged_origins = []
-        else
-          @turn[:event_seq] = event[:event_seq] if event[:event_seq]
+        elsif event[:event_seq]
+          @turn[:event_seq] = event[:event_seq]
         end
       end
 

data/lib/samagotchi/bridge.rb

@@ -481,7 +481,7 @@ module Samagotchi
       %w[GET snapshot] => :handle_snapshot,
       %w[GET tail] => :handle_tail
     }.freeze
-    ROUTE_PATH = %r|\A/session/([^/]+)/(.+)\z|u
+    ROUTE_PATH = %r{\A/session/([^/]+)/(.+)\z}u
 
     # @return [Array(Symbol, String), nil] the handler and the session id, or
     #   nil when no route takes this method and path
@@ -509,7 +509,7 @@ module Samagotchi
     # /recap in an attached TUI: the saved recap, and a new one asked for at
     # once (it arrives as :recap_ready). Answers mid-turn too.
     # 200 {enabled:, saved:, request:, min_user_turns:}. Returns [headers, status, body].
-    def handle_recap(session_id, _body = nil)
+    def handle_recap(_session_id, _body = nil)
       recap = @engine.recap
       return [{}, 200, { enabled: false }] unless recap
 
@@ -519,7 +519,7 @@ module Samagotchi
 
     # Cancel the active turn on this session's engine, if any.
     # Returns [headers, status, body].
-    def handle_cancel(session_id, body)
+    def handle_cancel(_session_id, body)
       # Optional reason from JSON body
       reason = :manual
       if body && !body.strip.empty?
@@ -546,6 +546,7 @@ module Samagotchi
       unless parsed.is_a?(Hash)
         return [{ "Allow" => "POST" }, 400, { error: "invalid_json" }]
       end
+
       qid = fetched(parsed, "id")
       selected = fetched(parsed, "selected")
       freeform = fetched(parsed, "freeform")
@@ -556,6 +557,7 @@ module Samagotchi
       if qid.to_s.strip.empty?
         return [{ "Allow" => "POST" }, 400, { error: "missing_fields", detail: "id required" }]
       end
+
       deadline = fetched(parsed, "deadline")
       return BAD_DEADLINE unless deadline_valid?(deadline)
       return deadline_passed("answer") if expired?("answer_expired", deadline, sid: session_id, id: qid)
@@ -722,6 +724,7 @@ module Samagotchi
         @on_exit_request.call(client_id, **options)
       end
       return deadline_passed("exit request") if reason == :expired
+
       if reason.nil?
         return [{}, 200, { status: "restarting", session_id: @session_id }] if restart
 
@@ -745,7 +748,7 @@ module Samagotchi
     # dropped with 408 deadline_passed. It is checked with the event log held,
     # right before the write, so nothing that holds the log (an exit check)
     # can delay an accepted turn past it. No deadline (an older client): taken.
-    def handle_post_turn(session_id, body)
+    def handle_post_turn(_session_id, body)
       parsed = parse_json(body)
       unless parsed.is_a?(Hash)
         return [{ "Allow" => "POST" }, 400, { error: "invalid_json" }]
@@ -801,7 +804,7 @@ module Samagotchi
 
     # Read-only snapshot surface. AC #4: too-old reconnects re-derive state
     # from here.
-    def handle_state(session_id, _body = nil)
+    def handle_state(_session_id, _body = nil)
       state = @engine.session_state_snapshot
       [{}, 200, { session_id: @session_id, session_state_snapshot: state.merge(event_id: event_id(state[:event_seq])) }]
     end
@@ -809,7 +812,7 @@ module Samagotchi
     # /stats for an attached client: the metrics, with the window and prompt
     # profile asked from the server when no turn has reported them yet (so,
     # unlike /state, it may wait on one short /props GET).
-    def handle_stats(session_id, _body = nil)
+    def handle_stats(_session_id, _body = nil)
       [{}, 200, { session_id: @session_id, metrics: @engine.stats_snapshot }]
     end
 
@@ -817,7 +820,7 @@ module Samagotchi
     # the messages elsewhere (the web server strips and formats them): it then
     # streams from the snapshot's event_seq, and the ring replays what came
     # after (or the stream resets). Returns [headers, status, body].
-    def handle_snapshot(session_id, _body = nil)
+    def handle_snapshot(_session_id, _body = nil)
       [{}, 200, snapshot_frame]
     end
 
@@ -825,7 +828,7 @@ module Samagotchi
     # that turn's answer (a queued turn's page re-reads late). The one
     # handler that gets the query (#dispatch); an older worker's route
     # ignores it and sends the newest answer.
-    def handle_tail(session_id, _body = nil, query = nil)
+    def handle_tail(_session_id, _body = nil, query = nil)
       turn_id = query && URI.decode_www_form(query).to_h["turn_id"].to_s
       [{}, 200, tail_frame(turn_id: turn_id.to_s.empty? ? nil : turn_id)]
     rescue ArgumentError
@@ -885,7 +888,7 @@ module Samagotchi
       headers = {}
       content_length = 0
       while (line = io.gets)
-        break if line == "\r\n" || line == "\n"
+        break if ["\r\n", "\n"].include?(line)
 
         key, value = line.split(":", 2)
         next unless key

data/lib/samagotchi/bridge_client/event_stream.rb

@@ -47,7 +47,7 @@ module Samagotchi
         @client = client
         @rediscover = rediscover
         params = []
-        params << ["snapshot", "1"] if snapshot
+        params << %w[snapshot 1] if snapshot
         params << ["client_id", client_id] if client_id
         @query = params.empty? ? "" : "?#{URI.encode_www_form(params)}"
         @reconnect_delays = reconnect_delays
@@ -138,7 +138,7 @@ module Samagotchi
             @on_event.call(event)
           end
         end
-      rescue EOFError, IOError, SystemCallError
+      rescue IOError, SystemCallError # EOFError is an IOError
         yielded ? :events : :unreachable
       ensure
         detach(sock)

data/lib/samagotchi/bridge_client.rb

@@ -287,7 +287,7 @@ module Samagotchi
           end
           chunk = sock.readpartial(4096)
           yield chunk
-        rescue EOFError, IOError, Errno::ECONNRESET, Errno::ECONNREFUSED
+        rescue IOError, Errno::ECONNRESET, Errno::ECONNREFUSED # EOFError is an IOError
           break
         end
       ensure