Skip to content

Commit 3ca7bcf

Browse files
parth0025claude
andcommitted
fix: remove duplicate CheckInstallStep mount and .env hot-reload (#41)
Two related sources of duplicate-handler accumulation in the bootstrap: 1. CheckInstallStep was mounted twice: once inside initializeControllers() and once unconditionally at the top level. The top-level mount has to stay because the install wizard must be reachable before MONGODB_URL is configured. Removed the inner mount so each startup registers it exactly once. 2. fs.watchFile on .env called initializeControllers() on every save, which re-ran .init(app) for ~60 route modules onto the same Express app instance (Express has no clean route un-registration) and spun another setInterval inside startInterval(). Removed the watcher; nodemon already restarts on file changes in dev, and production env changes require a process restart anyway. After this change initializeControllers() is only invoked once, from the MONGODB_URL startup gate. Closes #41 Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
1 parent 662715a commit 3ca7bcf

1 file changed

Lines changed: 0 additions & 6 deletions

File tree

index.js

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -140,7 +140,6 @@ function initializeControllers() {
140140
require('./Modules/notification-count/init').init(app);
141141
require('./Modules/notification/sendEmail/init').init(app);
142142
require('./Modules/trackerUserPermission/init').init(app);
143-
require('./Modules/CheckInstallStep/init').init(app);
144143
require('./Modules/SaasAdmin/init').init(app);
145144
if(process.env.NODE_ENV === "production") {
146145
require('./cron.js')
@@ -195,11 +194,6 @@ function initializeControllers() {
195194
require("./Modules/googleOAuth/init.js").init(app);
196195
}
197196

198-
// FIRES EVENT WHEN THE ENV IS UPDATED
199-
fs.watchFile(__dirname+'/.env', () => {
200-
initializeControllers();
201-
})
202-
203197
// SET MIDDLEWARE
204198
// require('./Config/setMiddleware.js').setMiddlewareWithC(app);
205199
// require('./Config/setMiddleware.js').setMiddleware(app);

0 commit comments

Comments
 (0)