mosaic/stack
1
0
Fork 0
Code Issues 10 Pull Requests Actions Packages Projects Releases 2 Wiki Activity

Refactor LlmService to Use Provider Pattern #127

New Issue
Closed
opened 2026-01-30 21:28:32 +00:00 by jason.woltje · 0 comments
jason.woltje commented 2026-01-30 21:28:32 +00:00
Owner
Copy Link

Refactor existing LlmService to delegate to LLM providers.

Objective: Migrate existing Ollama-specific code to use new provider abstraction.

Tasks:

  • Update llm.service.ts to use LlmManager
  • Remove Ollama-specific code from service
  • Delegate all LLM calls to providers
  • Maintain backward compatibility
  • Update existing tests
  • Ensure no breaking changes to API endpoints

Acceptance Criteria:

  • Existing chat functionality unbroken
  • All tests pass
  • Service uses provider pattern
  • No Ollama-specific code in service layer

Related: Epic #121, Phase 1 LLM Abstraction
Depends on: #122, #123, #126 (interface, Ollama provider, manager)

Refactor existing LlmService to delegate to LLM providers. Objective: Migrate existing Ollama-specific code to use new provider abstraction. Tasks: - Update llm.service.ts to use LlmManager - Remove Ollama-specific code from service - Delegate all LLM calls to providers - Maintain backward compatibility - Update existing tests - Ensure no breaking changes to API endpoints Acceptance Criteria: - Existing chat functionality unbroken - All tests pass - Service uses provider pattern - No Ollama-specific code in service layer Related: Epic #121, Phase 1 LLM Abstraction Depends on: #122, #123, #126 (interface, Ollama provider, manager)
jason.woltje added the p0apiapiphase-1 labels 2026-01-30 21:28:32 +00:00
jason.woltje added this to the M4-LLM (0.0.4) milestone 2026-01-30 23:40:48 +00:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
ai
AI/LLM integration
 api
Backend/API work
 api
auth
Authentication
 database
database
Database/schema work
 devops
Docker/deployment
 docs
Documentation
 frontend
graph
knowledge-module
migration
Data migration
 orchestrator
Orchestrator service (apps/orchestrator/)
 p0
Critical priority
 p1
High priority
 p2
Medium priority
 p3
Low priority
 performance
Performance work
 phase-1
phase-2
phase-3
phase-4
phase-5
plugin
MoltBot plugin work
 search
security
Security-related
 setup
Initial setup
 testing
Testing/QA
 web
Frontend work
No Label api api p0 phase-1
Milestone
No items
No Milestone M4-LLM (0.0.4)
Projects
Clear projects
No project
Assignees
Clear assignees
jason.woltje
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: mosaic/stack#127
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?