enable Biome useBlockStatements rule (#1508)

Author: JamieMagee

app.js

@@ -243,7 +243,9 @@ function createApp(config) {
     } else {
       // Don't log Azure robot liveness checks
       // https://feedback.azure.com/forums/169385-web-apps/suggestions/32120617-document-healthcheck-url-requirement-for-custom-co
-      if (!request?.url?.includes('robots933456.txt')) logger.error(`SvcRequestFailure: ${request.url}`, error)
+      if (!request?.url?.includes('robots933456.txt')) {
+        logger.error(`SvcRequestFailure: ${request.url}`, error)
+      }
 
       // set locals, only providing error in development
       response.locals.message = error.message

bin/config.js

@@ -37,7 +37,9 @@ function loadOne(spec, namespace) {
   const getPath = (namespace ? `${namespace}.` : '') + requirePath
   let target = get(providers, getPath)
   try {
-    if (!target) target = require(requirePath)
+    if (!target) {
+      target = require(requirePath)
+    }
     return objectPath ? get(target, objectPath) : target
   } catch (e) {
     const message = e instanceof Error ? e.message : String(e)

biome.json

@@ -43,6 +43,7 @@
       },
       "style": {
         "noParameterAssign": "off",
+        "useBlockStatements": "error",
         "useDefaultParameterLast": "off"
       },
       "complexity": {

business/aggregator.js

@@ -80,7 +80,9 @@ class AggregationService {
         mergeDefinitions(result, data.summary)
       }
     }
-    if (!tools.length) return null
+    if (!tools.length) {
+      return null
+    }
     set(result, 'described.tools', tools.reverse())
     const cdSummarized = this._findData('clearlydefined', summarized)
     this._overrideDeclaredLicense(result, cdSummarized, coordinates)
@@ -113,8 +115,12 @@ class AggregationService {
    */
   _findData(toolSpec, summarized) {
     const [tool, toolVersion] = toolSpec.split('/')
-    if (!summarized[tool]) return null
-    if (toolVersion) return { toolSpec, summary: summarized[tool][toolVersion] }
+    if (!summarized[tool]) {
+      return null
+    }
+    if (toolVersion) {
+      return { toolSpec, summary: summarized[tool][toolVersion] }
+    }
 
     const versions = Object.getOwnPropertyNames(summarized[tool])
     const latest = getLatestVersion(versions)
@@ -131,9 +137,13 @@ class AggregationService {
    */
   _normalizeFiles(result, cdSummarized, coordinates) {
     const cdFiles = get(cdSummarized, 'summary.files')
-    if (!cdFiles || !cdFiles.length) return
+    if (!cdFiles || !cdFiles.length) {
+      return
+    }
     const difference = (result.files?.length || 0) - cdFiles.length
-    if (!difference) return
+    if (!difference) {
+      return
+    }
     this.logger.info('difference between summary file count and cd file count', {
       count: difference,
       coordinates: coordinates.toString()