Feat: enable optimization from rcs-core by default (closes #80) (#81)

Author: JPeer264

__tests__/processAuto.ts

@@ -1,6 +1,7 @@
 import tmp from 'tmp';
 import path from 'path';
 import fs from 'fs-extra';
+import rcsCore from 'rcs-core';
 import { minify } from 'html-minifier';
 
 import rcs from '../lib';
@@ -131,3 +132,30 @@ test('should fillLibraries from html and css | issue #38', async () => {
   expect(minify(newFile2, { collapseWhitespace: true }))
     .toBe(minify(expectedFile2, { collapseWhitespace: true }));
 });
+
+test('should check if optimize has been called', async () => {
+  const optimizeSpy = jest.spyOn(rcsCore, 'optimize');
+
+  await rcs.process.auto(['**/*.{js,html}', 'css/style.css'], {
+    newPath: testCwd.name,
+    cwd: fixturesCwd,
+  });
+
+  expect(optimizeSpy).toHaveBeenCalledTimes(1);
+
+  optimizeSpy.mockRestore();
+});
+
+test('should check if optimize has been called test', async () => {
+  const optimizeSpy = jest.spyOn(rcsCore, 'optimize');
+
+  await rcs.process.auto(['**/*.{js,html}', 'css/style.css'], {
+    newPath: testCwd.name,
+    cwd: fixturesCwd,
+    optimize: false,
+  });
+
+  expect(optimizeSpy).not.toHaveBeenCalled();
+
+  optimizeSpy.mockRestore();
+});

docs/api/processAuto.md

@@ -18,6 +18,7 @@ Options:
 - overwrite `<Boolean>`: ensures that it does not overwrite the same file accidently. Default is `false`
 - cwd `<String>`: the working directory in which to serach. Default is `process.cwd()`
 - newPath `<String>`: in which folder the new files should go. Default is `rcs`
+- optimize `<Boolean>`: checks if the selectors should be [optimized](https://github.com/JPeer264/node-rcs-core/blob/main/docs/api/optimize.md). Default is `true`
 
 Example:
 

docs/api/processCss.md

@@ -13,6 +13,7 @@ Parameters:
 
 Options:
 
+- optimize `<Boolean>`: checks if the selectors should be [optimized](https://github.com/JPeer264/node-rcs-core/blob/main/docs/api/optimize.md). Default is `true`
 - *all options of [rcs.process](process.md)*
 - *plus options [rcsCore.fillLibrary](https://github.com/JPeer264/node-rcs-core/blob/master/docs/api/filllibraries.md)*
 

docs/api/processHtml.md

@@ -13,6 +13,7 @@ Parameters:
 
 Options:
 
+- optimize `<Boolean>`: checks if the selectors should be [optimized](https://github.com/JPeer264/node-rcs-core/blob/main/docs/api/optimize.md). Default is `true`
 - *all options of [rcs.process](process.md)*
 - *plus options [rcsCore.replace.html](https://github.com/JPeer264/node-rcs-core/blob/master/docs/api/replace.md#html)*
 

lib/process/process.ts

@@ -11,7 +11,7 @@ import { ReplaceHtmlOptions } from 'rcs-core/dest/replace/html';
 import { ReplacePugOptions } from 'rcs-core/dest/replace/pug';
 import { ReplaceCssOptions } from 'rcs-core/dest/replace/css';
 import { ReplaceJsOptions } from 'rcs-core/dest/replace/js';
-import { FillLibrariesOptions } from 'rcs-core/dest/fillLibraries';
+import { FillLibrariesOptions as RcsFillLibrariesOptions } from 'rcs-core/dest/fillLibraries';
 
 import save from '../helper/save';
 import replaceData from './replaceData';
@@ -26,6 +26,10 @@ export interface BaseOptions {
   overwrite?: boolean;
 }
 
+export interface FillLibrariesOptions extends RcsFillLibrariesOptions {
+  optimize?: boolean;
+}
+
 export interface AllOptions {
   pug: BaseOptions & ReplacePugOptions;
   any: BaseOptions & ReplaceStringOptions;
@@ -51,6 +55,7 @@ async function rcsProcess(type: 'pug', pathString: string | string[], opts?: All
 async function rcsProcess(type: 'any', pathString: string | string[], opts?: AllOptions['any']): Promise<void>;
 async function rcsProcess(type: any, pathString: string | string[], opts: any = {}): Promise<void> {
   const options = { ...optionsDefault, ...opts };
+  const shouldOptimize = options.optimize ?? true;
 
   let globString: string;
 
@@ -132,6 +137,10 @@ async function rcsProcess(type: any, pathString: string | string[], opts: any =
     ))
   ));
 
+  if (shouldOptimize) {
+    rcs.optimize();
+  }
+
   await new Promise((res, rej) => (
     // now all files can be renamed and saved
     // can be fired parallel