fix(compat/isMatchWith): align nested primitive matching with lodash in isMatchWith

src/compat/predicate/isMatch.spec.ts

@@ -33,6 +33,10 @@
     expect(isMatch({ a: { b: { c: 1, d: 2 }, e: 3 }, f: 4 }, { a: { b: { c: 1 } } })).toBe(true);
   });
 
+  it('should return `false` when nested source primitive does not match object target', () => {
+    expect(isMatch({ a: { b: 2 } }, { a: 1 })).toBe(false);
+  });
+
   it(`should match boolean values`, () => {
     // eslint-disable-next-line @typescript-eslint/ban-ts-comment
     // @ts-expect-error
@@ -60,7 +64,7 @@
 
     const Foo = function Foo(this: Foo) {
       this.a = 1;
     } as any as FooConstructor;
 
     Foo.prototype.b = 2;
 
@@ -80,7 +84,7 @@
 
     const Foo = function Foo(this: Foo) {
       this.a = 1;
     } as any as FooConstructor;
 
     Foo.prototype.b = 2;
 
@@ -150,7 +154,7 @@
 
     const Foo = function Foo(this: Foo, object: Partial<Foo>) {
       Object.assign(this, object);
     } as any as FooConstructor;
 
     // eslint-disable-next-line
     // @ts-ignore
@@ -285,16 +289,16 @@
       // eslint-disable-next-line @typescript-eslint/ban-ts-comment
       // @ts-expect-error
       expect(isMatch(1, { b: undefined })).toBe(true);
     } catch (e: any) {
       expect(false, e.message);
     }
 
     try {
       // eslint-disable-next-line @typescript-eslint/ban-ts-comment
       // @ts-expect-error
       expect(isMatch(1, { a: 1, b: undefined })).toBe(true);
     } catch (e: any) {
       expect(false, e.message);
     }
 
     // eslint-disable-next-line
@@ -304,8 +308,8 @@
       // eslint-disable-next-line @typescript-eslint/ban-ts-comment
       // @ts-expect-error
       expect(isMatch(1, { a: { c: undefined } })).toBe(true);
     } catch (e: any) {
       expect(false, e.message);
     }
 
     // eslint-disable-next-line

src/compat/predicate/isMatchWith.spec.ts

@@ -12,7 +12,7 @@
 describe('isMatchWith', () => {
   it('should provide correct `customizer` arguments', () => {
     const argsList: unknown[] = [];
     const object1: any = { a: [1, 2], b: null };
     const object2: any = { a: [1, 2], b: null };
 
     object1.b = object2;
@@ -338,6 +338,10 @@
     expect(isMatchWith({}, {}, rootCustomizer)).toBe(true);
   });
 
+  it('should return `false` when nested source primitive does not match object target', () => {
+    expect(isMatchWith({ a: { b: 2 } }, { a: 1 }, () => undefined)).toBe(false);
+  });
+
   it('should handle empty collections', () => {
     // eslint-disable-next-line @typescript-eslint/ban-ts-comment
     // @ts-expect-error

src/compat/predicate/isMatchWith.ts

@@ -124,9 +124,10 @@ export function isMatchWith(
         return Boolean(isEqual);
       }
 
-      return isMatchWithInternal(objValue, srcValue, doesMatch, stack);
+      return isMatchWithInternal(objValue, srcValue, doesMatch, stack, false);
     },
-    new Map()
+    new Map(),
+    true
   );
 }
 
@@ -141,7 +142,8 @@ function isMatchWithInternal(
     source: any,
     stack?: Map<any, any>
   ) => boolean | undefined,
-  stack?: Map<any, any>
+  stack?: Map<any, any>,
+  isRoot = false
 ): boolean {
   if (source === target) {
     return true;
@@ -155,7 +157,7 @@ function isMatchWithInternal(
       const sourceKeys = Object.keys(source);
 
       if (sourceKeys.length > 0) {
-        return isMatchWithInternal(target, { ...source }, compare, stack);
+        return isMatchWithInternal(target, { ...source }, compare, stack, isRoot);
       }
 
       return eq(target, source);
@@ -165,11 +167,15 @@ function isMatchWithInternal(
         return eq(target, source);
       }
 
-      if (typeof source === 'string') {
-        return source === '';
+      if (isRoot) {
+        if (typeof source === 'string') {
+          return source === '';
+        }
+
+        return true;
       }
 
-      return true;
+      return eq(target, source);
     }
   }
 }

src/compat/predicate/matches.spec.ts

@@ -25,6 +25,10 @@ describe('matches', () => {
     expect(isMatch5({ a: { b: { c: 1, d: 2 }, e: 3 }, f: 4 })).toBe(true);
   });
 
+  it('should return `false` when nested source primitive does not match object target', () => {
+    expect(matches({ a: 1 })({ a: { b: 2 } })).toBe(false);
+  });
+
   it(`should match inherited string keyed \`object\` properties`, () => {
     interface Foo {
       a: number;