@sap-ux/app-config-writer 1.0.7 → 1.0.9

package/dist/eslint-config/convert.js

@@ -192,6 +192,29 @@ async function removeFioriToolsFromExistingConfig(basePath, fs, logger) {
     logger?.debug(`Removed SAP Fiori tools plugin references from ${configPath}`);
     return JSON.stringify(eslintConfig, null, 2);
 }
+/**
+ * Finds the index of the `[` that is matched by the `]` at the given closing index, walking
+ * backwards through the string while tracking bracket depth.
+ *
+ * @param content - the full file content string
+ * @param closingIndex - the index of the `]` whose matching `[` we want to find
+ * @returns the index of the matching `[`, or -1 if not found
+ */
+function findMatchingOpenBracket(content, closingIndex) {
+    let depth = 0;
+    for (let i = closingIndex; i >= 0; i--) {
+        if (content.charAt(i) === ']') {
+            depth++;
+        }
+        else if (content.charAt(i) === '[') {
+            depth--;
+            if (depth === 0) {
+                return i;
+            }
+        }
+    }
+    return -1;
+}
 /**
  * Injects the SAP Fiori tools plugin import and config spread into the migrated flat-config file.
  *
@@ -216,10 +239,13 @@ async function injectFioriToolsIntoMigratedConfig(basePath, fs, config = 'recomm
     if (lastBracketIndex === -1) {
         throw new Error('Unexpected format of migrated eslint config. Could not inject the SAP Fiori tools plugin configuration.');
     }
+    const beforeBracket = content.slice(0, lastBracketIndex);
+    const matchingOpenIndex = findMatchingOpenBracket(content, lastBracketIndex);
+    const betweenBrackets = matchingOpenIndex === -1 ? '' : content.slice(matchingOpenIndex + 1, lastBracketIndex);
+    const isEmptyArray = /^\s*$/.test(betweenBrackets);
+    const separator = isEmptyArray ? '' : ',';
     content =
-        content.slice(0, lastBracketIndex) +
-            `,\n    ...fioriTools.configs['${config}'],\n` +
-            content.slice(lastBracketIndex);
+        beforeBracket + `${separator}\n    ...fioriTools.configs['${config}'],\n` + content.slice(lastBracketIndex);
     fs.write(migratedConfigPath, content);
     logger?.debug(`Injected SAP Fiori tools plugin into ${migratedConfigPath}`);
 }

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@sap-ux/app-config-writer",
   "description": "Add or update configuration for SAP Fiori tools application",
-  "version": "1.0.7",
+  "version": "1.0.9",
   "repository": {
     "type": "git",
     "url": "https://github.com/SAP/open-ux-tools.git",
@@ -30,7 +30,7 @@
     "semver": "7.7.4",
     "cross-spawn": "7.0.6",
     "@sap-ux/axios-extension": "2.0.2",
-    "@sap-ux/ui5-application-writer": "2.0.3",
+    "@sap-ux/ui5-application-writer": "2.0.4",
     "@sap-ux/btp-utils": "2.0.2",
     "@sap-ux/logger": "1.0.1",
     "@sap-ux/project-access": "2.1.2",
@@ -47,7 +47,7 @@
     "@types/cross-spawn": "6.0.6",
     "axios": "1.16.0",
     "nock": "14.0.11",
-    "@sap-ux/preview-middleware": "1.0.14"
+    "@sap-ux/preview-middleware": "1.0.16"
   },
   "engines": {
     "node": ">=22.x"