bug #93 Fix lazy-controller detection when controller contain static properties (update acorn ECMAScript version) (ameotoko)

This PR was squashed before being merged into the main branch.

Discussion
----------

Fix lazy-controller detection when controller contain static properties (update `acorn` ECMAScript version)

Fixes #84

`acorn` should be configured to use ECMAScript version 2022 instead of 2020, which does not support static class properties.

See #84 (comment) for explanation.

Commits
-------

a258b7c Fix lazy-controller detection when controller contain static properties (update `acorn` ECMAScript version)

Author: Kocal

dist/webpack/lazy-controller-loader.js

@@ -5583,7 +5583,7 @@ function getCommentsFromSource(source) {
     parse(source, {
         onComment: comments,
         sourceType: 'module',
-        ecmaVersion: 2020,
+        ecmaVersion: 2022,
     });
     return comments;
 }

src/util/get-stimulus-comment-options.ts

@@ -24,7 +24,7 @@ function getCommentsFromSource(source: string) {
     parse(source, {
         onComment: comments,
         sourceType: 'module',
-        ecmaVersion: 2020,
+        ecmaVersion: 2022,
     });
 
     return comments;

test/util/get-stimulus-comment-options.test.ts

@@ -42,4 +42,12 @@ describe('getStimulusCommentOptions', () => {
             errors: [],
         });
     });
+
+    it('parses source with static properties', () => {
+        const src = '/* stimulusFetch: "lazy" */ export default class extends Controller { static targets = []; }';
+        expect(getStimulusCommentOptions(src)).toEqual({
+            options: { stimulusFetch: 'lazy' },
+            errors: [],
+        });
+    });
 });