Bundle the code

Author: paescuj

.github/workflows/ci.yml

@@ -59,7 +59,7 @@ jobs:
           node-version: ${{ matrix.node }}
 
       - name: Test
-        run: pnpm exec vitest --coverage
+        run: pnpm test:all --coverage
 
       - name: Submit coverage
         uses: coverallsapp/github-action@master

bin/index.ts

@@ -8,9 +8,8 @@ import { assertDeprecated } from '../lib/assert.js';
 import * as defaults from '../lib/defaults.js';
 import { concurrently } from '../lib/index.js';
 import { castArray } from '../lib/utils.js';
-import { readPackageJson } from './read-package-json.js';
+import { version } from '../package.json' with { type: 'json' };
 
-const version = String(readPackageJson().version);
 const epilogue = `For documentation and more examples, visit:\nhttps://github.com/open-cli-tools/concurrently/tree/v${version}/docs`;
 
 // Clean-up arguments (yargs expects only the arguments after the program name)

bin/read-package-json.ts

@@ -21,7 +21,19 @@ export default defineConfig(
         },
     },
     eslint.configs.recommended,
-    tseslint.configs.recommended,
+    tseslint.configs.recommendedTypeChecked,
+    {
+        languageOptions: {
+            parserOptions: {
+                projectService: true,
+                tsconfigRootDir: import.meta.dirname,
+            },
+        },
+    },
+    {
+        files: ['**/*.js', '**/*.spec.ts', '**/__fixtures__/**/*', 'tests/**/*'],
+        extends: [tseslint.configs.disableTypeChecked],
+    },
     {
         rules: {
             curly: 'error',
@@ -36,6 +48,19 @@ export default defineConfig(
                     varsIgnorePattern: '^_',
                 },
             ],
+            '@typescript-eslint/prefer-promise-reject-errors': 'off',
+        },
+    },
+    {
+        files: ['**/*.ts'],
+        ignores: ['**/*.spec.ts', '**/__fixtures__/**/*', 'tests/**/*'],
+        rules: {
+            '@typescript-eslint/consistent-type-imports': 'error',
+            '@typescript-eslint/no-import-type-side-effects': 'error',
+            '@typescript-eslint/consistent-type-exports': [
+                'error',
+                { fixMixedExportsWithInlineTypeSpecifier: true },
+            ],
         },
     },
     { files: ['**/__fixtures__/**/*.{js,ts}'], rules: { 'no-console': 'off' } },
@@ -49,7 +74,7 @@ export default defineConfig(
             'simple-import-sort/exports': 'error',
             'import/first': 'error',
             'import/newline-after-import': 'error',
-            'import/no-duplicates': 'error',
+            'import/no-duplicates': ['error', { 'prefer-inline': true }],
         },
     },
     {

eslint.config.js

@@ -1,4 +1,4 @@
-import { CommandInfo } from '../command.js';
+import type { CommandInfo } from '../command.js';
 
 /**
  * A command parser encapsulates a specific logic for mapping `CommandInfo` objects

lib/command-parser/command-parser.d.ts

@@ -1,7 +1,7 @@
 import { quote } from 'shell-quote';
 
-import { CommandInfo } from '../command.js';
-import { CommandParser } from './command-parser.js';
+import type { CommandInfo } from '../command.js';
+import type { CommandParser } from './command-parser.js';
 
 /**
  * Replace placeholders with additional arguments.

lib/command-parser/expand-arguments.ts

@@ -1,5 +1,5 @@
-import { CommandInfo } from '../command.js';
-import { CommandParser } from './command-parser.js';
+import type { CommandInfo } from '../command.js';
+import type { CommandParser } from './command-parser.js';
 
 /**
  * Expands shortcuts according to the following table:

lib/command-parser/expand-shortcut.ts

@@ -1,9 +1,9 @@
 import fs from 'node:fs';
 
-import { CommandInfo } from '../command.js';
+import type { CommandInfo } from '../command.js';
 import JSONC from '../jsonc.js';
-import { escapeRegExp } from '../utils.js';
-import { CommandParser } from './command-parser.js';
+import { escapeRegExp, isObject } from '../utils.js';
+import type { CommandParser } from './command-parser.js';
 
 // Matches a negative filter surrounded by '(!' and ')'.
 const OMISSION = /\(!([^)]+)\)/;
@@ -14,7 +14,7 @@ const OMISSION = /\(!([^)]+)\)/;
  * configuration files of the current directory.
  */
 export class ExpandWildcard implements CommandParser {
-    static readDeno() {
+    static readDeno(this: void): unknown {
         try {
             let json: string = '{}';
 
@@ -30,7 +30,7 @@ export class ExpandWildcard implements CommandParser {
         }
     }
 
-    static readPackage() {
+    static readPackage(this: void): unknown {
         try {
             const json = fs.readFileSync('package.json', { encoding: 'utf-8' });
             return JSON.parse(json);
@@ -49,18 +49,23 @@ export class ExpandWildcard implements CommandParser {
 
     private relevantScripts(command: string): string[] {
         if (!this.packageScripts) {
-            this.packageScripts = Object.keys(this.readPackage().scripts || {});
+            const content = this.readPackage();
+            const scripts =
+                isObject(content) && isObject(content.scripts) ? Object.keys(content.scripts) : [];
+
+            this.packageScripts = scripts;
         }
 
         if (command === 'deno task') {
             if (!this.denoTasks) {
+                const content = this.readDeno();
+                const tasks =
+                    isObject(content) && isObject(content.tasks) ? Object.keys(content.tasks) : [];
+
                 // If Deno tries to run a task that doesn't exist,
                 // it can fall back to running a script with the same name.
                 // Therefore, the actual list of tasks is the union of the tasks and scripts.
-                this.denoTasks = [
-                    ...Object.keys(this.readDeno().tasks || {}),
-                    ...this.packageScripts,
-                ];
+                this.denoTasks = [...tasks, ...this.packageScripts];
             }
 
             return this.denoTasks;

lib/command-parser/expand-wildcard.ts

@@ -1,5 +1,5 @@
-import { CommandInfo } from '../command.js';
-import { CommandParser } from './command-parser.js';
+import type { CommandInfo } from '../command.js';
+import type { CommandParser } from './command-parser.js';
 
 /**
  * Strips quotes around commands so that they can run on the current shell.

lib/command-parser/strip-quotes.ts

@@ -1,12 +1,12 @@
-import { Buffer } from 'node:buffer';
-import {
+import type { Buffer } from 'node:buffer';
+import type {
     ChildProcess as BaseChildProcess,
     MessageOptions,
     SendHandle,
     SpawnOptions,
 } from 'node:child_process';
 import process from 'node:process';
-import { EventEmitter, Writable } from 'node:stream';
+import type { EventEmitter, Writable } from 'node:stream';
 
 import Rx from 'rxjs';