refactor(linter): remove for-of loop

Author: araujogui

src/linter/rules/missing-llm-description.mjs

@@ -8,27 +8,40 @@ import { LINT_MESSAGES } from '../constants.mjs';
  * @returns {Array<import('../types.d.ts').LintIssue>}
  */
 export const missingLlmDescription = entries => {
-  const issues = [];
+  return entries
+    .filter(entry => {
+      // Only process top-level headings
+      if (entry.heading.depth !== 1) {
+        return false;
+      }
 
-  for (const entry of entries) {
-    if (entry.heading.depth !== 1 || entry.llm_description) {
-      continue;
-    }
+      // Skip entries that have an llm_description property
+      if (entry.llm_description !== undefined) {
+        return false;
+      }
 
-    const descriptionNode = entry.content.children.find(
-      child => child.type === 'paragraph'
-    );
+      const hasParagraph = entry.content.children.some(
+        node => node.type === 'paragraph'
+      );
 
-    if (!descriptionNode) {
-      issues.push({
-        level: 'warn',
-        message: LINT_MESSAGES.missingLlmDescription,
-        location: {
-          path: entry.api_doc_source,
-        },
-      });
-    }
-  }
+      // Skip entries that contain a paragraph that can be used as a fallback.
+      if (hasParagraph) {
+        return false;
+      }
 
-  return issues;
+      return true;
+    })
+    .map(entry => mapToMissingEntryWarning(entry));
 };
+
+/**
+ * Maps a entry to a warning for missing llm description.
+ *
+ * @param {ApiDocMetadataEntry} entry
+ * @returns {import('../types.d.ts').LintIssue}
+ */
+const mapToMissingEntryWarning = entry => ({
+  level: 'warn',
+  message: LINT_MESSAGES.missingLlmDescription,
+  location: { path: entry.api_doc_source },
+});