Skip to content

Made env example local-dev friendly#158

Merged
ishaanxgupta merged 1 commit intoXortexAI:mainfrom
Ankit-Kotnala:dev/Ankit
May 7, 2026
Merged

Made env example local-dev friendly#158
ishaanxgupta merged 1 commit intoXortexAI:mainfrom
Ankit-Kotnala:dev/Ankit

Conversation

@Ankit-Kotnala
Copy link
Copy Markdown
Contributor

Summary

This PR updates .env.example so new contributors can set up the project locally without accidentally using production URLs or real-looking secrets.

Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request refactors the .env.example file by removing several LLM providers and monitoring configurations, focusing the setup on Google Gemini. It also introduces a new Authentication section for Google OAuth and JWT settings. Feedback includes correcting a likely typo in the Gemini model version, re-adding the ENVIRONMENT variable to ensure local development defaults, and maintaining consistent placeholder naming for authentication secrets.

Comment thread .env.example

# Google Gemini
GEMINI_API_KEY=your_gemini_api_key_here
GEMINI_MODEL=gemini-2.5-flash
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

medium

The model version gemini-2.5-flash appears to be a typo, as Gemini 2.5 is not a currently available version. For a working local setup, you should likely use gemini-1.5-flash or gemini-2.0-flash.

GEMINI_MODEL=gemini-1.5-flash

Comment thread .env.example
Comment on lines 42 to 44
# =============================================================================
# Monitoring & Observability
# Authentication
# =============================================================================
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

medium

The ENVIRONMENT variable was removed in this PR. Since the default value in src/config/settings.py is production, local instances will default to production mode. To ensure a local-friendly setup (e.g., disabling production-only monitoring or analytics), it is better to explicitly set this to development.

# =============================================================================
# General Settings
# =============================================================================
ENVIRONMENT=development

# =============================================================================
# Authentication
# =============================================================================

Comment thread .env.example
# =============================================================================
ENVIRONMENT=production
GOOGLE_CLIENT_ID=your_google_client_id_here
GOOGLE_CLIENT_SECRET=your-google_client_secret_here
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

medium

The placeholder for GOOGLE_CLIENT_SECRET uses a hyphen (your-google_client_secret_here), while GOOGLE_CLIENT_ID uses underscores. For consistency across the configuration file, it's recommended to use underscores for all placeholders.

GOOGLE_CLIENT_SECRET=your_google_client_secret_here

@ishaanxgupta ishaanxgupta merged commit 09953d8 into XortexAI:main May 7, 2026
1 check passed
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.

2 participants