Require Node.js 18

Author: sindresorhus

.github/funding.yml

@@ -10,11 +10,12 @@ jobs:
       fail-fast: false
       matrix:
         node-version:
-          - 14
-          - 12
+          - 22
+          - 20
+          - 18
     steps:
-      - uses: actions/checkout@v2
-      - uses: actions/setup-node@v2
+      - uses: actions/checkout@v4
+      - uses: actions/setup-node@v4
         with:
           node-version: ${{ matrix.node-version }}
       - run: npm install

.github/workflows/main.yml

@@ -1,5 +1,5 @@
 /**
-Create a lazy promise that defers execution until it's awaited or when `.then()` or `.catch()` is called.
+Create a lazy promise that defers execution until it's awaited or when `.then()`, `.catch()`, or `.finally()` is called.
 */
 export default class PLazy<ValueType> extends Promise<ValueType> { // eslint-disable-line @typescript-eslint/naming-convention
 	/**

index.d.ts

@@ -1,5 +1,3 @@
-// TODO: Use private class fields when ESLint support it.
-
 export default class PLazy extends Promise {
 	#executor;
 	#promise;
@@ -31,18 +29,17 @@ export default class PLazy extends Promise {
 	}
 
 	then(onFulfilled, onRejected) {
-		// TODO: Use `??=` when targeting Node.js 16.
-		this.#promise = this.#promise || new Promise(this.#executor);
+		this.#promise ??= new Promise(this.#executor);
 		return this.#promise.then(onFulfilled, onRejected);
 	}
 
 	catch(onRejected) {
-		this.#promise = this.#promise || new Promise(this.#executor);
+		this.#promise ??= new Promise(this.#executor);
 		return this.#promise.catch(onRejected);
 	}
 
 	finally(onFinally) {
-		this.#promise = this.#promise || new Promise(this.#executor);
+		this.#promise ??= new Promise(this.#executor);
 		return this.#promise.finally(onFinally);
 	}
 }

index.js

@@ -10,7 +10,7 @@ const lazyPromise = new PLazy<string>((resolve, reject) => {
 });
 
 expectType<PLazy<string>>(lazyPromise);
-expectType<PLazy<number>>(PLazy.from(async () => Promise.resolve(1)));
+expectType<PLazy<number>>(PLazy.from(async () => 1));
 expectType<PLazy<number>>(PLazy.from(() => 1));
 expectType<PLazy<number>>(PLazy.resolve(1));
 expectType<PLazy<never>>(PLazy.reject(new Error('fixture')));

index.test-d.ts

@@ -1,7 +1,7 @@
 {
 	"name": "p-lazy",
 	"version": "4.0.0",
-	"description": "Create a lazy promise that defers execution until it's awaited or when `.then()` or `.catch()` is called",
+	"description": "Create a lazy promise that defers execution until it's awaited or when `.then()`, `.catch()`, or `.finally()` is called",
 	"license": "MIT",
 	"repository": "sindresorhus/p-lazy",
 	"funding": "https://github.com/sponsors/sindresorhus",
@@ -11,9 +11,13 @@
 		"url": "https://sindresorhus.com"
 	},
 	"type": "module",
-	"exports": "./index.js",
+	"exports": {
+		"types": "./index.d.ts",
+		"default": "./index.js"
+	},
+	"sideEffects": false,
 	"engines": {
-		"node": ">=12"
+		"node": ">=18"
 	},
 	"scripts": {
 		"test": "xo && ava && tsd"
@@ -37,9 +41,14 @@
 		"bluebird"
 	],
 	"devDependencies": {
-		"ava": "^4.0.0",
-		"delay": "^5.0.0",
-		"tsd": "^0.19.1",
-		"xo": "^0.47.0"
+		"ava": "^6.2.0",
+		"tsd": "^0.31.2",
+		"xo": "^0.59.3"
+	},
+	"xo": {
+		"rules": {
+			"promise/prefer-await-to-then": "off",
+			"unicorn/no-thenable": "off"
+		}
 	}
 }

package.json

@@ -1,6 +1,6 @@
 # p-lazy
 
-> Create a lazy promise that defers execution until it's awaited or when `.then()` or `.catch()` is called
+> Create a lazy promise that defers execution until it's awaited or when `.then()`, or `.catch()`, or `.finally()` is called
 
 Useful if you're doing some heavy operations and would like to only do it when the promise is actually used.
 

readme.md

@@ -1,5 +1,5 @@
+import {setTimeout as delay} from 'node:timers/promises';
 import test from 'ava';
-import delay from 'delay';
 import PLazy from './index.js';
 
 const fixture = Symbol('fixture');