samagotchi 0.78.0 → 0.79.0

data/lib/samagotchi/llm/anthropic_wire.rb

@@ -32,8 +32,10 @@ module Samagotchi
       TOOL_ID_INVALID = /[^a-zA-Z0-9_-]/
       # What Messages takes as a tool name.
       TOOL_NAME = /\A[a-zA-Z0-9_-]{1,128}\z/
+      # The content block types a model's thinking is replayed in.
+      THINKING_TYPES = %w[thinking redacted_thinking].freeze
       # Blocks that can't carry a cache_control breakpoint.
-      UNMARKABLE = %w[thinking redacted_thinking].freeze
+      UNMARKABLE = THINKING_TYPES
 
       # A request's system blocks and messages.
       Translated = Data.define(:system, :messages)
@@ -105,6 +107,10 @@ module Samagotchi
         end
       end
 
+      # Whether +block+ is a thinking or redacted_thinking block (a Hash,
+      # string or symbol keys).
+      def thinking_block?(block) = block.is_a?(Hash) && THINKING_TYPES.include?(type_of(block))
+
       # +id+ as Messages takes it (^[a-zA-Z0-9_-]+$), the same in tool_use
       # and tool_result: a valid id as it is, another with its invalid
       # characters replaced and a short hash of it appended (so "a.b" and

data/lib/samagotchi/llm/chat_loop.rb

@@ -342,15 +342,6 @@ module Samagotchi
         nil
       end
 
-      # The shape the engine persists: role and content (parts stay an
-      # Array), plus a model turn's tool_calls and thinking (the host's
-      # reasoning, sent back as #wire_messages says), a result's tool_call_id, the image
-      # refs of a user message or a tool result, a prompt's turn_id (Engine,
-      # never sent) and a wake turn's report's turn_start (Steer, never sent), and a plugin tool result's tool_params and tool_labels
-      # (the live row's params line and label, never sent back), an
-      # edit/write result's tool_diffs (never sent), an empty-answer
-      # note's marker (TurnNote.empty, never sent), and an answer's display
-      # and display_refs (AnswerDisplay, AnswerRefs; never sent).
       # Removes the replayed thinking blocks a response's ThinkingDrop
       # names (AnthropicWire.block_identity) from the model entries'
       # anthropic_content: the edit the provider made for that request, so
@@ -375,6 +366,15 @@ module Samagotchi
         removed
       end
 
+      # The shape the engine persists: role and content (parts stay an
+      # Array), plus a model turn's tool_calls and thinking (the host's
+      # reasoning, sent back as #wire_messages says), a result's tool_call_id, the image
+      # refs of a user message or a tool result, a prompt's turn_id (Engine,
+      # never sent) and a wake turn's report's turn_start (Steer, never sent), and a plugin tool result's tool_params and tool_labels
+      # (the live row's params line and label, never sent back), an
+      # edit/write result's tool_diffs (never sent), an empty-answer
+      # note's marker (TurnNote.empty, never sent), and an answer's display
+      # and display_refs (AnswerDisplay, AnswerRefs; never sent).
       def plain(conversation)
         conversation.map do |entry|
           content = entry[:content].is_a?(Array) ? entry[:content] : entry[:content].to_s
@@ -471,8 +471,6 @@ module Samagotchi
 
       def thought?(entry) = entry[:thinking].is_a?(String) && !entry[:thinking].strip.empty?
 
-      THINKING_BLOCKS = %w[thinking redacted_thinking].freeze
-
       # A model entry's saved blocks (anthropic_content) to send back
       # verbatim, for an adapter that replays them: nil when there are none,
       # for an interrupted entry (its partial blocks have no signature), and
@@ -485,7 +483,7 @@ module Samagotchi
         return nil unless replays_content? && blocks.is_a?(Array) && !blocks.empty? && !entry[:interrupted]
         return nil unless Array(entry[:tool_calls]).all? { |call| paired.include?(call[:id]) }
 
-        blocks = blocks.reject { |block| block.is_a?(Hash) && THINKING_BLOCKS.include?((block["type"] || block[:type]).to_s) } unless keep_thinking
+        blocks = blocks.reject { |block| AnthropicWire.thinking_block?(block) } unless keep_thinking
         blocks.empty? ? nil : blocks
       end
 
@@ -519,6 +517,17 @@ module Samagotchi
           # The empty generations, never saved: the UIs draw the turn's
           # steps from them (ModelResult#empty_steps).
           @empty_steps = []
+          # Set as the run goes: the last response's refusal, the context
+          # window the last request was sized for, that request's text and
+          # image estimates, whether a refused reasoning is being asked
+          # again without it, and whether a preserved-thinking recovery
+          # settled nothing once already.
+          @refusal = nil
+          @window = nil
+          @request_chars = nil
+          @request_image_tokens = nil
+          @without_reasoning = false
+          @recovery_repeated = false
           @context = ContextStatus.new(conversation: conversation, llm_context: loop.llm_context)
           loop.llm_context_turn = LLMContextForget::Turn.new(conversation: conversation, context: @context,
                                                              emit: method(:emit))

data/lib/samagotchi/llm/model_listing.rb

@@ -1,5 +1,7 @@
 # frozen_string_literal: true
 
+require_relative "../log"
+
 module Samagotchi
   module LLM
     # A model a host lists. Unknown fields are nil.
@@ -51,7 +53,10 @@ module Samagotchi
 
       def init_model_listing(models_ttl)
         @models_ttl = models_ttl
+        @models_failure_ttl = [models_ttl, 60].min
         @models_mutex = Mutex.new
+        @models_failure_at = nil
+        @models_failure_logged_at = nil
       end
 
       # +model+'s entry in the cached list: its exact id, else any case.
@@ -93,16 +98,39 @@ module Samagotchi
       end
 
       def cached_models
+        now = monotonic
         @models_mutex.synchronize do
-          fresh = @models_at && (monotonic - @models_at) < @models_ttl
+          fresh = @models_at && (now - @models_at) < @models_ttl
           return @models if fresh
+
+          failed_recently = @models_failure_at && (now - @models_failure_at) < @models_failure_ttl
+          return [] if failed_recently
         end
         models = list_models
         @models_mutex.synchronize do
           @models = models
           @models_at = monotonic
+          @models_failure_at = nil
         end
         models
+      rescue StandardError => e
+        @models_mutex.synchronize { @models_failure_at = monotonic }
+        log_model_listing_failure(e)
+        raise
+      end
+
+      def log_model_listing_failure(error)
+        now = monotonic
+        should_log = @models_mutex.synchronize do
+          next false if @models_failure_logged_at && (now - @models_failure_logged_at) < @models_ttl
+
+          @models_failure_logged_at = now
+          true
+        end
+        return unless should_log
+
+        host = @host_name.to_s.empty? ? @base_url : @host_name
+        Log.warn(:model, "list_failed", host: host, error: error.class.name, msg: error.message.to_s[0, 500])
       end
 
       def monotonic = Process.clock_gettime(Process::CLOCK_MONOTONIC)

data/lib/samagotchi/llm/refusal_memo.rb

@@ -6,8 +6,9 @@ module Samagotchi
     # host's, HostRegistry keeps one per worker; ChatLoop: the worker's): a
     # +kind+ (a Symbol) for one model, or for the whole host when the model
     # is nil, with a value (true, or what the refusal named: an output cap).
-    # The turn, side-ask and recap threads share a holder, so every read and
-    # write takes the lock.
+    # The turn thread and the model-list reads share HostRegistry's adapter
+    # (an IdleClient's adapter is its own), so every read and write takes the
+    # lock.
     class RefusalMemo
       def initialize
         @mutex = Mutex.new

data/lib/samagotchi/question_desk.rb

@@ -349,7 +349,7 @@ module Samagotchi
         @pending = pending = pending.with(relayed_to: relayed_to)
         if session
           session.pending_question = pending
-          begin; session.save(state_dir: state_dir); rescue StandardError; nil; end
+          save_session(session)
         end
         pending.id
       end
@@ -439,7 +439,7 @@ module Samagotchi
       end
       if session
         session.pending_question = pending
-        begin; session.save(state_dir: state_dir); rescue StandardError; nil; end
+        save_session(session)
       end
       event = { type: :question_requested, pending_question: pending.to_h }
       event[:standing] = true if standing
@@ -559,12 +559,19 @@ module Samagotchi
     def cancel_controller = @cancel_controller_lookup.call
     def emit(event) = @emit.call(event)
 
+    def save_session(session)
+      session.save(state_dir: state_dir)
+    rescue StandardError => e
+      Log.warn(:turn, "question_save_failed", sid: session.id, error: e.class.name, msg: e.message.to_s[0, 200])
+      nil
+    end
+
     # The session file no longer holds a pending question.
     def clear_saved_question
       return unless session
 
       session.pending_question = nil
-      begin; session.save(state_dir: state_dir); rescue StandardError; nil; end
+      save_session(session)
     end
   end
 end

data/lib/samagotchi/reasoning_history.rb

@@ -2,6 +2,7 @@
 
 require_relative "config"
 require_relative "setting_explained"
+require_relative "llm/anthropic_wire"
 
 module Samagotchi
   # Which of its own reasoning a chat host's model gets back
@@ -125,7 +126,7 @@ module Samagotchi
         next false unless entry[:role].to_s == "model"
 
         (entry[:thinking].is_a?(String) && !entry[:thinking].strip.empty?) ||
-          Array(entry[:anthropic_content]).any? { |block| block.is_a?(Hash) && %w[thinking redacted_thinking].include?(block["type"].to_s) }
+          Array(entry[:anthropic_content]).any? { |block| LLM::AnthropicWire.thinking_block?(block) }
       end
     end
 

data/lib/samagotchi/version.rb

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

data/lib/samagotchi/web/public/app.js

@@ -56,7 +56,9 @@ import { actionLine, annotationActions, composerWithAction } from "./annotate_ac
 import { copyText, installCopy } from "./copy.js";
 import { createQuestionCards } from "./question_cards.js";
 import { splitSnapshotCards } from "./card.js";
-import { GRIP_OPEN_PX, PROMPT_LINE_PX, PROMPT_MIN_PX, gripFloor, gripFolds, promptHeight } from "./composer_size.js";
+import { PROMPT_LINE_PX, PROMPT_MIN_PX, promptHeight } from "./composer_size.js";
+import { GRIP_KEY, createComposerGrip } from "./composer_grip.js";
+import { readStored, writeStored } from "./storage.js";
 import { createCommandList } from "./command_list.js";
 import { createPromptHistory } from "./prompt_history.js";
 import { ALL_SESSIONS_HASH, sessionHash, sessionIdFromHash } from "./route.js";
@@ -64,7 +66,6 @@ import { scopeDir, scopeTemplate } from "./scope.js";
 import { html, nothing, render } from "./vendor/lit-html.js";
 import { easeScrollTo, FOLLOW_MS, shouldFollowScroll } from "./scroll.js";
 import { SHORT_WINDOW_QUERY, stripAutoHidden } from "./strip.js";
-import { COMPOSER_FOLD_KEY, composerFolded, foldHandle, opensOnKey } from "./composer_fold.js";
 import { waitingOn } from "./notify.js";
 import { applySessionEvent, inTree, sortedByUpdated } from "./sessions_list.js";
 import { createNotifications } from "./notifications.js";
@@ -204,19 +205,7 @@ let pageEditorUrl = null;
 const promptEl = $("#prompt");
 const actionBtn = $("#actionBtn");
 const cancelBtn = $("#cancelBtn");
-// The composer's fold (composer_fold.js): the grip, which is the handle
-// while folded, and the info bar's Stop turn while folded (the composer's
-// #cancelBtn is folded away then). +gripWillFold+: the drag in progress is
-// past the fold threshold.
-const composeGrip = $("#composeGrip");
-const gripStartTitle = composeGrip.title;
-let gripWillFold = false;
 const infoCancelBtn = $("#infoCancelBtn");
-// This visit's state (a visit is one select() call): opened by the page
-// (a focus, a restored line), archived (decided once, from the visit's
-// first getSession answer).
-const newFoldVisit = () => ({ open: false, archived: false, decided: false });
-let foldVisit = newFoldVisit();
 const modelPickEl = $("#modelPick");
 const composeHintEl = $("#composeHint");
 
@@ -754,8 +743,7 @@ function clearSelection() {
   closeStream();
   setMarkdownWarning(null);
   setCapabilities(null, null);
-  foldVisit = newFoldVisit();
-  applyComposerFolded();
+  composerGrip.newVisit();
   updateComposerMode();
   updateInfoBar();
   focusPrompt();
@@ -791,8 +779,7 @@ async function select(id) {
   closeStream();
   resetSessionState();
   resetTurnView();
-  foldVisit = newFoldVisit();
-  applyComposerFolded();
+  composerGrip.newVisit();
   updateInfoBar();
   contextChips.load(id);
   updateComposerMode();
@@ -837,11 +824,7 @@ async function resync({ endText = null } = {}) {
     if (id !== selected) return;
     // An archived session opens folded: decided once per visit, from its
     // first answer (the list may not hold it yet, or ever under ?dir).
-    if (!foldVisit.decided) {
-      foldVisit.decided = true;
-      foldVisit.archived = !!data.session?.archived;
-      applyComposerFolded();
-    }
+    composerGrip.decideVisit(data.session?.archived);
     const kept = keptCommandBubbles(historyEl, id);
     resetTurnView();
     renderSessionView(data, { kept });
@@ -1646,9 +1629,7 @@ function syncModelPick() {
 }
 // Leaving a session: the start page's remembered model again (no fetch).
 function restoreStartPick() {
-  let stored = null;
-  try { stored = localStorage.getItem(MODEL_KEY); } catch (_) {}
-  modelPicker.restore(stored);
+  modelPicker.restore(readStored(MODEL_KEY));
   renderNewChatLlmContext();
   renderNewChatThinking();
 }
@@ -1659,8 +1640,7 @@ async function loadModels() {
   } catch (_) {
     return;
   }
-  let stored = null;
-  try { stored = localStorage.getItem(MODEL_KEY); } catch (_) {}
+  const stored = readStored(MODEL_KEY);
   if (!modelPicker.setModels(payload, stored)) return;
   startPickTitle = payload.warning ? `${START_PICK_TITLE} · ${payload.warning}` : START_PICK_TITLE;
   modelPickEl.hidden = false;
@@ -1716,9 +1696,7 @@ chipsEl.addEventListener("click", (e) => {
   renderChips();
 });
 // ── Composer height: grows with the text; the top-edge grip sets a floor ───
-const GRIP_KEY = "chi_composer_floor";
-let promptFloor = null;
-try { promptFloor = Number(localStorage.getItem(GRIP_KEY)) || null; } catch (_) {}
+let promptFloor = Number(readStored(GRIP_KEY)) || null;
 
 // The empty box: one line in a session, three on the start page.
 function promptMinPx() {
@@ -1733,95 +1711,6 @@ function fitPrompt() {
   promptEl.style.overflowY = scrolls ? "auto" : "hidden";
 }
 
-// A drag down past the box folds the composer on release (a session only: the
-// start page's grip only resizes); from folded, a pull up opens it and the box
-// follows the pointer in the same gesture, a tap opens it and focuses the box.
-// No click listener: a click still fires after pointerdown's preventDefault
-// and would toggle twice.
-composeGrip.addEventListener("pointerdown", (e) => {
-  e.preventDefault();
-  composeGrip.setPointerCapture(e.pointerId);
-  composeGrip.classList.add("dragging");
-  const tapY = e.clientY;
-  let startY = tapY;
-  let startH = promptEl.offsetHeight;
-  let opened = false;
-  const move = (ev) => {
-    if (selected && isComposerFolded()) {
-      if (startY - ev.clientY < GRIP_OPEN_PX) return;
-      setComposerFolded(false);
-      opened = true;
-      // Re-based after the unfold (its fitPrompt sized the box: the minimum
-      // when empty, the text's height with a draft).
-      startH = promptEl.offsetHeight;
-      startY = ev.clientY;
-      return;
-    }
-    const requested = startH + (startY - ev.clientY);
-    promptFloor = gripFloor(requested, window.innerHeight, promptMinPx());
-    fitPrompt();
-    const willFold = !!selected && gripFolds({ requestedPx: requested, shownPx: promptEl.offsetHeight });
-    if (willFold !== gripWillFold) {
-      gripWillFold = willFold;
-      composeGrip.classList.toggle("will-fold", willFold);
-      drawFoldControls();
-    }
-  };
-  const end = () => {
-    composeGrip.classList.remove("dragging");
-    composeGrip.removeEventListener("pointermove", move);
-    composeGrip.removeEventListener("pointerup", up);
-    composeGrip.removeEventListener("pointercancel", cancel);
-    const folds = gripWillFold;
-    gripWillFold = false;
-    composeGrip.classList.remove("will-fold");
-    return folds;
-  };
-  const up = (ev) => {
-    const folds = end();
-    if (folds) {
-      // The next opening starts at the minimum (or the text's height).
-      promptFloor = null;
-      try { localStorage.removeItem(GRIP_KEY); } catch (_) {}
-      setComposerFolded(true);
-    } else if (selected && isComposerFolded()) {
-      if (Math.abs(ev.clientY - tapY) < GRIP_OPEN_PX) {
-        setComposerFolded(false);
-        promptEl.focus();
-      }
-    } else {
-      try { localStorage.setItem(GRIP_KEY, String(promptFloor || "")); } catch (_) {}
-      // A pull that opened the box leaves the focus in it, as a tap does.
-      if (opened) promptEl.focus();
-    }
-    drawFoldControls();
-  };
-  const cancel = () => {
-    end();
-    drawFoldControls();
-  };
-  composeGrip.addEventListener("pointermove", move);
-  composeGrip.addEventListener("pointerup", up);
-  composeGrip.addEventListener("pointercancel", cancel);
-});
-composeGrip.addEventListener("dblclick", () => {
-  if (selected && isComposerFolded()) return;
-  promptFloor = null;
-  try { localStorage.removeItem(GRIP_KEY); } catch (_) {}
-  fitPrompt();
-});
-// Enter or Space on the focused handle folds, or opens and focuses the box
-// (a session only; preventDefault: Space would scroll the page).
-composeGrip.addEventListener("keydown", (e) => {
-  if (!selected || (e.key !== "Enter" && e.key !== " ")) return;
-  e.preventDefault();
-  if (isComposerFolded()) {
-    setComposerFolded(false);
-    promptEl.focus();
-  } else {
-    setComposerFolded(true);
-  }
-});
 promptEl.addEventListener("input", fitPrompt);
 window.addEventListener("resize", fitPrompt);
 fitPrompt();
@@ -1906,102 +1795,27 @@ document.addEventListener("keydown", (e) => {
 cancelBtn.addEventListener("click", () => handleCancel());
 infoCancelBtn.addEventListener("click", () => handleCancel());
 
-// ── Composer fold (composer_fold.js) ───────────────────────────────────────
-// Dragging the composer's grip down folds the session's composer away; the
-// grip, now the handle on the info bar's top edge, pulled up or clicked, or
-// typing on the page, brings it back. The user's own choice is saved per
-// browser; an archived session opens folded for its visit, and expanding
-// there saves nothing. The page's own focus or a restored line opens it
-// for the visit only. The start page never folds.
-function composerFoldSaved() {
-  try {
-    return localStorage.getItem(COMPOSER_FOLD_KEY) === "1";
-  } catch (_) {
-    return false;
-  }
-}
-function isComposerFolded() {
-  return !!selected && composerFolded({ saved: composerFoldSaved(), archivedVisit: foldVisit.archived, visitOpen: foldVisit.open });
-}
-function applyComposerFolded() {
-  const folded = isComposerFolded();
-  const was = document.body.classList.contains("composer-folded");
-  document.body.classList.toggle("composer-folded", folded);
-  // Its height was measured while hidden.
-  if (was && !folded) fitPrompt();
-  drawFoldControls();
-}
-// The user's own fold or unfold (the grip, typing): saved, unless this
-// is an archived visit (both are for the visit then).
-function setComposerFolded(folded) {
-  if (foldVisit.archived) {
-    foldVisit.open = !folded;
-  } else {
-    try { localStorage.setItem(COMPOSER_FOLD_KEY, folded ? "1" : "0"); } catch (_) {}
-    foldVisit.open = false;
-  }
-  applyComposerFolded();
-}
-// The page opens it for this visit: nothing saved.
-function openForVisit() {
-  if (!isComposerFolded()) return;
-  foldVisit.open = true;
-  applyComposerFolded();
-}
-// Every focus of the message box: in a session it opens a folded composer
-// for the visit first (+open+ false: only when it shows already).
-function focusPrompt({ open = true } = {}) {
-  if (selected && isComposerFolded()) {
-    if (!open) return;
-    openForVisit();
-  }
-  promptEl.focus();
-}
-// The grip as the fold's handle (its title, marks, aria-expanded) and Stop turn
-// while folded. The start page's grip only resizes: no role, no fold state.
-function drawFoldControls() {
-  const folded = isComposerFolded();
-  infoCancelBtn.style.display = folded && liveTurn.running ? "" : "none";
-  if (!selected) {
-    composeGrip.removeAttribute("role");
-    composeGrip.removeAttribute("aria-expanded");
-    composeGrip.title = gripStartTitle;
-    for (const c of ["folded", "draft", "waiting"]) composeGrip.classList.remove(c);
-    return;
-  }
-  const id = selected;
-  const waiting = folded && !!(waitingOn(allSessions.find((s) => s.id === id)) || questionCards.pendingQuestion());
-  const h = foldHandle({ folded, draft: !!(promptEl.value.trim() || chips.length), waiting });
-  composeGrip.title = gripWillFold ? "Release to hide the message box" : h.title;
-  composeGrip.setAttribute("role", "button");
-  composeGrip.setAttribute("aria-expanded", folded ? "false" : "true");
-  composeGrip.classList.toggle("folded", h.folded);
-  composeGrip.classList.toggle("draft", h.draft);
-  composeGrip.classList.toggle("waiting", h.waiting);
-}
-promptEl.addEventListener("input", () => drawFoldControls());
-// Typing on the page opens a folded composer: the key lands in the box
-// (the class goes before the focus, so the browser types it there).
-document.addEventListener("keydown", (e) => {
-  if (!isComposerFolded()) return;
-  if (!opensOnKey(e, { active: document.activeElement, allView: allViewOpen(), lightbox: !lightboxEl.hidden })) return;
-  setComposerFolded(false);
-  promptEl.focus();
-});
-// A file dropped on the folded info bar: open (for the visit), then add it.
-const infoBarEl = $("#infoBar");
-infoBarEl.addEventListener("dragover", (e) => {
-  if (!isComposerFolded() || ![...(e.dataTransfer?.types || [])].includes("Files")) return;
-  e.preventDefault();
-});
-infoBarEl.addEventListener("drop", (e) => {
-  if (!isComposerFolded()) return;
-  const files = imageFiles(e.dataTransfer);
-  if (!files.length) return;
-  e.preventDefault();
-  openForVisit();
-  addImageFiles(files);
+// The composer's grip and fold (composer_grip.js).
+const composerGrip = createComposerGrip({
+  composeGrip: $("#composeGrip"),
+  promptEl,
+  infoCancelBtn,
+  infoBarEl: $("#infoBar"),
+  session: () => selected,
+  fitPrompt,
+  promptMinPx,
+  floor: () => promptFloor,
+  setFloor: (px) => { promptFloor = px; },
+  draft: () => !!(promptEl.value.trim() || chips.length),
+  waiting: () => !!(waitingOn(allSessions.find((s) => s.id === selected)) || questionCards.pendingQuestion()),
+  turnRunning: () => liveTurn.running,
+  allViewOpen,
+  lightboxOpen: () => !lightboxEl.hidden,
+  imageFiles,
+  addImageFiles,
 });
+// Bound here, not hoisted: no top-level code above this line may call them.
+const { openForVisit, focusPrompt, drawFoldControls } = composerGrip;
 // The / autocomplete (command_list.js): the session's commands while the
 // composer holds one "/word".
 const commandList = createCommandList({
@@ -2275,12 +2089,8 @@ const notifications = createNotifications({ button: $("#notifyBtn") });
 // The latest-sessions strip can be hidden for more chat room; the choice
 // stays per browser (it replaced zen mode, so a zen user starts hidden).
 function stripHiddenSaved() {
-  try {
-    const saved = localStorage.getItem("chi_strip_hidden");
-    return saved === null ? localStorage.getItem("chi_zen") === "1" : saved === "1";
-  } catch (_) {
-    return false;
-  }
+  const saved = readStored("chi_strip_hidden");
+  return saved === null ? readStored("chi_zen") === "1" : saved === "1";
 }
 // A short window hides the strip by itself, so the stage keeps room for a
 // card that waits (its question and choices); the sessions pill brings it
@@ -2296,7 +2106,7 @@ function applyStripHidden() {
 }
 function setStripHidden(hidden) {
   if (hidden) lists.familyPop.close();
-  try { localStorage.setItem("chi_strip_hidden", hidden ? "1" : "0"); } catch (_) {}
+  writeStored("chi_strip_hidden", hidden ? "1" : "0");
   stripShownWhileShort = !hidden && shortWindow.matches;
   applyStripHidden();
 }