Skip to content

[chore]: add dotenv to dev deps#1699

Merged
seanmcguire12 merged 1 commit into
mainfrom
add-dotenv-to-dev-deps
Feb 18, 2026
Merged

[chore]: add dotenv to dev deps#1699
seanmcguire12 merged 1 commit into
mainfrom
add-dotenv-to-dev-deps

Conversation

@seanmcguire12

@seanmcguire12 seanmcguire12 commented Feb 18, 2026

Copy link
Copy Markdown
Member

why

  • so that dotenv is installed with pnpm i

what changed

  • added dotenv to dev dependencies

Summary by cubic

Added dotenv 16.4.5 to devDependencies so it installs with pnpm i. This enables .env loading in local dev scripts without manual installation.

Written for commit 53f8c54. Summary will update on new commits. Review in cubic

@changeset-bot

changeset-bot Bot commented Feb 18, 2026

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: 53f8c54

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@greptile-apps

greptile-apps Bot commented Feb 18, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

Adds dotenv (v16.4.5) as a root-level workspace devDependency so it is installed via pnpm i. This aligns with the existing dotenv dependency in the evals package and supports local development workflows that rely on .env file loading (especially after the upcoming removal of automatic .env loading from the core library, per the existing changeset fair-cups-protect.md).

Confidence Score: 5/5

  • This PR is safe to merge — it only adds a well-known devDependency with no runtime or code logic changes.
  • The change is minimal and purely additive: a single pinned devDependency addition to the root package.json with the corresponding lockfile update. No source code is modified, no runtime behavior changes, and the version matches what's already used in the evals package.
  • No files require special attention.

Important Files Changed

Filename Overview
package.json Adds dotenv 16.4.5 as a root-level devDependency. Version matches the existing evals package dependency. No issues found.
pnpm-lock.yaml Auto-generated lockfile update reflecting the new root devDependency on dotenv 16.4.5. No issues.

Flowchart

flowchart TD
    A["Root package.json\n(workspace)"] -->|devDependencies| B["dotenv@16.4.5\n(newly added)"]
    C["packages/evals/package.json"] -->|dependencies| D["dotenv@16.4.5\n(existing)"]
    E["pnpm-lock.yaml"] -->|resolves| B
    E -->|resolves| D
    B --- F["Used for local dev\n(.env loading)"]
    D --- F
Loading

Last reviewed commit: 53f8c54

@greptile-apps greptile-apps Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

2 files reviewed, no comments

Edit Code Review Agent Settings | Greptile

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

No issues found across 2 files

Confidence score: 5/5

  • Automated review surfaced no issues in the provided summaries.
  • No files require special attention.
Architecture diagram
sequenceDiagram
    participant Dev as Developer
    participant Script as Local Dev Script
    participant Dotenv as dotenv (Library)
    participant FS as .env File
    participant Env as Node.js process.env

    Note over Dev, Env: Local Development Environment Bootstrapping

    Dev->>Script: Run pnpm script (dev/test/util)
    
    rect rgb(23, 37, 84)
    Note right of Script: Interaction enabled by NEW dependency
    Script->>Dotenv: NEW: config()
    Dotenv->>FS: Attempt to read .env file
    
    alt .env file found
        FS-->>Dotenv: Key=Value pairs
        Dotenv->>Env: NEW: Inject pairs into process.env
    else .env file missing
        FS-->>Dotenv: Error/Empty
        Dotenv-->>Script: No-op (use system env)
    end
    end

    Script->>Env: Access configuration (e.g., API keys)
    Env-->>Script: Return variable values
    Script->>Script: Proceed with local execution
Loading

@seanmcguire12
seanmcguire12 merged commit c815c55 into main Feb 18, 2026
147 checks passed
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