web: check status and a path filter on the log, visible focus in the rail !80

merged merged by cmc on 2026-08-26 05:14 UTC · krz/gitbay:log-polish into main

4 files changed, +63 −3

Layout: unified · split

internal/httpd/web.go +3 −1

 	type row struct {
 		SHA, ShortSHA, Subject, AuthorName, AuthorEmail, AuthorUser, Date string
 		Sig                                                               sigView
+		Check                                                             string // combined status, "" when none ran
 	}
 	names := s.authorNames()
+	checks, _ := s.st.CombinedStatusFor(p.Repo.ID, shas)
 	var rows []row
 	for _, sha := range shas {
 		v, parsed := s.sigFor(p.Repo, p.Dir, sha)
-		rw := row{SHA: sha, ShortSHA: sha[:10], Sig: v}
+		rw := row{SHA: sha, ShortSHA: sha[:10], Sig: v, Check: checks[sha]}
 		if parsed != nil {
 			rw.Subject = parsed.Subject
 			rw.AuthorName = names.name(parsed.AuthorEmail, parsed.AuthorName)

internal/store/statuses.go +41

	@@ -1,5 +1,7 @@
1	package store	1	package store
2		2
		3	import "strings"
		4
3	type CommitStatus struct {	5	type CommitStatus struct {
4	Context string	6	Context string
5	State string // pending \| success \| failure \| error	7	State string // pending \| success \| failure \| error
	@@ -50,6 +52,45 @@ func (s *Store) ListCommitStatuses(repoID int64, sha string) ([]CommitStatus, er
50		52
51	// CombinedStatus reduces per-context states to one: error/failure dominate,	53	// CombinedStatus reduces per-context states to one: error/failure dominate,
52	// then pending, then success; "" when no statuses exist.	54	// then pending, then success; "" when no statuses exist.
		55	// CombinedStatusFor returns the combined state for each of several commits
		56	// in one query. The log lists fifty commits at a time; asking per commit
		57	// turns one page into fifty round trips.
		58	func (s *Store) CombinedStatusFor(repoID int64, shas []string) (map[string]string, error) {
		59	out := map[string]string{}
		60	if len(shas) == 0 {
		61	return out, nil
		62	}
		63	args := make([]any, 0, len(shas)+1)
		64	args = append(args, repoID)
		65	for _, sha := range shas {
		66	args = append(args, sha)
		67	}
		68	q := `SELECT commit_sha, state FROM commit_statuses WHERE repo_id = ? AND commit_sha IN (?` +
		69	strings.Repeat(",?", len(shas)-1) + `)`
		70	rows, err := s.DB.Query(q, args...)
		71	if err != nil {
		72	return nil, err
		73	}
		74	defer rows.Close()
		75	for rows.Next() {
		76	var sha, state string
		77	if err := rows.Scan(&sha, &state); err != nil {
		78	return nil, err
		79	}
		80	// Same precedence as CombinedStatus, folded in as rows arrive.
		81	switch {
		82	case out[sha] == "failure":
		83	case state == "error" \|\| state == "failure":
		84	out[sha] = "failure"
		85	case state == "pending":
		86	out[sha] = "pending"
		87	case out[sha] == "":
		88	out[sha] = "success"
		89	}
		90	}
		91	return out, rows.Err()
		92	}
		93
53	func CombinedStatus(statuses []CommitStatus) string {	94	func CombinedStatus(statuses []CommitStatus) string {
54	if len(statuses) == 0 {	95	if len(statuses) == 0 {
55	return ""	96	return ""

internal/web/static/style.css +10

   outline-offset: 2px;
   border-radius: 1px;
 }
+/* the shell is black in both schemes, and --accent is a dark blue in the
+   light one — a ring nobody could see. The shell's own mark clears 7:1 on
+   black either way. */
+.rail :focus-visible { outline-color: var(--shell-mark); }
 /* visually hidden, still read aloud: for headings a sighted reader gets
    from the layout but a screen reader would otherwise miss */
 .teambody { padding: var(--sp-3); background: var(--bg); }
 .teambody .memberchip { margin-right: var(--sp-2); }
+form.logfilter {
+  display: flex; align-items: center; gap: var(--sp-2);
+  margin: 0 0 var(--sp-4); flex-wrap: wrap;
+}
+form.logfilter input { flex: 1; min-width: 12rem; max-width: 28rem; }
 /* account settings */
 table.keys td { padding: var(--sp-2) var(--sp-4) var(--sp-2) 0; }
 table.keys td.mono, .mono { font-family: var(--mono); font-size: var(--fs-1); overflow-wrap: anywhere; }

internal/web/templates/log.html +9 −2

 {{define "title"}}log · {{.Repo.OwnerName}}/{{.Repo.Name}}{{end}}
 {{define "content"}}
-{{if .FilePath}}<p class="meta">history of <a href="/{{.Repo.OwnerName}}/{{.Repo.Name}}/blob/{{.Ref}}/{{.FilePath}}"><code>{{.FilePath}}</code></a> · <a href="/{{.Repo.OwnerName}}/{{.Repo.Name}}/log">full log</a></p>{{end}}
 <h1 class="vh">Commits</h1>
+<form method="get" class="logfilter" action="/{{.Repo.OwnerName}}/{{.Repo.Name}}/log/{{.Ref}}">
+  <label for="path">Path</label>
+  <input type="text" id="path" name="path" value="{{.FilePath}}" placeholder="filter to a file or directory">
+  <button type="submit">Filter</button>
+  {{if .FilePath}}<a class="act" href="/{{.Repo.OwnerName}}/{{.Repo.Name}}/log/{{.Ref}}">Clear</a>{{end}}
+</form>
+{{if .FilePath}}<p class="meta">history of <a href="/{{.Repo.OwnerName}}/{{.Repo.Name}}/blob/{{.Ref}}/{{.FilePath}}"><code>{{.FilePath}}</code></a></p>{{end}}
 <ul class="loglist">
-{{range .Commits}}<li>
+{{range .Commits}}{{$c := .}}<li>
   <div class="commitmain">
     <p class="subject"><a href="/{{$.Repo.OwnerName}}/{{$.Repo.Name}}/commit/{{.SHA}}">{{.Subject}}</a></p>
     <p class="meta">{{template "authorname" dict "Name" .AuthorName "User" .AuthorUser "Email" .AuthorEmail}} · {{.Date}}</p>
   </div>
   <div class="commitside">
+    {{with .Check}}<a class="badge badge-{{.}}" href="/{{$.Repo.OwnerName}}/{{$.Repo.Name}}/commit/{{$c.SHA}}" title="checks: {{.}}">{{.}}</a>{{end}}
     {{template "sigbadge" .Sig}}
     <code><a href="/{{$.Repo.OwnerName}}/{{$.Repo.Name}}/commit/{{.SHA}}">{{.ShortSHA}}</a></code>
   </div>