Skip to content

Conversation

@adelrodriguez
Copy link
Collaborator

@adelrodriguez adelrodriguez commented Feb 1, 2026

Greptile Overview

Greptile Summary

This PR adds developer-friendly default values to all .env.template files across the monorepo, significantly improving the developer experience by allowing the applications to run locally without requiring developers to manually configure every environment variable.

Key changes:

  • Added placeholder values for OAuth providers (GitHub, Google) with dev- prefixed credentials
  • Configured AUTH_TRUSTED_ORIGINS with localhost origins for local development
  • Added default Sentry configuration with example invalid URLs to prevent accidental production data transmission
  • Included Inngest workflow defaults with dev credentials
  • Added Arcjet security key placeholder (ajkey_dev)
  • Configured mobile app with additional Sentry metadata (EXPO_PUBLIC_SENTRY_ORG, EXPO_PUBLIC_SENTRY_PROJECT, EXPO_PUBLIC_SENTRY_URL)

All default values are clearly marked as development placeholders (using dev- prefix, example.invalid domains, or obvious placeholder values), ensuring developers won't accidentally use these in production while enabling immediate local development without external service configuration.

Confidence Score: 5/5

  • This PR is safe to merge with no risks
  • Changes are limited to template files with development-friendly defaults that improve developer experience. All values are clearly marked as placeholders and use invalid/example domains to prevent accidental production usage. No code logic changes.
  • No files require special attention

Important Files Changed

Filename Overview
apps/api/.env.template Added dev-friendly default values for API environment variables including auth providers, Inngest, Arcjet, and Sentry
apps/app/.env.template Added default values for auth providers, trusted origins, and Sentry client configuration
apps/mobile/.env.template Added Sentry configuration defaults including org, project, and URL for Expo integration

Sequence Diagram

sequenceDiagram
    participant Dev as Developer
    participant EnvTemplate as .env.template
    participant LocalEnv as .env (local)
    participant App as Application
    
    Dev->>EnvTemplate: Copies template
    EnvTemplate-->>LocalEnv: Creates .env with default values
    Note over LocalEnv: Default values allow<br/>immediate local development
    Dev->>LocalEnv: Optionally updates values<br/>for specific integrations
    App->>LocalEnv: Reads environment variables
    LocalEnv-->>App: Provides configuration
    Note over App: App runs with sensible<br/>defaults or custom config
Loading

@coderabbitai
Copy link

coderabbitai bot commented Feb 1, 2026

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

  • 🔍 Trigger a full review
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch chore_add_default_values_for_environment_variables

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

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

Copy link
Collaborator Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

@adelrodriguez adelrodriguez marked this pull request as ready for review February 1, 2026 20:55
@adelrodriguez adelrodriguez merged commit 479925e into main Feb 2, 2026
8 checks passed
@adelrodriguez adelrodriguez deleted the chore_add_default_values_for_environment_variables branch February 2, 2026 15:06
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.

2 participants