Skip to content

Conversation

@Jay-IRL
Copy link
Contributor

@Jay-IRL Jay-IRL commented Jan 21, 2026

πŸ› Bug-fix PR

Before opening this PR please:

  1. make lint - passes ruff, mypy, pylint
  2. make test - all unit + integration tests green
  3. make coverage - β‰₯ 90 %
  4. make docker docker-run-ssl or make podman podman-run-ssl
  5. Update relevant documentation.
  6. Tested with sqlite and postgres + redis.
  7. Manual regression no longer fails. Ensure the UI and /version work correctly.

πŸ“Œ Summary

Code changes in toolops modules to support LLM configuration using "LLM Settings".

πŸ” Reproduction Steps

Link the issue and minimal steps to reproduce the bug.

🐞 Root Cause

Changes in LLM configuration using "LLM Settings"

πŸ’‘ Fix Description

Update toolops modules to use LLM configurations from "LLM Settings"

πŸ§ͺ Verification

Check Command Status
Lint suite make lint
Unit tests make test
Coverage β‰₯ 90 % make coverage
Manual regression no longer fails steps / screenshots

πŸ“ MCP Compliance (if relevant)

  • Matches current MCP spec
  • No breaking change to MCP clients

βœ… Checklist

  • Code formatted (make black isort pre-commit)
  • No secrets/credentials committed

@Jay-IRL Jay-IRL requested a review from crivetimihai as a code owner January 21, 2026 08:47
@crivetimihai crivetimihai added this to the Release 1.0.0-RC1 milestone Jan 21, 2026
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.

2 participants