@komaci/esm-generator 260.39.0 → 260.40.0

package/build/__tests__/adgAstGeneration.spec.js

@@ -363,7 +363,7 @@ export default class Component extends LightningElement {
         // Test that bundles with .ts files are processed correctly
         expect(() => {
             const bundleConfig = {
-                namespace: 'c',
+                namespace: 'lightning',
                 name: 'tsComponent',
                 type: 'internal',
                 namespaceMapping: {},

package/build/__tests__/index.spec.js

@@ -301,7 +301,7 @@ describe('Bundle processing', () => {
         const input = {
             moduleInfo: {
                 name: 'test',
-                namespace: 'c',
+                namespace: 'lightning',
                 type: 'bundle',
                 files: bundleSrcFileMap,
             },
@@ -319,7 +319,7 @@ describe('Bundle processing', () => {
     it('can handle multiple script files with the same name in different directories', () => {
         const bundleSrcFileMap = {
             'test.js': {},
-            'subdir/test.ts': {},
+            'subdir/test.js': {},
             'test.html': {},
         };
         const input = {
@@ -331,7 +331,7 @@ describe('Bundle processing', () => {
             },
             srcFileMap: {
                 'test.js': 'export default class Test {}',
-                'subdir/test.ts': 'export default class Test {}',
+                'subdir/test.js': 'export default class Test {}',
                 'test.html': '<template><div>Test</div></template>',
                 'test.css': 'p { text-align: center; }',
             },
@@ -346,7 +346,7 @@ describe('Bundle processing', () => {
                 return 'test';
             }
         }`;
-        const fooTsSource = `export default class Foo {
+        const fooJsSource = `export default class Foo {
             get fooProp() {
                 return 'foo';
             }
@@ -357,7 +357,7 @@ describe('Bundle processing', () => {
             type: 'internal',
             files: [
                 { fileName: 'test.js', source: testJsSource },
-                { fileName: 'foo.ts', source: fooTsSource },
+                { fileName: 'foo.js', source: fooJsSource },
                 { fileName: 'test.html', source: '<template><div>Test</div></template>' },
                 { fileName: 'test.css', source: 'p { text-align: center; }' },
             ],
@@ -378,7 +378,7 @@ describe('Bundle processing', () => {
             },
             srcFileMap: {
                 'test.js': testJsSource,
-                'foo.ts': fooTsSource,
+                'foo.js': fooJsSource,
                 'test.html': '<template><div>Test</div></template>',
                 'test.css': 'p { text-align: center; }',
             },
@@ -389,5 +389,105 @@ describe('Bundle processing', () => {
         expect(res).toContain('testProp');
         expect(res).not.toContain('fooProp');
     });
+    it('can process typescript files in the lightning namespace', () => {
+        const testTsSource = `import { LightningElement } from 'lwc';
+
+        export default class TestComponent extends LightningElement {
+            private count: number = 0;
+
+            get doubleCount(): number {
+                return this.count * 2;
+            }
+
+            increment(): void {
+                this.count++;
+            }
+        }`;
+        const bundleConfig = {
+            namespace: 'lightning',
+            name: 'testComponent',
+            type: 'internal',
+            files: [
+                { fileName: 'testComponent.ts', source: testTsSource },
+                {
+                    fileName: 'testComponent.html',
+                    source: '<template><div>Test</div></template>',
+                },
+            ],
+            enableKomaci: true,
+            namespaceMapping: {},
+            npmModuleMapping: {},
+        };
+        const metadata = (0, metadata_1.collectBundleMetadata)(bundleConfig);
+        const inputFiles = Object.fromEntries(metadata.files
+            .map(({ fileName, komaciDoc }) => [fileName, komaciDoc])
+            .filter(([, komaciDoc]) => !!komaciDoc));
+        const input = {
+            moduleInfo: {
+                name: 'testComponent',
+                namespace: 'lightning',
+                type: 'bundle',
+                files: inputFiles,
+            },
+            srcFileMap: {
+                'testComponent.ts': testTsSource,
+                'testComponent.html': '<template><div>Test</div></template>',
+            },
+        };
+        const res = (0, __1.generateKomaciModule)(input);
+        expect(res).not.toContain('ErrorFunction');
+        expect(res).toContain('registerAdgFunction');
+        expect(res).toContain('doubleCount');
+    });
+    it('throws error for typescript files in non-lightning namespace', () => {
+        const testTsSource = `import { LightningElement } from 'lwc';
+
+        export default class TestComponent extends LightningElement {
+            private count: number = 0;
+
+            get doubleCount(): number {
+                return this.count * 2;
+            }
+
+            increment(): void {
+                this.count++;
+            }
+        }`;
+        const bundleConfig = {
+            namespace: 'c',
+            name: 'testComponent',
+            type: 'internal',
+            files: [
+                { fileName: 'testComponent.ts', source: testTsSource },
+                {
+                    fileName: 'testComponent.html',
+                    source: '<template><div>Test</div></template>',
+                },
+            ],
+            enableKomaci: true,
+            namespaceMapping: {},
+            npmModuleMapping: {},
+        };
+        const metadata = (0, metadata_1.collectBundleMetadata)(bundleConfig);
+        const inputFiles = Object.fromEntries(metadata.files
+            .map(({ fileName, komaciDoc }) => [fileName, komaciDoc])
+            .filter(([, komaciDoc]) => !!komaciDoc));
+        const input = {
+            moduleInfo: {
+                name: 'testComponent',
+                namespace: 'c',
+                type: 'bundle',
+                files: inputFiles,
+            },
+            srcFileMap: {
+                'testComponent.ts': testTsSource,
+                'testComponent.html': '<template><div>Test</div></template>',
+            },
+        };
+        const res = (0, __1.generateKomaciModule)(input);
+        // TypeScript files in non-lightning namespaces should now throw an error
+        expect(res).toContain('ErrorFunction');
+        expect(res).toContain('unsupported file extension: ts');
+    });
 });
 //# sourceMappingURL=index.spec.js.map

package/build/componentAstProcessing.d.ts

@@ -5,11 +5,12 @@ import { ModuleContext, ModuleMetadata, HoistedFunctionMetadata } from '@komaci/
 /**
  * Retrieve the file path for the js of a component
  * @param generatorState GeneratorState containg the GeneratorInput, which in turn contains the JS raw source file
+ * @param allowTypeScript Optional parameter to control whether TypeScript files should be considered. Defaults to true for backward compatibility.
  * @returns Returns raw source code as string for the js of a component if it exists in the srcFileMap
  *          Returns undefined if no JS source file (whose filename also matches the LWC module name) was included in the
  *          GeneratorState.GeneratorInput by lwc-platform, other calling code.
  */
-export declare function getComponentSrcJs(generatorInput: GeneratorInput): string | undefined;
+export declare function getComponentSrcJs(generatorInput: GeneratorInput, allowTypeScript?: boolean): string | undefined;
 /**
  * AST is traversed to see if they contain getter functions that are not valid. For each getter that is valid, we will
  * update contextual information about the class and pass back info about the valid getters.

package/build/componentAstProcessing.js

@@ -7,15 +7,17 @@ const common_shared_1 = require("@komaci/common-shared");
 /**
  * Retrieve the file path for the js of a component
  * @param generatorState GeneratorState containg the GeneratorInput, which in turn contains the JS raw source file
+ * @param allowTypeScript Optional parameter to control whether TypeScript files should be considered. Defaults to true for backward compatibility.
  * @returns Returns raw source code as string for the js of a component if it exists in the srcFileMap
  *          Returns undefined if no JS source file (whose filename also matches the LWC module name) was included in the
  *          GeneratorState.GeneratorInput by lwc-platform, other calling code.
  */
-function getComponentSrcJs(generatorInput) {
+function getComponentSrcJs(generatorInput, allowTypeScript = true) {
     let sourceCode;
     const moduleName = generatorInput.moduleInfo.name;
     if (generatorInput.srcFileMap) {
-        const filename = Object.keys(generatorInput.srcFileMap).find((filename) => (filename.endsWith('.js') || filename.endsWith('.ts')) &&
+        const filename = Object.keys(generatorInput.srcFileMap).find((filename) => (filename.endsWith('.js') ||
+            (filename.endsWith('.ts') && allowTypeScript)) &&
             filename.startsWith(moduleName));
         if (filename) {
             sourceCode = generatorInput.srcFileMap[filename];

package/build/index.js

@@ -86,6 +86,15 @@ function normalizeSrcFileKeys(input) {
     }
 }
 exports.normalizeSrcFileKeys = normalizeSrcFileKeys;
+/**
+ * Determines if TypeScript files should be processed for the given namespace.
+ * TypeScript files are only processed for lightning and lightning/interop namespaces.
+ * @param namespace the module namespace
+ * @returns true if TypeScript files should be processed
+ */
+function shouldProcessTypeScriptForNamespace(namespace) {
+    return namespace === 'lightning' || namespace.startsWith('lightning/interop');
+}
 /**
  * Helper function to process and build the resolvable module based off the generator input and inital generator state.
  * @param input the Generator input
@@ -97,14 +106,16 @@ function processGeneratorInputAndState(input) {
         // Determine file type based on extension in file name.
         const segments = (0, common_shared_1.getFilepathSegments)(input.moduleInfo.fileName);
         const fileExt = segments[segments.length - 1];
+        const namespace = input.moduleInfo.namespace;
+        const isTypeScriptAllowed = shouldProcessTypeScriptForNamespace(namespace);
         let sourceFile;
-        const source = (0, componentAstProcessing_1.getComponentSrcJs)(input);
+        const source = (0, componentAstProcessing_1.getComponentSrcJs)(input, isTypeScriptAllowed);
         if (fileExt === 'html') {
             const templateMap = {};
             templateMap[input.moduleInfo.name] = input.moduleInfo.komaciDoc;
             sourceFile = (0, genericAstGeneration_1.generateResolvableModule)(input, undefined, templateMap, source);
         }
-        else if (fileExt === 'js' || fileExt === 'ts') {
+        else if (fileExt === 'js' || (fileExt === 'ts' && isTypeScriptAllowed)) {
             sourceFile = (0, genericAstGeneration_1.generateResolvableModule)(input, input.moduleInfo.komaciDoc, undefined, source);
         }
         else {
@@ -116,6 +127,8 @@ function processGeneratorInputAndState(input) {
     else if (input.moduleInfo.type === 'bundle') {
         // code to process a bundle
         let filename = '', fileExt;
+        const namespace = input.moduleInfo.namespace;
+        const isTypeScriptAllowed = shouldProcessTypeScriptForNamespace(namespace);
         const scriptDocs = {}; // {'test.js' : { filename: 'test', komaciDoc: KomaciDoc1 }}
         const htmlDocs = {};
         let sourceFile;
@@ -127,14 +140,14 @@ function processGeneratorInputAndState(input) {
             if (fileExt == 'html') {
                 htmlDocs[filename] = komaciDoc;
             }
-            else if (fileExt === 'js' || fileExt === 'ts') {
+            else if (fileExt === 'js' || (fileExt === 'ts' && isTypeScriptAllowed)) {
                 scriptDocs[filePath] = { filename, komaciDoc };
             }
             else {
                 throw new Error(types_1.ERROR_PREFIX + 'unsupported file extension: ' + fileExt);
             }
         }
-        const source = (0, componentAstProcessing_1.getComponentSrcJs)(input);
+        const source = (0, componentAstProcessing_1.getComponentSrcJs)(input, isTypeScriptAllowed);
         // Find the main component script file(s)
         const mainComponentScriptFiles = Object.values(scriptDocs).filter(({ filename }) => filename === input.moduleInfo.name);
         // For now we only support one main component script file.

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@komaci/esm-generator",
-    "version": "260.39.0",
+    "version": "260.40.0",
     "description": "Komaci generator for ADG ES modules",
     "homepage": "https://komaci.dev/",
     "repository": {