openclacky 1.5.9 → 1.5.11

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 565adf42059330fb821c57cb3b1200035fb866a1f2190ca87e093bc34de9edc0
-  data.tar.gz: 89db4dd987f39d4b77acd52121aa31bef54858f81d614557ac7b9984a1066ba0
+  metadata.gz: 1cef597eb5d6c066192ba6c2c587e0dcd62fdf2dd9ed422161f71718acbb7f25
+  data.tar.gz: 03bcd29f23eacd7b49a37db0494f7964757391ac52e8f3ef9787c49a6b2f8efb
 SHA512:
-  metadata.gz: e3bbb8988d833e3c6e6ce4cd13da64111e2ee15b400c2eb34b78bca9f51ae80525336baf923ba18cdbf85b5320d542f8050567792e7d5a437b28e2d09d29729d
-  data.tar.gz: ae508f3bdb44a35b8edd0c89ebcc53a64d3f7340214ded6c8dd47d0c757c66f5f99700f466c86278e8ee0a38ab838b8509c80ba8af2f253653f967b91f52a814
+  metadata.gz: 7543c4f518bf5a2e20778472e60909b5eda8f727015b66aad37c69b2d256b53ba3cd5126d2b385a06456a45a6531fe4fdaad7c6d7f8b247aeb2a76bf1446a9f1
+  data.tar.gz: d6ceb974a12d29096ae65bcbed3d6c6467880005101664919357572201846792faf4f89956ff5ca568519d74960674e214a39c61333e527741a9eb38083fcbf1

data/.clackyrules

@@ -70,6 +70,7 @@ the CI matrix - a green local `bundle exec rspec` on Ruby 3.x proves nothing abo
 - **IMPORTANT**: When developing new features, write RSpec tests as needed and ensure they pass
 - **DO NOT** write custom test scripts unless explicitly requested by the user
 - **DO NOT** create markdown documentation unless explicitly requested by the user
+- **DO NOT** run RuboCop for verification. `bundle exec rspec` is the only required check; the codebase has many pre-existing offenses, so RuboCop output is noise. Only run it when the user explicitly asks
 - **IMPORTANT**: When testing clacky commands or debugging, always use `bundle exec ruby bin/clacky` instead of the global `clacky` command. The global command loads the system-installed gem version (e.g., `openclacky-0.7.0`), not your local development code
 
 ### Tool Development

data/CHANGELOG.md

@@ -4,6 +4,66 @@ All notable changes to this project will be documented in this file.
 
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 
+## [1.5.11] - 2026-08-23
+
+### Added
+- Creator Studio: separate Marketplace and Brand sections for published extensions
+- Advisor extension for next-job suggestions
+- ask_user tool with multi-question support (replaces feedback_user)
+- Sidebar states for awaiting-feedback and just-finished sessions
+- Auto-purge file-trash entries older than 8 days
+- DeepSeek V4 flash vision exp model preset
+- IM channel toggle to mute tool-call narration and previews
+
+### Improved
+- Unified internal thread management with graceful shutdown
+
+### Fixed
+- Skill decrypt retry now picks up a renewed license key
+- DeepSeek weekends billed entirely at off-peak rates
+- Skill and extension zip downloads fall back to a secondary CDN host
+- Session status dot aligned with name and channel badge
+- Project session tag top margin
+- Unified modal close button and ext-studio modal semantics
+- Unified checkbox accent and distinct verified badge
+- Edit-model modal overflow on small screens
+- Subagent UI fixes
+- Star History charts migrated to star-history.dera.page (#491 - @CrustyMozarella)
+
+## [1.5.10] - 2026-08-19
+
+### Added
+- Import extensions from a local zip file
+- Per-model API format override (#484 - @kylezhang)
+- Project pinning (pin to top)
+- Copy Session ID action in session menu
+- Price ratio on model switcher panel
+- Model search filter and provider filter dropdown in settings
+- Remark field for model configs
+- Custom vision model config
+- GPT-5.6 Sol/Terra/Luna presets and pricing
+- GLM-5.3 model support
+- Global IM status-messages toggle (default off)
+- Extension-defined tools support; extensions can disable skills
+- Server restart state and HTTP restart flag
+- Actionable web search provider errors
+- Localized display names for broadcast skill commands
+
+### Improved
+- Skill autocomplete dropdown UI with localized source labels
+- Skill auto-creation and evolution reporting refinements
+
+### Fixed
+- MCP: preserve CRLF pairs split across SSE chunk boundaries (#489 - @MohammedAlkindi)
+- Feishu: auto-refresh token and retry when revoked (99991663) (#483 - @chengoak)
+- Preserve head+tail for oversized terminal output (#455 - @shipinliang)
+- openclacky start failed when no TTY
+- Capture master/worker crash logs under LaunchAgent
+- ReferenceError in model modal _syncRemarkField
+- Remark field always visible in model modal
+- api_format field only shown for custom providers and validated before mutating model config
+- Media gen output dir handling
+
 ## [1.5.9] - 2026-08-14
 
 ### Added

data/README.md

@@ -209,11 +209,11 @@ $ openclacky
 
 ## Star History
 
-<a href="https://www.star-history.com/?repos=clacky-ai%2Fopenclacky&type=date&legend=top-left">
+<a href="https://star-history.dera.page/#clacky-ai/openclacky&type=date&legend=top-left">
  <picture>
-   <source media="(prefers-color-scheme: dark)" srcset="https://api.star-history.com/chart?repos=clacky-ai/openclacky&type=date&theme=dark&legend=top-left" />
-   <source media="(prefers-color-scheme: light)" srcset="https://api.star-history.com/chart?repos=clacky-ai/openclacky&type=date&legend=top-left" />
-   <img alt="Star History Chart" src="https://api.star-history.com/chart?repos=clacky-ai/openclacky&type=date&legend=top-left" />
+   <source media="(prefers-color-scheme: dark)" srcset="https://star-history.dera.page/svg?repos=clacky-ai/openclacky&type=date&theme=dark&legend=top-left" />
+   <source media="(prefers-color-scheme: light)" srcset="https://star-history.dera.page/svg?repos=clacky-ai/openclacky&type=date&legend=top-left" />
+   <img alt="Star History Chart" src="https://star-history.dera.page/svg?repos=clacky-ai/openclacky&type=date&legend=top-left" />
  </picture>
 </a>
 

data/README_CN.md

@@ -204,11 +204,11 @@ $ openclacky
 
 ## Star 历史
 
-<a href="https://www.star-history.com/?repos=clacky-ai%2Fopenclacky&type=date&legend=top-left">
+<a href="https://star-history.dera.page/#clacky-ai/openclacky&type=date&legend=top-left">
  <picture>
-   <source media="(prefers-color-scheme: dark)" srcset="https://api.star-history.com/chart?repos=clacky-ai/openclacky&type=date&theme=dark&legend=top-left" />
-   <source media="(prefers-color-scheme: light)" srcset="https://api.star-history.com/chart?repos=clacky-ai/openclacky&type=date&legend=top-left" />
-   <img alt="Star History Chart" src="https://api.star-history.com/chart?repos=clacky-ai/openclacky&type=date&legend=top-left" />
+   <source media="(prefers-color-scheme: dark)" srcset="https://star-history.dera.page/svg?repos=clacky-ai/openclacky&type=date&theme=dark&legend=top-left" />
+   <source media="(prefers-color-scheme: light)" srcset="https://star-history.dera.page/svg?repos=clacky-ai/openclacky&type=date&legend=top-left" />
+   <img alt="Star History Chart" src="https://star-history.dera.page/svg?repos=clacky-ai/openclacky&type=date&legend=top-left" />
  </picture>
 </a>
 

data/README_JA.md

@@ -209,11 +209,11 @@ $ openclacky
 
 ## Star History
 
-<a href="https://www.star-history.com/?repos=clacky-ai%2Fopenclacky&type=date&legend=top-left">
+<a href="https://star-history.dera.page/#clacky-ai/openclacky&type=date&legend=top-left">
  <picture>
-   <source media="(prefers-color-scheme: dark)" srcset="https://api.star-history.com/chart?repos=clacky-ai/openclacky&type=date&theme=dark&legend=top-left" />
-   <source media="(prefers-color-scheme: light)" srcset="https://api.star-history.com/chart?repos=clacky-ai/openclacky&type=date&legend=top-left" />
-   <img alt="Star History Chart" src="https://api.star-history.com/chart?repos=clacky-ai/openclacky&type=date&legend=top-left" />
+   <source media="(prefers-color-scheme: dark)" srcset="https://star-history.dera.page/svg?repos=clacky-ai/openclacky&type=date&theme=dark&legend=top-left" />
+   <source media="(prefers-color-scheme: light)" srcset="https://star-history.dera.page/svg?repos=clacky-ai/openclacky&type=date&legend=top-left" />
+   <img alt="Star History Chart" src="https://star-history.dera.page/svg?repos=clacky-ai/openclacky&type=date&legend=top-left" />
  </picture>
 </a>
 

data/lib/clacky/agent/llm_caller.rb

@@ -95,6 +95,7 @@ module Clacky
 
         begin
           begin
+          Clacky::Shutdown.checkpoint!
           # Use active_messages (Time Machine) when undone, otherwise send full history.
           # to_api strips internal fields and handles orphaned tool_calls.
           messages_to_send = if respond_to?(:active_messages)
@@ -167,6 +168,7 @@ module Clacky
           end
 
         rescue Faraday::TimeoutError => e
+          Clacky::Shutdown.checkpoint!
           # Faraday::TimeoutError on our non-streaming POST almost always means
           # the *response* took longer than the 300s read-timeout to come back —
           # i.e. the model is trying to produce a huge output in one shot
@@ -198,7 +200,7 @@ module Clacky
               metadata: { attempt: retries, total: max_retries }
             )
             retrying_progress_opened = true
-            sleep retry_delay
+            Clacky::Shutdown.sleep(retry_delay)
             retry
           else
             # All retries exhausted for a timeout. Try the secondary gateway URL if available.
@@ -219,6 +221,7 @@ module Clacky
           end
 
         rescue Faraday::ConnectionFailed, Faraday::SSLError, Errno::ECONNREFUSED, Errno::ETIMEDOUT, Errno::EPIPE => e
+          Clacky::Shutdown.checkpoint!
           retries += 1
 
           # Errno::EPIPE means the underlying TCP socket is dead (server closed the
@@ -245,7 +248,7 @@ module Clacky
               metadata: { attempt: retries, total: max_retries }
             )
             retrying_progress_opened = true
-            sleep retry_delay
+            Clacky::Shutdown.sleep(retry_delay)
             retry
           else
             # All retries on the primary endpoint exhausted for a network-level
@@ -269,6 +272,7 @@ module Clacky
           end
 
         rescue RetryableError => e
+          Clacky::Shutdown.checkpoint!
           retries += 1
 
           # Probing failure: primary still down — renew cooling-off and retry with fallback.
@@ -298,7 +302,7 @@ module Clacky
             metadata: { attempt: retries, total: current_max }
           )
           retrying_progress_opened = true
-          sleep retry_delay
+          Clacky::Shutdown.sleep(retry_delay)
           retry
         else
           # All retries on the current endpoint exhausted.

data/lib/clacky/agent/memory_updater.rb

@@ -74,7 +74,7 @@ module Clacky
       private def with_memory_update_phase
         return yield unless @ui.respond_to?(:with_phase)
 
-        pid = @ui.phase_start(kind: "memory_update", label: "Updating long-term memory")
+        pid = @ui.phase_start(kind: "memory_update", label: Clacky::I18n.t("phase.memory_update"))
         summary = nil
         begin
           summary = yield
@@ -127,11 +127,6 @@ module Clacky
         # still reflects ONLY the user's task, not the memory update.
         subagent_cost = absorb_subagent_cost(result)
 
-        # A summary is only worth showing if the subagent actually wrote
-        # something. The common "No memory updates needed." path returns nil
-        # so the phase leaves no trace at all.
-        return unless subagent_wrote_memory?(subagent)
-
         "#{result[:iterations]} steps · $#{subagent_cost.round(4)}"
       end
 
@@ -142,43 +137,6 @@ module Clacky
         @config.memory_update_enabled != false
       end
 
-      # Inspect the subagent's history for a successful write/edit tool
-      # call targeting a memory file. Used to decide whether to surface a
-      # "Memory updated" info line (option C — silent when nothing changed).
-      # @param subagent [Clacky::Agent]
-      # @return [Boolean]
-      private def subagent_wrote_memory?(subagent)
-        return false unless subagent.respond_to?(:history) && subagent.history
-
-        subagent.history.to_a.any? do |msg|
-          next false unless msg.is_a?(Hash)
-
-          # Match OpenAI-style tool_calls on assistant messages …
-          tool_calls = msg[:tool_calls] || msg["tool_calls"]
-          if tool_calls.is_a?(Array) && tool_calls.any?
-            next true if tool_calls.any? do |tc|
-              name = tc.dig(:function, :name) || tc.dig("function", "name") || tc[:name] || tc["name"]
-              %w[write edit].include?(name.to_s)
-            end
-          end
-
-          # … and Anthropic-style content blocks with type=tool_use.
-          content = msg[:content] || msg["content"]
-          if content.is_a?(Array)
-            next true if content.any? do |block|
-              block.is_a?(Hash) &&
-                (block[:type] == "tool_use" || block["type"] == "tool_use") &&
-                %w[write edit].include?((block[:name] || block["name"]).to_s)
-            end
-          end
-
-          false
-        end
-      rescue StandardError
-        # Defensive: never let introspection errors break memory update.
-        false
-      end
-
       # Build the memory update prompt for the forked subagent.
       #
       # Architecture:

data/lib/clacky/agent/session_serializer.rb

@@ -392,7 +392,9 @@ module Clacky
           }
           all_files = image_files + disk_files
           ui.show_user_message(raw_text, created_at: msg[:created_at], files: all_files,
-                               editable: round[:editable] != false)
+                               editable: round[:editable] != false,
+                               skill_command: msg[:skill_command],
+                               skill_command_display: msg[:skill_command_display])
           replay_ext_events(msg, ui)
 
           round[:events].each do |ev|
@@ -740,16 +742,17 @@ module Clacky
             args_raw = tc[:arguments] || tc.dig(:function, :arguments) || {}
             args     = args_raw.is_a?(String) ? (JSON.parse(args_raw) rescue args_raw) : args_raw
 
-            # Special handling: request_user_feedback question is shown as an
-            # assistant message (matching real-time behavior), not as a tool call.
-            # Reconstruct the full formatted message including options (mirrors RequestUserFeedback#execute).
-            if name == "request_user_feedback"
-              question = args.is_a?(Hash) ? (args[:question] || args["question"]).to_s : ""
-              context  = args.is_a?(Hash) ? (args[:context]  || args["context"]).to_s  : ""
-              options  = args.is_a?(Hash) ? (args[:options]  || args["options"])        : nil
-              options  = Array(options) if options && !options.is_a?(Array)
+            # Special handling: the ask_user question is shown as an assistant
+            # message (matching real-time behavior), not as a tool call.
+            if Clacky::Tools::AskUser.feedback_tool?(name)
+              questions = Clacky::Tools::AskUser.normalize_questions(args)
+              context   = Clacky::Tools::AskUser.fetch_key(args, :context).to_s
+              first     = questions.first
 
-              ui.show_feedback_request(question, context, options || []) unless question.empty?
+              unless first.nil?
+                ui.show_feedback_request(first[:question], context, first[:options],
+                                         questions: questions)
+              end
             else
               ui.show_tool_call(name, args)
             end
@@ -794,14 +797,23 @@ module Clacky
         end
       end
 
-      # Replay a subagent transcript stored on a tool result message. Emits a
+      # Replay subagent transcripts stored on a tool result message. Emits a
       # bracketed sequence of UI events the frontend can render as a collapsible
       # sub-process block: subagent_start → (assistant_message / tool_call /
-      # tool_result)* → subagent_end. No-op when the message carries no transcript.
+      # tool_result)* → subagent_end. A fan-out batch renders one block per job,
+      # in the caller's original job order.
+      #
+      # Sessions saved before fan-out support stored a bare Hash here.
       def replay_subagent_transcript(msg, ui)
-        transcript = msg[:subagent_transcript]
-        return unless transcript.is_a?(Hash)
+        stored = msg[:subagent_transcript]
+        Array(stored.is_a?(Hash) ? [stored] : stored).each do |transcript|
+          next unless transcript.is_a?(Hash)
+
+          replay_one_subagent_transcript(transcript, ui)
+        end
+      end
 
+      def replay_one_subagent_transcript(transcript, ui)
         events = transcript[:events] || transcript["events"] || []
         return if events.empty?
 

data/lib/clacky/agent/skill_auto_creator.rb

@@ -22,8 +22,10 @@ module Clacky
 
         # Fork an isolated subagent to evaluate + create — does NOT touch main history
         subagent = fork_subagent
-        subagent.run(build_skill_creation_prompt)
-        nil
+        result = subagent.run(build_skill_creation_prompt)
+        cost = absorb_subagent_cost(result)
+
+        "$#{cost.round(4)}"
       end
 
       # Determine if this task is a candidate for skill auto-creation
@@ -84,19 +86,20 @@ module Clacky
           ## Action
 
           If **ALL** criteria are met:
-            → Call invoke_skill with:
+            → Call invoke_skill with EXACTLY these two parameters:
                - skill_name: "skill-creator"
-               - task: A clear description of what to automate and how (be specific)
-               - mode: "quick" (enables fast auto-creation without user interviews)
-               - suggested_name: A descriptive identifier (lowercase, hyphens OK)
+               - task: A clear description of what to automate and how (be specific).
+                 Start it with "Quick mode:" and state the suggested skill name
+                 (lowercase, hyphens OK) inside this text.
+
+          invoke_skill accepts ONLY skill_name and task. Passing any other
+          parameter raises an ArgumentError and the creation fails.
 
           Example invocation:
           ```
           invoke_skill(
             skill_name: "skill-creator",
-            task: "Create a skill to extract and summarize content from URLs. The skill should: 1) fetch the URL content, 2) parse the main text, 3) generate a concise summary. Expected input: URL. Expected output: markdown summary.",
-            mode: "quick",
-            suggested_name: "url-summarizer"
+            task: "Quick mode: create a skill named 'url-summarizer' (no user interviews). It should extract and summarize content from URLs: 1) fetch the URL content, 2) parse the main text, 3) generate a concise summary. Expected input: URL. Expected output: markdown summary."
           )
           ```
 

data/lib/clacky/agent/skill_evolution.rb

@@ -26,11 +26,12 @@ module Clacky
       def run_skill_evolution_hooks
         return unless skill_evolution_enabled?
         return if @is_subagent
+        # Preconditions unmet (short task, default skill, ...): the evolution
+        # never ran, so there is nothing to report — stay completely silent.
+        return unless skill_evolution_has_work?
 
         with_skill_evolution_phase do
-          if !skill_evolution_has_work?
-            Clacky::I18n.t("phase.skipped")
-          elsif @skill_execution_context
+          if @skill_execution_context
             maybe_reflect_on_skill
           else
             maybe_create_skill_from_task

data/lib/clacky/agent/skill_manager.rb

@@ -82,7 +82,7 @@ module Clacky
           return { matched: true, found: false, skill_name: skill_name, reason: :not_user_invocable, skill: skill }
         end
 
-        if @agent_profile && !skill.allowed_for_agent?(@agent_profile.name)
+        if @agent_profile && !@agent_profile.skill_allowed?(skill)
           return { matched: true, found: false, skill_name: skill_name, reason: :agent_not_allowed, skill: skill }
         end
 
@@ -266,22 +266,20 @@ module Clacky
       # is injected instructing the LLM to inform the user in their own language and
       # suggest similar skills — no error is raised, the LLM handles the reply.
       #
-      # @param user_input [String] Raw user input
+      # @param skill_command [Hash] Result of parse_skill_command for the user input
       # @param task_id [Integer] Current task ID (for message tagging)
       # @return [void]
-      def inject_skill_command_as_assistant_message(user_input, task_id)
-        result = parse_skill_command(user_input)
-
+      def inject_skill_command_as_assistant_message(skill_command, task_id)
         # Not a slash command at all — nothing to do
-        return unless result[:matched]
+        return unless skill_command[:matched]
 
-        skill_name = result[:skill_name]
+        skill_name = skill_command[:skill_name]
 
         # Slash command recognised but skill could not be dispatched — inject an
         # LLM-facing notice so the model explains the situation to the user in
         # their own language instead of silently ignoring the command.
-        unless result[:found]
-          notice = case result[:reason]
+        unless skill_command[:found]
+          notice = case skill_command[:reason]
           when :not_found
             suggestions = suggest_similar_skills(skill_name)
             msg = "[SYSTEM] The user entered the slash command /#{skill_name} but no matching skill was found. " \
@@ -302,8 +300,8 @@ module Clacky
           return
         end
 
-        skill     = result[:skill]
-        arguments = result[:arguments]
+        skill     = skill_command[:skill]
+        arguments = skill_command[:arguments]
 
         # fork_agent skills run in an isolated subagent
         if skill.fork_agent?
@@ -443,7 +441,7 @@ module Clacky
       def filter_skills_by_profile(skills)
         return skills unless @agent_profile
 
-        skills.select { |skill| skill.allowed_for_agent?(@agent_profile.name) }
+        skills.select { |skill| @agent_profile.skill_allowed?(skill) }
       end
 
       # Build template context for skill content expansion.
@@ -653,8 +651,7 @@ module Clacky
         # observe() attaches each trail to its own tool result message.
         # Kept out of the LLM payload via INTERNAL_FIELDS.
         if tool_call_id
-          @pending_subagent_transcripts[tool_call_id] =
-            extract_subagent_transcript(subagent, skill.identifier)
+          record_subagent_transcript(tool_call_id, subagent, skill.identifier)
         end
 
         # Mutate the subagent_instructions message in-place to become the result summary

data/lib/clacky/agent/skill_reflector.rb

@@ -26,11 +26,11 @@ module Clacky
         subagent = fork_subagent
         result = subagent.run(build_skill_reflection_prompt(skill_name))
 
-        cost = absorb_subagent_cost(result) if result
+        cost = absorb_subagent_cost(result)
 
         @skill_execution_context = nil
 
-        "#{skill_name} · $#{cost.round(4)}" if cost
+        "#{skill_name} · $#{cost.round(4)}"
       end
 
       private def should_reflect_on_skill?

data/lib/clacky/agent/tool_executor.rb

@@ -13,7 +13,7 @@ module Clacky
         case @config.permission_mode
         when :auto_approve, :confirm_all
           # Both modes auto-execute all file/shell tools without confirmation.
-          # The difference is only in request_user_feedback handling:
+          # The difference is only in ask_user handling:
           #   auto_approve → no human present, inject auto_reply
           #   confirm_all  → human present, truly wait for user input
           true

data/lib/clacky/agent/tool_registry.rb

@@ -55,10 +55,12 @@ module Clacky
       # todo_manager aliases
       "todo" => "todo_manager",
       "task_manager" => "todo_manager",
-      # request_user_feedback aliases
-      "ask_user" => "request_user_feedback",
-      "user_feedback" => "request_user_feedback",
-      "ask" => "request_user_feedback",
+      # ask_user aliases
+      "request_user_feedback" => "ask_user",
+      "user_feedback" => "ask_user",
+      "ask" => "ask_user",
+      "ask_question" => "ask_user",
+      "clarify" => "ask_user",
       # trash_manager aliases
       "trash" => "trash_manager",
       "delete" => "trash_manager",