Skip to content

fix: handle provider_reauthorization_required as reconnect-needed#27

Merged
tumberger merged 1 commit intomainfrom
04-09-fix_handle_provider_reauthorization
Apr 9, 2026
Merged

fix: handle provider_reauthorization_required as reconnect-needed#27
tumberger merged 1 commit intomainfrom
04-09-fix_handle_provider_reauthorization

Conversation

@tumberger
Copy link
Copy Markdown
Contributor

@tumberger tumberger commented Apr 9, 2026

The token exchange returns provider_reauthorization_required when a
provider needs re-authorization. Treat it the same as not-connected
so the CLI opens the browser for the connect flow.

Co-Authored-By: Claude Opus 4.6 (1M context) noreply@anthropic.com


Open with Devin

The token exchange returns provider_reauthorization_required when a
provider needs re-authorization. Treat it the same as not-connected
so the CLI opens the browser for the connect flow.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@tumberger tumberger marked this pull request as ready for review April 9, 2026 07:40
@tumberger tumberger merged commit 8e89e26 into main Apr 9, 2026
2 checks passed
Copy link
Copy Markdown
Contributor Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

Copy link
Copy Markdown

@devin-ai-integration devin-ai-integration bot left a comment

Choose a reason for hiding this comment

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

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 1 additional finding.

Open in Devin Review

@michiosw michiosw deleted the 04-09-fix_handle_provider_reauthorization branch April 10, 2026 09:04
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