chore(tests): sync node

Author: mceachen

scripts/sync-node-tests.ts

@@ -59,6 +59,17 @@ const skipTests: Record<string, Array<{ name: string; reason: string }>> = {
       reason: "Worker thread changeset serialization issue",
     },
   ],
+  "test-sqlite-template-tag.js": [
+    {
+      name: "a tag store keeps the database alive by itself",
+      reason: "Requires --expose-gc flag",
+    },
+    {
+      name: "tag store prevents circular reference leaks",
+      reason:
+        "Requires --expose-gc flag and Node.js internal GC test utilities",
+    },
+  ],
 };
 
 /**

test/node-compat/README.md

@@ -18,4 +18,4 @@ node --test test/node-compat/test-sqlite-statement-sync-columns.test.js
 ```
 
 Source: https://github.com/nodejs/node/tree/main/test/parallel
-Commit: 6beb65e00ef197fdfc392f79e7836bc1761bfd3d
+Commit: 6c44d31e2522021b724096939bc44defa42ec6a5

test/node-compat/test-sqlite-template-tag.test.js

@@ -9,6 +9,8 @@
  */
 
 "use strict";
+// Flags: --expose-gc
+
 const assert = require("assert");
 const { DatabaseSync } = require("@photostructure/sqlite");
 const { test, beforeEach } = require("node:test");
@@ -193,3 +195,39 @@ test("sql error messages are descriptive", () => {
     },
   );
 });
+
+test.skip("a tag store keeps the database alive by itself" /* Requires --expose-gc flag */, () => {
+  const sql = new DatabaseSync(":memory:").createTagStore();
+
+  sql.db.exec("CREATE TABLE test (data INTEGER)");
+
+  global.gc();
+
+  // eslint-disable-next-line no-unused-expressions
+  sql.run`INSERT INTO test (data) VALUES (1)`;
+});
+
+test.skip("tag store prevents circular reference leaks" /* Requires --expose-gc flag and Node.js internal GC test utilities */, async () => {
+  const { gcUntil } = require("../common/gc");
+
+  const before = process.memoryUsage().heapUsed;
+
+  // Create many SQLTagStore + DatabaseSync pairs with circular references
+  for (let i = 0; i < 1000; i++) {
+    const sql = new DatabaseSync(":memory:").createTagStore();
+    sql.db.exec("CREATE TABLE test (data INTEGER)");
+    // eslint-disable-next-line no-void
+    sql.db.setAuthorizer(() => void sql.db);
+  }
+
+  // GC until memory stabilizes or give up after 20 attempts
+  await gcUntil(
+    "tag store leak check",
+    () => {
+      const after = process.memoryUsage().heapUsed;
+      // Memory should not grow significantly (allow 50% margin for noise)
+      return after < before * 1.5;
+    },
+    20,
+  );
+});