feat: include standalone-ness in the analytics

isStandaloneBuild is already tested by updater.test.ts, so don't
bother mocking this for the analytics.test, only check that it is included.

Tested manually with the --version changes on linux.

src/cli/commands/version.ts

@@ -1,3 +1,9 @@
-import { getVersion } from '../../lib/version';
+import { getVersion, isStandaloneBuild } from '../../lib/version';
 
-export = getVersion;
+export = async () => {
+  let version = await getVersion();
+  if (isStandaloneBuild()) {
+    version += ' (standalone)';
+  }
+  return version;
+};

src/lib/analytics.js

@@ -49,13 +49,16 @@ function postAnalytics(data) {
     return Promise.resolve();
   }
 
+  const isStandalone = version.isStandaloneBuild();
+
   // get snyk version
   return version
     .getVersion()
     .then((version) => {
       data.version = version;
       data.os = osName(os.platform(), os.release());
       data.nodeVersion = process.version;
+      data.standalone = isStandalone;
 
       const seed = uuid.v4();
       const shasum = crypto.createHash('sha1');

src/lib/updater.ts

@@ -1,6 +1,7 @@
 import * as updateNotifier from 'update-notifier';
 import * as fs from 'fs';
 import * as p from 'path';
+import { isStandaloneBuild } from './version';
 
 export function updateCheck() {
   const pkgPath = p.join(__dirname, '../..', 'package.json');
@@ -10,10 +11,7 @@ export function updateCheck() {
     return false;
   }
 
-  const standalonePath = p.join(__dirname, '../', 'STANDALONE');
-  const isStandaloneBuild = fs.existsSync(standalonePath);
-
-  if (isStandaloneBuild) {
+  if (isStandaloneBuild()) {
     return false;
   }
 

src/lib/version.ts

@@ -1,3 +1,4 @@
+import * as fs from 'fs';
 import * as path from 'path';
 import { executeCommand } from './exec';
 
@@ -20,6 +21,11 @@ export function getVersion(): Promise<string> {
   });
 }
 
+export function isStandaloneBuild() {
+  const standalonePath = path.join(__dirname, '../', 'STANDALONE');
+  return fs.existsSync(standalonePath);
+}
+
 async function getBranchCommitAndDirty(promises): Promise<string> {
   try {
     const res: string[] = await Promise.all(promises);

test/analytics.test.ts

@@ -63,6 +63,7 @@ test('analytics', (t) => {
         'metadata',
         'args',
         'nodeVersion',
+        'standalone',
         'durationMs',
       ].sort(),
       'keys as expected',
@@ -97,6 +98,7 @@ test('analytics with args', (t) => {
         'metadata',
         'args',
         'nodeVersion',
+        'standalone',
         'durationMs',
       ].sort(),
       'keys as expected',
@@ -132,6 +134,7 @@ test('analytics with args and org', (t) => {
         'metadata',
         'args',
         'nodeVersion',
+        'standalone',
         'durationMs',
         'org',
       ].sort(),