Skip to content

refactor: centralize PR review agent profiles - #222

Closed
robbycochran wants to merge 1 commit into
refactor/workflow-code-layoutfrom
codex/pr-review-agent-cleanup
Closed

robbycochran wants to merge 1 commit into
refactor/workflow-code-layoutfrom
codex/pr-review-agent-cleanup

Conversation

@robbycochran

Copy link
Copy Markdown
Collaborator

Summary

  • Centralize production PR-review agent profile selection in one case block.
  • Remove repeated OpenCode/Codex lifecycle and workflow-selection conditionals.
  • Keep OpenCode and Codex as the two production adapters.
  • Document the Claude workflow as an offline configuration fixture, not a third production PR-review mode.

Validation

  • go build ./...
  • go vet ./...
  • CGO_ENABLED=0 go test ./...
  • make test-suite — 11/11 passed, 1 skipped
  • actionlint, shellcheck, shell syntax, and git diff --check

This PR is labeled stackrox-ai-review so the merged Codex review flow can review it.

@coderabbitai

coderabbitai Bot commented Sep 18, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository: stackrox/harness-openshell/.coderabbit.yaml

Review profile: CHILL

Plan: Enterprise

Run ID: 8a88c0d7-2c88-4358-9374-b33800bd8aab

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

Comment @coderabbitai help to get the list of available commands.

@stackrox-openshell-workflow stackrox-openshell-workflow Bot added the stackrox-ai-review Opt in to StackRox AI review label Sep 18, 2026
@robbycochran

Copy link
Copy Markdown
Collaborator Author

Closing this PR because it targets the divergent refactor/workflow-code-layout branch; recreate it on main if the refactor is still needed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

stackrox-ai-review Opt in to StackRox AI review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant