openclacky 0.7.7 → 0.7.8

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 5c3815310a11bea9ccad7ab6891b4ad85e248635af6919606e8963096699a1d7
-  data.tar.gz: 17f8059179701394a6951966ec0284ee6321750e2651c4de0ecbf37a4542c7fc
+  metadata.gz: 9b2a72212b8b9e8072acb09a055df1afdd8225ee81de41f11dce460a68ff1781
+  data.tar.gz: e8048bb38dc4a8ae8c85937165e10a855f8642d6759c8a2a94aa336b084e4188
 SHA512:
-  metadata.gz: b7b32ce62e12f9aa12db59b1f6640e4a6234c34c485d9b679df80654dc8722a369687e1fc0b5165a78a12f6210fcd02959133d6d3753c5c22873d071251b07fb
-  data.tar.gz: 20f4498c976579333b7e261490d287a33cec89b888c5ae1a640205c25f48c70c3863a551e1bef6173804b7670970ead88ed9a67702987da97963c0a981cd1e2a
+  metadata.gz: '095710e060b7c29882eb128be1a4008281435c13af07e7da01ec9ad1af27c30e252eb19e77f557882628409058d504f29024ea0dbb396ed20d326b26cfe1925c'
+  data.tar.gz: 2875ac5911322341be5fe396ad44a98e4cbcea51c95ed7d832e3978d5790542ca94ae0f065f6170484d9900be2f1ca190f6db0d25c14a33366f924f9d13a147d

data/.clacky/skills/commit/SKILL.md

@@ -1,7 +1,7 @@
 ---
+---
 name: commit
 description: Smart Git commit helper that analyzes changes and creates semantic commits
-disable-model-invocation: false
 user-invocable: true
 ---
 
@@ -472,4 +472,4 @@ This skill works best:
 
 - Created: 2025-02-01
 - Purpose: Improve commit quality and development workflow
-- Compatible with: Any git repository
+- Compatible with: Any git repository

data/CHANGELOG.md

@@ -7,6 +7,27 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+## [0.7.8] - 2026-03-06
+
+### Added
+- Skills panel in web UI: list all skills, enable/disable with toggle, view skill details
+- Hash-based routing (`#session/:id`, `#tasks`, `#skills`, `#settings`) with deep-link and refresh support
+- REST API endpoints for skills management (`GET /api/skills`, `PATCH /api/skills/:name/toggle`)
+- `disabled?` helper on `Skill` model for quick enabled/disabled state checks
+
+### Improved
+- Centralized `Router` object in web UI — single source of truth for all panel switching and sidebar highlight state
+- Web UI frontend split further: `skills.js` extracted as standalone module
+- Ctrl-C in web server now exits immediately via `StartCallback` trap override
+- Skill enable/disable now writes `disable-model-invocation: false` (retains field) instead of deleting it
+
+### Fixed
+- Sidebar highlight for Tasks and Skills stuck active after navigating away
+- Router correctly restores last view on page refresh via hash URL
+
+### Changed
+- Removed `plan_only` permission mode from agent, CLI, and web UI
+
 ## [0.7.7] - 2026-03-04
 
 ### Added

data/lib/clacky/agent/tool_executor.rb

@@ -16,8 +16,6 @@ module Clacky
         when :confirm_safes
           # Use SafeShell integration for safety check
           is_safe_operation?(tool_name, tool_params)
-        when :plan_only
-          false
         else
           false
         end
@@ -217,16 +215,6 @@ module Clacky
         }
       end
 
-      # Build planned result for plan-only mode
-      # @param call [Hash] Tool call
-      # @return [Hash] Formatted planned result
-      def build_planned_result(call)
-        {
-          id: call[:id],
-          content: JSON.generate({ planned: true, message: "Tool execution skipped (plan mode)" })
-        }
-      end
-
       # Check if a tool is potentially slow and should show progress
       # @param tool_name [String] Name of the tool
       # @param args [Hash] Tool arguments

data/lib/clacky/agent.rb

@@ -389,12 +389,6 @@ module Clacky
           end
         else
           # Permission check (if not in auto-approve mode)
-          if @config.is_plan_only?
-            @ui&.show_info("Planned: #{call[:name]}")
-            results << build_planned_result(call)
-            next
-          end
-
           confirmation = confirm_tool_use?(call)
           unless confirmation[:approved]
             # Show denial warning only for user-initiated denials (not system-injected preview errors)

data/lib/clacky/agent_config.rb

@@ -148,7 +148,7 @@ module Clacky
     # Default model for ClaudeCode environment
     CLAUDE_DEFAULT_MODEL = "claude-sonnet-4-5"
 
-    PERMISSION_MODES = [:auto_approve, :confirm_safes, :plan_only].freeze
+    PERMISSION_MODES = [:auto_approve, :confirm_safes].freeze
 
     attr_accessor :permission_mode, :max_tokens, :verbose,
                   :enable_compression, :enable_prompt_caching,
@@ -397,10 +397,6 @@ module Clacky
       true
     end
 
-    def is_plan_only?
-      @permission_mode == :plan_only
-    end
-
     private def validate_permission_mode(mode)
       mode ||= :confirm_safes
       mode = mode.to_sym

data/lib/clacky/cli.rb

@@ -27,7 +27,6 @@ module Clacky
       Permission modes:
         auto_approve    - Automatically execute all tools (use with caution)
         confirm_safes   - Auto-approve safe operations, confirm risky ones (default)
-        plan_only       - Generate plan without executing
 
       UI themes:
         hacker          - Matrix/hacker-style with bracket symbols (default)
@@ -42,7 +41,7 @@ module Clacky
         $ clacky agent --mode=auto_approve --path /path/to/project
     LONGDESC
     option :mode, type: :string, default: "confirm_safes",
-           desc: "Permission mode: auto_approve, confirm_safes, plan_only"
+           desc: "Permission mode: auto_approve, confirm_safes"
     option :theme, type: :string, default: "hacker",
            desc: "UI theme: hacker, minimal (default: hacker)"
     option :verbose, type: :boolean, aliases: "-v", default: false, desc: "Show detailed output"

data/lib/clacky/server/http_server.rb

@@ -33,14 +33,19 @@ module Clacky
           session_registry: @registry,
           session_builder:  method(:build_session)
         )
+        @skill_loader   = Clacky::SkillLoader.new
       end
 
       def start
+        # Override WEBrick's built-in signal traps via StartCallback,
+        # which fires after WEBrick sets its own INT/TERM handlers.
+        # This ensures Ctrl-C always exits immediately.
         server = WEBrick::HTTPServer.new(
           BindAddress:     @host,
           Port:            @port,
           Logger:          WEBrick::Log.new(File::NULL),
-          AccessLog:       []
+          AccessLog:       [],
+          StartCallback:   proc { trap("INT") { exit(0) }; trap("TERM") { exit(0) } }
         )
 
         # Mount API + WebSocket handler (takes priority).
@@ -69,10 +74,6 @@ module Clacky
           res["Pragma"]        = "no-cache"
         end
 
-        # Graceful shutdown on Ctrl-C
-        trap("INT")  { @scheduler.stop; server.shutdown }
-        trap("TERM") { @scheduler.stop; server.shutdown }
-
         puts "🌐 Clacky Web UI running at http://#{@host}:#{@port}"
         puts "   Press Ctrl-C to stop."
 
@@ -108,6 +109,7 @@ module Clacky
         when ["GET",    "/api/tasks"]         then api_list_tasks(res)
         when ["POST",   "/api/tasks"]         then api_create_task(req, res)
         when ["POST",   "/api/tasks/run"]     then api_run_task(req, res)
+        when ["GET",    "/api/skills"]         then api_list_skills(res)
         when ["GET",    "/api/config"]        then api_get_config(res)
         when ["POST",   "/api/config"]        then api_save_config(req, res)
         when ["POST",   "/api/config/test"]   then api_test_config(req, res)
@@ -125,6 +127,9 @@ module Clacky
           elsif method == "DELETE" && path.start_with?("/api/tasks/")
             name = URI.decode_www_form_component(path.sub("/api/tasks/", ""))
             api_delete_task(name, res)
+          elsif method == "PATCH" && path.match?(%r{^/api/skills/[^/]+/toggle$})
+            name = URI.decode_www_form_component(path.sub("/api/skills/", "").sub("/toggle", ""))
+            api_toggle_skill(name, req, res)
           else
             not_found(res)
           end
@@ -138,15 +143,12 @@ module Clacky
       end
 
       def api_create_session(req, res)
-        body = parse_json_body(req)
+        body        = parse_json_body(req)
         name        = body["name"]
-        working_dir = default_working_dir
+        working_dir = body["working_dir"]&.then { |d| File.expand_path(d) } || default_working_dir
 
-        # Validate working directory
-        unless Dir.exist?(working_dir)
-          json_response(res, 422, { error: "Directory does not exist: #{working_dir}" })
-          return
-        end
+        # Auto-create the working directory if it does not exist yet
+        FileUtils.mkdir_p(working_dir)
 
         session_id = build_session(name: name, working_dir: working_dir)
         json_response(res, 201, { session: @registry.list.find { |s| s[:id] == session_id } })
@@ -266,6 +268,40 @@ module Clacky
         end
       end
 
+      # ── Skills API ────────────────────────────────────────────────────────────
+
+      # GET /api/skills — list all loaded skills with metadata
+      def api_list_skills(res)
+        @skill_loader.load_all  # refresh from disk on each request
+        skills = @skill_loader.all_skills.map do |skill|
+          source = @skill_loader.loaded_from[skill.identifier]
+          {
+            name:        skill.identifier,
+            description: skill.context_description,
+            source:      source,
+            enabled:     !skill.disabled?
+          }
+        end
+        json_response(res, 200, { skills: skills })
+      end
+
+      # PATCH /api/skills/:name/toggle — enable or disable a skill
+      # Body: { enabled: true/false }
+      def api_toggle_skill(name, req, res)
+        body    = parse_json_body(req)
+        enabled = body["enabled"]
+
+        if enabled.nil?
+          json_response(res, 422, { error: "enabled field required" })
+          return
+        end
+
+        skill = @skill_loader.toggle_skill(name, enabled: enabled)
+        json_response(res, 200, { ok: true, name: skill.identifier, enabled: !skill.disabled? })
+      rescue Clacky::AgentError => e
+        json_response(res, 422, { error: e.message })
+      end
+
       # ── Config API ────────────────────────────────────────────────────────────
 
       # GET /api/config — return current model configurations
@@ -529,7 +565,6 @@ module Clacky
           broadcast_session_update(session_id)
           broadcast(session_id, { type: "error", session_id: session_id, message: e.message })
         end
-
         @registry.with_session(session_id) { |s| s[:thread] = thread }
       end
 
@@ -579,7 +614,6 @@ module Clacky
           broadcast_session_update(session_id)
           broadcast(session_id, { type: "error", session_id: session_id, message: e.message })
         end
-
         @registry.with_session(session_id) { |s| s[:thread] = thread }
       end
 

data/lib/clacky/server/scheduler.rb

@@ -158,7 +158,7 @@ module Clacky
 
         $stdout.puts "[Clacky Scheduler] Firing task '#{task_name}' (session: #{session_id})"
 
-        # Run the agent in a background thread so the scheduler tick is non-blocking
+        # Run the agent in a background thread so the scheduler tick is non-blocking.
         Thread.new do
           session = @registry.get(session_id)
           agent   = nil
@@ -174,6 +174,7 @@ module Clacky
           @registry.update(session_id, status: :error, error: e.message)
           $stderr.puts "[Clacky Scheduler] Task '#{task_name}' failed: #{e.message}"
         end
+
       rescue => e
         $stderr.puts "[Clacky Scheduler] Failed to fire task '#{schedule["task"]}': #{e.message}"
       end

data/lib/clacky/skill.rb

@@ -30,6 +30,13 @@ module Clacky
     attr_reader :allowed_tools, :context, :agent_type, :argument_hint, :hooks
     attr_reader :fork_agent, :model, :forbidden_tools, :auto_summarize
 
+    # Check if this skill is disabled (disable-model-invocation: true)
+    # @return [Boolean]
+    def disabled?
+      @disable_model_invocation == true
+    end
+
+
     # @param directory [Pathname, String] Path to the skill directory
     # @param source_path [Pathname, String, nil] Optional source path for priority resolution
     def initialize(directory, source_path: nil)

data/lib/clacky/skill_loader.rb

@@ -208,6 +208,34 @@ module Clacky
       load_single_skill(skill_dir, skill_dir, name, source_type)
     end
 
+    # Toggle a skill's disable-model-invocation field in its SKILL.md.
+    # System skills (source: :default) cannot be toggled — raises AgentError.
+    # @param name [String] Skill identifier
+    # @param enabled [Boolean] true = enable, false = disable
+    # @return [Skill] The reloaded skill
+    def toggle_skill(name, enabled:)
+      skill = @skills[name]
+      raise Clacky::AgentError, "Skill not found: #{name}" unless skill
+      raise Clacky::AgentError, "Cannot toggle system skill: #{name}" if @loaded_from[name] == :default
+
+      skill_file = skill.directory.join("SKILL.md")
+      fm = (skill.frontmatter || {}).dup
+
+      if enabled
+        fm["disable-model-invocation"] = false
+      else
+        fm["disable-model-invocation"] = true
+      end
+
+      skill_file.write(build_skill_content(fm, skill.content))
+
+      # Reload into registry
+      reloaded = Skill.new(skill.directory, source_path: skill.source_path)
+      @skills[reloaded.identifier] = reloaded
+      @skills_by_command[reloaded.slash_command] = reloaded
+      reloaded
+    end
+
     # Delete a skill
     # @param name [String] Skill name
     # @return [Boolean] True if deleted, false if not found

data/lib/clacky/ui2/components/input_area.rb

@@ -1232,8 +1232,6 @@ module Clacky
             :magenta
           when /confirm_safes/
             :cyan
-          when /plan_only/
-            :blue
           else
             :white
           end

data/lib/clacky/version.rb

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

data/lib/clacky/web/app.css

@@ -779,6 +779,248 @@ body {
 .form-textarea:focus { outline: none; border-color: #58a6ff; }
 .form-hint { font-size: 11px; color: #8b949e; margin-top: 4px; }
 
+/* ── Skills Panel ────────────────────────────────────────────────────────── */
+#skills-panel {
+  display: flex;
+  flex-direction: column;
+  height: 100%;
+  overflow: hidden;
+}
+#skills-header {
+  padding: 12px 20px;
+  border-bottom: 1px solid #30363d;
+  display: flex;
+  align-items: center;
+  justify-content: space-between;
+  background: #161b22;
+  flex-shrink: 0;
+  font-weight: 600;
+  font-size: 15px;
+}
+#btn-create-skill {
+  padding: 5px 12px;
+  background: #238636;
+  color: #fff;
+  border: none;
+  border-radius: 6px;
+  font-size: 13px;
+  cursor: pointer;
+  white-space: nowrap;
+}
+#btn-create-skill:hover { background: #2ea043; }
+#skills-body {
+  flex: 1;
+  overflow: hidden;
+  display: flex;
+  flex-direction: column;
+  padding: 20px 24px;
+  gap: 16px;
+}
+
+/* ── Skills Tabs ─────────────────────────────────────────────────────────── */
+#skills-tabs {
+  display: flex;
+  gap: 4px;
+  border-bottom: 1px solid #30363d;
+  flex-shrink: 0;
+}
+.skills-tab {
+  background: none;
+  border: none;
+  border-bottom: 2px solid transparent;
+  color: #8b949e;
+  cursor: pointer;
+  font-size: 13px;
+  font-weight: 500;
+  padding: 8px 16px;
+  margin-bottom: -1px;
+  transition: color .15s, border-color .15s;
+}
+.skills-tab:hover  { color: #e6edf3; }
+.skills-tab.active { color: #58a6ff; border-bottom-color: #58a6ff; }
+
+.skills-tab-content {
+  flex: 1;
+  overflow-y: auto;
+}
+
+/* ── My Skills list ──────────────────────────────────────────────────────── */
+#skills-list {
+  display: flex;
+  flex-direction: column;
+  gap: 8px;
+  padding-top: 12px;
+}
+.skills-empty {
+  color: #8b949e;
+  font-size: 13px;
+  padding: 20px 0;
+  text-align: center;
+}
+
+/* ── Skill Card ──────────────────────────────────────────────────────────── */
+.skill-card {
+  background: #161b22;
+  border: 1px solid #30363d;
+  border-radius: 8px;
+  padding: 14px 16px;
+  transition: border-color .15s;
+}
+.skill-card:hover { border-color: #484f58; }
+.skill-card-main {
+  display: flex;
+  align-items: center;
+  gap: 12px;
+}
+.skill-card-info { flex: 1; min-width: 0; }
+.skill-card-title {
+  display: flex;
+  align-items: center;
+  gap: 8px;
+  margin-bottom: 4px;
+}
+.skill-name {
+  font-size: 13px;
+  font-weight: 600;
+  color: #e6edf3;
+}
+.skill-card-desc {
+  font-size: 12px;
+  color: #8b949e;
+  line-height: 1.5;
+  white-space: nowrap;
+  overflow: hidden;
+  text-overflow: ellipsis;
+}
+
+/* ── Skill Badges ────────────────────────────────────────────────────────── */
+.skill-badge {
+  font-size: 10px;
+  font-weight: 600;
+  padding: 2px 7px;
+  border-radius: 10px;
+  letter-spacing: 0.3px;
+  flex-shrink: 0;
+}
+.skill-badge-system {
+  background: #1f3a5f;
+  color: #79c0ff;
+  border: 1px solid #1d4070;
+}
+.skill-badge-custom {
+  background: #1f3024;
+  color: #56d364;
+  border: 1px solid #1c4028;
+}
+
+/* ── Toggle Switch ───────────────────────────────────────────────────────── */
+.skill-toggle {
+  position: relative;
+  display: inline-flex;
+  align-items: center;
+  cursor: pointer;
+  flex-shrink: 0;
+}
+.skill-toggle-disabled {
+  cursor: not-allowed;
+  opacity: 0.4;
+}
+.skill-toggle-input {
+  opacity: 0;
+  width: 0;
+  height: 0;
+  position: absolute;
+}
+.skill-toggle-track {
+  display: inline-block;
+  width: 36px;
+  height: 20px;
+  background: #30363d;
+  border-radius: 10px;
+  transition: background .2s;
+  position: relative;
+}
+.skill-toggle-track::after {
+  content: "";
+  position: absolute;
+  width: 14px;
+  height: 14px;
+  background: #fff;
+  border-radius: 50%;
+  top: 3px;
+  left: 3px;
+  transition: left .2s;
+}
+.skill-toggle-input:checked + .skill-toggle-track {
+  background: #238636;
+}
+.skill-toggle-input:checked + .skill-toggle-track::after {
+  left: 19px;
+}
+
+/* ── Store Grid ──────────────────────────────────────────────────────────── */
+#skills-store-grid {
+  display: flex;
+  flex-direction: column;
+  gap: 10px;
+  padding-top: 12px;
+}
+.store-card {
+  background: #161b22;
+  border: 1px solid #30363d;
+  border-radius: 8px;
+  padding: 16px;
+  display: flex;
+  align-items: center;
+  gap: 14px;
+  transition: border-color .15s;
+}
+.store-card:hover { border-color: #484f58; }
+.store-card-icon {
+  font-size: 28px;
+  flex-shrink: 0;
+  width: 44px;
+  height: 44px;
+  display: flex;
+  align-items: center;
+  justify-content: center;
+  background: #0d1117;
+  border-radius: 8px;
+}
+.store-card-body { flex: 1; min-width: 0; }
+.store-card-title {
+  font-size: 13px;
+  font-weight: 600;
+  color: #e6edf3;
+  margin-bottom: 4px;
+}
+.store-card-desc {
+  font-size: 12px;
+  color: #8b949e;
+  line-height: 1.5;
+}
+.store-card-actions { flex-shrink: 0; }
+.btn-store-install {
+  background: #238636;
+  color: #fff;
+  border: none;
+  border-radius: 6px;
+  padding: 6px 14px;
+  font-size: 12px;
+  font-weight: 600;
+  cursor: pointer;
+  transition: background .15s;
+}
+.btn-store-install:hover { background: #2ea043; }
+.store-badge-installed {
+  font-size: 12px;
+  color: #56d364;
+  font-weight: 600;
+}
+
+/* ── Skills sidebar section ──────────────────────────────────────────────── */
+#skill-list-items { padding: 0 8px 8px; display: flex; flex-direction: column; gap: 2px; }
+
 /* ── Scrollbar ───────────────────────────────────────────────────────────── */
 ::-webkit-scrollbar       { width: 6px; }
 ::-webkit-scrollbar-track { background: transparent; }