Compare commits
7 Commits
| Author | SHA1 | Date | |
|---|---|---|---|
| 5d3ce5baf8 | |||
| baf829349c | |||
| 3145acc423 | |||
| ce3fc36835 | |||
| c267bc86a8 | |||
| c790a7236c | |||
| 67973b27aa |
@@ -15,6 +15,9 @@ jobs:
|
|||||||
with:
|
with:
|
||||||
go-version: '1.22'
|
go-version: '1.22'
|
||||||
|
|
||||||
|
- name: Vet
|
||||||
|
run: go vet ./...
|
||||||
|
|
||||||
- name: Run tests
|
- name: Run tests
|
||||||
run: go test -race ./...
|
run: go test -race ./...
|
||||||
|
|
||||||
|
|||||||
+3
-3
@@ -73,9 +73,9 @@ curl -s https://gitea-mobile.testing.leeworks.dev | head -5
|
|||||||
3. Tap on a PR to see details
|
3. Tap on a PR to see details
|
||||||
4. **Expected**: PR diff summary or review status displays correctly
|
4. **Expected**: PR diff summary or review status displays correctly
|
||||||
|
|
||||||
## Step 7: Core Functionality -- Triage Queue
|
## Step 7: Core Functionality -- Dashboard / Triage Queue
|
||||||
|
|
||||||
1. Navigate to the Triage tab (`/triage`)
|
1. Navigate to the Dashboard/Triage tab (`/`)
|
||||||
2. **Expected**: Unassigned issues and PRs awaiting review appear sorted by priority
|
2. **Expected**: Unassigned issues and PRs awaiting review appear sorted by priority
|
||||||
|
|
||||||
## Step 8: Create Issue (Write Operation)
|
## Step 8: Create Issue (Write Operation)
|
||||||
@@ -116,7 +116,7 @@ curl -s https://gitea-mobile.testing.leeworks.dev | head -5
|
|||||||
| 4 | Auth | Token saved, API calls work |
|
| 4 | Auth | Token saved, API calls work |
|
||||||
| 5 | Issues | List loads, filter works |
|
| 5 | Issues | List loads, filter works |
|
||||||
| 6 | PRs | List loads with review status |
|
| 6 | PRs | List loads with review status |
|
||||||
| 7 | Triage | Queue displays correctly |
|
| 7 | Dashboard/Triage | Queue displays correctly at `/` |
|
||||||
| 8 | Create issue | Issue created in Gitea |
|
| 8 | Create issue | Issue created in Gitea |
|
||||||
| 9 | Apply label | Label applied via API |
|
| 9 | Apply label | Label applied via API |
|
||||||
| 10 | PWA | Standalone mode, safe areas, dark mode |
|
| 10 | PWA | Standalone mode, safe areas, dark mode |
|
||||||
|
|||||||
@@ -1,6 +1,8 @@
|
|||||||
package middleware
|
package middleware
|
||||||
|
|
||||||
import (
|
import (
|
||||||
|
"crypto/rand"
|
||||||
|
"encoding/hex"
|
||||||
"log/slog"
|
"log/slog"
|
||||||
"net/http"
|
"net/http"
|
||||||
"time"
|
"time"
|
||||||
@@ -17,21 +19,39 @@ func (rw *responseWriter) WriteHeader(code int) {
|
|||||||
rw.ResponseWriter.WriteHeader(code)
|
rw.ResponseWriter.WriteHeader(code)
|
||||||
}
|
}
|
||||||
|
|
||||||
// Logging returns middleware that logs each HTTP request with structured logging.
|
// generateRequestID creates a short random hex string for request tracing.
|
||||||
|
func generateRequestID() string {
|
||||||
|
b := make([]byte, 8)
|
||||||
|
if _, err := rand.Read(b); err != nil {
|
||||||
|
return "unknown"
|
||||||
|
}
|
||||||
|
return hex.EncodeToString(b)
|
||||||
|
}
|
||||||
|
|
||||||
|
// Logging returns middleware that logs each HTTP request with structured fields:
|
||||||
|
// method, path, status, duration (ms), request-id, and remote address.
|
||||||
func Logging() func(http.Handler) http.Handler {
|
func Logging() func(http.Handler) http.Handler {
|
||||||
return func(next http.Handler) http.Handler {
|
return func(next http.Handler) http.Handler {
|
||||||
return http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
|
return http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
|
||||||
start := time.Now()
|
start := time.Now()
|
||||||
|
requestID := generateRequestID()
|
||||||
rw := &responseWriter{ResponseWriter: w, statusCode: http.StatusOK}
|
rw := &responseWriter{ResponseWriter: w, statusCode: http.StatusOK}
|
||||||
|
|
||||||
|
// Set request ID header for downstream correlation.
|
||||||
|
w.Header().Set("X-Request-ID", requestID)
|
||||||
|
|
||||||
next.ServeHTTP(rw, r)
|
next.ServeHTTP(rw, r)
|
||||||
|
|
||||||
|
duration := time.Since(start)
|
||||||
slog.Info("http request",
|
slog.Info("http request",
|
||||||
"method", r.Method,
|
"method", r.Method,
|
||||||
"path", r.URL.Path,
|
"path", r.URL.Path,
|
||||||
"status", rw.statusCode,
|
"status", rw.statusCode,
|
||||||
"duration", time.Since(start).String(),
|
"duration_ms", duration.Milliseconds(),
|
||||||
|
"duration", duration.String(),
|
||||||
|
"request_id", requestID,
|
||||||
"remote", r.RemoteAddr,
|
"remote", r.RemoteAddr,
|
||||||
|
"user_agent", r.UserAgent(),
|
||||||
)
|
)
|
||||||
})
|
})
|
||||||
}
|
}
|
||||||
|
|||||||
+53
-2
@@ -1,4 +1,12 @@
|
|||||||
/* Gitea Mobile — Mobile-first CSS (~5KB target) */
|
/* Gitea Mobile — Mobile-first CSS
|
||||||
|
* Dark-mode-first: dark colors are the :root defaults.
|
||||||
|
* Light mode is applied via @media (prefers-color-scheme: light).
|
||||||
|
*
|
||||||
|
* Size note: The original ~5KB target was based on the initial Phase 1 scope.
|
||||||
|
* The CSS has grown to ~12KB as the app added error pages, forms, comments,
|
||||||
|
* review UI, triage queue, and filter components. All rules are in active use.
|
||||||
|
* Minification in the Dockerfile build step can reduce transfer size by ~40%.
|
||||||
|
*/
|
||||||
|
|
||||||
/* Reset */
|
/* Reset */
|
||||||
*, *::before, *::after { box-sizing: border-box; margin: 0; padding: 0; }
|
*, *::before, *::after { box-sizing: border-box; margin: 0; padding: 0; }
|
||||||
@@ -533,7 +541,7 @@ a:active {
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
/* Dark mode is default; light mode override if needed */
|
/* Dark mode is default; light mode override for prefers-color-scheme: light */
|
||||||
@media (prefers-color-scheme: light) {
|
@media (prefers-color-scheme: light) {
|
||||||
:root {
|
:root {
|
||||||
--bg-primary: #ffffff;
|
--bg-primary: #ffffff;
|
||||||
@@ -543,6 +551,49 @@ a:active {
|
|||||||
--text-primary: #1f2328;
|
--text-primary: #1f2328;
|
||||||
--text-secondary: #656d76;
|
--text-secondary: #656d76;
|
||||||
--text-link: #0969da;
|
--text-link: #0969da;
|
||||||
|
--accent-green: #1a7f37;
|
||||||
|
--accent-red: #cf222e;
|
||||||
|
--accent-yellow: #9a6700;
|
||||||
|
--accent-blue: #0969da;
|
||||||
|
--accent-purple: #8250df;
|
||||||
|
}
|
||||||
|
|
||||||
|
.message.success {
|
||||||
|
background: #dafbe1;
|
||||||
|
border-color: #1a7f37;
|
||||||
|
}
|
||||||
|
|
||||||
|
.message.error {
|
||||||
|
background: #ffebe9;
|
||||||
|
border-color: #cf222e;
|
||||||
|
}
|
||||||
|
|
||||||
|
.message.info {
|
||||||
|
background: #ddf4ff;
|
||||||
|
border-color: #0969da;
|
||||||
|
}
|
||||||
|
|
||||||
|
.btn-primary {
|
||||||
|
background: #1a7f37;
|
||||||
|
}
|
||||||
|
|
||||||
|
.btn-primary:active {
|
||||||
|
background: #116329;
|
||||||
|
}
|
||||||
|
|
||||||
|
.btn-danger {
|
||||||
|
background: #ffebe9;
|
||||||
|
border-color: #cf222e;
|
||||||
|
}
|
||||||
|
|
||||||
|
.type-issue {
|
||||||
|
background: rgba(9, 105, 218, 0.1);
|
||||||
|
border-color: rgba(9, 105, 218, 0.3);
|
||||||
|
}
|
||||||
|
|
||||||
|
.type-pull {
|
||||||
|
background: rgba(26, 127, 55, 0.1);
|
||||||
|
border-color: rgba(26, 127, 55, 0.3);
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|||||||
Reference in New Issue
Block a user