Skip to content

fix: align memory embedding provider catalog - #7472

Merged
houko merged 6 commits into
mainfrom
fix/dashboard-memory-embedding-catalog
Aug 16, 2026
Merged

houko merged 6 commits into
mainfrom
fix/dashboard-memory-embedding-catalog

Conversation

@houko

@houko houko commented Aug 15, 2026 •

Copy link
Copy Markdown
Contributor

Summary

  • align the Memory embedding-provider catalog with runtime auto-detection support
  • keep provider, model, and API-key environment selections compatible when the provider changes
  • keep custom model entry stable while the user types
  • persist Auto-detect selections as explicit null clears and remove the corresponding TOML overrides server-side
  • return canonical cleared/default values from the PATCH response and cover the behavior with route and UI regressions

Verification

  • corepack pnpm exec vitest run src/pages/Memory/constants.test.ts src/pages/Memory/index.test.tsx (2 files, 15 tests passed)
  • corepack pnpm typecheck
  • corepack pnpm lint
  • CARGO_TARGET_DIR=/private/tmp/librefang-target-pr7472 cargo test -p librefang-api --lib routes::memory::tests::optional_string_patch_removes_explicit_nulls -- --exact
  • CARGO_TARGET_DIR=/private/tmp/librefang-target-pr7472 cargo test -p librefang-api --test memory_routes_integration patch_memory_config_null_clears_embedding_overrides -- --exact
  • CARGO_TARGET_DIR=/private/tmp/librefang-target-pr7472 cargo clippy -p librefang-api --all-targets -- -D warnings
  • cargo fmt --all -- --check
  • git diff --check

Out of scope

  • adding new runtime embedding providers or changing backend provider defaults
  • existing Polish and Ukrainian plural-key parity drift

@github-actions github-actions Bot added no-rust-required This task does not require Rust knowledge size/M 50-249 lines changed labels Aug 15, 2026
@github-actions github-actions Bot added size/L 250-999 lines changed and removed no-rust-required This task does not require Rust knowledge size/M 50-249 lines changed labels Aug 16, 2026
@houko
houko enabled auto-merge (squash) August 16, 2026 22:27
@houko
houko merged commit 11a2951 into main Aug 16, 2026
41 checks passed
@houko
houko deleted the fix/dashboard-memory-embedding-catalog branch August 16, 2026 22:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size/L 250-999 lines changed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant