cmd/pkgserver: add sort orders, change pagination rules

cmd/pkgserver/api.go

@@ -8,9 +8,9 @@ import (
 	"net/http"
 	"path"
 	"strconv"
-	"strings"
 
 	"hakurei.app/internal/info"
+	"hakurei.app/internal/pkg"
 	"hakurei.app/internal/rosa"
 )
 
@@ -36,14 +36,14 @@ func serveInfo(index *PackageIndex) func(http.ResponseWriter, *http.Request) {
 	}
 }
 
-func serveStatus(index *PackageIndex) func(w http.ResponseWriter, r *http.Request) {
+func serveStatus(index *PackageIndex, cache *pkg.Cache) func(w http.ResponseWriter, r *http.Request) {
 	return func(w http.ResponseWriter, r *http.Request) {
+		download := path.Dir(r.URL.Path) == "/status"
 		if index == nil {
 			http.Error(w, "index is nil", http.StatusInternalServerError)
 			return
 		}
-		base := path.Base(r.URL.Path)
-		name := strings.TrimSuffix(base, ".log")
+		name := path.Base(r.URL.Path)
 		p, ok := rosa.ResolveName(name)
 		if !ok {
 			http.NotFound(w, r)
@@ -56,7 +56,21 @@ func serveStatus(index *PackageIndex) func(w http.ResponseWriter, r *http.Reques
 			return
 		}
 		if len(pk.status) > 0 {
+			if download {
+				w.Header().Set("Content-Type", "application/octet-stream")
+			} else {
 				w.Header().Set("Content-Type", "text/plain; charset=utf-8")
+			}
+			if download {
+				var version string
+				if pk.Version != "\u0000" {
+					version = pk.Version
+				} else {
+					version = "unknown"
+				}
+				w.Header().Set("Content-Disposition", fmt.Sprintf("attachment; filename=\"%s-%s-%s.log\"", pk.Name, version, pkg.Encode(pk.ident.Value())))
+			}
+
 			w.Header().Set("Cache-Control", "no-cache, no-store, must-revalidate")
 			w.WriteHeader(http.StatusOK)
 			_, err := io.Copy(w, bytes.NewReader(pk.status))
@@ -111,10 +125,11 @@ func serveGet(index *PackageIndex) func(http.ResponseWriter, *http.Request) {
 
 const ApiVersion = "v1"
 
-func apiRoutes(index *PackageIndex) {
+func apiRoutes(index *PackageIndex, cache *pkg.Cache) {
 	http.HandleFunc(fmt.Sprintf("GET /api/%s/info", ApiVersion), serveInfo(index))
 	http.HandleFunc(fmt.Sprintf("GET /api/%s/get", ApiVersion), serveGet(index))
-	http.HandleFunc(fmt.Sprintf("GET /api/%s/status/", ApiVersion), serveStatus(index))
+	http.HandleFunc(fmt.Sprintf("GET /api/%s/status/", ApiVersion), serveStatus(index, cache))
+	http.HandleFunc("GET /status/", serveStatus(index, cache))
 }
 
 func WritePayload(w http.ResponseWriter, payload any) {

cmd/pkgserver/index.go

@@ -2,8 +2,8 @@ package main
 
 import (
 	"cmp"
-	"fmt"
 	"slices"
+	"unique"
 
 	"hakurei.app/internal/pkg"
 	"hakurei.app/internal/rosa"
@@ -29,7 +29,8 @@ type PackageIndexEntry struct {
 	Description string                `json:"description,omitempty"`
 	Website     string                `json:"website,omitempty"`
 	Version     string                `json:"version"`
-	Status      string `json:"report,omitempty"`
+	HasReport   bool                  `json:"report,omitempty"`
+	ident       unique.Handle[pkg.ID] `json:"-"`
 	status      []byte                `json:"-"`
 }
 
@@ -45,20 +46,18 @@ func createPackageIndex(cache *pkg.Cache, report *rosa.Report) (_ *PackageIndex,
 		id := cache.Ident(a)
 		st, n := report.ArtifactOf(id)
 		var status []byte
-		var statusUrl string
 		if n < 1 {
 			status = nil
-			statusUrl = ""
 		} else {
 			status = st
-			statusUrl = fmt.Sprintf("/api/%s/status/%s.log", ApiVersion, m.Name)
 		}
 		entry := PackageIndexEntry{
 			Name:        m.Name,
 			Description: m.Description,
 			Website:     m.Website,
 			Version:     v,
-			Status:      statusUrl,
+			HasReport:   len(status) > 0,
+			ident:       id,
 			status:      status,
 		}
 		work[p] = entry

cmd/pkgserver/main.go

@@ -50,7 +50,7 @@ func main() {
 			return err
 		}
 		uiRoutes()
-		apiRoutes(index)
+		apiRoutes(index, cache)
 		err = http.ListenAndServe(fmt.Sprintf(":%d", flagPort), nil)
 		if err != nil {
 			return err

cmd/pkgserver/ui/index.html

@@ -20,6 +20,12 @@
     <option value="50">50</option>
     <option value="100">100</option>
 </select></span>
+<span><label for="sort">Sort by: </label><select name="sort" id="sort">
+    <option value="0">Definition (ascending)</option>
+    <option value="1">Definition (descending)</option>
+    <option value="2">Name (ascending)</option>
+    <option value="3">Name (descending)</option>
+</select></span>
 </body>
 <footer>
     <p>&copy;<a href="https://hakurei.app/">Hakurei</a> (<span id="hakurei-version">unknown</span>). Licensed under the MIT license.</p>

cmd/pkgserver/ui/static/index.js

@@ -9,7 +9,7 @@ function toHTML(entry) {
     let v = entry.version != null ? `<span>${escapeHtml(entry.version)}</span>` : "";
     let d = entry.description != null ? `<p>${escapeHtml(entry.description)}</p>` : "";
     let w = entry.website != null ? `<a href="${encodeURI(entry.website)}">Website</a>` : "";
-    let r = entry.report != null ? `<a href="${encodeURI(entry.report)}">Log</a>` : "";
+    let r = entry.report ? `Log (<a href=\"${encodeURI('/api/v1/status/' + entry.name)}\">View</a> | <a href=\"${encodeURI('/status/' + entry.name)}\">Download</a>)` : "";
     let row = (document.createElement('tr'));
     row.innerHTML = `<td>
             <h2>${escapeHtml(entry.name)} ${v}</h2>
@@ -27,8 +27,8 @@ class InfoPayload {
 }
 async function infoRequest() {
     const res = await fetch(`${ENDPOINT}/info`);
-    const res_1 = await res.json();
-    return res_1;
+    const payload = await res.json();
+    return payload;
 }
 class GetPayload {
     count;
@@ -43,36 +43,27 @@ var SortOrders;
 })(SortOrders || (SortOrders = {}));
 async function getRequest(limit, index, sort) {
     const res = await fetch(`${ENDPOINT}/get?limit=${limit}&index=${index}&sort=${sort.valueOf()}`);
-    const res_1 = await res.json();
-    return res_1;
+    const payload = await res.json();
+    return payload;
 }
 class State {
     entriesPerPage = 10;
-    currentPage = 1;
     entryIndex = 0;
-    maxEntries = 100;
+    maxEntries = 0;
+    sort = SortOrders.DeclarationAscending;
     getEntriesPerPage() {
         return this.entriesPerPage;
     }
     setEntriesPerPage(entriesPerPage) {
         this.entriesPerPage = entriesPerPage;
-        if (this.currentPage > this.getMaxPage()) {
-            this.setCurrentPage(this.getMaxPage());
-        }
-    }
-    getCurrentPage() {
-        return this.currentPage;
-    }
-    setCurrentPage(page) {
-        this.currentPage = page;
-        this.setEntryIndex((this.getCurrentPage() - 1) * this.getEntriesPerPage());
-        document.getElementById("page-number").innerText = String(this.getCurrentPage());
+        this.setEntryIndex(Math.floor(this.getEntryIndex() / entriesPerPage) * entriesPerPage);
     }
     getEntryIndex() {
         return this.entryIndex;
     }
     setEntryIndex(entryIndex) {
         this.entryIndex = entryIndex;
+        this.updatePage();
         this.updateRange();
         this.updateListings();
     }
@@ -82,15 +73,23 @@ class State {
     setMaxEntries(max) {
         this.maxEntries = max;
     }
-    getMaxPage() {
-        return Math.ceil(this.getMaxEntries() / this.getEntriesPerPage());
+    getSortOrder() {
+        return this.sort;
+    }
+    setSortOrder(sortOrder) {
+        this.sort = sortOrder;
+        this.setEntryIndex(0);
+    }
+    updatePage() {
+        let page = Math.ceil(((this.getEntryIndex() + this.getEntriesPerPage()) - 1) / this.getEntriesPerPage());
+        document.getElementById("page-number").innerText = String(page);
     }
     updateRange() {
         let max = Math.min(this.getEntryIndex() + this.getEntriesPerPage(), this.getMaxEntries());
         document.getElementById("entry-counter").innerText = `${this.getEntryIndex() + 1}-${max} of ${this.getMaxEntries()}`;
     }
     updateListings() {
-        getRequest(this.getEntriesPerPage(), this.entryIndex, 0)
+        getRequest(this.getEntriesPerPage(), this.getEntryIndex(), this.getSortOrder())
             .then(res => {
             let table = document.getElementById("pkg-list");
             table.innerHTML = '';
@@ -102,16 +101,12 @@ class State {
 }
 let STATE;
 function prevPage() {
-    let current = STATE.getCurrentPage();
-    if (current > 1) {
-        STATE.setCurrentPage(STATE.getCurrentPage() - 1);
-    }
+    let index = STATE.getEntryIndex();
+    STATE.setEntryIndex(Math.max(0, index - STATE.getEntriesPerPage()));
 }
 function nextPage() {
-    let current = STATE.getCurrentPage();
-    if (current < STATE.getMaxPage()) {
-        STATE.setCurrentPage(STATE.getCurrentPage() + 1);
-    }
+    let index = STATE.getEntryIndex();
+    STATE.setEntryIndex(Math.min((Math.ceil(STATE.getMaxEntries() / STATE.getEntriesPerPage()) * STATE.getEntriesPerPage()) - STATE.getEntriesPerPage(), index + STATE.getEntriesPerPage()));
 }
 function escapeHtml(str) {
     return str
@@ -133,4 +128,7 @@ document.addEventListener("DOMContentLoaded", () => {
     document.getElementById("count").addEventListener("change", (event) => {
         STATE.setEntriesPerPage(parseInt(event.target.value));
     });
+    document.getElementById("sort").addEventListener("change", (event) => {
+        STATE.setSortOrder(parseInt(event.target.value));
+    });
 });

cmd/pkgserver/ui/static/index.ts

@@ -3,13 +3,13 @@ class PackageIndexEntry {
     description: string | null
     website: string | null
     version: string | null
-    report: string | null
+    report:  boolean
 }
 function toHTML(entry: PackageIndexEntry): HTMLTableRowElement {
     let v = entry.version != null ? `<span>${escapeHtml(entry.version)}</span>` : ""
     let d = entry.description != null ? `<p>${escapeHtml(entry.description)}</p>` : ""
     let w = entry.website != null ? `<a href="${encodeURI(entry.website)}">Website</a>` : ""
-    let r = entry.report != null ? `<a href="${encodeURI(entry.report)}">Log</a>` : ""
+    let r = entry.report ? `Log (<a href=\"${encodeURI('/api/v1/status/' + entry.name)}\">View</a> | <a href=\"${encodeURI('/status/' + entry.name)}\">Download</a>)` : ""
     let row = <HTMLTableRowElement>(document.createElement('tr'))
     row.innerHTML = `<td>
             <h2>${escapeHtml(entry.name)} ${v}</h2>
@@ -29,8 +29,8 @@ class InfoPayload {
 
 async function infoRequest(): Promise<InfoPayload> {
     const res = await fetch(`${ENDPOINT}/info`)
-    const res_1 = await res.json()
-    return res_1 as InfoPayload
+    const payload = await res.json()
+    return payload as InfoPayload
 }
 class GetPayload {
     count: number
@@ -45,36 +45,28 @@ enum SortOrders {
 }
 async function getRequest(limit: number, index: number, sort: SortOrders): Promise<GetPayload> {
     const res = await fetch(`${ENDPOINT}/get?limit=${limit}&index=${index}&sort=${sort.valueOf()}`)
-    const res_1 = await res.json()
-    return res_1 as GetPayload
+    const payload = await res.json()
+    return payload as GetPayload
 }
 class State {
     entriesPerPage: number = 10
-    currentPage: number = 1
     entryIndex: number = 0
     maxEntries: number = 0
+    sort: SortOrders = SortOrders.DeclarationAscending
+
     getEntriesPerPage(): number {
         return this.entriesPerPage
     }
     setEntriesPerPage(entriesPerPage: number) {
         this.entriesPerPage = entriesPerPage
-        if (this.currentPage > this.getMaxPage()) {
-            this.setCurrentPage(this.getMaxPage())
-        }
-    }
-    getCurrentPage(): number {
-        return this.currentPage
-    }
-    setCurrentPage(page: number) {
-        this.currentPage = page
-        this.setEntryIndex((this.getCurrentPage() - 1) * this.getEntriesPerPage())
-        document.getElementById("page-number").innerText = String(this.getCurrentPage())
+        this.setEntryIndex(Math.floor(this.getEntryIndex() / entriesPerPage) * entriesPerPage)
     }
     getEntryIndex(): number {
         return this.entryIndex
     }
     setEntryIndex(entryIndex: number) {
         this.entryIndex = entryIndex
+        this.updatePage()
         this.updateRange()
         this.updateListings()
     }
@@ -84,15 +76,23 @@ class State {
     setMaxEntries(max: number) {
         this.maxEntries = max
     }
-    getMaxPage(): number {
-        return Math.ceil(this.getMaxEntries() / this.getEntriesPerPage())
+    getSortOrder(): SortOrders {
+        return this.sort
+    }
+    setSortOrder(sortOrder: SortOrders) {
+        this.sort = sortOrder
+        this.setEntryIndex(0)
+    }
+    updatePage() {
+        let page = Math.ceil(((this.getEntryIndex() + this.getEntriesPerPage()) - 1) / this.getEntriesPerPage())
+        document.getElementById("page-number").innerText = String(page)
     }
     updateRange() {
         let max = Math.min(this.getEntryIndex() + this.getEntriesPerPage(), this.getMaxEntries())
         document.getElementById("entry-counter").innerText = `${this.getEntryIndex() + 1}-${max} of ${this.getMaxEntries()}`
     }
     updateListings() {
-        getRequest(this.getEntriesPerPage(), this.entryIndex, 0)
+        getRequest(this.getEntriesPerPage(), this.getEntryIndex(), this.getSortOrder())
             .then(res => {
                 let table = document.getElementById("pkg-list")
                 table.innerHTML = ''
@@ -107,16 +107,12 @@ class State {
 let STATE: State
 
 function prevPage() {
-    let current = STATE.getCurrentPage()
-    if (current > 1) {
-        STATE.setCurrentPage(STATE.getCurrentPage() - 1)
-    }
+    let index = STATE.getEntryIndex()
+    STATE.setEntryIndex(Math.max(0, index - STATE.getEntriesPerPage()))
 }
 function nextPage() {
-    let current = STATE.getCurrentPage()
-    if (current < STATE.getMaxPage()) {
-        STATE.setCurrentPage(STATE.getCurrentPage() + 1)
-    }
+    let index = STATE.getEntryIndex()
+    STATE.setEntryIndex(Math.min((Math.ceil(STATE.getMaxEntries() / STATE.getEntriesPerPage()) * STATE.getEntriesPerPage()) - STATE.getEntriesPerPage(), index + STATE.getEntriesPerPage()))
 }
 
 function escapeHtml(str: string): string {
@@ -141,4 +137,7 @@ document.addEventListener("DOMContentLoaded", () => {
     document.getElementById("count").addEventListener("change", (event) => {
         STATE.setEntriesPerPage(parseInt((event.target as HTMLSelectElement).value))
     })
+    document.getElementById("sort").addEventListener("change", (event) => {
+        STATE.setSortOrder(parseInt((event.target as HTMLSelectElement).value))
+    })
 })