fix(ci): upgrade node to v22 and update docs to reflect vitest usage

.devcontainer/devcontainer.json

@@ -20,8 +20,7 @@
       ],
       "settings": {
         "editor.formatOnSave": true,
-        "editor.defaultFormatter": "esbenp.prettier-vscode",
-        "files.eol": "\n"
+        "editor.defaultFormatter": "esbenp.prettier-vscode"
       }
     }
   },

.devcontainer/setup.sh

@@ -5,7 +5,7 @@ git config --global --add safe.directory $(pwd)
 
 # In your setup.sh or postCreateCommand
 sudo chown -R node:node /home/node/.cache/
-sudo chown -R node:node /workspaces/screeps-deploy-action/.git/hooks
+
 # 2. Re-connect Git Hooks
 pre-commit install
 pre-commit install-hooks

.editorconfig

@@ -1,8 +0,0 @@
-root = true
-
-[*]
-end_of_line = lf
-insert_final_newline = true
-charset = utf-8
-indent_style = space
-indent_size = 2

.gitattributes

@@ -1 +0,0 @@
-* text=auto eol=lf

.gitea/workflows/test.yaml

@@ -10,9 +10,9 @@ jobs:
     runs-on: pi
     steps:
     - uses: actions/checkout@v6
-    - uses: actions/setup-node@v6
+    - uses: actions/setup-node@v4
       with:
-        node-version: '24'
+        node-version: '22'
     - run: npm install
       shell: bash
     - run: npm test

.gitignore

@@ -1075,6 +1075,4 @@ FodyWeavers.xsd
 *.vsix
 
 /node_modules/
-/node_modules/.cache/
 /coverage/
-!/dist/

.pre-commit-config.yaml

@@ -5,20 +5,28 @@ repos:
   hooks:
   - id: check-yaml
   - id: check-json
+  - id: check-toml
+  - id: check-xml
   - id: check-added-large-files
     args: [--enforce-all]
     exclude: ^dist/index\.js$
+  - id: name-tests-test
   - id: detect-private-key
   - id: check-case-conflict
   - id: check-symlinks
+  - id: check-docstring-first
   - id: pretty-format-json
     args: [--autofix, --no-sort-keys, --no-ensure-ascii]
   - id: check-merge-conflict
   - id: no-commit-to-branch
 
 - repo: https://github.com/macisamuele/language-formatters-pre-commit-hooks
-  rev: v2.16.0
+  rev: v2.15.0
   hooks:
+  - id: pretty-format-ini
+    args: [--autofix]
+  - id: pretty-format-toml
+    args: [--autofix]
   - id: pretty-format-yaml
     args: [--autofix]
 
@@ -29,7 +37,7 @@ repos:
     types_or: [css, javascript]
 
 - repo: https://github.com/python-jsonschema/check-jsonschema
-  rev: 0.37.0
+  rev: 0.36.0
   hooks:
   - id: check-renovate
   - id: check-github-actions

__tests__/index.test.js

@@ -1,13 +1,8 @@
-import {
-  validateAuthentication,
-  replacePlaceholders,
-  readReplaceAndWriteFiles,
-  readFilesIntoDict,
-} from "../index.js";
-import fs from "fs";
-import path from "path";
-import os from "os";
-import { glob } from "glob";
+const { validateAuthentication, replacePlaceholders } = require("../index");
+const fs = require("fs");
+const path = require("path");
+const os = require("os");
+const { glob } = require("glob");
 
 describe("validateAuthentication", () => {
   it("should return null when only token is provided", () => {
@@ -72,83 +67,6 @@ describe("replacePlaceholders", () => {
   });
 });
 
-describe("readReplaceAndWriteFiles", () => {
-  let tempDir;
-
-  beforeEach(async () => {
-    tempDir = await fs.promises.mkdtemp(
-      path.join(os.tmpdir(), "replace-test-"),
-    );
-    process.env.GITHUB_SHA = "test-sha";
-    process.env.GITHUB_REF = "test-ref";
-  });
-
-  afterEach(async () => {
-    if (tempDir) {
-      await fs.promises.rm(tempDir, { recursive: true, force: true });
-    }
-  });
-
-  it("should find files and replace placeholders", async () => {
-    const fileName = "test.js";
-    const filePath = path.join(tempDir, fileName);
-    const content = "hash: {{gitHash}}, ref: {{gitRef}}, host: {{hostname}}";
-    await fs.promises.writeFile(filePath, content);
-
-    const pattern = "*.js";
-    // We pass tempDir as the prefix so glob searches inside it
-    await readReplaceAndWriteFiles(pattern, tempDir, "test-host");
-
-    const updatedContent = await fs.promises.readFile(filePath, "utf8");
-
-    expect(updatedContent).toContain("hash: test-sha");
-    expect(updatedContent).toContain("ref: test-ref");
-    expect(updatedContent).toContain("host: test-host");
-  });
-});
-
-describe("readFilesIntoDict", () => {
-  let tempDir;
-
-  beforeEach(async () => {
-    tempDir = await fs.promises.mkdtemp(path.join(os.tmpdir(), "read-test-"));
-    await fs.promises.mkdir(path.join(tempDir, "subdir"), { recursive: true });
-  });
-
-  afterEach(async () => {
-    if (tempDir) {
-      await fs.promises.rm(tempDir, { recursive: true, force: true });
-    }
-  });
-
-  it("should read files into a dictionary with correct keys", async () => {
-    const file1 = "file1.js";
-    const content1 = "content1";
-    await fs.promises.writeFile(path.join(tempDir, file1), content1);
-
-    const file2 = "subdir/file2.js";
-    const content2 = "content2";
-    await fs.promises.writeFile(path.join(tempDir, file2), content2);
-
-    const pattern = "**/*.js";
-    const result = await readFilesIntoDict(pattern, tempDir);
-
-    // Keys should be relative paths without extension
-    // On Windows, the path separator might differ, so we should be careful or just check contents
-
-    // Based on implementation:
-    // key = key.slice(prefix.length);
-    // key = path.basename(key, path.extname(key)); // Drop the file suffix -> THIS IS BUGGY for subdirs?
-
-    // Let's check the implementation of readFilesIntoDict again in index.js
-    // It does: key = path.basename(key, path.extname(key));
-    // This removes the directory part! So subdir/file2.js becomes file2
-
-    expect(result["file1"]).toBe(content1);
-    expect(result["file2"]).toBe(content2);
-  });
-});
-
 describe("glob functionality", () => {
   let tempDir;
 

dist/package.json

@@ -1,3 +0,0 @@
-{
-  "type": "module"
-}

index.js

@@ -1,9 +1,8 @@
-import { ScreepsAPI } from "screeps-api";
-import * as core from "@actions/core";
-import fs from "fs";
-import { glob } from "glob";
-import path from "path";
-import { fileURLToPath } from "url";
+const { ScreepsAPI } = require("screeps-api");
+const core = require("@actions/core");
+const fs = require("fs");
+const { glob } = require("glob");
+const path = require("path");
 
 /**
  * Replaces specific placeholder strings within the provided content with corresponding dynamic values.
@@ -18,7 +17,7 @@ import { fileURLToPath } from "url";
  * @param {string} content - The string content in which placeholders are to be replaced.
  * @returns {string} The content with placeholders replaced by their respective dynamic values.
  */
-export function replacePlaceholders(content, hostname) {
+function replacePlaceholders(content, hostname) {
   const deployTime = new Date().toISOString();
   return content
     .replace(/{{gitHash}}/g, process.env.GITHUB_SHA)
@@ -38,7 +37,7 @@ export function replacePlaceholders(content, hostname) {
  * @param {string} [prefix] - An optional directory prefix to prepend to the glob pattern. This allows searching within a specific directory.
  * @returns {Promise<string[]>} A promise that resolves with an array of file paths that were processed, or rejects with an error if the process fails.
  */
-export async function readReplaceAndWriteFiles(pattern, prefix, hostname) {
+async function readReplaceAndWriteFiles(pattern, prefix, hostname) {
   const globPattern = prefix ? path.join(prefix, pattern) : pattern;
   const files = await glob(globPattern);
 
@@ -59,7 +58,7 @@ export async function readReplaceAndWriteFiles(pattern, prefix, hostname) {
  * @param {string} prefix - Directory prefix for file paths.
  * @returns {Promise<Object>} - Promise resolving to a dictionary of file contents keyed by filenames.
  */
-export async function readFilesIntoDict(pattern, prefix) {
+async function readFilesIntoDict(pattern, prefix) {
   // Prepend the prefix to the glob pattern
   const globPattern = prefix ? path.join(prefix, pattern) : pattern;
   const files = await glob(globPattern);
@@ -89,7 +88,7 @@ export async function readFilesIntoDict(pattern, prefix) {
  * @param {string} password - The password.
  * @returns {string|null} - Returns an error message if validation fails, otherwise null.
  */
-export function validateAuthentication(token, username, password) {
+function validateAuthentication(token, username, password) {
   if (token) {
     if (username || password) {
       return "Token is defined along with username and/or password.";
@@ -111,7 +110,7 @@ export function validateAuthentication(token, username, password) {
 /**
  * Posts code to Screeps server.
  */
-export async function postCode() {
+async function postCode() {
   const protocol = core.getInput("protocol") || "https";
   const hostname = core.getInput("hostname") || "screeps.com";
   const port = core.getInput("port") || "443";
@@ -175,8 +174,9 @@ export async function postCode() {
       });
   }
 }
+postCode();
 
-const __filename = fileURLToPath(import.meta.url);
-if (process.argv[1] === __filename) {
-  postCode();
-}
+module.exports = {
+  validateAuthentication,
+  replacePlaceholders,
+};

package.json

@@ -2,16 +2,15 @@
   "name": "screeps-deploy-action",
   "version": "0.1.1",
   "description": "Deploys screeps code to the official game or an pirvate server.",
-  "type": "module",
   "main": "index.js",
   "scripts": {
     "start": "node index.js",
     "test": "vitest run --globals --coverage",
-    "build": "ncc build index.js -o dist -m --external utf-8-validate --external bufferutil"
+    "build": "ncc build index.js -o dist --external utf-8-validate --external bufferutil"
   },
   "dependencies": {
-    "@actions/core": "^3.0.0",
-    "glob": "^13.0.0",
+    "@actions/core": "^1.11.1",
+    "glob": "^11.0.1",
     "screeps-api": "^1.7.2"
   },
   "devDependencies": {