Cleanups (#570)

Author: paescuj

.github/ISSUE_TEMPLATE.md

@@ -0,0 +1,30 @@
+name: Bug Report
+description: File a bug report
+body:
+  - type: markdown
+    attributes:
+      value: |
+        Thank you for taking the time to fill out this bug report!
+        Before you get started, please ensure there's [no existing issue](https://github.com/open-cli-tools/concurrently/issues) with the same concern!
+  - type: textarea
+    attributes:
+      label: Describe the Bug
+      value: |
+        <!--
+
+        Please provide the following information:
+
+        - Description: Clear and concise description of what the bug is
+        - Expected Behavior: What you expected to happen
+        - Environment: At least the OS, Node.js, and concurrently's versions
+        - Reproduction: A way to reproduce the issue
+
+        Our time to work on this project is limited.
+        With an accurate report you're helping us to understand and solve the request faster.
+        Please understand that reports that do not meet the requirements may be closed without further notice.
+
+        Thank you!
+
+        -->
+    validations:
+      required: true

.github/ISSUE_TEMPLATE/bug-report.yml

@@ -0,0 +1,28 @@
+name: Feature Request
+description: Open a feature request
+body:
+  - type: markdown
+    attributes:
+      value: |
+        Thank you for taking the time to fill out this feature request!
+        Before you get started, please ensure there's [no existing issue](https://github.com/open-cli-tools/concurrently/issues) with the same concern!
+  - type: textarea
+    attributes:
+      label: Describe the Feature Request
+      value: |
+        <!--
+
+        Please provide the following information:
+
+        - Description: Clear and concise description of what you want
+        - Use Case: Why is this needed and what is a use case for it
+
+        Our time to work on this project is limited.
+        With an accurate report you're helping us to understand and solve the request faster.
+        Please understand that reports that do not meet the requirements may be closed without further notice.
+
+        Thank you!
+
+        -->
+    validations:
+      required: true

.github/ISSUE_TEMPLATE/feature-request.yml

@@ -13,11 +13,12 @@ runs:
   using: composite
   steps:
     - name: Install Node.js
-      uses: actions/setup-node@v4
+      uses: actions/setup-node@v5
       with:
         node-version-file: ${{ !inputs.node-version && '.node-version' || '' }}
         node-version: ${{ inputs.node-version }}
         registry-url: ${{ inputs.node-registry }}
+        package-manager-cache: false
 
     - name: Install pnpm
       uses: pnpm/action-setup@v4

.github/actions/setup/action.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 exec vitest --coverage
 
       - name: Submit coverage
         uses: coverallsapp/github-action@master

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

@@ -104,7 +104,7 @@ Check out documentation and other usage examples in the [`docs` directory](./doc
   - `prefix`: the prefix type to use when logging processes output.
     Possible values: `index`, `pid`, `time`, `command`, `name`, `none`, or a template (eg `[{time} process: {pid}]`).
     Default: the name of the process, or its index if no name is set.
-  - `prefixColors`: a list of colors or a string as supported by [chalk](https://www.npmjs.com/package/chalk) and additional style `auto` for an automatically picked color.
+  - `prefixColors`: a list of colors or a string as supported by [Chalk](https://www.npmjs.com/package/chalk) and additional style `auto` for an automatically picked color.
     If concurrently would run more commands than there are colors, the last color is repeated, unless if the last color value is `auto` which means following colors are automatically picked to vary.
     Prefix colors specified per-command take precedence over this list.
   - `prefixLength`: how many characters to show when prefixing with `command`. Default: `10`
@@ -224,6 +224,6 @@ It contains the following properties:
   So _null_ means the process didn't terminate normally. This will make **concurrently**
   to return non-zero exit code too.
 
-- Does this work with the npm-replacements [yarn](https://github.com/yarnpkg/yarn), [pnpm](https://pnpm.js.org/), or [Bun](https://bun.sh/)?
+- Does this work with the npm-replacements [yarn](https://yarnpkg.com/), [pnpm](https://pnpm.io/), or [Bun](https://bun.sh/)?
 
   Yes! In all examples above, you may replace "`npm`" with "`yarn`", "`pnpm`", or "`bun`".

.github/workflows/lint-format.yml

@@ -1,4 +1,4 @@
-/* eslint-disable no-console */
+import process from 'node:process';
 
 process.stdin.on('data', (chunk) => {
     const line = chunk.toString().trim();

README.md

@@ -4,13 +4,13 @@
  * (Windows doesn't provide the 'sleep' command by default,
  * see https://github.com/open-cli-tools/concurrently/issues/277)
  */
+import process from 'node:process';
 
-/* eslint-disable no-console */
-
-const seconds = process.argv[2];
-if (!seconds || isNaN(seconds) || process.argv.length > 3) {
+const seconds = +process.argv[2];
+if (!seconds || Number.isNaN(seconds) || process.argv.length > 3) {
     // Mimic behavior from native 'sleep' command
     console.error('usage: sleep seconds');
     process.exit(1);
 }
+
 await new Promise((resolve) => setTimeout(resolve, seconds * 1000));