fix: allow WebApplication app name in any case, add tests

lesya7 · cursoragent · lesya7 · commit 8aef61141847 · 2026-02-13T17:35:38.000-08:00
- Remove lowercase normalization; preserve folder/meta xml case
- Add tests for MyApp, myapp, MYAPP

Co-authored-by: Cursor &lt;cursoragent@cursor.com&gt;
diff --git a/src/resolve/adapters/webApplicationsSourceAdapter.ts b/src/resolve/adapters/webApplicationsSourceAdapter.ts
@@ -63,7 +63,7 @@ export class WebApplicationsSourceAdapter extends BundleSourceAdapter {
         ? source
         : new SourceComponent(
             {
-              name: source.name,
+              name: appName,
               type: source.type,
               content: source.content,
               xml: expectedXmlPath,
diff --git a/test/resolve/adapters/webApplicationsSourceAdapter.test.ts b/test/resolve/adapters/webApplicationsSourceAdapter.test.ts
@@ -192,6 +192,56 @@ describe('WebApplicationsSourceAdapter', () => {
     });
   });
 
+  describe('app name case', () => {
+    const buildAdapterWithAppName = (appName: string) => {
+      const appPath = join(BASE_PATH, appName);
+      const metaFile = join(appPath, `${appName}.webapplication-meta.xml`);
+      const distPath = join(appPath, 'dist');
+      const config = { outputDir: 'dist', routing: { trailingSlash: 'auto', fallback: '/index.html' } };
+      const vfs: VirtualDirectory[] = [
+        {
+          dirPath: appPath,
+          children: [
+            `${appName}.webapplication-meta.xml`,
+            { name: 'webapplication.json', data: Buffer.from(JSON.stringify(config)) },
+          ],
+        },
+        { dirPath: distPath, children: [{ name: 'index.html', data: Buffer.from('<html>test</html>') }] },
+      ];
+      const t = new VirtualTreeContainer(vfs);
+      return {
+        adapter: new WebApplicationsSourceAdapter(registry.types.webapplication, registryAccess, forceIgnore, t),
+        appPath,
+        metaFile,
+      };
+    };
+
+    it('should preserve mixed-case app name (MyApp)', () => {
+      const { adapter, appPath, metaFile } = buildAdapterWithAppName('MyApp');
+      const comp = adapter.getComponent(appPath);
+      expect(comp).to.not.be.undefined;
+      expect(comp!.name).to.equal('MyApp');
+      expect(comp!.fullName).to.equal('MyApp');
+      expect(comp!.xml).to.equal(metaFile);
+    });
+
+    it('should preserve lowercase app name (myapp)', () => {
+      const { adapter, appPath } = buildAdapterWithAppName('myapp');
+      const comp = adapter.getComponent(appPath);
+      expect(comp).to.not.be.undefined;
+      expect(comp!.name).to.equal('myapp');
+      expect(comp!.fullName).to.equal('myapp');
+    });
+
+    it('should preserve uppercase app name (MYAPP)', () => {
+      const { adapter, appPath } = buildAdapterWithAppName('MYAPP');
+      const comp = adapter.getComponent(appPath);
+      expect(comp).to.not.be.undefined;
+      expect(comp!.name).to.equal('MYAPP');
+      expect(comp!.fullName).to.equal('MYAPP');
+    });
+  });
+
   describe('webapplication.json validation', () => {
     // helper: build an adapter whose tree has the given webapplication.json content
     // plus optional extra VirtualDirectory entries for dist, etc.

Original file line number	Diff line number	Diff line change
`@@ -63,7 +63,7 @@ export class WebApplicationsSourceAdapter extends BundleSourceAdapter {`
`63`	`63`	`? source`
`64`	`64`	`: new SourceComponent(`
`65`	`65`	`{`
`66`		`- name: source.name,`
	`66`	`+ name: appName,`
`67`	`67`	`type: source.type,`
`68`	`68`	`content: source.content,`
`69`	`69`	`xml: expectedXmlPath,`