Validate monitoring_location_id in get_ratings and widen annotations in nearest

Copilot found that `get_ratings` accepts `monitoring_location_id` and
documents the same AGENCY-ID contract, but builds its STAC filter directly
without routing through `_get_args` — so the centralized validation never
ran. Call `_check_monitoring_location_id` at the top of `get_ratings` and
widen the annotation/docstring to `Iterable[str]` for consistency.

`get_nearest_continuous` inherits validation via its forwarded call to
`get_continuous`, so its behavior is already correct — but its annotation
and docstring still advertised `list[str]`. Widen both for parity.

Author: thodson-usgs

dataretrieval/waterdata/nearest.py

@@ -5,6 +5,7 @@
 
 from __future__ import annotations
 
+from collections.abc import Iterable
 from typing import Literal, get_args
 
 import pandas as pd
@@ -18,8 +19,8 @@
 
 def get_nearest_continuous(
     targets,
-    monitoring_location_id: str | list[str] | None = None,
-    parameter_code: str | list[str] | None = None,
+    monitoring_location_id: str | Iterable[str] | None = None,
+    parameter_code: str | Iterable[str] | None = None,
     *,
     window: str | pd.Timedelta = "PT7M30S",
     on_tie: OnTie = "first",
@@ -44,9 +45,9 @@ def get_nearest_continuous(
         Target timestamps. Naive datetimes are treated as UTC. Accepts a
         list, ``pandas.Series``, ``pandas.DatetimeIndex``, ``numpy.ndarray``,
         or anything ``pandas.to_datetime`` consumes.
-    monitoring_location_id : string or list of strings, optional
+    monitoring_location_id : string or iterable of strings, optional
         Forwarded to ``get_continuous``.
-    parameter_code : string or list of strings, optional
+    parameter_code : string or iterable of strings, optional
         Forwarded to ``get_continuous``.
     window : string or ``pandas.Timedelta``, default ``"PT7M30S"``
         Half-window around each target, as an ISO 8601 duration

dataretrieval/waterdata/ratings.py

@@ -22,7 +22,13 @@
 
 from dataretrieval.rdb import extract_rdb_comment, read_rdb
 
-from .utils import _DURATION_RE, BASE_URL, _default_headers, _format_api_dates
+from .utils import (
+    _DURATION_RE,
+    BASE_URL,
+    _check_monitoring_location_id,
+    _default_headers,
+    _format_api_dates,
+)
 
 logger = logging.getLogger(__name__)
 
@@ -33,7 +39,7 @@
 
 
 def get_ratings(
-    monitoring_location_id: str | list[str] | None = None,
+    monitoring_location_id: str | Iterable[str] | None = None,
     file_type: RATING_FILE_TYPE | list[RATING_FILE_TYPE] = "exsa",
     file_path: str | None = None,
     time: str | list[str] | None = None,
@@ -62,7 +68,7 @@ def get_ratings(
 
     Parameters
     ----------
-    monitoring_location_id : string or list of strings, optional
+    monitoring_location_id : string or iterable of strings, optional
         One or more identifiers in ``AGENCY-ID`` form (e.g.
         ``"USGS-01104475"``). If omitted, the spatial / temporal filters
         determine the result set.
@@ -142,6 +148,7 @@ def get_ratings(
         ... )
 
     """
+    monitoring_location_id = _check_monitoring_location_id(monitoring_location_id)
     file_types = _as_list(file_type)
     invalid = [ft for ft in file_types if ft not in _VALID_FILE_TYPES]
     if invalid: