Bypass the "Checks" page and go directly to the Travis build when clicking the "Details" links on a PR (#1636)

Fixes #1620

Co-authored-by: Federico Brigante <[email protected]>

Author: 2 people

readme.md

@@ -180,6 +180,7 @@ And [many more…](source/content.css)
 - [Hide comments in fewer clicks.](https://user-images.githubusercontent.com/1402241/43039221-1ddc91f6-8d29-11e8-9ed4-93459191a510.gif)
 - Use <kbd>←</kbd> and <kbd>→</kbd> to navigate through pages with pagination.
 - [Find a user’s most starred repositories in their profile.](https://user-images.githubusercontent.com/1402241/48474026-43e3ae80-e82c-11e8-93de-159ad4c6f283.png)
+- [Bypass the "Checks" page and go directly to the Travis build when clicking the "Details" links on a PR](https://user-images.githubusercontent.com/2103975/49071220-c6596e80-f22d-11e8-8a1e-bdcd62aa6ece.png)
 
 ### Previously part of Refined GitHub
 

source/content.js

@@ -84,6 +84,7 @@ import usefulNotFoundPage from './features/useful-not-found-page';
 import setDefaultRepositoriesTypeToSources from './features/set-default-repositories-type-to-sources';
 import markPrivateOrgs from './features/mark-private-orgs';
 import navigatePagesWithArrowKeys from './features/navigate-pages-with-arrow-keys';
+import bypassChecksTravis from './features/bypass-checks-travis';
 
 import * as pageDetect from './libs/page-detect';
 import {safeElementReady, enableFeature, safeOnAjaxedPages, injectCustomCSS} from './libs/utils';
@@ -306,6 +307,10 @@ function ajaxedPagesHandler() {
 	if (pageDetect.isPRCommit()) {
 		enableFeature(linkifyCommitSha);
 	}
+
+	if (pageDetect.isPRConversation()) {
+		enableFeature(bypassChecksTravis);
+	}
 }
 
 init();

source/features/bypass-checks-travis.js

@@ -0,0 +1,27 @@
+import select from 'select-dom';
+import domify from '../libs/domify';
+
+export default async function () {
+	// If anything errors, RGH will display the error next to the feature name
+	await Promise.all(select.all('[href="/apps/travis-ci"]').map(bypass));
+}
+
+async function bypass(check) {
+	const details = select('.status-actions', check.parentNode);
+
+	const response = await fetch(details.href, {
+		credentials: 'include'
+	});
+
+	if (!response.ok) {
+		return;
+	}
+
+	const dom = domify(await response.text());
+	// On errored build check pages, there's a link that points
+	// to the first errored Travis build instead of the current one.
+	// e.g. the failed "PR build" instead of the "branch build"
+	// .text-small selects the right one.
+	const directLink = select('[href^="https://travis-ci.com"].text-small', dom);
+	details.href = directLink.href;
+}