Merge pull request #84 from scrapfly/fix-fashionphile

Fix fashionphile

Author: mazen-r

fashionphile-scraper/fashionphile.py

@@ -10,8 +10,8 @@
 from typing import Dict, List
 from pathlib import Path
 from loguru import logger as log
-from urllib.parse import parse_qs, urlencode, urlparse
-from scrapfly import ScrapeConfig, ScrapflyClient, ScrapeApiResponse
+import re
+from scrapfly import ScrapeConfig, ScrapflyClient
 
 SCRAPFLY = ScrapflyClient(key=os.environ["SCRAPFLY_KEY"])
 
@@ -20,61 +20,153 @@
     "asp": True,
     # set the proxy country to US
     "country": "US",
+    "render_js": True,
 }
 
 output = Path(__file__).parent / "results"
 output.mkdir(exist_ok=True)
 
 
-def find_hidden_data(result: ScrapeApiResponse) -> dict:
-    """extract hidden NEXT_DATA from page html"""
-    data = result.selector.css("script#__NEXT_DATA__::text").get()
-    data = json.loads(data)
-    return data
+def convert_to_json_urls(urls):
+    converted = []
+    for url in urls:
+        # Replace '/p/' with '/products/' and add '.json' at the end
+        new_url = url.replace("/p/", "/products/") + ".json"
+        converted.append(new_url)
+    return converted
 
 
-async def scrape_products(urls: List[str]) -> dict:
-    """scrape fashionphile product pages for product data"""
+async def scrape_products(urls: List[str]) -> List[Dict]:
+    """
+    Scrape product data from Fashionphile product pages using the product API.
+    """
+    urls = convert_to_json_urls(urls)
     to_scrape = [ScrapeConfig(url, **BASE_CONFIG) for url in urls]
     products = []
     async for response in SCRAPFLY.concurrent_scrape(to_scrape):
-        data = find_hidden_data(response)
-        product = data["props"]["pageProps"]["initialState"]["productPageReducer"]["productData"]
-        products.append(product)
-    log.success(f"scraped {len(products)} product listings from product pages")
+        # Extract just the product data from the JSON content
+        content = response.result['result']['content']
+        product_data = json.loads(content)['product']
+        products.append(product_data)
+        log.success(f"scraped {len(products)} product listings from product pages")
     return products
 
 
-def update_url_parameter(url, **params):
-    """update url query parameter of an url with new values"""
-    current_params = parse_qs(urlparse(url).query)
-    updated_query_params = urlencode({**current_params, **params}, doseq=True)
-    return f"{url.split('?')[0]}?{updated_query_params}"
+def parse_price(price_text: str) -> int:
+    if not price_text:
+        return 0
+    # Remove $ and commas, convert to int
+    return int(re.sub(r'[$,]', '', price_text.strip()))
+
+
+def extract_product_from_card(card_selector) -> Dict:
+    """Extract product data from a product card HTML element"""
+    
+    # Get product ID from data attribute
+    product_id = card_selector.css('::attr(data-product-id)').get('')
+    
+    # Get brand name
+    brand_name = card_selector.css('.fp-card__vendor::text').get('').strip()
+    
+    # Get product name
+    product_name = card_selector.css('.fp-card__link__product-name::text').get('').strip()
+    
+    # Get condition
+    condition = card_selector.css('.fp-condition::text').get('').strip()
+        
+    # Get prices
+    regular_price_text = card_selector.css('.price-item--regular::text').get('').strip()
+    sale_price_text = card_selector.css('.price-item--sale.price-item--last::text').get('').strip()
+    
+    # If no sale price, use regular price as final price
+    if sale_price_text:
+        price_text = sale_price_text
+    elif regular_price_text:
+        price_text = regular_price_text
+    else:
+        # Fallback: try to find any price
+        price_text = card_selector.css('.price-item::text').get('$0').strip()
+    
+    price = parse_price(price_text)
+    
+    # Calculate discounted price
+    if regular_price_text and sale_price_text:
+        regular = parse_price(regular_price_text)
+        discounted_price = regular - price
+    else:
+        discounted_price = 0
+    
+
+    # Build result matching search_schema
+    result = {
+        "brand_name": brand_name,
+        "product_name" : product_name,
+        "condition": condition,
+        "discounted_price": discounted_price,
+        "price": price,
+        "id": int(product_id) if product_id else 0
+    }
+    
+    return result
 
 
 async def scrape_search(url: str, max_pages: int = 10) -> List[Dict]:
-    log.info(f"scraping search page {url}")
-    # scrape first page
+    # Scrape first page
     result_first_page = await SCRAPFLY.async_scrape(ScrapeConfig(url, **BASE_CONFIG))
-    data_first_page = find_hidden_data(result_first_page)
-    data_first_page = data_first_page["props"]["pageProps"]["serverState"]["initialResults"][
-        "prod_ecom_products_date_desc"
-    ]["results"][0]
-    results = data_first_page["hits"]
-
-    # find total page count
-    total_pages = data_first_page["nbPages"]
+    selector = result_first_page.selector
+    
+    # Find all product cards
+    product_cards = selector.css('.fp-algolia-product-card')
+    log.info(f"found {len(product_cards)} products on first page")
+    
+    # Extract data from each card
+    results = []
+    for card in product_cards:
+        try:
+            product_data = extract_product_from_card(card)
+            results.append(product_data)
+        except Exception as e:
+            log.warning(f"failed to extract product: {e}")
+            continue
+    
+    # Find total pages from pagination
+    pagination_href = selector.css('.ais-Pagination-item--lastPage a::attr(href)').get('')
+    if pagination_href:
+        match = re.search(r'page=(\d+)', pagination_href)
+        if match:
+            total_pages = int(match.group(1))
+        else:
+            total_pages = 1
+    else:
+        total_pages = 1
+    
     if max_pages and max_pages < total_pages:
         total_pages = max_pages
-
-    # scrape remaining pages
-    log.info(f"scraping search pagination ({total_pages-1} more pages)")
-    to_scrape = [
-        ScrapeConfig(update_url_parameter(url, page=page), **BASE_CONFIG) for page in range(2, total_pages + 1)
-    ]
-    async for result in SCRAPFLY.concurrent_scrape(to_scrape):
-        data = find_hidden_data(result)
-        data = data["props"]["pageProps"]["serverState"]["initialResults"]["prod_ecom_products_date_desc"]["results"][0]
-        results.extend(data["hits"])
-    log.success(f"scraped {len(results)} product listings from search pages")
-    return results
+    
+    log.info(f"total pages: {total_pages}")
+    
+    # Scrape remaining pages
+    if total_pages > 1:
+        log.info(f"scraping pagination ({total_pages-1} more pages)")
+        
+        # Build URLs for remaining pages
+        base_url = url.split('?')[0]
+        to_scrape = []
+        for page in range(2, total_pages + 1):
+            page_url = f"{base_url}?page={page}"
+            to_scrape.append(ScrapeConfig(page_url, **BASE_CONFIG))
+        
+        # Scrape concurrently
+        async for result in SCRAPFLY.concurrent_scrape(to_scrape):
+            product_cards = result.selector.css('.fp-algolia-product-card')
+            
+            for card in product_cards:
+                try:
+                    product_data = extract_product_from_card(card)
+                    results.append(product_data)
+                except Exception as e:
+                    log.warning(f"failed to extract product: {e}")
+                    continue
+    
+    log.info(f"scraped {len(results)} product listings from search pages")
+    return results