Enable minification for build artifact (#60)

### **PR Type**
Enhancement

___

### **Description**
- Enable minification in build script.

- Produce minified `dist/index.js` bundle.

___

<details> <summary><h3> File Walkthrough</h3></summary>

<table><thead><tr><th></th><th align="left">Relevant files</th></tr></thead><tbody><tr><td><strong>Configuration changes</strong></td><td><table>
<tr>
  <td>
    <details>
      <summary><strong>package.json</strong><dd><code>Add minification flag to build script</code>&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; </dd></summary>
<hr>

package.json

- Added `-m` flag to build script.

</details>

  </td>
  <td><a href="https://git.horstenkamp.eu/Screeps/screeps-deploy-action/src/branch/build/minify/package.json">+1/-1</a>&nbsp; &nbsp; &nbsp; </td>

</tr>
</table></td></tr><tr><td><strong>Build artifact</strong></td><td><table>
<tr>
  <td>
    <details>
      <summary><strong>index.js</strong><dd><code>Provide minified distribution bundle</code>&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; </dd></summary>
<hr>

dist/index.js

- Updated output to minified bundle.

</details>

  </td>
  <td><a href="https://git.horstenkamp.eu/Screeps/screeps-deploy-action/src/branch/build/minify/dist/index.js">+5/-51912</a></td>

</tr>
</table></td></tr></tr></tbody></table>

</details>

___

Reviewed-on: #60

.gitea/workflows/test.yaml

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

.gitignore

@@ -1077,3 +1077,4 @@ FodyWeavers.xsd
 /node_modules/
 /node_modules/.cache/
 /coverage/
+!/dist/

__tests__/index.test.js

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

index.js

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

package.json

@@ -6,7 +6,7 @@
   "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": "^2.0.0",