Auto-install npm dependencies on startup to fix Cannot find module 'whatsapp-web.js'

[Copilot]

The bot process crashes immediately on startup because node_modules/ is never present in a fresh clone — npm install must be run manually before node whatsapp_bot.js is invocable.

Changes

• app.py — ensure_npm_deps(): New function that checks for node_modules/ and runs npm install automatically if missing. Handles npm not found, non-zero exit (logs combined stdout+stderr), and 5-minute timeout.

• app.py — ensure_bot_file(): Now calls ensure_npm_deps() when whatsapp_bot.js is present, so dependencies are guaranteed installed before BotMonitor.start() launches the Node process.

• .gitignore: Added to exclude node_modules/, .wwebjs_auth/, .wwebjs_cache/, Python bytecode, runtime databases, and .env files — none of which should be committed.

# Before starting the bot, ensure npm deps are present
def ensure_npm_deps() -> None:
    node_modules = APP_DIR / "node_modules"
    if (APP_DIR / "package.json").exists() and not node_modules.exists():
        log.info("node_modules bulunamadı, 'npm install' çalıştırılıyor…")
        result = subprocess.run(['npm', 'install'], cwd=str(APP_DIR), ...)
        # logs success or combined stdout+stderr on failure

---

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

[eeea2222]

ok

[Copilot]

Pull request overview

This PR aims to prevent the WhatsApp bot from crashing on fresh clones by ensuring Node.js dependencies are installed before the Node bot process is started.

Changes:

• Added ensure_npm_deps() to auto-run npm install when node_modules/ is missing.
• Updated ensure_bot_file() to invoke ensure_npm_deps() when whatsapp_bot.js exists.
• Introduced a repository .gitignore to exclude Node/Python/runtime artifacts from version control.

Reviewed changes

Copilot reviewed 1 out of 2 changed files in this pull request and generated 3 comments.

File	Description
app.py	Adds startup-time npm dependency installation and wires it into bot startup checks.
.gitignore	Ignores Node dependencies, WhatsApp session/cache data, Python artifacts, and runtime files.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

ensure_npm_deps() logs npm install failures (non-zero exit, timeout, or missing npm) but still returns normally, so callers will proceed to start the bot and hit the same “Cannot find module …” crash. Consider making this function fail-fast (raise an exception or return a boolean that ensure_bot_file()/bot_start checks) so startup is aborted when dependencies cannot be installed.

[Copilot]

ensure_bot_file() logs that whatsapp_bot.js is missing but still returns bot_path, and /api/bot/start calls bot.start() unconditionally afterward. This leads to a confusing follow-on failure; consider raising FileNotFoundError (or another explicit exception) when the file is absent so callers can stop early and return a clear error.

Suggested change

	log.error(f"whatsapp_bot.js bulunamadı: {bot_path}")
	log.error("Bot başlatılamıyor. Dosyayı proje dizinine kopyalayın.")
	else:
	ensure_npm_deps()
	msg = f"whatsapp_bot.js bulunamadı: {bot_path}"
	log.error(msg)
	log.error("Bot başlatılamıyor. Dosyayı proje dizinine kopyalayın.")
	raise FileNotFoundError(msg)
	ensure_npm_deps()

[Copilot]

ensure_npm_deps() can be reached from the threaded Flask server (/api/bot/start) and potentially by multiple gunicorn workers; as written, concurrent calls can launch multiple npm install processes against the same directory, risking corrupted/partial installs. Consider adding a process-level/thread-level lock (and ideally an inter-process file lock) around the install path and re-checking node_modules after acquiring the lock.