chore: code review

Author: ovflowd

src/generators/legacy-json-all/index.mjs

@@ -53,14 +53,14 @@ export default {
     input.forEach(section => {
       // Copy the relevant properties from each section into our output
       propertiesToCopy.forEach(property => {
-        if (Array.isArray(section[property])) {
-          let toPush = section[property];
+        const items = section[property];
 
-          if (section.source) {
-            toPush = toPush.map(item => ({ ...item, source: section.source }));
-          }
+        if (Array.isArray(items)) {
+          const enrichedItems = section.source
+            ? items.map(item => ({ ...item, source: section.source }))
+            : items;
 
-          generatedValue[property].push(...toPush);
+          generatedValue[property].push(...enrichedItems);
         }
       });
     });

src/generators/legacy-json/utils/buildSection.mjs

@@ -64,9 +64,8 @@ export const createSectionBuilder = () => {
     }
 
     // Check if there are any non-empty fields in the meta object
-    const atLeastOneNonEmptyField = Object.values(meta).some(
-      value => value.length > 0
-    );
+    const atLeastOneNonEmptyField =
+      changes?.length || Object.keys(meta).length > 1;
 
     // Return undefined if the meta object is completely empty
     return atLeastOneNonEmptyField ? meta : undefined;