Repository navigation
refactor: move repo's to domain - #1801
Merged
Merged
Conversation
github-actions
Bot
added
the
type: feature
amitksingh1490
force-pushed
the
refactor/infra-to-domain
branch
from
October 29, 2025 13:41
a8ad980 to
deefe81
Compare
- Added `schema.rs` to define the database schema for conversations. - Created `forge_repo.rs` to implement the repository layer, aggregating various repository implementations. - Introduced `fs_snap.rs` for file snapshot management. - Updated `lib.rs` to include new modules for app configuration, caching, conversations, and file snapshots. - Modified `auth.rs` to accept a repository parameter for app configuration management. - Enhanced `forge_services.rs` to support repository integration in services. - Refactored `mcp` manager and service to utilize repository for cache management. - Updated provider registry to integrate repository for app configuration handling. - Implemented snapshot coordination in file services (create, patch, remove, undo).
amitksingh1490
marked this pull request as ready for review
October 30, 2025 11:18
- Updated Cargo.lock to include new dependencies: bytes, reqwest, reqwest-eventsource, strum, and url. - Refactored ForgeAPI to simplify the initialization of ForgeServices and ForgeRepo. - Modified Cargo.toml files to reflect the new dependencies in the respective crates. - Adjusted the structure of ForgeRepo to accept a generic infrastructure type, improving flexibility. - Streamlined the ForgeAuthService to remove unnecessary repository dependency. - Enhanced ForgeMcpManager and ForgeMcpService to eliminate repository dependencies, relying solely on infrastructure. - Updated various service implementations (FsCreate, FsPatch, FsRemove, FsUndo) to remove repository dependencies and utilize infrastructure directly. - Improved the ForgeProviderRegistry to work without a repository, simplifying its interface. - Ensured all changes maintain functionality while improving code clarity and reducing complexity.
…rations - Added `provider_repository.rs` to handle loading and merging provider configurations from embedded and custom sources. - Introduced `ForgeProviderRepository` struct with methods to initialize providers and manage configurations. - Implemented custom merging logic for provider configurations. - Added tests for loading and merging provider configurations. refactor: Update auth and forge_services modules - Moved `AppConfigRepository` import to `forge_domain` in `auth.rs`. - Cleaned up imports in `forge_services.rs` and removed unused `provider_registry`. - Added `provider_preferences.rs` for managing user preferences related to providers and models. feat: Add provider preferences service - Created `ForgeProviderPreferencesService` to manage default provider and model preferences. - Implemented methods to get and set default providers and models. - Added tests to ensure correct behavior of provider preferences service. refactor: Update provider service to include provider repository - Modified `ForgeProviderService` to require `ProviderRepository` for fetching providers. - Added methods to get a specific provider and all providers in `provider/service.rs`.
amitksingh1490
force-pushed
the
refactor/infra-to-domain
branch
from
October 31, 2025 07:15
7d99e31 to
c3d91a0
Compare
amitksingh1490
changed the title
amitksingh1490
added
type: chore
…or caching operations
… implement caching with CacacheStorage
…ile I/O operations
manthanabc
pushed a commit
to manthanabc/forge
that referenced
this pull request
Nov 3, 2025
Co-authored-by: autofix-ci[bot] <114827586+autofix-ci[bot]@users.noreply.github.com> Co-authored-by: Tushar Mathur <tusharmath@gmail.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.