Compare commits

..

4 Commits

Author SHA1 Message Date
zacharyhampton
406ff97260 - version bump 2025-12-04 23:08:37 -08:00
zacharyhampton
a8c9d0fd66 Replace REST autocomplete with GraphQL Search_suggestions query
- Replace /suggest REST endpoint with GraphQL Search_suggestions query
- Use search_location field instead of individual city/county/state/postal_code fields
- Fix coordinate order to [lon, lat] (GeoJSON standard) for radius searches
- Extract mpr_id from addr: prefix for single address lookups

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
2025-12-04 21:08:01 -08:00
Zachary Hampton
0b283e18bd Fix 403 error from Realtor.com API changes
- Update GraphQL endpoint to api.frontdoor.realtor.com
- Update HTTP headers with newer Chrome version and correct client name/version
- Improve error handling in handle_home method
- Fix response validation for missing/null data

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
2025-12-04 18:56:10 -08:00
Zachary Hampton
8bf1f9e24b Add regression test for listing_type=None including sold listings
Adds test_listing_type_none_includes_sold() to verify that when listing_type=None, sold listings are included in the results. This prevents regression of issue #142.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
2025-11-14 13:38:48 -08:00
4 changed files with 122 additions and 72 deletions

View File

@@ -81,21 +81,21 @@ class Scraper:
Scraper.session.mount("https://", adapter) Scraper.session.mount("https://", adapter)
Scraper.session.headers.update( Scraper.session.headers.update(
{ {
"accept": "application/json, text/javascript", 'sec-ch-ua-platform': '"macOS"',
"accept-language": "en-US,en;q=0.9", 'rdc-client-name': 'rdc-search-for-sale-desktop',
"cache-control": "no-cache", 'sec-ch-ua': '"Chromium";v="142", "Google Chrome";v="142", "Not_A Brand";v="99"',
"content-type": "application/json", 'sec-ch-ua-mobile': '?0',
"origin": "https://www.realtor.com", 'rdc-client-version': '0.1.0',
"pragma": "no-cache", 'user-agent': 'Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/142.0.0.0 Safari/537.36',
"priority": "u=1, i", 'accept': 'application/json',
"rdc-ab-tests": "commute_travel_time_variation:v1", 'content-type': 'application/json',
"sec-ch-ua": '"Not)A;Brand";v="99", "Google Chrome";v="127", "Chromium";v="127"', 'origin': 'https://www.realtor.com',
"sec-ch-ua-mobile": "?0", 'sec-fetch-site': 'same-site',
"sec-ch-ua-platform": '"Windows"', 'sec-fetch-mode': 'cors',
"sec-fetch-dest": "empty", 'sec-fetch-dest': 'empty',
"sec-fetch-mode": "cors", 'referer': 'https://www.realtor.com/',
"sec-fetch-site": "same-origin", 'accept-language': 'en-US,en;q=0.9',
"user-agent": "Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/127.0.0.0 Safari/537.36", 'priority': 'u=1, i',
} }
) )

View File

@@ -35,10 +35,7 @@ from .processors import (
class RealtorScraper(Scraper): class RealtorScraper(Scraper):
SEARCH_GQL_URL = "https://www.realtor.com/api/v1/rdc_search_srp?client_id=rdc-search-new-communities&schema=vesta" SEARCH_GQL_URL = "https://api.frontdoor.realtor.com/graphql"
PROPERTY_URL = "https://www.realtor.com/realestateandhomes-detail/"
PROPERTY_GQL = "https://graph.realtor.com/graphql"
ADDRESS_AUTOCOMPLETE_URL = "https://parser-external.geo.moveaws.com/suggest"
NUM_PROPERTY_WORKERS = 20 NUM_PROPERTY_WORKERS = 20
DEFAULT_PAGE_SIZE = 200 DEFAULT_PAGE_SIZE = 200
@@ -46,33 +43,70 @@ class RealtorScraper(Scraper):
super().__init__(scraper_input) super().__init__(scraper_input)
def handle_location(self): def handle_location(self):
# Get client_id from listing_type query = """query Search_suggestions($searchInput: SearchSuggestionsInput!) {
if self.listing_type is None: search_suggestions(search_input: $searchInput) {
client_id = "for-sale" geo_results {
elif isinstance(self.listing_type, list): type
client_id = self.listing_type[0].value.lower().replace("_", "-") if self.listing_type else "for-sale" text
else: geo {
client_id = self.listing_type.value.lower().replace("_", "-") _id
area_type
city
state_code
postal_code
county
centroid { lat lon }
slug_id
geo_id
}
}
}
}"""
params = { variables = {
"input": self.location, "searchInput": {
"client_id": client_id, "search_term": self.location
"limit": "1", }
"area_types": "city,state,county,postal_code,address,street,neighborhood,school,school_district,university,park",
} }
response = self.session.get( payload = {
self.ADDRESS_AUTOCOMPLETE_URL, "query": query,
params=params, "variables": variables,
) }
response = self.session.post(self.SEARCH_GQL_URL, json=payload)
response_json = response.json() response_json = response.json()
result = response_json["autocomplete"] if (
response_json is None
if not result: or "data" not in response_json
or response_json["data"] is None
or "search_suggestions" not in response_json["data"]
or response_json["data"]["search_suggestions"] is None
or "geo_results" not in response_json["data"]["search_suggestions"]
or not response_json["data"]["search_suggestions"]["geo_results"]
):
return None return None
return result[0] geo_result = response_json["data"]["search_suggestions"]["geo_results"][0]
geo = geo_result.get("geo", {})
result = {
"text": geo_result.get("text"),
"area_type": geo.get("area_type"),
"city": geo.get("city"),
"state_code": geo.get("state_code"),
"postal_code": geo.get("postal_code"),
"county": geo.get("county"),
"centroid": geo.get("centroid"),
}
if geo.get("area_type") == "address":
geo_id = geo.get("_id", "")
if geo_id.startswith("addr:"):
result["mpr_id"] = geo_id.replace("addr:", "")
return result
def get_latest_listing_id(self, property_id: str) -> str | None: def get_latest_listing_id(self, property_id: str) -> str | None:
query = """query Property($property_id: ID!) { query = """query Property($property_id: ID!) {
@@ -108,6 +142,7 @@ class RealtorScraper(Scraper):
return property_info["listings"][0]["listing_id"] return property_info["listings"][0]["listing_id"]
def handle_home(self, property_id: str) -> list[Property]: def handle_home(self, property_id: str) -> list[Property]:
"""Fetch single home with proper error handling."""
query = ( query = (
"""query Home($property_id: ID!) { """query Home($property_id: ID!) {
home(property_id: $property_id) %s home(property_id: $property_id) %s
@@ -116,23 +151,33 @@ class RealtorScraper(Scraper):
) )
variables = {"property_id": property_id} variables = {"property_id": property_id}
payload = { payload = {"query": query, "variables": variables}
"query": query,
"variables": variables,
}
response = self.session.post(self.SEARCH_GQL_URL, json=payload) try:
response_json = response.json() response = self.session.post(self.SEARCH_GQL_URL, json=payload)
data = response.json()
property_info = response_json["data"]["home"] # Check for errors or missing data
if "errors" in data or "data" not in data:
return []
if self.return_type != ReturnType.raw: if data["data"] is None or "home" not in data["data"]:
return [process_property(property_info, self.mls_only, self.extra_property_data, return []
self.exclude_pending, self.listing_type, get_key, process_extra_property_details)]
else:
return [property_info]
property_info = data["data"]["home"]
if property_info is None:
return []
# Process based on return type
if self.return_type != ReturnType.raw:
return [process_property(property_info, self.mls_only, self.extra_property_data,
self.exclude_pending, self.listing_type, get_key,
process_extra_property_details)]
else:
return [property_info]
except Exception:
return []
def general_search(self, variables: dict, search_type: str) -> Dict[str, Union[int, Union[list[Property], list[dict]]]]: def general_search(self, variables: dict, search_type: str) -> Dict[str, Union[int, Union[list[Property], list[dict]]]]:
""" """
@@ -363,19 +408,13 @@ class RealtorScraper(Scraper):
) )
elif search_type == "area": #: general search, came from a general location elif search_type == "area": #: general search, came from a general location
query = """query Home_search( query = """query Home_search(
$city: String, $search_location: SearchLocation,
$county: [String],
$state_code: String,
$postal_code: String
$offset: Int, $offset: Int,
) { ) {
home_search( home_search(
query: { query: {
%s %s
city: $city search_location: $search_location
county: $county
postal_code: $postal_code
state_code: $state_code
%s %s
%s %s
%s %s
@@ -511,24 +550,16 @@ class RealtorScraper(Scraper):
if not location_info.get("centroid"): if not location_info.get("centroid"):
return [] return []
coordinates = list(location_info["centroid"].values()) centroid = location_info["centroid"]
coordinates = [centroid["lon"], centroid["lat"]] # GeoJSON order: [lon, lat]
search_variables |= { search_variables |= {
"coordinates": coordinates, "coordinates": coordinates,
"radius": "{}mi".format(self.radius), "radius": "{}mi".format(self.radius),
} }
elif location_type == "postal_code": else: #: general search (city, county, postal_code, etc.)
search_variables |= { search_variables |= {
"postal_code": location_info.get("postal_code"), "search_location": {"location": location_info.get("text")},
}
else: #: general search, location
search_variables |= {
"city": location_info.get("city"),
"county": location_info.get("county"),
"state_code": location_info.get("state_code"),
"postal_code": location_info.get("postal_code"),
} }
if self.foreclosure: if self.foreclosure:

View File

@@ -1,6 +1,6 @@
[tool.poetry] [tool.poetry]
name = "homeharvest" name = "homeharvest"
version = "0.8.5" version = "0.8.6b"
description = "Real estate scraping library" description = "Real estate scraping library"
authors = ["Zachary Hampton <zachary@bunsly.com>", "Cullen Watson <cullen@bunsly.com>"] authors = ["Zachary Hampton <zachary@bunsly.com>", "Cullen Watson <cullen@bunsly.com>"]
homepage = "https://github.com/ZacharyHampton/HomeHarvest" homepage = "https://github.com/ZacharyHampton/HomeHarvest"

View File

@@ -87,6 +87,25 @@ def test_realtor_date_range_sold():
) )
def test_listing_type_none_includes_sold():
"""Test that listing_type=None includes sold listings (issue #142)"""
# Get properties with listing_type=None (should include all common types)
result_none = scrape_property(
location="Warren, MI",
listing_type=None
)
# Verify we got results
assert result_none is not None and len(result_none) > 0
# Verify sold listings are included
status_types = set(result_none['status'].unique())
assert 'SOLD' in status_types, "SOLD listings should be included when listing_type=None"
# Verify we get multiple listing types (not just one)
assert len(status_types) > 1, "Should return multiple listing types when listing_type=None"
def test_realtor_single_property(): def test_realtor_single_property():
results = [ results = [
scrape_property( scrape_property(