@cremini/skillpack 1.1.9 → 1.2.1

package/web/js/chat-apps-dialog.js

@@ -101,14 +101,11 @@ function populateForm() {
 
   // Restart required status
   if (state.restartRequired) {
-    const canRestart = config.runtimeControl?.canManagedRestart;
     setStatus(
-      canRestart
-        ? "Settings changed. Restart service to apply."
-        : "Settings changed. Restart the service manually to apply.",
+      "Settings changed. Restart service to apply.",
       "warning",
     );
-    updateRestartButton(canRestart);
+    updateRestartButton(true);
   } else {
     setStatus("", "");
     updateRestartButton(false);
@@ -139,17 +136,14 @@ async function handleSave() {
     const res = await saveConfigData(updates);
 
     state.config.adapters = res.adapters;
-    state.config.runtimeControl = res.runtimeControl;
     state.restartRequired = !!res.requiresRestart;
 
     if (res.requiresRestart) {
       setStatus(
-        res.runtimeControl?.canManagedRestart
-          ? "Settings saved. Restart service to apply changes."
-          : "Settings saved. Restart the service manually to apply changes.",
+        "Settings saved. Restart service to apply changes.",
         "warning",
       );
-      updateRestartButton(!!res.runtimeControl?.canManagedRestart);
+      updateRestartButton(true);
     } else {
       close();
     }

package/web/js/main.js

@@ -1,5 +1,4 @@
 import { state, loadConfig } from "./config.js";
-import { initSettings } from "./settings.js";
 import { initApiKeyDialog, updateApiKeyButton } from "./api-key-dialog.js";
 import { initChatAppsDialog, updateChatAppsButton } from "./chat-apps-dialog.js";
 import { initChat, showWelcome } from "./chat.js";
@@ -42,7 +41,6 @@ async function init() {
   }
   
   // Initialize all dialog modules
-  initSettings();
   initApiKeyDialog();
   initChatAppsDialog();
   initChat();

package/web/styles.css

@@ -72,13 +72,6 @@ body {
   line-height: 1.4;
 }
 
-.sidebar-settings-section {
-  padding: 16px 20px;
-  border-top: 1px solid var(--border-color);
-  background: var(--bg-secondary);
-  flex-shrink: 0;
-}
-
 .sidebar-skills-section {
   padding: 16px 20px;
   flex: 1;
@@ -874,26 +867,6 @@ body {
   width: 440px;
 }
 
-/* ---- Settings Modal ---- */
-.settings-trigger-btn {
-  display: flex;
-  align-items: center;
-  justify-content: center;
-  gap: 8px;
-  width: 100%;
-  padding: 12px;
-  background: var(--bg-tertiary);
-  border: 1px solid var(--border-color);
-  border-radius: var(--radius-sm);
-  color: var(--text-primary);
-  font-weight: 500;
-  cursor: pointer;
-  transition: all 0.2s ease;
-}
-.settings-trigger-btn:hover {
-  background: #ebebeb;
-}
-
 .settings-modal {
   margin: auto;
   padding: 0;
@@ -1073,3 +1046,68 @@ body {
   transform: none;
   box-shadow: none;
 }
+.label-hint {
+  color: var(--text-secondary);
+  font-size: 11px;
+  font-weight: 400;
+  margin-left: 4px;
+}
+
+/* OAuth UI */
+#apikey-oauth-section {
+  padding: 8px 0;
+}
+
+.oauth-status-card {
+  background: var(--bg-tertiary);
+  border: 1px solid var(--border-color);
+  border-radius: var(--radius-sm);
+  padding: 16px;
+  margin-bottom: 16px;
+}
+
+#oauth-status-indicator {
+  display: flex;
+  align-items: center;
+  gap: 8px;
+  font-size: 13px;
+  font-weight: 600;
+  margin-bottom: 8px;
+}
+
+.oauth-status-dot {
+  width: 8px;
+  height: 8px;
+  border-radius: 50%;
+}
+
+.oauth-disconnected .oauth-status-dot {
+  background-color: #94a3b8;
+}
+
+.oauth-connected .oauth-status-dot {
+  background-color: #22c55e;
+}
+
+.oauth-disconnected #oauth-status-text {
+  color: var(--text-secondary);
+}
+
+.oauth-connected #oauth-status-text {
+  color: #16a34a;
+}
+
+.oauth-hint {
+  font-size: 12px;
+  color: var(--text-muted);
+  line-height: 1.5;
+}
+
+.oauth-btn {
+  width: 100%;
+  margin-top: 8px;
+}
+
+.oauth-btn svg {
+  margin-right: 8px;
+}

package/web/js/settings.js

@@ -1,205 +0,0 @@
-import { state } from "./config.js";
-import { restartRuntime, saveConfigData } from "./api.js";
-
-// DOM Elements
-let dialog;
-let settingsBtn;
-let closeBtn;
-let saveBtn;
-let providerSelect;
-let apiKeyInput;
-let telegramTokenInput;
-let slackBotTokenInput;
-let slackAppTokenInput;
-let keyStatus;
-let restartBtn;
-
-export function initSettings() {
-  dialog = document.getElementById("settings-dialog");
-  settingsBtn = document.getElementById("open-settings-btn");
-  closeBtn = document.getElementById("close-settings-btn");
-  saveBtn = document.getElementById("save-settings-btn");
-  
-  providerSelect = document.getElementById("provider-select");
-  apiKeyInput = document.getElementById("api-key-input");
-  telegramTokenInput = document.getElementById("telegram-token-input");
-  slackBotTokenInput = document.getElementById("slack-bot-token-input");
-  slackAppTokenInput = document.getElementById("slack-app-token-input");
-  keyStatus = document.getElementById("key-status");
-  restartBtn = document.getElementById("restart-service-btn");
-
-  if (!dialog) return;
-
-  // Open/Close dialog
-  if (settingsBtn) {
-    settingsBtn.addEventListener("click", () => {
-      populateForm();
-      dialog.showModal();
-    });
-  }
-  
-  if (closeBtn) {
-    closeBtn.addEventListener("click", () => {
-      dialog.close();
-      keyStatus.textContent = ""; // clear status on close
-    });
-  }
-
-  // Save Settings
-  if (saveBtn) {
-    saveBtn.addEventListener("click", handleSave);
-  }
-  if (restartBtn) {
-    restartBtn.addEventListener("click", handleRestart);
-  }
-
-  // Placeholder logic
-  if (providerSelect) {
-    providerSelect.addEventListener("change", updatePlaceholder);
-  }
-}
-
-function updatePlaceholder() {
-  const p = providerSelect.value;
-  if (p === "openai") apiKeyInput.placeholder = "sk-proj-...";
-  else if (p === "anthropic") apiKeyInput.placeholder = "sk-ant-api03-...";
-  else apiKeyInput.placeholder = "sk-...";
-}
-
-function populateForm() {
-  const config = state.config;
-  if (!config) return;
-
-  if (state.restartRequired) {
-    setStatus(
-      config.runtimeControl?.canManagedRestart
-        ? "Settings saved. Restart service to apply changes."
-        : "Settings saved. Restart the service manually to apply changes.",
-      "warning",
-    );
-    updateRestartButton(true);
-  } else {
-    setStatus("", "");
-    updateRestartButton(false);
-  }
-
-  if (config.hasApiKey && config.apiKey) {
-    apiKeyInput.value = config.apiKey;
-  } else if (config.hasApiKey) {
-    apiKeyInput.value = "***************************************************";
-  } else {
-    apiKeyInput.value = "";
-  }
-
-  if (config.provider) {
-    providerSelect.value = config.provider;
-  }
-  updatePlaceholder();
-
-  const adapters = config.adapters || {};
-  if (adapters.telegram && adapters.telegram.token) {
-    telegramTokenInput.value = adapters.telegram.token;
-  } else {
-    telegramTokenInput.value = "";
-  }
-
-  if (adapters.slack) {
-    slackBotTokenInput.value = adapters.slack.botToken || "";
-    slackAppTokenInput.value = adapters.slack.appToken || "";
-  } else {
-    slackBotTokenInput.value = "";
-    slackAppTokenInput.value = "";
-  }
-}
-
-async function handleSave() {
-  const key = apiKeyInput.value.trim();
-  const provider = providerSelect.value;
-  const telegramToken = telegramTokenInput.value.trim();
-  const slackBotToken = slackBotTokenInput.value.trim();
-  const slackAppToken = slackAppTokenInput.value.trim();
-
-  const adapters = {};
-  if (telegramToken) adapters.telegram = { token: telegramToken };
-  if (slackBotToken || slackAppToken) {
-    adapters.slack = { 
-       botToken: slackBotToken || undefined, 
-       appToken: slackAppToken || undefined 
-    };
-  }
-
-  const updates = { provider, adapters };
-  if (key && key !== "***************************************************" && key !== state.config.apiKey) {
-    updates.key = key;
-  }
-
-  try {
-    const res = await saveConfigData(updates);
-    
-    // Update local config
-    state.config.provider = res.provider;
-    state.config.adapters = res.adapters;
-    state.config.runtimeControl = res.runtimeControl;
-    if (updates.key) {
-      state.config.hasApiKey = true;
-      state.config.apiKey = updates.key;
-    }
-    
-    if (state.config.hasApiKey && state.config.apiKey) {
-      apiKeyInput.value = state.config.apiKey;
-    } else if (state.config.hasApiKey) {
-      apiKeyInput.value = "***************************************************";
-    } else {
-      apiKeyInput.value = "";
-    }
-    state.restartRequired = !!res.requiresRestart;
-
-    if (res.requiresRestart) {
-      setStatus(
-        res.runtimeControl.canManagedRestart
-          ? "Settings saved. Restart service to apply changes."
-          : "Settings saved. Restart the service manually to apply changes.",
-        "warning",
-      );
-      updateRestartButton(res.runtimeControl.canManagedRestart);
-      return;
-    }
-
-    setStatus("Settings saved", "success");
-    updateRestartButton(false);
-
-  } catch (err) {
-    setStatus("Save failed: " + err.message, "error");
-  }
-}
-
-async function handleRestart() {
-  if (!restartBtn) return;
-
-  restartBtn.disabled = true;
-  if (saveBtn) saveBtn.disabled = true;
-  setStatus("Restarting service...", "warning");
-
-  try {
-    await restartRuntime();
-    setTimeout(() => {
-      window.location.reload();
-    }, 6000);
-  } catch (err) {
-    if (saveBtn) saveBtn.disabled = false;
-    restartBtn.disabled = false;
-    setStatus("Restart failed: " + err.message, "error");
-  }
-}
-
-function updateRestartButton(show) {
-  if (!restartBtn) return;
-  restartBtn.hidden = !show;
-  restartBtn.disabled = false;
-}
-
-function setStatus(message, status) {
-  if (!keyStatus) return;
-  keyStatus.textContent = message;
-  keyStatus.className = status ? `status-text ${status}` : "status-text";
-}