Skip to content

Refactor PR and issue label handling logic #951

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
sanjay-kv merged 1 commit into from
Oct 19, 2025
Merged

Refactor PR and issue label handling logic #951

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
33 changes: 4 additions & 29 deletions .github/workflows/autolabel-pr-issue.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -28,11 +28,9 @@ jobs:

// Check if triggered by issue event
if (context.eventName === 'issues') {
// Find all open PRs that link to this issue
const issueNumber = context.payload.issue.number;
console.log(`Issue #${issueNumber} labels were updated`);

// Search for PRs that mention this issue
const { data: pullRequests } = await github.rest.pulls.list({
owner: context.repo.owner,
repo: context.repo.repo,
Expand All @@ -46,7 +44,6 @@ jobs:
new RegExp(`(?:close[sd]?|fix(?:e[sd])?|resolve[sd]?)\\s+#${issueNumber}\\b`, 'i'),
new RegExp(`#${issueNumber}\\b`)
];

if (patterns.some(p => p.test(prText))) {
linkedPRs.push(pr.number);
}
Expand All @@ -60,7 +57,6 @@ jobs:
console.log(`Found linked PRs: ${linkedPRs.join(', ')}`);
return JSON.stringify({ prs: linkedPRs, issue: issueNumber });
} else {
// Triggered by PR event - original logic
prBody = context.payload.pull_request.body || '';
prTitle = context.payload.pull_request.title || '';

Expand All @@ -74,9 +70,7 @@ jobs:

patterns.forEach(pattern => {
const matches = [...textToSearch.matchAll(pattern)];
matches.forEach(match => {
issueNumbers.add(match[1]);
});
matches.forEach(match => issueNumbers.add(match[1]));
});

const issues = Array.from(issueNumbers);
Expand All @@ -97,19 +91,13 @@ jobs:
script: |
const extractData = JSON.parse('${{ steps.extract-issues.outputs.result }}');

// Labels to exclude from being applied to PRs
const excludedLabels = ['recode', 'hacktoberfest-accepted'];

let issueNumbers = [];
let prsToUpdate = [];

// Handle both PR and issue events
if (extractData.issue) {
// Issue event - update all linked PRs
issueNumbers = [extractData.issue];
prsToUpdate = extractData.prs || [];
} else {
// PR event - update the current PR
issueNumbers = extractData.issues || [];
prsToUpdate = extractData.prs || [];
}
Expand All @@ -130,15 +118,7 @@ jobs:
});

console.log(`Issue #${issueNumber} labels:`, issue.data.labels.map(l => l.name));

issue.data.labels.forEach(label => {
// Only add label if it's not in the excluded list
if (!excludedLabels.includes(label.name.toLowerCase())) {
allLabels.add(label.name);
} else {
console.log(`Excluding label: ${label.name}`);
}
});
issue.data.labels.forEach(label => allLabels.add(label.name));
} catch (error) {
console.log(`Could not fetch issue #${issueNumber}:`, error.message);
}
Expand Down Expand Up @@ -168,17 +148,14 @@ jobs:
return;
}

// Update each PR
for (const prNumber of prsToUpdate) {
try {
// First, get current PR labels
const { data: pr } = await github.rest.pulls.get({
owner: context.repo.owner,
repo: context.repo.repo,
pull_number: prNumber
});

// Remove all existing labels first (to handle removed issue labels)
const currentLabels = pr.labels.map(l => l.name);
if (currentLabels.length > 0) {
for (const label of currentLabels) {
Expand All @@ -195,17 +172,15 @@ jobs:
}
}

// Apply new labels
await github.rest.issues.addLabels({
owner: context.repo.owner,
repo: context.repo.repo,
issue_number: prNumber,
labels: labels
});

console.log(`Successfully applied ${labels.length} labels to PR #${prNumber}`);
console.log(`✅ Successfully applied ${labels.length} labels to PR #${prNumber}`);

// Add a comment to the PR
await github.rest.issues.createComment({
owner: context.repo.owner,
repo: context.repo.repo,
Expand All @@ -215,4 +190,4 @@ jobs:
} catch (error) {
console.error(`Error updating PR #${prNumber}:`, error.message);
}
}
}
Loading