Merge branch 'master' of github.com:poolifier/tatami

.github/release-please/manifest.json

@@ -1,3 +1,3 @@
 {
-  ".": "0.8.6"
+  ".": "0.8.7"
 }

CHANGELOG.md

@@ -6,6 +6,14 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.1.0/),
 and this project adheres to
 [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [0.8.7](https://github.com/poolifier/tatami-ng/compare/0.8.6...0.8.7) (2024-10-18)
+
+
+### 🐞 Bug Fixes
+
+* handle corner case promise like benchmark function ([ad9c3be](https://github.com/poolifier/tatami-ng/commit/ad9c3becc72359b8b89330fc324eef61040b84ba))
+* promise like rejection without catch handling ([57de829](https://github.com/poolifier/tatami-ng/commit/57de82906b809f4949ba0e300a98409aa0c7a756))
+
 ## [0.8.6](https://github.com/poolifier/tatami-ng/compare/0.8.5...0.8.6) (2024-10-17)
 
 

README.md

@@ -93,7 +93,7 @@ Bun versions >= 1.x are supported.
 <script type="module">
 import {
  ...
-} from 'https://cdn.jsdelivr.net/npm/[email protected].6/dist/browser/index.js'
+} from 'https://cdn.jsdelivr.net/npm/[email protected].7/dist/browser/index.js'
 </script>
 ```
 

biome.json

@@ -1,5 +1,5 @@
 {
-  "$schema": "https://biomejs.dev/schemas/1.9.3/schema.json",
+  "$schema": "https://biomejs.dev/schemas/1.9.4/schema.json",
   "organizeImports": {
     "enabled": true
   },

jsr.json

@@ -1,7 +1,7 @@
 {
   "$schema": "https://jsr.io/schema/config-file.v1.json",
   "name": "@poolifier/tatami-ng",
-  "version": "0.8.6",
+  "version": "0.8.7",
   "exports": "./src/index.js",
   "publish": {
     "include": [

package.json

@@ -3,8 +3,8 @@
   "name": "tatami-ng",
   "type": "module",
   "license": "MIT",
-  "version": "0.8.6",
-  "packageManager": "[email protected].30",
+  "version": "0.8.7",
+  "packageManager": "[email protected].31",
   "bin": {
     "tatami": "cli.js"
   },
@@ -51,13 +51,13 @@
     "peowly": "^1.3.2"
   },
   "devDependencies": {
-    "@biomejs/biome": "^1.9.3",
+    "@biomejs/biome": "^1.9.4",
     "@commitlint/cli": "^19.5.0",
     "@commitlint/config-conventional": "^19.5.0",
     "@types/bun": "^1.1.11",
     "husky": "^9.1.6",
     "lint-staged": "^15.2.10",
-    "npm-run-all2": "^6.2.3",
+    "npm-run-all2": "^6.2.4",
     "prettier": "^3.3.3"
   },
   "peerDependencies": {

src/benchmark.js

@@ -30,9 +30,9 @@ import {
 import { runtime } from './runtime.js'
 import { now } from './time.js'
 import {
-  isAsyncFnResource,
   isAsyncFunction,
   isFunction,
+  isFunctionAsyncResource,
   isObject,
 } from './utils.js'
 
@@ -175,7 +175,7 @@ export function bench(name, fn = undefined, opts = {}) {
     samples: opts.samples ?? defaultSamples,
     warmup: opts.warmup ?? true,
     baseline: false,
-    async: isAsyncFnResource(fn),
+    async: isFunctionAsyncResource(fn),
   })
 }
 
@@ -215,7 +215,7 @@ export function baseline(name, fn = undefined, opts = {}) {
     samples: opts.samples ?? defaultSamples,
     warmup: opts.warmup ?? true,
     baseline: true,
-    async: isAsyncFnResource(fn),
+    async: isFunctionAsyncResource(fn),
   })
 }
 

src/lib.js

@@ -21,9 +21,9 @@ import { now } from './time.js'
 import {
   AsyncFunction,
   checkDividend,
-  isAsyncFnResource,
   isAsyncFunction,
   isFunction,
+  isFunctionAsyncResource,
   isObject,
 } from './utils.js'
 
@@ -204,7 +204,7 @@ export async function measure(fn, opts = {}) {
       `expected boolean as 'async' option, got ${opts.async.constructor.name}`
     )
 
-  opts.async = opts.async ?? isAsyncFnResource(fn)
+  opts.async = opts.async ?? isFunctionAsyncResource(fn)
   opts.time = opts.time ?? defaultTime
   opts.samples = opts.samples ?? defaultSamples
   opts.warmup =

src/utils.js

@@ -37,7 +37,7 @@ export const isAsyncFunction = fn => {
  * @param {Function} fn the function to check
  * @returns {Boolean} true if the function is an async function or returns a promise
  */
-export const isAsyncFnResource = fn => {
+export const isFunctionAsyncResource = fn => {
   if (fn == null) {
     return false
   }
@@ -49,7 +49,11 @@ export const isAsyncFnResource = fn => {
     const promiseLike = isPromiseLike(fnCall)
     if (promiseLike) {
       // silence promise rejection
-      fnCall.then(() => {}).catch(() => {})
+      try {
+        fnCall.then(() => {})?.catch(() => {})
+      } catch {
+        // ignore
+      }
     }
     return promiseLike
   } catch {