convert to typescript

Author: silverwind

.eslintrc.yaml

@@ -1,2 +1,4 @@
 root: true
-extends: silverwind
+extends:
+  - silverwind
+  - silverwind-typescript

.github/workflows/ci.yaml

@@ -3,16 +3,13 @@ on: [push, pull_request]
 
 jobs:
   test:
-    strategy:
-      fail-fast: false
-      matrix:
-        node: [18, 20]
-        os: [ubuntu-latest, macos-latest, windows-latest]
-
     runs-on: ${{matrix.os}}
     steps:
-      - uses: actions/checkout@v3
-      - uses: actions/setup-node@v3
+      - uses: actions/checkout@v4
+      - uses: actions/setup-node@v4
+        with:
+          node-version: latest
+      - uses: oven-sh/setup-bun@v1
         with:
-          node-version: ${{matrix.node}}
+          bun-version: latest
       - run: make lint test

.gitignore

@@ -1,5 +1,4 @@
 /.vscode
+/dist
 /node_modules
 /npm-debug.log*
-/yarn-error.log
-/yarn.lock

Makefile

@@ -1,3 +1,6 @@
+SOURCE_FILES := index.ts
+DIST_FILES := dist/index.js
+
 node_modules: package-lock.json
 	npm install --no-save
 	@touch node_modules
@@ -7,19 +10,27 @@ deps: node_modules
 
 .PHONY: lint
 lint: node_modules
-	npx eslint --color .
+	npx eslint --ext js,jsx,ts,tsx --color .
+	npx tsc
 
 .PHONY: lint-fix
 lint-fix: node_modules
-	npx eslint --color . --fix
+	npx eslint --ext js,jsx,ts,tsx --color . --fix
+	npx tsc
 
 .PHONY: test
 test: node_modules
-	npx vitest
+	bun test
 
 .PHONY: test-update
 test-update: node_modules
-	npx vitest --update
+	bun test
+
+.PHONY: build
+build: node_modules $(DIST_FILES)
+
+$(DIST_FILES): $(SOURCE_FILES) package-lock.json vite.config.ts
+	npx vite build
 
 .PHONY: publish
 publish: node_modules
@@ -34,16 +45,16 @@ update: node_modules
 	@touch node_modules
 
 .PHONY: path
-patch: node_modules lint test
+patch: node_modules lint test build
 	npx versions patch package.json package-lock.json
 	@$(MAKE) --no-print-directory publish
 
 .PHONY: minor
-minor: node_modules lint test
+minor: node_modules lint test build
 	npx versions minor package.json package-lock.json
 	@$(MAKE) --no-print-directory publish
 
 .PHONY: major
-major: node_modules lint test
+major: node_modules lint test build
 	npx versions major package.json package-lock.json
 	@$(MAKE) --no-print-directory publish

child.js child.tschild.js renamed to child.ts

@@ -1,4 +1,4 @@
-import {daemonizeProcess} from "./index.js";
+import {daemonizeProcess} from "./index.ts";
 import {writeFileSync} from "node:fs";
 import {env, ppid} from "node:process";
 

index.js

@@ -5,11 +5,11 @@ import {readFile, copyFile, rm} from "node:fs/promises";
 import {join} from "node:path";
 
 const testDir = mkdtempSync(join(tmpdir(), "daemonize-process-"));
-const sleep = ms => new Promise(resolve => setTimeout(resolve, ms));
+const sleep = (ms: number) => new Promise(resolve => setTimeout(resolve, ms));
 
 beforeAll(async () => {
-  await copyFile(new URL("index.js", import.meta.url), join(testDir, "index.js"));
-  await copyFile(new URL("child.js", import.meta.url), join(testDir, "child.js"));
+  await copyFile(new URL("index.ts", import.meta.url), join(testDir, "index.ts"));
+  await copyFile(new URL("child.ts", import.meta.url), join(testDir, "child.ts"));
   await copyFile(new URL("package.json", import.meta.url), join(testDir, "package.json"));
 });
 
@@ -19,7 +19,7 @@ afterAll(async () => {
 
 test("simple", () => {
   return new Promise(resolve => {
-    const child = fork(join(testDir, "child.js"));
+    const child = fork(join(testDir, "child.ts"));
 
     child.on("exit", async () => {
       await sleep(1000); // give the child some time to exit
@@ -35,7 +35,7 @@ test("simple", () => {
 
       // verify that internal tracking variable is not leaked to the child
       expect(envVar).toEqual("false");
-      resolve();
+      resolve(undefined);
     });
   });
 });

index.test.js index.test.tsindex.test.js renamed to index.test.ts

@@ -0,0 +1,43 @@
+import {spawn} from "node:child_process";
+import {env, cwd, execPath, argv, exit} from "node:process";
+import type {SpawnOptions} from "node:child_process";
+
+const id = "_DAEMONIZE_PROCESS";
+
+type DaemonizeProcessOpts = {
+  /** The path to the script to be executed. Default: The current script. */
+  script?: string,
+  /** The command line arguments to be used. Default: The current arguments. */
+  arguments?: string[],
+  /** The path to the Node.js binary to be used. Default: The current Node.js binary. */
+  node?: string,
+  /** The exit code to be used when exiting the parent process. Default: `0`. */
+  exitCode?: number,
+} & SpawnOptions;
+
+export function daemonizeProcess(opts: DaemonizeProcessOpts = {}) {
+  if (id in env) {
+    // In the child, clean up the tracking environment variable
+    delete env[id];
+  } else {
+    // In the parent, set the tracking environment variable, fork the child and exit
+    const o: DaemonizeProcessOpts = {
+      // spawn options
+      env: Object.assign(env, opts.env, {[id]: "1"}),
+      cwd: cwd(),
+      stdio: "ignore",
+      detached: true,
+      // custom options
+      node: execPath,
+      script: argv[1],
+      arguments: argv.slice(2),
+      exitCode: 0,
+      ...opts,
+    };
+
+    const args: string[] = [o.script as string, ...(o.arguments as string[])];
+    const proc: any = spawn(o.node as string, args, o);
+    proc?.unref?.();
+    exit(o.exitCode);
+  }
+}