@komaci/esm-generator 262.20.0 → 262.21.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: 'lightning',
+                namespace: 'c',
                 name: 'tsComponent',
                 type: 'internal',
                 namespaceMapping: {},

package/build/__tests__/index.spec.js

@@ -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.js': {},
+            'subdir/test.ts': {},
             'test.html': {},
         };
         const input = {
@@ -331,7 +331,7 @@ describe('Bundle processing', () => {
             },
             srcFileMap: {
                 'test.js': 'export default class Test {}',
-                'subdir/test.js': 'export default class Test {}',
+                'subdir/test.ts': '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 fooJsSource = `export default class Foo {
+        const fooTsSource = `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.js', source: fooJsSource },
+                { fileName: 'foo.ts', source: fooTsSource },
                 { 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.js': fooJsSource,
+                'foo.ts': fooTsSource,
                 'test.html': '<template><div>Test</div></template>',
                 'test.css': 'p { text-align: center; }',
             },
@@ -389,105 +389,5 @@ 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,12 +5,11 @@ 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, allowTypeScript?: boolean): string | undefined;
+export declare function getComponentSrcJs(generatorInput: GeneratorInput): 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,17 +7,15 @@ 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, allowTypeScript = true) {
+function getComponentSrcJs(generatorInput) {
     let sourceCode;
     const moduleName = generatorInput.moduleInfo.name;
     if (generatorInput.srcFileMap) {
-        const filename = Object.keys(generatorInput.srcFileMap).find((filename) => (filename.endsWith('.js') ||
-            (filename.endsWith('.ts') && allowTypeScript)) &&
+        const filename = Object.keys(generatorInput.srcFileMap).find((filename) => (filename.endsWith('.js') || filename.endsWith('.ts')) &&
             filename.startsWith(moduleName));
         if (filename) {
             sourceCode = generatorInput.srcFileMap[filename];

package/build/index.js

@@ -86,15 +86,6 @@ 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
@@ -106,16 +97,14 @@ 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, isTypeScriptAllowed);
+        const source = (0, componentAstProcessing_1.getComponentSrcJs)(input);
         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' && isTypeScriptAllowed)) {
+        else if (fileExt === 'js' || fileExt === 'ts') {
             sourceFile = (0, genericAstGeneration_1.generateResolvableModule)(input, input.moduleInfo.komaciDoc, undefined, source);
         }
         else {
@@ -127,8 +116,6 @@ 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;
@@ -140,14 +127,14 @@ function processGeneratorInputAndState(input) {
             if (fileExt == 'html') {
                 htmlDocs[filename] = komaciDoc;
             }
-            else if (fileExt === 'js' || (fileExt === 'ts' && isTypeScriptAllowed)) {
+            else if (fileExt === 'js' || fileExt === 'ts') {
                 scriptDocs[filePath] = { filename, komaciDoc };
             }
             else {
                 throw new Error(types_1.ERROR_PREFIX + 'unsupported file extension: ' + fileExt);
             }
         }
-        const source = (0, componentAstProcessing_1.getComponentSrcJs)(input, isTypeScriptAllowed);
+        const source = (0, componentAstProcessing_1.getComponentSrcJs)(input);
         // 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": "262.20.0",
+    "version": "262.21.0",
     "description": "Komaci generator for ADG ES modules",
     "homepage": "https://komaci.dev/",
     "repository": {