fix: use native performance object in Node 18 (#38)

Fixes #17

.github/workflows/test_node.yml

@@ -7,7 +7,7 @@ jobs:
     runs-on: ubuntu-latest
     strategy:
       matrix:
-        node: [ '12', '14', '16' ]
+        node: [ '12', '14', '16', '18' ]
     steps:
       - uses: actions/checkout@v2
       - uses: actions/setup-node@v2

src/index.js

@@ -31,9 +31,19 @@ let clear
 if (
   perf &&
   perf.mark &&
+  perf.measure &&
   perf.getEntriesByName &&
   perf.getEntriesByType &&
-  perf.clearMeasures
+  perf.clearMarks &&
+  perf.clearMeasures &&
+  // In Node, we want to detect that this perf/correctness fix [1] is available, which
+  // landed in Node 16.15.0, 17.6.0, and 18.0.0. However, it's not observable, and
+  // we don't want to rely on fragile version checks.
+  // So we can rely on this observable change [2] to add clearResourceTimings, which
+  // landed a bit later (18.2.0), but is close enough for our purposes.
+  // [1]: https://github.com/nodejs/node/pull/42032
+  // [2]: https://github.com/nodejs/node/pull/42725
+  (process.browser || perf.clearResourceTimings)
 ) {
   mark = name => {
     throwIfEmpty(name)

src/now.js

@@ -1,6 +1,6 @@
 import perf from './performance'
 
-let nowForNode
+let nowPolyfillForNode
 
 if (!process.browser) {
   // implementation borrowed from:
@@ -11,9 +11,9 @@ if (!process.browser) {
     return hr[0] * 1e9 + hr[1]
   }
   const loadTime = getNanoSeconds()
-  nowForNode = () => ((getNanoSeconds() - loadTime) / 1e6)
+  nowPolyfillForNode = () => ((getNanoSeconds() - loadTime) / 1e6)
 }
 
-export default process.browser
-  ? perf && perf.now ? () => perf.now() : () => Date.now()
-  : nowForNode
+export default perf && perf.now
+  ? () => perf.now()
+  : process.browser ? () => Date.now() : nowPolyfillForNode

src/performance.js

@@ -1,5 +1,2 @@
 /* global performance */
-
-// TODO: Node's built-in performance API has poor performance for getEntriesByName()
-// so currently we avoid it: https://github.com/nolanlawson/marky/issues/29
-export default process.browser && typeof performance !== 'undefined' && performance
+export default typeof performance !== 'undefined' && performance