change the preprocess step to use an object instead of comments

Author: evoactivity

src/parse/preprocess.ts

@@ -32,12 +32,12 @@ export function preprocessTemplateRange(
 
   if (template.type === 'class-member') {
     // Replace with StaticBlock
-    prefix = 'static{/*';
-    suffix = '*/}';
+    prefix = 'static{t:`';
+    suffix = '`}';
   } else {
     // Replace with BlockStatement or ObjectExpression
-    prefix = '{/*';
-    suffix = '*/}';
+    prefix = '{t:`';
+    suffix = '`}';
 
     const nextToken = sliceByteRange(code, template.range.endByte).match(/\S+/);
 

src/print/index.ts

@@ -46,7 +46,10 @@ export const printer: Printer<Node | undefined> = {
       } else {
         let printed = estreePrinter.print(path, options, print, args);
 
-        assert('Expected Glimmer doc to be an array', Array.isArray(printed));
+        if (!Array.isArray(printed)) {
+          printed = [printed];
+        }
+
         trimPrinted(printed);
 
         // Remove semicolons so we can manage them ourselves

tests/unit-tests/preprocess.test.ts

@@ -8,40 +8,76 @@ import {
 const TEST_CASES = [
   {
     code: '<template>hi</template>',
-    expected: [`{/*hi               */}`],
+    expected: ['{t:`hi               `}'],
   },
   {
     code: '<template>/* hi */</template>',
-    expected: [`{/*~* hi *~               */}`],
+    expected: ['{t:`~* hi *~               `}'],
   },
   {
     code: '<template><div>hi</div></template>',
-    expected: [`{/*<div>hi<~div>               */}`],
+    expected: ['{t:`<div>hi<~div>               `}'],
   },
   {
     code: '<template>{{#if true}}hi{{/if}}</template>',
-    expected: [`{/*{{#if true}}hi{{~if}}               */}`],
+    expected: ['{t:`{{#if true}}hi{{~if}}               `}'],
   },
   {
     code: '<template>////////////////</template>',
-    expected: [`{/*~~~~~~~~~~~~~~~~               */}`],
+    expected: ['{t:`~~~~~~~~~~~~~~~~               `}'],
   },
   {
     code: '<template>💩</template>',
-    expected: [`{/*💩               */}`],
+    expected: ['{t:`💩               `}'],
   },
   {
     code: 'const a = <template>foo</template>; const b = <template>bar</template>;',
     expected: [
-      `const a = {/*foo               */}; const b = <template>bar</template>;`,
-      `const a = <template>foo</template>; const b = {/*bar               */};`,
+      'const a = {t:`foo               `}; const b = <template>bar</template>;',
+      'const a = <template>foo</template>; const b = {t:`bar               `};',
     ],
   },
   {
     code: `const a = <template>💩💩💩💩💩💩💩</template>; const b = <template>💩</template>`,
     expected: [
-      `const a = {/*💩💩💩💩💩💩💩               */}; const b = <template>💩</template>`,
-      `const a = <template>💩💩💩💩💩💩💩</template>; const b = {/*💩               */}`,
+      'const a = {t:`💩💩💩💩💩💩💩               `}; const b = <template>💩</template>',
+      'const a = <template>💩💩💩💩💩💩💩</template>; const b = {t:`💩               `}',
+    ],
+  },
+  {
+    code: 'class Thing { <template>hello</template> }',
+    expected: ['class Thing { static{t:`hello         `} }'],
+  },
+  {
+    code: `export default <template>     Explicit default export module top level component. Explicit default export module top level component. Explicit default export module top level component. Explicit default export module top level component. Explicit default export module top level component. </template>
+/*AMBIGUOUS*/`,
+    expected: [
+      `export default {t:` +
+        '`     Explicit default export module top level component. Explicit default export module top level component. Explicit default export module top level component. Explicit default export module top level component. Explicit default export module top level component.                `}' +
+        `
+/*AMBIGUOUS*/`,
+    ],
+  },
+  {
+    code: `class MyComponent
+  extends Component {
+    // prettier-ignore
+        <template>
+
+
+    <h1>   Class top level template. Class top level template. Class top level template. Class top level template. Class top level template. </h1>
+  </template>
+}`,
+    expected: [
+      `class MyComponent
+  extends Component {
+    // prettier-ignore
+        static{t:\`
+
+
+    <h1>   Class top level template. Class top level template. Class top level template. Class top level template. Class top level template. <~h1>
+           \`}
+}`,
     ],
   },
 ];