Merge pull request #71 from ayash911/feature/gate-debug-timers

Gate console.time/timeEnd performance logs behind DEBUG flag in data …

Author: yash-pouranik

apps/public-api/package.json

@@ -12,9 +12,9 @@
     "testEnvironment": "node"
   },
   "dependencies": {
-    "@urbackend/common": "*",
     "@kiroo/sdk": "^0.1.2",
     "@supabase/supabase-js": "^2.84.0",
+    "@urbackend/common": "*",
     "bcryptjs": "^3.0.2",
     "bullmq": "^5.70.1",
     "cookie-parser": "^1.4.7",
@@ -29,6 +29,8 @@
     "resend": "^6.6.0",
     "uuid": "^9.0.1",
     "zod": "^4.1.13"
+  },
+  "devDependencies": {
+    "jest": "^30.3.0"
   }
 }
-

apps/public-api/src/controllers/data.controller.js

@@ -5,6 +5,9 @@ const { getConnection } = require("@urbackend/common");
 const { getCompiledModel } = require("@urbackend/common");
 const { QueryEngine } = require("@urbackend/common");
 const { validateData, validateUpdateData } = require("@urbackend/common");
+const { performance } = require('perf_hooks');
+
+const isDebug = process.env.DEBUG === 'true';
 
 // Validate MongoDB ObjectId
 const isValidId = (id) => mongoose.Types.ObjectId.isValid(id);
@@ -16,7 +19,8 @@ const isDuplicateKeyError = (err) => {
 // INSERT DATA
 module.exports.insertData = async (req, res) => {
   try {
-    console.time("insert data");
+    let start;
+    if (isDebug) start = performance.now();
     const { collectionName } = req.params;
     const project = req.project;
 
@@ -60,7 +64,7 @@ module.exports.insertData = async (req, res) => {
       );
     }
 
-    console.timeEnd("insert data");
+    if (isDebug) console.log(`[DEBUG] insert data took ${(performance.now() - start).toFixed(2)}ms`);
     res.status(201).json(result);
   } catch (err) {
     console.error(err);
@@ -79,7 +83,8 @@ module.exports.insertData = async (req, res) => {
 // GET ALL DATA
 module.exports.getAllData = async (req, res) => {
   try {
-    console.time("getall");
+    let start;
+    if (isDebug) start = performance.now();
     const { collectionName } = req.params;
     const project = req.project;
 
@@ -103,7 +108,7 @@ module.exports.getAllData = async (req, res) => {
       .paginate();
 
     const data = await features.query.lean();
-    console.timeEnd("getall");
+    if (isDebug) console.log(`[DEBUG] getall took ${(performance.now() - start).toFixed(2)}ms`);
     res.json(data);
   } catch (err) {
     console.error(err);