skills/pbakaus_impeccable
1
0
Fork 0
mirror of https://github.com/pbakaus/impeccable.git synced 2026-09-12 22:26:38 +03:00
Code Issues Packages Projects Releases Wiki Activity

Reconcile the ai-assisted label for exempt authors too

Browse Source 
The exempt early-return skipped disclosure reconciliation, so a stale
ai-assisted label on a maintainer or member issue could never clear.
Disclosure labeling now mirrors the body for every author; the
exemption covers only the template and length gates. Per Greptile's
follow-up finding on #518.

AI-assisted: written with an AI agent under maintainer direction.
Co-authored-by: Cursor <cursoragent@cursor.com>
This commit is contained in:
Abdul Wahab
2026-08-05 22:48:30 +05:00
co-authored by Cursor
parent 281352ca53
commit ed4df9f8c2
2 changed files with 25 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files
scripts/github/issue-gate.mjs
+8 -5
View File
@@ -139,11 +139,17 @@ export function evaluateIssue(issue, options = {}) {
shouldReopen: false,
};
// Disclosure labeling mirrors the body for every author, exempt or not; the
// exemption below only covers the template and length gates.
const body = String(issue.body || '');
base.aiAssisted = AI_DISCLOSURE.test(body);
if (base.aiAssisted && !labels.has(AI_LABEL)) base.labelsToAdd.push(AI_LABEL);
if (!base.aiAssisted && labels.has(AI_LABEL)) base.labelsToRemove.push(AI_LABEL);
if (maintainers.has(author) || EXEMPT_ASSOCIATIONS.has(issue.authorAssociation)) {
return { ...base, exempt: true };
}
const body = String(issue.body || '');
const bodyHeadings = new Set(extractHeadings(body));
const anyTemplateHeading = templates.some(
(template) => template.headings.some((heading) => bodyHeadings.has(heading)),
@@ -179,9 +185,6 @@ export function evaluateIssue(issue, options = {}) {
if (verdict !== 'reject' && reasons.length > 0) verdict = 'needs-work';
const plan = { ...base, verdict, reasons };
plan.aiAssisted = AI_DISCLOSURE.test(body);
if (plan.aiAssisted && !labels.has(AI_LABEL)) plan.labelsToAdd.push(AI_LABEL);
if (!plan.aiAssisted && labels.has(AI_LABEL)) plan.labelsToRemove.push(AI_LABEL);
if (verdict === 'pass') {
if (labels.has(GATE_LABEL)) plan.labelsToRemove.push(GATE_LABEL);
@@ -326,8 +329,8 @@ function runCommentGate(repo, options) {
if (options.apply) minimizeComment(comment.nodeId);
}
// Exempt plans still reach here: they carry only disclosure-label actions.
function applyIssuePlan(repo, plan) {
if (plan.exempt) return;
if (plan.shouldReopen) reopenIssue(repo, plan.number);
for (const label of plan.labelsToRemove) removeLabel(repo, plan.number, label);
if (plan.labelsToAdd.length > 0) addLabels(repo, plan.number, plan.labelsToAdd);