n8n-nodes-variable 1.0.8 → 1.0.9

package/README.md

@@ -12,7 +12,7 @@ The **Variable** node lets you store, retrieve, update, and delete named variabl
 - **Workflow Global** — persists across executions using n8n's workflow static data
 - **Node Local** — persists for the specific node instance
 - **Custom Namespace** — workflow global storage with a fully dynamic namespace string (great for per-user / per-guild data)
-- **Cross-Workflow (Shared)** — variables are stored in a local SQLite database and shared across **all** workflows on this n8n instance
+- **Cross-Workflow (Shared)** — variables are stored in a local JSON file and shared across **all** workflows on this n8n instance
 
 ---
 
@@ -97,15 +97,15 @@ guild_{{$json.guild.id}} → per-guild settings
 
 ### Cross-Workflow (Shared)
 
-Variables are stored in a SQLite database file on the n8n host at:
+Variables are stored in a JSON file on the n8n host at:
 
 ```
-${N8N_USER_FOLDER ?? ~/.n8n}/n8n-nodes-variable.db
+${N8N_USER_FOLDER ?? ~/.n8n}/n8n-nodes-variable-data.json
 ```
 
-The database and table are **created automatically on first use** — no setup required. Data survives instance restarts and is accessible from any workflow on the same n8n instance. This is ideal for cross-workflow counters, shared feature flags, or any state that multiple workflows need to read and write.
+The file is **created automatically on first use** — no setup or dependencies required. Writes are performed atomically (write to a temporary file, then rename) to prevent corruption. Data survives instance restarts and is accessible from any workflow on the same n8n instance.
 
-> **Note:** the database lives on the n8n host machine. If you run n8n in a container or cloud environment, ensure the `.n8n` data directory is persisted to a volume so data is not lost on container restarts.
+> **Note:** the file lives on the n8n host machine. If you run n8n in a container or cloud environment, ensure the `.n8n` data directory is persisted to a volume so data is not lost on container restarts.
 
 ---
 

package/dist/nodes/Variable/helpers/dbStorage.d.ts

@@ -1,11 +1,12 @@
 /**
- * Cross-workflow variable storage backed by a local SQLite database.
+ * Cross-workflow variable storage backed by a local JSON file.
  *
- * The database is auto-created at:
- *   ${N8N_USER_FOLDER ?? ~/.n8n}/n8n-nodes-variable.db
+ * The file is auto-created at:
+ *   ${N8N_USER_FOLDER ?? ~/.n8n}/n8n-nodes-variable-data.json
  *
- * No configuration is required — the database file and table are created
- * automatically on first use and shared across all workflows on the instance.
+ * No configuration is required. Writes are performed atomically (write to a
+ * temporary file, then rename) to prevent data corruption on unexpected
+ * process exit. No native dependencies are needed.
  */
 import type { StoredVariableEntry } from './types';
 export declare function dbGetVariable(namespace: string, key: string): StoredVariableEntry | undefined;

package/dist/nodes/Variable/helpers/dbStorage.js

@@ -1,12 +1,13 @@
 "use strict";
 /**
- * Cross-workflow variable storage backed by a local SQLite database.
+ * Cross-workflow variable storage backed by a local JSON file.
  *
- * The database is auto-created at:
- *   ${N8N_USER_FOLDER ?? ~/.n8n}/n8n-nodes-variable.db
+ * The file is auto-created at:
+ *   ${N8N_USER_FOLDER ?? ~/.n8n}/n8n-nodes-variable-data.json
  *
- * No configuration is required — the database file and table are created
- * automatically on first use and shared across all workflows on the instance.
+ * No configuration is required. Writes are performed atomically (write to a
+ * temporary file, then rename) to prevent data corruption on unexpected
+ * process exit. No native dependencies are needed.
  */
 var __importDefault = (this && this.__importDefault) || function (mod) {
     return (mod && mod.__esModule) ? mod : { "default": mod };
@@ -18,12 +19,11 @@ exports.dbDeleteVariable = dbDeleteVariable;
 exports.dbHasVariable = dbHasVariable;
 exports.dbListVariables = dbListVariables;
 exports.dbClearNamespace = dbClearNamespace;
-const better_sqlite3_1 = __importDefault(require("better-sqlite3"));
 const fs_1 = __importDefault(require("fs"));
 const os_1 = __importDefault(require("os"));
 const path_1 = __importDefault(require("path"));
-const DB_FILENAME = 'n8n-nodes-variable.db';
-function getDbPath() {
+const DATA_FILENAME = 'n8n-nodes-variable-data.json';
+function getDataPath() {
     const userFolder = process.env['N8N_USER_FOLDER'] ?? path_1.default.join(os_1.default.homedir(), '.n8n');
     try {
         if (!fs_1.default.existsSync(userFolder)) {
@@ -31,107 +31,98 @@ function getDbPath() {
         }
     }
     catch {
-        // If we can't create the n8n folder, fall back to the OS temp directory
-        return path_1.default.join(os_1.default.tmpdir(), DB_FILENAME);
+        // If the n8n user folder can't be created, fall back to the OS temp dir
+        return path_1.default.join(os_1.default.tmpdir(), DATA_FILENAME);
     }
-    return path_1.default.join(userFolder, DB_FILENAME);
+    return path_1.default.join(userFolder, DATA_FILENAME);
 }
-// Singleton connection — reused across all node executions in the same process
-let _db = null;
-function getDb() {
-    if (_db)
-        return _db;
-    const dbPath = getDbPath();
-    _db = new better_sqlite3_1.default(dbPath);
-    // WAL journal mode allows concurrent reads alongside writes
-    _db.pragma('journal_mode = WAL');
-    _db.pragma('foreign_keys = ON');
-    // Auto-create the table if it doesn't exist
-    _db.exec(`
-    CREATE TABLE IF NOT EXISTS variables (
-      namespace  TEXT NOT NULL,
-      key        TEXT NOT NULL,
-      value      TEXT NOT NULL,
-      type       TEXT,
-      created_at TEXT,
-      updated_at TEXT,
-      PRIMARY KEY (namespace, key)
-    )
-  `);
-    return _db;
+function readData() {
+    const dataPath = getDataPath();
+    try {
+        const content = fs_1.default.readFileSync(dataPath, 'utf-8');
+        const parsed = JSON.parse(content);
+        if (typeof parsed === 'object' && parsed !== null && !Array.isArray(parsed)) {
+            return parsed;
+        }
+    }
+    catch {
+        // File doesn't exist yet or is corrupt — start fresh
+    }
+    return {};
+}
+function writeData(data) {
+    const dataPath = getDataPath();
+    const tmpPath = `${dataPath}.tmp`;
+    fs_1.default.writeFileSync(tmpPath, JSON.stringify(data), 'utf-8');
+    fs_1.default.renameSync(tmpPath, dataPath);
 }
 // ─── CRUD operations ─────────────────────────────────────────────────────────
 function dbGetVariable(namespace, key) {
-    const db = getDb();
-    const row = db
-        .prepare('SELECT value, type, created_at, updated_at FROM variables WHERE namespace = ? AND key = ?')
-        .get(namespace, key);
-    if (!row)
+    const data = readData();
+    const entry = data[namespace]?.[key];
+    if (!entry)
         return undefined;
     return {
-        value: JSON.parse(row.value),
-        type: row.type ?? undefined,
-        createdAt: row.created_at ?? undefined,
-        updatedAt: row.updated_at ?? undefined,
+        value: entry.value,
+        type: entry.type,
+        createdAt: entry.created_at,
+        updatedAt: entry.updated_at,
     };
 }
 function dbSetVariable(namespace, key, value, typeName, includeMetadata) {
-    const db = getDb();
+    const data = readData();
+    if (!data[namespace]) {
+        data[namespace] = {};
+    }
     const now = new Date().toISOString();
-    const serialized = JSON.stringify(value);
     if (includeMetadata) {
-        // Preserve the original created_at on conflict
-        db.prepare(`INSERT INTO variables (namespace, key, value, type, created_at, updated_at)
-       VALUES (?, ?, ?, ?, ?, ?)
-       ON CONFLICT(namespace, key) DO UPDATE SET
-         value      = excluded.value,
-         type       = excluded.type,
-         updated_at = excluded.updated_at`).run(namespace, key, serialized, typeName, now, now);
+        const existing = data[namespace][key];
+        data[namespace][key] = {
+            value,
+            type: typeName,
+            // Preserve original created_at on updates
+            created_at: existing?.created_at ?? now,
+            updated_at: now,
+        };
     }
     else {
-        db.prepare(`INSERT INTO variables (namespace, key, value, type, created_at, updated_at)
-       VALUES (?, ?, ?, NULL, NULL, NULL)
-       ON CONFLICT(namespace, key) DO UPDATE SET
-         value      = excluded.value,
-         type       = NULL,
-         created_at = NULL,
-         updated_at = NULL`).run(namespace, key, serialized);
+        data[namespace][key] = { value };
     }
+    writeData(data);
 }
 function dbDeleteVariable(namespace, key) {
-    const db = getDb();
-    const result = db
-        .prepare('DELETE FROM variables WHERE namespace = ? AND key = ?')
-        .run(namespace, key);
-    return result.changes > 0;
+    const data = readData();
+    if (!data[namespace] || !(key in data[namespace])) {
+        return false;
+    }
+    delete data[namespace][key];
+    writeData(data);
+    return true;
 }
 function dbHasVariable(namespace, key) {
-    const db = getDb();
-    const row = db
-        .prepare('SELECT 1 FROM variables WHERE namespace = ? AND key = ? LIMIT 1')
-        .get(namespace, key);
-    return row !== undefined;
+    const data = readData();
+    return Object.prototype.hasOwnProperty.call(data[namespace] ?? {}, key);
 }
 function dbListVariables(namespace) {
-    const db = getDb();
-    const rows = db
-        .prepare('SELECT key, value, type, created_at, updated_at FROM variables WHERE namespace = ? ORDER BY key')
-        .all(namespace);
+    const data = readData();
+    const ns = data[namespace] ?? {};
     const result = {};
-    for (const row of rows) {
-        result[row.key] = {
-            value: JSON.parse(row.value),
-            type: row.type ?? undefined,
-            createdAt: row.created_at ?? undefined,
-            updatedAt: row.updated_at ?? undefined,
+    for (const [k, entry] of Object.entries(ns)) {
+        result[k] = {
+            value: entry.value,
+            type: entry.type,
+            createdAt: entry.created_at,
+            updatedAt: entry.updated_at,
         };
     }
     return result;
 }
 function dbClearNamespace(namespace) {
-    const db = getDb();
-    const result = db
-        .prepare('DELETE FROM variables WHERE namespace = ?')
-        .run(namespace);
-    return result.changes;
+    const data = readData();
+    const count = Object.keys(data[namespace] ?? {}).length;
+    if (count > 0) {
+        delete data[namespace];
+        writeData(data);
+    }
+    return count;
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "n8n-nodes-variable",
-  "version": "1.0.8",
+  "version": "1.0.9",
   "description": "Local and global scoped variables for n8n workflows",
   "keywords": [
     "n8n-community-node-package",
@@ -38,7 +38,6 @@
     ]
   },
   "devDependencies": {
-    "@types/better-sqlite3": "^7.6.11",
     "@types/jest": "^29.5.12",
     "@types/node": "^18.19.50",
     "@typescript-eslint/eslint-plugin": "^6.21.0",
@@ -55,9 +54,6 @@
   "peerDependencies": {
     "n8n-workflow": "*"
   },
-  "dependencies": {
-    "better-sqlite3": "^9.4.0"
-  },
   "jest": {
     "preset": "ts-jest",
     "testEnvironment": "node",