fix: agent history parameter and conditional logfire instrumentation

- Fixed run_agent Modal function passing max_turns where history was expected
- Added history parameter to Modal spawn call for conversation continuity
- Made logfire instrumentation conditional on token being set (cleaner local dev)

Author: nikhilwoodruff

docs/src/app/globals.css

@@ -103,9 +103,9 @@ code {
 
 /* Agent response content */
 .response-content {
-  font-family: var(--font-inter), -apple-system, BlinkMacSystemFont, "Segoe UI", sans-serif;
-  font-size: 15px;
-  line-height: 1.7;
+  font-family: "Instrument Serif", Georgia, "Times New Roman", serif;
+  font-size: 17px;
+  line-height: 1.75;
   color: #1e293b;
   -webkit-font-smoothing: antialiased;
 }

docs/src/components/policy-chat.tsx

@@ -561,10 +561,11 @@ export function PolicyChat() {
                             : "bg-white border border-[var(--color-border)]"
                         }`}>
                           <div
-                            className="response-content font-[family-name:var(--font-inter)]"
+                            className="response-content"
                             style={{
-                              fontSize: '15px',
-                              lineHeight: 1.7,
+                              fontFamily: '"Instrument Serif", Georgia, serif',
+                              fontSize: '17px',
+                              lineHeight: 1.75,
                             }}
                           >
                             <ReactMarkdown remarkPlugins={[remarkBreaks, remarkGfm]}>

src/policyengine_api/agent_sandbox.py

@@ -3,7 +3,7 @@
 import json
 import re
 import time
-from typing import Any, Callable
+from typing import Callable
 
 import anthropic
 import modal
@@ -469,10 +469,17 @@ def run_agent(
     question: str,
     api_base_url: str = "https://v2.api.policyengine.org",
     call_id: str = "",
+    history: list[dict] | None = None,
     max_turns: int = 30,
 ) -> dict:
     """Run agentic loop to answer a policy question (Modal wrapper)."""
-    return _run_agent_impl(question, api_base_url, call_id, max_turns)
+    return _run_agent_impl(
+        question,
+        api_base_url,
+        call_id,
+        history=history,
+        max_turns=max_turns,
+    )
 
 
 if __name__ == "__main__":

src/policyengine_api/api/agent.py

@@ -127,7 +127,8 @@ async def run_agent(request: RunRequest) -> RunResponse:
         import modal
 
         run_fn = modal.Function.from_name("policyengine-sandbox", "run_agent")
-        call = run_fn.spawn(request.question, api_base_url, call_id)
+        history_dicts = [{"role": m.role, "content": m.content} for m in request.history]
+        call = run_fn.spawn(request.question, api_base_url, call_id, history_dicts)
 
         _calls[call_id] = {
             "call": call,

src/policyengine_api/main.py

@@ -17,7 +17,8 @@
 console = Console()
 
 # Configure Logfire (only if token is set)
-if settings.logfire_token:
+_logfire_enabled = bool(settings.logfire_token)
+if _logfire_enabled:
 
     def _scrubbing_callback(m: logfire.ScrubMatch):
         """Allow 'session' through for Claude stream debugging."""
@@ -67,8 +68,9 @@ async def lifespan(app: FastAPI):
     allow_headers=["*"],
 )
 
-# Instrument FastAPI with Logfire
-logfire.instrument_fastapi(app)
+# Instrument FastAPI with Logfire (only if configured)
+if _logfire_enabled:
+    logfire.instrument_fastapi(app)
 
 app.include_router(api_router)
 

src/policyengine_api/services/database.py

@@ -1,10 +1,14 @@
-import logfire
 from sqlmodel import Session, create_engine
 
 from policyengine_api.config.settings import settings
 
 engine = create_engine(settings.database_url, echo=settings.debug)
-logfire.instrument_sqlalchemy(engine=engine)
+
+# Only instrument with logfire if configured
+if settings.logfire_token:
+    import logfire
+
+    logfire.instrument_sqlalchemy(engine=engine)
 
 
 def get_session():