@sfdc-webapps/cli 1.0.2

package/test/unit/paths.spec.ts

@@ -0,0 +1,247 @@
+import { describe, it, expect } from 'vitest';
+import {
+  translatePathToBaseApp,
+  translatePathToTargetApp
+} from '../../src/utils/paths.js';
+
+describe('paths', () => {
+  describe('translatePathToBaseApp', () => {
+    it('should translate full path format to base app', () => {
+      const featurePath = 'digitalExperiences/webApplications/feature-simple/src/routes.tsx';
+      const featureName = 'feature-simple';
+      const baseAppPath = '/path/to/base-app';
+
+      const result = translatePathToBaseApp(featurePath, featureName, baseAppPath);
+
+      expect(result).toBe('digitalExperiences/webApplications/base-app/src/routes.tsx');
+    });
+
+    it('should keep bare src paths at root level', () => {
+      const featurePath = 'src/routes.tsx';
+      const featureName = 'feature-simple';
+      const baseAppPath = '/path/to/base-app';
+
+      const result = translatePathToBaseApp(featurePath, featureName, baseAppPath);
+
+      // Bare src/ paths (without webApp/ prefix) should stay at root
+      expect(result).toBe('src/routes.tsx');
+    });
+
+    it('should handle nested paths correctly', () => {
+      const featurePath = 'digitalExperiences/webApplications/feature-auth/src/components/Login.tsx';
+      const featureName = 'feature-auth';
+      const baseAppPath = '/path/to/my-base-app';
+
+      const result = translatePathToBaseApp(featurePath, featureName, baseAppPath);
+
+      expect(result).toBe('digitalExperiences/webApplications/my-base-app/src/components/Login.tsx');
+    });
+
+    it('should handle different base app names', () => {
+      const featurePath = 'digitalExperiences/webApplications/feature-test/src/app.tsx';
+      const featureName = 'feature-test';
+      const baseAppPath = '/projects/custom-base-app-name';
+
+      const result = translatePathToBaseApp(featurePath, featureName, baseAppPath);
+
+      expect(result).toBe('digitalExperiences/webApplications/custom-base-app-name/src/app.tsx');
+    });
+
+    it('should add prefix to webApp paths', () => {
+      const featurePath = 'webApp/src/routes.tsx';
+      const featureName = 'feature-simple';
+      const baseAppPath = '/path/to/base-app';
+
+      const result = translatePathToBaseApp(featurePath, featureName, baseAppPath);
+
+      // WebApp paths get the prefix added
+      expect(result).toBe('digitalExperiences/webApplications/base-app/src/routes.tsx');
+    });
+
+    it('should keep SFDX metadata paths at root level', () => {
+      const featurePath = 'classes/NavMenu.cls';
+      const featureName = 'feature-navigation-menu';
+      const baseAppPath = '/path/to/base-app';
+
+      const result = translatePathToBaseApp(featurePath, featureName, baseAppPath);
+
+      // SFDX metadata paths should remain unchanged
+      expect(result).toBe('classes/NavMenu.cls');
+    });
+
+    it('should keep triggers at root level', () => {
+      const featurePath = 'triggers/MyTrigger.trigger';
+      const featureName = 'feature-simple';
+      const baseAppPath = '/path/to/base-app';
+
+      const result = translatePathToBaseApp(featurePath, featureName, baseAppPath);
+
+      expect(result).toBe('triggers/MyTrigger.trigger');
+    });
+
+    it('should keep objects at root level', () => {
+      const featurePath = 'objects/CustomObject__c/fields/CustomField__c.field-meta.xml';
+      const featureName = 'feature-simple';
+      const baseAppPath = '/path/to/base-app';
+
+      const result = translatePathToBaseApp(featurePath, featureName, baseAppPath);
+
+      expect(result).toBe('objects/CustomObject__c/fields/CustomField__c.field-meta.xml');
+    });
+
+    it('should handle paths with same prefix but different feature name', () => {
+      const featurePath = 'digitalExperiences/webApplications/other-feature/src/routes.tsx';
+      const featureName = 'feature-simple';
+      const baseAppPath = '/path/to/base-app';
+
+      const result = translatePathToBaseApp(featurePath, featureName, baseAppPath);
+
+      // Should not match and return as-is
+      expect(result).toBe('digitalExperiences/webApplications/other-feature/src/routes.tsx');
+    });
+
+    it('should keep bare src paths with subdirectories at root level', () => {
+      const featurePath = 'src/components/Header/index.tsx';
+      const featureName = 'feature-ui';
+      const baseAppPath = '/path/to/base-app';
+
+      const result = translatePathToBaseApp(featurePath, featureName, baseAppPath);
+
+      // Bare src/ paths (without webApp/ prefix) should stay at root
+      expect(result).toBe('src/components/Header/index.tsx');
+    });
+  });
+
+  describe('translatePathToTargetApp', () => {
+    it('should translate full path format to target app', () => {
+      const featurePath = 'digitalExperiences/webApplications/feature-simple/src/routes.tsx';
+      const featureName = 'feature-simple';
+      const targetAppName = 'my-target-app';
+
+      const result = translatePathToTargetApp(featurePath, featureName, targetAppName);
+
+      expect(result).toBe('digitalExperiences/webApplications/my-target-app/src/routes.tsx');
+    });
+
+    it('should keep bare src paths at root level', () => {
+      const featurePath = 'src/routes.tsx';
+      const featureName = 'feature-simple';
+      const targetAppName = 'my-target-app';
+
+      const result = translatePathToTargetApp(featurePath, featureName, targetAppName);
+
+      // Bare src/ paths (without webApp/ prefix) should stay at root
+      expect(result).toBe('src/routes.tsx');
+    });
+
+    it('should handle nested paths correctly', () => {
+      const featurePath = 'digitalExperiences/webApplications/feature-auth/src/components/Login.tsx';
+      const featureName = 'feature-auth';
+      const targetAppName = 'my-app';
+
+      const result = translatePathToTargetApp(featurePath, featureName, targetAppName);
+
+      expect(result).toBe('digitalExperiences/webApplications/my-app/src/components/Login.tsx');
+    });
+
+    it('should handle different target app names', () => {
+      const featurePath = 'digitalExperiences/webApplications/feature-test/src/app.tsx';
+      const featureName = 'feature-test';
+      const targetAppName = 'production-app';
+
+      const result = translatePathToTargetApp(featurePath, featureName, targetAppName);
+
+      expect(result).toBe('digitalExperiences/webApplications/production-app/src/app.tsx');
+    });
+
+    it('should add prefix to webApp paths', () => {
+      const featurePath = 'webApp/src/routes.tsx';
+      const featureName = 'feature-simple';
+      const targetAppName = 'my-app';
+
+      const result = translatePathToTargetApp(featurePath, featureName, targetAppName);
+
+      // WebApp paths get the prefix added
+      expect(result).toBe('digitalExperiences/webApplications/my-app/src/routes.tsx');
+    });
+
+    it('should keep SFDX metadata paths at root level', () => {
+      const featurePath = 'classes/NavMenu.cls';
+      const featureName = 'feature-navigation-menu';
+      const targetAppName = 'my-app';
+
+      const result = translatePathToTargetApp(featurePath, featureName, targetAppName);
+
+      // SFDX metadata paths should remain unchanged
+      expect(result).toBe('classes/NavMenu.cls');
+    });
+
+    it('should keep triggers at root level', () => {
+      const featurePath = 'triggers/MyTrigger.trigger';
+      const featureName = 'feature-simple';
+      const targetAppName = 'my-app';
+
+      const result = translatePathToTargetApp(featurePath, featureName, targetAppName);
+
+      expect(result).toBe('triggers/MyTrigger.trigger');
+    });
+
+    it('should keep objects at root level', () => {
+      const featurePath = 'objects/CustomObject__c/fields/CustomField__c.field-meta.xml';
+      const featureName = 'feature-simple';
+      const targetAppName = 'my-app';
+
+      const result = translatePathToTargetApp(featurePath, featureName, targetAppName);
+
+      expect(result).toBe('objects/CustomObject__c/fields/CustomField__c.field-meta.xml');
+    });
+
+    it('should handle paths with same prefix but different feature name', () => {
+      const featurePath = 'digitalExperiences/webApplications/other-feature/src/routes.tsx';
+      const featureName = 'feature-simple';
+      const targetAppName = 'my-app';
+
+      const result = translatePathToTargetApp(featurePath, featureName, targetAppName);
+
+      // Should not match and return as-is
+      expect(result).toBe('digitalExperiences/webApplications/other-feature/src/routes.tsx');
+    });
+
+    it('should keep bare src paths with subdirectories at root level', () => {
+      const featurePath = 'src/components/Header/index.tsx';
+      const featureName = 'feature-ui';
+      const targetAppName = 'final-app';
+
+      const result = translatePathToTargetApp(featurePath, featureName, targetAppName);
+
+      // Bare src/ paths (without webApp/ prefix) should stay at root
+      expect(result).toBe('src/components/Header/index.tsx');
+    });
+
+    it('should work with hyphenated target app names', () => {
+      const featurePath = 'digitalExperiences/webApplications/feature-nav/src/nav.tsx';
+      const featureName = 'feature-nav';
+      const targetAppName = 'my-custom-app-name';
+
+      const result = translatePathToTargetApp(featurePath, featureName, targetAppName);
+
+      expect(result).toBe('digitalExperiences/webApplications/my-custom-app-name/src/nav.tsx');
+    });
+  });
+
+  describe('translatePathToBaseApp vs translatePathToTargetApp', () => {
+    it('should produce similar results with equivalent inputs', () => {
+      const featurePath = 'digitalExperiences/webApplications/feature-test/src/app.tsx';
+      const featureName = 'feature-test';
+      const appName = 'test-app';
+      const baseAppPath = '/path/to/test-app';
+
+      const baseResult = translatePathToBaseApp(featurePath, featureName, baseAppPath);
+      const targetResult = translatePathToTargetApp(featurePath, featureName, appName);
+
+      // Both should translate to the same app name
+      expect(baseResult).toBe(targetResult);
+      expect(baseResult).toBe('digitalExperiences/webApplications/test-app/src/app.tsx');
+    });
+  });
+});

package/test/unit/route-merger.spec.ts

@@ -0,0 +1,265 @@
+import { describe, it, expect } from 'vitest';
+import { Project } from 'ts-morph';
+import { mergeRoutes } from '../../src/utils/route-merger.js';
+
+describe('route-merger', () => {
+  describe('mergeRoutes', () => {
+    it('should merge routes according to documented example', () => {
+      const project = new Project({ useInMemoryFileSystem: true });
+
+      // Base app routes (target)
+      project.createSourceFile(
+        '/test/target-routes.tsx',
+        `import AppLayout from './appLayout';
+import Home from './Home';
+
+export const routes: RouteObject[] = [
+  {
+    path: '/',
+    element: <AppLayout />,
+    children: [
+      { index: true, element: <Home /> }
+    ]
+  }
+];`
+      );
+
+      // Feature routes
+      project.createSourceFile(
+        '/test/feature-routes.tsx',
+        `import AppLayout from './appLayout';
+import About from './About';
+import Contact from './Contact';
+
+export const routes: RouteObject[] = [
+  {
+    path: '/',
+    element: <AppLayout />,
+    children: [
+      { path: 'about', element: <About /> },
+      { path: 'contact', element: <Contact /> }
+    ]
+  }
+];`
+      );
+
+      const result = mergeRoutes('/test/feature-routes.tsx', '/test/target-routes.tsx', project);
+
+      // Verify the merged result contains all expected elements
+      expect(result).toContain("path: '/'");
+      expect(result).toContain('element: <AppLayout />');
+      expect(result).toContain('index: true');
+      expect(result).toContain('element: <Home />');
+      expect(result).toContain("path: 'about'");
+      expect(result).toContain('element: <About />');
+      expect(result).toContain("path: 'contact'");
+      expect(result).toContain('element: <Contact />');
+    });
+
+    it('should preserve all route properties including handle', () => {
+      const project = new Project({ useInMemoryFileSystem: true });
+
+      project.createSourceFile(
+        '/test/target2-routes.tsx',
+        `export const routes: RouteObject[] = [
+  {
+    path: '/',
+    element: <AppLayout />,
+    children: []
+  }
+];`
+      );
+
+      project.createSourceFile(
+        '/test/feature2-routes.tsx',
+        `export const routes: RouteObject[] = [
+  {
+    path: '/',
+    element: <AppLayout />,
+    children: [
+      {
+        path: 'about',
+        element: <About />,
+        handle: { showInNavigation: true, label: 'About' }
+      }
+    ]
+  }
+];`
+      );
+
+      const result = mergeRoutes('/test/feature2-routes.tsx', '/test/target2-routes.tsx', project);
+
+      expect(result).toContain('handle:');
+      expect(result).toContain('showInNavigation: true');
+      expect(result).toContain("label: 'About'");
+    });
+
+    it('should add new routes when paths do not match', () => {
+      const project = new Project({ useInMemoryFileSystem: true });
+
+      project.createSourceFile(
+        '/test/target3-routes.tsx',
+        `export const routes: RouteObject[] = [
+  {
+    path: '/dashboard',
+    element: <Dashboard />
+  }
+];`
+      );
+
+      project.createSourceFile(
+        '/test/feature3-routes.tsx',
+        `export const routes: RouteObject[] = [
+  {
+    path: '/settings',
+    element: <Settings />
+  }
+];`
+      );
+
+      const result = mergeRoutes('/test/feature3-routes.tsx', '/test/target3-routes.tsx', project);
+
+      expect(result).toContain("path: '/dashboard'");
+      expect(result).toContain('element: <Dashboard />');
+      expect(result).toContain("path: '/settings'");
+      expect(result).toContain('element: <Settings />');
+    });
+
+    it('should replace matching routes but preserve non-matching children', () => {
+      const project = new Project({ useInMemoryFileSystem: true });
+
+      project.createSourceFile(
+        '/test/target4-routes.tsx',
+        `export const routes: RouteObject[] = [
+  {
+    path: '/',
+    element: <OldLayout />,
+    children: [
+      { path: 'old-page', element: <OldPage /> },
+      { path: 'shared-page', element: <OldShared /> }
+    ]
+  }
+];`
+      );
+
+      project.createSourceFile(
+        '/test/feature4-routes.tsx',
+        `export const routes: RouteObject[] = [
+  {
+    path: '/',
+    element: <NewLayout />,
+    children: [
+      { path: 'shared-page', element: <NewShared /> },
+      { path: 'new-page', element: <NewPage /> }
+    ]
+  }
+];`
+      );
+
+      const result = mergeRoutes('/test/feature4-routes.tsx', '/test/target4-routes.tsx', project);
+
+      // Should have new layout
+      expect(result).toContain('element: <NewLayout />');
+
+      // Should preserve old-page from target
+      expect(result).toContain("path: 'old-page'");
+      expect(result).toContain('element: <OldPage />');
+
+      // Should have new-page from feature
+      expect(result).toContain("path: 'new-page'");
+      expect(result).toContain('element: <NewPage />');
+
+      // Should replace shared-page with feature version
+      expect(result).toContain('element: <NewShared />');
+      expect(result).not.toContain('element: <OldShared />');
+    });
+
+    it('should handle route deletion with __delete__ prefix', () => {
+      const project = new Project({ useInMemoryFileSystem: true });
+
+      project.createSourceFile(
+        '/test/target5-routes.tsx',
+        `export const routes: RouteObject[] = [
+  {
+    path: '/',
+    element: <AppLayout />,
+    children: [
+      { path: 'home', element: <Home /> },
+      { path: 'old', element: <OldPage /> },
+      { path: 'about', element: <About /> }
+    ]
+  }
+];`
+      );
+
+      project.createSourceFile(
+        '/test/feature5-routes.tsx',
+        `export const routes: RouteObject[] = [
+  {
+    path: '/',
+    element: <AppLayout />,
+    children: [
+      { path: '__delete__old', element: null }
+    ]
+  }
+];`
+      );
+
+      const result = mergeRoutes('/test/feature5-routes.tsx', '/test/target5-routes.tsx', project);
+
+      // Should keep home and about
+      expect(result).toContain("path: 'home'");
+      expect(result).toContain("path: 'about'");
+
+      // Should NOT have 'old' route
+      expect(result).not.toContain("path: 'old'");
+      expect(result).not.toContain('element: <OldPage />');
+    });
+
+    it('should handle multiple route deletions', () => {
+      const project = new Project({ useInMemoryFileSystem: true });
+
+      project.createSourceFile(
+        '/test/target6-routes.tsx',
+        `export const routes: RouteObject[] = [
+  {
+    path: '/',
+    element: <AppLayout />,
+    children: [
+      { path: 'keep1', element: <Keep1 /> },
+      { path: 'delete1', element: <Delete1 /> },
+      { path: 'keep2', element: <Keep2 /> },
+      { path: 'delete2', element: <Delete2 /> }
+    ]
+  }
+];`
+      );
+
+      project.createSourceFile(
+        '/test/feature6-routes.tsx',
+        `export const routes: RouteObject[] = [
+  {
+    path: '/',
+    element: <AppLayout />,
+    children: [
+      { path: '__delete__delete1', element: null },
+      { path: '__delete__delete2', element: null }
+    ]
+  }
+];`
+      );
+
+      const result = mergeRoutes('/test/feature6-routes.tsx', '/test/target6-routes.tsx', project);
+
+      // Should keep these routes
+      expect(result).toContain("path: 'keep1'");
+      expect(result).toContain("path: 'keep2'");
+
+      // Should NOT have deleted routes
+      expect(result).not.toContain("path: 'delete1'");
+      expect(result).not.toContain("path: 'delete2'");
+      expect(result).not.toContain('element: <Delete1 />');
+      expect(result).not.toContain('element: <Delete2 />');
+    });
+  });
+});