Skip to content

bump cogito#416

Merged
mudler merged 1 commit intomainfrom
chore/bump-cogito
Feb 14, 2026
Merged

bump cogito#416
mudler merged 1 commit intomainfrom
chore/bump-cogito

Conversation

@mudler
Copy link
Owner

@mudler mudler commented Feb 14, 2026

No description provided.

Signed-off-by: Ettore Di Giacinto <mudler@localai.io>
Copilot AI review requested due to automatic review settings February 14, 2026 20:25
@mudler mudler merged commit 0762d6f into main Feb 14, 2026
3 of 4 checks passed
@mudler mudler deleted the chore/bump-cogito branch February 14, 2026 20:25
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 updates the cogito dependency to a newer version from February 14, 2026, and refactors the code in core/agent/agent.go to work with the updated library API.

Changes:

  • Updated github.com/mudler/cogito dependency from version 0.8.2-0.20260206153401-a5346975d42b to 0.8.2-0.20260214201734-da0d4ceb2b44
  • Simplified code in core/agent/agent.go by removing an intermediary LLM call and associated error handling
  • Updated go.sum to reflect the new dependency version

Reviewed changes

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

File Description
go.mod Updates the cogito dependency version to the latest build
go.sum Adds checksums for the new cogito dependency version
core/agent/agent.go Removes intermediary LLM call, simplifying the code to directly use fragment's last message

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

}

result := a.cleanupLLMResponse(responseFragment.LastMessage().Content)
result := a.cleanupLLMResponse(fragment.LastMessage().Content)
Copy link

Copilot AI Feb 14, 2026

Choose a reason for hiding this comment

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

The removed code included error handling for the a.llm.Ask() call. Now fragment.LastMessage() is called directly without checking if fragment has messages or handling potential nil/empty cases. This could cause a panic if fragment.Messages is empty.

Copilot uses AI. Check for mistakes.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant