fix: change SQLTagStore.size from method to getter for API parity with node.js

Author: mceachen

CHANGELOG.md

@@ -2,6 +2,14 @@
 
 All notable changes to this project will be documented in this file.
 
+## [0.3.0] (2025-12-16)
+
+### Changed
+
+- **BREAKING**: `SQLTagStore.size` changed from method to getter for Node.js API parity (Node.js PR #60246)
+  - Before: `sql.size()`
+  - After: `sql.size`
+
 ## [0.2.1] (2025-12-01)
 
 ### Added
@@ -75,5 +83,7 @@ All notable changes to this project will be documented in this file.
 - macOS (x64, ARM64)
 - Linux (x64, ARM64), (glibc 2.28+, musl)
 
+[0.3.0]: https://github.com/PhotoStructure/node-sqlite/releases/tag/v0.3.0
+[0.2.1]: https://github.com/PhotoStructure/node-sqlite/releases/tag/v0.2.1
 [0.2.0]: https://github.com/PhotoStructure/node-sqlite/releases/tag/v0.2.0
 [0.0.1]: https://github.com/PhotoStructure/node-sqlite/releases/tag/v0.0.1

src/sql-tag-store.ts

@@ -49,7 +49,7 @@ export class SQLTagStore {
   /**
    * Returns the current number of cached statements.
    */
-  size(): number {
+  get size(): number {
     return this.cache.size();
   }
 

src/types/sql-tag-store-instance.ts

@@ -12,7 +12,7 @@ export interface SQLTagStoreInstance {
   /**
    * Returns the current number of cached statements.
    */
-  size(): number;
+  readonly size: number;
   /**
    * Clears all cached statements.
    */

test/sql-tag-store.test.ts

@@ -91,26 +91,26 @@ describe("SQLTagStore Tests", () => {
 
   test("TagStore capacity, size, and clear", () => {
     expect(sql.capacity).toBe(10);
-    expect(sql.size()).toBe(0);
+    expect(sql.size).toBe(0);
 
     expect(sql.run`INSERT INTO foo (text) VALUES (${"one"})`.changes).toBe(1);
-    expect(sql.size()).toBe(1);
+    expect(sql.size).toBe(1);
 
     expect(sql.get`SELECT * FROM foo WHERE text = ${"one"}`).toBeDefined();
-    expect(sql.size()).toBe(2);
+    expect(sql.size).toBe(2);
 
     // Using the same template string shouldn't increase the size
     expect(sql.get`SELECT * FROM foo WHERE text = ${"two"}`).toBeUndefined();
-    expect(sql.size()).toBe(2);
+    expect(sql.size).toBe(2);
 
     expect(
       (sql.all`SELECT * FROM foo` as Array<{ id: number; text: string }>)
         .length,
     ).toBe(1);
-    expect(sql.size()).toBe(3);
+    expect(sql.size).toBe(3);
 
     sql.clear();
-    expect(sql.size()).toBe(0);
+    expect(sql.size).toBe(0);
     expect(sql.capacity).toBe(10);
   });
 
@@ -141,7 +141,7 @@ describe("SQLTagStore Tests", () => {
 
   test("evicts finalized statements from cache", () => {
     expect(sql.run`INSERT INTO foo (text) VALUES (${"test"})`.changes).toBe(1);
-    expect(sql.size()).toBe(1);
+    expect(sql.size).toBe(1);
 
     // Get the cached statement through a different path - directly from db
     const stmt = db.prepare("INSERT INTO foo (text) VALUES (?)");
@@ -156,16 +156,16 @@ describe("SQLTagStore Tests", () => {
     expect(sql3.run`INSERT INTO foo (text) VALUES (${"cached"})`.changes).toBe(
       1,
     );
-    expect(sql3.size()).toBe(1);
+    expect(sql3.size).toBe(1);
 
     // Get a direct reference to a statement and finalize
     // Note: The cached statement is not directly accessible, so we test via size
     sql3.clear();
-    expect(sql3.size()).toBe(0);
+    expect(sql3.size).toBe(0);
 
     // Re-run should create a new statement
     expect(sql3.run`INSERT INTO foo (text) VALUES (${"new"})`.changes).toBe(1);
-    expect(sql3.size()).toBe(1);
+    expect(sql3.size).toBe(1);
   });
 
   test("LRU eviction when at capacity", () => {
@@ -183,18 +183,18 @@ describe("SQLTagStore Tests", () => {
       1,
     );
     expect(smallSql.get`SELECT * FROM foo WHERE id = ${1}`).toBeDefined();
-    expect(smallSql.size()).toBe(3);
+    expect(smallSql.size).toBe(3);
 
     // Add a 4th - should evict the oldest (first INSERT into foo)
     expect(smallSql.all`SELECT * FROM bar`.length).toBe(1);
-    expect(smallSql.size()).toBe(3);
+    expect(smallSql.size).toBe(3);
 
     // Verify LRU behavior - the INSERT into foo should be evicted
     // Accessing it again should add it back
     expect(smallSql.run`INSERT INTO foo (text) VALUES (${"c"})`.changes).toBe(
       1,
     );
-    expect(smallSql.size()).toBe(3);
+    expect(smallSql.size).toBe(3);
   });
 
   test("statement finalized property works", () => {