Skip to content

Commit 1569634

Browse files
Copilotfadidurah
andcommitted
Final refinements: improve placeholders, clarify P0 priority, document fallback
Co-authored-by: fadidurah <88730756+fadidurah@users.noreply.github.com>
1 parent 794c262 commit 1569634

File tree

1 file changed

+5
-5
lines changed

1 file changed

+5
-5
lines changed

.github/workflows/copilot-issue-response.yml

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -706,11 +706,11 @@ jobs:
706706
response += `### 📊 Help Us Prioritize\n\n`;
707707
response += `To help the team prioritize this issue, please provide information about customer impact:\n\n`;
708708
response += `\`\`\`\n`;
709-
response += `PING-COPILOT: We have [number] devices/users affected by this issue\n`;
709+
response += `PING-COPILOT: We have <number> devices/users affected by this issue\n`;
710710
response += `\`\`\`\n\n`;
711711
response += `Or if this is blocking production or P0 scenarios:\n`;
712712
response += `\`\`\`\n`;
713-
response += `PING-COPILOT: This is a P0/production-blocking issue affecting [description]\n`;
713+
response += `PING-COPILOT: This is a P0/production-blocking issue affecting <specific impact details>\n`;
714714
response += `\`\`\`\n\n`;
715715
response += `---\n\n`;
716716
}
@@ -804,15 +804,15 @@ jobs:
804804
}
805805
}
806806
807-
// Check for P0 indicators
807+
// Check for P0 indicators (always set P0 if detected, as it should override other priorities)
808808
if (requestLower.includes('p0') || requestLower.includes('production') ||
809809
requestLower.includes('critical') || requestLower.includes('blocking') ||
810810
requestLower.includes('urgent')) {
811811
priorityLevel = 'P0';
812812
if (!impactReason) impactReason = 'Production/P0 issue reported';
813813
}
814814
815-
// Check for P1 indicators
815+
// Check for P1 indicators (only if no higher priority already set)
816816
if (!priorityLevel && (requestLower.includes('p1') || requestLower.includes('major'))) {
817817
priorityLevel = 'P1';
818818
if (!impactReason) impactReason = 'Major issue reported';
@@ -857,7 +857,7 @@ jobs:
857857
owner: context.repo.owner,
858858
repo: context.repo.repo,
859859
name: priorityLevel,
860-
color: labelColors[priorityLevel] || 'fbca04',
860+
color: labelColors[priorityLevel] || 'fbca04', // Fallback shouldn't be reached with current logic
861861
description: `Priority ${priorityLevel} issue`
862862
});
863863
} catch (createError) {

0 commit comments

Comments
 (0)