Spaces:
Running
Running
Commit ·
8d46a58
1
Parent(s): 7fa8e8d
session logging
Browse files- agent/config.py +2 -0
- agent/core/agent_loop.py +10 -0
- agent/core/session.py +76 -1
- agent/main.py +2 -2
- configs/main_agent_config.json +3 -1
- pyproject.toml +2 -0
- uv.lock +4 -0
agent/config.py
CHANGED
|
@@ -19,6 +19,8 @@ class Config(BaseModel):
|
|
| 19 |
|
| 20 |
model_name: str
|
| 21 |
mcpServers: dict[str, MCPServerConfig] = {}
|
|
|
|
|
|
|
| 22 |
|
| 23 |
|
| 24 |
def substitute_env_vars(obj: Any) -> Any:
|
|
|
|
| 19 |
|
| 20 |
model_name: str
|
| 21 |
mcpServers: dict[str, MCPServerConfig] = {}
|
| 22 |
+
save_sessions: bool = True
|
| 23 |
+
session_dataset_repo: str = "smolagents/hf-agent-sessions"
|
| 24 |
|
| 25 |
|
| 26 |
def substitute_env_vars(obj: Any) -> Any:
|
agent/core/agent_loop.py
CHANGED
|
@@ -414,6 +414,16 @@ class Handlers:
|
|
| 414 |
@staticmethod
|
| 415 |
async def shutdown(session: Session) -> bool:
|
| 416 |
"""Handle shutdown (like shutdown in codex.rs:1329)"""
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| 417 |
session.is_running = False
|
| 418 |
await session.send_event(Event(event_type="shutdown"))
|
| 419 |
return True
|
|
|
|
| 414 |
@staticmethod
|
| 415 |
async def shutdown(session: Session) -> bool:
|
| 416 |
"""Handle shutdown (like shutdown in codex.rs:1329)"""
|
| 417 |
+
# Save session trajectory if enabled
|
| 418 |
+
if session.config.save_sessions:
|
| 419 |
+
print("💾 Saving session trajectory...")
|
| 420 |
+
repo_id = session.config.session_dataset_repo
|
| 421 |
+
url = await session.push_to_dataset(repo_id)
|
| 422 |
+
if url:
|
| 423 |
+
print(f"✅ Session saved to: {url}")
|
| 424 |
+
else:
|
| 425 |
+
print("❌ Failed to save session")
|
| 426 |
+
|
| 427 |
session.is_running = False
|
| 428 |
await session.send_event(Event(event_type="shutdown"))
|
| 429 |
return True
|
agent/core/session.py
CHANGED
|
@@ -1,6 +1,8 @@
|
|
| 1 |
import asyncio
|
|
|
|
| 2 |
import uuid
|
| 3 |
from dataclasses import dataclass
|
|
|
|
| 4 |
from enum import Enum
|
| 5 |
from typing import Any, Optional
|
| 6 |
|
|
@@ -55,11 +57,84 @@ class Session:
|
|
| 55 |
self.current_task: asyncio.Task | None = None
|
| 56 |
self.pending_approval: Optional[dict[str, Any]] = None
|
| 57 |
|
|
|
|
|
|
|
|
|
|
|
|
|
| 58 |
async def send_event(self, event: Event) -> None:
|
| 59 |
-
"""Send event back to client"""
|
| 60 |
await self.event_queue.put(event)
|
| 61 |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| 62 |
def interrupt(self) -> None:
|
| 63 |
"""Interrupt current running task"""
|
| 64 |
if self.current_task and not self.current_task.done():
|
| 65 |
self.current_task.cancel()
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| 1 |
import asyncio
|
| 2 |
+
import json
|
| 3 |
import uuid
|
| 4 |
from dataclasses import dataclass
|
| 5 |
+
from datetime import datetime
|
| 6 |
from enum import Enum
|
| 7 |
from typing import Any, Optional
|
| 8 |
|
|
|
|
| 57 |
self.current_task: asyncio.Task | None = None
|
| 58 |
self.pending_approval: Optional[dict[str, Any]] = None
|
| 59 |
|
| 60 |
+
# Session trajectory logging
|
| 61 |
+
self.logged_events: list[dict] = []
|
| 62 |
+
self.session_start_time = datetime.now().isoformat()
|
| 63 |
+
|
| 64 |
async def send_event(self, event: Event) -> None:
|
| 65 |
+
"""Send event back to client and log to trajectory"""
|
| 66 |
await self.event_queue.put(event)
|
| 67 |
|
| 68 |
+
# Log event to trajectory
|
| 69 |
+
self.logged_events.append(
|
| 70 |
+
{
|
| 71 |
+
"timestamp": datetime.now().isoformat(),
|
| 72 |
+
"event_type": event.event_type,
|
| 73 |
+
"data": event.data,
|
| 74 |
+
}
|
| 75 |
+
)
|
| 76 |
+
|
| 77 |
def interrupt(self) -> None:
|
| 78 |
"""Interrupt current running task"""
|
| 79 |
if self.current_task and not self.current_task.done():
|
| 80 |
self.current_task.cancel()
|
| 81 |
+
|
| 82 |
+
def get_trajectory(self) -> dict:
|
| 83 |
+
"""Serialize complete session trajectory for logging"""
|
| 84 |
+
return {
|
| 85 |
+
"session_id": self.session_id,
|
| 86 |
+
"session_start_time": self.session_start_time,
|
| 87 |
+
"session_end_time": datetime.now().isoformat(),
|
| 88 |
+
"model_name": self.config.model_name,
|
| 89 |
+
"messages": [msg.model_dump() for msg in self.context_manager.items],
|
| 90 |
+
"events": self.logged_events,
|
| 91 |
+
}
|
| 92 |
+
|
| 93 |
+
async def push_to_dataset(self, repo_id: str) -> Optional[str]:
|
| 94 |
+
"""
|
| 95 |
+
Push session trajectory to Hugging Face dataset
|
| 96 |
+
|
| 97 |
+
Args:
|
| 98 |
+
repo_id: HuggingFace dataset repo ID (e.g. 'username/dataset-name')
|
| 99 |
+
|
| 100 |
+
Returns:
|
| 101 |
+
URL to the uploaded file if successful, None otherwise
|
| 102 |
+
"""
|
| 103 |
+
try:
|
| 104 |
+
import os
|
| 105 |
+
|
| 106 |
+
from datasets import Dataset
|
| 107 |
+
|
| 108 |
+
# Get trajectory data
|
| 109 |
+
trajectory = self.get_trajectory()
|
| 110 |
+
|
| 111 |
+
# Convert to dataset row format
|
| 112 |
+
row = {
|
| 113 |
+
"session_id": trajectory["session_id"],
|
| 114 |
+
"session_start_time": trajectory["session_start_time"],
|
| 115 |
+
"session_end_time": trajectory["session_end_time"],
|
| 116 |
+
"model_name": trajectory["model_name"],
|
| 117 |
+
"messages": json.dumps(trajectory["messages"]),
|
| 118 |
+
"events": json.dumps(trajectory["events"]),
|
| 119 |
+
}
|
| 120 |
+
|
| 121 |
+
# Try to load existing dataset and append
|
| 122 |
+
try:
|
| 123 |
+
from datasets import load_dataset
|
| 124 |
+
|
| 125 |
+
existing_dataset = load_dataset(repo_id, split="train")
|
| 126 |
+
new_dataset = Dataset.from_dict(
|
| 127 |
+
{k: list(existing_dataset[k]) + [v] for k, v in row.items()}
|
| 128 |
+
)
|
| 129 |
+
except Exception:
|
| 130 |
+
# Dataset doesn't exist yet, create new one
|
| 131 |
+
new_dataset = Dataset.from_dict({k: [v] for k, v in row.items()})
|
| 132 |
+
|
| 133 |
+
# Push to hub
|
| 134 |
+
new_dataset.push_to_hub(repo_id, private=True, token=os.getenv("HF_TOKEN"))
|
| 135 |
+
|
| 136 |
+
return f"https://huggingface.co/datasets/{repo_id}"
|
| 137 |
+
|
| 138 |
+
except Exception as e:
|
| 139 |
+
print(f"Failed to push session to dataset: {e}")
|
| 140 |
+
return None
|
agent/main.py
CHANGED
|
@@ -415,8 +415,8 @@ async def main():
|
|
| 415 |
)
|
| 416 |
await submission_queue.put(shutdown_submission)
|
| 417 |
|
| 418 |
-
# Wait for tasks to complete
|
| 419 |
-
await asyncio.wait_for(agent_task, timeout=
|
| 420 |
listener_task.cancel()
|
| 421 |
|
| 422 |
print("✨ Goodbye!\n")
|
|
|
|
| 415 |
)
|
| 416 |
await submission_queue.put(shutdown_submission)
|
| 417 |
|
| 418 |
+
# Wait for tasks to complete (longer timeout to allow for session save)
|
| 419 |
+
await asyncio.wait_for(agent_task, timeout=30.0)
|
| 420 |
listener_task.cancel()
|
| 421 |
|
| 422 |
print("✨ Goodbye!\n")
|
configs/main_agent_config.json
CHANGED
|
@@ -1,5 +1,7 @@
|
|
| 1 |
{
|
| 2 |
-
"model_name": "anthropic/claude-
|
|
|
|
|
|
|
| 3 |
"mcpServers": {
|
| 4 |
"hf-mcp-server": {
|
| 5 |
"transport": "http",
|
|
|
|
| 1 |
{
|
| 2 |
+
"model_name": "anthropic/claude-opus-4-5-20251101",
|
| 3 |
+
"save_sessions": true,
|
| 4 |
+
"session_dataset_repo": "smolagents/hf-agent-sessions",
|
| 5 |
"mcpServers": {
|
| 6 |
"hf-mcp-server": {
|
| 7 |
"transport": "http",
|
pyproject.toml
CHANGED
|
@@ -5,6 +5,7 @@ description = "Add your description here"
|
|
| 5 |
readme = "README.md"
|
| 6 |
requires-python = ">=3.12"
|
| 7 |
dependencies = [
|
|
|
|
| 8 |
# Core dependencies (always required)
|
| 9 |
"pydantic>=2.12.3",
|
| 10 |
"python-dotenv>=1.2.1",
|
|
@@ -22,6 +23,7 @@ agent = [
|
|
| 22 |
"thefuzz>=0.22.1",
|
| 23 |
"nbconvert>=7.16.6",
|
| 24 |
"nbformat>=5.10.4",
|
|
|
|
| 25 |
]
|
| 26 |
|
| 27 |
# Evaluation/benchmarking dependencies
|
|
|
|
| 5 |
readme = "README.md"
|
| 6 |
requires-python = ">=3.12"
|
| 7 |
dependencies = [
|
| 8 |
+
"datasets>=4.4.1",
|
| 9 |
# Core dependencies (always required)
|
| 10 |
"pydantic>=2.12.3",
|
| 11 |
"python-dotenv>=1.2.1",
|
|
|
|
| 23 |
"thefuzz>=0.22.1",
|
| 24 |
"nbconvert>=7.16.6",
|
| 25 |
"nbformat>=5.10.4",
|
| 26 |
+
"datasets>=4.3.0", # For session logging to HF datasets
|
| 27 |
]
|
| 28 |
|
| 29 |
# Evaluation/benchmarking dependencies
|
uv.lock
CHANGED
|
@@ -902,12 +902,14 @@ name = "hf-agent"
|
|
| 902 |
version = "0.1.0"
|
| 903 |
source = { virtual = "." }
|
| 904 |
dependencies = [
|
|
|
|
| 905 |
{ name = "pydantic" },
|
| 906 |
{ name = "python-dotenv" },
|
| 907 |
]
|
| 908 |
|
| 909 |
[package.optional-dependencies]
|
| 910 |
agent = [
|
|
|
|
| 911 |
{ name = "fastmcp" },
|
| 912 |
{ name = "huggingface-hub" },
|
| 913 |
{ name = "litellm" },
|
|
@@ -946,6 +948,8 @@ eval = [
|
|
| 946 |
|
| 947 |
[package.metadata]
|
| 948 |
requires-dist = [
|
|
|
|
|
|
|
| 949 |
{ name = "datasets", marker = "extra == 'eval'", specifier = ">=4.3.0" },
|
| 950 |
{ name = "fastmcp", marker = "extra == 'agent'", specifier = ">=2.4.0" },
|
| 951 |
{ name = "hf-agent", extras = ["agent", "eval", "dev"], marker = "extra == 'all'" },
|
|
|
|
| 902 |
version = "0.1.0"
|
| 903 |
source = { virtual = "." }
|
| 904 |
dependencies = [
|
| 905 |
+
{ name = "datasets" },
|
| 906 |
{ name = "pydantic" },
|
| 907 |
{ name = "python-dotenv" },
|
| 908 |
]
|
| 909 |
|
| 910 |
[package.optional-dependencies]
|
| 911 |
agent = [
|
| 912 |
+
{ name = "datasets" },
|
| 913 |
{ name = "fastmcp" },
|
| 914 |
{ name = "huggingface-hub" },
|
| 915 |
{ name = "litellm" },
|
|
|
|
| 948 |
|
| 949 |
[package.metadata]
|
| 950 |
requires-dist = [
|
| 951 |
+
{ name = "datasets", specifier = ">=4.4.1" },
|
| 952 |
+
{ name = "datasets", marker = "extra == 'agent'", specifier = ">=4.3.0" },
|
| 953 |
{ name = "datasets", marker = "extra == 'eval'", specifier = ">=4.3.0" },
|
| 954 |
{ name = "fastmcp", marker = "extra == 'agent'", specifier = ">=2.4.0" },
|
| 955 |
{ name = "hf-agent", extras = ["agent", "eval", "dev"], marker = "extra == 'all'" },
|