fix: some fixes

Author: araujogui

src/linter/rules/__tests__/invalid-change-version.test.mjs

@@ -76,8 +76,8 @@ changes:
         level: 'error',
         message: 'Missing version field in the API doc entry',
         position: {
-          start: { line: 4 },
-          end: { line: 4 },
+          start: { line: 3 },
+          end: { line: 3 },
         },
       },
     ]);
@@ -89,8 +89,8 @@ changes:
         level: 'error',
         message: 'Missing version field in the API doc entry',
         position: {
-          start: { line: 3 },
-          end: { line: 3 },
+          start: { line: 4 },
+          end: { line: 4 },
         },
       },
     ]);

src/linter/rules/invalid-change-version.mjs

@@ -72,20 +72,20 @@ export const extractVersions = ({ context, node, report }) => {
   if (!isMap(node)) {
     context.report(
       report(
-        LINT_MESSAGES.invalidChangeProperty.replace('{{type}}', node.NODE_TYPE),
+        LINT_MESSAGES.invalidChangeProperty.replace('{{type}}', node.type),
         node
       )
     );
 
-    return null;
+    return;
   }
 
   const versionNode = findPropertyByName(node, 'version');
 
   if (!versionNode) {
     context.report(report(LINT_MESSAGES.missingChangeVersion, node));
 
-    return null;
+    return;
   }
 
   return normalizeNode(versionNode.value);
@@ -126,16 +126,22 @@ export const invalidChangeVersion = context => {
         report(
           LINT_MESSAGES.invalidChangeProperty.replace(
             '{{type}}',
-            changesNode.value.NODE_TYPE
+            changesNode.value.type
           ),
           changesNode.key
         )
       );
     }
 
-    changesNode.value.items.forEach(node =>
-      extractVersions({ context, node, report })
-        .filter(Boolean) // Filter already reported empt items
+    changesNode.value.items.forEach(node => {
+      const versions = extractVersions({ context, node, report });
+
+      if (!versions) {
+        return;
+      }
+
+      versions
+        .filter(Boolean) // Filter already reported empt items,
         .filter(isInvalid)
         .forEach(version =>
           context.report(
@@ -149,7 +155,7 @@ export const invalidChangeVersion = context => {
               version
             )
           )
-        )
-    );
+        );
+    });
   });
 };