lsm / neokai / 26668113987

82%

Build Type

push

github

Committed by web-flow

Commit Message

feat: unified provider registry with CRUD RPCs, credential store, and migration (#2044)

* feat: providers table schema, shared types, and repository

- Add ProviderRecord, CreateProviderParams, UpdateProviderParams to shared types
- Add providers table to schema (createTables + migration 148)
- Add ProviderRepository with full CRUD, reactive DB notifications
- Export ProviderCredentials from shared/provider index
- Add providers and provider_credentials to DB query excluded tables

* feat: provider CRUD RPCs, credential store, sync, and startup wiring

- Add CredentialStore with Keychain (macOS) and AES-256-GCM DB fallback
- Add ProviderCredentialManager for storeApiKey/storeOAuthTokens/get/remove
- Add provider-sync.ts: syncProviderToRegistry, syncAllProviders, removeProviderFromRegistry
- Add provider-handlers.ts: providers.list/get/create/update/delete/setDefault/test/healthCheck
- Add setCredentials() to all built-in providers (env fallback preserved)
- Wire provider handlers into RPC index with ProviderCredentialManager
- Custom endpoint compat layer: writes to both settings and providers table
- Add migrateProvidersIfNeeded: env vars → auth.json → customEndpoints → providers table
- Update app.ts startup: migrate → syncCustomEndpointProviders → syncAllProviders

* test: provider repository, handlers, and credential store

- ProviderRepository CRUD round-trip tests
- Provider RPC handler tests (create, update, delete, setDefault, test)
- DatabaseCredentialStore encryption round-trip tests

* refactor: address review feedback for provider overhaul PR

- Remove duplicate setCredentials declaration in Provider interface
- DRY: import customProviderIdFor instead of redefining/inlining custom: prefix
- Replace non-null assertions after INSERT/UPDATE with explicit throws
- Thread startup ProviderCredentialManager into RPC handlers
- Add tests for providers.get, providers.healthCheck, OAuth create/update

* refactor: address review round 2 — locks, fai... (continued)

Coverage Stats

9798 of 14272 branches covered (68.65%)

Branch coverage included in aggregate %.

288 of 788 new or added lines in 12 files covered. (36.55%)

15973 existing lines in 101 files now uncovered.

69081 of 99552 relevant lines covered (69.39%)

228.19 hits per line