fix(product, utils): handle metadata key deletion on product update (medusajs#12478)

Author: fPolic

.changeset/tall-beans-shave.md

@@ -0,0 +1,6 @@
+---
+"@medusajs/product": patch
+"@medusajs/utils": patch
+---
+
+fix(product, utils): handle metadata key deletion on product update

integration-tests/http/__tests__/product/admin/product.spec.ts

@@ -1912,7 +1912,7 @@ medusaIntegrationTestRunner({
 
           const payload = {
             metadata: {
-              "test-key": "",
+              "test-key": "", // item is deleted by setting to empty string
               "test-key-2": null,
             },
           }
@@ -1924,13 +1924,11 @@ medusaIntegrationTestRunner({
           )
 
           expect(response.status).toEqual(200)
-          expect(response.data.product.metadata).toEqual(
-            // BREAKING: Metadata updates are all-or-nothing in v2
-            {
-              "test-key": "",
-              "test-key-2": null,
-            }
-          )
+          expect(response.data.product.metadata).toEqual({
+            // "test-key" is deleted
+            "test-key-2": null, // updated
+            "test-key-3": "test-value-3", // preserved
+          })
         })
 
         it("updates products sales channels", async () => {

packages/core/utils/src/common/index.ts

@@ -72,7 +72,6 @@ export * from "./resolve-exports"
 export * from "./rules"
 export * from "./selector-constraints-to-string"
 export * from "./serialize-error"
-export * from "./set-metadata"
 export * from "./simple-hash"
 export * from "./string-to-select-relation-object"
 export * from "./stringify-circular"

packages/core/utils/src/common/set-metadata.ts

@@ -6,6 +6,8 @@ import {
   buildQuery,
   DALUtils,
   MedusaError,
+  isPresent,
+  mergeMetadata,
 } from "@medusajs/framework/utils"
 import { SqlEntityManager, wrap } from "@mikro-orm/postgresql"
 
@@ -148,6 +150,10 @@ export class ProductRepository extends DALUtils.mikroOrmBaseRepositoryFactory(
         )
       }
 
+      if (isPresent(productToUpdate.metadata)) {
+        productToUpdate.metadata = mergeMetadata(product.metadata ?? {}, productToUpdate.metadata)
+      }
+
       wrappedProduct.assign(productToUpdate)
     }