Revise launch config

.changeset/tidy-papayas-film.md

@@ -0,0 +1,6 @@
+---
+'@sap-ux/launch-config': minor
+---
+
+Reverted the use of Node.js `fs` modules and replaced them with `mem-fs` for writing launch config files & Removed `writeApplicationInfoSettings()` from `@sap-ux/launch-config`
+Refactoring create launch config functionalities.

packages/launch-config/package.json

@@ -34,8 +34,6 @@
         "@sap-ux/project-access": "workspace:*",
         "@sap-ux/ui5-config": "workspace:*",
         "@sap-ux/ui5-info": "workspace:*",
-        "@sap-ux/odata-service-inquirer": "workspace:*",
-        "@sap-ux/store": "workspace:*",
         "i18next": "23.5.1",
         "jsonc-parser": "3.2.0",
         "mem-fs": "2.1.0",

packages/launch-config/src/debug-config/config.ts

@@ -1,8 +1,7 @@
-import { DatasourceType, OdataVersion } from '@sap-ux/odata-service-inquirer';
 import { basename } from 'path';
 import { getLaunchConfig } from '../launch-config-crud/utils';
 import type { LaunchConfig, LaunchJSON, DebugOptions, LaunchConfigEnv } from '../types';
-import { FIORI_TOOLS_LAUNCH_CONFIG_HANDLER_ID } from '../types';
+import { FIORI_TOOLS_LAUNCH_CONFIG_HANDLER_ID, ProjectDataSourceType, oDataVersionV2, oDataVersionV4 } from '../types';
 
 // debug constants
 const testFlpSandboxHtml = 'test/flpSandbox.html';
@@ -56,13 +55,13 @@ function createLaunchConfig(
 /**
  * Configures the launch.json file based on provided options.
  *
+ * @param rootFolder
  * @param {string} cwd - The current working directory.
  * @param {DebugOptions} configOpts - Configuration options for the launch.json file.
  * @returns {LaunchJSON} The configured launch.json object.
  */
-export function configureLaunchJsonFile(cwd: string, configOpts: DebugOptions): LaunchJSON {
+export function configureLaunchJsonFile(rootFolder: string, cwd: string, configOpts: DebugOptions): LaunchJSON {
     const {
-        projectPath,
         isAppStudio,
         datasourceType,
         flpAppId,
@@ -73,22 +72,21 @@ export function configureLaunchJsonFile(cwd: string, configOpts: DebugOptions):
         isFioriElement,
         migratorMockIntent
     } = configOpts;
-
-    const projectName = basename(projectPath);
+    const projectName = basename(rootFolder);
     const flpAppIdWithHash = flpAppId && !flpAppId.startsWith('#') ? `#${flpAppId}` : flpAppId;
     const startHtmlFile = flpSandboxAvailable ? testFlpSandboxHtml : indexHtml;
     const runConfig = isAppStudio
         ? JSON.stringify({
               handlerId: FIORI_TOOLS_LAUNCH_CONFIG_HANDLER_ID,
-              runnableId: projectPath
+              runnableId: rootFolder
           })
         : undefined;
     const envUrlParam = getEnvUrlParams(sapClientParam);
 
     const launchFile: LaunchJSON = { version: '0.2.0', configurations: [] };
 
     // Add live configuration if the datasource is not from a metadata file
-    if (datasourceType !== DatasourceType.metadataFile) {
+    if (datasourceType !== ProjectDataSourceType.metadataFile) {
         const startCommand = `${startHtmlFile}${flpAppIdWithHash}`;
         const liveConfig = createLaunchConfig(
             `Start ${projectName}`,
@@ -102,10 +100,10 @@ export function configureLaunchJsonFile(cwd: string, configOpts: DebugOptions):
     }
 
     // Add mock configuration for OData V2 or V4
-    if (odataVersion && [OdataVersion.v2, OdataVersion.v4].includes(odataVersion)) {
+    if (odataVersion && [oDataVersionV2, oDataVersionV4].includes(odataVersion)) {
         const params = `${flpAppIdWithHash ?? ''}`;
         const mockCmdArgs =
-            isMigrator && odataVersion === OdataVersion.v2
+            isMigrator && odataVersion === oDataVersionV2
                 ? ['--open', `${testFlpSandboxMockServerHtml}${params}`]
                 : ['--config', './ui5-mock.yaml', '--open', `${testFlpSandboxHtml}${params}`];
         const mockConfig = createLaunchConfig(
@@ -120,7 +118,7 @@ export function configureLaunchJsonFile(cwd: string, configOpts: DebugOptions):
     }
 
     // Add local configuration
-    const shouldUseMockServer = isFioriElement && odataVersion === OdataVersion.v2 && isMigrator;
+    const shouldUseMockServer = isFioriElement && odataVersion === oDataVersionV2 && isMigrator;
     const localHtmlFile = shouldUseMockServer ? testFlpSandboxMockServerHtml : startHtmlFile;
     const startLocalCommand = `${localHtmlFile}${
         migratorMockIntent ? `#${migratorMockIntent.replace('#', '')}` : flpAppIdWithHash

packages/launch-config/src/debug-config/workspaceManager.ts

@@ -10,7 +10,7 @@ import { formatCwd, getLaunchJsonPath, isFolderInWorkspace, handleAppsNotInWorks
  * @param {any} vscode - The VS Code API object.
  * @returns {WorkspaceHandlerInfo} An object containing the path to the `launch.json` configuration file and the cwd for the launch configuration.
  */
-export function handleUnsavedWorkspace(projectPath: string, vscode: any): WorkspaceHandlerInfo {
+function handleUnsavedWorkspace(projectPath: string, vscode: any): WorkspaceHandlerInfo {
     const workspace = vscode.workspace;
     const wsFolder = workspace.getWorkspaceFolder(vscode.Uri.file(projectPath))?.uri?.fsPath;
     const nestedFolder = relative(wsFolder ?? projectPath, projectPath);
@@ -31,7 +31,7 @@ export function handleUnsavedWorkspace(projectPath: string, vscode: any): Worksp
  * @param {any} vscode - The VS Code API object.
  * @returns {WorkspaceHandlerInfo} An object containing the path to the `launch.json` configuration file and the cwd for the launch configuration.
  */
-export function handleSavedWorkspace(
+function handleSavedWorkspace(
     projectPath: string,
     projectName: string,
     targetFolder: string,
@@ -58,7 +58,7 @@ export function handleSavedWorkspace(
  * @param {any} vscode - The VS Code API object.
  * @returns {WorkspaceHandlerInfo} An object containing the path to the `launch.json` configuration file and the cwd for the launch configuration.
  */
-export function handleOpenFolderButNoWorkspaceFile(
+function handleOpenFolderButNoWorkspaceFile(
     projectPath: string,
     targetFolder: string,
     isAppStudio: boolean,
@@ -83,37 +83,38 @@ export function handleOpenFolderButNoWorkspaceFile(
  * This function handles different scenarios depending on whether a workspace is open,
  * whether the project is inside or outside of a workspace, and other factors.
  *
+ * @param rootFolder
  * @param {DebugOptions} options - The options used to determine how to manage the workspace configuration.
  * @param {string} options.projectPath -The project's path including project name.
  * @param {boolean} [options.isAppStudio] - A boolean indicating whether the current environment is BAS.
  * @param {boolean} [options.writeToAppOnly] - If true, write the launch configuration directly to the app folder, ignoring workspace settings.
  * @param {any} options.vscode - The VS Code API object.
  * @returns {WorkspaceHandlerInfo} An object containing the path to the `launch.json` configuration file, the cwd command, workspaceFolderUri if provided will enable reload.
  */
-export function handleWorkspaceConfig(options: DebugOptions): WorkspaceHandlerInfo {
-    const { projectPath, isAppStudio = false, writeToAppOnly = false, vscode } = options;
+export function handleWorkspaceConfig(rootFolder: string, options: DebugOptions): WorkspaceHandlerInfo {
+    const { isAppStudio = false, writeToAppOnly = false, vscode } = options;
 
-    const projectName = basename(projectPath);
-    const targetFolder = dirname(projectPath);
+    const projectName = basename(rootFolder);
+    const targetFolder = dirname(rootFolder);
 
     // Directly handle the case where we ignore workspace settings
     if (writeToAppOnly) {
-        return handleAppsNotInWorkspace(projectPath, isAppStudio, vscode);
+        return handleAppsNotInWorkspace(rootFolder, isAppStudio, vscode);
     }
     const workspace = vscode.workspace;
     const workspaceFile = workspace?.workspaceFile;
     // Handles the scenario where no workspace or folder is open in VS Code.
     if (!workspace) {
-        return handleAppsNotInWorkspace(projectPath, isAppStudio, vscode);
+        return handleAppsNotInWorkspace(rootFolder, isAppStudio, vscode);
     }
     // Handle case where a folder is open, but not a workspace file
     if (!workspaceFile) {
-        return handleOpenFolderButNoWorkspaceFile(projectPath, targetFolder, isAppStudio, vscode);
+        return handleOpenFolderButNoWorkspaceFile(rootFolder, targetFolder, isAppStudio, vscode);
     }
     // Handles the case where a previously saved workspace is open
     if (workspaceFile.scheme === 'file') {
-        return handleSavedWorkspace(projectPath, projectName, targetFolder, isAppStudio, vscode);
+        return handleSavedWorkspace(rootFolder, projectName, targetFolder, isAppStudio, vscode);
     }
     // Handles the case where an unsaved workspace is open
-    return handleUnsavedWorkspace(projectPath, vscode);
+    return handleUnsavedWorkspace(rootFolder, vscode);
 }

packages/launch-config/src/index.ts

@@ -1,5 +1,5 @@
 export * from './types';
-export { createLaunchConfig, configureLaunchConfig } from './launch-config-crud/create';
+export { createLaunchConfig } from './launch-config-crud/create';
 export { deleteLaunchConfig } from './launch-config-crud/delete';
 export { convertOldLaunchConfigToFioriRun } from './launch-config-crud/modify';
 export { getLaunchConfigs, getLaunchConfigByName } from './launch-config-crud/read';