diff --git a/extensions/markdown/npm-shrinkwrap.json b/extensions/markdown/npm-shrinkwrap.json index d1ac42b7a728283783ded522edeedad8951c1175..0f93176c5558515fed800fb4da3b2d8c7c5819db 100644 --- a/extensions/markdown/npm-shrinkwrap.json +++ b/extensions/markdown/npm-shrinkwrap.json @@ -3,9 +3,9 @@ "version": "0.2.0", "dependencies": { "applicationinsights": { - "version": "0.15.6", - "from": "applicationinsights@0.15.6", - "resolved": "https://registry.npmjs.org/applicationinsights/-/applicationinsights-0.15.6.tgz" + "version": "0.18.0", + "from": "applicationinsights@0.18.0", + "resolved": "https://registry.npmjs.org/applicationinsights/-/applicationinsights-0.18.0.tgz" }, "argparse": { "version": "1.0.9", @@ -58,14 +58,14 @@ "resolved": "https://registry.npmjs.org/uc.micro/-/uc.micro-1.0.3.tgz" }, "vscode-extension-telemetry": { - "version": "0.0.5", - "from": "vscode-extension-telemetry@>=0.0.5 <0.0.6", - "resolved": "https://registry.npmjs.org/vscode-extension-telemetry/-/vscode-extension-telemetry-0.0.5.tgz" + "version": "0.0.6", + "from": "vscode-extension-telemetry@>=0.0.6 <0.0.7", + "resolved": "https://registry.npmjs.org/vscode-extension-telemetry/-/vscode-extension-telemetry-0.0.6.tgz" }, "winreg": { - "version": "0.0.13", - "from": "winreg@0.0.13", - "resolved": "https://registry.npmjs.org/winreg/-/winreg-0.0.13.tgz" + "version": "1.2.3", + "from": "winreg@1.2.3", + "resolved": "https://registry.npmjs.org/winreg/-/winreg-1.2.3.tgz" } } } diff --git a/extensions/markdown/package.json b/extensions/markdown/package.json index dfd19b006b8a61fca30f941a6322aff2419270f4..aa40300de169ca7741359c7edf07e7f27b89d0c4 100644 --- a/extensions/markdown/package.json +++ b/extensions/markdown/package.json @@ -177,7 +177,7 @@ "highlight.js": "^9.3.0", "markdown-it": "^8.2.2", "markdown-it-named-headers": "0.0.4", - "vscode-extension-telemetry": "^0.0.5" + "vscode-extension-telemetry": "^0.0.6" }, "devDependencies": { "@types/node": "^7.0.4" diff --git a/extensions/markdown/src/extension.ts b/extensions/markdown/src/extension.ts index e3e72654850878c3f983472572d603bada55f81d..354d380c7fb6cda68a877f1248cd738db7c30e1c 100644 --- a/extensions/markdown/src/extension.ts +++ b/extensions/markdown/src/extension.ts @@ -30,6 +30,9 @@ var telemetryReporter: TelemetryReporter | null; export function activate(context: vscode.ExtensionContext) { const packageInfo = getPackageInfo(); telemetryReporter = packageInfo && new TelemetryReporter(packageInfo.name, packageInfo.version, packageInfo.aiKey); + if (telemetryReporter) { + context.subscriptions.push(telemetryReporter); + } const engine = new MarkdownEngine(); diff --git a/extensions/markdown/src/typings/vscode-extension-telemetry.d.ts b/extensions/markdown/src/typings/vscode-extension-telemetry.d.ts deleted file mode 100644 index f6177ef27a652fd02ce567819726c1920eb28dd5..0000000000000000000000000000000000000000 --- a/extensions/markdown/src/typings/vscode-extension-telemetry.d.ts +++ /dev/null @@ -1,6 +0,0 @@ -declare module 'vscode-extension-telemetry' { - export default class TelemetryReporter { - constructor(extensionId: string, extensionVersion: string, key: string); - sendTelemetryEvent(eventName: string, properties?: { [key: string]: string }, measures?: { [key: string]: number }): void; - } -} \ No newline at end of file diff --git a/extensions/typescript/npm-shrinkwrap.json b/extensions/typescript/npm-shrinkwrap.json index 058e3a3b44172c2319966ae92fcd278e2c31c864..093a64e196e710c7a20c51efef0a44cb2b4c5ed6 100644 --- a/extensions/typescript/npm-shrinkwrap.json +++ b/extensions/typescript/npm-shrinkwrap.json @@ -3,9 +3,9 @@ "version": "0.10.1", "dependencies": { "applicationinsights": { - "version": "0.15.6", - "from": "applicationinsights@0.15.6", - "resolved": "https://registry.npmjs.org/applicationinsights/-/applicationinsights-0.15.6.tgz" + "version": "0.18.0", + "from": "applicationinsights@0.18.0", + "resolved": "https://registry.npmjs.org/applicationinsights/-/applicationinsights-0.18.0.tgz" }, "semver": { "version": "4.3.6", @@ -13,14 +13,14 @@ "resolved": "https://registry.npmjs.org/semver/-/semver-4.3.6.tgz" }, "typescript": { - "version": "typescript@2.2.1-insiders.20170217", + "version": "2.2.1-insiders.20170217", "from": "typescript@typescript@2.2.1-insiders.20170217", "resolved": "https://registry.npmjs.org/typescript/-/typescript-2.2.1-insiders.20170217.tgz" }, "vscode-extension-telemetry": { - "version": "0.0.5", - "from": "vscode-extension-telemetry@>=0.0.5 <0.0.6", - "resolved": "https://registry.npmjs.org/vscode-extension-telemetry/-/vscode-extension-telemetry-0.0.5.tgz" + "version": "0.0.6", + "from": "vscode-extension-telemetry@>=0.0.6 <0.0.7", + "resolved": "https://registry.npmjs.org/vscode-extension-telemetry/-/vscode-extension-telemetry-0.0.6.tgz" }, "vscode-nls": { "version": "2.0.1", @@ -28,9 +28,9 @@ "resolved": "https://registry.npmjs.org/vscode-nls/-/vscode-nls-2.0.1.tgz" }, "winreg": { - "version": "0.0.13", - "from": "winreg@0.0.13", - "resolved": "https://registry.npmjs.org/winreg/-/winreg-0.0.13.tgz" + "version": "1.2.3", + "from": "winreg@1.2.3", + "resolved": "https://registry.npmjs.org/winreg/-/winreg-1.2.3.tgz" } } } diff --git a/extensions/typescript/package.json b/extensions/typescript/package.json index adf47d4b77f8b162fd4290517ac1d5ead0f87bf5..4fffc1019443abb7b0021d3bee2a3e0cae7cde1a 100644 --- a/extensions/typescript/package.json +++ b/extensions/typescript/package.json @@ -12,7 +12,7 @@ }, "dependencies": { "semver": "4.3.6", - "vscode-extension-telemetry": "^0.0.5", + "vscode-extension-telemetry": "^0.0.6", "vscode-nls": "^2.0.1", "typescript": "typescript@2.2.1-insiders.20170217" }, diff --git a/extensions/typescript/src/typescriptMain.ts b/extensions/typescript/src/typescriptMain.ts index 8c85d3845d57ac84626df3bb914f92a62a5dd74e..46d910a978c5ae3d805ddd5963ef9a78e796d8ef 100644 --- a/extensions/typescript/src/typescriptMain.ts +++ b/extensions/typescript/src/typescriptMain.ts @@ -408,7 +408,7 @@ class TypeScriptServiceClientHost implements ITypescriptServiceClientHost { configFileWatcher.onDidDelete(handleProjectCreateOrDelete, this, this.disposables); configFileWatcher.onDidChange(handleProjectChange, this, this.disposables); - this.client = new TypeScriptServiceClient(this, storagePath, globalState, workspaceState); + this.client = new TypeScriptServiceClient(this, storagePath, globalState, workspaceState, this.disposables); this.languages = []; this.languagePerId = Object.create(null); for (const description of descriptions) { diff --git a/extensions/typescript/src/typescriptServiceClient.ts b/extensions/typescript/src/typescriptServiceClient.ts index 1dcda834920c62d0902bb02dc2f2af49b6232326..7aae00d5e34941c75e41ffbb0732b8d529ed4a73 100644 --- a/extensions/typescript/src/typescriptServiceClient.ts +++ b/extensions/typescript/src/typescriptServiceClient.ts @@ -12,7 +12,7 @@ import * as fs from 'fs'; import * as electron from './utils/electron'; import { Reader } from './utils/wireProtocol'; -import { workspace, window, Uri, CancellationToken, OutputChannel, Memento, MessageItem, QuickPickItem, EventEmitter, Event, commands, WorkspaceConfiguration } from 'vscode'; +import { workspace, window, Uri, CancellationToken, Disposable, OutputChannel, Memento, MessageItem, QuickPickItem, EventEmitter, Event, commands, WorkspaceConfiguration } from 'vscode'; import * as Proto from './protocol'; import { ITypescriptServiceClient, ITypescriptServiceClientHost, API } from './typescriptService'; @@ -125,7 +125,7 @@ export default class TypeScriptServiceClient implements ITypescriptServiceClient private telemetryReporter: TelemetryReporter; - constructor(host: ITypescriptServiceClientHost, storagePath: string | undefined, globalState: Memento, private workspaceState: Memento) { + constructor(host: ITypescriptServiceClientHost, storagePath: string | undefined, globalState: Memento, private workspaceState: Memento, disposables: Disposable[]) { this.host = host; this.storagePath = storagePath; this.globalState = globalState; @@ -155,7 +155,7 @@ export default class TypeScriptServiceClient implements ITypescriptServiceClient this._apiVersion = new API('1.0.0'); this._checkGlobalTSCVersion = true; this.trace = this.readTrace(); - workspace.onDidChangeConfiguration(() => { + disposables.push(workspace.onDidChangeConfiguration(() => { this.trace = this.readTrace(); let oldglobalTsdk = this.globalTsdk; let oldLocalTsdk = this.localTsdk; @@ -167,9 +167,10 @@ export default class TypeScriptServiceClient implements ITypescriptServiceClient if (this.servicePromise === null && (oldglobalTsdk !== this.globalTsdk || oldLocalTsdk !== this.localTsdk)) { this.startService(); } - }); + })); if (this.packageInfo && this.packageInfo.aiKey) { this.telemetryReporter = new TelemetryReporter(this.packageInfo.name, this.packageInfo.version, this.packageInfo.aiKey); + disposables.push(this.telemetryReporter); } this.startService(); } diff --git a/extensions/typescript/src/typings/vscode-extension-telemetry.d.ts b/extensions/typescript/src/typings/vscode-extension-telemetry.d.ts deleted file mode 100644 index f6177ef27a652fd02ce567819726c1920eb28dd5..0000000000000000000000000000000000000000 --- a/extensions/typescript/src/typings/vscode-extension-telemetry.d.ts +++ /dev/null @@ -1,6 +0,0 @@ -declare module 'vscode-extension-telemetry' { - export default class TelemetryReporter { - constructor(extensionId: string, extensionVersion: string, key: string); - sendTelemetryEvent(eventName: string, properties?: { [key: string]: string }, measures?: { [key: string]: number }): void; - } -} \ No newline at end of file