Skip to content

Commit 5cf5643

Browse files
committed
Decouple cli and project services
1 parent 5cca759 commit 5cf5643

21 files changed

+469
-467
lines changed

src/analytics/AnalyticsService.ts

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -4,8 +4,8 @@ import { Version } from '../common/version';
44
import { GUAService } from './GUAService';
55
import { TelerikAnalyticsService } from './TelerikAnalyticsService';
66
import { AnalyticsBaseInfo, OperatingSystem } from './AnalyticsBaseInfo';
7-
import { ExtensionVersionService } from '../common/ExtensionVersionService';
8-
import * as ns from '../project/NsCliService';
7+
import { ExtensionHostServices as Services } from '../services/extensionHostServices';
8+
import * as utils from '../common/utilities';
99

1010
export class AnalyticsService {
1111
private _baseInfo: AnalyticsBaseInfo;
@@ -39,8 +39,8 @@ export class AnalyticsService {
3939
};
4040

4141
this._baseInfo = {
42-
cliVersion: ns.CliVersionInfo.getInstalledCliVersion().toString(),
43-
extensionVersion: require('../../../package.json').version,
42+
cliVersion: Services.cli.version.toString(),
43+
extensionVersion: utils.getInstalledExtensionVersion().toString(),
4444
operatingSystem: operatingSystem,
4545
userId: AnalyticsService.generateMachineId()
4646
};

src/common/extensionVersionService.ts

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
11
import * as https from 'https';
22
import * as vscode from 'vscode';
33
import {Version} from './version';
4+
import * as utils from './utilities';
45

56
export type LatestPublishedVersionCheckResult = {latestPublishedVersion: string, timestamp: number};
67

@@ -74,7 +75,7 @@ export class ExtensionVersionService {
7475

7576
public get isLatestInstalled(): Promise<{ result: boolean, error: string }> {
7677
return this.latestPublishedVersion.then(latestVersion => {
77-
let extensionVersion = Version.parse(require('../../package.json').version);
78+
let extensionVersion = utils.getInstalledExtensionVersion();
7879
let isLatest: boolean = extensionVersion.compareBySubminorTo(latestVersion) >= 0;
7980
let error = isLatest ? null : `A new version of the NativeScript extension is available. Open "Extensions" panel to update to v${latestVersion}.`;
8081
return {result: isLatest, error: error};

src/debug-adapter/utilities.ts renamed to src/common/utilities.ts

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,7 @@ import * as os from 'os';
99
import * as fs from 'fs';
1010
import * as url from 'url';
1111
import * as path from 'path';
12+
import {Version} from './Version';
1213

1314
export const enum Platform {
1415
Windows, OSX, Linux
@@ -376,3 +377,11 @@ export function lstrip(s: string, lStr: string): string {
376377
s.substr(lStr.length) :
377378
s;
378379
}
380+
381+
export function getInstalledExtensionVersion(): Version {
382+
return Version.parse(require('../../package.json').version);
383+
}
384+
385+
export function getMinSupportedCliVersion(): Version {
386+
return Version.parse(require('../../package.json').minNativescriptCliVersion);
387+
}

src/debug-adapter/adapter/adapterProxy.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22
* Copyright (C) Microsoft Corporation. All rights reserved.
33
*--------------------------------------------------------*/
44

5-
import * as utils from '../utilities';
5+
import * as utils from '../../common/utilities';
66
import {DebugAdapterServices as Services} from '../../services/debugAdapterServices';
77
import {DebugProtocol} from 'vscode-debugprotocol';
88

src/debug-adapter/adapter/pathTransformer.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22
* Copyright (C) Microsoft Corporation. All rights reserved.
33
*--------------------------------------------------------*/
44

5-
import * as utils from '../utilities';
5+
import * as utils from '../../common/utilities';
66
import {DebugAdapterServices as Services} from '../../services/debugAdapterServices';
77
import {DebugProtocol} from 'vscode-debugprotocol';
88

src/debug-adapter/adapter/sourceMaps/pathUtilities.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@
77
import * as Path from 'path';
88
import * as URL from 'url';
99
import {DebugAdapterServices as Services} from '../../../services/debugAdapterServices';
10-
import * as utils from '../../utilities';
10+
import * as utils from '../../../common/utilities';
1111

1212
export function getPathRoot(p: string) {
1313
if (p) {

src/debug-adapter/adapter/sourceMaps/sourceMapTransformer.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@ import * as fs from 'fs';
77
import {DebugAdapterServices as Services} from '../../../services/debugAdapterServices';
88
import {DebugProtocol} from 'vscode-debugprotocol';
99
import {ISourceMaps, SourceMaps} from './sourceMaps';
10-
import * as utils from '../../utilities';
10+
import * as utils from '../../../common/utilities';
1111

1212
interface IPendingBreakpoint {
1313
resolve: () => void;

src/debug-adapter/adapter/sourceMaps/sourceMaps.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@ import * as URL from 'url';
99
import * as FS from 'fs';
1010
import {SourceMapConsumer} from 'source-map';
1111
import * as PathUtils from './pathUtilities';
12-
import * as utils from '../../utilities';
12+
import * as utils from '../../../common/utilities';
1313
import {DebugAdapterServices as Services} from '../../../services/debugAdapterServices';
1414

1515

src/debug-adapter/connection/iosConnection.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ import * as net from 'net';
66
import * as stream from 'stream';
77
import {EventEmitter} from 'events';
88
import {INSDebugConnection} from './INSDebugConnection';
9-
import * as utils from '../utilities';
9+
import * as utils from '../../common/utilities';
1010
import {DebugAdapterServices as Services} from '../../services/debugAdapterServices';
1111

1212
interface IMessageWithId {

src/debug-adapter/consoleHelper.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@
33
*--------------------------------------------------------*/
44

55
import * as url from 'url';
6-
import * as Utilities from './utilities';
6+
import * as Utilities from '../common/utilities';
77

88
export function formatConsoleMessage(m: WebKitProtocol.Console.Message, isClientPath :boolean = false): { text: string, isError: boolean } {
99
let outputText: string;

0 commit comments

Comments
 (0)