fix: code smell

Author: BCsabaEngine

eslint.config.mjs

@@ -50,6 +50,7 @@ export default [
     },
 
     rules: {
+      curly: ['error', 'multi'],
       'simple-import-sort/imports': 'error',
       'simple-import-sort/exports': 'error',
       'unicorn/filename-case': 'off',

src/commandLine.ts

@@ -106,9 +106,8 @@ function parseArguments(): ICopyFilesArguments {
           break;
         case 'cachetime':
           result.cachetime = Number.parseInt(value, 10);
-          if (Number.isNaN(result.cachetime)) {
-            throw new TypeError(`Invalid cachetime: ${value}`);
-          }
+          if (Number.isNaN(result.cachetime)) throw new TypeError(`Invalid cachetime: ${value}`);
+
           break;
         default:
           throw new Error(`Unknown flag: ${flag}`);
@@ -190,9 +189,8 @@ function parseArguments(): ICopyFilesArguments {
           break;
         case 'cachetime':
           result.cachetime = Number.parseInt(nextArgument, 10);
-          if (Number.isNaN(result.cachetime)) {
-            throw new TypeError(`Invalid cachetime: ${nextArgument}`);
-          }
+          if (Number.isNaN(result.cachetime)) throw new TypeError(`Invalid cachetime: ${nextArgument}`);
+
           index++;
           break;
         default:

test/fixtures/sample-files/script.js

@@ -2,7 +2,5 @@ console.log('Hello from ESP32!');
 
 document.addEventListener('DOMContentLoaded', () => {
   const heading = document.querySelector('h1');
-  if (heading) {
-    heading.style.color = '#007bff';
-  }
+  if (heading) heading.style.color = '#007bff';
 });

test/unit/file.test.ts

@@ -87,9 +87,8 @@ describe('file', () => {
       vi.mocked(tinyglobby.globSync).mockReturnValue(mockFiles);
       vi.mocked(fs.readFileSync).mockImplementation((filePath: fs.PathOrFileDescriptor) => {
         const pathString = filePath.toString();
-        if (pathString.includes('file1') || pathString.includes('file2')) {
-          return content1;
-        }
+        if (pathString.includes('file1') || pathString.includes('file2')) return content1;
+
         return content3;
       });