Commit Graph

7 Commits

Author SHA1 Message Date
Jamie Tanna a9c8f7431f fix(lint): extend prefer-is-helpers to catch more cases (#45474)
This now covers everything that isn't `isObject` or `typeof x === 'number'`.

Co-authored-by: Claude Sonnet 5 <jamie.tanna+claude-code@mend.io>
2026-08-26 10:48:40 +00:00
Sebastian Poxhofer 16ef1715e9 refactor(linters): add rules for @sindresorhus/is helpers (#45229)
* feat(lint): add renovate/prefer-is-helpers and renovate/prefer-is-object rules

Flag ad-hoc type checks that have @sindresorhus/is equivalents:
- .filter(Boolean) -> .filter(isTruthy)
- typeof x === 'string' -> isString(x)
- x === null || x === undefined -> isNullOrUndefined(x) (and negation)

The typeof x === 'object' shape lives in a separate suggestion-only rule
(renovate/prefer-is-object, kept at warn) because isObject()/isPlainObject()
are not drop-in equivalents: null, arrays and functions behave differently.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

* refactor: fix renovate/prefer-is-helpers violations

Replace ad-hoc type checks with @sindresorhus/is helpers across lib/:
- 13x .filter(Boolean) -> .filter(isTruthy)
- 36x typeof x ==/!== 'string' -> isString(x) / !isString(x)
- 6x x === null || x === undefined -> isNullOrUndefined(x) (and negation)

Promote renovate/prefer-is-helpers to error now that the codebase is clean.
renovate/prefer-is-object stays at warn: its 15 remaining sites guard
parsed YAML/JSON or unknown values where isObject()/isPlainObject() are
not provably equivalent (null/array/function semantics).

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

* refactor: enforce renovate/prefer-is-object at error level

Resolve the 15 remaining warnings and raise the rule from warn to error:

- Migrate 10 sites to isObject() where the guarded value provably cannot
  be a function (parsed YAML/JSON, typed exec command arguments, or
  test-controlled fixtures): helm-values, woodpecker, bun, pre-commit
  managers, exec utils, file package cache, and the github http spec.
  In the bun manager this also fixes a latent crash where a null
  `workspaces` value would throw on the `in` operator.
- Keep 5 sites as reasoned oxlint-disable comments where excluding
  functions is load-bearing: the byte-exact fingerprint canonicalizer
  (2), the circularity walker, the zod result guard, and the generic
  hasKey helper.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

---------

Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
2026-08-13 14:22:03 +00:00
mado e4710c1df5 fix(util/fingerprint): incrementally hash input to avoid V8 max string length (#43605)
* fix(util/fingerprint): incrementally hash input to avoid V8 max string length

* test(util/fingerprint): cover backward-compat with safeStringify + hash

* style(util/fingerprint): satisfy prettier + oxlint (no-unnecessary-type-assertion)

* refactor(util/fingerprint): trim comments to WHY only

* fix(util/fingerprint): pre-check emittability to match safe-stable-stringify on toJSON-returns-undefined; satisfy biome import sort

* test(util/fingerprint): cover root function/symbol and toJSON-returns-undefined branches

* style(util/fingerprint): drop unnecessary non-null assertion

* perf(util/fingerprint): memoize isEmittable per object reference
2026-06-01 09:03:25 +00:00
Sergei Zharinov 930cf6633c refactor: Rewrite imports with .ts extensions (#40700)
fix: Additional vi.mock path corrections and prettier formatting
2026-01-26 22:48:17 +00:00
Adam Setch 39c907231b refactor(util/hash): replace hasha sha512 use cases (#23548) 2023-07-24 22:54:28 +00:00
RahulGautamSingh c45c875920 fix: modify fns to adapt to new safe-stable-stringify package update (#18573) 2022-11-20 06:19:19 +01:00
Rhys Arkins dbc4d0b669 fix: exclude unstable data from cache fingerprints (#18148) 2022-10-06 15:39:15 +02:00