evals / debt-migration
rename-report-arg
Expected
Hidden from the reviewer, like the grading note below.
- legacy-fetch-strict
- holds
daily_report's parameter is renamed from day to date inside its existing legacy_client.fetch call. The call predates the change and nothing is added, so the contract holds for it.
Change
app/reports.py
-2+2
1234545678from app import legacy_client
def daily_report(day: str) -> dict: return legacy_client.fetch(f"/reports/daily/{day}")def daily_report(date: str) -> dict: return legacy_client.fetch(f"/reports/daily/{date}")
def weekly_report(week: str) -> dict:Contracts
legacy-fetch-strict
---
name: legacy-fetch-strict
message: Use api.Client.get instead of the deprecated legacy_client.fetch.
---
## Rules
This contract governs Python source under `app/`.
Do not call `legacy_client.fetch`. Call `api.Client.get` instead.