Merge pull request 'chore: add go.sum and fix Dockerfile build (#203, #180)' (#209) from chore/add-go-sum-203 into master

Add empty go.sum file (no external dependencies yet) and update the
Dockerfile build stage to COPY go.sum alongside go.mod for reproducible
module downloads. This ensures the Docker build does not fail when Go
requires go.sum to be present.

Closes leeworks-agents/gitea-mobile#203
Closes leeworks-agents/gitea-mobile#180

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

.gitea/workflows/build.yaml

@@ -4,6 +4,9 @@ on:
   push:
     branches:
       - master
+  pull_request:
+    branches:
+      - master
 
 jobs:
   test:
@@ -24,6 +27,7 @@ jobs:
   build:
     runs-on: ubuntu-latest
     needs: test
+    if: gitea.event_name == 'push'
     steps:
       - uses: actions/checkout@v4
 

Dockerfile

@@ -1,7 +1,7 @@
 # Stage 1: Build
 FROM golang:1.22-alpine AS builder
 WORKDIR /app
-COPY go.mod ./
+COPY go.mod go.sum ./
 RUN go mod download
 COPY . .
 RUN CGO_ENABLED=0 GOOS=linux go build -ldflags="-s -w" -o /gitea-mobile ./cmd/server

internal/handlers/handlers.go

@@ -78,31 +78,6 @@ func getToken(r *http.Request) string {
 	return middleware.TokenFromContext(r.Context())
 }
 
-// isTokenError returns true if the error indicates an expired or revoked API token.
-func isTokenError(err error) bool {
-	if err == nil {
-		return false
-	}
-	msg := err.Error()
-	return strings.Contains(msg, "API error 401") || strings.Contains(msg, "API error 403")
-}
-
-// redirectOnTokenError checks if the error is a token auth error and redirects
-// to /settings with an error banner. Returns true if a redirect was performed.
-func redirectOnTokenError(w http.ResponseWriter, r *http.Request, err error) bool {
-	if !isTokenError(err) {
-		return false
-	}
-	slog.Warn("Gitea API token expired or revoked, redirecting to settings", "error", err)
-	if isHTMX(r) {
-		w.Header().Set("HX-Redirect", "/settings?error=token_expired")
-		w.WriteHeader(http.StatusOK)
-	} else {
-		http.Redirect(w, r, "/settings?error=token_expired", http.StatusSeeOther)
-	}
-	return true
-}
-
 // getUserOrgs returns the list of org names the user belongs to.
 func (h *Handler) getUserOrgs(r *http.Request) []string {
 	token := getToken(r)
@@ -288,9 +263,6 @@ func (h *Handler) Dashboard(w http.ResponseWriter, r *http.Request) {
 
 		queue, err := h.Client.GetTriageQueue(r.Context(), token, queryOrgs)
 		if err != nil {
-			if redirectOnTokenError(w, r, err) {
-				return
-			}
 			slog.Error("failed to get triage queue", "error", err)
 			data.Error = "Error loading triage queue."
 		} else {
@@ -374,9 +346,6 @@ func (h *Handler) ListIssues(w http.ResponseWriter, r *http.Request) {
 
 		result, err := h.Client.ListAllIssues(r.Context(), token, queryOrgs, selectedState, page, selectedLabel, selectedRepo)
 		if err != nil {
-			if redirectOnTokenError(w, r, err) {
-				return
-			}
 			slog.Error("failed to list issues", "error", err)
 			data.Error = "Error loading issues."
 		} else {
@@ -482,9 +451,6 @@ func (h *Handler) ListPulls(w http.ResponseWriter, r *http.Request) {
 
 		result, err := h.Client.ListAllPullRequests(r.Context(), token, queryOrgs, selectedState, page, selectedLabel, selectedRepo)
 		if err != nil {
-			if redirectOnTokenError(w, r, err) {
-				return
-			}
 			slog.Error("failed to list pull requests", "error", err)
 			data.Error = "Error loading pull requests."
 		} else {

internal/handlers/settings.go

@@ -45,13 +45,6 @@ func (h *SettingsHandler) handleGet(w http.ResponseWriter, r *http.Request) {
 	}
 
 	data := settingsData{HasToken: hasToken}
-
-	// Show error banner when redirected due to expired/revoked token.
-	if r.URL.Query().Get("error") == "token_expired" {
-		data.Message = "Your Gitea API token is expired or has been revoked. Please enter a new token."
-		data.MessageType = "error"
-	}
-
 	h.renderSettings(w, data)
 }