Skip to content
Open
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion README.md
Original file line number Diff line number Diff line change
Expand Up @@ -43,7 +43,7 @@ Or run locally:
git clone https://github.com/vercel-labs/coding-agent-template.git
cd coding-agent-template
pnpm install
# Set up .env.local with required variables
# Set up .env with required variables
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
# Set up .env with required variables
# Set up .env.local with required variables

The Quick Start section now references .env for environment variables, but the detailed "Local Development Setup" section still references .env.local. This inconsistency will confuse developers during setup.

View Details

Analysis

Documentation inconsistency: Quick Start references wrong environment file

What fails: README.md Quick Start section (line 46) instructs developers to "Set up .env with required variables" while the detailed "Local Development Setup" section (line 188) instructs to "Create a .env.local file with your values". This creates conflicting setup instructions for local development.

How to reproduce:

  1. Read line 46 in Quick Start - says to use .env
  2. Read line 188 in "Local Development Setup" - says to use .env.local
  3. Developer follows Quick Start and creates .env, then encounters different instructions in detailed setup

Result: Developers are confused about which file to use since Quick Start contradicts the detailed setup section.

Expected: Per Next.js environment variables documentation, .env.local is the standard convention for local development - it allows developers to override defaults without committing sensitive values to version control. The gitignore pattern .env* is specifically designed to ignore .env.local for local development.

Fix: Updated line 46 to reference .env.local to align with line 188 and follow Next.js best practices.

pnpm db:push
pnpm dev
```
Expand Down