← back to Terminal Status

terminal_status.py: fix set-variant — was calling nonexistent Owner.detect()/STORE

43fe59830febc45f2c3e5dd1a8af23ffd0a8748e · 2026-09-11 10:51:08 -0700 · Steve Abrams

The set-variant branch shipped calling Owner.detect() (no such method) and STORE
(undefined), so it crashed with AttributeError in EVERY session and the additive
🔵 stopped-marker never worked. Now uses the already-resolved caller + the real
store instance, matching every other command. Verified: set-variant stopped paints
base+🔵 (e.g. 🟣🔵), set-variant '' clears it.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

Files touched

• M terminal_status.py

Diff

commit 43fe59830febc45f2c3e5dd1a8af23ffd0a8748e
Author: Steve Abrams <steve@designerwallcoverings.com>
Date:   Fri Sep 11 10:51:08 2026 -0700

    terminal_status.py: fix set-variant — was calling nonexistent Owner.detect()/STORE
    
    The set-variant branch shipped calling Owner.detect() (no such method) and STORE
    (undefined), so it crashed with AttributeError in EVERY session and the additive
    🔵 stopped-marker never worked. Now uses the already-resolved caller + the real
    store instance, matching every other command. Verified: set-variant stopped paints
    base+🔵 (e.g. 🟣🔵), set-variant '' clears it.
    
    Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
---
 terminal_status.py | 8 +++++---
 1 file changed, 5 insertions(+), 3 deletions(-)

diff --git a/terminal_status.py b/terminal_status.py
index a2c9de3..14a644e 100644
--- a/terminal_status.py
+++ b/terminal_status.py
@@ -701,9 +701,11 @@ def main(argv=None):
         return 0
     caller = current_owner(rows)
     if args.command == "set-variant":
-        owner = Owner.detect()
-        record = STORE.set_variant(owner, args.variant)
-        print(f'/terminal-status → /dev/{owner.tty} {record["title"] or "CLEARED"}')
+        # Use the already-resolved caller + the real store instance (the branch shipped calling
+        # a nonexistent Owner.detect()/STORE, so set-variant crashed in EVERY session — the
+        # auto-🔵 stopped-marker never fired). Matches every other command's resolution. (2026-09-11)
+        record = store.set_variant(caller, args.variant)
+        print(f'/terminal-status → /dev/{caller.tty} {record["title"] or "CLEARED"}')
         return 0
 
     if args.command == "current":

← c40a4b0 terminal-status: add lightblue to PRIORITY (unblock scan cra  ·  back to Terminal Status  ·  terminal_status.py: set-variant --if-blocked (only 🔵-mark n 1f88b72 →