Skip to content

chore(OBE-11298): add Qodo PR agent config - #140

Open
JuanMantica45 wants to merge 1 commit into
Sentinel-One:masterfrom
JuanMantica45:jm-obe-11298-qodo-onboarding
Open

chore(OBE-11298): add Qodo PR agent config#140
JuanMantica45 wants to merge 1 commit into
Sentinel-One:masterfrom
JuanMantica45:jm-obe-11298-qodo-onboarding

Conversation

@JuanMantica45

Copy link
Copy Markdown
Contributor

Summary

Adds .pr_agent.toml to onboard this repo to Qodo (SentinelOne's AI code-review platform), using the same quiet-hardened Balanced preset already merged in manager and now proposed for pipeline, common, helm, and dataplane-private.

The preset turns off two noisy stock defaults flagged during the Qodo pilot:

  • enable_auto_checks_feedback = false — suppresses full CI-log-dump comments, which add no signal beyond the check itself.
  • pr_commands = ["/agentic_review"] — drops Qodo's auto-describe comment, which is redundant with our own PR descriptions.

The TODO: VERIFY comment about prefer_single_line_comments is carried over verbatim: pilot feedback recommended it, but it is not a documented Qodo config key (config reference), so it is deliberately left unset rather than shipped as a silent no-op.

Why

Config-only, so that when the Qodo app is installed on this repo it reviews with low-noise defaults from day one instead of the stock preset. Keeping the file byte-identical across our repos (apart from the repo name in the header comment) means they all behave the same under review and can be retuned in one pass.

Note: unlike our GitHub Enterprise repos, this repo lives on github.com, so the Qodo app installation is not covered by the s1-ghe-admin allowlist — that has to be arranged separately. This PR only lands the config.

Ticket: OBE-11298

Test plan

  • Config-only change; no Rust code touched, no behavior to unit test.
  • Verified .pr_agent.toml parses as valid TOML.
  • Verified the file body is byte-identical to the version merged in manager.

🤖 Generated with Claude Code

Sign up for freeto 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.

1 participant

@JuanMantica45