total-recall-brain 3.2.3 → 3.3.0

package/extension/background.js

@@ -1,7 +1,7 @@
 // Total Recall — Background Service Worker
 // Handles context menus, message routing, and passive browsing buffer.
 
-importScripts('lib/brain-client.js');
+importScripts('lib/preconfigured.js', 'lib/brain-client.js');
 
 // Destructure for convenience after importScripts populates self.BrainClient
 const { share, search, healthCheck } = self.BrainClient;

package/extension/lib/brain-client.js

@@ -2,64 +2,85 @@
 // Shared by background.js, popup, sidepanel, and options page.
 // Exposed as `self.BrainClient` namespace (no ES-module bundler required).
 
-const DEFAULT_BRAIN_URL = 'http://127.0.0.1:3000';
+(function () {
+  const DEFAULT_BRAIN_URL = 'http://127.0.0.1:3000';
 
-async function getConfig() {
-  const config = await chrome.storage.sync.get(['brainUrl', 'pat']);
-  return {
-    brainUrl: config.brainUrl || DEFAULT_BRAIN_URL,
-    pat: config.pat || ''
-  };
-}
+  async function getConfig() {
+    const config = await chrome.storage.sync.get(['brainUrl', 'pat']);
+    const pre = self.PreConfigured || {};
+    return {
+      brainUrl: config.brainUrl || pre.brainUrl || DEFAULT_BRAIN_URL,
+      pat: config.pat || pre.pat || ''
+    };
+  }
+
+  async function brainFetch(path, options = {}) {
+    const { brainUrl, pat } = await getConfig();
+    const url = `${brainUrl}${path}`;
+    const headers = { 'Content-Type': 'application/json', ...options.headers };
+    if (pat) headers['Authorization'] = `Bearer ${pat}`;
+
+    const res = await fetch(url, { ...options, headers });
+    if (res.status === 401) {
+      chrome.action.setBadgeText({ text: '!' });
+      chrome.action.setBadgeBackgroundColor({ color: '#ef4444' });
+      throw new Error('Authentication failed — check your PAT token');
+    }
+    if (!res.ok) throw new Error(`Brain API error: ${res.status}`);
+    return res.json();
+  }
 
-async function brainFetch(path, options = {}) {
-  const { brainUrl, pat } = await getConfig();
-  const url = `${brainUrl}${path}`;
-  const headers = { 'Content-Type': 'application/json', ...options.headers };
-  if (pat) headers['Authorization'] = `Bearer ${pat}`;
+  async function share(data) {
+    return brainFetch('/api/share', { method: 'POST', body: JSON.stringify(data) });
+  }
 
-  const res = await fetch(url, { ...options, headers });
-  if (res.status === 401) {
-    chrome.action.setBadgeText({ text: '!' });
-    chrome.action.setBadgeBackgroundColor({ color: '#ef4444' });
-    throw new Error('Authentication failed — check your PAT token');
+  async function search(query, topK = 3) {
+    return brainFetch('/api/memory/search/semantic', {
+      method: 'POST',
+      body: JSON.stringify({ query, top_k: topK })
+    });
   }
-  if (!res.ok) throw new Error(`Brain API error: ${res.status}`);
-  return res.json();
-}
 
-async function share(data) {
-  return brainFetch('/api/share', { method: 'POST', body: JSON.stringify(data) });
-}
+  async function healthCheck() {
+    return brainFetch('/health');
+  }
 
-async function search(query, topK = 3) {
-  return brainFetch('/api/memory/search/semantic', {
-    method: 'POST',
-    body: JSON.stringify({ query, top_k: topK })
-  });
-}
+  async function chat(messages, options = {}) {
+    const { brainUrl, pat } = await getConfig();
+    const url = `${brainUrl}/v1/chat/completions`;
+    const headers = { 'Content-Type': 'application/json' };
+    if (pat) headers['Authorization'] = `Bearer ${pat}`;
+    if (options.sessionId) headers['x-session-id'] = options.sessionId;
 
-async function chat(messages, options = {}) {
-  const { brainUrl, pat } = await getConfig();
-  const url = `${brainUrl}/v1/chat/completions`;
-  const headers = { 'Content-Type': 'application/json' };
-  if (pat) headers['Authorization'] = `Bearer ${pat}`;
-  if (options.sessionId) headers['x-session-id'] = options.sessionId;
+    const res = await fetch(url, {
+      method: 'POST',
+      headers,
+      body: JSON.stringify({
+        messages,
+        model: options.model || 'gemini',
+        brainId: options.brainId || 'global',
+        groundingNodes: options.groundingNodes || []
+      })
+    });
+    if (res.status === 401) throw new Error('Authentication failed — check your PAT token');
+    if (!res.ok) throw new Error(`Brain API error: ${res.status}`);
+    return res.json();
+  }
 
-  const res = await fetch(url, {
-    method: 'POST',
-    headers,
-    body: JSON.stringify({
-      messages,
-      model: options.model || 'gemini',
-      brainId: options.brainId || 'global',
-      groundingNodes: options.groundingNodes || []
-    })
-  });
-  if (res.status === 401) throw new Error('Authentication failed — check your PAT token');
-  if (!res.ok) throw new Error(`Brain API error: ${res.status}`);
-  return res.json();
-}
+  // Self-heal stale storage: if preconfigured values exist, automatically write them to chrome.storage.sync
+  try {
+    if (typeof chrome !== 'undefined' && chrome.storage && chrome.storage.sync) {
+      chrome.storage.sync.get(['brainUrl', 'pat']).then((config) => {
+        const pre = self.PreConfigured || {};
+        if (pre.pat && (!config.pat || config.pat !== pre.pat)) {
+          chrome.storage.sync.set({ brainUrl: pre.brainUrl, pat: pre.pat });
+        }
+      });
+    }
+  } catch (e) {
+    // Ignore in contexts where storage sync isn't available
+  }
 
-// Expose as a global namespace for non-module contexts (service worker, popup, etc.)
-self.BrainClient = { getConfig, brainFetch, share, search, healthCheck, chat };
+  // Expose as a global namespace for non-module contexts (service worker, popup, etc.)
+  self.BrainClient = { getConfig, brainFetch, share, search, healthCheck, chat };
+})();

package/extension/lib/preconfigured.js

@@ -0,0 +1,5 @@
+// Exposes default configuration dynamically injected by the server
+self.PreConfigured = {
+  brainUrl: '',
+  pat: ''
+};

package/extension/manifest.json

@@ -3,7 +3,7 @@
   "name": "Total Recall — Brain Extension",
   "version": "0.1.0",
   "description": "Contextual memory overlay, Share-to-Brain, and passive browsing context for Total Recall",
-  "permissions": ["activeTab", "contextMenus", "sidePanel", "storage"],
+  "permissions": ["activeTab", "contextMenus", "sidePanel", "storage", "tabs"],
   "optional_permissions": ["history", "webNavigation"],
   "host_permissions": ["http://127.0.0.1:*/*", "http://localhost:*/*"],
   "background": {

package/extension/options/options.html

@@ -66,6 +66,7 @@
     </div>
   </div>
 
+  <script src="../lib/preconfigured.js"></script>
   <script src="../lib/brain-client.js"></script>
   <script src="options.js"></script>
 </body>

package/extension/options/options.js

@@ -20,8 +20,9 @@
       'captureGranularity'
     ]);
 
-    brainUrlInput.value = settings.brainUrl || 'http://127.0.0.1:3000';
-    patInput.value = settings.pat || '';
+    const pre = self.PreConfigured || {};
+    brainUrlInput.value = settings.brainUrl || pre.brainUrl || 'http://127.0.0.1:3000';
+    patInput.value = settings.pat || pre.pat || '';
 
     // Blocklist: stored as array, displayed one per line
     if (Array.isArray(settings.blocklist)) {

package/extension/popup/popup.html

@@ -38,6 +38,7 @@
     <div id="status-msg" class="status-msg hidden"></div>
   </div>
 
+  <script src="../lib/preconfigured.js"></script>
   <script src="../lib/brain-client.js"></script>
   <script src="popup.js"></script>
 </body>

package/extension/popup/popup.js

@@ -59,14 +59,27 @@
     btnRemember.classList.add('loading');
     try {
       const [tab] = await chrome.tabs.query({ active: true, currentWindow: true });
+      const note = noteInput.value.trim();
+      const targetUrl = tab?.url || urlInput.value || '';
+      
+      // If we don't have a URL, we must have a note/excerpt
+      if (!targetUrl && !note) {
+        throw new Error('Please enter a note or browse to a webpage to remember.');
+      }
+      if (targetUrl && (targetUrl.startsWith('chrome://') || targetUrl.startsWith('about:') || targetUrl.startsWith('chrome-extension://'))) {
+        if (!note) {
+          throw new Error('Cannot capture system pages. Please enter a note to save.');
+        }
+      }
+
       const payload = {
-        url: tab?.url || urlInput.value,
+        url: targetUrl && !targetUrl.startsWith('chrome://') && !targetUrl.startsWith('about:') && !targetUrl.startsWith('chrome-extension://') ? targetUrl : undefined,
         title: tab?.title || '',
         action: 'remember',
         source: 'chrome-extension-popup'
       };
-      const note = noteInput.value.trim();
       if (note) payload.excerpt = note;
+
       await sendShare(payload);
       showStatus('Page remembered!');
       noteInput.value = '';
@@ -88,14 +101,26 @@
     btnResearch.classList.add('loading');
     try {
       const [tab] = await chrome.tabs.query({ active: true, currentWindow: true });
+      const note = noteInput.value.trim();
+      const targetUrl = tab?.url || urlInput.value || '';
+
+      if (!targetUrl && !note) {
+        throw new Error('Please enter research notes or browse to a webpage to queue research.');
+      }
+      if (targetUrl && (targetUrl.startsWith('chrome://') || targetUrl.startsWith('about:') || targetUrl.startsWith('chrome-extension://'))) {
+        if (!note) {
+          throw new Error('Cannot research system pages. Please enter research notes.');
+        }
+      }
+
       const payload = {
-        url: tab?.url || urlInput.value,
+        url: targetUrl && !targetUrl.startsWith('chrome://') && !targetUrl.startsWith('about:') && !targetUrl.startsWith('chrome-extension://') ? targetUrl : undefined,
         title: tab?.title || '',
         action: 'research',
         source: 'chrome-extension-popup'
       };
-      const note = noteInput.value.trim();
       if (note) payload.excerpt = note;
+
       await sendShare(payload);
       showStatus('Research queued!');
       noteInput.value = '';

package/extension/sidepanel/sidepanel.html

@@ -130,6 +130,7 @@
 
     </div>
   </div>
+  <script src="../lib/preconfigured.js"></script>
   <script src="../lib/brain-client.js"></script>
   <script src="sidepanel.js"></script>
 </body>

package/extension/sidepanel/sidepanel.js

@@ -183,9 +183,15 @@
     btnRemember.classList.add('loading');
     try {
       const tab = await getCurrentTab();
+      if (!tab || !tab.url) {
+        throw new Error('No active webpage found to capture. Make sure you are on a webpage tab.');
+      }
+      if (tab.url.startsWith('chrome://') || tab.url.startsWith('about:') || tab.url.startsWith('chrome-extension://')) {
+        throw new Error('Cannot capture system pages.');
+      }
       await sendShare({
         url: tab.url,
-        title: tab.title,
+        title: tab.title || 'Untitled Page',
         action: 'remember',
         source: 'chrome-extension-sidepanel'
       });
@@ -208,9 +214,15 @@
     btnResearch.classList.add('loading');
     try {
       const tab = await getCurrentTab();
+      if (!tab || !tab.url) {
+        throw new Error('No active webpage found to capture. Make sure you are on a webpage tab.');
+      }
+      if (tab.url.startsWith('chrome://') || tab.url.startsWith('about:') || tab.url.startsWith('chrome-extension://')) {
+        throw new Error('Cannot capture system pages.');
+      }
       await sendShare({
         url: tab.url,
-        title: tab.title,
+        title: tab.title || 'Untitled Page',
         action: 'research',
         source: 'chrome-extension-sidepanel'
       });
@@ -245,13 +257,19 @@
     btnSaveNote.disabled = true;
     btnSaveNote.classList.add('loading');
     try {
-      const tab = await getCurrentTab();
+      let tab = null;
+      try {
+        tab = await getCurrentTab();
+      } catch {
+        // ignore
+      }
+      const hasValidUrl = tab && tab.url && !tab.url.startsWith('chrome://') && !tab.url.startsWith('about:') && !tab.url.startsWith('chrome-extension://');
       await sendShare({
-        url: tab.url,
-        title: `Note: ${text.slice(0, 50)}`,
+        url: hasValidUrl ? tab.url : undefined,
+        title: hasValidUrl ? `Note: ${text.slice(0, 50)}` : undefined,
         excerpt: text,
         action: 'remember',
-        category: 'fact',
+        category: 'facts',
         source: 'chrome-extension-sidepanel'
       });
       noteInput.value = '';

package/frontend/src/pages/IntegrationsPage.tsx

@@ -1,5 +1,5 @@
 import { useMemo, useState, useEffect } from 'react'
-import { getApiBase, listApiKeys, connectClient, fetchActiveIntegrations } from '../api'
+import { getApiBase, listApiKeys, connectClient, fetchActiveIntegrations, fetchExtensionStatus } from '../api'
 import type { ApiKey } from '../api'
 
 type Preset = {
@@ -68,6 +68,7 @@ export default function IntegrationsPage() {
   const [message, setMessage] = useState<{ type: 'success' | 'error'; text: string } | null>(null)
   const [keys, setKeys] = useState<ApiKey[]>([])
   const [activeIdes, setActiveIdes] = useState<string[]>([])
+  const [extStatus, setExtStatus] = useState<{ available: boolean; connected: boolean } | null>(null)
 
   const baseUrl = useMemo(() => {
     const configured = getApiBase()
@@ -91,6 +92,12 @@ export default function IntegrationsPage() {
       })
       .catch(() => {})
 
+    fetchExtensionStatus()
+      .then(res => {
+        if (active) setExtStatus(res)
+      })
+      .catch(() => {})
+
     return () => { active = false }
   }, [])
 
@@ -221,6 +228,138 @@ export default function IntegrationsPage() {
         gridTemplateColumns: 'repeat(auto-fit, minmax(300px, 1fr))',
         gap: 16,
       }}>
+        {/* Chrome Extension Integration Card */}
+        <div style={{
+          border: '1px solid var(--border)',
+          background: 'var(--bg-secondary)',
+          borderRadius: 12,
+          padding: 18,
+          display: 'flex',
+          flexDirection: 'column',
+          gap: 12,
+          position: 'relative',
+        }}>
+          <div style={{ display: 'flex', justifyContent: 'space-between', alignItems: 'flex-start', gap: 12 }}>
+            <div>
+              <h3 style={{ margin: '0 0 4px' }}>Chrome Extension</h3>
+              <div style={{ color: 'var(--text-tertiary)', fontSize: 12 }}>Browser Overlay Panel & Capture</div>
+            </div>
+            {extStatus ? (
+              extStatus.connected ? (
+                <span style={{
+                  display: 'inline-flex',
+                  alignItems: 'center',
+                  gap: 6,
+                  padding: '4px 8px',
+                  borderRadius: 6,
+                  background: 'rgba(16, 185, 129, 0.1)',
+                  border: '1px solid rgba(16, 185, 129, 0.3)',
+                  fontSize: 12,
+                  color: '#34d399',
+                  fontWeight: 500,
+                }}>
+                  <span style={{
+                    width: 8,
+                    height: 8,
+                    borderRadius: '50%',
+                    background: '#10b981',
+                    boxShadow: '0 0 8px #10b981',
+                  }} />
+                  Connected
+                </span>
+              ) : (
+                <span style={{
+                  display: 'inline-flex',
+                  alignItems: 'center',
+                  gap: 6,
+                  padding: '4px 8px',
+                  borderRadius: 6,
+                  background: 'rgba(245, 158, 11, 0.1)',
+                  border: '1px solid rgba(245, 158, 11, 0.3)',
+                  fontSize: 12,
+                  color: '#fbbf24',
+                  fontWeight: 500,
+                }}>
+                  <span style={{ width: 8, height: 8, borderRadius: '50%', background: '#f59e0b' }} />
+                  Not Connected
+                </span>
+              )
+            ) : (
+              <span style={{
+                display: 'inline-flex',
+                alignItems: 'center',
+                gap: 6,
+                padding: '4px 8px',
+                borderRadius: 6,
+                background: 'rgba(255, 255, 255, 0.05)',
+                border: '1px solid rgba(255, 255, 255, 0.1)',
+                fontSize: 12,
+                color: 'var(--text-tertiary)'
+              }}>
+                <span style={{ width: 8, height: 8, borderRadius: '50%', background: '#94a3b8' }} />
+                Checking...
+              </span>
+            )}
+          </div>
+
+          <div style={{ fontSize: 12, color: 'var(--text-secondary)' }}>
+            Scopes utilized: <code>chat:write, memory:read, memory:write</code>
+          </div>
+
+          <div style={{
+            background: 'var(--bg-primary)',
+            border: '1px solid var(--border)',
+            borderRadius: 8,
+            padding: 10,
+            fontSize: 13,
+            color: 'var(--text-secondary)',
+            lineHeight: '1.4'
+          }}>
+            Install the Total Recall extension to passively capture context, store quick notes, and access the active brain search overlay inside any Chrome tab.
+          </div>
+
+          <div style={{
+            background: 'var(--bg-primary)',
+            border: '1px solid var(--border)',
+            borderRadius: 8,
+            padding: 10,
+            fontFamily: 'monospace',
+            fontSize: 11,
+            whiteSpace: 'pre-wrap',
+            color: 'var(--text-tertiary)',
+          }}>
+            1. Download zip & unzip locally.{"\n"}
+            2. Go to chrome://extensions in browser.{"\n"}
+            3. Enable "Developer mode" (top right).{"\n"}
+            4. Click "Load unpacked" & select unzipped folder.
+          </div>
+
+          <div style={{ display: 'flex', gap: 10, marginTop: 'auto', paddingTop: 8 }}>
+            <a
+              href={`${baseUrl}/api/extension/download`}
+              download
+              className="btn btn-sm btn-primary"
+              style={{
+                background: 'linear-gradient(135deg, #3b82f6 0%, #1d4ed8 100%)',
+                border: 'none',
+                boxShadow: '0 2px 4px rgba(59, 130, 246, 0.3)',
+                fontWeight: 600,
+                cursor: 'pointer',
+                textDecoration: 'none',
+                display: 'inline-flex',
+                alignItems: 'center',
+                justifyContent: 'center',
+                padding: '6px 12px',
+                borderRadius: '6px',
+                color: '#fff',
+                fontSize: '12px'
+              }}
+            >
+              ⬇ Download Extension Zip
+            </a>
+          </div>
+        </div>
+
         {filteredPresets.map(preset => {
           const snippet = preset.snippet(baseUrl)
           

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "total-recall-brain",
-  "version": "3.2.3",
+  "version": "3.3.0",
   "description": "Sovereign AI system — database-free, SSSS-powered personal intelligence engine deployable to any POSIX host.",
   "type": "module",
   "bin": {

package/src/cli/init.mjs

@@ -533,6 +533,36 @@ export default async function init(args) {
     if (opts.token) brainCfg.token = opts.token;
     fs.writeFileSync(path.join(cfgDir, 'brain.json'), JSON.stringify(brainCfg, null, 2), { mode: 0o600 });
     logOk(`Registered brain at ${opts.brain}. Run \`npx total-recall sync\` to pull instructions.`);
+  } else {
+    // Zero-config bootstrap: auto-generate localhost brain.json and valid Developer PAT out-of-the-box
+    const cfgDir = path.join(brainDir, 'config');
+    fs.mkdirSync(cfgDir, { recursive: true });
+    const brainJsonPath = path.join(cfgDir, 'brain.json');
+    let needsBootstrap = false;
+    let brainCfg = {};
+    if (fs.existsSync(brainJsonPath)) {
+      try {
+        brainCfg = JSON.parse(fs.readFileSync(brainJsonPath, 'utf8'));
+        if (!brainCfg.url || !brainCfg.token) needsBootstrap = true;
+      } catch {
+        needsBootstrap = true;
+      }
+    } else {
+      needsBootstrap = true;
+    }
+
+    if (needsBootstrap) {
+      try {
+        const { issueKey } = await import('../server/keys.mjs');
+        const keyData = issueKey('Default Local Developer Key', { scopes: ['*'] });
+        brainCfg.url = brainCfg.url || 'http://localhost:3000';
+        brainCfg.token = brainCfg.token || keyData.token;
+        fs.writeFileSync(brainJsonPath, JSON.stringify(brainCfg, null, 2), { mode: 0o600 });
+        logOk(`Bootstrap configuration successfully generated and pre-authorized at ${brainCfg.url}`);
+      } catch (err) {
+        logWarn(`Could not auto-generate bootstrap developer key: ${err.message}`);
+      }
+    }
   }
 
   let dashboardUrl = '';

package/src/cli/setup.mjs

@@ -541,8 +541,17 @@ export default async function setup(args) {
       ], { stdio: 'inherit' });
       if (result.status !== 0) throw new Error('Local deploy failed');
       ok('Local deploy complete!');
+
+      // Auto-issue a local developer key for zero-config extension / IDE connection
+      try {
+        const { issueKey } = await import('../server/keys.mjs');
+        const keyData = issueKey('Default Local Developer Key', { scopes: ['*'] });
+        pat = keyData.token;
+      } catch (err) {
+        warn(`Could not auto-generate local developer key: ${err.message}`);
+      }
     }
-    brainUrl = 'http://127.0.0.1:3000';
+    brainUrl = 'http://localhost:3000';
   }
 
   // ── Step 3: Domain (optional) ────────────────────────────────────────────────

package/src/server/rest.mjs

@@ -728,12 +728,17 @@ router.get('/api/brain/export', requireAuth, requireScope('brain:export'), async
 
 // ─── Chrome Extension Download ───────────────────────────────────────────────
 
-/**
- * GET /api/extension/download
- * Streams the Chrome extension directory as a .zip file for easy installation.
- * No auth required — the extension itself is not sensitive.
- */
 router.get('/api/extension/download', async (_req, res) => {
+  let preconfiguredPath = '';
+  let originalContent = '';
+  const cleanup = () => {
+    if (preconfiguredPath && originalContent) {
+      try {
+        fs.writeFileSync(preconfiguredPath, originalContent, 'utf8');
+      } catch {}
+    }
+  };
+
   try {
     // Extension lives at <package-root>/extension/
     const extDir = path.resolve(path.dirname(fileURLToPath(import.meta.url)), '../../extension');
@@ -741,6 +746,31 @@ router.get('/api/extension/download', async (_req, res) => {
       return res.status(404).json({ error: 'Chrome extension not found in this installation.' });
     }
 
+    // Read local brain configuration to get active URL and token
+    const brainPath = path.join(CONFIG_DIR, 'brain.json');
+    let brainUrl = '';
+    let patToken = '';
+    if (fs.existsSync(brainPath)) {
+      try {
+        const brain = JSON.parse(fs.readFileSync(brainPath, 'utf8'));
+        brainUrl = brain.url || '';
+        patToken = brain.token || '';
+      } catch {}
+    }
+
+    // Overwrite preconfigured.js temporarily with injected values
+    preconfiguredPath = path.join(extDir, 'lib', 'preconfigured.js');
+    if (fs.existsSync(preconfiguredPath)) {
+      originalContent = fs.readFileSync(preconfiguredPath, 'utf8');
+      const injectedContent = `// Exposes default configuration dynamically injected by the server
+self.PreConfigured = {
+  brainUrl: "${brainUrl}",
+  pat: "${patToken}"
+};
+`;
+      fs.writeFileSync(preconfiguredPath, injectedContent, 'utf8');
+    }
+
     res.setHeader('Content-Type', 'application/zip');
     res.setHeader('Content-Disposition', 'attachment; filename="total-recall-extension.zip"');
 
@@ -754,12 +784,20 @@ router.get('/api/extension/download', async (_req, res) => {
         res.setHeader('Content-Disposition', 'attachment; filename="total-recall-extension.tar.gz"');
         const tar = spawn('tar', ['czf', '-', '-C', path.dirname(extDir), 'extension'], { stdio: ['ignore', 'pipe', 'ignore'] });
         tar.stdout.pipe(res);
-        tar.on('error', err => { if (!res.headersSent) serverError(res, err); });
-        tar.on('close', () => { if (!res.writableEnded) res.end(); });
+        tar.on('error', err => { cleanup(); if (!res.headersSent) serverError(res, err); });
+        tar.on('close', () => { cleanup(); if (!res.writableEnded) res.end(); });
+      } else {
+        cleanup();
       }
     });
-    zip.on('close', code => { if (code !== 0 && !res.writableEnded) res.end(); });
-  } catch (err) { serverError(res, err); }
+    zip.on('close', code => {
+      cleanup();
+      if (code !== 0 && !res.writableEnded) res.end();
+    });
+  } catch (err) {
+    cleanup();
+    serverError(res, err);
+  }
 });
 
 /**