yashdev9274 / yashdev9274/supercli
improvement: authentication synchronization between terminal and browser
Nobody has claimed this yet.
- Dominant language
- TypeScript
- Stars
- 214
- Forks
- 41
- Avg merge
- 2h 26m
- Merged PRs (30d)
- 34
Description
Description
- While running the
supercode logincommand, the CLI prompted me with aY/Nquestion asking whether to open the browser automatically. - Before responding to the prompt, I manually copied the authentication URL and completed the authorization flow in the browser.
- After returning to the terminal, there was no synchronization between the web app and the CLI.
- I then selected
No, after which the TUI started polling for the authorization response and continued polling indefinitely.
Expected Behavior
If the user completes the authorization flow manually before responding to the Y/N prompt, the CLI should first check whether the user has already been authenticated.
Suggested flow:
- Perform a direct database/auth state check to see if authorization has already been completed.
- If authorization is not found, start polling for the authentication response.
This would prevent unnecessary polling and ensure that manual authorization flows are handled correctly.
Screenshots:
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start at the supercode login command and trace the Y/N prompt and TUI polling path. Inspect how the CLI reads authorization state, then reproduce the manual-browser-before-prompt sequence; done means an already-completed authorization is detected before polling, while incomplete authorization still follows the existing polling flow.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- typescript
- Domain
- authentication, cli
- Issue type
- Feature
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Quiet
- Clarity
- Mostly clear
- Newbie friendliness
- 52/100