Skip to content

feat: Clearer auth error in non-interactive sessions#10

Merged
samstarling merged 1 commit into
mainfrom
sam/2026-06-auth-login-guard
Jun 22, 2026
Merged

feat: Clearer auth error in non-interactive sessions#10
samstarling merged 1 commit into
mainfrom
sam/2026-06-auth-login-guard

Conversation

@samstarling

Copy link
Copy Markdown
Contributor

When semaloop auth login runs in a non-interactive shell (e.g. CI), there's no terminal to read the API key from. Previously this surfaced a cryptic low-level error ("reading API key: inappropriate ioctl for device").

This change means we check whether stdin is a terminal before prompting, and fail with an actionable message pointing to the non-interactive alternatives:

no terminal available to prompt for an API key, use --api-key or the SEMALOOP_API_KEY environment variable

@samstarling samstarling requested a review from robotrory June 19, 2026 08:01
@samstarling samstarling marked this pull request as draft June 19, 2026 13:16
@samstarling samstarling marked this pull request as ready for review June 19, 2026 13:16
@samstarling samstarling merged commit 60d8c50 into main Jun 22, 2026
7 checks passed
semaloop-release Bot pushed a commit that referenced this pull request Jun 22, 2026
# [1.4.0](v1.3.0...v1.4.0) (2026-06-22)

### Features

* Clearer auth error in non-interactive sessions ([#10](#10)) ([60d8c50](60d8c50))
@semaloop-release

Copy link
Copy Markdown

🎉 This PR is included in version 1.4.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

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