Skip to content

Commit 3c0e546

Browse files
update the code based on telemetry data changes
1 parent e052e28 commit 3c0e546

File tree

4 files changed

+13
-5
lines changed

4 files changed

+13
-5
lines changed

bin/cli.js

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@ const finder = require('find-package-json');
77
const recast = require('ember-template-recast');
88
const fs = require('fs');
99
const path = require('path');
10-
const { appResolver, detectTypeAndName } = require('../transforms/no-implicit-this/helpers/util');
10+
const { appResolver, detectTypeAndName, TELEMETRY_KEY } = require('../transforms/no-implicit-this/helpers/util');
1111
const { gatherSingleTelemetryForUrl, getTelemetry } = require('ember-codemods-telemetry-helpers');
1212
const appLocation = process.argv[2];
1313
const args = process.argv.slice(3);
@@ -93,7 +93,13 @@ function findAppName(f) {
9393
debug('Gathering telemetry data from %s ...', appLocation);
9494

9595
// This is for collecting metadata for the app just once to generate the map of lookupnames to local properties
96-
await gatherSingleTelemetryForUrl(appLocation, appResolver, lookupNames, appName);
96+
await gatherSingleTelemetryForUrl(
97+
appLocation,
98+
{ telemetryKey: TELEMETRY_KEY },
99+
appResolver,
100+
lookupNames,
101+
appName
102+
);
97103

98104
let telemetry = getTelemetry();
99105

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

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
{
2-
"single-telemetry": {
2+
"telemetry-data": {
33
"component:handlebars-with-prefix-component-properties-only": {
44
"type": "Component",
55
"localProperties": ["baz", "bang"],

transforms/no-implicit-this/helpers/plugin.js

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,7 @@ const path = require('path');
66
// sorta. telemetry needs to be defined.
77
// telemtry can be populated with -mock-telemetry.json
88
const KNOWN_HELPERS = require('./known-helpers');
9+
const { TELEMETRY_KEY } = require('./util');
910

1011
function getTelemetryObjByName(name, telemetry) {
1112
let telemetryLookupName = Object.keys(telemetry).find(item => item.split(':').pop() === name);
@@ -18,7 +19,7 @@ function transform(root, options = {}) {
1819
let b = recast.builders;
1920

2021
let scopedParams = [];
21-
let telemetry = options.telemetry ? options.telemetry['single-telemetry'] : {};
22+
let telemetry = options.telemetry ? options.telemetry[TELEMETRY_KEY] : {};
2223

2324
let customHelpers = options.customHelpers || [];
2425

transforms/no-implicit-this/helpers/util.js

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,6 @@
11
const path = require('path');
22
const globby = require('globby');
3+
const TELEMETRY_KEY = 'telemetry-data';
34

45
/**
56
* Generates a lookup name for a backing js class.
@@ -112,4 +113,4 @@ function appResolver(lookupNames, currAppName) {
112113
return mapping;
113114
}
114115

115-
module.exports = { appResolver, detectTypeAndName };
116+
module.exports = { appResolver, detectTypeAndName, TELEMETRY_KEY };

0 commit comments

Comments
 (0)