feat: add sort by name / modified time to tracer directory listing (#116)

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: hiyouga <16256802+hiyouga@users.noreply.github.com>
Co-authored-by: hiyouga <hiyouga@buaa.edu.cn>

Author: Copilot

src_py/agenthub/integration/tracer.py

@@ -29,7 +29,7 @@
 from pathlib import Path
 from typing import Any
 
-from flask import Flask, Response, render_template_string
+from flask import Flask, Response, render_template_string, request
 
 from ..types import UniMessage
 
@@ -330,6 +330,11 @@ def inline_thinking_summary(item: dict[str, Any]) -> str:
                 <div class="bg-white rounded-lg shadow-sm border border-gray-200 p-4 mb-6">
                     <p class="text-sm text-gray-600"><strong>Path:</strong> {{ breadcrumb|safe }}</p>
                 </div>
+                <div class="flex items-center gap-2 mb-3">
+                    <span class="text-xs text-gray-500">Sort by:</span>
+                    <a href="{{ sort_name_url }}" class="px-3 py-1 text-xs rounded border transition-colors {% if current_sort == 'name' %}bg-blue-600 text-white border-blue-600{% else %}bg-white text-gray-700 border-gray-300 hover:bg-gray-50{% endif %}">Name</a>
+                    <a href="{{ sort_mtime_url }}" class="px-3 py-1 text-xs rounded border transition-colors {% if current_sort == 'mtime' %}bg-blue-600 text-white border-blue-600{% else %}bg-white text-gray-700 border-gray-300 hover:bg-gray-50{% endif %}">Modified Time</a>
+                </div>
                 <div class="bg-white rounded-lg shadow-sm border border-gray-200 overflow-hidden">
                     {% if items %}
                         {% for item in items %}
@@ -339,9 +344,14 @@ def inline_thinking_summary(item: dict[str, Any]) -> str:
                                         <span class="mr-2">{% if item.is_dir %}📁{% else %}📄{% endif %}</span>
                                         <span class="text-sm">{{ item.name }}</span>
                                     </span>
-                                    {% if item.size %}
-                                    <span class="text-xs text-gray-500">{{ item.size }}</span>
-                                    {% endif %}
+                                    <span class="flex items-center gap-4">
+                                        {% if item.mtime %}
+                                        <span class="text-xs text-gray-400">{{ item.mtime }}</span>
+                                        {% endif %}
+                                        {% if item.size %}
+                                        <span class="text-xs text-gray-500">{{ item.size }}</span>
+                                        {% endif %}
+                                    </span>
                                 </a>
                             </div>
                         {% endfor %}
@@ -635,9 +645,19 @@ def browse(subpath: str = "") -> str | Response:
                     return f"Error reading file: {str(e)}", 500
 
             # If it's a directory, list its contents
+            sort_by = request.args.get("sort", "name")
+            if sort_by not in ("name", "mtime"):
+                sort_by = "name"
+
             items = []
             try:
-                for entry in sorted(full_path.iterdir(), key=lambda x: (not x.is_dir(), x.name)):
+                entries = list(full_path.iterdir())
+                if sort_by == "mtime":
+                    entries.sort(key=lambda x: (not x.is_dir(), -x.stat().st_mtime))
+                else:
+                    entries.sort(key=lambda x: (not x.is_dir(), x.name))
+
+                for entry in entries:
                     # Calculate relative path from cache_dir
                     try:
                         relative_path = entry.resolve().relative_to(self.cache_dir.resolve())
@@ -648,6 +668,7 @@ def browse(subpath: str = "") -> str | Response:
                         "name": entry.name,
                         "is_dir": entry.is_dir(),
                         "url": f"/{relative_path}",
+                        "mtime": datetime.fromtimestamp(entry.stat().st_mtime).strftime("%Y-%m-%d %H:%M:%S"),
                     }
                     if entry.is_file():
                         size = entry.stat().st_size
@@ -670,7 +691,18 @@ def browse(subpath: str = "") -> str | Response:
                     breadcrumb_parts.append(f'<a href="/{path_to_part}">{part}</a>')
             breadcrumb = " / ".join(breadcrumb_parts)
 
-            return render_template_string(DIRECTORY_TEMPLATE, items=items, breadcrumb=breadcrumb)
+            base_url = "/" + subpath if subpath else "/"
+            sort_name_url = base_url + "?sort=name"
+            sort_mtime_url = base_url + "?sort=mtime"
+
+            return render_template_string(
+                DIRECTORY_TEMPLATE,
+                items=items,
+                breadcrumb=breadcrumb,
+                current_sort=sort_by,
+                sort_name_url=sort_name_url,
+                sort_mtime_url=sort_mtime_url,
+            )
 
         return app
 

src_py/tests/test_tracer.py

@@ -330,3 +330,91 @@ def test_format_config_with_system_and_tools(temp_cache_dir):
     assert "tools:" in txt_content
     assert "get_weather" in txt_content
     assert "parameters" in txt_content
+
+
+def test_web_app_sort_by_name(temp_cache_dir):
+    """Test directory listing sorted by name."""
+    tracer = Tracer(cache_dir=temp_cache_dir)
+
+    model = "fake-model"
+    history = [{"role": "user", "content_items": [{"type": "text", "text": "Test"}]}]
+    config = {}
+    tracer.save_history(model, history, "zebra/conv", config)
+    tracer.save_history(model, history, "apple/conv", config)
+    tracer.save_history(model, history, "mango/conv", config)
+
+    app = tracer.create_web_app()
+
+    with app.test_client() as client:
+        response = client.get("/?sort=name")
+        assert response.status_code == 200
+        html = response.data.decode()
+
+        # Dirs should appear alphabetically: apple, mango, zebra
+        pos_apple = html.index("apple")
+        pos_mango = html.index("mango")
+        pos_zebra = html.index("zebra")
+        assert pos_apple < pos_mango < pos_zebra
+
+        # Sort controls should be present
+        assert "sort=name" in html
+        assert "sort=mtime" in html
+
+
+def test_web_app_sort_by_mtime(temp_cache_dir):
+    """Test directory listing sorted by modification time (most recent first)."""
+    tracer = Tracer(cache_dir=temp_cache_dir)
+
+    model = "fake-model"
+    history = [{"role": "user", "content_items": [{"type": "text", "text": "Test"}]}]
+    config = {}
+
+    tracer.save_history(model, history, "alpha/conv", config)
+    tracer.save_history(model, history, "beta/conv", config)
+    tracer.save_history(model, history, "gamma/conv", config)
+
+    # Explicitly set directory mtimes so the order is deterministic
+    alpha_dir = Path(temp_cache_dir) / "alpha"
+    beta_dir = Path(temp_cache_dir) / "beta"
+    gamma_dir = Path(temp_cache_dir) / "gamma"
+    os.utime(alpha_dir, (1000, 1000))
+    os.utime(beta_dir, (2000, 2000))
+    os.utime(gamma_dir, (3000, 3000))
+
+    app = tracer.create_web_app()
+
+    with app.test_client() as client:
+        response = client.get("/?sort=mtime")
+        assert response.status_code == 200
+        html = response.data.decode()
+
+        # Most recently modified directory (gamma, mtime=3000) should appear before alpha (mtime=1000)
+        pos_gamma = html.index("gamma")
+        pos_alpha = html.index("alpha")
+        assert pos_gamma < pos_alpha
+
+        # Sort controls should be present
+        assert "sort=name" in html
+        assert "sort=mtime" in html
+
+
+def test_web_app_sort_default_is_name(temp_cache_dir):
+    """Test that the default sort order is by name."""
+    tracer = Tracer(cache_dir=temp_cache_dir)
+
+    model = "fake-model"
+    history = [{"role": "user", "content_items": [{"type": "text", "text": "Test"}]}]
+    config = {}
+    tracer.save_history(model, history, "zebra/conv", config)
+    tracer.save_history(model, history, "apple/conv", config)
+
+    app = tracer.create_web_app()
+
+    with app.test_client() as client:
+        response = client.get("/")
+        assert response.status_code == 200
+        html = response.data.decode()
+
+        pos_apple = html.index("apple")
+        pos_zebra = html.index("zebra")
+        assert pos_apple < pos_zebra