fix: replace node-persist with minimal in-repo file storage

jest.config.ts

@@ -3,6 +3,7 @@ import type { Config } from "jest";
 const config: Config = {
   "preset": "ts-jest",
   "testEnvironment": "node",
+  "setupFilesAfterEnv": ["<rootDir>/jest.setup.ts"],
   "transform": {
     "^.+\\.tsx?$": ["ts-jest", {
       "tsconfig": "tsconfig.spec.json",

jest.setup.ts

@@ -0,0 +1,14 @@
+import fs from "node:fs";
+import os from "node:os";
+import path from "node:path";
+import { HAPStorage } from "./src/lib/model/HAPStorage";
+
+// Redirect the HAPStorage singleton into a fresh temporary directory for every test file,
+// so suites which exercise persistence (e.g. through Accessory.publish()) never touch
+// a real "persist" directory inside the repository.
+const storageDir = fs.mkdtempSync(path.join(os.tmpdir(), "hap-nodejs-jest-"));
+HAPStorage.setCustomStoragePath(storageDir);
+
+afterAll(() => {
+  fs.rmSync(storageDir, { recursive: true, force: true });
+});

package.json

@@ -14,7 +14,7 @@
     "check": "npm install && npm outdated",
     "clean": "rimraf dist && rimraf coverage",
     "lint": "eslint 'src/**/*.{js,ts,json}'",
-    "build": "rimraf dist && tsc && node .github/node-persist-ignore.js",
+    "build": "rimraf dist && tsc",
     "generate-definitions": "ts-node src/lib/definitions/generate-definitions.ts",
     "prepublishOnly": "npm run build",
     "postpublish": "npm run clean",
@@ -58,7 +58,6 @@
     "debug": "^4.4.3",
     "fast-srp-hap": "^2.0.4",
     "futoin-hkdf": "^1.5.3",
-    "node-persist": "^0.0.12",
     "source-map-support": "^0.5.21",
     "tslib": "^2.8.1",
     "tweetnacl": "^1.0.3"

src/index.ts

@@ -7,6 +7,7 @@ import { readFileSync } from "node:fs";
  * @group Utils
  */
 export * as uuid from "./lib/util/uuid";
+export * from "./lib/model/HAPFileStorage";
 export * from "./lib/model/HAPStorage";
 export * from "./lib/Accessory";
 export * from "./lib/Bridge";

src/lib/model/AccessoryInfo.spec.ts

@@ -6,8 +6,8 @@ import { AssertionError } from "assert";
 describe("AccessoryInfo", () => {
   describe("#load()", () => {
     it("should default category to Categories.OTHER when missing", () => {
-      const mockStorage = HAPStorage.storage();
-      (mockStorage.getItem as jest.Mock).mockReturnValueOnce({
+      const username = "0E:AE:FC:45:7B:91";
+      HAPStorage.storage().setItemSync(AccessoryInfo.persistKey(username), {
         displayName: "Test",
         pincode: "123-45-678",
         signSk: "aa".repeat(64),
@@ -16,7 +16,7 @@ describe("AccessoryInfo", () => {
         // category intentionally omitted
       });
 
-      const info = AccessoryInfo.load("0E:AE:FC:45:7B:91");
+      const info = AccessoryInfo.load(username);
       expect(info).not.toBeNull();
       expect(info!.category).toBe(Categories.OTHER);
       expect(typeof info!.category).toBe("number");

src/lib/model/AccessoryInfo.ts

@@ -230,7 +230,7 @@ export class AccessoryInfo {
       signPk: this.signPk.toString("hex"),
       pairedClients: {},
       // moving permissions into an extra object, so there is nothing to migrate from old files.
-      // if the legacy node-persist storage should be upgraded some time, it would be reasonable to combine the storage
+      // if the legacy storage format should be upgraded some time, it would be reasonable to combine the storage
       // of public keys (pairedClients object) and permissions.
       pairedClientsPermission: {},
       configVersion: this.configVersion,