openclacky 1.5.13 → 1.5.15

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

@@ -290,6 +290,37 @@ const BillingView = (() => {
 
   // ── Tooltips ─────────────────────────────────────────────────────────────────
 
+  // Measured while hidden so the real size is known before choosing a side.
+  // The clamp matters on narrow viewports: flipping a 200px-wide tooltip left of
+  // a cursor near the left edge would otherwise push it off-screen.
+  //
+  // Below MOBILE_TOOLTIP_MAX_WIDTH the tooltip is as wide as 58%-73% of the chart,
+  // so following the pointer covers the data no matter which side it flips to.
+  // There it docks just outside `chart` instead, keeping the plot fully visible.
+  const MOBILE_TOOLTIP_MAX_WIDTH = 768;
+
+  function _placeTooltip(tooltip, e, chart) {
+    tooltip.style.display = "block";
+    tooltip.style.visibility = "hidden";
+    const rect = tooltip.getBoundingClientRect();
+
+    if (chart && window.innerWidth <= MOBILE_TOOLTIP_MAX_WIDTH) {
+      const chartRect = chart.getBoundingClientRect();
+      const below = chartRect.bottom + 8;
+      const top = below + rect.height > window.innerHeight ? chartRect.top - 8 - rect.height : below;
+      tooltip.style.left = `${Math.max(8, Math.min(chartRect.left, window.innerWidth - 8 - rect.width))}px`;
+      tooltip.style.top = `${Math.max(8, top)}px`;
+      tooltip.style.visibility = "visible";
+      return;
+    }
+
+    const ovfX = e.clientX + 15 + rect.width - window.innerWidth;
+    const ovfY = e.clientY - 10 + rect.height - window.innerHeight;
+    tooltip.style.left = ovfX > 0 ? `${Math.max(8, e.clientX - 15 - rect.width)}px` : `${e.clientX + 15}px`;
+    tooltip.style.top = ovfY > 0 ? `${Math.max(8, e.clientY - 10 - rect.height)}px` : `${e.clientY - 10}px`;
+    tooltip.style.visibility = "visible";
+  }
+
   function _bindChartTooltip() {
     const container = document.getElementById("billing-chart-container");
     const tooltip = document.getElementById("billing-tooltip");
@@ -334,14 +365,7 @@ const BillingView = (() => {
           <span class="tooltip-value">${output}</span>
         </div>
       `;
-      tooltip.style.display = "block";
-      tooltip.style.visibility = "hidden";
-      const rect = tooltip.getBoundingClientRect();
-      const ovfX = e.clientX + 15 + rect.width - window.innerWidth;
-      const ovfY = e.clientY - 10 + rect.height - window.innerHeight;
-      tooltip.style.left = ovfX > 0 ? `${e.clientX - 15 - rect.width}px` : `${e.clientX + 15}px`;
-      tooltip.style.top = ovfY > 0 ? `${e.clientY - 10 - rect.height}px` : `${e.clientY - 10}px`;
-      tooltip.style.visibility = "visible";
+      _placeTooltip(tooltip, e, container);
     });
 
     container.addEventListener("mouseleave", () => {
@@ -376,27 +400,59 @@ const BillingView = (() => {
           <span class="tooltip-value">${cell.dataset.cost}</span>
         </div>
       `;
-      tooltip.style.display = "block";
-      tooltip.style.left = `${e.clientX + 15}px`;
-      tooltip.style.top = `${e.clientY - 10}px`;
+      _placeTooltip(tooltip, e, grid);
     });
 
     grid.addEventListener("mouseleave", () => {
       tooltip.style.display = "none";
+      tooltip.style.visibility = "";
     });
   }
 
+  // The dots are r="3" in viewBox units, which the responsive SVG scales down to
+  // under 5px on a phone — far too small to hit. Snap to the nearest point on the
+  // x axis instead so the whole column width is clickable.
+  function _nearestTrendDot(svg, clientX, clientY) {
+    const count = parseInt(svg.dataset.count, 10);
+    const xStep = parseFloat(svg.dataset.xStep);
+    const plotLeft = parseFloat(svg.dataset.plotLeft);
+    if (!count || !xStep) return null;
+    const ctm = svg.getScreenCTM();
+    if (!ctm) return null;
+    const point = svg.createSVGPoint();
+    point.x = clientX;
+    point.y = clientY;
+    const x = point.matrixTransform(ctm.inverse()).x;
+    const index = Math.max(0, Math.min(count - 1, Math.round((x - plotLeft) / xStep)));
+    return svg.querySelector(`.billing-trend-dot[data-index="${index}"]`);
+  }
+
   function _bindTrendTooltip() {
     const svg = document.querySelector(".billing-trend-svg");
     const tooltip = document.getElementById("billing-tooltip");
     if (!svg || !tooltip) return;
 
+    const crosshairX = svg.querySelector(".billing-trend-crosshair-x");
+    const crosshairY = svg.querySelector(".billing-trend-crosshair-y");
+
     svg.addEventListener("mousemove", (e) => {
-      const dot = e.target.closest(".billing-trend-dot");
+      const dot = _nearestTrendDot(svg, e.clientX, e.clientY);
       if (!dot) {
         tooltip.style.display = "none";
         return;
       }
+      svg.querySelectorAll(".billing-trend-dot.is-active").forEach(d => d.classList.remove("is-active"));
+      dot.classList.add("is-active");
+      if (crosshairX && crosshairY) {
+        const cx = dot.getAttribute("cx");
+        const cy = dot.getAttribute("cy");
+        crosshairX.setAttribute("x1", cx);
+        crosshairX.setAttribute("x2", cx);
+        crosshairY.setAttribute("y1", cy);
+        crosshairY.setAttribute("y2", cy);
+        crosshairX.classList.add("is-visible");
+        crosshairY.classList.add("is-visible");
+      }
       tooltip.innerHTML = `
         <div class="tooltip-header">
           <span class="tooltip-date">${dot.dataset.date}</span>
@@ -406,18 +462,14 @@ const BillingView = (() => {
           <span class="tooltip-value">${dot.dataset.cost}</span>
         </div>
       `;
-      tooltip.style.display = "block";
-      tooltip.style.visibility = "hidden";
-      const rect = tooltip.getBoundingClientRect();
-      const ovf = e.clientX + 15 + rect.width - window.innerWidth;
-      tooltip.style.left = ovf > 0 ? `${e.clientX - 15 - rect.width}px` : `${e.clientX + 15}px`;
-      tooltip.style.top = `${e.clientY - 10}px`;
-      tooltip.style.visibility = "visible";
+      _placeTooltip(tooltip, e, svg);
     });
 
     svg.addEventListener("mouseleave", () => {
       tooltip.style.display = "none";
       tooltip.style.visibility = "";
+      svg.querySelectorAll(".billing-trend-dot.is-active").forEach(d => d.classList.remove("is-active"));
+      svg.querySelectorAll(".billing-trend-crosshair").forEach(l => l.classList.remove("is-visible"));
     });
   }
 
@@ -641,7 +693,7 @@ const BillingView = (() => {
           <h4>${I18n.t("billing.costTrend") || "Cost Trend"}</h4>
         </div>
         <div class="billing-trend-chart">
-          <svg viewBox="0 0 ${w} ${h}" preserveAspectRatio="xMidYMid meet" class="billing-trend-svg">
+          <svg viewBox="0 0 ${w} ${h}" preserveAspectRatio="xMidYMid meet" class="billing-trend-svg" data-plot-left="${pad.left}" data-x-step="${xStep}" data-count="${costs.length}">
             ${yLabels.map(l => `
               <line x1="${pad.left}" y1="${l.y}" x2="${w - pad.right}" y2="${l.y}" class="billing-trend-grid-line" />
               <text x="${pad.left - 6}" y="${l.y + 4}" class="billing-trend-y-label">${currencySymbol}${_formatCost(l.val)}</text>
@@ -656,11 +708,13 @@ const BillingView = (() => {
               </linearGradient>
             </defs>
             <polygon points="${areaPoints}" fill="url(#billing-trend-grad)" class="billing-trend-area" />
+            <line class="billing-trend-crosshair billing-trend-crosshair-x" x1="0" y1="${pad.top}" x2="0" y2="${pad.top + plotH}" />
+            <line class="billing-trend-crosshair billing-trend-crosshair-y" x1="${pad.left}" y1="0" x2="${w - pad.right}" y2="0" />
             <polyline points="${points}" fill="none" class="billing-trend-line" />
             ${costs.map((c, i) => {
               const cx = pad.left + i * xStep;
               const cy = pad.top + plotH - ((c - minCost) / range) * plotH;
-              return `<circle cx="${cx}" cy="${cy}" r="3" class="billing-trend-dot" data-date="${days[i].date}" data-cost="${currencySymbol}${_formatCost(c)}" />`;
+              return `<circle cx="${cx}" cy="${cy}" r="3" class="billing-trend-dot" data-index="${i}" data-date="${days[i].date}" data-cost="${currencySymbol}${_formatCost(c)}" />`;
             }).join("")}
           </svg>
         </div>

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

@@ -273,7 +273,10 @@ const ExtensionsStore = (() => {
       _error   = null;
       _emit("extensions:loading");
       try {
-        const res  = await fetch("/api/store/extensions/brand");
+        const params = new URLSearchParams();
+        if (_sort) params.set("sort", _sort);
+        const qs   = params.toString();
+        const res  = await fetch("/api/store/extensions/brand" + (qs ? "?" + qs : ""));
         const data = await res.json();
         const all  = data.extensions || [];
         const matched = _query

data/lib/clacky/web/features/model-tester/store.js

@@ -1,9 +1,9 @@
 // ── ModelTester · store — model connection test + save (shared helper) ────
 //
 // Network helpers shared by the onboarding wizard and the settings model modal:
-// test a model connection and persist a model config. No own panel, no state to
-// hold — it mirrors test/save outcomes onto the extension bus so extensions can
-// observe model-config changes.
+// test a model connection, persist a model config, and list the models an
+// endpoint advertises. No own panel, no state to hold — it mirrors test/save
+// outcomes onto the extension bus so extensions can observe model-config changes.
 //
 // `ModelTester` stays the single public facade.
 //
@@ -72,7 +72,40 @@ window.ModelTester = (function () {
     }
   }
 
-  return { testConnection, saveModel };
+  // Ask the server to query `<base_url>/models` for the ids the endpoint
+  // advertises. The key travels in the POST body (never the URL) because it
+  // is used for this one request only; a masked key is resolved server-side
+  // from the stored config via `id`/`index`, same as testConnection.
+  async function listModels({ base_url, api_key, api_format, index, id } = {}) {
+    const body = { base_url, api_key };
+    if (typeof id === "string" && id) body.id = id;
+    if (typeof index === "number") body.index = index;
+    if (api_format) body.api_format = api_format;
+
+    let data;
+    let status = 0;
+    try {
+      const res = await fetch("/api/config/models/list", {
+        method:  "POST",
+        headers: { "Content-Type": "application/json" },
+        body:    JSON.stringify(body)
+      });
+      status = res.status;
+      data = await res.json();
+    } catch (e) {
+      return { ok: false, message: e.message };
+    }
+
+    // Validation failures (400/422) carry `error` instead of `message`;
+    // never render an empty reason tail.
+    if (!data.ok) return { ok: false, message: data.message || data.error || ("HTTP " + status) };
+
+    const models = Array.isArray(data.models) ? data.models : [];
+    _emit("modeltester:listed", { base_url, count: models.length });
+    return { ok: true, models, message: data.message || "" };
+  }
+
+  return { testConnection, saveModel, listModels };
 })();
 
 const ModelTester = window.ModelTester;

data/lib/clacky/web/features/skills/store.js

@@ -262,7 +262,7 @@ const SkillsStore = (() => {
         const res  = await fetch(`/api/skills/${encodeURIComponent(name)}/content`);
         const data = await res.json();
         if (!data.ok) throw new Error(data.error || "Load failed");
-        return { ok: true, content: data.content || "", path: data.path };
+        return { ok: true, content: data.content || "", path: data.path, fields: data.fields };
       } catch (e) {
         return { ok: false, error: e.message };
       }
@@ -285,6 +285,23 @@ const SkillsStore = (() => {
       }
     },
 
+    /** Update a custom skill's editable fields (name/name_zh/description/description_zh/body). */
+    async updateSkillFields(name, fields) {
+      try {
+        const res  = await fetch(`/api/skills/${encodeURIComponent(name)}/content`, {
+          method:  "PUT",
+          headers: { "Content-Type": "application/json" },
+          body:    JSON.stringify({ fields })
+        });
+        const data = await res.json();
+        if (!res.ok || !data.ok) throw new Error(data.error || "Save failed");
+        await Skills.load();
+        return { ok: true };
+      } catch (e) {
+        return { ok: false, error: e.message };
+      }
+    },
+
     /** Import a skill: validate url/path, open a session, run /skill-add. */
     async importSkill(url) {
       const trimmed = (url || "").trim();

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

@@ -596,16 +596,137 @@ const SkillsView = (() => {
   async function _editSkill(skill) {
     const res = await Skills.fetchSkillContent(skill.name);
     if (!res.ok) { alert(I18n.t("skills.editFail") + ": " + res.error); return; }
+    _openSkillFormEditor(skill, res);
+  }
+
+  function _openSkillFormEditor(skill, res) {
+    const f = res.fields || {};
+
+    let overlay = document.getElementById("skill-editor-overlay");
+    if (overlay) overlay.remove();
+
+    overlay = document.createElement("div");
+    overlay.id = "skill-editor-overlay";
+    overlay.className = "modal-overlay";
+
+    const cancelLabel = I18n.t("modal.cancel");
+    const saveLabel   = I18n.t("modal.save");
+
+    overlay.innerHTML = `
+      <div class="modal-box skill-editor-box">
+        <div class="modal-header">
+          <h3 class="modal-title"></h3>
+          <button class="modal-close-btn" title="${cancelLabel}"></button>
+        </div>
+        <div class="modal-body">
+          <div class="modal-field">
+            <label class="modal-label" for="se-name">${I18n.t("skills.edit.name")}</label>
+            <input type="text" id="se-name" class="modal-input skill-editor-name">
+            <span class="form-hint">${I18n.t("skills.edit.nameHint")}</span>
+          </div>
+          <div class="modal-field">
+            <label class="modal-label" for="se-name-zh">${I18n.t("skills.edit.nameZh")}</label>
+            <input type="text" id="se-name-zh" class="modal-input skill-editor-name-zh">
+          </div>
+          <div class="modal-field">
+            <label class="modal-label" for="se-desc">${I18n.t("skills.edit.description")}</label>
+            <textarea id="se-desc" class="modal-input skill-editor-description" rows="2"></textarea>
+          </div>
+          <div class="modal-field">
+            <label class="modal-label" for="se-desc-zh">${I18n.t("skills.edit.descriptionZh")}</label>
+            <textarea id="se-desc-zh" class="modal-input skill-editor-description-zh" rows="2"></textarea>
+          </div>
+          <div class="modal-field">
+            <label class="modal-label" for="se-body">${I18n.t("skills.edit.body")}</label>
+            <textarea id="se-body" class="modal-input skill-editor-body-input" rows="10"></textarea>
+          </div>
+        </div>
+        <div class="modal-footer">
+          <span class="skill-editor-status"></span>
+          <button class="btn-secondary skill-editor-cancel">${cancelLabel}</button>
+          <button class="btn-secondary skill-editor-markdown">${I18n.t("skills.edit.editMarkdown")}</button>
+          <button class="btn-primary skill-editor-save">${saveLabel}</button>
+        </div>
+      </div>`;
+
+    document.body.appendChild(overlay);
+    overlay.querySelector(".modal-title").textContent = I18n.t("skills.edit.title") + ": " + skill.name;
+
+    const nameInput   = overlay.querySelector(".skill-editor-name");
+    const nameZhInput = overlay.querySelector(".skill-editor-name-zh");
+    const descInput   = overlay.querySelector(".skill-editor-description");
+    const descZhInput = overlay.querySelector(".skill-editor-description-zh");
+    const bodyInput   = overlay.querySelector(".skill-editor-body-input");
+    const status      = overlay.querySelector(".skill-editor-status");
+    const closeBtn    = overlay.querySelector(".modal-close-btn");
+    const cancelBtn   = overlay.querySelector(".skill-editor-cancel");
+    const markdownBtn = overlay.querySelector(".skill-editor-markdown");
+    const saveBtn     = overlay.querySelector(".skill-editor-save");
+
+    nameInput.value   = f.name || skill.name || "";
+    nameZhInput.value = f.name_zh || "";
+    descInput.value   = f.description || "";
+    descZhInput.value = f.description_zh || "";
+    bodyInput.value   = f.body || "";
+
+    // Auto-grow textareas so the modal body is the only scroll container.
+    [descInput, descZhInput, bodyInput].forEach((ta) => {
+      const autoResize = () => {
+        ta.style.height = "auto";
+        ta.style.height = ta.scrollHeight + "px";
+      };
+      ta.addEventListener("input", autoResize);
+      autoResize();
+    });
 
-    CodeEditor.open({
-      content: res.content,
-      title: skill.name,
-      language: "markdown",
-      onSave: async (newContent) => {
-        const r = await Skills.updateSkillContent(skill.name, newContent);
-        if (!r.ok) throw new Error(r.error);
+    function close() {
+      overlay.remove();
+    }
+
+    closeBtn.addEventListener("click", close);
+    cancelBtn.addEventListener("click", close);
+
+    markdownBtn.addEventListener("click", () => {
+      close();
+      CodeEditor.open({
+        content: res.content,
+        title: skill.name,
+        language: "markdown",
+        onSave: async (newContent) => {
+          const r = await Skills.updateSkillContent(skill.name, newContent);
+          if (!r.ok) throw new Error(r.error);
+        }
+      });
+    });
+
+    saveBtn.addEventListener("click", async () => {
+      const name = nameInput.value.trim();
+      if (!name) {
+        nameInput.classList.add("input-error");
+        nameInput.focus();
+        return;
       }
+
+      const fields = {
+        name:           name,
+        name_zh:        nameZhInput.value.trim(),
+        description:    descInput.value.trim(),
+        description_zh: descZhInput.value.trim(),
+        body:           bodyInput.value
+      };
+
+      saveBtn.disabled = true;
+      status.textContent = I18n.t("modal.saving");
+      status.className = "skill-editor-status";
+      const r = await Skills.updateSkillFields(skill.name, fields);
+      if (r.ok) { close(); return; }
+
+      status.textContent = I18n.t("skills.edit.saveFail") + (r.error ? ": " + r.error : "");
+      status.className = "skill-editor-status skill-editor-status-error";
+      saveBtn.disabled = false;
     });
+
+    setTimeout(() => nameInput.focus(), 50);
   }
 
   async function _doImportFromBar() {

data/lib/clacky/web/features/workspace/store.js

@@ -13,6 +13,7 @@ const WorkspaceStore = (() => {
   let _sessionId  = null;
   let _workingDir = null;
 
+  const _defaultAppCache = {};
   const _listeners = {};
 
   function _on(event, handler) {
@@ -30,7 +31,22 @@ const WorkspaceStore = (() => {
   }
 
   function _absPath(relPath) {
-    return _workingDir.replace(/\/+$/, "") + "/" + relPath;
+    // Absolute input (POSIX, Windows drive letter, or ~-home) is used as-is so
+    // file:// links from chat can point outside the working directory. A
+    // leading ~ is expanded by the backend via File.expand_path.
+    if (/^([A-Za-z]:[\\/]|\/|~)/.test(relPath)) return relPath;
+    return (_workingDir || "").replace(/\/+$/, "") + "/" + relPath;
+  }
+
+  // Throw with the backend's error message when available so toasts show the
+  // real cause ("file not found", "unsupported OS", …) instead of "HTTP 404".
+  async function _respError(resp) {
+    let detail = `HTTP ${resp.status}`;
+    try {
+      const data = await resp.json();
+      if (data && data.error) detail = data.error;
+    } catch (_e) { /* non-JSON body */ }
+    throw new Error(detail);
   }
 
   const state = {
@@ -43,6 +59,9 @@ const WorkspaceStore = (() => {
     on: _on,
     state,
 
+    /** Resolve an entry-relative (or absolute) path to an absolute path. */
+    absPath(relPath) { return _absPath(relPath); },
+
     /** Update active session context. Returns { changed, hadSession }. */
     setSession(session) {
       const newId      = session ? session.id : null;
@@ -55,11 +74,21 @@ const WorkspaceStore = (() => {
       return { changed, hadSession };
     },
 
+    /** ws-dispatcher signals a finished agent task; the view reloads the tree
+     *  because the agent may have created/edited files under the working dir. */
+    notifyTaskCompleted(sessionId) {
+      _emit("workspace:taskCompleted", { sessionId });
+    },
+
     async fetchEntries(relPath) {
       const url  = `/api/sessions/${encodeURIComponent(_sessionId)}/files?path=${encodeURIComponent(relPath || "")}`;
       const resp = await fetch(url);
-      if (!resp.ok) throw new Error(`HTTP ${resp.status}`);
+      if (!resp.ok) await _respError(resp);
       const data = await resp.json();
+      // The root the server actually listed is the authoritative base for
+      // the relative entry paths it returns — keeps _absPath in sync with
+      // what the tree shows even if a session-context update lands stale.
+      if (data.root) _workingDir = data.root;
       return data.entries || [];
     },
 
@@ -69,7 +98,51 @@ const WorkspaceStore = (() => {
         headers: { "Content-Type": "application/json" },
         body: JSON.stringify({ path: _absPath(entry.path), action: "reveal" })
       });
-      if (!resp.ok) throw new Error(`HTTP ${resp.status}`);
+      if (!resp.ok) await _respError(resp);
+    },
+
+    /** Installed applications that can open this file (macOS; [] elsewhere). */
+    async fetchOpenWithApps(entry) {
+      const resp = await fetch("/api/file/apps?path=" + encodeURIComponent(_absPath(entry.path)));
+      if (!resp.ok) return [];
+      const data = await resp.json();
+      return data.apps || [];
+    },
+
+    /** Default application the OS would use to open this file; null if none.
+     *  Cached per extension so tab switches don't refetch. */
+    async fetchDefaultApp(entry) {
+      const ext = (entry.path.match(/\.([^./\\]+)$/) || [])[1];
+      if (!ext) return null;
+      const key = ext.toLowerCase();
+      if (Object.prototype.hasOwnProperty.call(_defaultAppCache, key)) {
+        return _defaultAppCache[key];
+      }
+      const resp = await fetch("/api/file/default-app?path=" + encodeURIComponent(_absPath(entry.path)));
+      const data = resp.ok ? await resp.json() : null;
+      const app = data && data.ok ? data.app || null : null;
+      _defaultAppCache[key] = app;
+      return app;
+    },
+
+    /** Open the file with one of the apps returned by fetchOpenWithApps. */
+    async openWithApp(entry, app) {
+      const resp = await fetch("/api/file-action", {
+        method: "POST",
+        headers: { "Content-Type": "application/json" },
+        body: JSON.stringify({ path: _absPath(entry.path), action: "open_with", app })
+      });
+      if (!resp.ok) await _respError(resp);
+    },
+
+    /** Open the file with the OS default application. */
+    async openWithDefault(entry) {
+      const resp = await fetch("/api/file-action", {
+        method: "POST",
+        headers: { "Content-Type": "application/json" },
+        body: JSON.stringify({ path: _absPath(entry.path), action: "open" })
+      });
+      if (!resp.ok) await _respError(resp);
     },
 
     async displayPath(entry) {
@@ -78,7 +151,7 @@ const WorkspaceStore = (() => {
         headers: { "Content-Type": "application/json" },
         body: JSON.stringify({ path: _absPath(entry.path), action: "display-path" })
       });
-      if (!resp.ok) throw new Error(`HTTP ${resp.status}`);
+      if (!resp.ok) await _respError(resp);
       const data = await resp.json();
       return data.path;
     },
@@ -89,7 +162,7 @@ const WorkspaceStore = (() => {
         headers: { "Content-Type": "application/json" },
         body: JSON.stringify({ path: _absPath(entry.path), action: "download" })
       });
-      if (!resp.ok) throw new Error(`HTTP ${resp.status}`);
+      if (!resp.ok) await _respError(resp);
       return resp.blob();
     },
 
@@ -99,7 +172,7 @@ const WorkspaceStore = (() => {
         headers: { "Content-Type": "application/json" },
         body: JSON.stringify({ path: _absPath(entry.path), action: "download" })
       });
-      if (!resp.ok) throw new Error(`HTTP ${resp.status}`);
+      if (!resp.ok) await _respError(resp);
       return resp.text();
     },
 
@@ -109,7 +182,7 @@ const WorkspaceStore = (() => {
         headers: { "Content-Type": "application/json" },
         body: JSON.stringify({ path: _absPath(entry.path), action: "save", content })
       });
-      if (!resp.ok) throw new Error(`HTTP ${resp.status}`);
+      if (!resp.ok) await _respError(resp);
       _emit("workspace:fileSaved", { sessionId: _sessionId, path: entry.path, name: entry.name });
     },
   };