Skip to content

Improve logging, shutdown handling, and environment validation#5

Merged
IsaacHatilima merged 1 commit intoreviewfrom
codex/review-repository-and-suggest-changes
Dec 25, 2025
Merged

Improve logging, shutdown handling, and environment validation#5
IsaacHatilima merged 1 commit intoreviewfrom
codex/review-repository-and-suggest-changes

Conversation

@IsaacHatilima
Copy link
Owner

Summary

  • enable consistent HTTP request logging across environments and add graceful SIGINT shutdown handling
  • introduce Zod-based environment validation and wire configs to the validated env accessors
  • document required configuration with an updated README and sample .env file

Testing

  • not run (not requested)

Codex Task

Copy link

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

JWT_ACCESS_SECRET: z.string({required_error: "JWT_ACCESS_SECRET is required"}),

PORT: z.coerce.number().int().positive().default(3000),
SWAGGER_ENABLED: z.coerce.boolean().default(false),

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P1 Badge Stop coercing SWAGGER_ENABLED="false" to true

The new env schema uses z.coerce.boolean() for SWAGGER_ENABLED (src/utils/environment-variables.ts) so any non-empty string parses to true; as shown by the .env.example value SWAGGER_ENABLED=false, this now always enables Swagger even when the flag is set to "false" or "0", exposing docs in environments where they were meant to be disabled.

Useful? React with 👍 / 👎.

@IsaacHatilima IsaacHatilima merged commit 92464c2 into review Dec 25, 2025
1 check passed
@IsaacHatilima IsaacHatilima deleted the codex/review-repository-and-suggest-changes branch December 25, 2025 10:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant