Improved model managment by zilaei · Pull Request #286 · helsingborg-stad/F-AI

zilaei · 2025-09-04T16:50:39Z

This PR adds model managment to settings. This PR also improves the UI in the zoo edit page, for displaying available models when creating new assistants.

Settings:

Zoo:

Features

New backend service for models
Model settings UI
Improved model selection UI in the zoo
Future proofed support for model visibility settings (active / depricated)

Backend

Created MongoModelService with full CRUD operations
Added /model API endpoints for CRUD's
Added model specific scoops

Frontend

New model settings page route /settings/models/:
- Create / Edit modal with capability management
- Delete confirmation modal
- Badges for model capabilities (vision, function calling and so on)
Better model selector in the zoo:
- Accordion with model information
Reusable badge component for model capabilities
Storybook stories for the new components

Testing

Remove models in MondoDB chat_models and restart the backend
Add New scoopes needed for administrating models:

"scopes": [
    ...,
    "model.read",
    "model.write",
    "model.delete",
    "model.admin"
  ],

- Create ModelSelector component to display selected model compactly - Create ModelSelectorModal component to wrap ModelAccordion in a modal - Update AssistantDetails to use new ModelSelector instead of inline accordion - Improves UX by reducing initial form length refactor(frontend/assistant): use class directives for conditional styling refactor(frontend/assistant): remove redundant enhancedDescription div in ModelItem and feat(frontend/assistant): introduce reusable Badge component - Add componenet `Badge.svelte` for model badges - Replace hardcoded badge-related logic in ModelHeader, ModelCapabilities, and ModelSelector with Badge component feat(frontend/assistant): install and use `lucide-svelte` for Badge icons - Replace SVG strings with `lucide-svelte` components for badge icons. - Add `lucide-svelte` as a dependency. - Refactor Badge component for better type safety and dynamic icon rendering. feat(frontend/models): create a new page for models in settings - New `ModelSettings` components: `ModelList`, `ModelItem`, `ModelFormModal`, `DeleteConfirmModal`, and `PageHeader`. - Add modal-based create, edit, and delete flows for models. fix(frontend/models): handle delete model errors in `DeleteConfirmModal` - Added error state management - Form enhance callback to handle deletion failurs - Added UI for display error messages fix(frontend/models): check if the error message type before handling it as a string feat(api/models): add CRUD operations and lifecycle fields for models - Introduced `Model` with fields: `created_at`, `updated_at`, `status`, `visibility`, and `version`. - Added model management in `IAssistantService` with methods for create, update, delete, and retrieval. - Implemented comprehensive test coverage for CRUD operations, including edge cases like keys with slashes, duplicate keys, optimistic locking, and model lifecycle validation. - Integrated `model_router` into `api_router`. feat(models): implement MongoDB-backed model service - Added MongoModelService for model. - Created API CRUD endpoints for models. - Implemented tests for MongoModelService, covering edge cases and lifecycle scenarios. - Integrated new model service to main app setup This commit will break the frontend. `backendApi.ts` needs to consider the new endpoint created for the model service.

…r model management - Introduced new user permission checks: read, write, delete, access settings, and admin for models. - Added CRUD methods in `backendApi.ts` to interact with the models API.

…ranslations - Updated `fr.json`, `en.json`, and `sv.json` to include new labels for "General" and "Models" settings.

- Introduced new components: `CapabilityToggle`, `MaxTokensInput`, `CapabilitiesSection`, and `CapabilityBadges`. - Added support for managing model capabilities (e.g., image processing, reasoning, code execution, function calling) and token limits in the model settings.

- Made meta a required field with default values. - Updated helpers and API logic to align with meta changes. - Removed unused GetAvailableModels API and its references.

… helpers

- Replaced `formData` with individual reactive state variables. - Improved readability and reduced complexity in the reactive logic. - Minor UI cleanup, SVG with `lucide-svelte` component. - Fix linting and remove comments

…iption fallback feat(components): extracted color picker, from assistant, to a separate reusable component - Added customizable colors and selection handling. feat(frontend/models): add primary color support to ModelFormModal feat(models): add primary color handling to meta object

…component fix(models): use correct fields for avatar and color handling in ModelSelector and utilities refactor(models): enhanced model handling with `$derived.by` refactor(models): add utilities for snake_case to camelCase conversion and update meta handling feat(models): normalize backend response with `ModelRepository` and `ModelTransformer` - Added a new `ModelRepository` to centralize API interactions for models. - Implemented `ModelTransformer` for bidirectional conversion between camelCase and snake_case. refactor(utils): remove helper for `getModelAvatar` access

fix(model): remove component hover effect

…classes directly before endpoints - Kept shared `ModelResponse` class at the top - since it's used by multiple endpoints - Moved `CreateModelRequest`, `GetModelsResponse` and `UpdateModelRequest` placed directly before their corresponding endpoint

- Backend routes now have a cleaner path without the redundant `/models` suffix - Frontend API client updated to use the fixed endpoints

…sistantService.py`

…ent-to-model conversion logic - Fix will improve code by removing duplications

…d_at` to Unix epoch for missing/unknown timestamps

…instead of manually specifying all fields

…at` and `updated_at`

….py`

…` from `setup_default_groups.py`

- "vison" was incorrectly used

zilaei force-pushed the feat/user-friendly-model-info branch from d028b67 to 4c8fbe8 Compare September 9, 2025 14:04

zilaei marked this pull request as ready for review September 10, 2025 11:52

zilaei requested a review from MasterKenth September 11, 2025 08:19

zilaei added 16 commits September 11, 2025 10:41

feat(frontend/models): extend permissions and add API integrations fo…

9b4bfd4

…r model management - Introduced new user permission checks: read, write, delete, access settings, and admin for models. - Added CRUD methods in `backendApi.ts` to interact with the models API.

refactor(frontend/settings): improve navigation and action button

c8b74c8

feat(frontend/settings): add labels for General and Models settings t…

0a621e4

…ranslations - Updated `fr.json`, `en.json`, and `sv.json` to include new labels for "General" and "Models" settings.

refactor(models): remove optional meta fields

28a27ff

- Made meta a required field with default values. - Updated helpers and API logic to align with meta changes. - Removed unused GetAvailableModels API and its references.

refactor(models): extract buildMetaObject and extractCapabilities…

77d356a

… helpers

feat(storybook): add stories for ModelSettings components

e785c01

fix(model): set default function calling capability false

46b5265

fix(model): remove component hover effect

feat(backend/models): extend default setup data with meta data

421d103

refactor: move ModelRepository to backendApi

4582bc2

refactor: merge ModelRepository into backendApi

73f5a53

add web search capability support

96eee64

zilaei force-pushed the feat/user-friendly-model-info branch from 5a9684d to 96eee64 Compare September 11, 2025 08:44

zilaei added 10 commits September 11, 2025 16:12

fix(backend/main): routes should be ordered by alphabetic order

5f8a248

fix(model): remove redundant suffix "/models" from endpoints

96344a1

- Backend routes now have a cleaner path without the redundant `/models` suffix - Frontend API client updated to use the fixed endpoints

fix(backend/assistants): remove unused _model_service from `MongoAs…

108078d

…sistantService.py`

fix(backend/models): create helper function _doc_to_model for docum…

2186c19

…ent-to-model conversion logic - Fix will improve code by removing duplications

fix(backend/models): set MongoModelService created_at and `update…

2b18f14

…d_at` to Unix epoch for missing/unknown timestamps

fix(backend/models): use model.model_dump() in MongoModelService …

af0cb47

…instead of manually specifying all fields

fix(backend/models): set explicit timestamps for Model.py `created_…

7a14526

…at` and `updated_at`

fix(backend/models): remove model.delete from `setup_default_groups…

7dec903

….py`

fix(backend/models): remove unused model.admin and `settings.models…

da3c0e0

…` from `setup_default_groups.py`

fix(models): refactor capability "vision" to "imagegen"

57c5957

- "vison" was incorrectly used

MasterKenth approved these changes Sep 12, 2025

View reviewed changes

zilaei merged commit b30f1f4 into master Sep 12, 2025
2 checks passed

zilaei deleted the feat/user-friendly-model-info branch September 12, 2025 11:46

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Improved model managment#286

Improved model managment#286
zilaei merged 27 commits into
masterfrom
feat/user-friendly-model-info

zilaei commented Sep 4, 2025 •

edited

Loading

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Conversation

zilaei commented Sep 4, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Features

Backend

Frontend

Testing

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

zilaei commented Sep 4, 2025 •

edited

Loading