Fix: Publish types (+ validate and use types + generate type maps)

declaration.tsconfig.json

@@ -0,0 +1,8 @@
+{
+	"extends": "./tsconfig",
+	"compilerOptions": {
+		"declaration": true,
+		"noEmit": false,
+		"emitDeclarationOnly": true
+	}
+}

index.js

@@ -2,10 +2,14 @@
 const supportsColor = require('supports-color');
 const hasFlag = require('has-flag');
 
+/**
+@param {string} versionString
+@returns {{ major: number, minor: number, patch: number }}
+*/
 function parseVersion(versionString) {
 	if (/^\d{3,4}$/.test(versionString)) {
 		// Env var doesn't always use dots. example: 4601 => 46.1.0
-		const m = /(\d{1,2})(\d{2})/.exec(versionString);
+		const m = /(\d{1,2})(\d{2})/.exec(versionString) ?? [];
 		return {
 			major: 0,
 			minor: parseInt(m[1], 10),
@@ -21,11 +25,23 @@ function parseVersion(versionString) {
 	};
 }
 
+/**
+@param {{ isTTY?: boolean | undefined }} stream
+@returns {boolean}
+*/
 function supportsHyperlink(stream) {
-	const {env} = process;
-
-	if ('FORCE_HYPERLINK' in env) {
-		return !(env.FORCE_HYPERLINK.length > 0 && parseInt(env.FORCE_HYPERLINK, 10) === 0);
+	const {
+		CI,
+		FORCE_HYPERLINK,
+		NETLIFY,
+		TEAMCITY_VERSION,
+		TERM_PROGRAM,
+		TERM_PROGRAM_VERSION,
+		VTE_VERSION
+	} = process.env;
+
+	if (FORCE_HYPERLINK) {
+		return !(FORCE_HYPERLINK.length > 0 && parseInt(FORCE_HYPERLINK, 10) === 0);
 	}
 
 	if (hasFlag('no-hyperlink') || hasFlag('no-hyperlinks') || hasFlag('hyperlink=false') || hasFlag('hyperlink=never')) {
@@ -37,7 +53,7 @@ function supportsHyperlink(stream) {
 	}
 
 	// Netlify does not run a TTY, it does not need `supportsColor` check
-	if ('NETLIFY' in env) {
+	if (NETLIFY) {
 		return true;
 	}
 
@@ -54,18 +70,18 @@ function supportsHyperlink(stream) {
 		return false;
 	}
 
-	if ('CI' in env) {
+	if (CI) {
 		return false;
 	}
 
-	if ('TEAMCITY_VERSION' in env) {
+	if (TEAMCITY_VERSION) {
 		return false;
 	}
 
-	if ('TERM_PROGRAM' in env) {
-		const version = parseVersion(env.TERM_PROGRAM_VERSION);
+	if (TERM_PROGRAM) {
+		const version = parseVersion(TERM_PROGRAM_VERSION || '');
 
-		switch (env.TERM_PROGRAM) {
+		switch (TERM_PROGRAM) {
 			case 'iTerm.app':
 				if (version.major === 3) {
 					return version.minor >= 1;
@@ -80,13 +96,13 @@ function supportsHyperlink(stream) {
 		}
 	}
 
-	if ('VTE_VERSION' in env) {
+	if (VTE_VERSION) {
 		// 0.50.0 was supposed to support hyperlinks, but throws a segfault
-		if (env.VTE_VERSION === '0.50.0') {
+		if (VTE_VERSION === '0.50.0') {
 			return false;
 		}
 
-		const version = parseVersion(env.VTE_VERSION);
+		const version = parseVersion(VTE_VERSION);
 		return version.major > 0 || version.minor >= 50;
 	}
 

package.json

@@ -10,15 +10,16 @@
         "url": "github.com/jamestalmage"
     },
     "engines": {
-        "node": ">=8"
+        "node": ">=14.18"
     },
     "scripts": {
         "prepublishOnly": "npm run create-types",
-        "test": "xo && nyc ava",
-        "create-types": "tsc index.js --allowJs  --declaration --emitDeclarationOnly"
+        "test": "xo && nyc ava && tsc",
+        "create-types": "tsc -p declaration.tsconfig.json"
     },
     "files": [
         "index.js",
+        "index.d.ts",
         "browser.js"
     ],
     "browser": "browser.js",
@@ -33,10 +34,12 @@
         "supports-color": "^7.0.0"
     },
     "devDependencies": {
+        "@tsconfig/node14": "^1.0.3",
+        "@types/supports-color": "^8.1.1",
         "ava": "^2.2.0",
         "codecov": "^3.5.0",
         "nyc": "^14.1.1",
-        "typescript": "^3.7.2",
+        "typescript": "^4.9.5",
         "xo": "^0.24.0"
     },
     "nyc": {

tsconfig.json

@@ -0,0 +1,12 @@
+{
+	"extends": "@tsconfig/node14/tsconfig.json",
+	"files": [
+		"index.js"
+	],
+	"compilerOptions": {
+		"allowJs": true,
+		"checkJs": true,
+		"noEmit": true,
+		"removeComments": true
+	}
+}