Skip to content

Commit 83b80ee

Browse files
committed
ci(verify-command): don't self-react; pass comment_id so the bot acks
The relay runs under a human PAT, so reacting 👀 here looks like the requester reacting to themselves. Drop it and pass comment_id to TechEngine, which adds the reaction as TechEngineBot. Refs #1
1 parent 8b24e4d commit 83b80ee

1 file changed

Lines changed: 5 additions & 8 deletions

File tree

.github/workflows/verify-command.yml

Lines changed: 5 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -38,14 +38,10 @@ jobs:
3838
script: |
3939
const { owner, repo } = context.repo;
4040
const issue_number = context.issue.number;
41-
// 👀 reaction so the requester sees the command was picked up.
42-
try {
43-
await github.rest.reactions.createForIssueComment({
44-
owner, repo, comment_id: context.payload.comment.id, content: 'eyes',
45-
});
46-
} catch (e) {
47-
core.info(`reaction skipped: ${e.message}`);
48-
}
41+
// No reaction here: this step runs under the relay token (a human PAT),
42+
// so reacting would look like the requester reacting to themselves.
43+
// The 👀 acknowledgement is added by TechEngineBot on the engine side
44+
// (it holds the bot token); we just pass the comment id through.
4945
const pr = await github.rest.pulls.get({ owner, repo, pull_number: issue_number });
5046
await github.rest.repos.createDispatchEvent({
5147
owner: 'GetTechAPI',
@@ -54,6 +50,7 @@ jobs:
5450
client_payload: {
5551
pr_number: String(issue_number),
5652
head_sha: pr.data.head.sha,
53+
comment_id: context.payload.comment.id,
5754
requested_by: context.payload.comment.user.login,
5855
},
5956
});

0 commit comments

Comments
 (0)