dsh-subscriptions 0.2.0 → 0.2.2

package/lib/providers/codex-search.js

@@ -15,7 +15,9 @@ export class CodexWebSearchProvider {
     constructor(options) {
         this.options = options;
     }
-    available() { return this.options.enabled?.() ?? true; }
+    available() {
+        return this.options.enabled?.() ?? true;
+    }
     async search(request, signal) {
         throwIfAborted(signal);
         let session;
@@ -50,10 +52,10 @@ export class CodexWebSearchProvider {
                 response = await fetchFn(CODEX_SEARCH_URL, {
                     method: 'POST',
                     headers: {
-                        'authorization': `Bearer ${session.accessToken}`,
+                        authorization: `Bearer ${session.accessToken}`,
                         'chatgpt-account-id': session.accountId,
                         'content-type': 'application/json',
-                        'originator': 'codex_cli_rs',
+                        originator: 'codex_cli_rs',
                         ...attributionHeaders(),
                     },
                     body: JSON.stringify(body),
@@ -73,7 +75,9 @@ export class CodexWebSearchProvider {
                 await discard(response);
                 throw new WebError('Codex Web Search was rate-limited; retry later', 'CODEX_SEARCH_RATE_LIMIT');
             }
-            if (response.status >= 500 && response.status <= 599 && attempt < MAX_ATTEMPTS) {
+            if (response.status >= 500 &&
+                response.status <= 599 &&
+                attempt < MAX_ATTEMPTS) {
                 await discard(response);
                 await abortableDelay(retryBaseMs * 2 ** (attempt - 1), signal);
                 continue;
@@ -106,14 +110,20 @@ export function normalizeCodexSearchResponse(value) {
         for (const candidate of value.results) {
             if (!record(candidate))
                 continue;
-            const rawUrl = safeString(candidate.url, 8192) ?? safeString(candidate.source_url, 8192);
+            const rawUrl = safeString(candidate.url, 8192) ??
+                safeString(candidate.source_url, 8192);
             const url = rawUrl === undefined ? undefined : httpUrl(rawUrl);
             if (url === undefined || seen.has(url))
                 continue;
             seen.add(url);
-            const title = safeString(candidate.title, 1000) ?? safeString(candidate.source_title, 1000);
+            const title = safeString(candidate.title, 1000) ??
+                safeString(candidate.source_title, 1000);
             const snippet = safeString(candidate.snippet, 4000) ?? safeString(candidate.text, 4000);
-            sources.push({ url, ...(title ? { title } : {}), ...(snippet ? { snippet } : {}) });
+            sources.push({
+                url,
+                ...(title ? { title } : {}),
+                ...(snippet ? { snippet } : {}),
+            });
         }
     return { content: value.output, sources, truncated: false };
 }
@@ -124,6 +134,8 @@ function safeString(value, max) {
     if (typeof value !== 'string')
         return undefined;
     const text = value.trim();
+    // The class is the C0 control range being rejected, not an accidental escape.
+    // eslint-disable-next-line no-control-regex -- reject NUL through unit separator in provider text
     if (!text || text.length > max || /[\u0000-\u001f]/.test(text))
         return undefined;
     return text;
@@ -131,7 +143,9 @@ function safeString(value, max) {
 function httpUrl(value) {
     try {
         const parsed = new URL(value);
-        return parsed.protocol === 'http:' || parsed.protocol === 'https:' ? parsed.href : undefined;
+        return parsed.protocol === 'http:' || parsed.protocol === 'https:'
+            ? parsed.href
+            : undefined;
     }
     catch {
         return undefined;
@@ -141,7 +155,9 @@ async function discard(response) {
     try {
         await response.body?.cancel();
     }
-    catch { /* best effort */ }
+    catch {
+        /* best effort */
+    }
 }
 function throwIfAborted(signal) {
     if (signal?.aborted)
@@ -156,9 +172,18 @@ function abortableDelay(ms, signal) {
         return Promise.resolve();
     }
     return new Promise((resolve, reject) => {
-        const timer = setTimeout(() => { cleanup(); resolve(); }, ms);
-        const onAbort = () => { cleanup(); reject(cancelled(signal?.reason)); };
-        const cleanup = () => { clearTimeout(timer); signal?.removeEventListener('abort', onAbort); };
+        const timer = setTimeout(() => {
+            cleanup();
+            resolve();
+        }, ms);
+        const onAbort = () => {
+            cleanup();
+            reject(cancelled(signal?.reason));
+        };
+        const cleanup = () => {
+            clearTimeout(timer);
+            signal?.removeEventListener('abort', onAbort);
+        };
         signal?.addEventListener('abort', onAbort, { once: true });
     });
 }
@@ -196,6 +221,8 @@ async function boundedText(response, signal) {
         try {
             reader.releaseLock();
         }
-        catch { /* best effort */ }
+        catch {
+            /* best effort */
+        }
     }
 }