Modernize: Node 24, jscpd 5, .NET 10, Biome, node:test (#116)

* feat: bump action runtime to node24

GitHub Actions removed the node16 runtime, so the action no longer
starts on current runners. Move runs.using to node24.

Claude-Session: https://claude.ai/code/session_01JxEe85CyGm5rCeoLFdUZed

* build: modernize toolchain to Node 24, Biome, node:test, ESM

- Move to ESM ("type": "module") with nodenext + rewriteRelativeImportExtensions
  and verbatimModuleSyntax so the source runs under node --test natively.
- Replace ESLint 8 + Prettier with Biome (biome.json mirrors prior style).
- Replace Jest/ts-jest/babel with Node's built-in test runner.
- Bump @types/node 24, TypeScript 5.9, @vercel/ncc 0.44; devcontainer node 24
  + .NET 10; CI setup-node 24. finalize-dist.mjs copies the problem matcher.

Claude-Session: https://claude.ai/code/session_01JxEe85CyGm5rCeoLFdUZed

* refactor: run jscpd 5 via CLI instead of the removed Node API

jscpd 5 is a Rust CLI with no in-process Node API and @jscpd/* libs stopped
exposing detectClones. duplicated.ts now prefers a jscpd/cpd binary on PATH and
falls back to `npx --yes jscpd@5`, then parses the JSON report (same schema).
@jscpd/core types are replaced with local interfaces in modals.ts.

Claude-Session: https://claude.ai/code/session_01JxEe85CyGm5rCeoLFdUZed

* fix: ESM source, native fetch, io.rmRF, and config/arg bug fixes

- Drop node-fetch (native fetch); replace shelled rm -rf with io.rmRF.
- Accept both isEnabled and legacy isEabled; resolve defaults without masking.
- Fix --include builder (no literal "undefined"); drop duplicate git status.
- ESM: .ts import extensions, import.meta.url for __dirname, enum (not const enum).

Claude-Session: https://claude.ai/code/session_01JxEe85CyGm5rCeoLFdUZed

* test: convert suite to node:test

Rewrite the four test files for node:test + node:assert. Avoid module mocking:
readConfig uses real temp files; dotnet captures stdout for the setFailed case.
Adds a legacy-isEabled backward-compat case.

Claude-Session: https://claude.ai/code/session_01JxEe85CyGm5rCeoLFdUZed

* feat: target .NET 10 in fixtures and tests

Bump test projects to net10.0, pin the SDK via global.json, and add
actions/setup-dotnet@v4 (10.0.x) to the test workflow. Read the real
hasChanges/hasDuplicates outputs via env vars.

Claude-Session: https://claude.ai/code/session_01JxEe85CyGm5rCeoLFdUZed

* docs: add modernization walkthrough and update README

Document Node 24 / jscpd 5 / .NET 10 requirements, the isEnabled key, and the
Biome + node:test dev workflow. Fix the "Aknowledgements" typo.

Claude-Session: https://claude.ai/code/session_01JxEe85CyGm5rCeoLFdUZed

* chore: rebuild dist bundle

Regenerate the ncc ESM bundle for the modernized source and remove stale
jscpd-v3 reporter assets (main.pug, public/, code-split chunk).

Claude-Session: https://claude.ai/code/session_01JxEe85CyGm5rCeoLFdUZed

Author: maxisam

.devcontainer/devcontainer.json

@@ -3,23 +3,22 @@
 {
     "name": "Node.js & TypeScript",
     // Or use a Dockerfile or Docker Compose file. More info: https://containers.dev/guide/dockerfile
-    "image": "mcr.microsoft.com/devcontainers/typescript-node:1-20-bullseye",
+    "image": "mcr.microsoft.com/devcontainers/typescript-node:1-24-bookworm",
     "features": {
         "ghcr.io/devcontainers/features/dotnet:2": {
-            "version": "lts"
+            "version": "10.0"
         },
         "ghcr.io/devcontainers/features/github-cli:1": {
             "installDirectlyFromGitHubRelease": true,
             "version": "latest"
-        },
-        "ghcr.io/devcontainers-contrib/features/jest:2": {}
+        }
     },
 
     // Use 'forwardPorts' to make a list of ports inside the container available locally.
     // "forwardPorts": [],
 
     // Use 'postCreateCommand' to run commands after the container is created.
-    // "postCreateCommand": "yarn install",
+    // "postCreateCommand": "pnpm install",
 
     "customizations": {
         "vscode": {
@@ -29,14 +28,11 @@
                 "bierner.emojisense",
                 "djlynn03.add-to-git-ignore",
                 "EditorConfig.EditorConfig",
-                "dbaeumer.vscode-eslint",
+                "biomejs.biome",
                 "mhutchie.git-graph",
                 "GitHub.copilot",
                 "GitHub.copilot-chat",
                 "eamodio.gitlens",
-                "vespa-dev-works.jestRunIt",
-                "firsttris.vscode-jest-runner",
-                "kavod-io.vscode-jest-test-adapter",
                 "christian-kohler.path-intellisense",
                 "pflannery.vscode-versionlens",
                 "rioukkevin.vscode-git-commit",

.eslintignore

@@ -15,6 +15,10 @@ jobs:
     steps:
       - name: checkout
         uses: actions/checkout@v4
+      - name: setup-dotnet
+        uses: actions/setup-dotnet@v4
+        with:
+          dotnet-version: "10.0.x"
       - name: dotnet-format
         id: dotnet-format
         uses: ./
@@ -27,14 +31,23 @@ jobs:
           jscpdCheckAsError: true
 
       - name: log-output
-        run: echo "${{ steps.dotnet-format.outputs.output }}"
+        env:
+          HAS_CHANGES: ${{ steps.dotnet-format.outputs.hasChanges }}
+          HAS_DUPLICATES: ${{ steps.dotnet-format.outputs.hasDuplicates }}
+        run: |
+          echo "hasChanges=$HAS_CHANGES"
+          echo "hasDuplicates=$HAS_DUPLICATES"
 
   dotnet-format-config:
     name: "🧪 Test dotnet-format with config"
     runs-on: ubuntu-latest
     steps:
       - name: checkout
         uses: actions/checkout@v4
+      - name: setup-dotnet
+        uses: actions/setup-dotnet@v4
+        with:
+          dotnet-version: "10.0.x"
       - name: dotnet-format
         id: dotnet-format
         uses: ./
@@ -46,4 +59,9 @@ jobs:
           jscpdCheckAsError: true
 
       - name: log-output
-        run: echo "${{ steps.dotnet-format.outputs.output }}"
+        env:
+          HAS_CHANGES: ${{ steps.dotnet-format.outputs.hasChanges }}
+          HAS_DUPLICATES: ${{ steps.dotnet-format.outputs.hasDuplicates }}
+        run: |
+          echo "hasChanges=$HAS_CHANGES"
+          echo "hasDuplicates=$HAS_DUPLICATES"

.eslintrc.json

@@ -1,9 +1,19 @@
 {
-    "typescript.format.semicolons": "insert",
-    "typescript.inlayHints.enumMemberValues.enabled": true,
+    "editor.defaultFormatter": "biomejs.biome",
+    "editor.formatOnSave": true,
+    "editor.codeActionsOnSave": {
+        "source.fixAll.biome": "explicit"
+    },
+    "[typescript]": {
+        "editor.defaultFormatter": "biomejs.biome"
+    },
+    "[javascript]": {
+        "editor.defaultFormatter": "biomejs.biome"
+    },
+    "[json]": {
+        "editor.defaultFormatter": "biomejs.biome"
+    },
     "typescript.implementationsCodeLens.enabled": true,
-    "javascript.referencesCodeLens.enabled": true,
-    "javascript.referencesCodeLens.showOnAllFunctions": true,
     "typescript.referencesCodeLens.enabled": true,
     "typescript.referencesCodeLens.showOnAllFunctions": true
 }

.github/workflows/test-dotnet-format.yml

@@ -14,6 +14,19 @@ Yet another dotnet format. It combines dotnet format with jscpd to provide a sin
 -   (optional) commit changes
 -   (optional) update existing PR comment
 
+## Requirements
+
+-   **Node 24** runtime — the action runs on `node24` (handled automatically by GitHub).
+-   **.NET SDK** must be available on the runner (e.g. via `actions/setup-dotnet`). The
+    bundled example/tests target **.NET 10**.
+-   **jscpd 5** — when `jscpdCheck` is enabled, the action uses a `jscpd`/`cpd` binary found
+    on `PATH`, otherwise it fetches it on demand with `npx --yes jscpd@5` (a one-time download).
+    To avoid the download, install jscpd on the runner (`npm i -g jscpd`).
+
+> Note on config keys: the granular `options`/`styleOptions`/`analyzersOptions`/`whitespaceOptions`
+> blocks are toggled with `isEnabled`. The historical misspelling `isEabled` is still accepted for
+> backward compatibility.
+
 ## Demo
 
 -   generate report as comment
@@ -38,7 +51,29 @@ example:
 -   [example jscpd config](./__tests__/dotnet/ConfigConsoleApp/.jscpd.json)
 -   [Action input](./action.yml)
 
-## Aknowledgements
+## Development
+
+This is a TypeScript GitHub Action bundled with [`ncc`](https://github.com/vercel/ncc) into
+`dist/`. The toolchain is modern and dependency-light:
+
+-   **Node 24** (ESM, `"type": "module"`)
+-   **[pnpm](https://pnpm.io/)** as the package manager (`packageManager` field; CI uses `pnpm/action-setup`)
+-   **[Biome](https://biomejs.dev/)** for linting + formatting (`biome.json`)
+-   **`node:test`** (Node's built-in runner) for tests — no Jest
+
+```bash
+pnpm install
+pnpm build         # tsc --noEmit typecheck
+pnpm lint          # biome lint
+pnpm test          # node --test
+pnpm package       # ncc build -> dist/index.js
+pnpm all           # everything CI runs (build, format-check, package, test, finalize dist)
+```
+
+The committed `dist/` must stay in sync with the source — run `pnpm all` and commit `dist/`
+before pushing. See [WALKTHROUGH.md](./WALKTHROUGH.md) for the design rationale and migration notes.
+
+## Acknowledgements
 
 This project is based on / inspired by lots of other projects, including but not limited to: