openclacky 0.9.32 → 0.9.33

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: a4dd6332b6e7425bea0dd817603ad5af83e4d23b5742b79f5ca97f2d0fc18a0c
-  data.tar.gz: 640788854a81c8760999e866dce8364c6e2547603098f24582f6b6b51837797b
+  metadata.gz: c1e2a5b101fac2a1079e97a31598bdb183dc9649714bc54a5f2f949688a5d70a
+  data.tar.gz: c30e593f71f54293dec3f0b862b8fd77536a07a101b04acdec329282c3b91255
 SHA512:
-  metadata.gz: 96423895e7df89b17c5eb7196aca0f829e1f5544c14def222d888faddb35a39c78f3df1bdea40d3ce884c9d3edf7b3a4ac8f8447e5ab7394e569ed9d9ffd8038
-  data.tar.gz: 2f85e85244ddfa8720a9c2cf6fc053d68671e051ce0d2ccafbbb01581b3fc460e1c7cd10c4a05dc93a88614c3f6505e61aaa4510f821a283a39974069f2694a6
+  metadata.gz: 2574bc424afc1137f366df2eb14d8e27b54272713329d8bfb18daea5901208ef0878e8e23c0c21a22a0c90920fd07d851fb1cba14969af7eb8a4179fae124b30
+  data.tar.gz: 3860a3573c05cd0824029363bd9ea13c3c2e11c46ed9679231a9dd370a98630923d043cc2f83df00ec682ef5dbb5f2361b3acb24862a8129c2166ba4a5cdfcba

data/CHANGELOG.md

@@ -7,6 +7,13 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+## [0.9.33] - 2026-04-20
+
+### Fixed
+- **Skill evolution targets only user skills**: auto-evolution (skill auto-creation and skill reflection) now skips default and brand skills — only user-created skills in `~/.clacky/skills/` or `.clacky/skills/` are eligible for improvement
+- **Skill auto-creation and reflection run in isolated subagents**: these background analysis tasks no longer inject messages into the main conversation history; they now fork a dedicated subagent that runs fully independently, preventing any interference with the current session
+- **User feedback prompt no longer interrupts agent flow**: removed stray `STOP.` prefix from the in-conversation user-feedback message, allowing the agent to handle feedback naturally without halting unexpectedly
+
 ## [0.9.32] - 2026-04-20
 
 ### Added

data/lib/clacky/agent/skill_auto_creator.rb

@@ -5,10 +5,10 @@ module Clacky
     # Scenario 1: Auto-create new skills from complex task patterns.
     #
     # After completing a complex task (high iteration count, no existing skill used),
-    # inject a system prompt asking the LLM to analyze if the workflow is reusable
-    # and worth capturing as a new skill.
+    # forks a subagent to analyze if the workflow is reusable and worth capturing
+    # as a new skill.
     #
-    # If the LLM determines it's valuable, it can invoke skill-creator in "quick mode"
+    # If the LLM determines it's valuable, it invokes skill-creator in "quick mode"
     # to generate a new skill automatically.
     module SkillAutoCreator
       # Default minimum iterations to consider auto-creating a skill
@@ -19,7 +19,11 @@ module Clacky
       def maybe_create_skill_from_task
         return unless should_auto_create_skill?
 
-        inject_skill_creation_prompt
+        @ui&.show_info("Analyzing task for skill creation opportunity...")
+
+        # Fork an isolated subagent to evaluate + create — does NOT touch main history
+        subagent = fork_subagent
+        subagent.run(build_skill_creation_prompt)
       end
 
       # Determine if this task is a candidate for skill auto-creation
@@ -47,19 +51,6 @@ module Clacky
         }
       end
 
-      # Inject skill creation prompt as a system message
-      # The LLM will analyze and decide whether to create a new skill
-      private def inject_skill_creation_prompt
-        @history.append({
-          role: "user",
-          content: build_skill_creation_prompt,
-          system_injected: true,
-          skill_auto_create: true
-        })
-
-        @ui&.show_info("Analyzing task for skill creation opportunity...")
-      end
-
       # Build the skill auto-creation prompt content
       # @return [String]
       private def build_skill_creation_prompt

data/lib/clacky/agent/skill_manager.rb

@@ -205,7 +205,8 @@ module Clacky
           skill_name: skill.identifier,
           start_iteration: @iterations,
           arguments: arguments,
-          slash_command: slash_command
+          slash_command: slash_command,
+          source: skill.source
         }
 
         # For encrypted brand skills with supporting scripts: decrypt to a tmpdir so the

data/lib/clacky/agent/skill_reflector.rb

@@ -4,12 +4,12 @@ module Clacky
   class Agent
     # Scenario 2: Reflect on skill execution and suggest improvements.
     #
-    # After a skill completes, inject a system prompt asking the LLM to analyze:
+    # After a skill completes, forks a subagent to analyze:
     #   - Were instructions clear enough?
     #   - Any missing edge cases?
     #   - Any improvements needed?
     #
-    # If the LLM identifies concrete improvements, it can invoke skill-creator
+    # If the LLM identifies concrete improvements, it invokes skill-creator
     # to update the skill.
     module SkillReflector
       # Minimum iterations for a skill execution to warrant reflection.
@@ -27,6 +27,11 @@ module Clacky
         # platform-management skills invoked incidentally should not be reflected on.
         return unless @skill_execution_context[:slash_command]
 
+        # Skip default and brand skills — they are system-owned and should not be
+        # auto-improved by the evolution system.
+        source = @skill_execution_context[:source]
+        return if source == :default || source == :brand
+
         skill_name = @skill_execution_context[:skill_name]
         start_iteration = @skill_execution_context[:start_iteration]
         iterations = @iterations - start_iteration
@@ -34,28 +39,15 @@ module Clacky
         # Only reflect if the skill actually ran for a meaningful number of iterations
         return if iterations < MIN_SKILL_ITERATIONS
 
-        inject_skill_reflection_prompt(skill_name, iterations)
+        # Fork an isolated subagent to reflect + improve — does NOT touch main history
+        @ui&.show_info("Reflecting on skill execution: #{skill_name}")
+        subagent = fork_subagent
+        subagent.run(build_skill_reflection_prompt(skill_name, iterations))
 
         # Clear the context so we don't reflect again
         @skill_execution_context = nil
       end
 
-      # Inject reflection prompt into history as a system message
-      # The LLM will respond in the next user interaction (non-blocking)
-      #
-      # @param skill_name [String] Identifier of the skill that was executed
-      # @param iterations [Integer] Number of iterations the skill ran for
-      private def inject_skill_reflection_prompt(skill_name, iterations)
-        @history.append({
-          role: "user",
-          content: build_skill_reflection_prompt(skill_name, iterations),
-          system_injected: true,
-          skill_reflection: true
-        })
-
-        @ui&.show_info("Reflecting on skill execution: #{skill_name}")
-      end
-
       # Build the reflection prompt content
       # @param skill_name [String]
       # @param iterations [Integer]

data/lib/clacky/agent.rb

@@ -395,7 +395,7 @@ module Clacky
               # Add user feedback as a new user message with system_injected marker
               @history.append({
                 role: "user",
-                content: "STOP. The user has a question/feedback for you: #{action_result[:feedback]}\n\nPlease respond to the user's question/feedback before continuing with any actions.",
+                content: "The user has a question/feedback for you: #{action_result[:feedback]}\n\nPlease respond to the user's question/feedback before continuing with any actions.",
                 system_injected: true
               })
               # Continue loop to let agent respond to feedback

data/lib/clacky/skill.rb

@@ -36,6 +36,11 @@ module Clacky
     attr_reader :fork_agent, :model, :forbidden_tools, :auto_summarize
     attr_reader :brand_skill, :brand_config
 
+    # Source location of this skill — set by SkillLoader after registration.
+    # One of: :default, :global_claude, :global_clacky, :project_claude, :project_clacky, :brand
+    # @return [Symbol, nil]
+    attr_accessor :source
+
     # Warnings accumulated during load (e.g. name was invalid and fell back to dir name).
     # Non-empty means the skill loaded but something was auto-corrected.
     # @return [Array<String>]

data/lib/clacky/skill_loader.rb

@@ -433,6 +433,7 @@ module Clacky
 
       @skills[id]        = skill
       @loaded_from[id]   = source
+      skill.source       = source
 
       # Invalid skills have no usable slug — skip slash command registration but
       # still keep them in @skills so they appear (greyed-out) in the UI.
@@ -466,16 +467,7 @@ module Clacky
 
         begin
           skill = Skill.new(Pathname.new(skill_dir))
-
-          # Check for duplicates (higher priority skills override)
-          if @skills.key?(skill.identifier)
-            next  # Skip if already loaded from higher priority location
-          end
-
-          # Register skill
-          @skills[skill.identifier] = skill
-          @skills_by_command[skill.slash_command] = skill
-          @loaded_from[skill.identifier] = :default
+          register_skill(skill, source: :default)
         rescue StandardError => e
           @errors << "Failed to load default skill #{skill_name}: #{e.message}"
         end

data/lib/clacky/version.rb

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

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: openclacky
 version: !ruby/object:Gem::Version
-  version: 0.9.32
+  version: 0.9.33
 platform: ruby
 authors:
 - windy