Skip to content

chore: update models in model configs#83

Merged
p- merged 3 commits into
mainfrom
p--models-update
May 21, 2026
Merged

chore: update models in model configs#83
p- merged 3 commits into
mainfrom
p--models-update

Conversation

@p-
Copy link
Copy Markdown
Contributor

@p- p- commented May 19, 2026

No description provided.

Copilot AI review requested due to automatic review settings May 19, 2026 07:41
Copy link
Copy Markdown
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

Updates model configuration files to point to newer model versions (gpt-5.4, claude-sonnet-4.6), switches to the responses API type, and consolidates by removing the codeql_python-specific config in favor of the shared model_config.

Changes:

  • Bump model versions and add api_type: responses settings in model_config.yaml and model_config_lowercost.yaml.
  • Delete model_config_codeql_python.yaml and update the audit taskflow to reference the generic model_config instead.
Show a summary per file
File Description
src/seclab_taskflows/configs/model_config.yaml Updates models to gpt-5.4 / claude-sonnet-4.6 and replaces temperature with api_type: responses for code_analysis and general_tasks.
src/seclab_taskflows/configs/model_config_lowercost.yaml Updates to gpt-5.4-mini and adds api_type: responses settings.
src/seclab_taskflows/configs/model_config_codeql_python.yaml Deletes the codeql_python-specific model config.
src/seclab_taskflows/taskflows/audit/remote_sources_local.yaml Switches the audit taskflow to use the generic model_config after the codeql_python config removal.

Copilot's findings

Tip

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

  • Files reviewed: 4/4 changed files
  • Comments generated: 0

@p- p- merged commit 1eafe13 into main May 21, 2026
13 checks passed
@p- p- deleted the p--models-update branch May 21, 2026 12:01
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.

3 participants