Skip to content

feat: validate environment configuration at startup - #387

Open
Jossy-ossy wants to merge 2 commits into
Divinity-Labs:mainfrom
Jossy-ossy:fix/issue-338
Open

Jossy-ossy wants to merge 2 commits into
Divinity-Labs:mainfrom
Jossy-ossy:fix/issue-338

Conversation

@Jossy-ossy

Copy link
Copy Markdown
Contributor

Summary

  • Centralize runtime environment access in the Zod-backed config module and validate numeric, boolean, network, and production-required settings.
  • Replace direct environment reads across service modules with typed config accessors.
  • Document per-network overrides and add validation regression coverage.

Closes #338

@drips-wave

drips-wave Bot commented Oct 16, 2026

Copy link
Copy Markdown

@Jossy-ossy Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

@Jossy-ossy

Copy link
Copy Markdown
Contributor Author

@Miracle656 I resolved the merge conflicts and updated this branch. Please take another look and merge when ready.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Fourteen files read process.env directly, and nothing validates it

1 participant