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",
+        "editor.defaultFormatter": "esbenp.prettier-vscode"
-        "files.eol": "\n"
       }
     }
   },

.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]
 

__tests__/index.test.js

@@ -1,9 +1,4 @@
-const {
+const { validateAuthentication, replacePlaceholders } = require("../index");
-  validateAuthentication,
-  replacePlaceholders,
-  readReplaceAndWriteFiles,
-  readFilesIntoDict,
-} = require("../index");
 const fs = require("fs");
 const path = require("path");
 const os = require("os");
@@ -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;
 

index.js

@@ -174,15 +174,9 @@ async function postCode() {
       });
   }
 }
-
+postCode();
-if (require.main === module) {
-  postCode();
-}
 
 module.exports = {
   validateAuthentication,
   replacePlaceholders,
-  postCode,
-  readReplaceAndWriteFiles,
-  readFilesIntoDict,
 };

package.json

@@ -6,11 +6,11 @@
   "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": "^2.0.0",
+    "@actions/core": "^1.11.1",
-    "glob": "^13.0.0",
+    "glob": "^11.0.1",
     "screeps-api": "^1.7.2"
   },
   "devDependencies": {