Skip to content

Inline claude auto review workflow#130

Open
amdove wants to merge 2 commits intomainfrom
add-claude-auto-review
Open

Inline claude auto review workflow#130
amdove wants to merge 2 commits intomainfrom
add-claude-auto-review

Conversation

@amdove
Copy link
Copy Markdown
Contributor

@amdove amdove commented Apr 29, 2026

Summary

  • Replaces the thin caller with the full inlined workflow
  • posit-dev/team-operator is public, so it cannot call reusable workflows from internal/private repos even with permissions (GitHub limitation)
  • Prompt defaults to APPROVE with inline comments; only blocks on critical issues

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@amdove amdove changed the title Switch to reusable Claude auto-review workflow from ptd-workspace Switch to reusable Claude auto-review workflow Apr 29, 2026
@amdove amdove changed the title Switch to reusable Claude auto-review workflow ci: switch to reusable Claude auto-review workflow Apr 29, 2026
@amdove amdove requested a review from timtalbot April 29, 2026 21:35
posit-dev/team-operator is public so it cannot call reusable workflows
from internal/private repos. Inline the full workflow directly.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@amdove amdove changed the title ci: switch to reusable Claude auto-review workflow Inline claude auto review workflow May 4, 2026
Copy link
Copy Markdown
Contributor

@statik statik left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks for working on this! some questions about this change so marked as request changes


- uses: aws-actions/configure-aws-credentials@v4
with:
role-to-assume: arn:aws:iam::${{ env.PTD_AWS_ACCOUNT }}:role/claude-code
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

was it deliberate to move off of bedrock APIs?

timeout_minutes: "60"
direct_prompt: |
Please review this PR following the guidelines in `.claude/review-guidelines.md`. Use the GitHub review system:
claude_code_oauth_token: ${{ secrets.CLAUDE_CODE_OAUTH_TOKEN }}
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would like to understand more about this change to claude code oauth token instead of bedrock APIs, I thought we would still use bedrock for this use case.

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