2.12.26: refactor writing files (#1683)

Author: larshp

package-lock.json

@@ -41,16 +41,16 @@
     },
     "homepage": "https://github.com/abaplint/transpiler_poc#readme",
     "devDependencies": {
-        "@abaplint/core": "^2.115.16",
+        "@abaplint/core": "^2.115.17",
         "@eslint/compat": "^2.0.1",
         "@eslint/eslintrc": "^3.3.3",
         "@eslint/js": "^9.39.2",
         "@types/chai": "^4.3.20",
         "@types/mocha": "^10.0.10",
-        "@types/node": "^24.10.8",
+        "@types/node": "^24.10.9",
         "@types/sql.js": "^1.4.9",
-        "@typescript-eslint/eslint-plugin": "^8.53.0",
-        "@typescript-eslint/parser": "^8.53.0",
+        "@typescript-eslint/eslint-plugin": "^8.53.1",
+        "@typescript-eslint/parser": "^8.53.1",
         "chai": "^4.5.0",
         "cross-fetch": "^4.1.0",
         "dotenv": "^17.2.3",
@@ -59,8 +59,8 @@
         "fast-xml-parser": "^5.3.3",
         "globals": "^16.5.0",
         "mocha": "^11.7.5",
-        "pg": "^8.17.1",
-        "pg-cursor": "^2.16.1",
+        "pg": "^8.17.2",
+        "pg-cursor": "^2.16.2",
         "snowflake-sdk": "^2.3.3",
         "source-map-support": "^0.5.21",
         "sql.js": "^1.13.0",

package.json

@@ -1,6 +1,6 @@
 {
   "name": "@abaplint/transpiler-cli",
-  "version": "2.12.25",
+  "version": "2.12.26",
   "description": "Transpiler - Command Line Interface",
   "funding": "https://github.com/sponsors/larshp",
   "bin": {
@@ -27,10 +27,10 @@
   "author": "abaplint",
   "license": "MIT",
   "devDependencies": {
-    "@abaplint/core": "^2.115.16",
-    "@abaplint/transpiler": "^2.12.25",
+    "@abaplint/core": "^2.115.17",
+    "@abaplint/transpiler": "^2.12.26",
     "@types/glob": "^8.1.0",
-    "@types/node": "^24.10.8",
+    "@types/node": "^24.10.9",
     "@types/progress": "^2.0.7",
     "glob": "=7.2.0",
     "progress": "^2.0.3",

packages/cli/package-lock.json

@@ -71,4 +71,14 @@ export class FileOperations {
     return files;
   }
 
+  public static async writeFiles(files: {path: string, contents: string}[]) {
+    const limit = this.setupPLimit();
+    const promises = files.map((file) => {
+      return limit(async () => {
+        await fsPromises.writeFile(file.path, file.contents);
+      });
+    });
+    await Promise.all(promises);
+  }
+
 }

packages/cli/package.json

@@ -70,8 +70,12 @@ async function loadLib(config: ITranspilerConfig): Promise<Transpiler.IFile[]> {
   return files;
 }
 
-function writeObjects(outputFiles: Transpiler.IOutputFile[], config: ITranspilerConfig, outputFolder: string, files: Transpiler.IFile[]) {
+async function writeObjects(outputFiles: Transpiler.IOutputFile[],
+  config: ITranspilerConfig, outputFolder: string, files: Transpiler.IFile[]) {
+
   const writeSourceMaps = config.write_source_map || false;
+  const filesToWrite: {path: string, contents: string}[] = [];
+
   for (const output of outputFiles) {
     let contents = output.chunk.getCode();
     if (writeSourceMaps === true
@@ -92,15 +96,17 @@ function writeObjects(outputFiles: Transpiler.IOutputFile[], config: ITranspiler
           map = map.replace(`"${f.filename}"`, withPath);
         }
       }
-      fs.writeFileSync(outputFolder + path.sep + name, map);
+      filesToWrite.push({path: outputFolder + path.sep + name, contents: map});
     }
 
     if (output.object.type.toUpperCase() === "PROG") {
       // hmm, will this work for INCLUDEs ?
       contents = `if (!globalThis.abap) await import("./_init.mjs");\n` + contents;
     }
-    fs.writeFileSync(outputFolder + path.sep + output.filename, contents);
+    filesToWrite.push({path: outputFolder + path.sep + output.filename, contents});
   }
+
+  await FileOperations.writeFiles(filesToWrite);
 }
 
 async function build(config: ITranspilerConfig, files: Transpiler.IFile[]) {
@@ -135,7 +141,7 @@ async function run() {
     fs.mkdirSync(outputFolder);
   }
 
-  writeObjects(output.objects, config, outputFolder, files);
+  await writeObjects(output.objects, config, outputFolder, files);
   console.log(output.objects.length + " objects written to disk");
 
   if (config.write_unit_tests === true) {

packages/cli/src/file_operations.ts

@@ -1,6 +1,6 @@
 {
   "name": "@abaplint/runtime",
-  "version": "2.12.25",
+  "version": "2.12.26",
   "description": "Transpiler - Runtime",
   "main": "build/src/index.js",
   "typings": "build/src/index.d.ts",

packages/cli/src/index.ts

@@ -1,6 +1,6 @@
 {
   "name": "@abaplint/transpiler",
-  "version": "2.12.25",
+  "version": "2.12.26",
   "description": "Transpiler",
   "main": "build/src/index.js",
   "typings": "build/src/index.d.ts",
@@ -29,7 +29,7 @@
   "author": "abaplint",
   "license": "MIT",
   "dependencies": {
-    "@abaplint/core": "^2.115.16",
+    "@abaplint/core": "^2.115.17",
     "source-map": "^0.7.6"
   },
   "devDependencies": {