Skip to content

feat(agents): let callers choose the report tool - #1550

Merged
ian-at-strix merged 3 commits into
mainfrom
feat/drop-base-tools
Oct 21, 2026
Merged

ian-at-strix merged 3 commits into
mainfrom
feat/drop-base-tools

Conversation

@ian-at-strix

@ian-at-strix ian-at-strix commented Oct 20, 2026 •

Copy link
Copy Markdown
Contributor

run_strix_scan and build_strix_agent now take report_tool (default create_vulnerability_report). Every agent, root and children, gets that tool in place of create_vulnerability_report, and the rendered system prompt names it.

strix-pro passes create_pr_vulnerability_report for PR reviews. Until now subagents there had both tools, and their prompt told them to use the generic one. They filed confirmed bugs with it in 217 calls across 123 eval runs, and the review never counted them: about 3.7 points of catch rate. Same pattern as #1496 (root_finish_tool).

@ian-at-strix ian-at-strix changed the title feat(agents): let callers remove base tools from scan agents feat(agents): let callers choose the report tool Oct 20, 2026
@ian-at-strix
ian-at-strix marked this pull request as ready for review October 20, 2026 23:46
@greptile-apps

greptile-apps Bot commented Oct 20, 2026 •

Copy link
Copy Markdown
Contributor

RetriggerView in GreptileConfidence Score: 5/5

[Medium impact] The changes since the previous review introduce no new blocking issue.

Summary

Lets callers choose the reporting tool for root and child agents.

  • Scan callers can choose which tool every agent uses to file findings.

Reviews (2) · Last reviewed commit: "refactor(agents): name the report tool v..." · Reviewed by Greptile

Comment thread strix/agents/prompt.py Outdated
Comment thread strix/agents/prompt.py Outdated
@ian-at-strix

Copy link
Copy Markdown
Contributor Author

@greptileai

@ian-at-strix
ian-at-strix merged commit 3a3855f into main Oct 21, 2026
12 of 15 checks passed
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