From 4c888b05003704b239cc2aae34c46b8d9bf0aa12 Mon Sep 17 00:00:00 2001
From: DeNNiiInc <dennii.inc@gmail.com>
Date: Mon, 29 Dec 2025 10:50:16 +1100
Subject: [PATCH] Fix database insert - add user_ip column

---
 lib/runner.js | 17 +++++++----------
 1 file changed, 7 insertions(+), 10 deletions(-)

diff --git a/lib/runner.js b/lib/runner.js
index e9c52d0..1c80731 100644
--- a/lib/runner.js
+++ b/lib/runner.js
@@ -243,24 +243,21 @@ async function _executeTest(url, options) {
     // Prepare Database Insert
     const insertQuery = `
       INSERT INTO test_results (
-        id, url, user_uuid, is_mobile, 
+        id, url, user_uuid, user_ip, is_mobile, 
         scores, metrics, filmstrip, timestamp
-      ) VALUES ($1, $2, $3, $4, $5, $6, $7, NOW())
+      ) VALUES ($1, $2, $3, $4, $5, $6, $7, $8, NOW())
       RETURNING id
     `;
     
-    // Convert objects to JSON strings for DB text columns if needed, 
-    // or rely on pg to stringify objects for JSONB columns if schema uses JSONB.
-    // Based on previous errors, schema might be expecting JSON.
-    
     const values = [
       testId, 
       lhr.finalUrl, 
-      options.userUuid || null, // Ensure options is available or pass it in
+      options.userUuid || null,
+      options.userIp || null,
       isMobile,
-      JSON.stringify(summary.scores),
-      JSON.stringify(summary.metrics),
-      JSON.stringify(filmstripData)
+      summary.scores,
+      summary.metrics,
+      filmstripData
     ];
 
     try {