From cb9625d613c3f428a34da4db12bce5bb8c169543 Mon Sep 17 00:00:00 2001
From: rob <robdickson444@hotmail.com>
Date: Fri, 16 Jan 2026 14:46:35 -0400
Subject: [PATCH] Add improved pagination to admin pending tools page
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

- Page number links with ellipsis for large ranges
- First/Last page buttons (« and »)
- "Go to page" input for direct page jumping
- Current page highlighted in indigo

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
---
 src/cmdforge/web/templates/admin/pending.html | 49 ++++++++++++++++++-
 1 file changed, 48 insertions(+), 1 deletion(-)

diff --git a/src/cmdforge/web/templates/admin/pending.html b/src/cmdforge/web/templates/admin/pending.html
index 893f701..5f26b8e 100644
--- a/src/cmdforge/web/templates/admin/pending.html
+++ b/src/cmdforge/web/templates/admin/pending.html
@@ -91,13 +91,48 @@
         <div class="text-sm text-gray-700">
             Page {{ meta.page }} of {{ meta.total_pages }} ({{ meta.total }} total)
         </div>
-        <div class="flex space-x-2">
+        <div class="flex items-center space-x-2">
+            <!-- First/Previous -->
             {% if meta.page > 1 %}
+            <a href="?page=1" class="px-2 py-1 border rounded text-sm hover:bg-gray-100" title="First page">&laquo;</a>
             <a href="?page={{ meta.page - 1 }}" class="px-3 py-1 border rounded text-sm hover:bg-gray-100">Previous</a>
             {% endif %}
+
+            <!-- Page numbers -->
+            {% set start_page = [1, meta.page - 2] | max %}
+            {% set end_page = [meta.total_pages, meta.page + 2] | min %}
+
+            {% if start_page > 1 %}
+            <span class="text-gray-400">...</span>
+            {% endif %}
+
+            {% for p in range(start_page, end_page + 1) %}
+            {% if p == meta.page %}
+            <span class="px-3 py-1 bg-indigo-600 text-white rounded text-sm">{{ p }}</span>
+            {% else %}
+            <a href="?page={{ p }}" class="px-3 py-1 border rounded text-sm hover:bg-gray-100">{{ p }}</a>
+            {% endif %}
+            {% endfor %}
+
+            {% if end_page < meta.total_pages %}
+            <span class="text-gray-400">...</span>
+            {% endif %}
+
+            <!-- Next/Last -->
             {% if meta.page < meta.total_pages %}
             <a href="?page={{ meta.page + 1 }}" class="px-3 py-1 border rounded text-sm hover:bg-gray-100">Next</a>
+            <a href="?page={{ meta.total_pages }}" class="px-2 py-1 border rounded text-sm hover:bg-gray-100" title="Last page">&raquo;</a>
             {% endif %}
+
+            <!-- Jump to page -->
+            <span class="text-gray-400 mx-2">|</span>
+            <form class="flex items-center space-x-1" onsubmit="jumpToPage(event)">
+                <label class="text-sm text-gray-600">Go to:</label>
+                <input type="number" id="page-input" min="1" max="{{ meta.total_pages }}"
+                       class="w-16 px-2 py-1 border rounded text-sm focus:outline-none focus:ring-1 focus:ring-indigo-500"
+                       placeholder="{{ meta.page }}">
+                <button type="submit" class="px-2 py-1 bg-indigo-600 text-white rounded text-sm hover:bg-indigo-700">Go</button>
+            </form>
         </div>
     </div>
     {% endif %}
@@ -134,6 +169,18 @@ function toggleFindings(toolId) {
     el.classList.toggle('hidden');
 }
 
+function jumpToPage(event) {
+    event.preventDefault();
+    const input = document.getElementById('page-input');
+    const page = parseInt(input.value);
+    const maxPage = parseInt(input.max);
+    if (page && page >= 1 && page <= maxPage) {
+        window.location.href = `?page=${page}`;
+    } else if (page) {
+        alert(`Please enter a page number between 1 and ${maxPage}`);
+    }
+}
+
 async function approveTool(toolId) {
     if (!confirm('Approve this tool?')) return;
     try {