misc: some comments updated

Author: yash-pouranik

backend/app.js

@@ -46,7 +46,7 @@ if (process.env.NODE_ENV !== 'test') {
 }
 
 
-// IMPORTS 
+// rate limiter and loggerr IMPORTS 
 const { limiter, logger } = require('./middleware/api_usage');
 
 // Route Imports
@@ -59,11 +59,7 @@ const storageRoute = require('./routes/storage');
 // ROUTES SETUP 
 app.use('/api/auth', dashboardLimiter, authRoute); // Developer Auth
 app.use('/api/projects', dashboardLimiter, projectRoute); // Project Mgmt
-
-// Logger added to userAuth route
 app.use('/api/userAuth', limiter, logger, userAuthRoute);
-
-// Data & Storage Routes (Protected)
 app.use('/api/data', limiter, cors(adminCorsOptions), logger, dataRoute);
 app.use('/api/storage', limiter, cors(adminCorsOptions), logger, storageRoute);
 
@@ -72,7 +68,7 @@ app.get('/', (req, res) => {
     res.status(200).json({ status: "success", message: "urBackend API is running 🚀" })
 });
 
-//  FAULT TOLERANCE: Global Error Handler
+// Global Error Handler
 app.use((err, req, res, next) => {
     if (err instanceof SyntaxError && err.status === 400 && 'body' in err) {
         return res.status(400).json({
@@ -87,7 +83,7 @@ app.use((err, req, res, next) => {
         message: err.message
     });
 });
-//  DB CONNECTION & SERVER START 
+//  DB and server initialization
 // (Only connect if NOT in Test Mode)
 if (process.env.NODE_ENV !== 'test') {
 
@@ -121,7 +117,7 @@ if (process.env.NODE_ENV !== 'test') {
         console.log(`Server running on port ${PORT}`);
     });
 
-    // 🛡️ GRACEFUL SHUTDOWN
+    // handle gracefll shutdwn
     const gracefulShutdown = async () => {
         console.log('🛑 SIGTERM/SIGINT received. Shutting down gracefully...');
 

backend/controllers/data.controller.js

@@ -4,10 +4,10 @@ const Project = require("../models/Project");
 const { getConnection } = require("../utils/connection.manager");
 const { getCompiledModel } = require("../utils/injectModel");
 
-// Helper: CodeQL ko satisfy karne ke liye ID validate karna zaroori hai
+// Validate MongoDB ObjectId
 const isValidId = (id) => mongoose.Types.ObjectId.isValid(id);
 
-// 1. INSERT DATA
+// INSERT DATA
 module.exports.insertData = async (req, res) => {
     try {
         const { collectionName } = req.params;
@@ -38,7 +38,7 @@ module.exports.insertData = async (req, res) => {
             }
         }
 
-        // Sanitize removes MongoDB operators like $ne, $gt etc.
+        // Prevnt NoSQL injection
         const safeData = sanitize(cleanData);
 
         let docSize = 0;
@@ -65,7 +65,7 @@ module.exports.insertData = async (req, res) => {
     }
 };
 
-// 2. GET ALL DATA
+// GET ALL DATA
 module.exports.getAllData = async (req, res) => {
     try {
         const { collectionName } = req.params;
@@ -84,13 +84,13 @@ module.exports.getAllData = async (req, res) => {
     }
 };
 
-// 3. GET SINGLE DOC
+// GET SINGLE DOC
 module.exports.getSingleDoc = async (req, res) => {
     try {
         const { collectionName, id } = req.params;
         const project = req.project;
 
-        // CodeQL Fix: User input ID ko pehle validate karo
+        // ensure valid mongose objct id
         if (!isValidId(id)) return res.status(400).json({ error: "Invalid ID format." });
 
         const collectionConfig = project.collections.find(c => c.name === collectionName);
@@ -108,7 +108,7 @@ module.exports.getSingleDoc = async (req, res) => {
     }
 };
 
-// 4. UPDATE DATA
+// UPDATE DATA
 module.exports.updateSingleData = async (req, res) => {
     try {
         const { collectionName, id } = req.params;
@@ -149,7 +149,7 @@ module.exports.updateSingleData = async (req, res) => {
     }
 };
 
-// 5. DELETE DATA
+// DELETE DATA
 module.exports.deleteSingleDoc = async (req, res) => {
     try {
         const { collectionName, id } = req.params;

backend/utils/injectModel.js

@@ -34,30 +34,30 @@ function getCompiledModel(connection, collectionData, projectId, isExternal) {
     }
 
 
-    // 1. Get per-connection cache
+    // Get per-connection cache
     if (!modelRegistry.has(connection)) {
         modelRegistry.set(connection, new Map());
     }
 
     const connectionModels = modelRegistry.get(connection);
 
-    // 2. If already compiled for THIS connection
+    // If already compiled for THIS connection
     if (connectionModels.has(collectionName)) {
         return connectionModels.get(collectionName);
     }
 
-    // 3. If model already exists on connection (edge case)
+    // If model already exists on connection (edge case)
     if (connection.models[collectionName]) {
         const existingModel = connection.models[collectionName];
         connectionModels.set(collectionName, existingModel);
         return existingModel;
     }
 
-    // 4. Build schema + compile
+    // Build schema + compile
     const schema = buildMongooseSchema(collectionData.model);
     const model = connection.model(collectionName, schema);
 
-    // 5. Cache it
+    // Cache it
     connectionModels.set(collectionName, model);
 
     return model;

frontend/src/App.jsx

@@ -12,21 +12,12 @@ import CreateCollection from './pages/CreateCollection';
 import NotFound from './pages/NotFound';
 import Analytics from './pages/Analytics';
 
-// 1. Import Database Page
 import Database from './pages/Database';
-
-// 2. Import Storage Page
 import Storage from './pages/Storage';
-
-// 3. Import Docs Page
 import Docs from './pages/Docs';
-
 import Auth from './pages/Auth';
 import OtpVerification from './pages/OtpVerification';
-
-
 import Settings from './pages/Settings';
-
 import ProjectSettings from './pages/ProjectSettings';
 
 
@@ -77,7 +68,6 @@ function App() {
           </ProtectedRoute>
         } />
 
-        {/* 2. Database Route */}
         <Route path="/project/:projectId/database" element={
           <ProtectedRoute>
             <MainLayout>
@@ -86,15 +76,14 @@ function App() {
           </ProtectedRoute>
         } />
 
-        {/* 2. Register Storage Route */}
         <Route path="/project/:projectId/storage" element={<ProtectedRoute><MainLayout><Storage /></MainLayout></ProtectedRoute>} />
 
-        {/* 3. Register Docs Route */}
+
         <Route path="/docs" element={
           <Docs />
         } />
 
-        {/* 2. Add Analytics Route */}
+
         <Route path="/project/:projectId/analytics" element={
           <ProtectedRoute>
             <MainLayout>
@@ -103,13 +92,10 @@ function App() {
           </ProtectedRoute>
         } />
 
-        {/* 2. Register Auth Route */}
         <Route path="/project/:projectId/auth" element={<ProtectedRoute><MainLayout><Auth /></MainLayout></ProtectedRoute>} />
 
-        {/* 2. Register Settings Route */}
         <Route path="/settings" element={<ProtectedRoute><MainLayout><Settings /></MainLayout></ProtectedRoute>} />
 
-        {/* Project Specific Settings (New) */}
         <Route path="/project/:projectId/settings" element={<ProtectedRoute><MainLayout><ProjectSettings /></MainLayout></ProtectedRoute>} />
 
 
@@ -121,7 +107,6 @@ function App() {
           </ProtectedRoute>
         } />
 
-        {/* 2. Catch-all 404 Route (Sabse last mein hona chahiye) */}
         <Route path="*" element={<NotFound />} />
 
       </Routes>