Skip to content

feat: add ability to specify guardrails model #135#136

Merged
navalnica merged 1 commit intodevelopmentfrom
135-add-separate-model-config-to-out-of-scope-checker
Feb 12, 2026
Merged

feat: add ability to specify guardrails model #135#136
navalnica merged 1 commit intodevelopmentfrom
135-add-separate-model-config-to-out-of-scope-checker

Conversation

@bahdan111
Copy link
Contributor

@bahdan111 bahdan111 commented Feb 12, 2026

Applicable issues

Resolves #135

Description of changes

Add separate llm_model_config to OutOfScopeConfig for guardrails, allowing independent model configuration from the supreme agent.

Changes:

  • statgpt/common/schemas/channel.py: Added llm_model_config field to OutOfScopeConfig with default LLMModelConfig
  • statgpt/app/chains/out_of_scope_checker.py: Updated to use out_of_scope.llm_model_config
  • configurations/clients/sample/channels.yaml: Added example config

Checklist

By submitting this pull request, I confirm that my contribution is made under the terms of the MIT license.

@Fedir-Yatsenko Fedir-Yatsenko self-requested a review February 12, 2026 08:57
@Fedir-Yatsenko Fedir-Yatsenko added enhancement New feature or request python Pull requests that update python code labels Feb 12, 2026
Copy link
Collaborator

@Fedir-Yatsenko Fedir-Yatsenko left a comment

Choose a reason for hiding this comment

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

Looks good

@bahdan111
Copy link
Contributor Author

bahdan111 commented Feb 12, 2026

/deploy-review

GitHub actions run: 21940016646
Environment URL: review-environment | pipeline

@navalnica navalnica marked this pull request as ready for review February 12, 2026 09:28
@navalnica navalnica requested a review from ypldan as a code owner February 12, 2026 09:28
@bahdan111
Copy link
Contributor Author

tested, changed the model via config and made sure change propagated

@navalnica
Copy link
Contributor

tested, changed the model via config and made sure change propagated

how did you make sure changes propagated?

@bahdan111
Copy link
Contributor Author

bahdan111 commented Feb 12, 2026

tested, changed the model via config and made sure change propagated

how did you make sure changes propagated?

sent question and then checked grafana logs for review env

@navalnica
Copy link
Contributor

sent question and then checked grafana logs for review env

both guardrails (new LLM) and agent logs (old LLM)?

@bahdan111
Copy link
Contributor Author

sent question and then checked grafana logs for review env

both guardrails (new LLM) and agent logs (old LLM)?

yeah, so checked that default model works, without changing the config, and then changed the config to a new model

@navalnica navalnica merged commit ce988e3 into development Feb 12, 2026
11 of 12 checks passed
@navalnica navalnica deleted the 135-add-separate-model-config-to-out-of-scope-checker branch February 12, 2026 11:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request python Pull requests that update python code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add separate llm_model_config to OutOfScopeConfig

3 participants