tinywebui 0.1.4 → 0.1.5

package/src/config/approval.js

@@ -21,41 +21,47 @@
  * `autoApproveTools` never does.
  */
 export const APPROVAL_MODES = ['writes', 'all', 'off'];
-
 /** 'ask' or 'auto' for one concrete call. */
 export function approvalFor(cfg, hub, name, args) {
-  // The harness's own tools are part of the harness, not third-party code:
-  // they never ask, whatever the mode or overrides say. Approval is for MCP.
-  if (hub?.isLocal?.(name)) {
-    if (!hub.isGated?.(name, args)) return 'auto';
-    // manage_mcp can add a server that runs a command on this machine, so it
-    // asks even when the mode is 'off'. Only an explicit per-tool "auto" skips it.
-    return (cfg.autoApproveTools || []).includes(name) ? 'auto' : 'ask';
-  }
-  if ((cfg.confirmTools || []).includes(name)) return 'ask';
-  if ((cfg.autoApproveTools || []).includes(name)) return 'auto';
-  // Missing means off, not the config default: a caller that builds its own
-  // cfg (tests, evals) has no one to ask and must not stall waiting for one.
-  const mode = cfg.toolApproval || 'off';
-  if (mode === 'off') return 'auto';
-  if (mode === 'all') return 'ask';
-  return hub?.isReadOnly?.(name, args) ? 'auto' : 'ask';
+    // The harness's own tools are part of the harness, not third-party code:
+    // they never ask, whatever the mode or overrides say. Approval is for MCP.
+    if (hub?.isLocal?.(name)) {
+        if (!hub.isGated?.(name, args))
+            return 'auto';
+        // manage_mcp can add a server that runs a command on this machine, so it
+        // asks even when the mode is 'off'. Only an explicit per-tool "auto" skips it.
+        return (cfg.autoApproveTools || []).includes(name) ? 'auto' : 'ask';
+    }
+    if ((cfg.confirmTools || []).includes(name))
+        return 'ask';
+    if ((cfg.autoApproveTools || []).includes(name))
+        return 'auto';
+    // Missing means off, not the config default: a caller that builds its own
+    // cfg (tests, evals) has no one to ask and must not stall waiting for one.
+    const mode = cfg.toolApproval || 'off';
+    if (mode === 'off')
+        return 'auto';
+    if (mode === 'all')
+        return 'ask';
+    return hub?.isReadOnly?.(name, args) ? 'auto' : 'ask';
 }
-
 /** The per-tool override the settings panel shows: 'ask', 'auto' or 'default'. */
 export function overrideFor(cfg, name) {
-  if ((cfg.confirmTools || []).includes(name)) return 'ask';
-  if ((cfg.autoApproveTools || []).includes(name)) return 'auto';
-  return 'default';
+    if ((cfg.confirmTools || []).includes(name))
+        return 'ask';
+    if ((cfg.autoApproveTools || []).includes(name))
+        return 'auto';
+    return 'default';
 }
-
 /** The config patch that sets one tool's override. */
 export function setOverride(cfg, name, policy) {
-  const confirm = new Set(cfg.confirmTools || []);
-  const auto = new Set(cfg.autoApproveTools || []);
-  confirm.delete(name);
-  auto.delete(name);
-  if (policy === 'ask') confirm.add(name);
-  if (policy === 'auto') auto.add(name);
-  return { confirmTools: [...confirm], autoApproveTools: [...auto] };
+    const confirm = new Set(cfg.confirmTools || []);
+    const auto = new Set(cfg.autoApproveTools || []);
+    confirm.delete(name);
+    auto.delete(name);
+    if (policy === 'ask')
+        confirm.add(name);
+    if (policy === 'auto')
+        auto.add(name);
+    return { confirmTools: [...confirm], autoApproveTools: [...auto] };
 }