-
-
Notifications
You must be signed in to change notification settings - Fork 0
Closed
Labels
agent-qaTesting and verification agentTesting and verification agentarea-workflowsGitHub Actions workflowsGitHub Actions workflowsenhancementNew feature or requestNew feature or requestpriority:P2Normal: Standard enhancement or bug fix, moderate impactNormal: Standard enhancement or bug fix, moderate impact
Description
Origin
PR #202 comment 2639354259 from Copilot
Current Behavior
Pattern regex uses \d+ to match any digits in branch name copilot/sub-pr-{number}.
No validation that extracted number matches the original PR number parameter.
Enhancement
Validate extracted PR number:
if ($headRef -match 'copilot/sub-pr-(\d+)') {
$extractedPR = [int]$matches[1]
return $extractedPR -eq $PRNumber
}
return $falseRationale
Prevents false positives if repository has multiple Copilot follow-up branches.
Acceptance Criteria
- Extract PR number from branch name
- Compare against $PRNumber parameter
- Return false if mismatch
- Add Pester test for validation logic
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
agent-qaTesting and verification agentTesting and verification agentarea-workflowsGitHub Actions workflowsGitHub Actions workflowsenhancementNew feature or requestNew feature or requestpriority:P2Normal: Standard enhancement or bug fix, moderate impactNormal: Standard enhancement or bug fix, moderate impact