Skip to content

ci(sdlc): enable the code-review agent - #4168

Open
dearsimanovich-epam wants to merge 1 commit into
feat/port-agent-toolingfrom
ci/enable-code-review-agent
Open

dearsimanovich-epam wants to merge 1 commit into
feat/port-agent-toolingfrom
ci/enable-code-review-agent

Conversation

@dearsimanovich-epam

@dearsimanovich-epam dearsimanovich-epam commented Aug 12, 2026

Copy link
Copy Markdown
Contributor

Description:

Flips the harness on. One file, a rename: agents/_code-reviewagents/code-review. That underscore
is the dispatcher's on-switch — match-agents.py globs agents/*/agent.yml and skips only
underscore-prefixed directories.

Deliberately separate from #4163 so merge order can't produce a broken state. The agent invokes
/code-review-and-quality, so this must land after #4166; until it does, #4163 stays inert on its
own.

Issues:

  • No tracking issue — infra work.

Base is feat/port-agent-tooling, not development — the rename only makes sense on top of #4163,
and this keeps the diff to the one file it changes. GitHub retargets it to development automatically
when #4163 merges.

Merge order: #4163 (harness) and #4166 (skills) first, in either order, then this.

Verified locally:

  • with this tree the dispatcher discovers code-review
  • STAGE_CODE_REVIEW_ENABLED=false still suppresses it, so it can be turned off without a revert

Still required, and not repo state — set these before or after merging:

DIAL route var DIAL_CORE_URL + secret DIAL_API_KEY (optionally DIAL_ROUTE_PATH, DIAL_MODEL)
or direct secret ANTHROPIC_API_KEY

run-agent.yml picks the route automatically from whether DIAL_CORE_URL is set. Neither is
configured today — the repo currently has only ACTIONS_BOT_TOKEN and the DockerHub secrets, and no
repo variables at all. Without credentials the agent job fails fast rather than silently passing.

Two things to expect on the first run:

  • Test it on an ordinary code PR. The trust gate rejects any PR touching .claude/**,
    .github/workflows/**, agents/**, or adding a symlink — including this one. That's by design; a
    normal feature PR is what exercises the agent.
  • Check the model pin. model: claude-sonnet-4-6 resolves as anthropic.claude-sonnet-4-6 on the
    DIAL route, matching what claude-security-review.yml already uses. If that deployment isn't
    available, set DIAL_MODEL or bump the pin — an unavailable deployment fails the run.

Breaking Changes: none, but this is the point where AI review starts commenting on PRs. STAGE_CODE_REVIEW_ENABLED=false is the kill switch.

Checklist:

  • the pull request name complies with Conventional Commits
  • the pull request name ends with (Issue #<TICKET_ID>) — no tracking issue for this work

Renames `agents/_code-review` to `agents/code-review`, which is the harness's
on-switch: match-agents.py globs `agents/*/agent.yml` and skips only
underscore-prefixed directories.

Kept separate from the harness PR so merge order cannot produce a broken state.
The agent invokes `/code-review-and-quality`, so this must land after the skill
exists; until then the harness stays inert on its own.

Verified locally: the dispatcher discovers `code-review` with this tree, and
STAGE_CODE_REVIEW_ENABLED=false still suppresses it, so there is a way to turn it
off without a revert.

Still requires credentials, which are not repo state: either DIAL_CORE_URL +
DIAL_API_KEY, or ANTHROPIC_API_KEY.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@dearsimanovich-epam dearsimanovich-epam self-assigned this Aug 12, 2026
@dearsimanovich-epam
dearsimanovich-epam changed the base branch from development to feat/port-agent-tooling August 12, 2026 09:18
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