TOMToolkit / tom_jpl / 32419142863

91%

dev: 86%

Pull #23

github

Cap and throttle the MPC fallback lookups to avoid hammering their server

The first real run against an already-populated database surfaced this
immediately: 497 individually-stuck targets (every historically-inactive
Scout candidate never processed under the new rename-based logic), which
meant 498 sequential HTTP requests to minorplanetcenter.net in one
invocation with no delay between them -- a burst of traffic to a small,
publicly-run service that's worth avoiding regardless of whether it
actually trips a rate limit.

Add --max-fallback-lookups (default 50) to cap how many individual lookups
one run attempts -- a large backlog is worked down gradually across
multiple cron runs instead of all at once, with a warning logged noting
how many were deferred -- and --fallback-lookup-delay (default 1.0s) to
sleep between each request. Both are threaded through
Command.handle() -> _update_designations() -> _fallback_lookup_designations().

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

642 of 692 new or added lines in 7 files covered. (92.77%)

1290 of 1419 relevant lines covered (90.91%)

0.91 hits per line