samagotchi 0.102.0 → 0.104.0

data/lib/samagotchi/title_command.rb

@@ -3,12 +3,13 @@
 module Samagotchi
   # /title, the session's own name (Session#title): with no argument, the
   # title and what shows without one; `/title TEXT` names the session
-  # (whitespace collapsed, cut at 80 characters as a preview is), saved at
-  # once; `/title --clear` drops it, and the lists show the first message
-  # again. Anyone who may send the session a command may name it (a parent
-  # agent its delegates too). It runs mid-turn as well (anytime): the save
-  # takes the session's lock and writes a running turn's checkpoint, as a
-  # turn's own saves do.
+  # (whitespace collapsed, cut at 80 characters as a preview is);
+  # `/title --clear` drops it, and the lists show the first message again.
+  # Anyone who may send the session a command may name it (a parent agent
+  # its delegates too). It doesn't save: its host saves a changed result at
+  # once (Worker, the REPL). It runs mid-turn as well (anytime): the
+  # worker's save takes the session's lock and writes a running turn's
+  # checkpoint, as a turn's own saves do.
   class TitleCommand
     NAME = "/title"
     CLEAR = "--clear"
@@ -17,10 +18,8 @@ module Samagotchi
     LIMIT = 80
 
     # @param engine [Engine] #session
-    # @param save [#call] saves the session after a change
-    def initialize(engine:, save:)
+    def initialize(engine:)
       @engine = engine
-      @save = save
     end
 
     # @param args [String] what follows /title
@@ -42,9 +41,8 @@ module Samagotchi
       session.title = title
       # The command itself as the session's first line (chi send --new
       # "/title X", a web first message) became its preview: it goes, and
-      # the save derives the preview from the first user message again.
+      # the host's save derives the preview from the first user message again.
       session.first_preview = "" if session.first_preview.to_s.start_with?(NAME)
-      @save.call(session)
       [listing(session), true, :ok]
     end
 

data/lib/samagotchi/tools/task_runtime.rb

@@ -99,9 +99,10 @@ module Samagotchi
       # status. Only while this chi runs. A record a refresh finished in
       # between (failed, no exit code) is corrected; a stopped task (its
       # stop flag on disk) is left as task_stop records it.
-      # The thread is named EXIT_STATUS_THREAD (a spec joins them before it
-      # removes the tasks' folder) here, before it returns: named from
-      # inside, a thread not yet scheduled was missed by that join.
+      # The thread is named EXIT_STATUS_THREAD (a spec joins them with
+      # #quiesce! before it removes the tasks' folder) here, before it
+      # returns: named from inside, a thread not yet scheduled was missed by
+      # that join.
       # @return [Thread]
       def record_exit_status(waiter, path, task_id:)
         # Absolute now: safe against a later chdir.
@@ -132,6 +133,19 @@ module Samagotchi
       # record_exit_status's threads' name.
       EXIT_STATUS_THREAD = "task_exit_status"
 
+      # Joins every live EXIT_STATUS_THREAD, for up to +timeout+ s in all: a
+      # spec calls it (with_task_dir) after stopping its tasks and before it
+      # removes their folder, which a recorder may still write into. chi
+      # itself doesn't call it.
+      # @return [Array<Thread>] the ones still alive after +timeout+ (empty
+      #   when quiet)
+      def quiesce!(timeout: 5)
+        deadline = Process.clock_gettime(Process::CLOCK_MONOTONIC) + timeout
+        threads = Thread.list.select { |thread| thread.name == EXIT_STATUS_THREAD && thread != Thread.current }
+        threads.each { |thread| thread.join([deadline - Process.clock_gettime(Process::CLOCK_MONOTONIC), 0].max) }
+        threads.select(&:alive?)
+      end
+
       # Reasons a refresh finishes a record with when it read no exit code.
       MISSING_CODE_REASONS = ["process_ended_without_exit_code", NOT_CHIS_PROCESS].freeze
 

data/lib/samagotchi/version.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
 module Samagotchi
-  VERSION = "0.102.0"
+  VERSION = "0.104.0"
   # Sent as the User-Agent of every HTTP request chi makes.
   USER_AGENT = "chi/#{VERSION}"
 end