Skip to content

refactor: create llm package with Ollama implementation#153

Merged
theantichris merged 4 commits intomainfrom
refactor/llm-package
Dec 27, 2025
Merged

refactor: create llm package with Ollama implementation#153
theantichris merged 4 commits intomainfrom
refactor/llm-package

Conversation

@theantichris
Copy link
Owner

No description provided.

@theantichris theantichris added this to the Root Command UX & Inputs milestone Dec 27, 2025
@theantichris theantichris self-assigned this Dec 27, 2025
Copilot AI review requested due to automatic review settings December 27, 2025 03:45
@github-project-automation github-project-automation bot moved this to Backlog in theantichris Dec 27, 2025
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR refactors the codebase by extracting Ollama-specific LLM functionality into a dedicated internal/llm package. The changes improve code organization by separating concerns and making the LLM client logic reusable.

Key changes:

  • Created new internal/llm package with Ollama chat implementation
  • Moved chat-related types (ChatRequest, ChatResponse, ChatMessage) from main to the llm package
  • Removed getChatResponse function from main and replaced with llm.Chat function

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.

File Description
internal/llm/ollama.go New file containing Ollama chat implementation with exported types and Chat function
main.go Removed chat-related types and getChatResponse function; now uses llm.Chat for communication
main_test.go Updated to use llm.ChatMessage type; removed TestGetChatResponse test function
.harper-dictionary.txt Added ChatResponse to dictionary

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@theantichris theantichris merged commit adb40d6 into main Dec 27, 2025
5 checks passed
@theantichris theantichris deleted the refactor/llm-package branch December 27, 2025 03:57
@github-project-automation github-project-automation bot moved this from Backlog to Done in theantichris Dec 27, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants