price_scanner 0.2.1 → 0.2.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 0ecd0d4d00035cf9a9fae5d1e36ef4e86c817432128eae3b02f1a46cef7d0dfa
-  data.tar.gz: 7a594ab4fb0af0456f4465b867929a7a041e8f19883f0a4e67932a408a1bde14
+  metadata.gz: 2014de33dd81d654b4fd82992474605dd110ff5154299dec1bfca80ec17f91db
+  data.tar.gz: b341b9072d034078d95c8a335d0d8f9d9b62bd2d00dc4379441a48cebe3091b9
 SHA512:
-  metadata.gz: eaa881c8021e3402d543589efdfb4d0cc081e59b3650714952525fcd9b932a3885c8d245f4f5052637fd33f1b3167edddab3d778c7a8046ece33d60ee4999aac
-  data.tar.gz: 1ff594ed3d82a7e37848b84e0888cefc3e6a80515e62b91267bfb764c80dbb93335b532ae9adb376bf948c6cd68d4461ad35d10bfe694b3239f768d0b21be8a7
+  metadata.gz: edffb337ea738fa8541612a49d1a2417f40c4d3463ef4b0509eef224e34172278f1ea585b2c34e470aea0feb457540d7be2218a4fdc97f2fb2e67e8c75244190
+  data.tar.gz: e0a1d51e6a441400d41c6c77c25ca9a5d083ec8e2d758a76685321281e4e14c7bf0a05d3bbac27cd60ecc04e0babf8f2bcd0aad23e471246fcbf6f957fcc890c

data/CHANGELOG.md

@@ -0,0 +1,35 @@
+# Changelog
+
+## 0.2.3
+
+- Fix comma-as-thousands-separator not recognized in PRICE_PATTERN (`7,999.00 €` → was parsed as `999.00 €`)
+- Affects prices in English/international format: `$1,299.99`, `8,289.00 €`, etc.
+- Safe change: requires exactly 3 digits after separator, so decimal commas (`19,99 zł`) still work correctly
+
+## 0.2.2
+
+- Fix negative price detection with spaced dash ("- 1.040 zł") — savings badges with space between minus and price were not filtered
+- Refactor `negative_price?` with `rindex_non_space` helper (DRY)
+- Distinguish range separators ("Pack of 3 - 29,99 zł") from negative prices
+
+## 0.2.1
+
+- Fix false price extraction from model numbers (IP65, HC940, H265, 2K 30MP)
+- Prevent digits before currency symbol from being matched as prices
+
+## 0.2.0
+
+- Remove ConsentDetector from gem (moved to smart_offers app)
+
+## 0.1.1
+
+- Remove rubycritic dependency
+- Auto-require all price_scanner modules
+
+## 0.1.0
+
+- Initial release
+- `PriceScanner::Parser` — normalize prices, extract currency, strip price mentions
+- `PriceScanner::Detector` — extract prices from text, filter negatives/per-unit/ranges/savings
+- Multi-currency support: PLN, EUR, USD, GBP
+- Smart filtering: negative prices, per-unit prices, price ranges, savings amounts

data/lib/price_scanner/detector.rb

@@ -4,9 +4,9 @@ module PriceScanner
   # Extracts prices from text using regex patterns with smart filtering.
   module Detector
     PRICE_PATTERN = /
-      (?:zł|pln|€|\$|£)[\s\u00a0]*(?:\d{1,3}(?:[.\s\u00a0]\d{3})+|\d{1,4})(?:[.,]\d{1,2})?  |
-      (?<![a-zA-Z\d])(?:\d{1,3}(?:[.\s\u00a0]\d{3})+|\d{1,4})[.,]\d{2}[\s\u00a0]*(?:zł|pln|€|\$|£|eur|usd|gbp)(?!\d)  |
-      (?<![a-zA-Z\d])(?:\d{1,3}(?:[.\s\u00a0]\d{3})+|\d{1,4})[\s\u00a0]*(?:zł|pln|€|\$|£)(?!\d)
+      (?:zł|pln|€|\$|£)[\s\u00a0]*(?:\d{1,3}(?:[.,\s\u00a0]\d{3})+|\d{1,4})(?:[.,]\d{1,2})?  |
+      (?<![a-zA-Z\d])(?:\d{1,3}(?:[.,\s\u00a0]\d{3})+|\d{1,4})[.,]\d{2}[\s\u00a0]*(?:zł|pln|€|\$|£|eur|usd|gbp)(?!\d)  |
+      (?<![a-zA-Z\d])(?:\d{1,3}(?:[.,\s\u00a0]\d{3})+|\d{1,4})[\s\u00a0]*(?:zł|pln|€|\$|£)(?!\d)
     /ix
 
     PER_UNIT_PATTERN = %r{(?:/\s*|za\s+)(?:kg|g|mg|l|ml|szt|m[²³23]?|cm|mm|op|opak|pcs|pc|unit|each|ea|kaps|tabl|tab)\b}i
@@ -72,7 +72,21 @@ module PriceScanner
     end
 
     def negative_price?(text_str, match_index)
-      match_index.positive? && NEGATIVE_PREFIXES.include?(text_str[match_index - 1])
+      return false unless match_index.positive?
+
+      # Find the non-whitespace char before price: "-1.040" or "- 1.040"
+      dash_pos = rindex_non_space(text_str, match_index - 1)
+      return false unless dash_pos && NEGATIVE_PREFIXES.include?(text_str[dash_pos])
+
+      # Dash at start of text = negative; after digit = range separator ("3 - 29,99")
+      before_dash = rindex_non_space(text_str, dash_pos - 1)
+      before_dash.nil? || text_str[before_dash] !~ /\d/
+    end
+
+    def rindex_non_space(text_str, from)
+      i = from
+      i -= 1 while i >= 0 && text_str[i] =~ /\s/
+      i >= 0 ? i : nil
     end
 
     def per_unit_price?(text_str, match_end)
@@ -133,7 +147,7 @@ module PriceScanner
     end
 
     private_class_method :scan_raw_prices, :find_price_at, :build_price_result,
-                         :negative_price?, :per_unit_price?,
+                         :negative_price?, :rindex_non_space, :per_unit_price?,
                          :filter_range_prices, :find_range_indices, :range_between?,
                          :filter_savings_by_difference, :savings_amount?, :matches_savings_pattern?
   end

data/lib/price_scanner/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module PriceScanner
-  VERSION = "0.2.1"
+  VERSION = "0.2.3"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: price_scanner
 version: !ruby/object:Gem::Version
-  version: 0.2.1
+  version: 0.2.3
 platform: ruby
 authors:
 - Justyna
@@ -21,6 +21,7 @@ files:
 - ".reek.yml"
 - ".rspec"
 - ".rubocop.yml"
+- CHANGELOG.md
 - LICENSE
 - README.md
 - Rakefile