Revert to standard methods and target ES2022 for Stimulus compatibility

The arrow function approach didn't work because class fields are transformed differently by SWC, preventing Stimulus from discovering controller methods.

Changes:
- Reverted arrow functions back to standard class methods
- Changed SWC target from es2015 to es2022 to minimize class transformations
- Added optimizer config to preserve class structure
- Kept keepClassNames: true and loose: false for spec-compliant transforms

ES2022 has native support for modern class features, so SWC should apply minimal transformations that preserve Stimulus's ability to discover and bind controller methods.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>

Author: justin808

client/app/controllers/comments_controller.js

@@ -10,7 +10,7 @@ marked.use(mangle());
 export default class CommentsController extends Controller {
   static targets = ['commentList', 'commentAuthor', 'commentText', 'commentRefresh', 'alertDiv', 'errorList'];
 
-  resetText = () => {
+  resetText() {
     const inputAuthor = this.commentAuthorTarget;
     const inputText = this.commentTextTarget;
     const alertDiv = this.alertDivTarget;
@@ -41,12 +41,12 @@ export default class CommentsController extends Controller {
     }
   }
 
-  refreshCommentList = () => {
+  refreshCommentList() {
     const refreshBtn = this.commentRefreshTarget;
     refreshBtn.click();
   }
 
-  connect = () => {
+  connect() {
     console.log('connected to Stimulus comments_controller');
 
     const protocol = window.location.protocol === 'https:' ? 'wss://' : 'ws://';

config/swc.config.js

@@ -13,17 +13,24 @@ module.exports = {
     transform: {
       react: {
         runtime: "automatic"
+      },
+      // Disable optimizer to preserve class structure for Stimulus
+      optimizer: {
+        globals: {
+          vars: {}
+        }
       }
     },
-    // Critical for Stimulus: preserve class names and use spec-compliant transforms
+    // Critical for Stimulus: preserve class names and method bindings
     keepClassNames: true,
     // Use spec-compliant class transforms (not loose mode)
     loose: false,
-    // Don't use external helpers to avoid any potential binding issues
+    // Don't use external helpers
     externalHelpers: false,
-    target: "es2015"
+    // Target ES2022 to avoid transforming class features
+    target: "es2022"
   },
-  // Preserve original semantics for class fields/methods
+  // Preserve ES6 module semantics
   module: {
     type: "es6"
   }