Add telemetry option

Author: gitKrystan

.eslintrc.js

@@ -1,34 +1,29 @@
 module.exports = {
-    root: true,
-    parserOptions: {
-      ecmaVersion: 2017
+  root: true,
+  parserOptions: {
+    ecmaVersion: 2017,
+  },
+  env: {
+    node: true,
+    es6: true,
+  },
+  extends: ['eslint:recommended', 'prettier'],
+  plugins: ['prettier'],
+  rules: {
+    'prettier/prettier': 'error',
+  },
+  overrides: [
+    {
+      files: ['**/*.ts'],
+      parser: '@typescript-eslint/parser',
+      plugins: ['prettier', '@typescript-eslint'],
+      extends: ['eslint:recommended', 'plugin:@typescript-eslint/recommended', 'prettier'],
     },
-    env: {
-      node: true,
-      es6: true
-    },
-    extends: ["eslint:recommended", "prettier"],
-    plugins: ["prettier"],
-    rules: {
-      "prettier/prettier": "error"
-    },
-    overrides: [
-      {
-        files: ['**/*.ts'],
-        parser: '@typescript-eslint/parser',
-        plugins: ['prettier', '@typescript-eslint'],
-        extends: [
-          'eslint:recommended',
-          'plugin:@typescript-eslint/recommended',
-          'prettier',
-          'prettier/@typescript-eslint',
-        ],
+    {
+      files: ['**/*.test.js'],
+      env: {
+        jest: true,
       },
-      {
-        files: ['**/*.test.js'],
-        env: {
-          jest: true
-        }
-      }
-    ]
+    },
+  ],
 };

TODO.md

@@ -0,0 +1,69 @@
+```ts
+  const { Webpack } = require('@embroider/webpack');
+  return require('@embroider/compat').compatBuild(app, Webpack, {
+    // staticComponents: true,
+    // staticModifiers: true,
+    // staticHelpers: true,
+    skipBabel: [
+      {
+        package: 'qunit',
+      },
+    ],
+  });
+```
+
+1. Find '@embroider/webpack' entry point
+2. Look at second arg to `compatBuild`
+3. ...?
+
+
+# TODO
+
+- [ ] Make telemetry pluggable
+- [ ] Maybe Discourse can make their own Telemetry plugin
+- [ ] Embroider telemetry plugin[1]
+
+## Problems
+
+[1] Embroider compat tries to link up your modules by emitting the implicit imports.
+
+## Questions for tomorrow
+
+1. How does the angle bracket codemod work?
+
+## Questions for EF4
+
+1. Is this even possible?!?
+
+```ts
+{{ambiguous}} ->
+  1. <Ambiguous>
+  2. {{(ambiguous)}}
+  3. {{this.ambiguous}}
+  4. {{ambiguous}} //modifier
+
+  1. leave it alone
+  2. change to {{this.ambiguous}}
+  (3. error?)
+
+enum DisambiguateResult {
+    LeaveIt,
+    ChangeIt,
+    CantDoIt,
+}
+
+enum Namespace {
+    Helper,
+    Component,
+    ComponentOrHelper,
+    Modifier,
+}
+
+function disambiguate(headName: string, namespace: Namespace, ...?): DisambiguateResult {
+    // ...
+    // SOMETHING SOMETHING PLUG IN EMBROIDER
+}
+
+
+
+```

helpers/types.ts

@@ -1,6 +1,4 @@
 /** Type predicate. Checks if the given value is a `Record<string, unknown>`. */
-export function isRecord<R extends Record<string, unknown>>(
-  value: unknown
-): value is R {
+export function isRecord<R extends Record<string, unknown>>(value: unknown): value is R {
   return value !== null && typeof value === 'object';
-}
+}

package.json

@@ -64,12 +64,12 @@
     "chalk": "^4.1.1",
     "common-tags": "^1.8.0",
     "coveralls": "^3.1.0",
-    "eslint": "^6.8.0",
-    "eslint-config-prettier": "^6.15.0",
+    "eslint": "^8.41.0",
+    "eslint-config-prettier": "^8.8.0",
     "eslint-plugin-prettier": "^3.4.0",
     "execa": "^3.4.0",
     "jest": "^29.1.0",
-    "prettier": "^1.19.1",
+    "prettier": "^2.8.8",
     "release-it": "^14.6.2",
     "release-it-lerna-changelog": "^3.1.0",
     "ts-jest": "^29.1.0",

test/fixtures/3.13/input/app/templates/application.hbs

@@ -1,7 +1,7 @@
 {{not false}}
 
-{{#if (not foo)}}
-  {{foo}}
+{{#if (not this.foo)}}
+  {{this.foo}}
 {{/if}}
 
 {{outlet}}

test/helpers/test-runner.ts

@@ -43,8 +43,8 @@ export class TestRunner {
       emberServe.stderr?.pipe(process.stderr);
     }
 
-    const serverWaiter = new Promise<void>(resolve => {
-      emberServe.stdout?.on('data', data => {
+    const serverWaiter = new Promise<void>((resolve) => {
+      emberServe.stdout?.on('data', (data) => {
         if (data.toString().includes('Build successful')) {
           resolve();
         }
@@ -75,7 +75,7 @@ export class TestRunner {
 
       await execa('diff', ['-rq', actual, expectedApp], { cwd: this.inputDir, stdio: 'inherit' });
     } catch (e) {
-      console.log(isRecord(e) ? e['stdout']: 'codemod did not run successfully');
+      console.log(isRecord(e) ? e['stdout'] : 'codemod did not run successfully');
 
       throw new Error('codemod did not run successfully');
     }

test/helpers/utils.ts

@@ -33,10 +33,10 @@ export async function kill(subprocess: execa.ExecaChildProcess): Promise<void> {
   } catch (e) {
     console.log(`PID ${subprocess.pid} has stopped.`);
     console.log(`\tKilled: ${subprocess.killed}`);
-    console.log(`\tCancelled: ${isRecord(e) ? e['isCanceled']: 'unknown'}`);
+    console.log(`\tCancelled: ${isRecord(e) ? e['isCanceled'] : 'unknown'}`);
   }
 
-  return new Promise(resolve => {
+  return new Promise((resolve) => {
     setTimeout(() => resolve(), 3000);
   });
 }

test/run-test.ts

@@ -44,7 +44,7 @@ const allVersions = ['3.10', '3.13'];
   }
 
   process.exit(didSucceed ? 0 : 1);
-})().catch(e => {
+})().catch((e) => {
   console.error(e);
   process.exit(1);
 });

transforms/no-implicit-this/__testfixtures__/-mock-telemetry.d.json.ts

@@ -2,4 +2,4 @@ export type Telemetry = Record<string, Record<string, unknown>>;
 
 declare const telemetry: Telemetry;
 
-export default telemetry;
+export default telemetry;

transforms/no-implicit-this/helpers/options.ts

@@ -2,15 +2,19 @@ import { getOptions as getCLIOptions } from 'codemod-cli';
 import fs from 'node:fs';
 import path from 'node:path';
 import { ZodError, ZodType, z } from 'zod';
-import { Telemetry, getTelemetry } from './telemetry';
+import Resolver, { RuntimeResolver } from './resolver';
 
 export interface Options {
-  customHelpers: string[],
-  telemetry: Telemetry
+  customHelpers: string[];
+  resolver: Resolver;
 }
 
 const CLIOptions = z.object({
   config: z.string().optional(),
+  // FIXME: Defaulting to 'runtime' isn't quite correct
+  telemetry: z.union([z.literal('runtime'), z.literal('embroider')]).default('runtime'),
+  // FIXME: Optionally allow angle-bracket conversion for components
+  // FIXME: Optionally add parens for helpers
 });
 
 type CLIOptions = z.infer<typeof CLIOptions>;
@@ -29,7 +33,8 @@ export function getOptions(): Options {
   const cliOptions = parse(getCLIOptions(), CLIOptions);
   return {
     customHelpers: getCustomHelpersFromConfig(cliOptions.config),
-    telemetry: getTelemetry(),
+    resolver:
+      cliOptions.telemetry === 'runtime' ? RuntimeResolver.build() : (null as unknown as Resolver),
   };
 }