Move some analysis to main loop (#225)

Author: bartveneman

.gitignore

@@ -2,4 +2,5 @@ node_modules
 .nyc_output
 coverage
 .DS_Store
-dist
+dist
+.vscode

benchmark/run.js

@@ -0,0 +1,52 @@
+import byteSize from './format-filesize.js'
+import { analyze as analyzeCss } from '../dist/analyzer.modern.js'
+import * as fs from 'fs'
+const files = [
+  ['bol-com-20190617', 'Bol.com', 127],
+  ['bootstrap-5.0.0', 'Bootstrap 5.0.0', 49],
+  ['css-tricks-20190319', 'CSS-Tricks', 50],
+  ['facebook-20190319', 'Facebook.com', 75],
+  ['github-20210501', 'GitHub.com', 95],
+  ['gazelle-20210905', 'Gazelle.nl', 325],
+  ['lego-20190617', 'Lego.com', 62],
+  ['smashing-magazine-20190319', 'Smashing Magazine.com', 300],
+  ['trello-20190617', 'Trello.com', 86]
+]
+
+let maxLen = -1
+
+files.forEach(([, name]) => {
+  if (name.length > maxLen) {
+    maxLen = name.length
+  }
+})
+
+console.log('Running benchmark on /dist/analyzer.js:')
+
+const suite = []
+
+files.forEach(([filename, name, expectedDuration]) => {
+  const css = fs.readFileSync(`./src/__fixtures__/${filename}.css`, 'utf-8')
+  const fileSize = byteSize(css.length)
+  suite.push([
+    `${name.padEnd(maxLen + 2)} (${fileSize.padStart(6)})`,
+    () => analyzeCss(css),
+    expectedDuration
+  ])
+})
+
+const RUN_COUNT = 20
+
+suite.forEach(([name, fn, expectedDuration]) => {
+  const start = new Date()
+  for (let i = 0; i < RUN_COUNT; i++) {
+    fn();
+  }
+  const duration = Math.floor((new Date() - start) / RUN_COUNT)
+  const overtime = expectedDuration - duration
+  console.log(
+    name,
+    `${duration}ms`.padStart(6, ' '),
+    `(${overtime >= 0 ? '-' : '+'}${Math.abs(overtime)}ms ${Math.round(Math.abs(overtime) / duration * 100)}%)`
+  )
+})

src/aggregate-collection.js

@@ -82,7 +82,7 @@ class AggregateCollection {
     const max = sorted[sorted.length - 1]
 
     const sum = this.items.reduce((total, num) => (total += num))
-    const mode = Mode(sorted)
+    const mode = Mode(this.items)
     const median = Median(sorted)
 
     return {

src/context-collection.js

@@ -5,7 +5,6 @@ class ContextCollection {
     this.list = new CountableCollection()
     /** @type {[index; string]: CountableCollection} */
     this.contexts = {}
-    this.contextCount = 0
   }
 
   /**
@@ -18,7 +17,6 @@ class ContextCollection {
 
     if (!this.contexts[context]) {
       this.contexts[context] = new CountableCollection()
-      this.contextCount++
     }
 
     this.contexts[context].push(item)