Skip to content

fix(lint): clear all 15 oxlint warnings - #620

Merged
unohee merged 2 commits into
Intrect-io:mainfrom
aniruddhaadak80:fix/lint-warnings
Sep 15, 2026
Merged

unohee merged 2 commits into
Intrect-io:mainfrom
aniruddhaadak80:fix/lint-warnings

Conversation

@aniruddhaadak80

@aniruddhaadak80 aniruddhaadak80 commented Sep 10, 2026

Copy link
Copy Markdown
Contributor

Summary

Clears all 15 oxlint warnings (npm run lint: 15 warnings / 0 errors before, 0 / 0 after), while preserving the daily reporter's persisted-watermark contract.

  • Removes unused catch bindings and verified dead private fields from the browser UI.
  • Removes a useless object-spread fallback and fixes optional header access in a test.
  • Keeps readWatermark() and wires it into generateDailyReports(): a completed UTC-day report is skipped after restart, while a partial failure writes no watermark and remains retryable.
  • Adds focused regression coverage for both watermark paths.

Related issue

Closes AGT-4295.

Type of change

  • Bug fix
  • New feature
  • Refactor / chore
  • Docs

Verification

  • npm run lint — 0 warnings, 0 errors across 844 files
  • npm run typecheck
  • npm run build
  • Focused tests — 47 passed, including the real browser 403 prompt/retry path and both watermark paths
  • Full suite under Node 22 — 5,982 passed, 8 failed, 6 skipped; the same 8 environment/tool-discovery failures reproduce on current main

@unohee
unohee merged commit 8bb0bae into Intrect-io:main Sep 15, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants