@openwebf/webf 0.24.1 → 0.24.2

package/test/analyzer.test.ts

@@ -366,5 +366,49 @@ describe('Analyzer', () => {
       expect(classObj.props[1].type.value).toBe(FunctionArgumentType.promise);
       expect(classObj.props[2].type.value).toBe(FunctionArgumentType.int);
     });
+
+    it('should preserve complex CustomEvent generic types', () => {
+      const warnSpy = jest.spyOn(console, 'warn').mockImplementation(() => {});
+
+      const blob = new IDLBlob('/test/source.d.ts', '/test/target', 'test', 'test');
+      blob.raw = `
+        interface VideoError {
+          code: int;
+          message: string;
+        }
+
+        interface Events {
+          error: CustomEvent<VideoError>;
+          loadedmetadata: CustomEvent<{ duration: double; videoWidth: int; videoHeight: int }>;
+          volumechange: CustomEvent<{ volume: double; muted: boolean }>;
+        }
+      `;
+
+      const propertyCollector = {
+        properties: new Set<string>(),
+        files: new Set<string>(),
+        interfaces: new Set<string>(),
+      };
+      const unionTypeCollector = {
+        types: new Set<ParameterType[]>(),
+      };
+
+      analyzer(blob, propertyCollector, unionTypeCollector);
+
+      const eventsObj = blob.objects.find(o => (o as ClassObject).name === 'Events') as ClassObject;
+      expect(eventsObj).toBeDefined();
+
+      const errorProp = eventsObj.props.find(p => p.name === 'error');
+      expect(errorProp?.type.value).toBe('CustomEvent<VideoError>');
+
+      const loadedProp = eventsObj.props.find(p => p.name === 'loadedmetadata');
+      expect(loadedProp?.type.value).toBe('CustomEvent<{ duration: number; videoWidth: number; videoHeight: number }>');
+
+      const volumeProp = eventsObj.props.find(p => p.name === 'volumechange');
+      expect(volumeProp?.type.value).toBe('CustomEvent<{ volume: number; muted: boolean }>');
+
+      expect(warnSpy).not.toHaveBeenCalled();
+      warnSpy.mockRestore();
+    });
   });
-});
+});

package/test/commands.test.ts

@@ -24,7 +24,7 @@ mockFs.readFileSync = jest.fn().mockImplementation((filePath: any) => {
   if (pathStr.includes('gitignore.tpl')) return 'gitignore template';
   if (pathStr.includes('react.package.json.tpl')) return '<%= packageName %> <%= version %> <%= description %>';
   if (pathStr.includes('react.tsconfig.json.tpl')) return 'react tsconfig';
-  if (pathStr.includes('react.tsup.config.ts.tpl')) return 'tsup config';
+  if (pathStr.includes('react.tsdown.config.ts.tpl')) return 'tsdown config';
   if (pathStr.includes('react.index.ts.tpl')) return 'index template';
   if (pathStr.includes('vue.package.json.tpl')) return '<%= packageName %> <%= version %> <%= description %>';
   if (pathStr.includes('vue.tsconfig.json.tpl')) return 'vue tsconfig';
@@ -456,6 +456,78 @@ describe('Commands', () => {
       }));
     });
 
+    it('should copy Flutter README.md into generated React package root', async () => {
+      const options = {
+        flutterPackageSrc: '/flutter/src',
+        framework: 'react',
+        packageName: 'test-package'
+      };
+
+      // Mock TypeScript validation
+      mockTypeScriptValidation('/flutter/src');
+
+      const originalExistsSync = mockFs.existsSync as jest.Mock;
+      mockFs.existsSync = jest.fn().mockImplementation((filePath: any) => {
+        const pathStr = filePath.toString();
+        if (pathStr === '/flutter/src/README.md') return true;
+        if (pathStr === path.join(path.resolve('/dist'), 'README.md')) return false;
+        return originalExistsSync(filePath);
+      });
+
+      const originalReadFileSync = mockFs.readFileSync as jest.Mock;
+      mockFs.readFileSync = jest.fn().mockImplementation((filePath: any, encoding?: any) => {
+        const pathStr = filePath.toString();
+        if (pathStr === '/flutter/src/README.md') {
+          return '# Flutter README\n\nHello';
+        }
+        return originalReadFileSync(filePath, encoding);
+      });
+
+      await generateCommand('/dist', options);
+
+      expect(mockFs.writeFileSync).toHaveBeenCalledWith(
+        path.join(path.resolve('/dist'), 'README.md'),
+        '# Flutter README\n\nHello',
+        'utf-8'
+      );
+    });
+
+    it('should copy Flutter README.md into generated Vue package root', async () => {
+      const options = {
+        flutterPackageSrc: '/flutter/src',
+        framework: 'vue',
+        packageName: 'test-package'
+      };
+
+      // Mock TypeScript validation
+      mockTypeScriptValidation('/flutter/src');
+
+      const originalExistsSync = mockFs.existsSync as jest.Mock;
+      mockFs.existsSync = jest.fn().mockImplementation((filePath: any) => {
+        const pathStr = filePath.toString();
+        if (pathStr === '/flutter/src/README.md') return true;
+        if (pathStr === path.join(path.resolve('/dist'), 'README.md')) return false;
+        return originalExistsSync(filePath);
+      });
+
+      const originalReadFileSync = mockFs.readFileSync as jest.Mock;
+      mockFs.readFileSync = jest.fn().mockImplementation((filePath: any, encoding?: any) => {
+        const pathStr = filePath.toString();
+        if (pathStr === '/flutter/src/README.md') {
+          return '# Flutter README\n\nHello';
+        }
+        return originalReadFileSync(filePath, encoding);
+      });
+
+      await generateCommand('/dist', options);
+
+      expect(mockFs.writeFileSync).toHaveBeenCalledWith(
+        path.join(path.resolve('/dist'), 'README.md'),
+        '# Flutter README\n\nHello',
+        'utf-8'
+      );
+    });
+
     it('should generate an aggregated README in dist from markdown docs', async () => {
       const options = {
         flutterPackageSrc: '/flutter/src',
@@ -751,7 +823,7 @@ describe('Commands', () => {
               name: 'test-package',
               version: '1.0.0',
               dependencies: { react: '^18.0.0' },
-              scripts: { build: 'tsup' }
+              scripts: { build: 'tsdown' }
             });
           }
           return '';
@@ -1228,7 +1300,7 @@ describe('Commands', () => {
               name: 'test-package',
               version: '1.0.0',
               scripts: {
-                build: 'tsup'
+                build: 'tsdown'
               },
               dependencies: { react: '^18.0.0' }
             });
@@ -1277,7 +1349,7 @@ describe('Commands', () => {
               name: 'test-package',
               version: '1.0.0',
               scripts: {
-                build: 'tsup'
+                build: 'tsdown'
               },
               dependencies: { react: '^18.0.0' }
             });

package/test/generator.test.ts

@@ -395,6 +395,43 @@ describe('Generator', () => {
       const indexWrite = mockFs.writeFileSync.mock.calls.find(call => call[0].toString().includes('index.ts'));
       expect(indexWrite).toBeUndefined();
     });
+
+    it('should merge type-only Element exports into existing index.ts', async () => {
+      mockAnalyzer.analyzer.mockImplementation((blob: any) => {
+        const props = new ClassObject();
+        props.name = blob.filename === 'test' ? 'TestProperties' : 'ComponentProperties';
+        const events = new ClassObject();
+        events.name = blob.filename === 'test' ? 'TestEvents' : 'ComponentEvents';
+        blob.objects = [props, events];
+      });
+
+      mockFs.existsSync.mockImplementation((p: any) => {
+        const s = p.toString();
+        if (s.includes(path.join('/test/target', 'src', 'index.ts'))) return true;
+        return true;
+      });
+      mockFs.readFileSync.mockImplementation((p: any) => {
+        const s = p.toString();
+        if (s.includes(path.join('/test/target', 'src', 'index.ts'))) {
+          return '/* empty index scaffold */\n';
+        }
+        return 'test content';
+      });
+
+      await reactGen({
+        source: '/test/source',
+        target: '/test/target',
+        command: 'test command'
+      });
+
+      const indexWrite = mockFs.writeFileSync.mock.calls.find(call => call[0].toString().includes('index.ts'));
+      expect(indexWrite).toBeDefined();
+      const content = String(indexWrite![1]);
+      expect(content).toContain('export { Test }');
+      expect(content).toContain('export type { TestElement }');
+      expect(content).toContain('export { Component }');
+      expect(content).toContain('export type { ComponentElement }');
+    });
   });
 
   describe('vueGen', () => {

package/test/react.test.ts

@@ -184,6 +184,11 @@ describe('React Generator', () => {
       // Should include custom props with generated TS types
       expect(result).toContain('title: __webfTypes.dom_string;');
       expect(result).toContain('disabled?: __webfTypes.boolean;');
+
+      // Element interface should surface properties for refs
+      expect(result).toContain('export interface TestComponentElement extends WebFElementWithMethods<{');
+      expect(result).toContain('title: __webfTypes.dom_string;');
+      expect(result).toContain('disabled?: __webfTypes.boolean;');
       
       // And still include standard HTML props
       expect(result).toContain('id?: string;');