@factorypure/client-helpers 1.1.28 → 1.1.29

package/dist/index.js

@@ -908,6 +908,7 @@ class SearchExclusionRule {
         return null;
     }
 }
+const skipSkuRegexCache = new Map();
 /**
  * Rule for SKU skip filtering
  */
@@ -921,28 +922,32 @@ class SkipSkuRule {
     canBeOverridden = true;
     overridableBy = ['sku_match', 'calculated_sku_match', 'alt_sku_match'];
     evaluate(context) {
-        const nots = [];
-        const formatted = context.vendorScrapeOptions.default_skip_skus
-            .filter((s) => s.toLowerCase() !== context.variant.sku.toLowerCase())
-            .map((s) => `${s}`);
-        nots.push(...formatted);
-        nots.push(...context.vendorScrapeOptions.vendor_skip_skus.filter((s) => s.toLowerCase() !== context.variant.sku.toLowerCase()));
-        // Get SKU alternates for comparison
+        const variantSkuLower = context.variant.sku.toLowerCase();
+        const allSkus = [
+            ...context.vendorScrapeOptions.default_skip_skus,
+            ...context.vendorScrapeOptions.vendor_skip_skus,
+        ];
+        const nots = allSkus.filter((s) => s.toLowerCase() !== variantSkuLower);
+        if (nots.length === 0)
+            return null;
         const skuAlternates = context.variantScrapeOptions.sku_alternates || [];
-        const skuAlternatesLower = skuAlternates.map((alt) => alt.toLowerCase());
-        const hasExclusion = nots.some((sku) => {
-            const escapedSku = sku.replace(/[-\/\\^$*+?.()|[\]{}]/g, '\\$&');
-            const skuRegex = new RegExp(`(?: |\/|\&|\=|"|'|\`)${escapedSku}(?: |\/|\&|\=|"|'|\`)`, 'gi');
-            const skuMatches = context.result.title?.match(skuRegex);
-            // If this skip SKU is found in the title
-            if (skuMatches && skuMatches.length > 0) {
-                // Check if this skip SKU is actually one of the variant's alternates
-                const isAlternate = skuAlternatesLower.includes(sku.toLowerCase());
-                // Only count as exclusion if it's NOT an alternate
-                return !isAlternate;
-            }
-            return false;
-        });
+        const cacheKey = variantSkuLower + '\x00' + nots.join('\x00') + '\x01' + skuAlternates.join('\x00');
+        let cached = skipSkuRegexCache.get(cacheKey);
+        if (!cached) {
+            const alternatesSet = new Set(skuAlternates.map((a) => a.toLowerCase()));
+            // Only build regex for SKUs that are not alternates (alternates can never be exclusions)
+            const nonAlternateNots = nots.filter((s) => !alternatesSet.has(s.toLowerCase()));
+            const regex = nonAlternateNots.length > 0
+                ? new RegExp(`(?: |\/|\&|\=|"|'|\`)(?:${nonAlternateNots
+                    .map((s) => s.replace(/[-\/\\^$*+?.()|[\]{}]/g, '\\$&'))
+                    .join('|')})(?: |\/|\&|\=|"|'|\`)`, 'i')
+                : null;
+            cached = { regex: regex, alternatesSet };
+            skipSkuRegexCache.set(cacheKey, cached);
+        }
+        if (!cached.regex)
+            return null;
+        const hasExclusion = cached.regex.test(context.result.title ?? '');
         if (hasExclusion) {
             return {
                 ruleId: this.id,
@@ -955,6 +960,7 @@ class SkipSkuRule {
         return null;
     }
 }
+const vendorExclusionRegexCache = new Map();
 /**
  * Rule for vendor exclusions
  */
@@ -967,18 +973,42 @@ class VendorExclusionRule {
     enabled = true;
     canBeOverridden = true;
     overridableBy = ['sku_match', 'calculated_sku_match', 'alt_sku_match', 'match_confirmation_override'];
+    //  evaluate(context: FilterContext): FilterResult | null {
+    //     const nots: string[] = []
+    //     const formatted = context.vendorScrapeOptions.default_skip_vendors
+    //         .filter((s) => s.toLowerCase() !== context.variant.vendor.toLowerCase())
+    //         .map((s) => `${s}`)
+    //     nots.push(...formatted)
+    //     const hasExclusion = nots.some((vendor) => {
+    //         const escaped = vendor.replace(/[-\/\\^$*+?.()|[\]{}]/g, '\\$&')
+    //         const matchRegex = new RegExp(`(?: |\/|\&|\=|"|'|\`)${escaped}(?: |\/|\&|\=|"|'|\`)`, 'gi')
+    //         const matches = context.result.title?.match(matchRegex)
+    //         return matches && matches.length > 0
+    //     })
+    //     if (hasExclusion) {
+    //         return {
+    //             ruleId: this.id,
+    //             severity: this.severity,
+    //             message: HIDE_REASONS.VENDOR_EXCLUSION,
+    //             metadata: { title: context.result.title },
+    //             timestamp: new Date().toISOString(),
+    //         }
+    //     }
+    //     return null
+    // }
     evaluate(context) {
-        const nots = [];
-        const formatted = context.vendorScrapeOptions.default_skip_vendors
-            .filter((s) => s.toLowerCase() !== context.variant.vendor.toLowerCase())
-            .map((s) => `${s}`);
-        nots.push(...formatted);
-        const hasExclusion = nots.some((vendor) => {
-            const escaped = vendor.replace(/[-\/\\^$*+?.()|[\]{}]/g, '\\$&');
-            const matchRegex = new RegExp(`(?: |\/|\&|\=|"|'|\`)${escaped}(?: |\/|\&|\=|"|'|\`)`, 'gi');
-            const matches = context.result.title?.match(matchRegex);
-            return matches && matches.length > 0;
-        });
+        const variantVendorLower = context.variant.vendor.toLowerCase();
+        const vendors = context.vendorScrapeOptions.default_skip_vendors.filter((s) => s.toLowerCase() !== variantVendorLower);
+        if (vendors.length === 0)
+            return null;
+        const cacheKey = variantVendorLower + '\x00' + vendors.join('\x00');
+        let matchRegex = vendorExclusionRegexCache.get(cacheKey);
+        if (!matchRegex) {
+            const combinedPattern = vendors.map((s) => s.replace(/[-\/\\^$*+?.()|[\]{}]/g, '\\$&')).join('|');
+            matchRegex = new RegExp(`(?: |\/|\&|\=|"|'|\`)(?:${combinedPattern})(?: |\/|\&|\=|"|'|\`)`, 'i');
+            vendorExclusionRegexCache.set(cacheKey, matchRegex);
+        }
+        const hasExclusion = matchRegex.test(context.result.title ?? '');
         if (hasExclusion) {
             return {
                 ruleId: this.id,
@@ -1255,8 +1285,9 @@ class RefurbishedUsedRule {
     evaluate(context) {
         const refurbishedKeywords = getRefurbishedKeywords(context.variant);
         const normalizedTitle = context.result.title?.toLowerCase().replace(/[^a-z0-9-]/g, ' ') ?? '';
+        const titleWords = normalizedTitle.split(/\s+/);
         const details = context.result.details_and_offers;
-        const isRefurbished = refurbishedKeywords.some((keyword) => normalizedTitle.split(/\s+/).includes(keyword)) ||
+        const isRefurbished = refurbishedKeywords.some((keyword) => titleWords.includes(keyword)) ||
             (details && details.includes('Pre-owned'));
         if (isRefurbished) {
             return {
@@ -1624,6 +1655,9 @@ export function createCustomFilterRule(config) {
  * This provides more flexibility and better extensibility than the legacy filterScrapeResults
  */
 export const filterScrapeResultsV2 = ({ scrapeResults, variant, variantScrapeOptions, vendorScrapeOptions, globalScrapeOptions, filterConfig, customRules, comparisonMap, matchConfirmationMap, }) => {
+    if (scrapeResults.length === 0 || !scrapeResults) {
+        return [];
+    }
     const registry = createDefaultFilterRegistry();
     // Register custom rules if provided
     if (customRules) {
@@ -2024,11 +2058,9 @@ export const getRefurbishedKeywords = (variant) => {
     ];
     if (variant?.sku) {
         const skuBasedKeywords = [
+            `${variant.sku.toLowerCase()}-r`,
+            `${variant.sku.toUpperCase()}-R`,
             `${variant.sku}-R`,
-            `${variant.sku}-R`.toLowerCase(),
-            `${variant.sku}-R`.toUpperCase(),
-            `${variant.sku?.toLowerCase()}-R`,
-            `${variant.sku?.toUpperCase()}-R`,
         ];
         refurbishedKeywords.push(...skuBasedKeywords);
     }

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@factorypure/client-helpers",
-    "version": "1.1.28",
+    "version": "1.1.29",
     "description": "",
     "main": "./dist/index.js",
     "types": "./dist/index.d.ts",