Skip to content

fix: fetch real secrets from backend API in SecretVault (#3987)#3995

Merged
mrveiss merged 1 commit intoDev_new_guifrom
fix/issue-3987
Apr 8, 2026
Merged

fix: fetch real secrets from backend API in SecretVault (#3987)#3995
mrveiss merged 1 commit intoDev_new_guifrom
fix/issue-3987

Conversation

@mrveiss
Copy link
Copy Markdown
Owner

@mrveiss mrveiss commented Apr 8, 2026

Summary

Replaces hardcoded mock secrets with real backend secret data.

Changes

  • Removed 4 hardcoded mock secrets from component
  • Added API integration via SecretsApiClient
  • Fetch secrets from /api/secrets/ endpoint on mount
  • Implemented delete/revoke secret functionality with confirmation
  • Added loading state while fetching secrets
  • Added error and success notifications
  • Enhanced UI with secret metadata (created_at, updated_at)
  • Proper scope badge support (user/global, session/chat)

Features

  • Real secrets now display from backend API
  • Users can delete/revoke secrets
  • Loading and error states show appropriately
  • Metadata displays creation/update dates
  • Secret values masked until revealed

Testing

  • Component loads secrets from real API
  • Delete functionality works with confirmation
  • Error handling displays user-friendly messages
  • TypeScript compilation successful

Closes #3987

Replace hardcoded mock secrets with real backend data:
- Removed hardcoded mockSecrets array
- Added API integration via SecretsApiClient
- Implemented getSecrets() from /api/secrets/ endpoint
- Added loading state and error handling
- Implemented delete/revoke functionality
- Enhanced display with metadata (created_at, updated_at)
- Proper TypeScript typing and error handling

Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>
@mrveiss mrveiss merged commit 4ed2a0b into Dev_new_gui Apr 8, 2026
1 of 3 checks passed
@mrveiss mrveiss deleted the fix/issue-3987 branch April 8, 2026 10:03
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