gaby: labels page: skip labels authored by bots

For #64.

Change-Id: I9db76f39f41d2ad734257beb01b55d6de6705cfa
Reviewed-on: https://go-review.googlesource.com/c/oscar/+/634976
LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
Reviewed-by: Hyang-Ah Hana Kim <hyangah@gmail.com>

Author: jba

internal/gaby/labels.go

@@ -31,6 +31,7 @@ type labelsPage struct {
 
 type labelsResult struct {
 	*github.Issue // the issue we're reporting on
+	Problem       string
 	Category      labels.Category
 	Explanation   string
 	BodyHTML      safehtml.HTML
@@ -93,17 +94,22 @@ func (g *Gaby) populateLabelsPage(r *http.Request) *labelsPage {
 
 	// Find issues in database.
 	for i := range github.LookupIssues(g.db, project, issueMin, issueMax) {
-		cat, exp, err := labels.IssueCategory(r.Context(), g.llm, i)
-		if err != nil {
-			p.Error = err
-			return p
+		lr := &labelsResult{Issue: i}
+		if i.PullRequest != nil {
+			lr.Problem = "skipping: pull request"
+		} else if isBot(i.User.Login) {
+			lr.Problem = "skipping: author is a bot"
+		} else {
+			cat, exp, err := labels.IssueCategory(r.Context(), g.llm, i)
+			if err != nil {
+				p.Error = err
+				return p
+			}
+			lr.Category = cat
+			lr.Explanation = exp
+			lr.BodyHTML = htmlutil.MarkdownToSafeHTML(i.Body)
 		}
-		p.Results = append(p.Results, &labelsResult{
-			Issue:       i,
-			Category:    cat,
-			Explanation: exp,
-			BodyHTML:    htmlutil.MarkdownToSafeHTML(i.Body),
-		})
+		p.Results = append(p.Results, lr)
 	}
 	return p
 }
@@ -135,3 +141,8 @@ func (pm *labelsParams) inputs() []FormInput {
 		},
 	}
 }
+
+func isBot(author string) bool {
+	// TODO: generalize.
+	return author == "gopherbot" || author == "gabyhelp"
+}

internal/gaby/static/labels.css

@@ -0,0 +1,11 @@
+/*
+Copyright 2024 The Go Authors. All rights reserved.
+Use of this source code is governed by a BSD-style
+license that can be found in the LICENSE file.
+*/
+td {
+    text-align: left;
+    vertical-align: top;
+}
+
+

internal/gaby/tmpl/labelspage.tmpl

@@ -18,17 +18,21 @@ license that can be found in the LICENSE file.
 		<table width="40%">
 			<tr><td>Issue</td><td><a href="{{.HTMLURL}}">#{{.Number}}</a></td></tr>
 			<tr><td>Title</td><td><strong>{{.Title}}</strong></td></tr>
-			<tr><td valign="top">Body</td>
-				<td><details><summary>Contents</Summary>{{.BodyHTML}}</details></td>
-			</tr>
-			<tr><td>Author</td><td>{{.User.Login}}</td></tr>
-			<tr><td>State</td><td>{{.State}}</td></tr>
-			<tr><td>Labels</td>
-				<td>{{range .Labels}}{{.Name}} {{end}}</td>
-			</tr>
-			<tr><td colspan=2 height="10rem"></td><tr>
-			<tr><td>Category</td><td>{{.Category.Name}} ({{.Category.Description}})</td></tr>
-			<tr><td valign="top">Explanation</td><td>{{.Explanation}}</td></tr>
+			{{with .Problem}}
+				<tr><td>Skipped</td><td>{{.}}</td></tr>
+			{{else}}
+				<tr><td>Body</td>
+					<td><details><summary>Contents</Summary>{{.BodyHTML}}</details></td>
+				</tr>
+				<tr><td>Author</td><td>{{.User.Login}}</td></tr>
+				<tr><td>State</td><td>{{.State}}</td></tr>
+				<tr><td>Labels</td>
+					<td>{{range .Labels}}{{.Name}} {{end}}</td>
+				</tr>
+				<tr><td colspan=2 height="10rem"></td><tr>
+				<tr><td>Category</td><td>{{.Category.Name}} ({{.Category.Description}})</td></tr>
+				<tr><td valign="top">Explanation</td><td>{{.Explanation}}</td></tr>
+			{{end}}
 		</table>
 		</div>
 	  {{- end}}