ts-data-forge 7.0.0 → 7.0.1

package/src/functional/result/impl/result-flat-map.mts

@@ -12,9 +12,9 @@ import { type UnwrapErr, type UnwrapOk } from './types.mjs';
  *
  * ```ts
  * const parseNumber = (input: string): Result<number, string> => {
- *   const num = Number.parseInt(input, 10);
+ *   const num = Num.safeParseInt(input);
  *
- *   return Number.isNaN(num) ? Result.err('not a number') : Result.ok(num);
+ *   return Result.mapErr(num, () => 'not a number');
  * };
  *
  * const parsed = Result.flatMap(Result.ok('42'), parseNumber);

package/src/functional/result.test.mts

@@ -1,4 +1,5 @@
 import { expectType } from '../expect-type.mjs';
+import { Num } from '../number/index.mjs';
 import { type Err, type Ok } from '../types.mjs';
 import { Optional } from './optional/index.mjs';
 import { pipe } from './pipe.mjs';
@@ -467,7 +468,7 @@ describe('Result test', () => {
 
     test('should support chaining multiple flatMaps', () => {
       const parseNumber = (s: string): Result<number, string> => {
-        const n = Number(s);
+        const n = Result.unwrapOkOr(Num.safeParseFloat(s), Number.NaN);
 
         return Number.isNaN(n) ? Result.err('Not a number') : Result.ok(n);
       };
@@ -518,7 +519,7 @@ describe('Result test', () => {
 
     test('should work with pipe when curried', () => {
       const parseNumber = (s: string): Result<number, string> => {
-        const n = Number(s);
+        const n = Result.unwrapOkOr(Num.safeParseFloat(s), Number.NaN);
 
         return Number.isNaN(n) ? Result.err('Not a number') : Result.ok(n);
       };

package/src/functional/ternary-result/impl/ternary-result-flat-map.mts

@@ -15,9 +15,11 @@ import { type UnwrapErr, type UnwrapOk, type UnwrapWarn } from './types.mjs';
  *
  * ```ts
  * const parse = (value: string): TernaryResult<number, string, string> =>
- *   Number.isNaN(Number(value))
- *     ? TernaryResult.err('NaN')
- *     : TernaryResult.ok(Number(value));
+ *   pipe(Num.safeParseFloat(value)).map((result) =>
+ *     Result.isErr(result)
+ *       ? TernaryResult.err('NaN')
+ *       : TernaryResult.ok(result.value),
+ *   ).value;
  *
  * const doubled = TernaryResult.flatMap(TernaryResult.ok('3'), (text) =>
  *   TernaryResult.map(parse(text), (num) => num * 2),

package/src/functional/ternary-result.test.mts

@@ -1,5 +1,8 @@
 import { expectType } from '../expect-type.mjs';
+import { Num } from '../number/index.mjs';
 import { Optional } from './optional/index.mjs';
+import { pipe } from './pipe.mjs';
+import { Result } from './result/index.mjs';
 import { TernaryResult } from './ternary-result/index.mjs';
 
 describe('TernaryResult test', () => {
@@ -69,9 +72,11 @@ describe('TernaryResult test', () => {
 
   test('flatMap propagates Warn and Err while keeping warnings', () => {
     const parse = (value: string): TernaryResult<number, string, never> =>
-      Number.isNaN(Number(value))
-        ? TernaryResult.err('NaN')
-        : TernaryResult.ok(Number(value));
+      pipe(Num.safeParseFloat(value)).map((result) =>
+        Result.isErr(result)
+          ? TernaryResult.err('NaN')
+          : TernaryResult.ok(result.value),
+      ).value;
 
     const warn = TernaryResult.warn('3', 'slow');
 

package/src/guard/has-key.mts

@@ -47,9 +47,7 @@ export const hasKey = <
 >(
   obj: R,
   key: K,
-): obj is HasKeyReturnType<R, K> =>
-  // eslint-disable-next-line ts-data-forge/prefer-is-record-and-has-key
-  Object.hasOwn(obj, key);
+): obj is HasKeyReturnType<R, K> => Object.hasOwn(obj, key);
 
 /**
  * @internal

package/src/guard/is-non-null-object.mts

@@ -33,5 +33,4 @@
  */
 // eslint-disable-next-line @typescript-eslint/no-restricted-types
 export const isNonNullObject = (u: unknown): u is object =>
-  // eslint-disable-next-line ts-data-forge/prefer-is-non-null-object
   typeof u === 'object' && u !== null;

package/src/guard/key-is-in.mts

@@ -46,6 +46,4 @@ export const keyIsIn = <
 >(
   key: K,
   obj: R,
-): key is K & keyof typeof obj =>
-  // eslint-disable-next-line ts-data-forge/prefer-is-record-and-has-key
-  Object.hasOwn(obj, key);
+): key is K & keyof typeof obj => Object.hasOwn(obj, key);

package/src/json/json.mts

@@ -350,18 +350,15 @@ const keysDeepImpl = (
   // eslint-disable-next-line @typescript-eslint/prefer-readonly-parameter-types
   mut_keys: string[],
 ): void => {
-  for (const k of Object.keys(obj)) {
+  for (const [k, v] of Object.entries(obj)) {
     mut_keys.push(k);
 
-    const o = obj[k];
-
-    if (isRecord(o)) {
-      keysDeepImpl(o, mut_keys);
+    if (isRecord(v)) {
+      keysDeepImpl(v, mut_keys);
     }
 
-    // eslint-disable-next-line ts-data-forge/prefer-arr-is-array
-    if (Array.isArray(o)) {
-      for (const li of o) {
+    if (Array.isArray(v)) {
+      for (const li of v) {
         if (isRecord(li)) {
           keysDeepImpl(li, mut_keys);
         }

package/src/json/json.test.mts

@@ -1,6 +1,7 @@
 import { Arr } from '../array/index.mjs';
 import { Result } from '../functional/index.mjs';
 import { hasKey, isRecord, isString } from '../guard/index.mjs';
+import { Num } from '../number/index.mjs';
 import { Json } from './json.mjs';
 
 describe('parse', () => {
@@ -154,7 +155,7 @@ describe('parse', () => {
   test('should handle reviver returning different types', () => {
     const transformReviver = (key: string, value: unknown): unknown => {
       if (key === 'number' && typeof value === 'string') {
-        return Number.parseInt(value, 10);
+        return Result.unwrapOkOr(Num.safeParseInt(value), Number.NaN);
       }
 
       return value;
@@ -465,7 +466,6 @@ describe('stringifySelected', () => {
 
     // Note: stringifySelected works with JSON.stringify's replacer parameter
     // which may not work as expected with arrays
-    // eslint-disable-next-line ts-data-forge/prefer-arr-is-array
     assert.isTrue(Array.isArray(parsed));
 
     expect(parsed).toHaveLength(3);

package/src/number/branded-types/non-zero-safe-int.mts

@@ -31,9 +31,9 @@ const {
 >({
   integerOrSafeInteger: 'SafeInteger',
   nonZero: true,
-  // eslint-disable-next-line total-functions/no-unsafe-type-assertion, math/prefer-number-min-safe-integer, ts-data-forge/prefer-as-int
+  // eslint-disable-next-line total-functions/no-unsafe-type-assertion, math/prefer-number-min-safe-integer
   MIN_VALUE: Number.MIN_SAFE_INTEGER as SafeInt,
-  // eslint-disable-next-line total-functions/no-unsafe-type-assertion, math/prefer-number-max-safe-integer, ts-data-forge/prefer-as-int
+  // eslint-disable-next-line total-functions/no-unsafe-type-assertion, math/prefer-number-max-safe-integer
   MAX_VALUE: Number.MAX_SAFE_INTEGER as SafeUint,
   typeNameInMessage,
 } as const);

package/src/number/branded-types/positive-safe-int.mts

@@ -32,7 +32,7 @@ const {
 >({
   integerOrSafeInteger: 'SafeInteger',
   MIN_VALUE: 1,
-  // eslint-disable-next-line total-functions/no-unsafe-type-assertion, math/prefer-number-max-safe-integer, ts-data-forge/prefer-as-int
+  // eslint-disable-next-line total-functions/no-unsafe-type-assertion, math/prefer-number-max-safe-integer
   MAX_VALUE: Number.MAX_SAFE_INTEGER as SafeUint,
   typeNameInMessage,
 } as const);

package/src/number/branded-types/safe-int.mts

@@ -32,9 +32,9 @@ const {
   SafeUint
 >({
   integerOrSafeInteger: 'SafeInteger',
-  // eslint-disable-next-line total-functions/no-unsafe-type-assertion, math/prefer-number-min-safe-integer, ts-data-forge/prefer-as-int
+  // eslint-disable-next-line total-functions/no-unsafe-type-assertion, math/prefer-number-min-safe-integer
   MIN_VALUE: Number.MIN_SAFE_INTEGER as SafeInt,
-  // eslint-disable-next-line total-functions/no-unsafe-type-assertion, math/prefer-number-max-safe-integer, ts-data-forge/prefer-as-int
+  // eslint-disable-next-line total-functions/no-unsafe-type-assertion, math/prefer-number-max-safe-integer
   MAX_VALUE: Number.MAX_SAFE_INTEGER as SafeUint,
   typeNameInMessage,
 } as const);

package/src/number/branded-types/safe-uint.mts

@@ -29,7 +29,7 @@ const {
 >({
   integerOrSafeInteger: 'SafeInteger',
   MIN_VALUE: 0,
-  // eslint-disable-next-line total-functions/no-unsafe-type-assertion, math/prefer-number-max-safe-integer, ts-data-forge/prefer-as-int
+  // eslint-disable-next-line total-functions/no-unsafe-type-assertion, math/prefer-number-max-safe-integer
   MAX_VALUE: Number.MAX_SAFE_INTEGER as SafeUint,
   typeNameInMessage,
 } as const);

package/src/number/num-arithmetic-types.mts

@@ -52,7 +52,6 @@ type Sign = 'any' | 'neg' | 'nonneg' | 'nonpos' | 'nonzero' | 'pos';
 type Level = 'finite' | 'int' | 'number' | 'safeint';
 
 /** Classifies a numeric type's sign. */
-// transformer-ignore-next-line convert-to-readonly
 export type SignOf<N> = [N] extends [PositiveNumber]
   ? 'pos'
   : [N] extends [NegativeNumber]
@@ -66,7 +65,6 @@ export type SignOf<N> = [N] extends [PositiveNumber]
           : 'any';
 
 /** Classifies a numeric type's level. */
-// transformer-ignore-next-line convert-to-readonly
 export type LevelOf<N> = [N] extends [SafeInt]
   ? 'safeint'
   : [N] extends [Int]
@@ -77,7 +75,6 @@ export type LevelOf<N> = [N] extends [SafeInt]
 
 // --- boolean fact algebra over a sign ----------------------------------------
 
-// transformer-ignore-next-line convert-to-readonly
 type And<A extends boolean, B extends boolean> = [A, B] extends [true, true]
   ? true
   : false;
@@ -171,7 +168,6 @@ type DivLevel<LA extends Level, LB extends Level> = 'number' extends LA | LB
   : 'finite';
 
 /** Floor division yields an integer; the safe range is preserved. */
-// transformer-ignore-next-line convert-to-readonly
 type DivIntLevel<LA extends Level, LB extends Level> = 'number' extends LA | LB
   ? 'number'
   : [LA, LB] extends ['safeint', 'safeint']

package/src/number/num.mts

@@ -134,7 +134,7 @@ export namespace Num {
       ? Result.err(
           new Error(`safeParseInt: "${s}" is not a valid base-10 integer`),
         )
-      : // eslint-disable-next-line total-functions/no-unsafe-type-assertion, ts-data-forge/prefer-as-int
+      : // eslint-disable-next-line total-functions/no-unsafe-type-assertion
         Result.ok(Math.trunc(viaNumber) as Int);
   };
 
@@ -214,7 +214,7 @@ export namespace Num {
       ? Result.err(
           new Error(`safeParseFloat: "${s}" is not a valid finite number`),
         )
-      : // eslint-disable-next-line total-functions/no-unsafe-type-assertion, ts-data-forge/prefer-as-int
+      : // eslint-disable-next-line total-functions/no-unsafe-type-assertion
         Result.ok(viaNumber as FiniteNumber);
   };
 
@@ -649,7 +649,7 @@ export namespace Num {
    * @param num - The number to round
    * @returns The rounded integer as an Int branded type
    */
-  // eslint-disable-next-line total-functions/no-unsafe-type-assertion, ts-data-forge/prefer-as-int
+  // eslint-disable-next-line total-functions/no-unsafe-type-assertion
   export const roundToInt = (num: number): Int => Math.trunc(num + 0.5) as Int;
 
   /**

package/src/number/num.test.mts

@@ -164,7 +164,7 @@ describe('Num test', () => {
     test('rejects empty / whitespace-only input (unlike Number)', () => {
       assert.isTrue(Result.isErr(Num.safeParseInt('')));
 
-      assert.isTrue(Result.isErr(Num.safeParseInt('   ')));
+      assert.isTrue(Result.isErr(Num.safeParseInt(' '.repeat(3))));
     });
 
     test('rejects non-finite words', () => {
@@ -222,7 +222,7 @@ describe('Num test', () => {
     test('rejects empty / whitespace-only input (unlike Number)', () => {
       assert.isTrue(Result.isErr(Num.safeParseFloat('')));
 
-      assert.isTrue(Result.isErr(Num.safeParseFloat('   ')));
+      assert.isTrue(Result.isErr(Num.safeParseFloat(' '.repeat(3))));
     });
 
     test('rejects non-finite values', () => {

package/src/object/object.mts

@@ -505,10 +505,10 @@ const deepOmitImpl = (
 
   const head = path[0];
 
-  const tail = path.slice(1);
-
   if (!hasKey(record, head)) return record;
 
+  const tail = path.slice(1);
+
   if (!Arr.isNonEmpty(tail)) {
     return Object.fromEntries(
       Object.entries(record).filter(([k]) => k !== head),

package/src/others/fast-deep-equal.mts

@@ -1,5 +1,3 @@
-/* eslint-disable ts-data-forge/prefer-arr-is-array */
-
 import { isMap, isRegExp, isSet, isTypedArray } from '@sindresorhus/is';
 import { hasKey, isRecord } from '../guard/index.mjs';
 

package/src/others/map-nullable.test.mts

@@ -1,6 +1,7 @@
 import { type DeepReadonly } from 'ts-type-forge';
 import { expectType } from '../expect-type.mjs';
-import { pipe } from '../functional/index.mjs';
+import { pipe, Result } from '../functional/index.mjs';
+import { Num } from '../number/index.mjs';
 import { mapNullable } from './map-nullable.mjs';
 
 describe(mapNullable, () => {
@@ -306,7 +307,7 @@ describe(mapNullable, () => {
       const extractAge = mapNullable((data: FormDataType) => data.age);
 
       const parseAge = mapNullable((ageStr: string) =>
-        Number.parseInt(ageStr, 10),
+        Result.unwrapOkOr(Num.safeParseInt(ageStr), Number.NaN),
       );
 
       const validateAge = mapNullable((age: number) =>
@@ -336,7 +337,7 @@ describe(mapNullable, () => {
       const extractAge = mapNullable((data: FormDataType) => data.age);
 
       const parseAge = mapNullable((ageStr: string) =>
-        Number.parseInt(ageStr, 10),
+        Result.unwrapOkOr(Num.safeParseInt(ageStr), Number.NaN),
       );
 
       const extractedAge = extractAge(incompleteFormData);

package/src/others/unknown-to-string.mts

@@ -78,7 +78,7 @@ export const unknownToString = (
 
         // Special handling for Map
         if (isMap(value)) {
-          const entries = Array.from(value.entries());
+          const entries = Array.from(value);
 
           const entriesStr = prettyPrintObject
             ? JSON.stringify(entries, undefined, 2)
@@ -89,7 +89,7 @@ export const unknownToString = (
 
         // Special handling for Set
         if (isSet(value)) {
-          const values = Array.from(value.values());
+          const values = Array.from(value);
 
           const valuesStr = prettyPrintObject
             ? JSON.stringify(values, undefined, 2)