feat: fix mergeWithConfig

Author: Lonli-Lokli

packages/ts-result/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@lonli-lokli/ts-result",
-  "version": "2.7.0",
+  "version": "2.8.0",
   "private": false,
   "sideEffects": false,
   "main": "dist/index.js",

packages/ts-result/src/lib/ts-result.spec.ts

@@ -811,6 +811,11 @@ describe('Result combining methods', () => {
       }
     });
 
+    test('single pending correctly processed', () => {
+      const merged = mergeWithConfig([pending], { priority: 'failure' });
+      expect(merged.isPending()).toBe(true);
+    });
+
     test('success case works the same in both modes', () => {
       const successResults = [success(1), success(2)];
       const defaultMerge = mergeWithConfig(successResults, {

packages/ts-result/src/lib/ts-result.ts

@@ -220,7 +220,7 @@ class ResultConstructor<F, S, T extends ResultType = ResultType>
     }
 
     const successes = values.filter((r) => r.isSuccess());
-    if (successes.length === values.length) {
+    if (successes.length === values.length && successes.length > 0) {
       return success(successes.map((r) => r.value as S));
     }