browser-debugger-cli 0.7.2 → 0.9.0

package/dist/telemetry/filterDsl.js

@@ -5,8 +5,10 @@
  * Supports domain matching, status codes, HTTP methods, MIME types, size thresholds,
  * header inspection, and various state filters.
  */
-import { matchesWildcard } from '../telemetry/filters.js';
+import { RESOURCE_TYPE_ABBREVIATIONS, resourceTypeFromName } from '../constants.js';
+import { matchesWildcard } from './filters.js';
 import { createLogger } from '../ui/logging/index.js';
+import { findSimilar } from '../utils/suggestions.js';
 import { extractHostname } from '../utils/url.js';
 const log = createLogger('network');
 const SIZE_UNITS = {
@@ -22,11 +24,29 @@ const VALID_FILTER_TYPES = [
     'mime-type',
     'resource-type',
     'larger-than',
+    'duration',
     'has-response-header',
     'is',
     'scheme',
 ];
-const VALID_IS_VALUES = ['from-cache', 'running'];
+/** CDP resource types (the keys of the TYP abbreviations) */
+const RESOURCE_TYPES = Object.keys(RESOURCE_TYPE_ABBREVIATIONS);
+const DURATION_PATTERN = /^(\d+(?:\.\d+)?)\s*(ms|s)?$/i;
+/**
+ * Parse a duration into milliseconds.
+ *
+ * @param value - e.g. "500", "500ms", "1.5s"
+ * @returns Milliseconds
+ * @throws Error for another format
+ */
+export function parseDuration(value) {
+    const match = DURATION_PATTERN.exec(value);
+    if (!match)
+        throw new Error(`Invalid duration: "${value}". Use format like "500ms" or "1.5s"`);
+    const amount = parseFloat(match[1] ?? '0');
+    return match[2]?.toLowerCase() === 's' ? amount * 1000 : amount;
+}
+const VALID_IS_VALUES = ['from-cache', 'running', 'failed'];
 const SIZE_PATTERN = /^(\d+(?:\.\d+)?)\s*(b|kb|mb|gb)?$/i;
 const QUOTED_TOKEN_PATTERN = /(?:[^\s"]+|"[^"]*")+/g;
 /**
@@ -51,8 +71,64 @@ function extractOperator(value) {
         return ['>', value.slice(1)];
     if (value.startsWith('<'))
         return ['<', value.slice(1)];
+    if (value.startsWith('='))
+        return ['=', value.slice(1)];
     return ['=', value];
 }
+/** Filter types whose value may start with a comparison operator */
+const COMPARABLE_TYPES = new Set(['status-code', 'larger-than', 'duration']);
+/**
+ * Check a filter value of a type with a fixed vocabulary or format.
+ *
+ * @param type - Filter type
+ * @param value - Value (after any operator)
+ * @returns An error, or null if the value is valid
+ */
+function validateFilterValue(type, value) {
+    switch (type) {
+        case 'is':
+            return validateIsValue(value);
+        case 'larger-than':
+            return validateSizeValue(value);
+        case 'status-code':
+            return validateStatusCode(value);
+        case 'duration':
+            return validateDuration(value);
+        case 'resource-type':
+            return validateResourceTypes(value);
+        default:
+            return null;
+    }
+}
+/**
+ * Check a duration value.
+ *
+ * @param value - Duration text
+ * @returns An error, or null if valid
+ */
+function validateDuration(value) {
+    return DURATION_PATTERN.test(value)
+        ? null
+        : createError(`Invalid duration: "${value}"`, 'Use format like "500ms", "1.5s" or "1000"');
+}
+/**
+ * Check that every comma-separated resource type exists.
+ *
+ * @param value - e.g. "XHR,Fetch"
+ * @returns An error with a did-you-mean, or null if all are known
+ */
+function validateResourceTypes(value) {
+    const unknown = value
+        .split(',')
+        .map((type) => type.trim())
+        .find((type) => resourceTypeFromName(type) === undefined);
+    if (unknown === undefined)
+        return null;
+    const similar = findSimilar(unknown, RESOURCE_TYPES);
+    return createError(`Unknown resource type: "${unknown}"`, similar.length > 0
+        ? `Did you mean: ${similar[0]}?`
+        : `Valid types: ${RESOURCE_TYPES.join(', ')}`);
+}
 function findSimilarTypes(input) {
     const lowerInput = input.toLowerCase();
     return VALID_FILTER_TYPES.filter((type) => {
@@ -119,27 +195,13 @@ function parseFilterToken(token) {
     const typeError = validateFilterType(type);
     if (typeError)
         return typeError;
-    let operator = '=';
-    let value = rawValue;
-    if (type === 'status-code' || type === 'larger-than') {
-        [operator, value] = extractOperator(rawValue);
-    }
-    if (type === 'is') {
-        const isError = validateIsValue(value);
-        if (isError)
-            return isError;
-        value = value.toLowerCase();
-    }
-    if (type === 'larger-than') {
-        const sizeError = validateSizeValue(value);
-        if (sizeError)
-            return sizeError;
-    }
-    if (type === 'status-code') {
-        const statusError = validateStatusCode(value);
-        if (statusError)
-            return statusError;
-    }
+    const [operator, operand] = COMPARABLE_TYPES.has(type)
+        ? extractOperator(rawValue)
+        : ['=', rawValue];
+    const valueError = validateFilterValue(type, operand);
+    if (valueError)
+        return valueError;
+    const value = type === 'is' ? operand.toLowerCase() : operand;
     return { type: type, value, negated, operator };
 }
 function tokenize(input) {
@@ -220,7 +282,7 @@ function matchesFilter(request, filter) {
         case 'domain':
             return matchesWildcard(extractHostname(request.url), filter.value);
         case 'status-code':
-            if (request.status === undefined)
+            if (!request.status)
                 return false;
             return compareNumeric(request.status, parseInt(filter.value, 10), filter.operator);
         case 'method':
@@ -229,14 +291,14 @@ function matchesFilter(request, filter) {
             if (!request.mimeType)
                 return false;
             const normalizedMime = normalizeMimeType(request.mimeType);
-            const filterValue = filter.value.toLowerCase();
+            const filterValue = filter.value.toLowerCase().replace(/\*$/, '');
             return normalizedMime === filterValue || normalizedMime.startsWith(filterValue);
         }
         case 'resource-type': {
             if (!request.resourceType)
                 return false;
-            const types = filter.value.split(',').map((t) => t.trim().toLowerCase());
-            return types.includes(request.resourceType.toLowerCase());
+            const types = filter.value.split(',').map((t) => resourceTypeFromName(t));
+            return types.includes(request.resourceType);
         }
         case 'larger-than': {
             const size = request.encodedDataLength ?? 0;
@@ -244,6 +306,9 @@ function matchesFilter(request, filter) {
             const effectiveOperator = filter.operator === '=' ? '>' : filter.operator;
             return compareNumeric(size, threshold, effectiveOperator);
         }
+        case 'duration':
+            return (request.duration !== undefined &&
+                compareNumeric(request.duration, parseDuration(filter.value), filter.operator === '=' ? '>=' : filter.operator));
         case 'has-response-header': {
             if (!request.responseHeaders)
                 return false;
@@ -251,10 +316,13 @@ function matchesFilter(request, filter) {
             return Object.keys(request.responseHeaders).some((h) => h.toLowerCase() === headerName);
         }
         case 'is':
-            if (filter.value === 'from-cache')
-                return matchesCacheHeaders(request.responseHeaders);
+            if (filter.value === 'from-cache') {
+                return request.fromCache === true || matchesCacheHeaders(request.responseHeaders);
+            }
             if (filter.value === 'running')
                 return request.status === undefined;
+            if (filter.value === 'failed')
+                return request.status === 0;
             return false;
         case 'scheme':
             return extractScheme(request.url).toLowerCase() === filter.value.toLowerCase();
@@ -277,15 +345,17 @@ export function applyFilters(requests, filters) {
 export function getFilterHelpText() {
     return `
 Filter syntax:
-  status-code:>=400       HTTP status codes (supports =, >=, <=, >, <)
+  status-code:>=400       HTTP status codes (supports =, >=, <=, >, <; status-code:=500 or :500)
   domain:api.*            Domain with wildcards
   method:POST             HTTP method
   mime-type:application/json
   resource-type:XHR,Fetch CDP resource types (comma-separated)
   larger-than:100KB       Size threshold (B, KB, MB, GB)
+  duration:>1s            Request time (ms or s; supports =, >=, <=, >, <; a bare value means at least)
   has-response-header:set-cookie
-  is:from-cache           Cached responses
+  is:from-cache           Served from browser cache (or a CDN cache hit)
   is:running              In-progress requests
+  is:failed               Requests that got no response (DNS, refused, aborted, blocked)
   scheme:https            URL scheme
 
 Negation (use ! to avoid CLI conflicts with -):

package/dist/telemetry/filterPresets.js

@@ -3,7 +3,7 @@
  *
  * Provides convenient shortcuts for frequently used filter combinations.
  */
-import { CommandError } from '../ui/errors/index.js';
+import { CommandError } from '../errors/index.js';
 import { EXIT_CODES } from '../utils/exitCodes.js';
 import { getSuggestion } from '../utils/suggestions.js';
 /**
@@ -12,7 +12,7 @@ import { getSuggestion } from '../utils/suggestions.js';
 export const FILTER_PRESETS = {
     errors: {
         name: 'errors',
-        description: 'Failed requests (4xx and 5xx status codes)',
+        description: 'HTTP error responses (4xx and 5xx status codes)',
         filter: 'status-code:>=400',
     },
     api: {
@@ -45,11 +45,21 @@ export const FILTER_PRESETS = {
         description: 'JavaScript files',
         filter: 'resource-type:Script',
     },
+    failed: {
+        name: 'failed',
+        description: 'Requests that got no response (DNS, refused, aborted, blocked)',
+        filter: 'is:failed',
+    },
     pending: {
         name: 'pending',
         description: 'In-progress requests (no response yet)',
         filter: 'is:running',
     },
+    slow: {
+        name: 'slow',
+        description: 'Requests that took at least 1 second',
+        filter: 'duration:>=1s',
+    },
 };
 /**
  * Resolve a preset name to its filter string.

package/dist/telemetry/filters.d.ts

@@ -16,7 +16,7 @@ export interface BodyFetchDecision {
  * Domains to exclude by default (common tracking/analytics)
  * These generate high volume but are rarely useful for debugging
  */
-export declare const DEFAULT_EXCLUDED_DOMAINS: readonly ["analytics.google.com", "googletagmanager.com", "googleadservices.com", "doubleclick.net", "google-analytics.com", "clarity.ms", "bat.bing.com", "facebook.com", "connect.facebook.net", "tiktok.com", "linkedin.com", "twitter.com", "snapchat.com", "mixpanel.com", "segment.com", "segment.io", "amplitude.com", "heap.io", "fullstory.com", "hotjar.com", "logrocket.com", "smartlook.com", "exactag.com", "criteo.com", "adroll.com", "outbrain.com", "taboola.com", "confirmit.com", "newrelic.com", "datadoghq.com", "sentry.io"];
+export declare const DEFAULT_EXCLUDED_DOMAINS: readonly ["analytics.google.com", "googletagmanager.com", "googleadservices.com", "doubleclick.net", "google-analytics.com", "clarity.ms", "bat.bing.com", "facebook.com", "connect.facebook.net", "tiktok.com", "linkedin.com", "twitter.com", "snapchat.com", "mixpanel.com", "segment.com", "segment.io", "amplitude.com", "heap.io", "fullstory.com", "hotjar.com", "logrocket.com", "smartlook.com", "exactag.com", "criteo.com", "adroll.com", "outbrain.com", "taboola.com", "confirmit.com", "newrelic.com", "datadoghq.com", "sentry.io", "googlesyndication.com", "googletagservices.com", "adservice.google.com", "amazon-adsystem.com", "adnxs.com", "rubiconproject.com", "pubmatic.com", "casalemedia.com", "indexww.com", "openx.net", "3lift.com", "teads.tv", "bidswitch.net", "sharethrough.com", "adsafeprotected.com", "moatads.com", "doubleverify.com", "scorecardresearch.com", "imrworldwide.com", "chartbeat.com", "chartbeat.net", "permutive.com", "permutive.app", "optimizely.com", "quantserve.com", "quantcount.com", "krxd.net", "demdex.net", "omtrdc.net", "everesttech.net", "xiti.com", "ati-host.net"];
 /**
  * Console message types to exclude by default (Redux/React DevTools noise)
  */

package/dist/telemetry/filters.js

@@ -38,6 +38,38 @@ export const DEFAULT_EXCLUDED_DOMAINS = [
     'newrelic.com',
     'datadoghq.com',
     'sentry.io',
+    'googlesyndication.com',
+    'googletagservices.com',
+    'adservice.google.com',
+    'amazon-adsystem.com',
+    'adnxs.com',
+    'rubiconproject.com',
+    'pubmatic.com',
+    'casalemedia.com',
+    'indexww.com',
+    'openx.net',
+    '3lift.com',
+    'teads.tv',
+    'bidswitch.net',
+    'sharethrough.com',
+    'adsafeprotected.com',
+    'moatads.com',
+    'doubleverify.com',
+    'scorecardresearch.com',
+    'imrworldwide.com',
+    'chartbeat.com',
+    'chartbeat.net',
+    'permutive.com',
+    'permutive.app',
+    'optimizely.com',
+    'quantserve.com',
+    'quantcount.com',
+    'krxd.net',
+    'demdex.net',
+    'omtrdc.net',
+    'everesttech.net',
+    'xiti.com',
+    'ati-host.net',
 ];
 /**
  * Pre-compiled Set of excluded domains for O(1) exact match lookup.
@@ -313,8 +345,11 @@ export function shouldFetchBodyWithReason(url, mimeType, encodedDataLength, opti
         (mimeType?.includes('javascript') ?? false) ||
         (mimeType?.includes('text') ?? false) ||
         (mimeType?.includes('html') ?? false);
-    if (!isTextResponse) {
-        return { should: false, reason: 'Non-text response type' };
+    if (!isTextResponse && !options.fetchAllBodies) {
+        return {
+            should: false,
+            reason: 'Binary response (start the session with --all to capture it)',
+        };
     }
     if (encodedDataLength > maxBodySize) {
         const sizeStr = `${(encodedDataLength / 1024 / 1024).toFixed(2)}MB`;
@@ -328,7 +363,7 @@ export function shouldFetchBodyWithReason(url, mimeType, encodedDataLength, opti
     if (!shouldFetch) {
         return {
             should: false,
-            reason: 'Auto-optimization (see DEFAULT_SKIP_BODY_PATTERNS)',
+            reason: 'Skipped by default for this kind of resource (start the session with --all to capture it)',
         };
     }
     return { should: true };