@sap-ux/fiori-migration-writer 0.0.2 → 0.0.3

package/dist/ProjectMigrator.js

@@ -1,6 +1,6 @@
 import { FioriElementsVersion, FileName } from './project-spec-types.js';
 import { determineMessage, readFile, updateFile } from './utils/index.js';
-import { createMemFsEditor, getCurrentEditor, runWithEditor, exists } from './utils/fs-adapter.js';
+import { createMemFsEditor, getCurrentEditor, runWithEditor, exists, loadProjectIntoEditor } from './utils/fs-adapter.js';
 import { commitFileSystemChanges } from './files/file-system.js';
 import { ui5VersionRequestInfo } from '@sap-ux/ui5-info';
 import { getAppProgrammingLanguage } from '@sap-ux/project-access';
@@ -50,6 +50,9 @@ export class ProjectMigrator {
             let messages = [];
             let result = false;
             try {
+                // Load the project tree into mem-fs so existence checks see untracked files
+                // on disk and migration writes never silently overwrite them.
+                await loadProjectIntoEditor(editor, projectRoot);
                 // Load or fetch project information
                 const { projectInfo, messages: projectInfoMessages } = await loadOrFetchProjectInfo(projectRoot, importProjectInfoOpt);
                 messages = messages.concat(projectInfoMessages);

package/dist/adapters/ui5-config-adapter.js

@@ -47,6 +47,40 @@ function addMockserverMiddleware(orderedYaml, templateData) {
         orderedYaml.server.customMiddleware.unshift(mockserverMiddleware);
     }
 }
+/**
+ * Create a new UI5Config instance with metadata and type pre-configured.
+ *
+ * @param templateData - Template data providing the config name
+ * @returns Configured UI5Config instance
+ */
+async function createBaseConfig(templateData) {
+    const config = await UI5Config.newInstance('', { validateSchema: false });
+    config.setMetadata({
+        name: templateData.ui5Yaml?.name?.toLowerCase() || templateData.project.name?.toLowerCase() || 'app'
+    });
+    config.setType('application');
+    return config;
+}
+/**
+ * Post-process a UI5Config YAML string: add specVersion 4.0 at the top and
+ * optionally set webapp and appreload paths.
+ *
+ * @param yamlContent - Raw YAML string from UI5Config.toString()
+ * @param webappPath - Optional webapp path to inject
+ * @returns Final YAML string with specVersion and optional paths applied
+ */
+function postProcessYaml(yamlContent, webappPath) {
+    const yamlJson = parse(yamlContent);
+    const orderedYaml = { specVersion: '4.0' };
+    Object.keys(yamlJson).forEach((key) => {
+        orderedYaml[key] = yamlJson[key];
+    });
+    if (webappPath) {
+        setWebappPath(orderedYaml, webappPath);
+        setAppreloadPath(orderedYaml, webappPath);
+    }
+    return stringify(orderedYaml);
+}
 /**
  * Build backend configuration from template data and neo-app destinations
  *
@@ -86,12 +120,7 @@ function buildBackendConfigs(templateData, neoappDestinations, messages, destina
  */
 export async function generateUI5YamlContent(options) {
     const { templateData, neoappDestinations, messages, destination, firstNeoAppDestination, webappPath, setUI5Version, strict = false } = options;
-    const config = await UI5Config.newInstance('', { validateSchema: false });
-    // Set metadata and type
-    config.setMetadata({
-        name: templateData.ui5Yaml?.name?.toLowerCase() || templateData.project.name?.toLowerCase() || 'app'
-    });
-    config.setType('application');
+    const config = await createBaseConfig(templateData);
     // Build backend configuration using helpers
     const backends = buildBackendConfigs(templateData, neoappDestinations, messages, destination, firstNeoAppDestination, strict);
     // Add fiori-tools-proxy middleware with all backends
@@ -104,20 +133,7 @@ export async function generateUI5YamlContent(options) {
     if (previewMiddleware) {
         config.addCustomMiddleware(previewMiddleware);
     }
-    const yamlContent = config.toString();
-    // Post-process: Add specVersion 4.0 (UI5Config doesn't set this)
-    const yamlJson = parse(yamlContent);
-    // Add specVersion at the top
-    const orderedYaml = { specVersion: '4.0' };
-    Object.keys(yamlJson).forEach((key) => {
-        orderedYaml[key] = yamlJson[key];
-    });
-    // Apply webappPath if provided
-    if (webappPath) {
-        setWebappPath(orderedYaml, webappPath);
-        setAppreloadPath(orderedYaml, webappPath);
-    }
-    return stringify(orderedYaml);
+    return postProcessYaml(config.toString(), webappPath);
 }
 /**
  * Generate ui5-local.yaml content using @sap-ux/ui5-config builder
@@ -135,12 +151,7 @@ export async function generateUI5YamlContent(options) {
  */
 export async function generateUI5LocalYamlContent(options) {
     const { templateData, neoappDestinations, messages, destination, firstNeoAppDestination, webappPath, strict = false } = options;
-    const config = await UI5Config.newInstance('', { validateSchema: false });
-    // Set metadata and type
-    config.setMetadata({
-        name: templateData.ui5Yaml?.name?.toLowerCase() || templateData.project.name?.toLowerCase() || 'app'
-    });
-    config.setType('application');
+    const config = await createBaseConfig(templateData);
     // Add framework section with SAPUI5 libraries (ui5-local.yaml specific)
     if (templateData.project.localUI5Version && templateData.ui5Yaml?.sapUiLibs) {
         const libraries = [...templateData.ui5Yaml.sapUiLibs];
@@ -165,9 +176,7 @@ export async function generateUI5LocalYamlContent(options) {
         config.addCustomMiddleware(previewMiddleware);
     }
     const yamlContent = config.toString();
-    // Parse and post-process
     const yamlJson = parse(yamlContent);
-    // Add specVersion at the top
     const orderedYaml = { specVersion: '4.0' };
     Object.keys(yamlJson).forEach((key) => {
         orderedYaml[key] = yamlJson[key];
@@ -192,12 +201,7 @@ export async function generateUI5LocalYamlContent(options) {
  * @returns Promise resolving to ui5-mock.yaml content as string
  */
 export async function generateUI5MockYamlContent(templateData, webappPath, setUI5Version) {
-    const config = await UI5Config.newInstance('', { validateSchema: false });
-    // Set metadata and type
-    config.setMetadata({
-        name: templateData.ui5Yaml?.name?.toLowerCase() || templateData.project.name?.toLowerCase() || 'app'
-    });
-    config.setType('application');
+    const config = await createBaseConfig(templateData);
     // Add fiori-tools-proxy middleware with UI5 version if provided
     const proxyConfig = {
         ignoreCertErrors: false,
@@ -214,19 +218,6 @@ export async function generateUI5MockYamlContent(templateData, webappPath, setUI
     config.addFioriToolsProxyMiddleware(proxyConfig);
     // Add fiori-tools-appreload middleware
     config.addFioriToolsAppReloadMiddleware();
-    const yamlContent = config.toString();
-    // Post-process: Add specVersion and webappPath
-    const yamlJson = parse(yamlContent);
-    // Add specVersion at the top
-    const orderedYaml = { specVersion: '4.0' };
-    Object.keys(yamlJson).forEach((key) => {
-        orderedYaml[key] = yamlJson[key];
-    });
-    // Apply webappPath if provided
-    if (webappPath) {
-        setWebappPath(orderedYaml, webappPath);
-        setAppreloadPath(orderedYaml, webappPath);
-    }
-    return stringify(orderedYaml);
+    return postProcessYaml(config.toString(), webappPath);
 }
 //# sourceMappingURL=ui5-config-adapter.js.map

package/dist/index.d.ts

@@ -45,12 +45,12 @@ export { BulkProjectMigrator } from './BulkProjectMigrator.js';
 export { createMemFsEditor } from './utils/fs-adapter.js';
 export * from './types.js';
 export * from './i18n.js';
-export { findProjectRoot, getProjectType, getMainService, hasDependency, getMinUI5VersionAsArray, type CdsUi5PluginInfo, readFile, readJSON, fileExists, writeFile, updateFile, updateJSON, deleteFile, findAllManifest, isFioriToolsProject, determineMessage, createDirectory } from './utils/index.js';
+export { findProjectRoot, getProjectType, getMainService, hasDependency, getMinUI5VersionAsArray, type CdsUi5PluginInfo, readFile, readJSON, fileExists, writeFile, updateFile, updateJSON, deleteFile, findAllManifest, isFioriToolsProject, determineMessage } from './utils/index.js';
 export { buildMainBackend, buildProxyConfig, buildPreviewMiddleware } from './adapters/ui5-config-helpers.js';
 export { generateUI5YamlContent, generateUI5LocalYamlContent, generateUI5MockYamlContent } from './adapters/ui5-config-adapter.js';
 export type { UI5YamlOptions } from './adapters/ui5-config-adapter.js';
 export { readManifest, getUI5Version } from './utils/manifest-and-version-utils.js';
-export { doesPropertyExist, stripSpaces, escapeSingleQuotes, escapeDoubleQuotes, doesDirectoryExists } from './utils/file-system-utils.js';
+export { stripSpaces, escapeSingleQuotes, escapeDoubleQuotes, doesDirectoryExists } from './utils/file-system-utils.js';
 export { generateSapLibsStr, getSourceTemplate } from './utils/migration-utils.js';
 export { getWebIDEProjectPathsAsLabels } from './utils/project-discovery.js';
 export { handleGitIgnoreFile, handlePackageJsonFile, handleLocateReuseLibsFile } from './utils/template-generators/index.js';

package/dist/index.js

@@ -56,13 +56,13 @@ export * from './i18n.js';
 // Public API - Utilities (re-export selective functions for backward compatibility)
 export { findProjectRoot, getProjectType, getMainService, hasDependency, getMinUI5VersionAsArray, readFile, readJSON, fileExists, writeFile, updateFile, updateJSON, deleteFile, findAllManifest, isFioriToolsProject, 
 // Additional exports for test compatibility
-determineMessage, createDirectory } from './utils/index.js';
+determineMessage } from './utils/index.js';
 // Additional exports from adapters for test compatibility
 export { buildMainBackend, buildProxyConfig, buildPreviewMiddleware } from './adapters/ui5-config-helpers.js';
 export { generateUI5YamlContent, generateUI5LocalYamlContent, generateUI5MockYamlContent } from './adapters/ui5-config-adapter.js';
 // Additional exports from utils for test compatibility
 export { readManifest, getUI5Version } from './utils/manifest-and-version-utils.js';
-export { doesPropertyExist, stripSpaces, escapeSingleQuotes, escapeDoubleQuotes, doesDirectoryExists } from './utils/file-system-utils.js';
+export { stripSpaces, escapeSingleQuotes, escapeDoubleQuotes, doesDirectoryExists } from './utils/file-system-utils.js';
 export { generateSapLibsStr, getSourceTemplate } from './utils/migration-utils.js';
 export { getWebIDEProjectPathsAsLabels } from './utils/project-discovery.js';
 export { handleGitIgnoreFile, handlePackageJsonFile, handleLocateReuseLibsFile } from './utils/template-generators/index.js';

package/dist/migration-process/legacy-helpers.js

@@ -31,81 +31,48 @@ export function buildLegacyPaths(rootPath, legacyPath) {
  * @param rootPath - Project root path
  * @param paths - Legacy paths object
  */
+/**
+ * Move all mem-fs files rooted at `fromPath` to be rooted at `toPath`.
+ * Iterates the store after collecting to avoid modifying during iteration.
+ *
+ * mem-fs keys use the OS-native separator, so comparisons use `sep`.
+ *
+ * @param fs
+ * @param fromPath
+ * @param toPath
+ */
+function moveMemFsDirectory(fs, fromPath, toPath) {
+    const filesToMove = [];
+    fs.store.each((file) => {
+        const filePath = file.path;
+        if (filePath === fromPath || filePath.startsWith(fromPath + sep)) {
+            const relativePath = filePath === fromPath ? '' : filePath.substring(fromPath.length + 1);
+            const newPath = relativePath ? join(toPath, relativePath) : toPath;
+            filesToMove.push({ oldPath: filePath, newPath });
+        }
+    });
+    for (const { oldPath, newPath } of filesToMove) {
+        const fileContent = fs.read(oldPath);
+        fs.write(newPath, fileContent);
+        fs.delete(oldPath);
+    }
+}
 async function memFsMove(rootPath, paths) {
-    const fs = getCurrentEditor();
-    if (!fs || !hasStore(fs)) {
+    const editor = getCurrentEditor();
+    if (!editor || !hasStore(editor)) {
         return;
     }
     // Move webapp files from src/main/webapp to webapp
     if (await exists(paths.ffLegacyWebappPath)) {
-        // Iterate through all files in mem-fs store
-        // mem-fs stores files with absolute paths as keys
-        const filesToMove = [];
-        fs.store.each((file) => {
-            const filePath = file.path;
-            // Check if this file is under the legacy webapp path.
-            // mem-fs keys use the OS-native separator, so compare with `sep` (not a hardcoded '/')
-            // or the match fails on Windows and no files are moved.
-            if (filePath === paths.ffLegacyWebappPath || filePath.startsWith(paths.ffLegacyWebappPath + sep)) {
-                // Calculate relative path from legacy webapp
-                const relativePath = filePath === paths.ffLegacyWebappPath
-                    ? ''
-                    : filePath.substring(paths.ffLegacyWebappPath.length + 1);
-                const newPath = relativePath
-                    ? join(rootPath, DirName.Webapp, relativePath)
-                    : join(rootPath, DirName.Webapp);
-                filesToMove.push({ oldPath: filePath, newPath });
-            }
-        });
-        // Move files (do this after iteration to avoid modifying during iteration)
-        for (const { oldPath, newPath } of filesToMove) {
-            const fileContent = fs.read(oldPath);
-            fs.write(newPath, fileContent);
-            fs.delete(oldPath);
-        }
+        moveMemFsDirectory(editor, paths.ffLegacyWebappPath, join(rootPath, DirName.Webapp));
     }
-    // Move test/qunit to webapp/test
+    // Move test/qunit to webapp/test/qunit
     if (await exists(paths.ffLegacyTestQunitPath)) {
-        const filesToMove = [];
-        fs.store.each((file) => {
-            const filePath = file.path;
-            if (filePath === paths.ffLegacyTestQunitPath || filePath.startsWith(paths.ffLegacyTestQunitPath + sep)) {
-                const relativePath = filePath === paths.ffLegacyTestQunitPath
-                    ? ''
-                    : filePath.substring(paths.ffLegacyTestQunitPath.length + 1);
-                const newPath = relativePath
-                    ? join(paths.ffNewTestPath, 'qunit', relativePath)
-                    : join(paths.ffNewTestPath, 'qunit');
-                filesToMove.push({ oldPath: filePath, newPath });
-            }
-        });
-        for (const { oldPath, newPath } of filesToMove) {
-            const fileContent = fs.read(oldPath);
-            fs.write(newPath, fileContent);
-            fs.delete(oldPath);
-        }
+        moveMemFsDirectory(editor, paths.ffLegacyTestQunitPath, join(paths.ffNewTestPath, 'qunit'));
     }
-    // Move test/uiveri5 to webapp/test
+    // Move test/uiveri5 to webapp/test/uiveri5
     if (await exists(paths.ffLegacyTestuiveri5Path)) {
-        const filesToMove = [];
-        fs.store.each((file) => {
-            const filePath = file.path;
-            if (filePath === paths.ffLegacyTestuiveri5Path ||
-                filePath.startsWith(paths.ffLegacyTestuiveri5Path + sep)) {
-                const relativePath = filePath === paths.ffLegacyTestuiveri5Path
-                    ? ''
-                    : filePath.substring(paths.ffLegacyTestuiveri5Path.length + 1);
-                const newPath = relativePath
-                    ? join(paths.ffNewTestPath, 'uiveri5', relativePath)
-                    : join(paths.ffNewTestPath, 'uiveri5');
-                filesToMove.push({ oldPath: filePath, newPath });
-            }
-        });
-        for (const { oldPath, newPath } of filesToMove) {
-            const fileContent = fs.read(oldPath);
-            fs.write(newPath, fileContent);
-            fs.delete(oldPath);
-        }
+        moveMemFsDirectory(editor, paths.ffLegacyTestuiveri5Path, join(paths.ffNewTestPath, 'uiveri5'));
     }
 }
 /**

package/dist/migration-process/setup.d.ts

@@ -1,13 +1,13 @@
 import type { ImportProjectInfo } from '../types.js';
 /**
  * Sets up necessary directories for migration
- * Creates localService directory if data source exists, and test directory if webapp exists
+ * Note: mem-fs handles directory creation implicitly when files are written
  *
- * @param projectInfo - Project information
- * @param rootPath - Root path of the project
- * @param hasDataSource - Whether project has a data source
+ * @param _projectInfo - Project information (unused - kept for API compatibility)
+ * @param _rootPath - Root path of the project (unused - kept for API compatibility)
+ * @param _hasDataSource - Whether project has a data source (unused - kept for API compatibility)
  */
-export declare function setupMigrationDirectories(projectInfo: ImportProjectInfo, rootPath: string, hasDataSource: boolean): Promise<void>;
+export declare function setupMigrationDirectories(_projectInfo: ImportProjectInfo, _rootPath: string, _hasDataSource: boolean): Promise<void>;
 /**
  * Removes package-lock.json if it exists
  * Package-lock is regenerated during installation

package/dist/migration-process/setup.js

@@ -1,25 +1,16 @@
 import { join } from 'node:path';
-import { DirName } from '../project-spec-types.js';
-import { TemplateFileName } from '../index.js';
-import { createDirectory, fileExists, deleteFile } from '../utils/index.js';
+import { fileExists, deleteFile } from '../utils/index.js';
 /**
  * Sets up necessary directories for migration
- * Creates localService directory if data source exists, and test directory if webapp exists
+ * Note: mem-fs handles directory creation implicitly when files are written
  *
- * @param projectInfo - Project information
- * @param rootPath - Root path of the project
- * @param hasDataSource - Whether project has a data source
+ * @param _projectInfo - Project information (unused - kept for API compatibility)
+ * @param _rootPath - Root path of the project (unused - kept for API compatibility)
+ * @param _hasDataSource - Whether project has a data source (unused - kept for API compatibility)
  */
-export async function setupMigrationDirectories(projectInfo, rootPath, hasDataSource) {
-    // Create localService directory if project has a data source
-    if (hasDataSource) {
-        createDirectory(join(rootPath, projectInfo.webappPath, DirName.LocalService));
-    }
-    // Create test directory if webapp exists
-    const webAppPath = join(rootPath, projectInfo.webappPath);
-    if (await fileExists(webAppPath)) {
-        createDirectory(join(rootPath, projectInfo.webappPath, TemplateFileName.Test));
-    }
+export async function setupMigrationDirectories(_projectInfo, _rootPath, _hasDataSource) {
+    // mem-fs handles directory creation implicitly when files are written to paths
+    // No explicit directory creation needed
 }
 /**
  * Removes package-lock.json if it exists

package/dist/template/details.js

@@ -3,7 +3,7 @@
  */
 import { join } from 'node:path';
 import { TemplateFileName } from '../index.js';
-import { fileExists, readFile } from '../utils/index.js';
+import { readFile, fileExists } from '../utils/index.js';
 import { xmlToJson } from '../utils/service.js';
 import { isAppFreestyle } from '../utils/common.js';
 /**

package/dist/utils/common.d.ts

@@ -11,7 +11,7 @@ export declare const v4MockServerDep: {
 };
 export { packageJsonTmplName, propertyEditorTaskProjectTypes, getBaseTemplatesMap, getFLPSandboxTemplatesMap, getFLPSandboxMockServerTemplatesMap, getCommonV4TemplatesMap, getSapTemplatesMap, getFeTemplatesMap, getAdaptationTemplatesMap, getLibraryTemplatesMap, getFFTestSuiteMap } from './template-maps.js';
 export { findAllWebIDEProjectFolders, getWebIDEProjectPathsAsLabels, findProjectsByManifest } from './project-discovery.js';
-export { stripSpaces, escapeSingleQuotes, escapeDoubleQuotes, doesDirectoryExists, doesPropertyExist, createDirectory } from './file-system-utils.js';
+export { normalizePath, stripSpaces, escapeSingleQuotes, escapeDoubleQuotes, doesDirectoryExists } from './file-system-utils.js';
 export { getUI5Version, checkManifestUI5Version, readManifest, isAppFreestyle, isGenerateIndex } from './manifest-and-version-utils.js';
 export { generateSapLibsStr, MigrationError, determineMessage, generateTemplate, getSourceTemplate, generateToolsId, determineSapUxLayer, buildUrlParam } from './migration-utils.js';
 //# sourceMappingURL=common.d.ts.map

package/dist/utils/common.js

@@ -19,7 +19,7 @@ export { packageJsonTmplName, propertyEditorTaskProjectTypes, getBaseTemplatesMa
 // Re-export from project-discovery
 export { findAllWebIDEProjectFolders, getWebIDEProjectPathsAsLabels, findProjectsByManifest } from './project-discovery.js';
 // Re-export from file-system-utils
-export { stripSpaces, escapeSingleQuotes, escapeDoubleQuotes, doesDirectoryExists, doesPropertyExist, createDirectory } from './file-system-utils.js';
+export { normalizePath, stripSpaces, escapeSingleQuotes, escapeDoubleQuotes, doesDirectoryExists } from './file-system-utils.js';
 // Re-export from manifest-and-version-utils
 export { getUI5Version, checkManifestUI5Version, readManifest, isAppFreestyle, isGenerateIndex } from './manifest-and-version-utils.js';
 // Re-export from migration-utils

package/dist/utils/file-access.d.ts

@@ -12,7 +12,7 @@ export declare function readFile(pathOrFs: string | Editor, path?: string): stri
  *
  * @param pathOrFs - Path to file, or Editor instance
  * @param path - Path to file (if first param is Editor)
- * @returns Parsed JSON object with indentation metadata for round-trip preservation
+ * @returns Parsed JSON object
  */
 export declare function readJSON<T = any>(pathOrFs: string | Editor, path?: string): Promise<T>;
 /**

package/dist/utils/file-access.js

@@ -7,7 +7,7 @@
  */
 // @ts-expect-error - no type definitions available
 import parseJson from 'json-parse-even-better-errors';
-import { readFile as fsReadFile } from 'node:fs/promises';
+import { readFile as fsReadFile, access } from 'node:fs/promises';
 import { editorHasPath, getCurrentEditor, exists as fsAdapterExists, getEditor } from './fs-adapter.js';
 /**
  * Read a text file
@@ -26,7 +26,7 @@ export function readFile(pathOrFs, path) {
  *
  * @param pathOrFs - Path to file, or Editor instance
  * @param path - Path to file (if first param is Editor)
- * @returns Parsed JSON object with indentation metadata for round-trip preservation
+ * @returns Parsed JSON object
  */
 export async function readJSON(pathOrFs, path) {
     const filePath = typeof pathOrFs === 'string' ? pathOrFs : path;
@@ -40,23 +40,7 @@ export async function readJSON(pathOrFs, path) {
         const fs = getEditor(pathOrFs);
         content = fs.read(filePath);
     }
-    // Parse with JSON.parse for consistent SyntaxError behavior
-    const result = JSON.parse(content);
-    // Also parse with parseJson to extract indentation metadata
-    // This is needed for updateJSON to preserve formatting
-    try {
-        const resultWithIndent = parseJson(content);
-        // Copy indent metadata to the result object
-        const indent = Symbol.for('indent');
-        if (resultWithIndent[indent]) {
-            result[indent] = resultWithIndent[indent];
-        }
-    }
-    catch (error) {
-        // If parseJson fails, we still have the result from JSON.parse
-        // updateJSON will use default 4-space indentation
-    }
-    return result;
+    return JSON.parse(content);
 }
 /**
  * Check if a file exists
@@ -66,15 +50,24 @@ export async function readJSON(pathOrFs, path) {
  * @returns true if file exists, false otherwise
  */
 export async function fileExists(pathOrFs, path) {
-    const editor = getCurrentEditor();
-    const filePath = typeof pathOrFs === 'string' ? pathOrFs : path;
-    // When called with just a path string and no editor context, check filesystem directly
-    if (typeof pathOrFs === 'string' && !editor) {
+    // String path: delegate to the single shared existence check (mem-fs staged OR real fs).
+    if (typeof pathOrFs === 'string') {
         return fsAdapterExists(pathOrFs);
     }
-    // Otherwise use mem-fs editor
-    const fs = getEditor(pathOrFs);
-    return editorHasPath(fs, filePath);
+    // Explicit editor argument: check that editor's staged files, then fall back to real fs.
+    // The real-fs fallback prevents overwriting files not yet loaded into the given editor
+    // (e.g., the mockserver.js overwrite bug).
+    const filePath = path;
+    if (editorHasPath(pathOrFs, filePath)) {
+        return true;
+    }
+    try {
+        await access(filePath);
+        return true;
+    }
+    catch {
+        return false;
+    }
 }
 /**
  * Write a text file

package/dist/utils/file-discovery.js

@@ -8,6 +8,7 @@ import fastGlob from 'fast-glob';
 import { dirname, basename, join, normalize } from 'node:path';
 import { access } from 'node:fs/promises';
 import { readJSON } from './file-access.js';
+import { normalizePath } from './file-system-utils.js';
 /**
  * Reuse library type enum
  * Matches @sap/ux-project-access ReuseLibType
@@ -92,8 +93,8 @@ export async function findAllProjectRoots(paths, sapuxRequired = false) {
 async function findLibraryProjectRoot(manifestPath, workspaceBoundary) {
     let current = dirname(manifestPath);
     // Normalize paths for comparison
-    const normalizedBoundary = workspaceBoundary.replaceAll('\\', '/');
-    const normalizedCurrent = (path) => path.replaceAll('\\', '/');
+    const normalizedBoundary = normalizePath(workspaceBoundary);
+    const normalizedCurrent = (path) => normalizePath(path);
     // Helper to check if a file/directory exists using Node.js fs API
     // (not mem-fs, since project markers are real filesystem entries)
     const pathExists = async (path) => {

package/dist/utils/file-system-utils.d.ts

@@ -1,4 +1,12 @@
 import type { Editor } from 'mem-fs-editor';
+/**
+ * Normalize path separators to forward slashes for cross-platform compatibility
+ * mem-fs and many tools expect forward slashes regardless of OS
+ *
+ * @param path - Path to normalize
+ * @returns Path with forward slashes
+ */
+export declare const normalizePath: (path: string) => string;
 /**
  * stripSpaces
  *
@@ -25,19 +33,4 @@ export declare const escapeDoubleQuotes: (s: string) => string;
  * @param directory - Directory path (if first param is Editor)
  */
 export declare function doesDirectoryExists(directoryOrFs: string | Editor, directory?: string): boolean;
-/**
- * Check if property exists on object
- *
- * @param obj
- * @param fieldName
- */
-export declare function doesPropertyExist(obj: unknown, fieldName: string): boolean;
-/**
- * Create directory if it doesn't exist
- * Note: mem-fs handles directories implicitly, so this is a no-op
- *
- * @param directoryOrFs - Directory path or Editor instance
- * @param directory - Directory path (if first param is Editor)
- */
-export declare function createDirectory(directoryOrFs: string | Editor, directory?: string): boolean;
 //# sourceMappingURL=file-system-utils.d.ts.map

package/dist/utils/file-system-utils.js

@@ -1,4 +1,12 @@
 import { editorHasPath, getEditor } from './fs-adapter.js';
+/**
+ * Normalize path separators to forward slashes for cross-platform compatibility
+ * mem-fs and many tools expect forward slashes regardless of OS
+ *
+ * @param path - Path to normalize
+ * @returns Path with forward slashes
+ */
+export const normalizePath = (path) => path.replaceAll('\\', '/');
 /**
  * stripSpaces
  *
@@ -30,26 +38,4 @@ export function doesDirectoryExists(directoryOrFs, directory) {
     // Use editorHasPath instead of fs.exists() to avoid side effect of staging paths
     return editorHasPath(fs, dir);
 }
-/**
- * Check if property exists on object
- *
- * @param obj
- * @param fieldName
- */
-export function doesPropertyExist(obj, fieldName) {
-    return Object.hasOwn(obj, fieldName);
-}
-/**
- * Create directory if it doesn't exist
- * Note: mem-fs handles directories implicitly, so this is a no-op
- *
- * @param directoryOrFs - Directory path or Editor instance
- * @param directory - Directory path (if first param is Editor)
- */
-export function createDirectory(directoryOrFs, directory) {
-    // mem-fs handles directories implicitly when writing files
-    // Check if it exists for consistency
-    const exists = doesDirectoryExists(directoryOrFs, directory);
-    return !exists; // Return true if it didn't exist (was "created")
-}
 //# sourceMappingURL=file-system-utils.js.map

package/dist/utils/fs-adapter.d.ts

@@ -28,6 +28,17 @@ export declare function createMemFsEditor(): Editor;
  * @returns Result of the callback
  */
 export declare function runWithEditor<T>(editor: Editor, callback: () => T | Promise<T>): T | Promise<T>;
+/**
+ * Load every file under a project folder into the mem-fs store.
+ *
+ * Pulling files into the store makes existence checks that scan the store (editorHasPath)
+ * see the real project tree, so later writes do not overwrite untracked files. Loading via
+ * the store's get() does not mark files as modified, so commit() still skips them.
+ *
+ * @param editor - Mem-fs editor whose store should be populated
+ * @param projectRoot - Absolute path to the project folder to load
+ */
+export declare function loadProjectIntoEditor(editor: Editor, projectRoot: string): Promise<void>;
 /**
  * Get the current editor from context
  *
@@ -51,11 +62,14 @@ export declare function isMemFsEnabled(): boolean;
  */
 export declare function editorHasPath(editor: Editor, path: string): boolean;
 /**
- * Check if file/directory exists (async)
- * Uses editor from context if available, otherwise checks real filesystem
+ * Check if a file/directory exists (async).
+ *
+ * When an editor is in context, the migration tree is already loaded into mem-fs, so a staged
+ * path is authoritative. A real-filesystem check is still used as a fallback so files that were
+ * never loaded into mem-fs are not reported missing (and therefore not silently overwritten).
  *
  * @param path - Path to check
- * @returns True if exists in mem-fs (when editor available) OR on real filesystem
+ * @returns True if staged in mem-fs OR present on the real filesystem
  */
 export declare function exists(path: string): Promise<boolean>;
 /**

package/dist/utils/fs-adapter.js

@@ -2,6 +2,8 @@ import { create as createMemFs } from 'mem-fs';
 import { create as createEditor } from 'mem-fs-editor';
 import { access } from 'node:fs/promises';
 import { AsyncLocalStorage } from 'node:async_hooks';
+import fastGlob from 'fast-glob';
+import { normalizePath } from './file-system-utils.js';
 /**
  * AsyncLocalStorage for tracking the current migration's editor
  */
@@ -42,6 +44,32 @@ export function createMemFsEditor() {
 export function runWithEditor(editor, callback) {
     return editorContext.run(editor, callback);
 }
+/**
+ * Load every file under a project folder into the mem-fs store.
+ *
+ * Pulling files into the store makes existence checks that scan the store (editorHasPath)
+ * see the real project tree, so later writes do not overwrite untracked files. Loading via
+ * the store's get() does not mark files as modified, so commit() still skips them.
+ *
+ * @param editor - Mem-fs editor whose store should be populated
+ * @param projectRoot - Absolute path to the project folder to load
+ */
+export async function loadProjectIntoEditor(editor, projectRoot) {
+    const files = await fastGlob('**/*', {
+        cwd: projectRoot,
+        ignore: ['**/node_modules/**', '**/dist/**', '**/.git/**'],
+        absolute: true,
+        onlyFiles: true,
+        dot: true
+    });
+    const store = editor.store;
+    for (const file of files) {
+        // get() lazily reads the file into the store without marking it modified.
+        // The store resolves and keys the path itself; editorHasPath normalizes separators
+        // at comparison time, so both sides stay consistent cross-platform.
+        store.get(file);
+    }
+}
 /**
  * Get the current editor from context
  *
@@ -70,7 +98,7 @@ export function isMemFsEnabled() {
 export function editorHasPath(editor, path) {
     // Normalize path separators to forward slashes for consistent comparison
     // mem-fs stores all paths with forward slashes regardless of OS
-    const normalizedPath = path.replaceAll('\\', '/');
+    const normalizedPath = normalizePath(path);
     const directoryPrefix = normalizedPath.endsWith('/') ? normalizedPath : normalizedPath + '/';
     let found = false;
     editor.store.each((file) => {
@@ -78,7 +106,7 @@ export function editorHasPath(editor, path) {
             return;
         }
         // Normalize file path as well for comparison
-        const normalizedFilePath = file.path.replaceAll('\\', '/');
+        const normalizedFilePath = normalizePath(file.path);
         // Check exact match OR files under this directory
         if (normalizedFilePath === normalizedPath || normalizedFilePath.startsWith(directoryPrefix)) {
             found = true;
@@ -87,19 +115,20 @@ export function editorHasPath(editor, path) {
     return found;
 }
 /**
- * Check if file/directory exists (async)
- * Uses editor from context if available, otherwise checks real filesystem
+ * Check if a file/directory exists (async).
+ *
+ * When an editor is in context, the migration tree is already loaded into mem-fs, so a staged
+ * path is authoritative. A real-filesystem check is still used as a fallback so files that were
+ * never loaded into mem-fs are not reported missing (and therefore not silently overwritten).
  *
  * @param path - Path to check
- * @returns True if exists in mem-fs (when editor available) OR on real filesystem
+ * @returns True if staged in mem-fs OR present on the real filesystem
  */
 export async function exists(path) {
     const editor = getCurrentEditor();
-    if (editor) {
-        // In mem-fs mode, check mem-fs only
-        return editorHasPath(editor, path);
+    if (editor && editorHasPath(editor, path)) {
+        return true;
     }
-    // No editor context - check real filesystem
     try {
         await access(path);
         return true;

package/dist/utils/index.d.ts

@@ -2,7 +2,6 @@ export * from './common.js';
 export * from './UI5Theme.js';
 export { validateMetadata } from './service.js';
 export { determineMessage } from './migration-utils.js';
-export { createDirectory } from './file-system-utils.js';
 export { readFile, readJSON, fileExists, writeFile, updateFile, updateJSON, deleteFile } from './file-access.js';
 export { findAllProjectRoots, getReuseLibs, ReuseLibType } from './file-discovery.js';
 export { findProjectRoot, getProjectType, getMainService, hasDependency, getMinUI5VersionAsArray, type CdsUi5PluginInfo } from '@sap-ux/project-access';

package/dist/utils/index.js

@@ -8,8 +8,6 @@ export * from './UI5Theme.js';
 export { validateMetadata } from './service.js';
 // Migration utilities
 export { determineMessage } from './migration-utils.js';
-// File system utilities
-export { createDirectory } from './file-system-utils.js';
 // Re-export native file access to replace @sap/ux-project-access file I/O
 export { readFile, readJSON, fileExists, writeFile, updateFile, updateJSON, deleteFile } from './file-access.js';
 // Re-export native file discovery to replace @sap/ux-project-access

package/dist/utils/migration-utils.js

@@ -4,7 +4,7 @@ import { readPackageUpAsync } from './read-package-up-compat.js';
 import { v4 as uuidV4 } from 'uuid';
 import { i18nText } from '../i18n.js';
 import { writeFile } from './file-access.js';
-import { createDirectory } from './file-system-utils.js';
+import uniq from 'lodash/uniq.js';
 // Get current directory using import.meta.url (ESM)
 const __dirname = dirname(fileURLToPath(import.meta.url));
 // Matches strings starting with "sap" or containing ".sap" (e.g., "sap.m", "com.sap.ui")
@@ -17,14 +17,12 @@ const sapLibsRegexFilter = /(?:^sap|\.sap)/;
  * @param libsStrInput
  */
 export function generateSapLibsStr(manifestLibs, libsStrInput) {
-    let libStrOut = '';
     const mlibs = Object.keys(manifestLibs || {});
     // remove spaces and split at comma
     const strlibs = libsStrInput?.replaceAll(' ', '').split(',') || [];
     const libs = mlibs.concat(strlibs);
-    const uniquelibs = libs.filter((v, i, a) => a.indexOf(v) === i);
-    libStrOut = uniquelibs.filter((ele) => sapLibsRegexFilter.test(ele)).join(', ');
-    return libStrOut;
+    const uniquelibs = uniq(libs);
+    return uniquelibs.filter((ele) => sapLibsRegexFilter.test(ele)).join(', ');
 }
 /**
  * Custom error class for migration errors
@@ -82,14 +80,12 @@ export async function generateTemplate(templateData, projectRoot, templateRoot,
     const { fullTemplateName, targetFile } = resolveTemplatePaths(projectRoot, templateRoot, templateName, templateProps);
     // 2. Resolve template data
     const resolvedData = resolveTemplateData(templateData, templateProps);
-    // 3. Ensure target directory exists (uses fs-adapter)
-    createDirectory(dirname(targetFile));
-    // 4. Render template
+    // 3. Render template (mem-fs handles directory creation implicitly)
     const { isRendered = true, opts = {} } = templateProps;
     let content = renderTemplate(templateRoot, fullTemplateName, resolvedData, opts, isRendered);
-    // 5. Apply file-specific handlers
+    // 4. Apply file-specific handlers
     content = await applyFileSpecificHandlers(templateName, content, projectRoot, targetFile, templateData);
-    // 6. Write file (uses fs-adapter)
+    // 5. Write file (mem-fs handles directory creation implicitly)
     // Do not catch - let errors propagate to caller for proper handling
     writeFile(targetFile, content);
 }

package/package.json

@@ -10,7 +10,7 @@
   "bugs": {
     "url": "https://github.com/SAP/open-ux-tools/issues?q=is%3Aopen+is%3Aissue+label%3Abug+label%3Afiori-migration-writer"
   },
-  "version": "0.0.2",
+  "version": "0.0.3",
   "license": "Apache-2.0",
   "main": "dist/index.js",
   "files": [
@@ -25,7 +25,7 @@
     "ejs": "3.1.10",
     "fast-glob": "3.3.3",
     "fast-xml-parser": "5.10.1",
-    "fs-extra": "11.3.6",
+    "fs-extra": "11.4.1",
     "i18next": "26.3.6",
     "json-parse-even-better-errors": "5.0.0",
     "lodash": "4.18.1",
@@ -37,17 +37,17 @@
     "uuid": "11.1.1",
     "vscode-uri": "3.1.0",
     "yaml": "2.9.0",
-    "@sap-ux/adp-tooling": "1.0.64",
-    "@sap-ux/axios-extension": "2.1.1",
-    "@sap-ux/i18n": "1.0.5",
-    "@sap-ux/btp-utils": "2.0.6",
-    "@sap-ux/nodejs-utils": "1.0.10",
-    "@sap-ux/launch-config": "1.0.24",
-    "@sap-ux/project-access": "2.3.0",
+    "@sap-ux/axios-extension": "2.1.2",
+    "@sap-ux/btp-utils": "2.0.7",
+    "@sap-ux/i18n": "1.0.6",
+    "@sap-ux/nodejs-utils": "1.0.11",
+    "@sap-ux/adp-tooling": "1.0.65",
+    "@sap-ux/project-access": "2.3.1",
     "@sap-ux/store": "2.0.7",
-    "@sap-ux/ui5-application-writer": "2.1.5",
-    "@sap-ux/ui5-config": "1.0.9",
-    "@sap-ux/ui5-info": "1.0.6"
+    "@sap-ux/ui5-application-writer": "2.1.6",
+    "@sap-ux/ui5-config": "1.0.10",
+    "@sap-ux/ui5-info": "1.0.7",
+    "@sap-ux/launch-config": "1.0.25"
   },
   "devDependencies": {
     "@jest/globals": "30.4.1",
@@ -56,10 +56,10 @@
     "@types/lodash": "4.17.24",
     "@types/mem-fs": "1.1.2",
     "@types/mem-fs-editor": "7.0.1",
-    "@types/semver": "7.7.1",
+    "@types/semver": "7.8.0",
     "jest-specific-snapshot": "^8.0.0",
     "rimraf": "6.1.3",
-    "@sap-ux/eslint-plugin-fiori-tools": "10.17.1"
+    "@sap-ux/eslint-plugin-fiori-tools": "10.17.2"
   },
   "engines": {
     "node": ">=22.x"