jackwener / jackwener/OpenCLI

auth refresh: include recovery command for not_logged_in rows

Open
#1,883 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
JavaScript
Stars
29.3k
Forks
2.9k
Avg merge
15h 36m
Merged PRs (30d)
70

Description

## Context
PR #1881 added `opencli auth refresh`. Review noted that `not_logged_in` rows should be directly actionable for agents instead of requiring them to synthesize the login command.

## Goal
Add a safe, structured recovery hint to auth refresh/status rows where appropriate, for example:

```json
{
"site": "github",
"status": "not_logged_in",
"recovery": "opencli github login"
}
```

## Requirements
- No secrets or account identifiers in recovery fields.
- Keep existing table output readable.
- Apply consistently to `auth refresh` and consider `auth status` if the row shape supports it cleanly.
- Align with existing AuthRequiredError recovery-command style.

Follow-up from PR #1881 / thread #OpenCLI:7bbdfcd2.

Contributor guide

Open the contributing guide

Research direction

Start by reading PR #1881 and the existing AuthRequiredError recovery-command style. Trace the auth refresh and auth status row outputs, then verify structured and table formats. Done means not_logged_in rows show safe, consistent recovery commands without secrets or account identifiers.

Written by the indexing model from the issue text.

Assessment

Tech stack
javascript
Domain
authentication, cli
Issue type
Feature
Difficulty
3/5
Estimated time
1-2 days
Activity status
Quiet
Clarity
Mostly clear
Newbie friendliness
58/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.