openclacky 1.3.6 → 1.3.7

data/lib/clacky/web/app.js

@@ -31,7 +31,7 @@ function escapeHtml(str) {
 // Single source of truth for panel visibility and URL hash.
 //
 // Views:
-//   welcome            → /#
+//   welcome            → /#new     (new-session landing page)
 //   session/{id}       → /#session/{id}
 //   tasks              → /#tasks
 //   skills             → /#skills
@@ -54,11 +54,12 @@ const PANELS = [
   "skills-panel",
   "channels-panel",
   "mcp-panel",
+  "extensions-panel",
   "trash-panel",
   "profile-panel",
   "billing-panel",
   "settings-panel",
-  "creator-panel",
+  "ext-slot-main-workspace",
 ];
 
 const Router = (() => {
@@ -75,7 +76,12 @@ const Router = (() => {
   }
 
   // Update the URL hash without triggering a hashchange handler loop.
+  // Only arm the skip-flag if the hash is actually going to change — otherwise
+  // the browser fires no hashchange event and the flag would linger and
+  // silently swallow the next real hashchange.
   function _setHash(hash) {
+    const target = hash ? "#" + hash : "";
+    if (location.hash === target || (target === "" && location.hash === "")) return;
     _skipNextHashChange = true;
     location.hash = hash;
   }
@@ -84,10 +90,14 @@ const Router = (() => {
   function _parseHash(hash) {
     const h = (hash || "").replace(/^#\/?/, "");
     if (!h)                           return { view: "welcome",  params: {} };
+    if (h === "new")                  return { view: "welcome",  params: {} };
     if (h === "tasks")                return { view: "tasks",    params: {} };
     if (h === "skills")               return { view: "skills",   params: {} };
     if (h === "channels")             return { view: "channels", params: {} };
     if (h === "mcp")                  return { view: "mcp",      params: {} };
+    if (h === "extensions")           return { view: "extensions", params: {} };
+    const mExtDetail = h.match(/^extensions\/(.+)$/);
+    if (mExtDetail)                   return { view: "extensions", params: { detailId: mExtDetail[1] } };
     if (h === "trash")                return { view: "trash",    params: {} };
     if (h === "profile")              return { view: "profile",  params: {} };
     // Legacy: #memories redirects to #profile (memories are now merged into
@@ -95,7 +105,8 @@ const Router = (() => {
     if (h === "memories")             return { view: "profile",  params: {} };
     if (h === "billing")              return { view: "billing",  params: {} };
     if (h === "settings")             return { view: "settings", params: {} };
-    if (h === "creator")              return { view: "creator",  params: {} };
+    const mExt = h.match(/^ext\/(.+)$/);
+    if (mExt)                         return { view: "ext-workspace", params: { id: mExt[1] } };
     const m = h.match(/^session\/(.+)$/);
     if (m)                            return { view: "session",  params: { id: m[1] } };
     return { view: "welcome", params: {} };
@@ -109,10 +120,10 @@ const Router = (() => {
     skills:   "skills-sidebar-item",
     channels: "channels-sidebar-item",
     mcp:      "mcp-sidebar-item",
+    extensions: "extensions-sidebar-item",
     trash:    "trash-sidebar-item",
     profile:  "profile-sidebar-item",
     billing:  "billing-sidebar-item",
-    creator:  "creator-sidebar-item",
   };
 
   // Remove active highlight from all Router-managed sidebar items.
@@ -121,6 +132,10 @@ const Router = (() => {
       const el = $(id);
       if (el) el.classList.remove("active");
     });
+    // Extension workspace nav items tag themselves with data-ext-workspace;
+    // the Router owns their active highlight the same way as built-in items.
+    document.querySelectorAll("[data-ext-workspace].active")
+      .forEach(el => el.classList.remove("active"));
   }
 
   // Core: apply a view change. Called both from navigate() and hashchange.
@@ -238,6 +253,18 @@ const Router = (() => {
         Sessions.renderList();
         break;
 
+      case "extensions":
+        $("extensions-panel").style.display = "flex";
+        if (params.detailId) {
+          _setHash(`extensions/${params.detailId}`);
+          Extensions.onPanelShow({ detailId: params.detailId });
+        } else {
+          _setHash("extensions");
+          Extensions.onPanelShow();
+        }
+        Sessions.renderList();
+        break;
+
       case "trash":
         _setHash("trash");
         $("trash-panel").style.display = "flex";
@@ -259,13 +286,6 @@ const Router = (() => {
         Sessions.renderList();
         break;
 
-      case "creator":
-        _setHash("creator");
-        $("creator-panel").style.display = "flex";
-        Creator.onPanelShow();
-        Sessions.renderList();
-        break;
-
       case "settings":
         _setHash("settings");
         $("settings-panel").style.display = "";
@@ -284,10 +304,26 @@ const Router = (() => {
         $("onboard-panel").style.display = "flex";
         break;
 
+      case "ext-workspace": {
+        const ws = window.Clacky && Clacky.ext && Clacky.ext._getWorkspace(params.id);
+        if (!ws) { await _apply("welcome"); return; }
+        _setHash(`ext/${params.id}`);
+        const container = $("ext-slot-main-workspace");
+        container.replaceChildren();
+        container.style.display = "block";
+        try { ws.render(container, {}); }
+        catch (err) { console.error("ext workspace render failed:", err); }
+        Sessions.renderList();
+        break;
+      }
+
       default:  // "welcome"
-        _setHash("");
+        _setHash("new");
         $("welcome").style.display = "";
         Sessions.renderList();
+        if (window.NewSessionView && NewSessionView.onPanelShow) {
+          NewSessionView.onPanelShow();
+        }
         break;
     }
 
@@ -296,6 +332,9 @@ const Router = (() => {
     _clearSidebarActive();
     const activeItem = SIDEBAR_ITEMS[view];
     if (activeItem) $(activeItem)?.classList.add("active");
+    if (view === "ext-workspace" && params.id) {
+      document.querySelector(`[data-ext-workspace="${params.id}"]`)?.classList.add("active");
+    }
   }
 
   // Listen for browser back/forward (or manual hash edits).
@@ -308,6 +347,14 @@ const Router = (() => {
     _apply(view, params).catch(err => console.error("Router._apply failed:", err));
   });
 
+  // Extensions request navigation via a CustomEvent so ext.js has no direct
+  // dependency on Router. Detail is { view, params } — same shape as navigate().
+  document.addEventListener("clacky:ext:navigate", (e) => {
+    const { view, params } = e.detail || {};
+    if (!view) return;
+    _apply(view, params || {}).catch(err => console.error("Router._apply failed:", err));
+  });
+
   return {
     get current() { return _current; },
     get params()  { return _params; },
@@ -328,6 +375,8 @@ const Router = (() => {
   };
 })();
 
+Clacky.Router = Router;
+
 // ── Modal utility ─────────────────────────────────────────────────────────
 const Modal = (() => {
   /** Show a yes/no confirmation dialog. Returns a Promise<boolean>. */
@@ -484,6 +533,8 @@ const Modal = (() => {
   return { confirm, confirmOnce, prompt, rename };
 })();
 
+Clacky.Modal = Modal;
+
 // ── Toast helper ──────────────────────────────────────────────────────────
 // Non-blocking notification stack. Replaces alert() for success/error/info
 // feedback. Supports an optional action button (e.g. "Go check").
@@ -700,9 +751,6 @@ window.bootAfterBrand = async function() {
   // key_setup blocks boot entirely; onboard.js calls _bootUI() when done.
   if (needsOnboard && phase === "key_setup") return;
 
-  // Show creator sidebar entry if user_licensed
-  Creator.updateSidebarVisibility();
-
   // Initialize skill autocomplete
   SkillAC.init();
 

data/lib/clacky/web/auth.js

@@ -148,3 +148,5 @@ const Auth = (() => {
     get passed() { return _authPassed; },
   };
 })();
+
+Clacky.Auth = Auth;

data/lib/clacky/web/components/notify.js

@@ -146,6 +146,8 @@ const Notify = (() => {
   return { init, toggle, enabled, setEnabled, onTaskFinished };
 })();
 
+Clacky.Notify = Notify;
+
 // Initialize on load (button binding + initial icon state).
 if (document.readyState === "loading") {
   document.addEventListener("DOMContentLoaded", () => Notify.init());

data/lib/clacky/web/components/sidebar.js

@@ -23,16 +23,12 @@ const Sidebar = (() => {
     document.getElementById("skills-sidebar-item").addEventListener("click", () => Router.navigate("skills"));
     document.getElementById("channels-sidebar-item").addEventListener("click", () => Router.navigate("channels"));
     document.getElementById("mcp-sidebar-item").addEventListener("click", () => Router.navigate("mcp"));
+    document.getElementById("extensions-sidebar-item").addEventListener("click", () => Router.navigate("extensions"));
     document.getElementById("trash-sidebar-item").addEventListener("click", () => Router.navigate("trash"));
     document.getElementById("profile-sidebar-item").addEventListener("click", () => Router.navigate("profile"));
     document.getElementById("billing-sidebar-item").addEventListener("click", () => Router.navigate("billing"));
 
     // memories-sidebar-item is retained as a hidden legacy placeholder — no click handler.
-
-    // creator-sidebar-item is conditionally rendered (only when user_licensed).
-    // This ?. is a legitimate business guard, not defensive padding — the
-    // element genuinely may not exist in the DOM for unlicensed users.
-    document.getElementById("creator-sidebar-item")?.addEventListener("click", () => Router.navigate("creator"));
   }
 
   return { init };