openclacky 1.5.20 → 1.5.21

data/lib/clacky/web/settings.js

@@ -1599,12 +1599,14 @@ const Settings = (() => {
       const btn         = document.getElementById("btn-browser-setup");
       const toggleWrap  = document.getElementById("browser-toggle-wrap");
       const toggleInput = document.getElementById("browser-toggle-input");
+      const shotWrap    = document.getElementById("browser-screenshot-settings");
 
       // File doesn't exist → not set up yet
       if (data.enabled === undefined || data.enabled === null) {
         desc.textContent = I18n.t("settings.browser.desc");
         btn.textContent  = I18n.t("settings.browser.btn");
         toggleWrap.style.display = "none";
+        shotWrap.style.display   = "none";
         return;
       }
 
@@ -1620,9 +1622,51 @@ const Settings = (() => {
         toggleInput.dataset.bound = "1";
         toggleInput.addEventListener("change", _toggleBrowser);
       }
+
+      _renderScreenshotSettings(data);
     } catch (_) { /* non-critical */ }
   }
 
+  function _renderScreenshotSettings(data) {
+    const wrap    = document.getElementById("browser-screenshot-settings");
+    const quality = document.getElementById("browser-screenshot-quality");
+
+    wrap.style.display = "flex";
+
+    const limit = data.screenshot_max_base64_bytes;
+    quality.querySelectorAll(".settings-segmented-btn").forEach((btn) => {
+      btn.classList.toggle("active", Number(btn.dataset.bytes) === limit);
+    });
+
+    if (!quality.dataset.bound) {
+      quality.dataset.bound = "1";
+      quality.addEventListener("click", _onScreenshotQualityClick);
+    }
+  }
+
+  async function _onScreenshotQualityClick(event) {
+    const btn = event.target.closest(".settings-segmented-btn");
+    if (!btn || btn.classList.contains("active")) return;
+    await _saveScreenshotSettings({ screenshot_max_base64_bytes: Number(btn.dataset.bytes) });
+  }
+
+  async function _saveScreenshotSettings(payload) {
+    try {
+      const res  = await fetch("/api/browser/settings", {
+        method:  "POST",
+        headers: { "Content-Type": "application/json" },
+        body:    JSON.stringify(payload)
+      });
+      const data = await res.json();
+      if (!data.ok) throw new Error(data.error || "save failed");
+      _renderScreenshotSettings(data);
+      Modal.toast(I18n.t("settings.browser.screenshot.saved"), "success");
+    } catch (_) {
+      Modal.toast(I18n.t("settings.browser.screenshot.saveFailed"), "error");
+      await _loadBrowserStatus();
+    }
+  }
+
   async function _toggleBrowser() {
     const toggleInput = document.getElementById("browser-toggle-input");
     toggleInput.disabled = true;
@@ -1691,14 +1735,14 @@ const Settings = (() => {
         const data = await res.json();
         if (data.ok) {
           status.textContent = I18n.t(successKey);
-          status.className   = "model-test-result success";
+          status.className   = "model-test-result result-ok";
         } else {
           status.textContent = data.error || I18n.t("settings.network.invalidUrl");
-          status.className   = "model-test-result error";
+          status.className   = "model-test-result result-fail";
         }
       } catch (e) {
         status.textContent = e.message || I18n.t("settings.network.invalidUrl");
-        status.className   = "model-test-result error";
+        status.className   = "model-test-result result-fail";
       }
     }
 
@@ -1728,8 +1772,17 @@ const Settings = (() => {
     const saveBtn = document.getElementById("btn-save-search");
     const testBtn = document.getElementById("btn-test-search");
     const status  = document.getElementById("settings-search-status");
+    const dsHint  = document.getElementById("search-deepseek-hint");
+    const builtinNote = document.getElementById("search-builtin-note");
+    const customBtn = document.getElementById("btn-custom-searcher");
     if (!dropdown || !saveBtn) return;
 
+    const DS_WEB = "deepseek_web";
+    // Per-call billing for hosted web tools starts 2026-11-01 00:00 Beijing
+    // time (UTC+8); before that the platform records calls at cost 0.
+    const DS_BILLING_START_MS = Date.parse("2026-10-31T16:00:00Z");
+    const defaultKeyPlaceholder = keyInput ? keyInput.placeholder : "";
+
     let select = dropdown.__customSelect;
     if (!select && trigger) {
       select = CustomSelect.init({
@@ -1740,7 +1793,60 @@ const Settings = (() => {
     }
 
     function _syncKeyVisibility() {
-      keyWrap.style.display = (select && select.value) ? "" : "none";
+      const value = select ? select.value : "";
+      keyWrap.style.display = value ? "" : "none";
+
+      // Keep the trigger's i18n key in step with the selection, so a language
+      // switch does not relabel the closed dropdown with another provider's name.
+      const valueSpan = document.getElementById("settings-search-provider-value");
+      if (valueSpan) {
+        if (value === DS_WEB) {
+          valueSpan.setAttribute("data-i18n", "settings.search.deepseek");
+        } else if (value) {
+          valueSpan.removeAttribute("data-i18n");
+        } else {
+          valueSpan.setAttribute("data-i18n", "settings.search.builtin");
+        }
+      }
+
+      const isDeepSeek = value === DS_WEB;
+      if (dsHint) {
+        dsHint.style.display = isDeepSeek ? "" : "none";
+        if (isDeepSeek) {
+          const hintKey = Date.now() < DS_BILLING_START_MS
+            ? "settings.search.deepseekHintFree"
+            : "settings.search.deepseekHint";
+          dsHint.setAttribute("data-i18n", hintKey);
+          dsHint.textContent = I18n.t(hintKey);
+        }
+      }
+      if (builtinNote) builtinNote.style.display = value ? "none" : "";
+      if (keyInput) {
+        keyInput.placeholder = isDeepSeek
+          ? I18n.t("settings.search.deepseekKeyPlaceholder")
+          : defaultKeyPlaceholder;
+      }
+      // Anything already shown refers to the provider the user just left.
+      if (status) {
+        status.textContent = "";
+        status.className = "model-test-result";
+      }
+    }
+
+    // Assigning onclick (not addEventListener) keeps re-opening Settings from
+    // stacking duplicate handlers, since open() re-runs this initializer.
+    if (customBtn) {
+      customBtn.onclick = async () => {
+        customBtn.disabled = true;
+        try {
+          Router.navigate("chat");
+          await Sessions.startWith(I18n.t("settings.search.customPrompt"), {
+            name: I18n.t("settings.search.customSessionName"),
+          });
+        } finally {
+          customBtn.disabled = false;
+        }
+      };
     }
 
     try {
@@ -1749,8 +1855,12 @@ const Settings = (() => {
       if (data.ok) {
         const rows = (data.available || []).map(name =>
           `<div class="custom-select-option" data-value="${_esc(name)}">${_esc(name)}</div>`).join("");
+        const deepseekRow =
+          `<div class="custom-select-option" data-value="${DS_WEB}">${_esc(I18n.t("settings.search.deepseek"))}</div>`;
         dropdown.innerHTML =
-          `<div class="custom-select-option" data-value="">${I18n.t("settings.search.builtin")}</div>` + rows;
+          `<div class="custom-select-option" data-value="">${I18n.t("settings.search.builtin")}</div>` +
+          deepseekRow +
+          rows;
         if (select) select.setValue(data.provider || "");
         keyInput.value = data.key_masked || "";
       }
@@ -1759,7 +1869,7 @@ const Settings = (() => {
 
     function _showStatus(message, ok) {
       status.textContent = message;
-      status.className   = `model-test-result ${ok ? "success" : "error"}`;
+      status.className   = `model-test-result ${ok ? "result-ok" : "result-fail"}`;
     }
 
     if (!saveBtn.dataset.bound) {
@@ -1767,15 +1877,26 @@ const Settings = (() => {
       saveBtn.addEventListener("click", async () => {
         status.textContent = "";
         status.className   = "model-test-result";
+
+        const provider = select.value;
+        const key = keyInput.value.trim();
+        // DeepSeek falls back to the configured model key, so an empty field is
+        // valid there. Every other searcher is a script that reads the key from
+        // CLACKY_SEARCH_KEY and cannot run without one.
+        if (provider && provider !== DS_WEB && !key) {
+          _showStatus(I18n.t("settings.search.keyRequired"), false);
+          return;
+        }
+
         try {
           const res = await fetch("/api/config/search", {
             method:  "PATCH",
             headers: { "Content-Type": "application/json" },
-            body:    JSON.stringify({ provider: select.value, key: keyInput.value })
+            body:    JSON.stringify({ provider: select.value, key: key })
           });
           const data = await res.json();
           if (data.ok) {
-            _showStatus(I18n.t("settings.search.saved"), true);
+            Modal.toast(I18n.t("settings.search.saved"), "success");
           } else {
             _showStatus(data.error || I18n.t("settings.search.failed"), false);
           }
@@ -1987,6 +2108,20 @@ const Settings = (() => {
         badge.className    = "brand-status-value badge-active";
       }
 
+      // Only "unreachable" is ambiguous on its own: it means heartbeats have
+      // been failing past the grace period, not that the license is invalid.
+      const hint = document.getElementById("brand-status-hint");
+      if (hint) {
+        if (data.warning && data.warning_type === "unreachable") {
+          // CJK copy reads best with full-width brackets; latin locales don't.
+          const note = I18n.t("settings.brand.hint.unreachable");
+          hint.textContent   = I18n.lang() === "zh" ? `（${note}）` : `(${note})`;
+          hint.style.display = "";
+        } else {
+          hint.style.display = "none";
+        }
+      }
+
       // Fetch full brand info for expiry date and support QR code
       fetch("/api/brand").then(r => r.json()).then(info => {
         const expiresEl = document.getElementById("brand-status-expires");

data/lib/clacky/web/ws-dispatcher.js

@@ -289,7 +289,7 @@ function _dispatchEvent(ev) {
 
     // ── Session list ───────────────────────────────────────────────────
     case "session_list": {
-      Sessions.setAll(ev.sessions || [], !!ev.has_more, ev.groups || {});
+      Sessions.setAll(ev.sessions || [], !!ev.has_more, ev.groups || {}, ev.next_cursor || null);
       Projects.setAll(ev.projects || []);
       Sessions.renderList();
       Projects.renderSection();
@@ -369,18 +369,11 @@ function _dispatchEvent(ev) {
       }
       if (!sid) break;
       // A creation update is emitted only after the session's initial metadata
-      // has been persisted. Promote it into the canonical list; ordinary full
-      // snapshots remain patches so reconnects and old-session updates cannot
-      // disturb the pagination boundary.
-      //
-      // Exception: the session the user is currently viewing. It may have been
-      // opened from search / a deep link and live only in the `_extraSessions`
-      // cache (outside the loaded page), which the sidebar does not render —
-      // patching it would leave the list without a row until a reload. A full
-      // snapshot carries every column the list needs, and its fresh
-      // `updated_at` sorts it to the top, exactly where a reload would put it.
+      // has been persisted. Other full snapshots join the canonical list only
+      // when they sort inside the loaded pages — exactly where a reload would
+      // put them — so stale snapshots cannot disturb the pagination boundary.
       const listed = Sessions.all.some(s => s.id === sid);
-      if (ev.session && (ev.created || (sid === Sessions.activeId && !listed))) {
+      if (ev.session && (ev.created || (!listed && Sessions.belongsToLoadedPage(ev.session)))) {
         Sessions.add(ev.session);
       } else {
         Sessions.patch(sid, patch);
@@ -418,7 +411,6 @@ function _dispatchEvent(ev) {
       Sessions.patch(ev.session_id, { name: ev.name });
       Sessions.renderList();
       Projects.renderSection();
-      // Title is now shown only in the sidebar; chat-header element was removed.
       break;
     }
 
@@ -672,6 +664,10 @@ function _dispatchEvent(ev) {
       if (typeof Share !== "undefined") Share.maybePromptOnComplete();
       break;
 
+    case "feedback_countdown":
+      Sessions.updateFeedbackCountdown(ev);
+      break;
+
     case "request_feedback":
       if (ev.session_id !== Sessions.activeId) break;
       Sessions.showFeedbackRequest(ev.question, ev.context, ev.options, ev.questions);

data/lib/clacky.rb

@@ -111,6 +111,7 @@ require_relative "clacky/utils/browser_detector"
 require_relative "clacky/utils/scripts_manager"
 require_relative "clacky/utils/searcher_manager"
 require_relative "clacky/search_config"
+require_relative "clacky/deepseek_web_tools"
 require_relative "clacky/utils/model_pricing"
 require_relative "clacky/utils/gitignore_parser"
 require_relative "clacky/utils/limit_stack"

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: openclacky
 version: !ruby/object:Gem::Version
-  version: 1.5.20
+  version: 1.5.21
 platform: ruby
 authors:
 - windy
@@ -346,6 +346,7 @@ files:
 - lib/clacky/agent/chunk_index.rb
 - lib/clacky/agent/cost_tracker.rb
 - lib/clacky/agent/fake_tool_call_detector.rb
+- lib/clacky/agent/feedback.rb
 - lib/clacky/agent/goal_manager.rb
 - lib/clacky/agent/goal_state.rb
 - lib/clacky/agent/history_navigation.rb
@@ -377,6 +378,7 @@ files:
 - lib/clacky/cli.rb
 - lib/clacky/cli_guidance.rb
 - lib/clacky/client.rb
+- lib/clacky/deepseek_web_tools.rb
 - lib/clacky/default_extensions/advisor/advisors/general.md
 - lib/clacky/default_extensions/advisor/api/handler.rb
 - lib/clacky/default_extensions/advisor/ext.yml
@@ -590,6 +592,7 @@ files:
 - lib/clacky/server/git_panel.rb
 - lib/clacky/server/http_server.rb
 - lib/clacky/server/model_prices.rb
+- lib/clacky/server/pid_files.rb
 - lib/clacky/server/preview.rb
 - lib/clacky/server/project_manager.rb
 - lib/clacky/server/scheduler.rb