fix(every/fill/filter/find/findIndex/findLastIndex/indexOf/join): fix lodash compatibility

[D-Sketon]

add ArrayLike<T> | null | undefined for every/fill/filter/find/findIndex/findLastIndex/indexOf/join
add fromIndex for findIndex

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
es-toolkit	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Oct 14, 2024 2:54pm

[codecov-commenter]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 97.29%. Comparing base (76ec5f1) to head (bd4962e).
Report is 5 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main     #707   +/-   ##
=======================================
  Coverage   97.28%   97.29%           
=======================================
  Files         283      287    +4     
  Lines        2433     2477   +44     
  Branches      685      703   +18     
=======================================
+ Hits         2367     2410   +43     
- Misses         60       61    +1     
  Partials        6        6           

[raon0211]

I’m not completely convinced we need to explicitly mention that every and other functions accept null or undefined in the signature section of documentation.

The function signature ends up looking quite verbose, making it harder to focus on the key points.

How about we keep the original function signature in the docs and add a special info section like this:

 function every<T>(arr: T[]): boolean;
 function every<T>(arr: T[], doesMatch: (item: T, index: number, arr: T[]) => unknown): boolean;
 function every<T>(arr: T[], doesMatch: Partial<T>): boolean;
 function every<T>(arr: T[], doesMatch: [keyof T, unknown]): boolean;
 function every<T>(arr: T[], doesMatch: string): boolean;

 function every<T extends Record<string, unknown>>(
   object: T,
   doesMatch: (value: T[keyof T], key: keyof T, object: T) => unknown
 ): boolean;
 function every<T extends Record<string, unknown>>(object: T, doesMatch: Partial<T[keyof T]>): boolean;
 function every<T extends Record<string, unknown>>(object: T, doesMatch: [keyof T, unknown]): boolean;
 function every<T extends Record<string, unknown>>(object: T, doesMatch: string): boolean;

::: info `arr` can be `ArrayLike<T>`, `null`, or `undefined`

The `every` function is designed for complete compatibility with lodash:

- If arr is an `ArrayLike<T>`, it gets converted into an array using `Array.from(...)`.
- If arr is `null` or `undefined`, it will be treated as an empty array.

:::

::: info `object` can be `null` or `undefined`

To ensure full compatibility with lodash, the every function handles `object` in this way:

- If `object` is `null` or `undefined`, it will be converted into an empty object.

:::

[D-Sketon]

done

[raon0211]

Great work. Thanks so much!