Merge pull request #550 from ember-codemods/no-telemetry

Author: gitKrystan

.gitignore

@@ -1,3 +1,5 @@
 node_modules
 *.log
 .eslintcache
+transforms/**/*.js
+!transforms/ember-object/__testfixtures__/**/*.js

README.md

@@ -50,6 +50,14 @@ If you have any _lazily loaded_ modules, such as modules from Ember Engines,
 you'll need to make sure that the URL you provide loads these modules as well.
 Otherwise, the codemod will not be able to detect them or analyze them.
 
+To disable this feature, run with `NO_TELEMETRY=true` and omit the path to your local server:
+
+```shell
+NO_TELEMETRY=true npx ember-native-class-codemod [OPTIONS] path/of/files/ or/some**/*glob.js
+```
+
+DANGER: Disabling the telemetry may result in incorrect behavior. Carefully vet the results.
+
 ### Types
 
 The `type` option can be used to further narrow down transforms to a particular type of

bin/cli.js

@@ -4,11 +4,11 @@
 const { gatherTelemetryForUrl, analyzeEmberObject } = require('ember-codemods-telemetry-helpers');
 
 (async () => {
-  await gatherTelemetryForUrl(process.argv[2], analyzeEmberObject);
+  let args = process.argv.slice(1);
+  if (process.env['NO_TELEMETRY'] !== 'true') {
+    await gatherTelemetryForUrl(process.argv[2], analyzeEmberObject);
+    args = process.argv.slice(2);
+  }
 
-  require('codemod-cli').runTransform(
-    __dirname,
-    'ember-object',
-    process.argv.slice(2) /* paths or globs */
-  );
+  require('codemod-cli').runTransform(__dirname, 'ember-object', args);
 })();

transforms/ember-object/__testfixtures__/-mock-telemetry.json

@@ -1,8 +1,8 @@
 {
   "runtime": {
     "computedProperties": ["computedMacro", "anotherMacro", "numPlusOne", "numPlusPlus", "error", "errorService"],
-    "observedProperties": [],
-    "observerProperties": {},
+    "observedProperties": ["prop"],
+    "observerProperties": { "observerProp": ["prop"] },
     "offProperties": { "offProp": ["prop1", "prop2"] },
     "overriddenActions": ["overriddenActionMethod"],
     "overriddenProperties": ["overriddenMethod"],

transforms/ember-object/__testfixtures__/decorators-invalid-2.options.json

@@ -0,0 +1,3 @@
+{
+  "noTelemetry": "false"
+}

transforms/ember-object/__testfixtures__/decorators-invalid-4.options.json

@@ -0,0 +1,3 @@
+{
+  "noTelemetry": "false"
+}

transforms/ember-object/__testfixtures__/object-literal-with-decorators-invalid-1.options.json

@@ -0,0 +1,3 @@
+{
+  "noTelemetry": false
+}

transforms/ember-object/__testfixtures__/object-literal-with-decorators-invalid-2.options.json

@@ -0,0 +1,3 @@
+{
+  "noTelemetry": false
+}

transforms/ember-object/__testfixtures__/object-literal-with-decorators-invalid-3.options.json

@@ -0,0 +1,3 @@
+{
+  "noTelemetry": false
+}

transforms/ember-object/__testfixtures__/object-literal-with-decorators-invalid-4.options.json

@@ -0,0 +1,3 @@
+{
+  "noTelemetry": false
+}