Merge pull request #172 from snyk/chore/create-bundle-func-split

fix: separated of getting supported files from creating remote bundle

src/bundles.ts

@@ -172,10 +172,6 @@ export async function remoteBundleFactory(options: RemoteBundleFactoryOptions):
   return remoteBundle;
 }
 
-interface CreateBundleFromFoldersOptions extends ConnectionOptions, AnalyzeFoldersOptions {
-  // pass
-}
-
 /**
  * Get supported filters and test baseURL for correctness and availability
  *
@@ -218,21 +214,40 @@ export interface FileBundle extends RemoteBundle {
   skippedOversizedFiles?: string[];
 }
 
+interface CreateBundleFromFoldersOptions extends ConnectionOptions, AnalyzeFoldersOptions {
+  // pass
+}
+
 /**
  * Creates a remote bundle and returns response from the bundle API
  *
  * @param {CreateBundleFromFoldersOptions} options
  * @returns {Promise<FileBundle | null>}
  */
 export async function createBundleFromFolders(options: CreateBundleFromFoldersOptions): Promise<FileBundle | null> {
-  const baseDir = determineBaseDir(options.paths);
-  const [supportedFiles, fileIgnores] = await Promise.all([
-    // Fetch supporte files to save network traffic
-    getSupportedFiles(options.baseURL, options.source, options.requestId, options.languages),
-    // Scan for custom ignore rules
-    collectIgnoreRules(options.paths, options.symlinksEnabled, options.defaultFileIgnores),
-  ]);
+  // Fetch supported files to save network traffic
+  const supportedFiles = await getSupportedFiles(options.baseURL, options.source, options.requestId, options.languages);
 
+  // Collect files and create a remote bundle
+  return await createBundleWithCustomFiles(options, supportedFiles);
+}
+
+/**
+ * Creates a remote bundle and returns response from the bundle API
+ * This function is used to create a bundle with a custom list of supported file extensions
+ *
+ * @param {CreateBundleFromFoldersOptions} options
+ * @param {SupportedFiles} supportedFiles
+ * @returns {Promise<FileBundle | null>}
+ */
+export async function createBundleWithCustomFiles(
+  options: CreateBundleFromFoldersOptions,
+  supportedFiles: SupportedFiles,
+): Promise<FileBundle | null> {
+  // Scan for custom ignore rules
+  const fileIgnores = await collectIgnoreRules(options.paths, options.symlinksEnabled, options.defaultFileIgnores);
+
+  const baseDir = determineBaseDir(options.paths);
   emitter.scanFilesProgress(0);
   const bundleFiles = [];
   const skippedOversizedFiles = [];

src/index.ts

@@ -1,5 +1,5 @@
 import { analyzeFolders, extendAnalysis } from './analysis';
-import { createBundleFromFolders } from './bundles';
+import { createBundleFromFolders, createBundleWithCustomFiles } from './bundles';
 import { emitter } from './emitter';
 import { startSession, checkSession, getAnalysis, getIpFamily, IpFamily } from './http';
 import { MAX_FILE_SIZE } from './constants';
@@ -23,6 +23,7 @@ export {
   getGlobPatterns,
   analyzeFolders,
   createBundleFromFolders,
+  createBundleWithCustomFiles,
   extendAnalysis,
   emitter,
   MAX_FILE_SIZE,