Skip to content

feat: implement comprehensive duplicate provider connection handling#112

Open
toanalien wants to merge 1 commit intodecolua:masterfrom
toanalien:feat/handle-duplicate-provider
Open

feat: implement comprehensive duplicate provider connection handling#112
toanalien wants to merge 1 commit intodecolua:masterfrom
toanalien:feat/handle-duplicate-provider

Conversation

@toanalien
Copy link
Contributor

  • Add duplicate detection utilities with secure API key hashing (SHA-256)
  • Implement OAuth duplicate detection by email/user ID/project ID
  • Implement API key duplicate detection by hashed key comparison
  • Add DuplicateWarningModal component with Replace/Keep Both/Cancel options
  • Add visual duplicate badge indicators with warning variant
  • Add POST /api/providers/check-duplicate endpoint for pre-check
  • Update POST /api/providers to return 409 Conflict on duplicates
  • Auto-update OAuth connections when duplicates detected in callback
  • Add comprehensive documentation in docs/DUPLICATE_HANDLING.md
  • Support provider-specific duplicate detection (GitHub, Antigravity, Cursor, etc.)
  • Preserve priority and settings when replacing duplicates
  • Allow keeping both connections as fallback with lower priority

- Add duplicate detection utilities with secure API key hashing (SHA-256)
- Implement OAuth duplicate detection by email/user ID/project ID
- Implement API key duplicate detection by hashed key comparison
- Add DuplicateWarningModal component with Replace/Keep Both/Cancel options
- Add visual duplicate badge indicators with warning variant
- Add POST /api/providers/check-duplicate endpoint for pre-check
- Update POST /api/providers to return 409 Conflict on duplicates
- Auto-update OAuth connections when duplicates detected in callback
- Add comprehensive documentation in docs/DUPLICATE_HANDLING.md
- Support provider-specific duplicate detection (GitHub, Antigravity, Cursor, etc.)
- Preserve priority and settings when replacing duplicates
- Allow keeping both connections as fallback with lower priority
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