Skip to content

various fixes#66

Merged
m-y-mo merged 1 commit intomainfrom
model_config
Mar 11, 2026
Merged

various fixes#66
m-y-mo merged 1 commit intomainfrom
model_config

Conversation

@m-y-mo
Copy link
Contributor

@m-y-mo m-y-mo commented Mar 11, 2026

No description provided.

@m-y-mo m-y-mo requested review from JarLob and p- as code owners March 11, 2026 12:26
Copilot AI review requested due to automatic review settings March 11, 2026 12:26
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

Streamlines and tunes seclab taskflows/config to remove redundant audit steps, increase parallelism for local iterative audits, and reduce default reasoning effort for the code_analysis model configuration.

Changes:

  • Removes a redundant repo-context-only follow-up task from identify_applications.yaml, leaving the existing per-component analyses as the source of entry points/user actions.
  • Enables async execution for the iterative local issue-audit task to allow parallel audits.
  • Lowers code_analysis reasoning effort from high to medium in the default model config.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
src/seclab_taskflows/taskflows/audit/identify_applications.yaml Removes a redundant task and keeps the needed toolboxes on the remaining analysis step.
src/seclab_taskflows/taskflows/audit/audit_issue_local_iter.yaml Switches the iterative audit task to async: true for parallel processing.
src/seclab_taskflows/configs/model_config.yaml Adjusts default code_analysis reasoning effort to medium.

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

@m-y-mo m-y-mo merged commit bc9d2a3 into main Mar 11, 2026
13 checks passed
@m-y-mo m-y-mo deleted the model_config branch March 11, 2026 12:37
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