Skip to content

Comments

chore: point PR review workflows to team-skills#2246

Merged
nick-inkeep merged 1 commit intomainfrom
feat/pr-review-to-team-skills
Feb 22, 2026
Merged

chore: point PR review workflows to team-skills#2246
nick-inkeep merged 1 commit intomainfrom
feat/pr-review-to-team-skills

Conversation

@nick-inkeep
Copy link
Collaborator

Summary

  • Updates both Claude CI workflows to load PR review agents and skills from inkeep/team-skills/ci/ instead of repo-local files or internal-cc-plugins
  • Old pr-review agent files intentionally kept in place as inert fallback until validated
  • Requires team-skills PR #5 to be merged first

Changes

claude-code-review.yml

  • Add GitHub App token generation step (INTERNAL_CI_APP_ID) for team-skills access
  • Add shallow clone of inkeep/team-skills to /tmp/team-skills
  • Add --plugin-dir /tmp/team-skills/ci/pr-review to claude_args

closed-pr-review-auto-improver.yml

  • Change token repositories from internal-cc-pluginsteam-skills
  • Clone team-skills instead of internal-cc-plugins
  • Update --plugin-dir to /tmp/team-skills/ci/closed-pr-review-auto-improver
  • Update prompt to reference team-skills paths for agent edits and PR target

Test plan

  • Merge team-skills PR update docs #5 first
  • Merge this PR
  • Open a test PR to trigger claude-code-review.yml — verify agents load from team-skills
  • Merge test PR to trigger closed-pr-review-auto-improver.yml — verify auto-improver targets team-skills
  • After validation, submit cleanup PR (remove old agent files)

🤖 Generated with Claude Code

Update both Claude CI workflows to load agents and skills from
inkeep/team-skills/ci/ instead of repo-local files or internal-cc-plugins.

claude-code-review.yml:
- Add GitHub App token generation step for team-skills access
- Add shallow clone of team-skills to /tmp/team-skills
- Add --plugin-dir /tmp/team-skills/ci/pr-review to claude_args

closed-pr-review-auto-improver.yml:
- Change token repositories from internal-cc-plugins to team-skills
- Clone team-skills instead of internal-cc-plugins
- Update --plugin-dir to /tmp/team-skills/ci/closed-pr-review-auto-improver
- Update prompt to reference team-skills paths for agent edits and PR target

Old pr-review agent files are intentionally kept in place as inert fallback
until the new workflow is validated. Cleanup will follow in a separate PR.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@changeset-bot
Copy link

changeset-bot bot commented Feb 22, 2026

⚠️ No Changeset found

Latest commit: 33a1408

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@vercel
Copy link

vercel bot commented Feb 22, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
agents-api Ready Ready Preview, Comment Feb 22, 2026 1:42am
agents-docs Ready Ready Preview, Comment Feb 22, 2026 1:42am
agents-manage-ui Building Building Preview, Comment Feb 22, 2026 1:42am

Request Review

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.

1 participant