feat: first pass at functional prototype without subprocess support (b…

…coe#5)

BREAKING CHANGE: dropped subprocess support for the time being, while we march towards an initial implementation.

.gitignore

@@ -1,2 +1,3 @@
 .DS_Store
 node_modules
+.nyc_output

README.md

@@ -14,13 +14,19 @@ The above example will collect coverage for `foo.js` using v8's profiler.
 
 TODO:
 
-- [ ] write logic for converting v8 coverage output to [Istanbul Coverage.json format](https://github.com/gotwarlost/istanbul/blob/master/coverage.json.md).
+- [x] write logic for converting v8 coverage output to [Istanbul Coverage.json format](https://github.com/gotwarlost/istanbul/blob/master/coverage.json.md).
+  * https://github.com/bcoe/v8-to-istanbul
+
 - [ ] talk to Node.js project about silencing messages:
 
    > `Debugger listening on ws://127.0.0.1:56399/e850110a-c5df-41d8-8ef2-400f6829617f`.
 
-- [ ] figure out why `detailed` mode does not appear to be working.
-- [ ] figure out a better way to determine that all processes in event loop
+- [x] figure out why `detailed` mode does not appear to be working.
+  * this is fixed in v8, as long as you start with `--inspect-brk` you
+    can collect coverage in detailed mode.
+- [x] figure out a better way to determine that all processes in event loop
    have terminated (except the inspector session).
-- [ ] process.exit() can't perform an async operation; how can we track coverage
+- [x] process.exit() can't perform an async operation; how can we track coverage
   for scripts that exit?
+  * we can now listen for the `Runtime.executionContextDestroyed` event.
+- [ ]

bin/c8.js

@@ -1,10 +1,58 @@
 #!/usr/bin/env node
 
 const argv = require('yargs').parse()
+const CRI = require('chrome-remote-interface')
+const getPort = require('get-port');
 const foreground = require('foreground-child')
-const sw = require('spawn-wrap')
+const waitTillPortOpen = require('wait-till-port-open')
 
-if (argv._.length) {
-  sw([require.resolve('./wrap')])
-  foreground(process.argv.slice(2))
+getPort().then(async port => {
+  foreground(
+    ['node', `--inspect-brk=${port}`].concat(process.argv.slice(2)),
+    (done) => {
+      console.info('actually got here')
+    }
+  )
+  try {
+    await waitTillPortOpen(port)
+    const client = await CRI({port: port})
+
+    const {Debugger, Runtime, Profiler} = client
+    await Runtime.runIfWaitingForDebugger()
+    await Runtime.enable()
+    await Profiler.enable()
+    await Profiler.startPreciseCoverage({callCount: true, detailed: true})
+    await Debugger.enable()
+    await Debugger.paused()
+    await Debugger.resume()
+
+    client.on('event', async (message) => {
+      // console.info(message)
+      if (message.method === 'Runtime.executionContextDestroyed') {
+        await outputCoverage(Profiler)
+        client.close()
+      }
+    })
+
+  } catch (err) {
+    console.error(err)
+    process.exit(1)
+  }
+})
+
+async function outputCoverage (Profiler) {
+  const IGNORED_PATHS = [
+    /\/bin\/wrap.js/,
+    /\/node_modules\//,
+    /node-spawn-wrap/
+  ]
+  let {result} = await Profiler.takePreciseCoverage()
+  result = result.filter(coverage => {
+    for (var ignored, i = 0; (ignored = IGNORED_PATHS[i]) !== undefined; i++) {
+      if (ignored.test(coverage.url)) return false
+    }
+    if (!/^\//.test(coverage.url)) return false
+    else return true
+  })
+  console.log(JSON.stringify(result, null, 2))
 }

package.json

@@ -25,7 +25,7 @@
     "chrome-remote-interface": "^0.25.2",
     "foreground-child": "^1.5.6",
     "get-port": "^3.2.0",
-    "spawn-wrap": "=1.3.8",
+    "wait-till-port-open": "^1.0.0",
     "yargs": "^10.0.3"
   },
   "devDependencies": {