Bundle the code

Author: paescuj

.github/workflows/test.yml .github/workflows/ci.yml.github/workflows/test.yml renamed to .github/workflows/ci.yml

@@ -1,30 +1,39 @@
-name: Test
+name: CI
 
 on:
   push:
     branches:
       - main
-    paths-ignore:
-      - '**.md'
-      - '**.ya?ml'
-      - '!pnpm-lock.yaml'
-      - '!.github/workflows/test.yml'
   pull_request:
     branches:
       - main
-    paths-ignore:
-      - '**.md'
-      - '**.ya?ml'
-      - '!pnpm-lock.yaml'
-      - '!.github/workflows/test.yml'
 
 concurrency:
   group: ${{ github.workflow }}-${{ github.ref }}
   cancel-in-progress: true
 
 jobs:
+  lint:
+    name: Lint
+    runs-on: ubuntu-latest
+    steps:
+      - name: Checkout repository
+        uses: actions/checkout@v5
+
+      - name: Setup
+        uses: ./.github/actions/setup
+
+      - name: Lint
+        run: pnpm run lint
+
+      - name: Format
+        run: pnpm run format
+
+      - name: Typecheck
+        run: pnpm run typecheck
+
   test:
-    name: Node.js ${{ matrix.node }} on ${{ matrix.os.name }}
+    name: Test (Node.js ${{ matrix.node }}, ${{ matrix.os.name }})
     runs-on: ${{ matrix.os.version }}
     strategy:
       fail-fast: false
@@ -49,11 +58,8 @@ jobs:
         with:
           node-version: ${{ matrix.node }}
 
-      - name: Install concurrently 🐶🥣
-        run: pnpm add --global concurrently
-
-      - name: Build & Test
-        run: concurrently --prefix none --group "pnpm:build" "pnpm:test --coverage" "pnpm:test:smoke"
+      - name: Test
+        run: pnpm run test:all --coverage
 
       - name: Submit coverage
         uses: coverallsapp/github-action@master

.github/workflows/lint-format.yml

@@ -0,0 +1,2 @@
+#!/usr/bin/env node
+import('./dist/bin.js');

bin.js

@@ -1,14 +1,12 @@
-#!/usr/bin/env node
 import yargs from 'yargs';
 import { hideBin } from 'yargs/helpers';
 
-import { assertDeprecated } from '../src/assert.js';
-import * as defaults from '../src/defaults.js';
-import { concurrently } from '../src/index.js';
-import { castArray } from '../src/utils.js';
-import { readPackageJson } from './read-package-json.js';
+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 { 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/concurrently.ts bin/index.tsbin/concurrently.ts renamed to bin/index.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',
@@ -30,6 +42,19 @@ export default defineConfig(
             'no-console': 'error',
             'prefer-const': 'error',
             'prefer-object-spread': 'error',
+            '@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 },
+            ],
         },
     },
     {
@@ -42,7 +67,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 }],
         },
     },
     {

bin/read-package-json.ts

@@ -1,5 +1,6 @@
-import EventEmitter from 'events';
-import { PassThrough, Writable } from 'stream';
+import EventEmitter from 'node:events';
+import { PassThrough, Writable } from 'node:stream';
+
 import { vi } from 'vitest';
 
 import { ChildProcess, CloseEvent, Command, CommandInfo } from '../command.js';