Rule proposal: no-useless-boolean-cast

[geoffswift]

Description

We already have this good thing happening with prefer-native-coercion-functions

// ❌
const example = values.filter((value) => Boolean(value));

// ✅
const example = values.filter(Boolean);

Some times though you can have a situation where there's an unnecessary coercion though, when the callback is more complex.

The standard rule no-extra-boolean-cast stops me doing something like this

// ❌
if (Boolean(example))

// ✅
if (example)

Examples

// ❌
const example = records.filter(({ fieldname }) => Boolean(fieldname));

// ✅
const example = records.filter(({ fieldname }) => fieldname);

// ❌
const example = records.every((record) => Boolean(record.fieldname));

// ✅
const example = records.every((record) => record.fieldname);

// ❌
const example = records.some((record) => Boolean(record.fieldname));

// ✅
const example = records.some((record) => record.fieldname);

Proposed rule name

no-useless-boolean-cast

Additional Info

This might be giving some performance benefits.