samagotchi 0.87.0 → 0.89.0

data/lib/samagotchi/context_providers.rb

@@ -134,8 +134,7 @@ module Samagotchi
 
       record.scripts.each do |file, sha|
         path = File.join(provenance.scripts_dir, file)
-        recorded = MemoryBundle::Provenance.recorded_sha(sha)
-        next if File.file?(path) && MemoryBundle::Provenance.file_sha(path) == recorded
+        next if File.file?(path) && MemoryBundle::Provenance.sha_matches?(path, sha)
 
         raise Invalid, "bundle #{source.provider}'s scripts/#{file} differs from the installed one (reinstall the bundle)"
       end

data/lib/samagotchi/guardrails/parent_policy.rb

@@ -105,6 +105,13 @@ module Samagotchi
         STRATEGIES.key?(word) ? word : HARD
       end
 
+      # Whether a session's own value is set but not a level (a hand-edited
+      # file: read as hard); the worker says so once when it loads.
+      def unknown_level?(value)
+        word = value.to_s.strip.downcase
+        !word.empty? && word != UNSET && !STRATEGIES.key?(word)
+      end
+
       # What a parent may answer on a session whose own value is
       # +session_value+: config.yml's (+config+) unless the session's is
       # stricter. Config's own object when it binds, so its label stays.
@@ -243,6 +250,19 @@ module Samagotchi
         end
       end
 
+      # Whether a parent may usefully answer +pending+ under +strategy+:
+      # some option it may pick is not a deny (free text alone is a deny
+      # with a reason). A delegate's question it may is reported to the
+      # parent's model; one it may not goes to the parent's user
+      # (Tools::DelegateRelay.relay?). Who may answer what stays #refusal's.
+      def parent_decides?(pending, strategy:)
+        deny = deny_labels(pending)
+        allowed_options(pending, strategy: strategy).any? { |label| !deny.include?(label) }
+      end
+
+      # The deny-tagged options' labels.
+      def deny_labels(pending) = Array(pending.options).zip(effects_of(pending)).filter_map { |label, tag| label.to_s if tag == DENY }
+
       # The question's deny-tagged option's label, or nil.
       def deny_label(pending)
         index = effects_of(pending).index(DENY)

data/lib/samagotchi/hooks/bundle_loader.rb

@@ -126,6 +126,8 @@ module Samagotchi
         # Why the file doesn't match the sha256 recorded at install, or nil.
         # With none recorded, only a required hook counts as a mismatch.
         def sha_mismatch(file, recorded, required:)
+          return nil if MemoryBundle::Provenance.unchecked?(recorded)
+
           expected = MemoryBundle::Provenance.recorded_sha(recorded)
           return (required ? "no sha256 recorded for it (reinstall the bundle)" : nil) if expected.empty?
 

data/lib/samagotchi/installed_gem.rb

@@ -29,6 +29,17 @@ module Samagotchi
       File.file?(path) ? path : nil
     end
 
+    # What Bundler changed in ENV, mapped back to the values from before it
+    # (nil: unset); {} outside Bundler. Child chi commands use this in gem
+    # mode so they load the same installed gem without inheriting this app's
+    # bundle.
+    def unbundled_env_changes(env = ENV.to_h)
+      return {} unless defined?(::Bundler) && ::Bundler.respond_to?(:unbundled_env)
+
+      clean = ::Bundler.unbundled_env
+      (env.keys | clean.keys).each_with_object({}) { |key, changes| changes[key] = clean[key] unless clean[key] == env[key] }
+    end
+
     def real_path(path)
       File.realpath(path)
     rescue SystemCallError

data/lib/samagotchi/memory_bundle/asset_installer.rb

@@ -17,6 +17,8 @@ module Samagotchi
     # scripts/*. Each replaces what an earlier version installed. Installer
     # runs it after the memories; it adds to the installer's results and
     # warnings, and #run returns what the record (Provenance#write) keeps.
+    # A dev install (+dev+, `chi bundle install --dev`) links each file to
+    # the source instead of copying it, and warns of no checksum mismatch.
     class AssetInstaller
       # What was installed: the paths Provenance#write records (each
       # empty / nil on a dry run).
@@ -32,7 +34,7 @@ module Samagotchi
       # @param results, warnings, conflicts the Installer's (results and
       #   warnings are added to)
       def initialize(name:, source_dir:, manifest:, provenance:, existing:, results:, warnings:, conflicts:,
-                     force: false, strict: true, upgrade: false, dry_run: false)
+                     force: false, strict: true, upgrade: false, dry_run: false, dev: false)
         @name = name
         @bundle_dir = source_dir
         @manifest = manifest
@@ -45,6 +47,7 @@ module Samagotchi
         @strict = strict
         @upgrade = upgrade
         @dry_run = dry_run
+        @dev = dev
       end
 
       # @return [Installed]
@@ -52,8 +55,8 @@ module Samagotchi
       #   script the bundle doesn't have
       def run
         install_hooks
-        prune_dropped_hooks if @upgrade && @existing && !@dry_run
-        verify_hook_checksums if @manifest && @strict
+        prune_dropped_hooks if (@upgrade || (@dev && @existing&.dev)) && @existing && !@dry_run
+        verify_hook_checksums if @manifest && @strict && !@dev
         incoming_rules = Dir.glob(File.join(@bundle_dir, "guardrails", "*.yml")).sort
                             .reject { |src| linked_out?(src, "guardrails/#{File.basename(src)}") }
         guardrail_files = install_guardrails(incoming_rules)
@@ -109,7 +112,7 @@ module Samagotchi
 
         FileUtils.mkdir_p(@provenance.hooks_dir)
         unchanged = File.exist?(dest) && FileUtils.identical?(src, dest)
-        FileUtils.cp(src, dest)
+        place(src, dest)
         @results[basename] = if unchanged
                                { status: "skipped", reason: "already up to date" }
                              elsif @upgrade && @existing
@@ -132,7 +135,7 @@ module Samagotchi
           else
             FileUtils.mkdir_p(@provenance.hooks_dir)
             dest = File.join(@provenance.hooks_dir, basename)
-            FileUtils.cp(src, dest)
+            place(src, dest)
             @results[basename] = { status: "installed" }
             @hook_files[basename] = dest
           end
@@ -140,13 +143,13 @@ module Samagotchi
       end
 
       # An upgrade removes the hooks the previous version shipped and this
-      # one doesn't (a local file too, with a warning).
+      # one doesn't (a local file too, with a warning; a dev link silently).
       def prune_dropped_hooks
         @existing.hooks.each_key do |old_key_str|
           next if @bundle_hooks.include?(old_key_str)
 
           old_dest = File.join(@provenance.hooks_dir, old_key_str)
-          if File.exist?(old_dest) && !@force
+          if File.exist?(old_dest) && !@force && !File.symlink?(old_dest)
             @warnings << "Bundle no longer includes hook #{old_key_str} but local file exists — removing"
           end
           FileUtils.rm_f(old_dest)
@@ -193,7 +196,7 @@ module Samagotchi
         FileUtils.mkdir_p(target)
         incoming_rules.to_h do |src|
           dest = File.join(target, File.basename(src))
-          FileUtils.cp(src, dest)
+          place(src, dest)
           @results["guardrails/#{File.basename(src)}"] =
             unchanged.include?(src) ? { status: "skipped", reason: "already up to date" } : { status: "installed" }
           [File.basename(src), dest]
@@ -227,7 +230,7 @@ module Samagotchi
         unchanged = File.file?(dest) && FileUtils.identical?(src, dest)
         FileUtils.rm_rf(@provenance.plugin_dir)
         FileUtils.mkdir_p(@provenance.plugin_dir)
-        FileUtils.cp(src, dest)
+        place(src, dest)
         @results[plugin[:file]] = if unchanged
                                     { status: "skipped", reason: "already up to date" }
                                   elsif @upgrade && @provenance.installed?
@@ -237,7 +240,7 @@ module Samagotchi
                                   end
         expected = @manifest.checksum_for_plugin
         actual = Digest::SHA256.hexdigest(File.binread(dest))
-        if @strict && expected && actual != expected
+        if @strict && !@dev && expected && actual != expected
           @warnings << "Checksum mismatch for plugin #{plugin[:file]}: expected #{expected[0..7]}..., got #{actual[0..7]}..."
         end
         dest
@@ -253,6 +256,8 @@ module Samagotchi
           src = File.join(@bundle_dir, "scripts", file)
           raise Installer::InstallError, "the manifest names scripts/#{file}, which the bundle doesn't have" unless File.file?(src)
           raise Installer::InstallError, "scripts/#{file} is a symlink (not installed)" if linked?(src)
+
+          @warnings << "scripts/#{file} is not executable; chmod +x #{src}" if @dev && !File.executable?(src)
         end
         return {} if @dry_run
 
@@ -266,21 +271,28 @@ module Samagotchi
         FileUtils.mkdir_p(@provenance.scripts_dir)
         scripts.to_h do |file, declared|
           dest = File.join(@provenance.scripts_dir, file)
-          FileUtils.cp(File.join(@bundle_dir, "scripts", file), dest)
+          place(File.join(@bundle_dir, "scripts", file), dest)
           @results["scripts/#{file}"] = if unchanged.include?(file) then { status: "skipped", reason: "already up to date" }
                                         elsif @upgrade && @provenance.installed? then { status: "updated" }
                                         else { status: "installed" }
                                         end
-          FileUtils.chmod(0o755, dest)
+          FileUtils.chmod(0o755, dest) unless @dev
           actual = Digest::SHA256.hexdigest(File.binread(dest))
           expected = declared.to_s.delete_prefix("sha256:")
-          if @strict && !expected.empty? && actual != expected
+          if @strict && !@dev && !expected.empty? && actual != expected
             @warnings << "Checksum mismatch for script #{file}: expected #{expected[0..7]}..., got #{actual[0..7]}..."
           end
           [file, dest]
         end
       end
 
+      # Copies +src+ to +dest+, or (a dev install) links +dest+ to it. An
+      # existing link goes first: a copy over it would write into the source.
+      def place(src, dest)
+        FileUtils.rm_f(dest) if File.symlink?(dest)
+        @dev ? File.symlink(File.realpath(src), dest) : FileUtils.cp(src, dest)
+      end
+
       # Whether the bundle file +src+ reads through a symlink: it is one, or
       # it (or its dir, hooks/ say) resolves outside the bundle dir. A tar,
       # zip or git repo keeps links; copying through one would install any

data/lib/samagotchi/memory_bundle/installed_bundle.rb

@@ -22,13 +22,15 @@ module Samagotchi
     #   includes    a profile's recorded members, nil for a plain bundle
     #   session_settings
     #               the keys a session may set (chi --set), [] when none
+    #   dev         a dev install (`chi bundle install --dev`): its source is
+    #               the folder it links to and its files carry Provenance::DEV_SHA
     #   error       why manifest.json couldn't be read (the rest empty), else nil
     class InstalledBundle < Data.define(:name, :version, :scope, :source, :installed_at, :trust_level, :source_commit,
                                         :requires_chi, :files, :hooks, :guardrails, :scripts, :plugin, :needs,
-                                        :includes, :context_providers, :session_settings, :error)
+                                        :includes, :context_providers, :session_settings, :dev, :error)
       OPTIONAL = { version: nil, scope: nil, source: nil, installed_at: nil, trust_level: nil, source_commit: nil,
                    requires_chi: nil, files: {}, hooks: {}, guardrails: {}, scripts: {}, plugin: nil, needs: nil,
-                   includes: nil, context_providers: nil, session_settings: [], error: nil }.freeze
+                   includes: nil, context_providers: nil, session_settings: [], dev: false, error: nil }.freeze
 
       def initialize(name:, **fields)
         super(name: name, **OPTIONAL, **fields)
@@ -60,7 +62,8 @@ module Samagotchi
             scripts: mapping(h["scripts"]) { |meta| sha(meta) },
             plugin: PluginRef.parse(h["plugin"]), needs: h["needs"], context_providers: h["context_providers"],
             includes: h["includes"].is_a?(Array) ? h["includes"].map(&:to_s) : nil,
-            session_settings: h["session_settings"].is_a?(Array) ? h["session_settings"].map(&:to_s) : [])
+            session_settings: h["session_settings"].is_a?(Array) ? h["session_settings"].map(&:to_s) : [],
+            dev: h["dev"] == true)
       end
 
       def self.string(value) = value&.to_s

data/lib/samagotchi/memory_bundle/installer.rb

@@ -40,7 +40,7 @@ module Samagotchi
       # installs from a dir in it only, unless force); ShippedUpdate,
       # Profile and SystemBundle pass the one they install from.
       def initialize(source:, name:, scope: nil, force: false, strict: true, upgrade: false, dry_run: false, prepared: nil,
-                     shipped_dir: SourceNormalizer::SHIPPED_DIR)
+                     shipped_dir: SourceNormalizer::SHIPPED_DIR, dev: false)
         @source = source
         @shipped_dir = shipped_dir
         @prepared = prepared
@@ -50,6 +50,7 @@ module Samagotchi
         @strict = strict
         @upgrade = upgrade
         @dry_run = dry_run
+        @dev = dev
         @results = {}
         @warnings = []
         @placeholder_warnings = []
@@ -98,7 +99,7 @@ module Samagotchi
         installed = AssetInstaller.new(
           name: @name, source_dir: @bundle_dir, manifest: @manifest, provenance: @provenance, existing: @existing,
           results: @results, warnings: @warnings, conflicts: @conflicts,
-          force: @force, strict: @strict, upgrade: @upgrade, dry_run: @dry_run
+          force: @force, strict: @strict, upgrade: @upgrade, dry_run: @dry_run, dev: @dev
         ).run
 
         # Files the previous version shipped and this one doesn't.
@@ -172,10 +173,14 @@ module Samagotchi
       def check_source
         refuse_case_twin
         refuse_shipped_name
+        refuse_dev_mismatch
         now = incoming_identity
         return if now.nil?
 
         was = @existing && SourceIdentity.of(@existing.source)
+        if @dev && @existing&.dev && was != now
+          raise InstallError, "#{@name} is a dev install of #{@existing.source}; uninstall it before installing from #{@source}"
+        end
         return if was.nil? || was == now
         return @source_change = [@existing.source, @source.to_s] if @upgrade
 
@@ -191,6 +196,19 @@ module Samagotchi
         raise InstallError, "#{@name} differs only in case from the installed #{twin}" if twin
       end
 
+      # A dev install (--dev, linked to its source) and a normal one don't
+      # replace each other: neither an upgrade nor a plain install over a
+      # dev bundle, nor --dev over a normal one.
+      def refuse_dev_mismatch
+        return if @existing.nil? || @existing.dev == @dev
+
+        if @existing.dev
+          raise InstallError, "#{@name} is a dev install of #{@existing.source}: it follows its source already " \
+                              "(to leave dev mode: chi bundle uninstall #{@name}, then chi bundle install without --dev)"
+        end
+        raise InstallError, "#{@name} is installed normally; `chi bundle uninstall #{@name}` first, then chi bundle install --dev"
+      end
+
       # A shipped name (in any case) from anything but this chi's own
       # shipped dir. A recorded source may be an older gem's (SourceIdentity
       # matches the path); an incoming one must be a dir in SHIPPED_DIR, so
@@ -421,7 +439,8 @@ module Samagotchi
           scripts_files: installed.script_files,
           context_providers: @manifest.context_providers,
           file_descriptions: @manifest.file_descriptions,
-          session_settings: @manifest.session_settings
+          session_settings: @manifest.session_settings,
+          dev: @dev
         )
       end
 

data/lib/samagotchi/memory_bundle/provenance.rb

@@ -116,9 +116,20 @@ module Samagotchi
         Digest::SHA256.hexdigest(File.binread(path))
       end
 
-      # Whether the file's content is the one recorded (either format).
+      # What a dev install (`chi bundle install --dev`) records instead of
+      # a sha256: its files are links to the source, which change under it.
+      # Not a hex digest, so no file can be mistaken for it.
+      DEV_SHA = "sha256:dev"
+
+      # Whether +recorded+ is DEV_SHA: the file isn't checked.
+      def self.unchecked?(recorded)
+        recorded_sha(recorded) == recorded_sha(DEV_SHA)
+      end
+
+      # Whether the file's content is the one recorded (either format); a
+      # dev install's (DEV_SHA) always is.
       def self.sha_matches?(path, recorded)
-        file_sha(path) == recorded_sha(recorded)
+        unchecked?(recorded) || file_sha(path) == recorded_sha(recorded)
       end
 
       # Writes provenance for an installation: merges into existing manifest.json
@@ -139,9 +150,11 @@ module Samagotchi
       # @param scripts_files [Hash] file name => installed script; the
       #   sha256 of each is recorded (ContextFetch checks it before a run)
       # @param context_providers [Array<ContextProviders::Provider>]
+      # @param dev [Boolean] a dev install: its files are links to the
+      #   source, recorded as DEV_SHA (no sha256 check) with "dev": true
       def write(files:, scope:, version:, source_path:, hooks: {}, trust_level: nil, source_commit: nil, hooks_files: {},
                 guardrails_files: {}, plugin_file: nil, requires_chi: nil, needs: nil, conflicts: [], includes: nil,
-                scripts_files: {}, context_providers: [], file_descriptions: {}, session_settings: [])
+                scripts_files: {}, context_providers: [], file_descriptions: {}, session_settings: [], dev: false)
         FileUtils.mkdir_p(@bundle_dir)
         bases_dir = File.join(@bundle_dir, "bases")
         FileUtils.mkdir_p(bases_dir)
@@ -196,6 +209,7 @@ module Samagotchi
         normalized_hooks.each do |k, hook|
           src = hooks_files[k] || hooks_files[k.to_sym]
           hook = hook.with(sha256: BundleHook.of_file(src.to_s).sha256) if src && File.exist?(src.to_s)
+          hook = hook.with(sha256: DEV_SHA) if dev
           hooks_map[k] = hook.to_record
           next unless src && File.exist?(src.to_s)
 
@@ -216,9 +230,10 @@ module Samagotchi
           "hooks" => hooks_map
         }
         manifest_data["trust_level"] = trust_level.to_s if trust_level && !trust_level.to_s.empty?
+        manifest_data["dev"] = true if dev
         if guardrails_files.is_a?(Hash) && !guardrails_files.empty?
           manifest_data["guardrails"] = guardrails_files.sort.to_h do |basename, path|
-            [basename.to_s, { "sha256" => "sha256:#{Digest::SHA256.hexdigest(File.read(path.to_s))}" }]
+            [basename.to_s, { "sha256" => dev ? DEV_SHA : "sha256:#{Digest::SHA256.hexdigest(File.read(path.to_s))}" }]
           end
         end
         manifest_data["source_commit"] = source_commit.to_s if source_commit && !source_commit.to_s.empty?
@@ -226,7 +241,7 @@ module Samagotchi
         if plugin_file
           content = File.binread(plugin_file.to_s)
           manifest_data["plugin"] = { "file" => File.basename(plugin_file.to_s),
-                                      "sha256" => "sha256:#{Digest::SHA256.hexdigest(content)}" }
+                                      "sha256" => dev ? DEV_SHA : "sha256:#{Digest::SHA256.hexdigest(content)}" }
           FileUtils.mkdir_p(File.join(bases_dir, "plugin"))
           File.binwrite(plugin_base_path(File.basename(plugin_file.to_s)), content)
         end
@@ -235,7 +250,7 @@ module Samagotchi
         manifest_data["includes"] = includes.map(&:to_s) if includes.is_a?(Array)
         unless scripts_files.empty?
           manifest_data["scripts"] = scripts_files.sort.to_h do |file, path|
-            [file.to_s, { "sha256" => "sha256:#{Digest::SHA256.hexdigest(File.binread(path.to_s))}" }]
+            [file.to_s, { "sha256" => dev ? DEV_SHA : "sha256:#{Digest::SHA256.hexdigest(File.binread(path.to_s))}" }]
           end
         end
         manifest_data["context_providers"] = context_providers.map(&:to_h) unless context_providers.empty?

data/lib/samagotchi/relay.rb

@@ -16,6 +16,8 @@ module Samagotchi
     # How much of the child's task the card keeps, and the question line.
     TASK_CHARS = 200
     TASK_LINE_CHARS = 80
+    # A hook's question (QuestionDesk#ask_for_hook), relayed as such (P4).
+    HOOK_KIND = "hook"
 
     module_function
 
@@ -33,6 +35,14 @@ module Samagotchi
       asked = inner ? inner[:asked].to_s : pending.question.to_s
       task = inner ? inner[:task].to_s : task_of(child)
       who = chain.reverse.join(" → ")
+      relay = { id: relay_id, child_id: child.id, child_short: short, child_question_id: pending.id.to_s,
+                task: task, chain: chain, asked: asked, more: more.to_i }
+      if pending.kind.to_s == HOOK_KIND
+        # The asking hook's own header, kept down a chain (a relay of a relay).
+        relay[:header] = inner ? inner[:header].to_s : pending.header.to_s
+        return hook_card(pending, who, more, relay)
+      end
+
       {
         question: question_text(who, task, asked),
         options: relabel(Array(pending.options), facts[:scopes], who),
@@ -43,11 +53,33 @@ module Samagotchi
         **{ effects: pending.effects, user_only: pending.user_only }.compact,
         kind: Guardrails::Approval::KIND,
         approval: facts,
-        relay: { id: relay_id, child_id: child.id, child_short: short, child_question_id: pending.id.to_s,
-                 task: task, chain: chain, asked: asked, more: more.to_i }
+        relay: relay
       }
     end
 
+    # A hook's question (P4: one its parent can't usefully answer): its
+    # kind, hook, options and effects as the child asked them, and no
+    # approval facts (ParentApprovals.approval? stays false on it, so the
+    # policy judges it by its own tags, as on the child).
+    def hook_card(pending, who, more, relay)
+      {
+        question: question_text(who, relay[:task], relay[:asked]),
+        options: Array(pending.options).map(&:to_s),
+        header: hook_header(who, relay[:header], more),
+        multi_select: !!pending.multi_select,
+        allow_freeform: !!pending.allow_freeform,
+        **{ effects: pending.effects, user_only: pending.user_only, hook: pending.hook }.compact,
+        kind: HOOK_KIND,
+        relay: relay
+      }
+    end
+
+    # delegate ab12 asks: Cost cap (+1 more delegate waiting)
+    def hook_header(who, header, more)
+      text = "delegate #{who} asks: #{header.to_s.strip.empty? ? "a question" : header}"
+      more.to_i.positive? ? "#{text} (+#{more} more delegate#{"s" if more.to_i > 1} waiting)" : text
+    end
+
     # delegate ab12 ("fix the flaky spec in …") asks:
     #   execute: git push origin main
     #     in /path/repo (repo samagotchi, branch main)

data/lib/samagotchi/relay_desk.rb

@@ -63,6 +63,19 @@ module Samagotchi
       end
     end
 
+    # Whether the parent's user (or a parent agent) answered or dismissed a
+    # relay of this child's question: it isn't relayed again (a refused
+    # answer leaves it on the child's card). A closed one (a Stop, the
+    # child gone) is not settled. Forgotten after KEEP_SECONDS or a restart.
+    def settled?(child_id, child_question_id)
+      @mutex.synchronize do
+        prune
+        @relays.each_value.any? do |relay|
+          relay[:state] == "answered" && relay[:child_id] == child_id.to_s && relay[:child_question_id] == child_question_id.to_s
+        end
+      end
+    end
+
     # Whether a relay is open, or settled recently enough that its child
     # may still ask about it: a restart would drop them.
     def active?

data/lib/samagotchi/session_manager.rb

@@ -263,21 +263,13 @@ module Samagotchi
       # A gem-mode worker activates the newest installed chi itself: under
       # `bundle exec` (a Gemfile that has samagotchi) Bundler's RUBYOPT would
       # hold it to the bundle's version, and its badge would never clear.
-      child_env.merge!(unbundled_env_changes) if InstalledGem.spec
+      child_env.merge!(InstalledGem.unbundled_env_changes) if InstalledGem.spec
       opts[:env] = child_env unless child_env.empty?
       opts
     end
 
-    # What Bundler changed in ENV (RUBYOPT, BUNDLE_*), mapped back to the
-    # values from before it (nil: unset); {} outside Bundler.
-    private_class_method def self.unbundled_env_changes(env = ENV.to_h)
-      return {} unless defined?(::Bundler) && ::Bundler.respond_to?(:unbundled_env)
-
-      clean = ::Bundler.unbundled_env
-      (env.keys | clean.keys).each_with_object({}) { |key, changes| changes[key] = clean[key] unless clean[key] == env[key] }
-    end
-
     GUARDRAILS_ENV_PREFIX = "SAMAGOTCHI_GUARDRAILS_"
+    WORKER_ENV_PREFIX = "SAMAGOTCHI_WORKER_"
     # Tools::Builtins.parent_env's (Tools::TaskRuntime::MARKER_ENV_KEYS).
     SESSION_MARKER_ENV = %w[SAMAGOTCHI_PARENT_SESSION SAMAGOTCHI_SESSION_MODEL SAMAGOTCHI_SESSION_THINKING
                             SAMAGOTCHI_SESSION_REASONING_HISTORY].freeze

data/lib/samagotchi/terminal_ui/attach_launcher.rb

@@ -195,7 +195,7 @@ module Samagotchi
 
         verb = flags.size == 1 ? "applies" : "apply"
         names = flags.size > 2 ? "#{flags[0..-2].join(", ")} and #{flags.last}" : flags.join(" and ")
-        err.puts "(#{names} #{verb} to a new session; #{session_id}'s prompt is already built)"
+        err.puts "(#{names} #{verb} to a new session; #{session_id} keeps its own)"
       end
 
       # @return [BridgeClient, nil] the running worker's, or nil when none runs

data/lib/samagotchi/terminal_ui/question_prompt.rb

@@ -96,7 +96,8 @@ module Samagotchi
         case reason.to_s
         when "answered_on_child" then "(answered in #{@delegate})"
         when "child_gone" then "(#{@delegate}'s worker is gone)"
-        when "dismissed" then "(denied)"
+        # A relayed hook question's dismissal: the hook decides what it means.
+        when "dismissed" then approval? ? "(denied)" : "(dismissed)"
         else "(left open in #{@delegate})"
         end
       end

data/lib/samagotchi/terminal_ui.rb

@@ -737,7 +737,7 @@ module Samagotchi
       return if @toolkit_flags.empty?
 
       names = @toolkit_flags.size > 1 ? "#{@toolkit_flags[0..-2].join(", ")} and #{@toolkit_flags.last}" : @toolkit_flags.first
-      warn "(#{names} #{@toolkit_flags.size == 1 ? "applies" : "apply"} to a new session; #{session.id}'s prompt is already built)"
+      warn "(#{names} #{@toolkit_flags.size == 1 ? "applies" : "apply"} to a new session; #{session.id} keeps its own)"
     end
     private :warn_toolkit_flags_ignored