feat: add database connection pool and timeout configurations

[nevil-mathew]

Summary by CodeRabbit

• Chores
  • Added several new environment configuration keys for database pooling and timeout settings, each with default values, optional flags, and explanatory messages.
  • Integrated those new configuration keys into the app so they are available at runtime.
  • APP_NAME default remains unchanged.

[coderabbitai]

Walkthrough

Adds database connection pool and Postgres timeout environment variables to the environment configuration; APP_NAME default remains "MentorED".

Changes

Cohort / File(s)	Summary
Database Configuration src/envVariables.js	Added databaseEnvironmentVariables containing DB_POOL_MAX, DB_POOL_MIN, DB_POOL_ACQUIRE_MS, DB_POOL_IDLE_MS, DB_POOL_EVICT_MS, PG_STATEMENT_TIMEOUT_MS, PG_IDLE_TX_TIMEOUT_MS; merged these into environmentVariables; APP_NAME default unchanged ("MentorED").

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

• Verify defaults and optional flags for each new env var.
• Confirm the spread merge of databaseEnvironmentVariables into environmentVariables does not overwrite or reorder unrelated keys.
• Run/adjust any tests or docs that reference APP_NAME if present.

Poem

🐰 I hop through configs, neat and spry,

Pool sizes, timeouts set on high,
Defaults tucked in a cozy bed,
MentorED hums — carrot dreams ahead 🥕

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	You can run @coderabbitai generate docstrings to improve docstring coverage.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title Check	✅ Passed	The pull request title "feat: add database connection pool and timeout configurations" directly aligns with the changeset, which adds new database environment variables (DB_POOL_MAX, DB_POOL_MIN, DB_POOL_ACQUIRE_MS, DB_POOL_IDLE_MS, DB_POOL_EVICT_MS, PG_STATEMENT_TIMEOUT_MS, PG_IDLE_TX_TIMEOUT_MS) to src/envVariables.js. The title is concise, uses a conventional format with a descriptive "feat:" prefix, and clearly identifies both the pool configuration and timeout configuration aspects of the change. It contains no noise, vague terms, or misleading information, making it easy for teammates to understand the primary purpose of the pull request from version history alone.

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch database-connection-pool

---

📜 Recent review details

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 65f98c4 and 0629c7e.

📒 Files selected for processing (1)

• src/envVariables.js (2 hunks)

🚧 Files skipped from review as they are similar to previous changes (1)

• src/envVariables.js

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}