openclacky 1.3.11 → 1.4.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: eccc2d78824c13072991199a00edd5b848216334422ce37876cd639854abb4ab
-  data.tar.gz: c29754fc48b5b9638168fa167c5335c32ccf5c64a3db94d3e64ac2638581857e
+  metadata.gz: 4c8311a2bd1bd4cd050fce3e806479c580afb4da476201e59a041c2293347b5f
+  data.tar.gz: 520ba1c8c1821aede0b4766be3f321b3c121a91d3d9fe9e5dada0034479a3d4e
 SHA512:
-  metadata.gz: 6d2e6ef2dcf07a50779241a3ed5b384c2fbf34e4265208874567b12d1280e29f2fe79124286ab7b73251bfc836de01ebf9d8ad03f91ae12ad55ac4575128bfd3
-  data.tar.gz: 77088322e7cecc8c0cc0f7138786263c3de14dff2d1b33e99cee763437918e226bd34f0ef318a12d4f7292bcad5a935a8ca2ca16c9a15d8d2bd7262eebee9ccd
+  metadata.gz: e2d171e71ae9eb9cbecfc79bbf6ac1f3977268baf5a4026a57caffc378501ca6ca1bad8426237bcd949564ac1986d1b5b4bfe4f549f108d35f3e216edee75bdc
+  data.tar.gz: 8ba4d9fe3a50aacd4a247aae6877f391c73297c4e63b8c428b99a09f9c2d9ab642104ff5d6370c04556b679b2e3d041785f1833727605b3205638e7c7ea01126

data/CHANGELOG.md

@@ -4,6 +4,65 @@ 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
+- Volcengine Ark Seedance video generation provider (async submit + poll)
+- Entry points config and extension info editing in ext-studio
+- Search box in skills panel tab bar
+- Default letter gradient icon for extensions without a custom icon
+- Skeleton loading for published extensions list in ext-studio
+
+### Improved
+- Extensions market renders `display_name` instead of raw slug
+- ext-studio extension picker and cards show display name; unit tags aggregated by type
+- Session list "Today / Yesterday" labels localized
+
+### Fixed
+- Terminal Pass 1 echo-strip no longer swallows real output (C-5701)
+- OS metadata files (`.DS_Store`, `__MACOSX`) excluded from packed extension zips (C-5728)
+- TUI exit now restores original terminal settings via `stty` (C-5724)
+- Local image cache busted with mtime version param when file is overwritten (C-5703)
+- Generated media usage correctly associated with sessions (C-5739)
+- Mobile header search action alignment (C-5733)
+- Session search shortcut hint adapted for web (C-5734)
+- Memory card expand button height aligned with other action buttons
+- Billing model filter select styling improved
+- Modal overlay click-to-close disabled (prevents accidental dismissal)
+- Version number / date separator added in extension version history
+- All users can now publish extensions (permission gate removed)
+- Invalid extension version format shows inline error in publish modal
+- Brand update button hover color uses correct CSS variable
+
 ## [1.3.11] - 2026-07-12
 
 ### 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/skill_manager.rb

@@ -452,6 +452,7 @@ module Clacky
       # @return [Hash<String, Proc>]
       def build_template_context
         {
+          "session_id"      => -> { @session_id },
           "memories_meta"   => -> { load_memories_meta },
           "all_skills_meta" => -> { load_all_skills_meta }
         }

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
@@ -111,7 +131,7 @@ class ExtStudioExt < Clacky::ApiExtension
     exts = Array(result[:extensions]).map do |ext|
       {
         id: ext["name"] || ext["slug"] || ext["id"],
-        name: ext["name"],
+        name: ext["display_name"] || ext["name"],
         version: (ext["latest_version"] || {})["version"] || ext["version"],
         status: ext["status"],
         units: ext["units"] || {}
@@ -149,6 +169,122 @@ 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
+  # Writes display metadata back to the local ext.yml without touching other fields.
+  post "/set_meta" do
+    ext_id = require_ext_id!
+
+    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
+
+    yml_path = File.join(container[:dir], "ext.yml")
+    error!("ext.yml not found", status: 404) unless File.exist?(yml_path)
+
+    manifest = Psych.safe_load(File.read(yml_path), permitted_classes: [], aliases: true) || {}
+
+    manifest["name"]        = presence(json_body["name"])        if json_body.key?("name")
+    manifest["description"] = presence(json_body["description"]) if json_body.key?("description")
+
+    if json_body.key?("entry_points")
+      eps = json_body["entry_points"]
+      by_panel = Hash.new { |h, k| h[k] = [] }
+      Array(eps).each { |ep| by_panel[ep["unit_id"].to_s] << { "slot" => ep["slot"] } if ep["slot"] }
+      panels = Array((manifest["contributes"] || {})["panels"])
+      panels.each do |panel|
+        pid = panel["id"].to_s
+        slots = by_panel[pid]
+        if slots.any?
+          panel["entry_points"] = slots
+        else
+          panel.delete("entry_points")
+        end
+      end
+    end
+
+    File.write(yml_path, Psych.dump(manifest))
+    Clacky::ExtensionLoader.load_all(force: true)
+
+    json(ok: true, ext_id: ext_id)
+  end
+
   # POST /api/ext/ext-studio/set_version
   # body: { ext_id, version }
   # Writes the new version string back to the local ext.yml.
@@ -176,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.
@@ -250,6 +430,14 @@ class ExtStudioExt < Clacky::ApiExtension
     raw = container[:raw] || {}
     ext_issues = issues.select { |i| i.ext == ext_id }
     dir = container[:dir]
+    ext_units = result.units.select { |u| u.ext_id == ext_id }
+    panels = Array((raw["contributes"] || {})["panels"])
+    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,
@@ -259,7 +447,11 @@ class ExtStudioExt < Clacky::ApiExtension
       layer: container[:layer].to_s,
       dir: dir,
       mtime: File.mtime(File.join(dir, "ext.yml")).to_i,
-      units: result.units.select { |u| u.ext_id == ext_id }.map { |u| serialize_unit(u) },
+      units: ext_units.map { |u| serialize_unit(u) },
+      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 }
     }
@@ -269,6 +461,13 @@ class ExtStudioExt < Clacky::ApiExtension
     { kind: unit.kind.to_s, id: unit.id, layer: unit.layer.to_s }
   end
 
+  private def unit_counts(units)
+    units.each_with_object({}) do |unit, counts|
+      kind = unit.kind.to_s
+      counts[kind] = (counts[kind] || 0) + 1
+    end
+  end
+
   private def serialize_issue(issue)
     {
       ext: issue.ext,