• Maintainability | napi/parser/test/parse.test.ts:704-717
  These range tests removed the @ts-ignore but still access .range on AST nodes. That’s fine if the underlying types were fixed, but the deleted TODO explicitly said the TS definitions weren’t corrected yet. If the type definitions were not updated in this PR, the only reason this compiles is likely that the node is typed loosely (or any) somewhere, which can hide real typing regressions.

At minimum, consider reintroducing a targeted @ts-expect-error with a specific reason if the types are still known-incomplete, or ensure the tests assert range via a well-defined narrowing/type guard to keep this from being “accidentally untyped”.

• Readability | napi/parser/test/visit.test.ts:14-18
  Using null! here avoids @ts-ignore, but it introduces a non-null assertion to force a runtime error path. For tests that intentionally pass invalid inputs, prefer an explicit unsafe cast (null as unknown as VisitorObject) or a helper like asAny(null) if your codebase has one, so the intent is clearly “invalid type” rather than “I assert this isn’t null”.

null! can also read as accidental rather than deliberate invalid-input testing.

• Maintainability | tasks/e2e/test/utils.ts:61-70
  Assigning mockedWindow via {} as (typeof globalThis)["window"] makes the mock look like a real Window, but it’s still an empty object at runtime. That can mask missing properties and lead to false confidence in tests: code under test might access window.document (etc.) and crash, but this typing suggests it’s present.

If the goal is just to satisfy globalThis.window assignment typing, it’s better to type it as Window | undefined on the global, or assign globalThis.window = mockedWindow as unknown as Window with an explicit comment that it’s a minimal stub. Also, the delete globalThis.window suppression should explain why delete is needed (e.g., window is non-optional in lib DOM typings).

Summary of changes

• Enabled typescript/ban-ts-comment in oxlintrc.json with ts-expect-error allowed only when accompanied by a description.
• Updated several tests to replace // @ts-ignore with // @ts-expect-error (and added brief descriptions in some places).
• Added an oxlint-disable-next-line to keep // @ts-nocheck in lazy-deserialization.test.ts.
• Removed several @ts-ignore lines in parse.test.ts around range assertions.
• Replaced an unsafe new Visitor() call in visit.test.ts with new Visitor(null!) to exercise runtime validation.
• Tightened the mocked window typing in tasks/e2e/test/utils.ts and made the delete globalThis.window suppression an explicit @ts-expect-error.