Skip to content

Commit c698bb6

Browse files
committed
chore: add more logs
Signed-off-by: exploreriii <133720349+exploreriii@users.noreply.github.com>
1 parent a39f424 commit c698bb6

2 files changed

Lines changed: 93 additions & 8 deletions

File tree

.github/scripts/bots/assign-03-intermediate-check.js

Lines changed: 51 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,28 +1,59 @@
1-
const { hasIntermediateEligibility } = require('../lib/eligibility/has-eligibility-03-intermediate');
2-
const { rejectionRouter } = require('../lib/comments/rejection-router');
1+
const { hasIntermediateEligibility } =
2+
require('../lib/eligibility/has-eligibility-03-intermediate');
3+
const { rejectionRouter } =
4+
require('../lib/comments/rejection-router');
35

46
const INTERMEDIATE_LABEL = 'Intermediate';
57

68
function isIntermediate(issue) {
7-
return (issue.labels ?? []).some(l => l.name === INTERMEDIATE_LABEL);
9+
return (issue.labels ?? []).some(
10+
l => l.name?.toLowerCase() === INTERMEDIATE_LABEL.toLowerCase()
11+
);
812
}
913

1014
module.exports = async ({ github, context }) => {
15+
console.log('[intermediate-enforce] Workflow triggered', {
16+
action: context.payload.action,
17+
issue: context.payload.issue?.number,
18+
assignee: context.payload.assignee?.login,
19+
labels: context.payload.issue?.labels?.map(l => l.name),
20+
});
21+
1122
const { issue, assignee } = context.payload;
1223
const { owner, repo } = context.repo;
1324

14-
if (!issue || !assignee) return;
15-
if (!isIntermediate(issue)) return;
25+
if (!issue || !assignee) {
26+
console.log('[intermediate-enforce] Exit: missing issue or assignee');
27+
return;
28+
}
29+
30+
if (!isIntermediate(issue)) {
31+
console.log('[intermediate-enforce] Exit: issue is not Intermediate');
32+
return;
33+
}
1634

1735
const username = assignee.login;
1836

37+
console.log('[intermediate-enforce] Checking eligibility', {
38+
issue: issue.number,
39+
username,
40+
});
41+
1942
const result = await hasIntermediateEligibility({
2043
github,
2144
owner,
2245
repo,
2346
username,
2447
});
2548

49+
console.log('[intermediate-enforce] Eligibility result', {
50+
issue: issue.number,
51+
username,
52+
eligible: result.eligible,
53+
reason: result.reason,
54+
context: result.context,
55+
});
56+
2657
if (result.eligible) {
2758
console.log('[intermediate-enforce] Assignment allowed', {
2859
issue: issue.number,
@@ -32,6 +63,11 @@ module.exports = async ({ github, context }) => {
3263
}
3364

3465
// ❌ Not eligible → unassign
66+
console.log('[intermediate-enforce] Reverting assignment', {
67+
issue: issue.number,
68+
username,
69+
});
70+
3571
await github.rest.issues.removeAssignees({
3672
owner,
3773
repo,
@@ -46,12 +82,22 @@ module.exports = async ({ github, context }) => {
4682
});
4783

4884
if (body) {
85+
console.log('[intermediate-enforce] Posting rejection comment', {
86+
issue: issue.number,
87+
username,
88+
reason: result.reason,
89+
});
90+
4991
await github.rest.issues.createComment({
5092
owner,
5193
repo,
5294
issue_number: issue.number,
5395
body,
5496
});
97+
} else {
98+
console.log('[intermediate-enforce] No rejection message generated', {
99+
reason: result.reason,
100+
});
55101
}
56102

57103
console.log('[intermediate-enforce] Assignment reverted', {

.github/scripts/bots/assign-04-advanced-check.js

Lines changed: 42 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -6,15 +6,31 @@ const { rejectionRouter } =
66
const ADVANCED_LABEL = 'Advanced';
77

88
function isAdvanced(issue) {
9-
return (issue.labels ?? []).some(l => l.name === ADVANCED_LABEL);
9+
return (issue.labels ?? []).some(
10+
l => l.name?.toLowerCase() === ADVANCED_LABEL.toLowerCase()
11+
);
1012
}
1113

1214
module.exports = async ({ github, context }) => {
15+
console.log('[advanced-enforce] Workflow triggered', {
16+
action: context.payload.action,
17+
issue: context.payload.issue?.number,
18+
assignee: context.payload.assignee?.login,
19+
labels: context.payload.issue?.labels?.map(l => l.name),
20+
});
21+
1322
const { issue, assignee } = context.payload;
1423
const { owner, repo } = context.repo;
1524

16-
if (!issue || !assignee) return;
17-
if (!isAdvanced(issue)) return;
25+
if (!issue || !assignee) {
26+
console.log('[advanced-enforce] Exit: missing issue or assignee');
27+
return;
28+
}
29+
30+
if (!isAdvanced(issue)) {
31+
console.log('[advanced-enforce] Exit: issue is not Advanced');
32+
return;
33+
}
1834

1935
const username = assignee.login;
2036

@@ -30,6 +46,14 @@ module.exports = async ({ github, context }) => {
3046
username,
3147
});
3248

49+
console.log('[advanced-enforce] Eligibility result', {
50+
issue: issue.number,
51+
username,
52+
eligible: result.eligible,
53+
reason: result.reason,
54+
context: result.context,
55+
});
56+
3357
if (result.eligible) {
3458
console.log('[advanced-enforce] Assignment allowed', {
3559
issue: issue.number,
@@ -39,6 +63,11 @@ module.exports = async ({ github, context }) => {
3963
}
4064

4165
// ❌ Not eligible → revert assignment
66+
console.log('[advanced-enforce] Reverting assignment', {
67+
issue: issue.number,
68+
username,
69+
});
70+
4271
await github.rest.issues.removeAssignees({
4372
owner,
4473
repo,
@@ -53,12 +82,22 @@ module.exports = async ({ github, context }) => {
5382
});
5483

5584
if (body) {
85+
console.log('[advanced-enforce] Posting rejection comment', {
86+
issue: issue.number,
87+
username,
88+
reason: result.reason,
89+
});
90+
5691
await github.rest.issues.createComment({
5792
owner,
5893
repo,
5994
issue_number: issue.number,
6095
body,
6196
});
97+
} else {
98+
console.log('[advanced-enforce] No rejection message generated', {
99+
reason: result.reason,
100+
});
62101
}
63102

64103
console.log('[advanced-enforce] Assignment reverted', {

0 commit comments

Comments
 (0)