loopbackio / loopback-next / 39628942496

55%

push

github

refactor: resolve typescript-eslint v8 findings

- Drop unused `catch` bindings (`no-unused-vars`).
- Use optional chains where they are equivalent (`prefer-optional-chain`).
- Use `??=` for `if (x == null) x = y` and for `!x` guards on values that are
  objects or arrays, where it is equivalent (`prefer-nullish-coalescing`).
- Remove the `route()` overload that `RestApplication` declared twice
  (`unified-signatures`).
- Remove `eslint-disable` directives that no longer suppress anything.

`||` is kept wherever the left operand may be `''` or `0`. The passport-login
example keeps its explicit `!user.credentials` guard, since
`user.credentials?.password !== password` accepts a user without credentials
when `password` is `undefined`.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Signed-off-by: Raymond Feng <enjoyjava@gmail.com>

3435 of 10767 branches covered (31.9%)

Branch coverage included in aggregate %.

23 of 29 new or added lines in 15 files covered. (79.31%)

9561 of 12446 relevant lines covered (76.82%)

1308.14 hits per line