8 Commits

9 changed files with 82 additions and 16 deletions

3
AUTHORS.md Normal file
View File

@@ -0,0 +1,3 @@
### Authors
- joe fleming ([w33ble](https://github.com/w33ble))

8
CHANGELOG.md Normal file
View File

@@ -0,0 +1,8 @@
### Changelog
#### 0.1.0 (18 October 2018)
- docs: update readme, add sample env [`a28d6a2`](https://git.w33ble.com/w33ble/gh-action-bot/commit/a28d6a25096efb9c8acea3e8d26e7f7f50dfd6b5)
- feat: add retest comment, delete it [`ced41f4`](https://git.w33ble.com/w33ble/gh-action-bot/commit/ced41f4a495823205e5769e4531f923a982d29b4)
- feat: process 2 pages of events [`c117cb8`](https://git.w33ble.com/w33ble/gh-action-bot/commit/c117cb847d5cb7a6b57b705e0c192cc725c0d646)
- feat: fetch data for prs that need retest [`d1cfd7c`](https://git.w33ble.com/w33ble/gh-action-bot/commit/d1cfd7c20b17802f2de1b9153ae9a8392ed1bc44)
- feat: add history tracking [`b6b5c27`](https://git.w33ble.com/w33ble/gh-action-bot/commit/b6b5c27ffc43ff2e1c1ccf041958021cf7c8c4d4)

View File

@@ -1,10 +1,16 @@
# gh-action-bot # gh-action-bot
Action bot for Github. Action bot for Github. Checks a repo's event stream and leaves a ghost comment when your criteria is met.
[![GitHub license](https://img.shields.io/badge/license-MIT-blue.svg)](https://raw.githubusercontent.com/w33ble/gh-action-bot/master/LICENSE) [![GitHub license](https://img.shields.io/badge/license-MIT-blue.svg)](https://raw.githubusercontent.com/w33ble/gh-action-bot/master/LICENSE)
[![npm](https://img.shields.io/npm/v/gh-action-bot.svg)](https://www.npmjs.com/package/gh-action-bot) [![Project Status](https://img.shields.io/badge/status-stable-green.svg)](https://nodejs.org/api/documentation.html#documentation_stability_index)
[![Project Status](https://img.shields.io/badge/status-experimental-orange.svg)](https://nodejs.org/api/documentation.html#documentation_stability_index)
## Usage
`node . owner/repo`
You'll need to set some environment flags. Check `sample-env` for what is available, and you can copy the file to `.env` in the root and add your settings there as well.
#### License #### License

View File

@@ -1,6 +1,7 @@
{ {
"name": "gh-action-bot", "name": "gh-action-bot",
"version": "0.0.0", "version": "0.1.0",
"private": true,
"description": "Action bot for Github", "description": "Action bot for Github",
"main": "index", "main": "index",
"module": "index.mjs", "module": "index.mjs",

6
sample-env Normal file
View File

@@ -0,0 +1,6 @@
GITHUB_ACCESS_TOKEN=xxx
COMMENT_ACTOR=ci-comment-username
COMMENT_BODY_REGEXP=build failed
COMMENT_BODY_REGEXP_FLAGS=i
PULL_LABEL_FILTER=
PULL_RETEST_BODY=retest

View File

@@ -1,28 +1,43 @@
import dotenv from 'dotenv';
import createRepo from './lib/create_repo.mjs'; import createRepo from './lib/create_repo.mjs';
import getComments from './lib/get_comments.mjs'; import getComments from './lib/get_comments.mjs';
import getPull from './lib/get_pull.mjs'; import getPull from './lib/get_pull.mjs';
import getCommits from './lib/get_commits.mjs'; import getCommits from './lib/get_commits.mjs';
import getCommitStatus from './lib/get_commit_status.mjs'; import getCommitStatus from './lib/get_commit_status.mjs';
import { createComment, deleteComment } from './lib/comments.mjs';
import History from './lib/history.mjs'; import History from './lib/history.mjs';
// load env vars from .env file
dotenv.config();
export default async function() { export default async function() {
// parse repo name from cli and create repo instance // parse repo name from cli and create repo instance
const repo = createRepo(process.argv.splice(2)[0]); const repo = createRepo(process.argv.splice(2)[0]);
const {
COMMENT_BODY_REGEXP,
COMMENT_BODY_REGEXP_FLAGS,
COMMENT_ACTOR,
PULL_LABEL_FILTER,
PULL_RETEST_BODY,
} = process.env;
// load the history module // load the history module
const history = new History(); const history = new History();
// fetch comment info from event stream, filter for only new comments // fetch comment info from event stream, filter for only new comments
const comments = (await getComments(repo, { const comments = (await getComments(repo, {
body: /build failed/i, body: new RegExp(COMMENT_BODY_REGEXP, COMMENT_BODY_REGEXP_FLAGS),
actor: 'elasticmachine', actor: COMMENT_ACTOR,
})).filter(comment => !history.get(comment.id)); })).filter(comment => !history.get(comment.id));
// read pull data, filter out any closed pulls // read pull data
const pulls = (await Promise.all( const pulls = (await Promise.all(
comments.map(async comment => { comments.map(async comment => {
const pull = await getPull(repo, comment.number); const pull = await getPull(repo, comment.number);
if (pull.state !== 'open') return false;
if (pull.state !== 'open') return false; // filter out any closed pulls
if (PULL_LABEL_FILTER && !pull.labels.includes(PULL_LABEL_FILTER)) return false; // filter on label
return { comment, pull }; return { comment, pull };
}) })
)).filter(Boolean); )).filter(Boolean);
@@ -38,8 +53,15 @@ export default async function() {
}) })
)).filter(Boolean); )).filter(Boolean);
console.log(records); console.log(`Found ${records.length} outstanding failures`);
process.exit();
await Promise.all(
records.map(async record => {
console.log(`Re-testing PR #${record.pull.number}`);
const comment = await createComment(repo, records[0].pull.number, PULL_RETEST_BODY);
await deleteComment(repo, comment.id);
})
);
/* /*
@@ -47,12 +69,12 @@ export default async function() {
- [x] keep track of seen comment ids, only process new ones - [x] keep track of seen comment ids, only process new ones
- [x] check the pr's status and only retest if no longer "Pending" - [x] check the pr's status and only retest if no longer "Pending"
- [ ] add a retest comment - [x] add a retest comment
- POST /repos/:owner/:repo/issues/:number/comments - POST /repos/:owner/:repo/issues/:number/comments
- [x] delete the retest comment
- [ ] delete the build comment - [ ] delete the build comment
- DELETE /repos/:owner/:repo/issues/comments/:comment_id
- [ ] delete ALL build failure comments - [ ] delete ALL build failure comments
- [ ] delete the retest comment - DELETE /repos/:owner/:repo/issues/comments/:comment_id
*/ */
} }

7
src/lib/comments.mjs Normal file
View File

@@ -0,0 +1,7 @@
export async function createComment(repo, issueId, body) {
return repo.issues(issueId).comments.create({ body });
}
export async function deleteComment(repo, commentId) {
return repo.issues.comments(commentId).remove();
}

View File

@@ -8,8 +8,11 @@ function usageError() {
export default function(repo) { export default function(repo) {
if (typeof repo !== 'string' || repo.length === 0) usageError(); if (typeof repo !== 'string' || repo.length === 0) usageError();
const settings = {
token: process.env.GITHUB_ACCESS_TOKEN,
};
const octo = new Octokat(); const octo = new Octokat(settings);
const [owner, name] = repo.split('/'); const [owner, name] = repo.split('/');
if (!owner || !name) usageError(); if (!owner || !name) usageError();

View File

@@ -1,7 +1,17 @@
export default async function getEvents(repo, { body, actor } = {}) { export default async function getEvents(repo, { body, actor } = {}) {
const { items: events } = await repo.events.fetch(); const events = await repo.events.fetch();
return events const items = await (async () => {
// process 2 pages of events
if (events.nextPage) {
const moreEvents = await events.nextPage.fetch();
return [...events.items, ...moreEvents.items];
}
return events.items;
})();
return items
.map(comment => { .map(comment => {
// only comments created by specific user // only comments created by specific user
if (comment.type !== 'IssueCommentEvent') return false; if (comment.type !== 'IssueCommentEvent') return false;