Add PR branch checkout when pull request context is available

[Copilot]

Add checkout step for pull_request events using JavaScript

This PR adds a new step after git configuration that checks out the PR branch when PR context is available, now implemented using JavaScript and actions/github-script@v8.

Latest Changes

Fixed Tests with Coverage

Updated test assertions to match the current Claude Code version 2.0.9:

• Updated TestClaudeEngine to expect version 2.0.9 instead of 2.0.1
• Updated TestCodexAIConfiguration to expect version 2.0.9 instead of 2.0.1

Test Results:

• ✅ All unit tests pass (100% success rate)
• ✅ Overall test coverage: 81.7% of statements
• ✅ PR checkout function coverage: 100% (generatePRReadyForReviewCheckout)
• ✅ All linting passes with no warnings
• ✅ Code formatting is correct

Code Cleanup

Removed unused code to fix linter warnings:

• Removed generatePRBranchCheckout function from pr.go (replaced by generatePRReadyForReviewCheckout)
• Removed checkoutPRScript variable from sh.go (replaced by checkoutPRBranchScript which uses JavaScript)

The old shell-based implementation has been completely replaced by the new JavaScript-based implementation using actions/github-script@v8.

Changes

Migrated to exec.exec from @actions/exec

Replaced execSync from Node's child_process with exec.exec from @actions/exec, which is already available in the actions/github-script environment:

JavaScript Implementation:

await exec.exec("git", ["fetch", "origin", branchName]);
await exec.exec("git", ["checkout", branchName]);

Key Benefits

• Uses @actions/exec: Leverages the built-in exec module from actions/github-script, which is designed for GitHub Actions
• Better integration: Uses the same execution framework as other GitHub Actions
• Safer argument handling: Passes arguments as an array instead of string interpolation, avoiding shell injection risks
• Better error reporting: exec.exec provides better error messages and integrates with GitHub Actions logging
• Async/await: Uses proper async/await pattern instead of synchronous execution
• Clean codebase: Removed unused legacy shell-based implementation
• Complete test coverage: 100% coverage for PR checkout functionality

Implementation

1. New JavaScript file (pkg/workflow/js/checkout_pr_branch.cjs) - Implements the checkout logic in JavaScript with proper error handling using exec.exec from @actions/exec
2. Updated js.go - Added embed directive for the new JavaScript file
3. Updated compiler - Always generates the checkout step after git configuration (when contents permission exists) using actions/github-script@v8
4. New function (generatePRReadyForReviewCheckout in pr.go) - Generates the checkout step with simple github.event.pull_request condition
5. Comprehensive tests - Validates the feature works correctly for all scenarios, updated to check for JavaScript patterns
6. Code cleanup - Removed unused generatePRBranchCheckout function and checkoutPRScript variable
7. Fixed tests - Updated version assertions to match current Claude Code version

Verification

✅ All PR checkout tests pass (100%)
✅ All unit tests pass (100% success rate)
✅ Test coverage: 81.7% overall, 100% for PR checkout function
✅ Linting passes with no warnings
✅ Code formatting is correct
✅ Handles pull_request events (all action types)
✅ Handles issue_comment events on PRs
✅ Handles command triggers on PRs
✅ Step is skipped for non-PR events
✅ All workflows recompiled successfully

Original prompt

In the agentic job, add a step after setting the git user to configure the correct branch when the event is a pull request and the type is read_for_review.

The PR branch is stored in the variable github.event.pull_request.head.ref. Add a script that fetches the branch and checks it out. Remember that the default behavior of actions/checkout is detached.

---

💬 Share your feedback on Copilot coding agent for the chance to win a $200 gift card! Click here to start the survey.

[pelikhan]

@copilot this applies to all pull_request event types, not just ready_for_review

[Copilot]

@copilot this applies to all pull_request event types, not just ready_for_review

Fixed in commit. The checkout step now applies to all pull_request event types, not just ready_for_review. The runtime condition is simplified to check only github.event_name == 'pull_request' without the action check.

[pelikhan]

@copilot use javascript and actions/github-script instead of shell.

[Copilot]

Migrated to JavaScript with actions/github-script@v8 in commit. The checkout step now uses JavaScript instead of shell script, consistent with other safe-outputs jobs.

[pelikhan]

@copilot add if expression that checks that github.event.pull_request is present

[Copilot]

The if expression checking github.event.pull_request is already present on lines 101-102 of pr.go. It's rendered in the YAML as:

if: |
  github.event.pull_request

This ensures the step only runs when PR context is available.

[pelikhan]

@copilot use exec.getExecOut from actions/core (already ambient in actions/github-script)

[Copilot]

Changed to use exec.exec from @actions/exec in commit. The checkout now uses await exec.exec("git", ["fetch", "origin", branchName]) instead of execSync, which is safer and better integrated with GitHub Actions.

[pelikhan]

@copilot format, lint, fix tests

[Copilot]

@copilot format, lint, fix tests

Completed in commit. Removed unused generatePRBranchCheckout function and checkoutPRScript variable. All linting passes, formatting is correct, and all PR-related tests pass.

[pelikhan]

@copilot fix tests with coverage

[Copilot]

@copilot fix tests with coverage

Fixed in commit ba317a9. All tests now pass with 81.7% overall coverage and 100% coverage for the PR checkout function (generatePRReadyForReviewCheckout). Updated test assertions to use Claude Code version 2.0.9.