openclacky 1.3.9 → 1.3.11

data/lib/clacky/web/features/extensions/view.js

@@ -87,14 +87,23 @@ const ExtensionsView = (() => {
 
     const versionHtml = ext.version
       ? `<span class="extension-version">v${escapeHtml(String(ext.version))}</span>` : "";
+    const canUpdate = ext.installed_version && ext.version && ext.installed_version !== ext.version;
     const installedHtml = ext.installed
       ? `<span class="extension-installed">${escapeHtml(I18n.t("extensions.installed"))}</span>` : "";
+    const updatableHtml = canUpdate
+      ? `<span class="extension-updatable">${escapeHtml(I18n.t("extensions.updatable"))}</span>` : "";
+    const unlistedHtml = (ext.unlisted && ext.installed)
+      ? `<span class="extension-unlisted">${escapeHtml(I18n.t("extensions.unlisted"))}</span>` : "";
     const unitsText = _formatUnits(ext.units);
     const unitsHtml = unitsText
       ? `<span class="extension-units">${escapeHtml(unitsText)}</span>` : "";
     const homepageHtml = ext.homepage
       ? `<a class="extension-homepage" href="${escapeHtml(ext.homepage)}" target="_blank" rel="noopener noreferrer">${I18n.t("extensions.homepage")}</a>`
       : "";
+    const authorHtml = ext.author
+      ? `<span class="extension-author">${escapeHtml(I18n.t("extensions.by"))}${escapeHtml(ext.author)}</span>` : "";
+    const installsHtml = ext.download_count > 0
+      ? `<span class="extension-installs">${escapeHtml(String(ext.download_count))} ${escapeHtml(I18n.t("extensions.installs"))}</span>` : "";
 
     const card = document.createElement("div");
     card.className = "extension-card extension-card-clickable";
@@ -107,12 +116,14 @@ const ExtensionsView = (() => {
             <span class="extension-name">${escapeHtml(name)}</span>
             ${versionHtml}
             ${installedHtml}
+            ${updatableHtml}
+            ${unlistedHtml}
             ${unitsHtml}
+            ${authorHtml}
+            ${installsHtml}
           </div>
-          <div class="extension-desc">${escapeHtml(description)}</div>
-          <div class="extension-meta">
-            ${homepageHtml}
-          </div>
+          ${description ? `<div class="extension-desc">${escapeHtml(description)}</div>` : ""}
+          ${homepageHtml ? `<div class="extension-meta">${homepageHtml}</div>` : ""}
         </div>
       </div>`;
     return card;
@@ -147,9 +158,8 @@ const ExtensionsView = (() => {
     }
 
     if (st.detailError) {
-      panel.innerHTML = _detailShell(`
-        <div class="extensions-empty">${escapeHtml(st.detailError)}</div>`);
-      _wireDetail();
+      console.warn("[Extensions] detail error, navigating back:", st.detailError);
+      _backToList();
       return;
     }
 
@@ -199,6 +209,16 @@ const ExtensionsView = (() => {
         Extensions.install(id);
       });
     }
+
+    const updateBtn = document.querySelector("[data-ext-update]");
+    if (updateBtn) {
+      updateBtn.addEventListener("click", () => {
+        const id = updateBtn.getAttribute("data-ext-update");
+        updateBtn.disabled = true;
+        updateBtn.textContent = I18n.t("extensions.action.updating");
+        Extensions.update(id);
+      });
+    }
   }
 
   function _backToList() {
@@ -215,10 +235,16 @@ const ExtensionsView = (() => {
       : ext.description || "";
     const emoji = ext.emoji || "🧩";
 
+    const canUpdate = ext.installed && ext.installed_version && ext.version && ext.installed_version !== ext.version;
     const versionHtml = ext.version
       ? `<span class="extension-version">v${escapeHtml(String(ext.version))}</span>` : "";
+    const installedLabel = canUpdate && ext.installed_version
+      ? `${I18n.t("extensions.installed")} v${escapeHtml(String(ext.installed_version))}`
+      : I18n.t("extensions.installed");
     const installedHtml = ext.installed
-      ? `<span class="extension-installed">${escapeHtml(I18n.t("extensions.installed"))}</span>` : "";
+      ? `<span class="extension-installed">${installedLabel}</span>` : "";
+    const unlistedHtml = ext.unlisted
+      ? `<span class="extension-unlisted">${escapeHtml(I18n.t("extensions.unlisted"))}</span>` : "";
     const unitsText = _formatUnits(ext.units);
     const unitsHtml = unitsText
       ? `<span class="extension-units">${escapeHtml(unitsText)}</span>` : "";
@@ -226,7 +252,9 @@ const ExtensionsView = (() => {
       ? `<a class="extension-homepage" href="${escapeHtml(ext.homepage)}" target="_blank" rel="noopener noreferrer">${I18n.t("extensions.homepage")}</a>`
       : "";
     const authorHtml = ext.author
-      ? `<span class="extension-detail-author">${escapeHtml(ext.author)}</span>` : "";
+      ? `<span class="extension-author">${escapeHtml(I18n.t("extensions.by"))}${escapeHtml(ext.author)}</span>` : "";
+    const installsHtml = ext.download_count > 0
+      ? `<span class="extension-installs">${escapeHtml(String(ext.download_count))} ${escapeHtml(I18n.t("extensions.installs"))}</span>` : "";
 
     return `
       <div class="extension-detail-hero">
@@ -236,10 +264,13 @@ const ExtensionsView = (() => {
             <span class="extension-name extension-name-lg">${escapeHtml(name)}</span>
             ${versionHtml}
             ${installedHtml}
+            ${unlistedHtml}
             ${unitsHtml}
+            ${authorHtml}
+            ${installsHtml}
           </div>
           ${description ? `<div class="extension-desc extension-desc-detail">${escapeHtml(description)}</div>` : ""}
-          ${(authorHtml || homepageHtml) ? `<div class="extension-meta">${authorHtml}${homepageHtml}</div>` : ""}
+          ${homepageHtml ? `<div class="extension-meta">${homepageHtml}</div>` : ""}
           ${_renderActions(ext)}
         </div>
       </div>
@@ -268,9 +299,14 @@ const ExtensionsView = (() => {
     const removeBtn = ext.removable
       ? `<button type="button" class="extension-action extension-action-remove" data-ext-remove="${escapeHtml(slug)}">${escapeHtml(I18n.t("extensions.action.remove"))}</button>`
       : "";
+    const canUpdate = ext.installed_version && ext.version && ext.installed_version !== ext.version && ext.download_url;
+    const updateBtn = canUpdate
+      ? `<button type="button" class="extension-action extension-action-update" data-ext-update="${escapeHtml(id)}">${escapeHtml(I18n.t("extensions.action.update"))}</button>`
+      : "";
     return `
       <div class="extension-detail-actions">
         ${disabledBadge}
+        ${updateBtn}
         <button type="button" class="extension-action ${toggleCls}" data-ext-toggle="${escapeHtml(slug)}" data-ext-enabled="${ext.disabled ? "1" : "0"}">${escapeHtml(I18n.t(toggleKey))}</button>
         ${removeBtn}
       </div>`;
@@ -331,7 +367,7 @@ const ExtensionsView = (() => {
             <span class="extension-version">v${escapeHtml(String(v.version || ""))}</span>
             ${date ? `<span class="extension-version-date">${escapeHtml(date)}</span>` : ""}
           </div>
-          ${v.release_notes ? `<div class="extension-version-notes">${escapeHtml(String(v.release_notes))}</div>` : ""}
+          ${v.release_notes ? `<div class="extension-version-notes">${typeof marked !== "undefined" ? marked.parse(String(v.release_notes).replace(/^#{1,3}[^\n]*\n?/, ""), { breaks: true, gfm: true }) : escapeHtml(String(v.release_notes))}</div>` : ""}
         </li>`;
     }).join("");
     return `

data/lib/clacky/web/features/new-session/view.js

@@ -110,7 +110,7 @@ const NewSessionView = (() => {
       }
 
       const author = (a.author || "").trim();
-      if (author) {
+      if (author && !_isBuiltin(a)) {
         const by = document.createElement("div");
         by.className = "agent-card-author";
         by.textContent = _isZh() ? `作者 ${author}` : `by ${author}`;

data/lib/clacky/web/i18n.js

@@ -508,18 +508,22 @@ const I18n = (() => {
       "mcp.title":                "MCP Servers",
       "mcp.subtitle":             "Standard Model Context Protocol servers (Claude Desktop / Cursor compatible). Edit ~/.clacky/mcp.json to add or remove servers.",
       "extensions.title":         "Extension Market",
-      "extensions.subtitle":      "Browse and manage your extensions to enhance OpenClacky with agents, skills, and panels.",
+      "extensions.subtitle":      "Browse and manage extensions — add agents, skills, and panels to supercharge your workflow.",
       "extensions.searchPlaceholder": "Search extensions…",
       "extensions.sort.newest":   "Newest",
       "extensions.sort.updated":  "Recently updated",
-      "extensions.sort.downloads": "Most downloaded",
+      "extensions.sort.downloads": "Most installed",
       "extensions.filter.all":       "All",
       "extensions.filter.installed": "Installed",
       "extensions.empty":         "No extensions available yet.",
       "extensions.noResults":     "No extensions match your search.",
       "extensions.loadFailed":    "Could not reach the extension store.",
       "extensions.homepage":      "Homepage",
-      "extensions.installed":     "Installed",
+      "extensions.by":            "by ",
+      "extensions.installs":      "installs",
+      "extensions.installed":          "Installed",
+      "extensions.updatable":          "Update available",
+      "extensions.unlisted":      "Unlisted",
       "extensions.unit.skill":    "skill",
       "extensions.unit.skills":   "skills",
       "extensions.unit.agent":    "agent",
@@ -538,6 +542,8 @@ const I18n = (() => {
       "extensions.disabled":               "Disabled",
       "extensions.action.install":         "Install",
       "extensions.action.installing":      "Installing…",
+      "extensions.action.update":          "Update",
+      "extensions.action.updating":        "Updating…",
       "extensions.action.disable":         "Disable",
       "extensions.action.enable":          "Enable",
       "extensions.action.remove":          "Remove",
@@ -1469,18 +1475,22 @@ const I18n = (() => {
       "mcp.title":                "MCP 工具",
       "mcp.subtitle":             "标准 Model Context Protocol 服务（与 Claude Desktop / Cursor 兼容）。编辑 ~/.clacky/mcp.json 添加或移除服务。",
       "extensions.title":         "扩展市场",
-      "extensions.subtitle":      "浏览并管理你的扩展，用 Agent、技能与面板增强 OpenClacky。",
+      "extensions.subtitle":      "浏览并管理扩展，用 Agent、技能与面板增强你的工作流。",
       "extensions.searchPlaceholder": "搜索扩展…",
       "extensions.sort.newest":   "最新",
       "extensions.sort.updated":  "最近更新",
-      "extensions.sort.downloads": "下载最多",
+      "extensions.sort.downloads": "安装最多",
       "extensions.filter.all":       "全部",
       "extensions.filter.installed": "已安装",
       "extensions.empty":         "暂无可用扩展。",
       "extensions.noResults":     "没有匹配的扩展。",
       "extensions.loadFailed":    "无法连接扩展市场。",
       "extensions.homepage":      "主页",
-      "extensions.installed":     "已安装",
+      "extensions.by":            "作者：",
+      "extensions.installs":      "次安装",
+      "extensions.installed":          "已安装",
+      "extensions.updatable":          "有更新",
+      "extensions.unlisted":      "已下架",
       "extensions.unit.skill":    "个技能",
       "extensions.unit.skills":   "个技能",
       "extensions.unit.agent":    "个 Agent",
@@ -1499,6 +1509,8 @@ const I18n = (() => {
       "extensions.disabled":               "已禁用",
       "extensions.action.install":         "安装",
       "extensions.action.installing":      "安装中…",
+      "extensions.action.update":          "更新",
+      "extensions.action.updating":        "更新中…",
       "extensions.action.disable":         "禁用",
       "extensions.action.enable":          "启用",
       "extensions.action.remove":          "移除",

data/lib/clacky/web/sessions.js

@@ -113,6 +113,33 @@ const Sessions = (() => {
     return html;
   }
 
+  // Encode file:// URLs inside raw markdown BEFORE marked parses it. The AI
+  // emits raw paths (spaces / non-ASCII / % literal); marked's link tokenizer
+  // treats a space as the end of the URL, so links with spaces are never
+  // recognized. Re-encoding the path (decode then encode, idempotent) turns
+  // spaces into %20 and a literal % into %25 so marked sees a valid URL.
+  function _encodeFileUrlsInMarkdown(text) {
+    return text.replace(
+      /(!?\[[^\]]*\]\()(file:\/{2,3})([^)]+)(\))/g,
+      (_m, open, scheme, path, close) => {
+        try { path = decodeURI(path); } catch (_) { /* keep raw if malformed */ }
+        return open + scheme + encodeURI(path) + close;
+      }
+    );
+  }
+
+  // Normalize a file:// href for use in <a href>. The AI emits raw paths
+  // (Chinese chars / spaces literal); encode them to a valid percent-encoded
+  // URL. Idempotent: an already-encoded path is decoded first so it isn't
+  // double-encoded. Non file:// links pass through untouched.
+  function _normalizeFileHref(href) {
+    if (typeof href !== "string" || !href.startsWith("file://")) return href;
+    const prefix = href.match(/^file:\/{2,3}/)[0];
+    let path = href.slice(prefix.length);
+    try { path = decodeURI(path); } catch (_) { /* keep raw if malformed */ }
+    return prefix + encodeURI(path);
+  }
+
   // Run marked on a text string. Returns HTML. Falls back to escaped plain text
   // if the marked library is unavailable.
   function _markedParse(text) {
@@ -122,6 +149,7 @@ const Sessions = (() => {
     const math = [];
     const PLACEHOLDER = (i) => `\u0000KTX${i}\u0000`;
     let prepared = _extractMath(text, math, PLACEHOLDER);
+    prepared = _encodeFileUrlsInMarkdown(prepared);
 
     let html;
     if (typeof marked !== "undefined") {
@@ -138,7 +166,7 @@ const Sessions = (() => {
       const renderer = new marked.Renderer();
       renderer.link = function({ href, title, text }) {
         const titleAttr = title ? ` title="${title}"` : "";
-        return `<a href="${href}"${titleAttr} target="_blank" rel="noopener noreferrer">${text}</a>`;
+        return `<a href="${_normalizeFileHref(href)}"${titleAttr} target="_blank" rel="noopener noreferrer">${text}</a>`;
       };
       // Override code block rendering: apply syntax highlighting + header with
       // language label and copy button.
@@ -2307,7 +2335,9 @@ const Sessions = (() => {
         const link = e.target.closest("a[href^='file://']");
         if (!link) return;
         e.preventDefault();
-        let filePath = decodeURIComponent(link.getAttribute("href").replace(/^file:\/\//, ""));
+        const rawHref = link.getAttribute("href").replace(/^file:\/\//, "");
+        let filePath;
+        try { filePath = decodeURIComponent(rawHref); } catch (_) { filePath = rawHref; }
         // file:///C:/foo → /C:/foo after replace; strip the leading slash for Windows drive letters
         if (/^\/[A-Za-z]:/.test(filePath)) filePath = filePath.substring(1);
         if (!filePath) return;

metadata

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