openclacky 1.3.4 → 1.3.5

data/lib/clacky/ui2/ui_controller.rb

@@ -17,7 +17,7 @@ module Clacky
       include Clacky::UIInterface
 
       attr_reader :layout, :renderer, :running, :inline_input, :input_area
-      attr_accessor :config
+      attr_accessor :config, :available_models
 
       def initialize(config = {})
         @renderer = ViewRenderer.new
@@ -38,6 +38,7 @@ module Clacky
         @todo_area = Components::TodoArea.new
         @welcome_banner = Components::WelcomeBanner.new
         @inline_input = nil  # Created when needed
+        @feedback_countdown = nil  # Active auto-approve feedback countdown session
         @layout = LayoutManager.new(
           input_area: @input_area,
           todo_area: @todo_area
@@ -874,7 +875,7 @@ module Clacky
 
       # Show error message
       # @param message [String] Error message
-      def show_error(message, code: nil, top_up_url: nil)
+      def show_error(message, code: nil, top_up_url: nil, raw_message: nil)
         output = @renderer.render_error(message)
         append_output(output)
       end
@@ -1054,6 +1055,99 @@ module Clacky
         end
       end
 
+      # Auto-approve countdown for request_user_feedback: show a single live
+      # countdown line. If the user presses any key before timeout, collect
+      # their answer and return it (intervention). Otherwise return :timeout so
+      # the agent auto-decides and continues.
+      # @param seconds [Integer] Countdown duration
+      # @return [String, Symbol] feedback string, "" (bare Enter), or :timeout
+      # Show a live single-line countdown in the output area while keeping the
+      # normal input box usable. The agent thread blocks here until either:
+      #   * the user submits a message in the regular input box -> returns the text
+      #   * the user starts typing (cancels the auto-timeout but keeps waiting)
+      #   * the countdown reaches zero with no interaction -> returns :timeout
+      # @param seconds [Integer] Countdown duration
+      # @return [String, Symbol] feedback text, or :timeout
+      def request_feedback_with_countdown(seconds: 10)
+        theme = ThemeManager.current_theme
+
+        queue = Queue.new
+        entry_id = @layout.append_output(countdown_prompt(seconds, theme))
+
+        session = {
+          queue: queue,
+          entry_id: entry_id,
+          intervened: false,
+          watchdog: nil
+        }
+        @feedback_countdown = session
+
+        session[:watchdog] = Thread.new do
+          remaining = seconds.to_i
+          while remaining.positive?
+            break if session[:intervened]
+
+            @layout.replace_entry(entry_id, countdown_prompt(remaining, theme))
+            sleep 1
+            remaining -= 1
+          end
+          queue.push(:timeout) unless session[:intervened]
+        end
+
+        result = queue.pop
+        session[:watchdog].kill if session[:watchdog]&.alive?
+        @feedback_countdown = nil
+        @layout.remove_entry(entry_id) if entry_id
+        @layout.recalculate_layout
+        @layout.render_all
+
+        if result == :timeout
+          append_output(theme.format_text("  No response — continuing automatically.", :thinking))
+        elsif result.to_s.strip.empty?
+          append_output(theme.format_text("  → (continue)", :success))
+          result = ""
+        end
+
+        result
+      end
+
+      # Cancel the countdown's auto-timeout when the user starts interacting,
+      # but keep waiting for their submitted answer. Returns true if a
+      # countdown was active.
+      private def intervene_feedback_countdown
+        session = @feedback_countdown
+        return false unless session && !session[:intervened]
+
+        session[:intervened] = true
+        session[:watchdog].kill if session[:watchdog]&.alive?
+        if session[:entry_id]
+          @layout.remove_entry(session[:entry_id])
+          session[:entry_id] = nil
+          @layout.recalculate_layout
+          @layout.render_all
+        end
+        true
+      end
+
+      private def countdown_prompt(remaining, theme)
+        theme.format_text("  Auto-continuing in #{remaining}s — type your answer to step in…", :info)
+      end
+
+      # Whether a keystroke should count as the user stepping into a countdown.
+      # Plain typing and pastes do; pure scroll/navigation keys do not.
+      private def countdown_intervening_key?(key)
+        case key
+        when Hash
+          key[:type] == :rapid_input
+        when String
+          key.length >= 1 && key.ord >= 32
+        when :backspace, :enter
+          true
+        else
+          false
+        end
+      end
+
       # Show diff between old and new content
       # @param old_content [String] Old content
       # @param new_content [String] New content
@@ -1206,7 +1300,7 @@ module Clacky
       # Check if API key is configured and show warning if missing
       private def check_api_key_configuration
         config = Clacky::AgentConfig.load
-        
+
         if !config.models_configured?
           show_warning("No models configured! Please run /config to set up your models and API keys.")
         elsif config.api_key.nil? || config.api_key.empty?
@@ -1250,7 +1344,7 @@ module Clacky
         while @running
           # Process any pending resize events
           @layout.process_pending_resize
-          
+
           key = @layout.screen.read_key(timeout: 0.1)
           next unless key
 
@@ -1291,6 +1385,13 @@ module Clacky
           return
         end
 
+        # During an auto-approve feedback countdown the normal input box stays
+        # live; the first meaningful keystroke cancels the auto-timeout so the
+        # user can finish typing their answer without being rushed.
+        if @feedback_countdown && !@feedback_countdown[:intervened] && countdown_intervening_key?(key)
+          intervene_feedback_countdown
+        end
+
         result = @input_area.handle_key(key)
 
         # Handle height change first
@@ -1416,6 +1517,14 @@ module Clacky
           append_output(output)
         end
 
+        # If an auto-approve feedback countdown is waiting for an answer, this
+        # submission IS that answer — deliver it to the waiting agent thread
+        # instead of starting a brand-new turn.
+        if (session = @feedback_countdown)
+          session[:queue].push(data[:text].to_s)
+          return
+        end
+
         # Then call callback (allows interrupting previous agent before processing new input)
         @input_callback&.call(data[:text], data[:files])
       end
@@ -1425,31 +1534,31 @@ module Clacky
       # @return [Hash, nil] Hash with updated config values, or nil if cancelled
       public def show_config_modal(current_config, test_callback: nil)
         modal = Components::ModalComponent.new
-        
+
         loop do
           # Build menu choices
           choices = []
-          
+
           # Add model list
           current_config.models.each_with_index do |model, idx|
             is_current = (idx == current_config.current_model_index)
             model_name = model["model"] || "unnamed"
             masked_key = mask_api_key(model["api_key"])
-            
+
             # Add type badge if present
             type_badge = case model["type"]
                         when "default" then "[default] "
                         when "lite" then "[lite] "
                         else ""
                         end
-            
+
             display_name = "#{type_badge}#{model_name} (#{masked_key})"
             choices << {
               name: display_name,
               value: { action: :switch, model_id: model["id"] }
             }
           end
-          
+
           # Add action buttons
           choices << { name: "─" * 50, disabled: true }
           choices << { name: "[+] Add New Model", value: { action: :add } }
@@ -1458,16 +1567,16 @@ module Clacky
             choices << { name: "[-] Delete Model", value: { action: :delete } } if current_config.models.length > 1
           end
           choices << { name: "[X] Close", value: { action: :close } }
-          
+
           # Show menu
           result = modal.show(
-            title: "Model Configuration", 
+            title: "Model Configuration",
             choices: choices,
             on_close: -> { @layout.rerender_all }
           )
-          
+
           return nil if result.nil?
-          
+
           case result[:action]
           when :switch
             # Just signal the caller which model to switch to.
@@ -1699,16 +1808,16 @@ module Clacky
 
         result # Return selected task_id or nil
       end
-      
+
       # Show form for editing a model
       # @param model [Hash, nil] Existing model hash or nil for new model
       # @return [Hash, nil] Updated model hash or nil if cancelled
       private def show_model_edit_form(model, test_callback: nil)
         modal = Components::ModalComponent.new
-        
+
         is_new = model.nil?
         model ||= {}
-        
+
         # For new models, show provider selection first
         selected_provider = nil
         if is_new
@@ -1718,32 +1827,32 @@ module Clacky
           end
           provider_choices << { name: "─" * 40, disabled: true }
           provider_choices << { name: "Custom (manual configuration)", value: "custom" }
-          
+
           # Show provider selection
           selected_provider = modal.show(
             title: "Select Provider",
             choices: provider_choices,
             on_close: -> { @layout.rerender_all }
           )
-          
+
           # User cancelled
           return nil if selected_provider.nil?
         end
-        
+
         # Prepare masked API key for display
         masked_key = mask_api_key(model["api_key"])
-        
+
         # Pre-fill values from provider preset if selected
         provider_preset = nil
         if selected_provider && selected_provider != "custom"
           provider_preset = Clacky::Providers.get(selected_provider)
         end
-        
+
         # Get default values from provider or existing model
         default_model = provider_preset ? provider_preset["default_model"] : model["model"]
         default_base_url = provider_preset ? provider_preset["base_url"] : model["base_url"]
         default_api_key = model["api_key"] || ""
-        
+
         # Define fields
         fields = [
           {
@@ -1763,7 +1872,7 @@ module Clacky
             default: default_base_url || ""
           }
         ]
-        
+
         # Create validator if test_callback provided
         validator = if test_callback
           lambda do |values|
@@ -1772,14 +1881,14 @@ module Clacky
             model_name = values[:model].to_s.empty? ? model["model"] : values[:model]
             base_url = values[:base_url].to_s.empty? ? model["base_url"] : values[:base_url]
             anthropic_format = model["anthropic_format"] # Not editable in form, use model's value
-            
+
             test_config_values = {
               "api_key" => api_key,
               "model" => model_name,
               "base_url" => base_url,
               "anthropic_format" => anthropic_format
             }
-            
+
             # For new models, require all fields
             if is_new
               if test_config_values["api_key"].to_s.empty?
@@ -1792,7 +1901,7 @@ module Clacky
                 return { success: false, error: "Base URL is required" }
               end
             end
-            
+
             # Create a temporary config for testing
             temp_config = Clacky::AgentConfig.new(models: [test_config_values], current_model_index: 0)
             test_callback.call(temp_config)
@@ -1800,7 +1909,7 @@ module Clacky
         else
           nil
         end
-        
+
         # Determine modal title based on provider
         modal_title = if is_new && selected_provider && selected_provider != "custom"
           provider_name = Clacky::Providers.get(selected_provider)&.dig("name") || selected_provider
@@ -1810,7 +1919,7 @@ module Clacky
         else
           "Edit Model"
         end
-        
+
         # Show modal and collect values
         result = modal.show(
           title: modal_title,
@@ -1818,9 +1927,9 @@ module Clacky
           validator: validator,
           on_close: -> { @layout.rerender_all }
         )
-        
+
         return nil if result.nil?
-        
+
         # Merge with existing model values or provider defaults
         {
           api_key: result[:api_key].to_s.empty? ? model["api_key"] : result[:api_key],
@@ -1829,7 +1938,7 @@ module Clacky
           provider: selected_provider
         }
       end
-      
+
       # Mask API key for display
       private def mask_api_key(api_key)
         if api_key && !api_key.empty?

data/lib/clacky/ui_interface.rb

@@ -28,7 +28,7 @@ module Clacky
     # === Status messages ===
     def show_info(message, prefix_newline: true); end
     def show_warning(message); end
-    def show_error(message, code: nil, top_up_url: nil); end
+    def show_error(message, code: nil, top_up_url: nil, raw_message: nil); end
     def show_success(message); end
     def log(message, level: :info); end
 
@@ -131,6 +131,15 @@ module Clacky
     # === Blocking interaction ===
     def request_confirmation(message, default: true); end
 
+    # Auto-approve countdown for request_user_feedback. Shows a live countdown
+    # and lets the user press a key to take over and answer. Returns :timeout
+    # when no one intervenes (agent should auto-decide and continue), or a
+    # feedback string / "" when the user steps in. Non-interactive UIs (web,
+    # json, channel) have no human watching a TTY, so they default to :timeout.
+    def request_feedback_with_countdown(seconds: 10)
+      :timeout
+    end
+
     # === Input control (CLI layer) ===
     def clear_input; end
     def set_input_tips(message, type: :info); end

data/lib/clacky/utils/encoding.rb

@@ -75,6 +75,31 @@ module Clacky
         to_utf8(data)
       end
 
+      # Decode a raw PTY byte stream to valid UTF-8, auto-detecting the source
+      # encoding. UTF-8 is tried first (Linux/macOS and modern programs); when
+      # the bytes are not valid UTF-8 they are decoded as GBK/CP936 (Simplified
+      # Chinese Windows powershell.exe / cmd.exe default output); anything that
+      # still fails is scrubbed.
+      #
+      # MUST be called on complete byte runs — callers slice on "\n" (0x0A),
+      # which is never a trailing byte of a GBK or UTF-8 multibyte sequence, so
+      # a character is never split across the boundary.
+      #
+      # @param data [String, nil] raw PTY bytes (binary/ASCII-8BIT)
+      # @return [String] valid UTF-8 string
+      def self.pty_to_utf8(data)
+        return "" if data.nil? || data.empty?
+
+        s = data.dup.force_encoding("UTF-8")
+        return s if s.valid_encoding?
+
+        data.dup
+            .force_encoding("GBK")
+            .encode("UTF-8", invalid: :replace, undef: :replace, replace: "?")
+      rescue Encoding::InvalidByteSequenceError, Encoding::UndefinedConversionError
+        to_utf8(data)
+      end
+
       # Return an ASCII-safe UTF-8 copy of *str* suitable for security regex
       # pattern matching.  Any byte that is not valid in the source encoding, or
       # that cannot be represented in UTF-8, is replaced with '?'.  The

data/lib/clacky/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Clacky
-  VERSION = "1.3.4"
+  VERSION = "1.3.5"
 end

data/lib/clacky/web/app.css

@@ -650,38 +650,38 @@ body {
   position: relative;
   display: flex;
   align-items: center;
-  border-radius: 5px;
+  border-radius: 6px;
   overflow: visible;
 }
 .btn-split-main {
-  height: 1.375rem;
-  padding: 0 0.5rem;
+  height: 1.5rem;
+  padding: 0 0.625rem;
   border: none;
-  border-radius: 5px 0 0 5px;
+  border-radius: 6px 0 0 6px;
   background: var(--color-accent-primary);
   color: #fff;
-  font-size: 0.6875rem;
+  font-size: 0.75rem;
   font-weight: 500;
   white-space: nowrap;
   line-height: 1;
   cursor: pointer;
-  transition: background 0.15s, color 0.15s, box-shadow 0.15s;
+  transition: background 0.15s, box-shadow 0.15s;
 }
 .btn-split-main:hover {
   background: var(--color-button-primary-hover);
-  box-shadow: 0 2px 4px rgba(0,0,0,0.1);
+  box-shadow: 0 2px 6px rgba(0,0,0,0.15);
 }
 .btn-split-arrow {
-  height: 1.375rem;
-  padding: 0 0.25rem;
+  height: 1.5rem;
+  padding: 0 0.35rem;
   border: none;
   border-left: 1px solid rgba(255,255,255,0.3);
-  border-radius: 0 5px 5px 0;
+  border-radius: 0 6px 6px 0;
   background: var(--color-accent-primary);
   color: #fff;
-  font-size: 0.625rem;
+  font-size: 0.75rem;
   cursor: pointer;
-  transition: background 0.15s, color 0.15s;
+  transition: background 0.15s;
 }
 .btn-split-arrow:hover {
   background: var(--color-button-primary-hover);
@@ -690,7 +690,7 @@ body {
 /* ── Dropdown menu ───────────────────────────────────────────────────────── */
 .new-session-dropdown {
   position: absolute;
-  top: calc(100% + 0.25rem);
+  top: 100%;
   right: 0;
   min-width: 11.25rem;
   background: var(--color-bg-card);
@@ -698,13 +698,13 @@ body {
   border-radius: 7px;
   box-shadow: var(--shadow-md);
   z-index: 200;
-  padding: 0.25rem 0;
+  padding: 0.375rem 0 0.25rem;
   overflow: hidden;
 }
 .new-session-dropdown[hidden] { display: none; }
 .dropdown-item {
-  padding: 0.25rem 0.625rem;
-  font-size: 0.6875rem;
+  padding: 0.4rem 0.875rem;
+  font-size: 0.8125rem;
   color: var(--color-text-secondary);
   cursor: pointer;
   white-space: nowrap;
@@ -1799,7 +1799,7 @@ body {
   white-space: nowrap;
   transition: background .12s;
 }
-.task-run-btn:hover { background: var(--color-accent-hover); }
+.task-run-btn:hover { background: var(--color-button-primary-hover); }
 .task-action-btn {
   display: inline-flex;
   align-items: center;
@@ -2456,6 +2456,36 @@ body {
   .msg-copy-btn { opacity: 0.65; }
 }
 
+/* ── Inline "start" button on unchecked task-list items ──────────────────── */
+.msg-assistant li.task-list-item { position: relative; }
+.msg-todo-spawn {
+  display: inline-flex;
+  align-items: center;
+  margin-left: 0.5rem;
+  padding: 0.05rem 0.5rem;
+  font-size: 0.75rem;
+  line-height: 1.4;
+  border: 1px solid var(--color-border-primary);
+  border-radius: 5px;
+  background: var(--color-bg-primary);
+  color: var(--color-text-secondary);
+  cursor: pointer;
+  opacity: 0;
+  vertical-align: middle;
+  transition: opacity 0.15s ease, color 0.15s ease, background 0.15s ease, border-color 0.15s ease;
+}
+.msg-assistant:hover .msg-todo-spawn,
+.msg-todo-spawn:focus-visible { opacity: 1; }
+.msg-todo-spawn:hover {
+  color: var(--color-text-primary);
+  background: var(--color-bg-tertiary);
+  border-color: var(--color-border-secondary);
+}
+.msg-todo-spawn:disabled { opacity: 0.5; cursor: default; }
+@media (hover: none) {
+  .msg-todo-spawn { opacity: 0.7; }
+}
+
 /* ── Markdown rendering inside assistant messages ────────────────────────── */
 .msg-assistant p:last-child { margin-bottom: 0; }
 .msg-assistant h1, .msg-assistant h2, .msg-assistant h3,
@@ -2690,7 +2720,7 @@ body {
   align-self: flex-start;
   display: flex;
   flex-direction: column;
-  gap: 0.75rem;
+  gap: 0.5rem;
 }
 .msg-error .retry-btn {
   align-self: flex-start;
@@ -2721,6 +2751,56 @@ body {
 .msg-error a:hover {
   opacity: 0.8;
 }
+.error-raw-detail {
+  margin-top: -0.125rem;
+  font-size: 0.75rem;
+  align-self: flex-start;
+  max-width: 100%;
+}
+.error-raw-detail summary {
+  cursor: pointer;
+  font-size: 0.7rem;
+  font-weight: 500;
+  user-select: none;
+  color: var(--color-error);
+  opacity: 0.6;
+  list-style: none;
+  display: inline-flex;
+  align-items: center;
+  gap: 0.25rem;
+  transition: opacity 0.15s;
+}
+.error-raw-detail summary::-webkit-details-marker { display: none; }
+.error-raw-detail summary::before {
+  content: "";
+  display: inline-block;
+  width: 0;
+  height: 0;
+  border-top: 4px solid transparent;
+  border-bottom: 4px solid transparent;
+  border-left: 5px solid currentColor;
+  transition: transform 0.15s;
+  flex-shrink: 0;
+}
+.error-raw-detail[open] summary::before {
+  transform: rotate(90deg);
+}
+.error-raw-detail summary:hover {
+  opacity: 0.9;
+}
+.error-raw-detail pre {
+  margin: 0.375rem 0 0;
+  padding: 0.5rem 0.625rem;
+  background: rgba(255, 59, 48, 0.05);
+  border: 1px solid rgba(255, 59, 48, 0.12);
+  border-radius: 5px;
+  font-family: var(--font-mono, monospace);
+  font-size: 0.7rem;
+  line-height: 1.5;
+  white-space: pre-wrap;
+  word-break: break-all;
+  color: var(--color-text-secondary);
+}
 .msg-success   { color: var(--color-success); align-self: flex-start; font-size: 0.8125rem; }
 .tool-name     { color: var(--color-warning); font-weight: 600; }
 .progress-msg  { color: var(--color-accent-primary); font-size: 0.75rem; align-self: center; }
@@ -6929,10 +7009,10 @@ body {
   background: var(--color-bg-hover);
 }
 .code-editor-body .cm-activeLine {
-  background: var(--color-bg-hover);
+  background: color-mix(in srgb, var(--color-bg-hover) 60%, transparent);
 }
 .code-editor-body .cm-selectionBackground {
-  background: color-mix(in srgb, var(--color-accent-primary) 20%, transparent) !important;
+  background: color-mix(in srgb, var(--color-accent-primary) 40%, transparent) !important;
 }
 .code-editor-footer {
   display: flex;
@@ -10046,7 +10126,7 @@ body.setup-mode[data-theme="dark"] {
   border-right: 1px solid var(--color-border-primary);
 }
 .billing-period-btn:hover {
-  background: var(--color-bg-tertiary);
+  background: var(--color-bg-hover);
   color: var(--color-text-primary);
 }
 .billing-period-btn.active {
@@ -10906,7 +10986,7 @@ body.setup-mode[data-theme="dark"] {
   white-space: nowrap;
   transition: opacity 0.15s ease, transform 0.05s ease;
 }
-.btn-mcp-cta:hover { opacity: 0.85; }
+.btn-mcp-cta:hover { background: var(--color-button-primary-hover) }
 .btn-mcp-cta:active { transform: scale(0.98); }
 .btn-mcp-cta-large {
   padding: 0.625rem 1.125rem;
@@ -11019,6 +11099,49 @@ body.setup-mode[data-theme="dark"] {
   color: var(--color-error, #d33);
 }
 
+/* ── Accent Color Swatches ───────────────────────────────────────────── */
+.settings-accent-swatches {
+  display: flex;
+  flex-wrap: wrap;
+  gap: 0.5rem;
+  padding: 0.25rem 0;
+}
+
+.settings-accent-swatch {
+  width: 2.25rem;
+  height: 2.25rem;
+  border-radius: 50%;
+  border: none;
+  cursor: pointer;
+  padding: 0;
+  transition: transform 0.15s;
+  outline: none;
+  position: relative;
+}
+
+.settings-accent-swatch:hover {
+  transform: scale(1.15);
+}
+
+.settings-accent-swatch.active::after {
+  content: "";
+  position: absolute;
+  width: 0.375rem;
+  height: 0.625rem;
+  border-right: 0.125rem solid white;
+  border-bottom: 0.125rem solid white;
+  top: 46%;
+  left: 50%;
+  transform: translate(-50%, -50%) rotate(45deg);
+}
+
+.swatch-indigo         { background: #4f46e5; }
+.swatch-aurora-blue    { background: #3B82F6; }
+.swatch-forest-green   { background: #10B981; }
+.swatch-sunrise-orange { background: #F59E0B; }
+.swatch-rose-violet    { background: #8B5CF6; }
+.swatch-coral-red      { background: #EF4444; }
+
 /* ── Sessions List ───────────────────────────────────────────────────── */
 .billing-sessions-row {
   margin-top: 1rem;