chore: update lint workflow to include manual stage hooks

This PR contains the following updates:

| Package | Change | [Age](https://docs.renovatebot.com/merge-confidence/) | [Confidence](https://docs.renovatebot.com/merge-confidence/) |
|---|---|---|---|
| [@vitest/coverage-v8](https://vitest.dev/guide/coverage) ([source](https://github.com/vitest-dev/vitest/tree/HEAD/packages/coverage-v8)) | [`4.1.5` → `4.1.6`](https://renovatebot.com/diffs/npm/@vitest%2fcoverage-v8/4.1.5/4.1.6) | ![age](https://developer.mend.io/api/mc/badges/age/npm/@vitest%2fcoverage-v8/4.1.6?slim=true) | ![confidence](https://developer.mend.io/api/mc/badges/confidence/npm/@vitest%2fcoverage-v8/4.1.5/4.1.6?slim=true) |
| [vitest](https://vitest.dev) ([source](https://github.com/vitest-dev/vitest/tree/HEAD/packages/vitest)) | [`4.1.5` → `4.1.6`](https://renovatebot.com/diffs/npm/vitest/4.1.5/4.1.6) | ![age](https://developer.mend.io/api/mc/badges/age/npm/vitest/4.1.6?slim=true) | ![confidence](https://developer.mend.io/api/mc/badges/confidence/npm/vitest/4.1.5/4.1.6?slim=true) |

---

### Release Notes

<details>
<summary>vitest-dev/vitest (@&#8203;vitest/coverage-v8)</summary>

### [`v4.1.6`](https://github.com/vitest-dev/vitest/releases/tag/v4.1.6)

[Compare Source](https://github.com/vitest-dev/vitest/compare/v4.1.5...v4.1.6)

#####    🐞 Bug Fixes

- **browser**: Provide project reference in `ToMatchScreenshotResolvePath`  -  by [@&#8203;macarie](https://github.com/macarie) and [@&#8203;sheremet-va](https://github.com/sheremet-va) in [#&#8203;10138](https://github.com/vitest-dev/vitest/issues/10138) [<samp>(31882)</samp>](https://github.com/vitest-dev/vitest/commit/31882607c)
- Global `sequence.concurrent: true` with top-level `test(..., { concurrent: false })` + depreacte `sequential` test API and options  -  by [@&#8203;hi-ogawa](https://github.com/hi-ogawa), **Codex** and [@&#8203;sheremet-va](https://github.com/sheremet-va) in [#&#8203;10196](https://github.com/vitest-dev/vitest/issues/10196) [<samp>(2847d)</samp>](https://github.com/vitest-dev/vitest/commit/2847dfa2a)
- **browser**: Simplify orchestrator otel carrier  -  by [@&#8203;hi-ogawa](https://github.com/hi-ogawa) in [#&#8203;10285](https://github.com/vitest-dev/vitest/issues/10285) [<samp>(18af9)</samp>](https://github.com/vitest-dev/vitest/commit/18af98cee)

#####    🏎 Performance

- Stringify diff objects only once  -  by [@&#8203;sheremet-va](https://github.com/sheremet-va) in [#&#8203;10276](https://github.com/vitest-dev/vitest/issues/10276) [<samp>(9f7b1)</samp>](https://github.com/vitest-dev/vitest/commit/9f7b1528c)

#####     [View changes on GitHub](https://github.com/vitest-dev/vitest/compare/v4.1.5...v4.1.6)

</details>

---

### Configuration

📅 **Schedule**: (UTC)

- Branch creation
  - "on monday,on friday"
- Automerge
  - At any time (no schedule defined)

🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about these updates again.

---

 - [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check this box

---

This PR has been generated by [Mend Renovate](https://github.com/renovatebot/renovate).
<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiI0My4xNjkuNSIsInVwZGF0ZWRJblZlciI6IjQzLjE2OS41IiwidGFyZ2V0QnJhbmNoIjoibWFpbiIsImxhYmVscyI6WyJhdXRvbWF0aW9uIiwibnBtIl19-->

Reviewed-on: https://git.horstenkamp.eu/Screeps/screeps-deploy-action/pulls/82
Co-authored-by: Renovate Bot <renovate@horstenkamp.eu>
Co-committed-by: Renovate Bot <renovate@horstenkamp.eu>

.devcontainer/devcontainer.json

@@ -20,7 +20,8 @@
       ],
       "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

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

.gitattributes

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

.gitea/workflows/lint.yaml

@@ -9,10 +9,10 @@ jobs:
     name: pre-commit Linting
     runs-on: pi
     steps:
-    - uses: actions/checkout@v6
+    - uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd # v6
-    - uses: actions/setup-python@v6
+    - uses: actions/setup-python@a309ff8b426b58ec0e2a45f0f869d46889d02405 # v6
     - run: pip install pre-commit
       shell: bash
     - name: Pre Commit
-      run: SKIP=no-commit-to-branch pre-commit run -a
+      run: SKIP=no-commit-to-branch pre-commit run -a --hook-stage pre-commit --hook-stage manual
       shell: bash

.gitea/workflows/test.yaml

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

.gitignore

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

.pre-commit-config.yaml

@@ -1,3 +1,4 @@
+default_stages: [pre-commit, manual]
 exclude: ^(dist|node_modules)/
 repos:
 - repo: https://github.com/pre-commit/pre-commit-hooks
@@ -5,28 +6,21 @@ 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
+    stages: [pre-commit]
 
 - repo: https://github.com/macisamuele/language-formatters-pre-commit-hooks
-  rev: v2.15.0
+  rev: v2.16.0
   hooks:
-  - id: pretty-format-ini
-    args: [--autofix]
-  - id: pretty-format-toml
-    args: [--autofix]
   - id: pretty-format-yaml
     args: [--autofix]
 
@@ -37,18 +31,37 @@ repos:
     types_or: [css, javascript]
 
 - repo: https://github.com/python-jsonschema/check-jsonschema
-  rev: 0.36.0
+  rev: 0.37.2
   hooks:
   - id: check-renovate
   - id: check-github-actions
+    stages: [manual]
   - id: check-github-workflows
+    stages: [manual]
 
 - repo: https://github.com/pre-commit/pre-commit-hooks
   rev: v6.0.0
   hooks:
   - id: end-of-file-fixer
-    exclude: (.txt$|.ipynb$|README.md$|readme.mde$)
+    exclude: (.txt$|.ipynb$|README.md$)
   - id: trailing-whitespace
     exclude: (.txt$|README.md$)
   - id: mixed-line-ending
     args: [--fix=lf]
+
+- repo: local
+  hooks:
+  - id: build
+    name: rebuild-distribution
+    entry: npm run build
+    language: system
+    files: ^(index\.js|monitor\.js|package\.json|package-lock\.json)$
+    pass_filenames: false
+    stages: [pre-commit]
+  - id: add-dist
+    name: stage-distribution
+    entry: git add dist/
+    language: system
+    files: ^(index\.js|monitor\.js|package\.json|package-lock\.json)$
+    pass_filenames: false
+    stages: [pre-commit]

__tests__/index.test.js

@@ -1,8 +1,13 @@
-const { validateAuthentication, replacePlaceholders } = require("../index");
+import {
-const fs = require("fs");
+  validateAuthentication,
-const path = require("path");
+  replacePlaceholders,
-const os = require("os");
+  readReplaceAndWriteFiles,
-const { glob } = require("glob");
+  readFilesIntoDict,
+} from "../index.js";
+import fs from "fs";
+import path from "path";
+import os from "os";
+import { glob } from "glob";
 
 describe("validateAuthentication", () => {
   it("should return null when only token is provided", () => {
@@ -67,6 +72,83 @@ 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

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

index.js

@@ -1,8 +1,9 @@
-const { ScreepsAPI } = require("screeps-api");
+import { ScreepsAPI } from "screeps-api";
-const core = require("@actions/core");
+import * as core from "@actions/core";
-const fs = require("fs");
+import fs from "fs";
-const { glob } = require("glob");
+import { glob } from "glob";
-const path = require("path");
+import path from "path";
+import { fileURLToPath } from "url";
 
 /**
  * Replaces specific placeholder strings within the provided content with corresponding dynamic values.
@@ -17,7 +18,7 @@ const path = require("path");
  * @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.
  */
-function replacePlaceholders(content, hostname) {
+export function replacePlaceholders(content, hostname) {
   const deployTime = new Date().toISOString();
   return content
     .replace(/{{gitHash}}/g, process.env.GITHUB_SHA)
@@ -37,7 +38,7 @@ 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.
  */
-async function readReplaceAndWriteFiles(pattern, prefix, hostname) {
+export async function readReplaceAndWriteFiles(pattern, prefix, hostname) {
   const globPattern = prefix ? path.join(prefix, pattern) : pattern;
   const files = await glob(globPattern);
 
@@ -58,7 +59,7 @@ 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.
  */
-async function readFilesIntoDict(pattern, prefix) {
+export 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);
@@ -88,7 +89,7 @@ async function readFilesIntoDict(pattern, prefix) {
  * @param {string} password - The password.
  * @returns {string|null} - Returns an error message if validation fails, otherwise null.
  */
-function validateAuthentication(token, username, password) {
+export function validateAuthentication(token, username, password) {
   if (token) {
     if (username || password) {
       return "Token is defined along with username and/or password.";
@@ -110,7 +111,7 @@ function validateAuthentication(token, username, password) {
 /**
  * Posts code to Screeps server.
  */
-async function postCode() {
+export async function postCode() {
   const protocol = core.getInput("protocol") || "https";
   const hostname = core.getInput("hostname") || "screeps.com";
   const port = core.getInput("port") || "443";
@@ -174,9 +175,8 @@ async function postCode() {
       });
   }
 }
-postCode();
 
-module.exports = {
+const __filename = fileURLToPath(import.meta.url);
-  validateAuthentication,
+if (process.argv[1] === __filename) {
-  replacePlaceholders,
+  postCode();
-};
+}

package.json

@@ -2,15 +2,16 @@
   "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 --external utf-8-validate --external bufferutil"
+    "build": "ncc build index.js -o dist -m --external utf-8-validate --external bufferutil"
   },
   "dependencies": {
-    "@actions/core": "^1.11.1",
+    "@actions/core": "^3.0.0",
-    "glob": "^11.0.1",
+    "glob": "^13.0.0",
     "screeps-api": "^1.7.2"
   },
   "devDependencies": {