Fix handling of IssueErrors thrown from hed-validator (second attempt)

This does not issue warnings for missing HEDVersion fields for every dataset.

Author: happy5214

bids-validator/validators/hed.js

@@ -29,32 +29,24 @@ async function checkHedStrings(tsvs, jsonContents, jsonFiles) {
     )
   }
 
-  const issues = []
-  for (const [sidecarName, sidecarContents] of Object.entries(jsonContents)) {
-    try {
-      const sidecarFile = buildSidecar(sidecarName, sidecarContents, jsonFiles)
-      issues.push(...validateFile(sidecarFile, hedSchemas))
-    } catch (e) {
-      issues.push(new Issue({ code: 109 }))
-      return issues
-    }
-  }
+  const sidecarIssues = validateFiles(
+    buildSidecars(jsonContents, jsonFiles),
+    hedSchemas,
+  )
 
-  if (issues.some((issue) => issue.isError())) {
-    return issues
+  if (sidecarIssues.some((issue) => issue.isError() || issue.code === 109)) {
+    return sidecarIssues
   }
 
-  for (const tsv of tsvs) {
-    try {
-      const tsvFile = buildTsv(tsv, jsonContents)
-      issues.push(...validateFile(tsvFile, hedSchemas))
-    } catch (e) {
-      issues.push(new Issue({ code: 109 }))
-      return issues
-    }
-  }
+  const tsvIssues = validateFiles(buildTsvs(tsvs, jsonContents), hedSchemas)
 
-  return issues
+  return [...sidecarIssues, ...tsvIssues]
+}
+
+function* buildSidecars(jsonContents, jsonFiles) {
+  for (const [sidecarName, sidecarContents] of Object.entries(jsonContents)) {
+    yield buildSidecar(sidecarName, sidecarContents, jsonFiles)
+  }
 }
 
 function buildSidecar(sidecarName, sidecarContents, jsonFiles) {
@@ -63,6 +55,12 @@ function buildSidecar(sidecarName, sidecarContents, jsonFiles) {
   return new hedValidator.bids.BidsSidecar(sidecarName, sidecarContents, file)
 }
 
+function* buildTsvs(tsvs, jsonContents) {
+  for (const tsv of tsvs) {
+    yield buildTsv(tsv, jsonContents)
+  }
+}
+
 function buildTsv(tsv, jsonContents) {
   const potentialSidecars = utils.files.potentialLocations(
     tsv.file.relativePath.replace('.tsv', '.json'),
@@ -81,12 +79,20 @@ function buildTsv(tsv, jsonContents) {
   )
 }
 
-function validateFile(file, hedSchemas) {
-  const issues = file.validate(hedSchemas)
-  if (issues === null) {
-    throw new Error()
+function validateFiles(fileGenerator, hedSchemas) {
+  const issues = []
+  for (const file of fileGenerator) {
+    try {
+      const fileIssues = file.validate(hedSchemas)
+      if (fileIssues === null) {
+        return [new hedValidator.bids.BidsIssue(109)]
+      }
+      issues.push(fileIssues)
+    } catch (issueError) {
+      return hedValidator.bids.BidsHedIssue.fromHedIssues(issueError, file.file)
+    }
   }
-  return issues
+  return issues.flat()
 }
 
 function getSidecarFileObject(sidecarName, jsonFiles) {