Clean up template plugin compat code for Ember < 3.24

Author: dfreeman

packages/ember-css-modules/index.js

@@ -66,11 +66,8 @@ module.exports = {
     this.parentPreprocessorRegistry.add(
       'htmlbars-ast-plugin',
       HtmlbarsPlugin.instantiate({
-        emberVersion: this.checker.for('ember-source').version,
-        options: {
-          fileExtension: this.getFileExtension(),
-          includeExtensionInModulePath: this.includeExtensionInModulePath(),
-        },
+        fileExtension: this.getFileExtension(),
+        includeExtensionInModulePath: this.includeExtensionInModulePath(),
       })
     );
   },

packages/ember-css-modules/lib/htmlbars-plugin/index.js

@@ -1,31 +1,24 @@
 'use strict';
 
 const utils = require('./utils');
-const semver = require('semver');
 
 module.exports = class ClassTransformPlugin {
   constructor(env, options) {
     this.syntax = env.syntax;
     this.builders = env.syntax.builders;
     this.options = options;
     this.stylesModule = this.determineStylesModule(env);
-    this.isGlimmer = this.detectGlimmer();
     this.visitor = this.buildVisitor(env);
-
-    // Alias for 2.15 <= Ember < 3.1
-    this.visitors = this.visitor;
   }
 
-  static instantiate({ emberVersion, options }) {
+  static instantiate(options) {
     return {
       name: 'ember-css-modules',
-      plugin: semver.lt(emberVersion, '2.15.0-alpha')
-        ? LegacyAdapter.bind(null, this, options)
-        : (env) => new this(env, options),
+      plugin: (env) => new this(env, options),
       parallelBabel: {
         requireFile: __filename,
         buildUsing: 'instantiate',
-        params: { emberVersion, options },
+        params: options,
       },
       baseDir() {
         return `${__dirname}/../..`;
@@ -54,17 +47,6 @@ module.exports = class ClassTransformPlugin {
     return name;
   }
 
-  detectGlimmer() {
-    if (!this.syntax.parse) {
-      return false;
-    }
-
-    // HTMLBars builds ConcatStatements with StringLiterals + raw PathExpressions
-    // Glimmer builds ConcatStatements with TextNodes + MustacheStatements
-    let ast = this.syntax.parse('<div class="foo {{bar}}"></div>');
-    return ast.body[0].attributes[0].value.parts[0].type === 'TextNode';
-  }
-
   buildVisitor(env) {
     if (env.moduleName === env.filename) {
       // No-op for the stage 1 Embroider pass (which only contains relative paths)
@@ -133,7 +115,6 @@ module.exports = class ClassTransformPlugin {
 
     utils.removeAttr(node, localClassAttr);
 
-    let stringBuilder = this.isGlimmer ? 'text' : 'string';
     let classAttr = utils.getAttr(node, 'class');
     let parts = [];
     let classAttrValue;
@@ -149,26 +130,18 @@ module.exports = class ClassTransformPlugin {
         parts.push(
           this.builders.mustache(
             this.builders.path('concat'),
-            utils.concatStatementToParams(
-              this.builders,
-              classAttrValue,
-              this.isGlimmer
-            )
+            utils.concatStatementToParams(this.builders, classAttrValue)
           )
         );
       } else if (classAttrValue.type === 'TextNode') {
-        parts.push(this.builders[stringBuilder](classAttrValue.chars));
+        parts.push(this.builders.text(classAttrValue.chars));
       } else if (classAttrValue.type === 'MustacheStatement') {
-        if (classAttrValue.params.length || this.isGlimmer) {
-          parts.push(classAttrValue);
-        } else {
-          parts.push(this.builders.path(classAttrValue.path.original));
-        }
+        parts.push(classAttrValue);
       }
     }
 
     utils.pushAll(parts, this.localToPath(localClassAttr.value));
-    this.divide(parts, this.isGlimmer ? 'text' : 'string');
+    this.divide(parts, 'text');
     node.attributes.unshift(
       this.builders.attr('class', this.builders.concat(parts))
     );
@@ -225,11 +198,7 @@ module.exports = class ClassTransformPlugin {
 
   concatLocalPath(node) {
     let concatPath = this.builders.path('concat');
-    let concatParts = utils.concatStatementToParams(
-      this.builders,
-      node,
-      this.isGlimmer
-    );
+    let concatParts = utils.concatStatementToParams(this.builders, node);
     let concatStatement = this.builders.mustache(concatPath, concatParts);
     return this.dynamicLocalPath(concatStatement);
   }
@@ -268,22 +237,3 @@ module.exports = class ClassTransformPlugin {
     return parts;
   }
 };
-
-// For Ember < 2.15
-class LegacyAdapter {
-  constructor(plugin, options, env) {
-    this.plugin = plugin;
-    this.options = options;
-    this.meta = env.meta;
-    this.syntax = null;
-  }
-
-  transform(ast) {
-    let plugin = new this.plugin(
-      Object.assign({ syntax: this.syntax }, this.meta),
-      this.options
-    );
-    this.syntax.traverse(ast, plugin.visitor);
-    return ast;
-  }
-}

packages/ember-css-modules/lib/htmlbars-plugin/utils.js

@@ -44,11 +44,7 @@ function textToString(builders, node) {
   }
 }
 
-function concatStatementToParams(builders, node, isGlimmer) {
-  if (!isGlimmer) {
-    return node.parts;
-  }
-
+function concatStatementToParams(builders, node) {
   return node.parts.map(function (part) {
     if (part.type === 'MustacheStatement') {
       if (!part.params.length && !part.hash.pairs.length) {

packages/ember-css-modules/tests/helpers/render-with-styles.js

@@ -1,7 +1,6 @@
 /* global require, define */
 
 import Ember from 'ember';
-import { VERSION } from '@ember/version';
 import ClassTransformPlugin from 'ecm-template-transform';
 
 const { compile } = Ember.__loader.require('ember-template-compiler');
@@ -22,10 +21,7 @@ export default function registerStyles(styles) {
     }
 
     const { plugin } = ClassTransformPlugin.instantiate({
-      emberVersion: VERSION,
-      options: {
-        includeExtensionInModulePath: false,
-      },
+      includeExtensionInModulePath: false,
     });
 
     const plugins = { ast: [plugin] };