@abaplint/transpiler-cli 2.14.0 → 2.14.1

package/README.md

@@ -51,7 +51,7 @@ output/
 Names must be valid single directory names on Windows and POSIX. Names are
 unique without regard to case, and `project` is reserved. A conflicting derived
 name needs an explicit `libs[].name`. Duplicate ABAP objects across libraries
-are rejected; a project object replaces a library object of the same type/name.
+are rejected by default; a project object replaces a library object of the same type/name.
 
 Class locals, test classes, function groups, MIME data, and
 source maps follow their owning object. Source maps include ABAP source content,
@@ -68,3 +68,30 @@ files automatically.
 The transpiler library retains flat output unless its caller supplies an output
 folder map as the third argument to `Transpiler.run()`. This layout is enabled
 automatically by the CLI.
+
+## Duplicate dependencies
+
+By default, two libraries defining the same ABAP object (type and name) cause
+an `Ambiguous dependency object` error. To keep existing dependency repositories
+and skip later copies, set this top-level option in `abap_transpile.json`:
+
+```json
+{
+  "skip_duplicate_dependencies": true
+}
+```
+
+The first library in `libs` containing an object wins. Every file of that
+object in later libraries is skipped, including metadata, class locals, and
+binary assets. The CLI warns once per skipped object per library, naming the
+selected library. Imports, output folders, and source maps use the selected
+copy. Project objects still take precedence over dependencies, and shared
+`package.devc.xml` metadata keeps its existing behavior.
+
+Library order now affects generated output. Different definitions can change
+DDIC types, available methods, and runtime behavior, or cause syntax errors in
+code expecting the skipped version. This option does not check that copies are
+equivalent or suppress syntax checking. Put the intended provider first and
+run your application's tests. For selective control, use `libs[].exclude_filter`
+to exclude only the unwanted object's files instead. Use a clean generated
+output directory when changing providers, since old output is not removed.

package/build/bundle.js

@@ -340,7 +340,7 @@ async function build(config, files) {
         options.ignoreSourceMap = true;
     }
     const t = new Transpiler.Transpiler(options);
-    const { reg, folders, sources } = (0, libraries_1.libraryRegistry)(files, libraries);
+    const { reg, folders, sources } = (0, libraries_1.libraryRegistry)(files, libraries, config.skip_duplicate_dependencies);
     const output = await t.run(reg, new Progress(), folders);
     return { output, sources };
 }
@@ -559,13 +559,14 @@ async function loadLibraries(config) {
     }
     return result;
 }
-function libraryRegistry(files, libraries) {
+function libraryRegistry(files, libraries, skipDuplicateDependencies = false) {
     const reg = new abaplint.Registry();
     const folders = new Map();
     const objects = new Map();
     const sources = new Map();
     // Libraries first: addFile() then replaces a dependency object with the project's object.
     for (const lib of libraries) {
+        const skipped = new Set();
         for (const file of lib.files) {
             const memory = new abaplint.MemoryFile(file.filename, file.contents);
             const type = memory.getObjectType()?.toUpperCase();
@@ -576,6 +577,14 @@ function libraryRegistry(files, libraries) {
             const owner = objects.get(key);
             // abapGit repeats package.devc.xml across repositories; the registry keeps the last copy.
             if (key !== "DEVC:PACKAGE" && owner !== undefined && owner !== lib.name) {
+                if (skipDuplicateDependencies === true) {
+                    if (!skipped.has(key)) {
+                        console.warn("Skipping duplicate dependency object " + key + " in " + lib.name + "; using " + owner);
+                        skipped.add(key);
+                    }
+                    // Skip every file of the losing object, including locals, metadata, and binary assets.
+                    continue;
+                }
                 throw new Error("Ambiguous dependency object " + key + " in " + owner + " and " + lib.name);
             }
             objects.set(key, lib.name);

package/build/types.d.ts

@@ -33,6 +33,8 @@ export interface ITranspilerConfig {
          */
         exclude_filter?: string[];
     }[];
+    /** skip duplicate dependency objects after the first library in libs, with a warning; defaults to false */
+    skip_duplicate_dependencies?: boolean;
     write_unit_tests?: boolean;
     write_source_map?: boolean;
     options: ITranspilerOptions;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@abaplint/transpiler-cli",
-  "version": "2.14.0",
+  "version": "2.14.1",
   "description": "Transpiler - Command Line Interface",
   "funding": "https://github.com/sponsors/larshp",
   "bin": {
@@ -28,7 +28,7 @@
   "license": "MIT",
   "devDependencies": {
     "@abaplint/core": "^2.120.71",
-    "@abaplint/transpiler": "^2.14.0",
+    "@abaplint/transpiler": "^2.14.1",
     "@types/glob": "^9.0.0",
     "@types/node": "^26.6.2",
     "@types/progress": "^2.0.7",

package/schema.json

@@ -5,6 +5,35 @@
     "ITranspilerConfig": {
       "additionalProperties": false,
       "properties": {
+        "converter": {
+          "additionalProperties": false,
+          "description": "experimental",
+          "properties": {
+            "input_folder": {
+              "anyOf": [
+                {
+                  "type": "string"
+                },
+                {
+                  "items": {
+                    "type": "string"
+                  },
+                  "type": "array"
+                }
+              ],
+              "uniqueItems": true
+            },
+            "output_folder": {
+              "minLength": 1,
+              "type": "string"
+            }
+          },
+          "required": [
+            "input_folder",
+            "output_folder"
+          ],
+          "type": "object"
+        },
         "exclude_filter": {
           "description": "list of regex, case insensitive",
           "items": {
@@ -13,6 +42,10 @@
           "type": "array",
           "uniqueItems": true
         },
+        "incremental_output": {
+          "description": "Skip unchanged generated files and remove tracked obsolete output",
+          "type": "boolean"
+        },
         "input_filter": {
           "description": "list of regex, case insensitive, empty gives all files, positive list",
           "items": {
@@ -32,7 +65,8 @@
               },
               "type": "array"
             }
-          ]
+          ],
+          "uniqueItems": true
         },
         "libs": {
           "items": {
@@ -60,6 +94,12 @@
                 ]
               },
               "folder": {
+                "description": "relative to the current working directory",
+                "type": "string"
+              },
+              "name": {
+                "description": "output directory name; defaults to the repository or local folder name",
+                "minLength": 1,
                 "type": "string"
               },
               "url": {
@@ -74,8 +114,13 @@
           "$ref": "#/definitions/ITranspilerOptions"
         },
         "output_folder": {
+          "minLength": 1,
           "type": "string"
         },
+        "skip_duplicate_dependencies": {
+          "description": "skip duplicate dependency objects after the first library in libs, with a warning; defaults to false",
+          "type": "boolean"
+        },
         "write_source_map": {
           "type": "boolean"
         },