openclacky 1.4.0 → 1.4.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 29e2af05d27f902fc645a1399eb16d16d41681ec03312ee861fdd594292368c2
-  data.tar.gz: bbb62ec065f07867a2085b6dc0f81d5702c34cf0ff4e10e0b88513daa5fb4541
+  metadata.gz: 4c8311a2bd1bd4cd050fce3e806479c580afb4da476201e59a041c2293347b5f
+  data.tar.gz: 520ba1c8c1821aede0b4766be3f321b3c121a91d3d9fe9e5dada0034479a3d4e
 SHA512:
-  metadata.gz: bf7f4ad60821686404ee3b43607e59c51cb0d38420171b94eaea2c6d2877d71c03052f9147b56f61aeb5fc01155127d4aab15a8f0b6ecc0b73d6b294fb0bbe3c
-  data.tar.gz: ba3a05a10e80fd8b48f10f0778388c5417ce8fa79e0f3936cb6ef734117609bc85f56868227e174675e4132d0c619a8b385619c7aaf62e75039fc3f4895d15f8
+  metadata.gz: e2d171e71ae9eb9cbecfc79bbf6ac1f3977268baf5a4026a57caffc378501ca6ca1bad8426237bcd949564ac1986d1b5b4bfe4f549f108d35f3e216edee75bdc
+  data.tar.gz: 8ba4d9fe3a50aacd4a247aae6877f391c73297c4e63b8c428b99a09f9c2d9ab642104ff5d6370c04556b679b2e3d041785f1833727605b3205638e7c7ea01126

data/CHANGELOG.md

@@ -4,6 +4,35 @@ 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.4.1] - 2026-07-20
+
+### Added
+- Extension README edit/validate flow and screenshot upload in ext-studio publish
+- Kimi k3 and k2.7-code models added to Kimi preset
+- New session split-button with improved styling
+- Rename-on-name-conflict flow in ext-studio publish
+- Skip OpenClacky AI Keys card for branded onboarding
+
+### Improved
+- Secondary API host added with reduced failover latency
+- Extensions default sort changed to most-installed (downloads first)
+
+### Fixed
+- WSL file panel "Copy Path" now copies Windows-accessible UNC path via `wslpath` (C-5693)
+- Edit button hidden on compressed user messages archived to chunk (C-5723)
+- Message-edit Enter key blocked during IME composition (C-5742)
+- Markdown images with `file://~/` tilde paths now render correctly (C-5747)
+- Local `file://` path resolution unified for Windows drive letters (C-5746, C-5708)
+- Brand extension installs under its real ID, not as unpacked (C-5735)
+- Backup success status no longer stays displayed forever (C-5711)
+- `grep` tool defaults to recursive behavior
+- `display_name` fields included in installed extensions API response
+- Published list reloads after setup entry point and publish
+- Unpublish button size matches iterate button in ext-studio
+- `device_token` correctly passed in screenshot upload multipart fields
+- MCP tool list rendering fixed and display styling improved
+- Uninstall script now removes all openclacky gem versions
+
 ## [1.4.0] - 2026-07-14
 
 ### Added

data/lib/clacky/agent/session_serializer.rb

@@ -257,8 +257,10 @@ module Clacky
             end
 
           if is_real_user_msg
-            # Start a new round at each real user message
-            current_round = { user_msg: msg, events: [] }
+            # Start a new round at each real user message.
+            # editable: true — this message still lives in the active in-memory
+            # @history, so truncate_from_created_at can locate and truncate it.
+            current_round = { user_msg: msg, events: [], editable: true }
             rounds << current_round
           elsif current_round
             current_round[:events] << msg
@@ -325,7 +327,8 @@ module Clacky
               preview_path: f[:preview_path] || f["preview_path"] }
           }
           all_files = image_files + disk_files
-          ui.show_user_message(raw_text, created_at: msg[:created_at], files: all_files)
+          ui.show_user_message(raw_text, created_at: msg[:created_at], files: all_files,
+                               editable: round[:editable] != false)
 
           round[:events].each do |ev|
             # Skip system-injected messages (e.g. synthetic skill content, memory prompts)
@@ -480,7 +483,11 @@ module Clacky
                 created_at: synthetic_ts,
                 _from_chunk: true
               },
-              events: []
+              events: [],
+              # editable: false — this message was archived into a chunk MD and no
+              # longer exists in the active in-memory @history, so it cannot be
+              # truncated/edited (truncate_from_created_at would silently no-op).
+              editable: false
             }
             rounds << current_round
           elsif current_round

data/lib/clacky/agent.rb

@@ -1862,11 +1862,10 @@ module Clacky
       files = []
       content.scan(/(!?)\[([^\]]*)\]\(file:\/\/([^)]+)\)/) do
         inline = $1 == "!"
-        # URL-decode percent-encoded characters (e.g. Chinese filenames encoded by AI)
-        raw_path = CGI.unescape($3)
-        name   = File.basename(raw_path)
-        path   = File.expand_path(raw_path)
-        Clacky::Logger.info("[parse_file_links] raw=#{$3.inspect} expanded=#{path.inspect} exist=#{File.exist?(path)}")
+        # Resolve the AI-emitted path: decode, WSL drive-letter normalize, expand.
+        path   = Clacky::Utils::EnvironmentDetector.resolve_local_path($3)
+        name   = File.basename(path)
+        Clacky::Logger.info("[parse_file_links] raw=#{$3.inspect} resolved=#{path.inspect} exist=#{File.exist?(path)}")
         files << { name: name, path: path, inline: inline }
       end
       { text: content, files: files }

data/lib/clacky/brand_config.rb

@@ -622,7 +622,7 @@ module Clacky
     #
     # Returns { success: true, extension: {...} } or
     #         { success: false, error: "...", already_exists: Boolean }.
-    def upload_extension!(ext_id, zip_data, force: false, status: nil, changelog: nil, origin: 'marketplace')
+    def upload_extension!(ext_id, zip_data, force: false, status: nil, changelog: nil, readme: nil, origin: 'marketplace')
       identity = Clacky::Identity.load
       return { success: false, error: "Device not bound to a platform account" } unless identity.bound?
 
@@ -636,6 +636,7 @@ module Clacky
       fields["origin"]    = origin.to_s      if origin
       fields["status"]    = status.to_s    if status
       fields["changelog"] = changelog.to_s if changelog
+      fields["readme"]    = readme.to_s    if readme
 
       body_bytes, boundary = build_multipart(fields, "extension_zip", "#{ext_id}.zip", zip_data)
 
@@ -700,6 +701,55 @@ module Clacky
       end
     end
 
+    # Update an extension's readme text without publishing a new version.
+    # Uses PATCH /api/v1/client/extensions/:id.
+    # Returns { success:, extension:, error: }
+    def update_extension_readme!(ext_id, readme)
+      identity = Clacky::Identity.load
+      return { success: false, error: "Device not bound to a platform account" } unless identity.bound?
+
+      path    = "/api/v1/client/extensions/#{URI.encode_www_form_component(ext_id)}"
+      payload = { "device_token" => identity.device_token, "readme" => readme.to_s }
+      result  = platform_client.patch(path, payload)
+
+      if result[:success]
+        { success: true, extension: result[:data]["extension"] }
+      else
+        body = result[:data] || {}
+        { success: false, error: result[:error] || body["code"] || "Update readme failed" }
+      end
+    rescue StandardError => e
+      { success: false, error: "Network error: #{e.message}" }
+    end
+
+    # Upload any file to the platform's model-agnostic upload endpoint.
+    # Returns an orphan blob URL — no extension record required.
+    # Uses POST /api/v1/client/uploads.
+    # Parameters:
+    #   data         (bytes)  - raw file bytes
+    #   filename     (string) - filename including extension
+    #   content_type (string) - MIME type
+    # Returns { success:, url:, filename:, blob_key:, error: }
+    def upload_file!(data, filename:, content_type: "image/png")
+      identity = Clacky::Identity.load
+      return { success: false, error: "Device not bound to a platform account" } unless identity.bound?
+
+      path   = "/api/v1/client/uploads"
+      fields = { "device_token" => identity.device_token }
+      body_bytes, boundary = build_multipart(fields, "file", filename, data, content_type: content_type)
+      result = platform_client.multipart_post(path, body_bytes, boundary, read_timeout: 60)
+
+      if result[:success]
+        d = result[:data] || {}
+        { success: true, url: d["url"], filename: d["filename"], blob_key: d["blob_key"] }
+      else
+        body = result[:data] || {}
+        { success: false, error: result[:error] || body["code"] || "File upload failed" }
+      end
+    rescue StandardError => e
+      { success: false, error: "Network error: #{e.message}" }
+    end
+
     # Search the public extension marketplace. Anonymous — no license required.
     # Uses GET /api/v1/extensions. Returns { success:, extensions: [], error: }.
     def search_extensions!(query: nil, sort: nil)
@@ -1579,7 +1629,7 @@ module Clacky
 
     # Assemble a binary multipart/form-data body: text fields + one file part.
     # Kept binary-safe so null bytes in the ZIP survive. Returns [body, boundary].
-    private def build_multipart(fields, file_field, filename, file_bytes)
+    private def build_multipart(fields, file_field, filename, file_bytes, content_type: "application/zip")
       boundary = "----ClackyMultipart#{SecureRandom.hex(8)}"
       crlf     = "\r\n"
       parts    = []
@@ -1591,7 +1641,7 @@ module Clacky
       end
       parts << "--#{boundary}#{crlf}"
       parts << "Content-Disposition: form-data; name=\"#{file_field}\"; filename=\"#{filename}\"#{crlf}"
-      parts << "Content-Type: application/zip#{crlf}#{crlf}"
+      parts << "Content-Type: #{content_type}#{crlf}#{crlf}"
       parts << file_bytes.b
       parts << "#{crlf}--#{boundary}--#{crlf}"
       [parts.map(&:b).join, boundary]

data/lib/clacky/default_extensions/ext-studio/api/handler.rb

@@ -79,11 +79,24 @@ class ExtStudioExt < Clacky::ApiExtension
       res      = Clacky::ExtensionPackager.pack(ext_id, out_dir: tmp)
       zip_data = File.binread(res.path)
 
+      # Read readme from local README.md if present, so first-publish carries it.
+      local_readme = nil
+      result_load  = Clacky::ExtensionLoader.load_all(force: false)
+      container    = Array(result_load.containers).find { |id, _| id == res.ext_id }&.last
+      if container
+        readme_path = File.join(container[:dir], "README.md")
+        if File.exist?(readme_path)
+          local_readme = File.read(readme_path, encoding: "utf-8").strip
+          local_readme = nil if local_readme.empty?
+        end
+      end
+
       result = brand.upload_extension!(
         res.ext_id, zip_data,
         force:     json_body["force"] == true,
         status:    presence(json_body["status"]),
-        changelog: presence(json_body["changelog"])
+        changelog: presence(json_body["changelog"]),
+        readme:    local_readme
       )
 
       if result[:success]
@@ -91,8 +104,15 @@ class ExtStudioExt < Clacky::ApiExtension
         ver = (ext["latest_version"] || {})["version"]
         json(ok: true, ext_id: res.ext_id, version: ver, status: ext["status"])
       elsif result[:already_exists]
+        # The same user has already published this extension — they need to use
+        # force: true to publish a new version.
         json(ok: false, already_exists: true,
              error: "#{res.ext_id} already published. Publish a new version with force.")
+      elsif result[:error].to_s.include?("already taken")
+        # Another user has claimed this name globally — the creator must rename
+        # their extension before publishing.
+        json(ok: false, name_taken: true,
+             error: result[:error])
       else
         error!(result[:error] || "publish failed", status: 502)
       end
@@ -149,6 +169,81 @@ class ExtStudioExt < Clacky::ApiExtension
     json(ok: true, ext_id: ext_id)
   end
 
+  # PATCH /api/ext/ext-studio/readme
+  # body: { ext_id, readme }
+  # Two modes:
+  #   - Extension already published → save to platform via API
+  #   - Extension not yet published → write readme field to local ext.yml
+  patch "/readme" do
+    ext_id = require_ext_id!
+    readme = json_body["readme"].to_s
+
+    # Check whether this extension has been published on the platform.
+    brand    = Clacky::BrandConfig.load
+    identity = Clacky::Identity.load
+    published = false
+
+    if identity.bound?
+      my_result = brand.fetch_my_extensions!
+      if my_result[:success]
+        published = Array(my_result[:extensions]).any? { |e| (e["name"] || e["slug"]) == ext_id }
+      end
+    end
+
+    if published
+      # Already on the platform — update readme there.
+      result = brand.update_extension_readme!(ext_id, readme)
+      error!(result[:error] || "readme update failed", status: 502) unless result[:success]
+    else
+      # Not yet published — persist readme to local README.md so it's included on first publish.
+      result    = Clacky::ExtensionLoader.load_all(force: false)
+      container = Array(result.containers).find { |id, _| id == ext_id }&.last
+      error!("extension not found: #{ext_id}", status: 404) unless container
+
+      readme_path = File.join(container[:dir], "README.md")
+      if readme.strip.empty?
+        File.delete(readme_path) if File.exist?(readme_path)
+      else
+        File.write(readme_path, readme, encoding: "utf-8")
+      end
+    end
+
+    json(ok: true, ext_id: ext_id, saved_to: published ? "platform" : "local")
+  end
+
+  # POST /api/ext/ext-studio/screenshot
+  # Multipart body: { ext_id (field), file (image file) }
+  # Uploads a screenshot image via the platform's model-agnostic upload API
+  # and returns the CDN URL. Does NOT require the extension to be published yet.
+  post "/screenshot" do
+    q = req.query
+    ext_id = q["ext_id"].to_s.strip
+    error!("ext_id required", status: 422) if ext_id.empty?
+
+    uploaded = q["file"]
+    error!("missing file", status: 422) unless uploaded
+
+    # WEBrick::HTTPUtils::FormData inherits from String — .to_s is the file bytes,
+    # .filename is the original filename, .["content-type"] gives the MIME type.
+    filename     = uploaded.respond_to?(:filename) ? (uploaded.filename || "screenshot.png") : "screenshot.png"
+    content_type = (uploaded.respond_to?(:[]) ? uploaded["content-type"].to_s : "")
+    content_type = "image/png" if content_type.empty?
+    data         = uploaded.to_s
+    error!("missing file", status: 422) if data.empty?
+
+    unless Clacky::Identity.load.bound?
+      error!("device not bound to a platform account", status: 428, needs_binding: true)
+    end
+
+    result = Clacky::BrandConfig.load.upload_file!(
+      data,
+      filename:     filename,
+      content_type: content_type
+    )
+    error!(result[:error] || "screenshot upload failed", status: 502) unless result[:success]
+    json(ok: true, url: result[:url], filename: result[:filename])
+  end
+
   # POST /api/ext/ext-studio/set_meta
   # body: { ext_id, name?, description?, entry_points? }
   # entry_points: [{ unit_id, slot }] — stored under contributes.panels[id].entry_points
@@ -217,6 +312,50 @@ class ExtStudioExt < Clacky::ApiExtension
   end
 
 
+  # POST /api/ext/ext-studio/set_id
+  # body: { ext_id, new_id }
+  # Renames an extension: updates the `id` field in ext.yml and renames the
+  # local container directory. Used when the user wants to republish under a
+  # different name (e.g. after a name-taken conflict on the marketplace).
+  post "/set_id" do
+    ext_id = require_ext_id!
+    new_id = presence(json_body["new_id"])
+    error!("new_id required", status: 422) unless new_id
+
+    slug_format = /\A[a-z0-9][a-z0-9\-]*[a-z0-9]\z/
+    unless new_id.length <= 64 && new_id.match?(slug_format)
+      error!("new_id must be lowercase letters, digits and hyphens (e.g. my-ext)", status: 422)
+    end
+
+    result    = Clacky::ExtensionLoader.load_all(force: false)
+    container = Array(result.containers).find { |id, _| id == ext_id }&.last
+    error!("extension not found: #{ext_id}", status: 404) unless container
+
+    old_dir = container[:dir]
+    new_dir = File.join(File.dirname(old_dir), new_id)
+    error!("An extension named '#{new_id}' already exists locally", status: 409) if File.exist?(new_dir)
+
+    yml_path = File.join(old_dir, "ext.yml")
+    error!("ext.yml not found", status: 404) unless File.exist?(yml_path)
+
+    # Update the id field in ext.yml, preserving all other content.
+    content = File.read(yml_path)
+    if content =~ /^id:/
+      content = content.sub(/^id:.*$/, "id: #{new_id}")
+    else
+      content = "id: #{new_id}\n" + content
+    end
+    File.write(yml_path, content)
+
+    # Rename the container directory.
+    File.rename(old_dir, new_dir)
+
+    # Reload so the new id is visible immediately.
+    Clacky::ExtensionLoader.load_all(force: true)
+
+    json(ok: true, old_id: ext_id, new_id: new_id)
+  end
+
   # body: { idea? }
   # Spawns a session bound to the ext-developer agent, optionally seeded with
   # the user's idea as the first task — the "let AI build it for me" entry.
@@ -296,6 +435,9 @@ class ExtStudioExt < Clacky::ApiExtension
     entry_points = panels.flat_map do |p|
       Array(p["entry_points"]).map { |ep| { panel_id: p["id"], slot: ep["slot"] } }
     end
+    readme_path    = File.join(dir, "README.md")
+    readme_content = File.exist?(readme_path) ? File.read(readme_path, encoding: "utf-8").strip : nil
+    readme_content = nil if readme_content&.empty?
     {
       id: ext_id,
       name: raw["name"] || ext_id,
@@ -309,6 +451,7 @@ class ExtStudioExt < Clacky::ApiExtension
       unit_counts: unit_counts(ext_units),
       contributes: raw["contributes"] || {},
       entry_points: entry_points,
+      readme: readme_content,
       error_count: ext_issues.count { |i| i.level == :error },
       warning_count: ext_issues.count { |i| i.level == :warning }
     }