Skip to content

Conversation

@TheUntraceable
Copy link

@TheUntraceable TheUntraceable commented Jan 29, 2026


Summary by cubic

Updated the missing API key error to reference "atmn login" instead of "atmn auth" to match the current CLI command and reduce confusion. Also removed invalid "hour" and "day" reset intervals from plan validation.

Written for commit fdc9640. Summary will update on new commits.

Greptile Overview

Greptile Summary

Corrected error message to reference the correct CLI authentication command atmn login instead of the non-existent atmn auth command, ensuring consistency with the rest of the codebase.

Key Changes:

  • Bug fixes: Fixed incorrect CLI command reference in API key error message (atmn/src/lib/env/keys.ts:51)

Confidence Score: 5/5

  • This PR is safe to merge with no risk
  • This is a simple, single-line documentation fix that corrects an outdated command reference. The change aligns with the actual CLI command defined in the codebase and matches all other error messages throughout the project.
  • No files require special attention

Important Files Changed

Filename Overview
atmn/src/lib/env/keys.ts Updated error message to reference correct CLI command atmn login instead of non-existent atmn auth

Sequence Diagram

sequenceDiagram
    participant User
    participant Application
    participant KeyManager
    participant EnvReader

    User->>Application: Request operation
    Application->>KeyManager: getKey(environment)
    KeyManager->>EnvReader: readApiKeys()
    EnvReader-->>KeyManager: Return available keys
    KeyManager->>KeyManager: Validate key exists
    alt Key not found
        KeyManager-->>Application: Throw authentication error
        Application-->>User: Show error with login command
        Note over User: Runs atmn login
    else Key exists
        KeyManager-->>Application: Return valid key
        Application->>Application: Process request
    end
Loading

(2/5) Greptile learns from your feedback when you react with thumbs up/down!

Copilot AI review requested due to automatic review settings January 29, 2026 18:56
@vercel
Copy link

vercel bot commented Jan 29, 2026

@TheUntraceable is attempting to deploy a commit to the Autumn Team on Vercel.

A member of the Team first needs to authorize it.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates an error message to reference the correct CLI command for authentication. The error message previously instructed users to run atmn auth, but the actual command is atmn login.

Changes:

  • Updated error message in getKey() function to reference atmn login instead of the non-existent atmn auth command

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link

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

Choose a reason for hiding this comment

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

No issues found across 1 file

Confidence score: 5/5

  • Automated review surfaced no issues in the provided summaries.
  • No files require special attention.

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.

1 participant