mirror of
https://github.com/outbackdingo/optimclaw.git
synced 2026-08-26 23:50:17 +00:00
* fix: strip reasoning from LLM responses and persist assistant messages reliably - Filter out `type: "reasoning"` output items from NEAR AI Responses API parsing so chain-of-thought never reaches the UI (nearai.rs) - Rewrite clean_response with regex-based tag stripping that is code-aware (preserves tags inside fenced blocks and inline backticks), supports 9+ tag names (think, thought, reasoning, reflection, etc.), handles <final> extraction, pipe-delimited tags, and case/whitespace tolerance (reasoning.rs) - Add Reasoning::complete() helper so all non-agentic LLM call sites (summarize, suggest, heartbeat, compaction) get automatic response cleaning; thread SafetyLayer through to those callers - Change persist_turn from fire-and-forget tokio::spawn to awaited async so both user and assistant messages are written before returning, preventing data loss on shutdown/restart - Pass input_count through seed_response_chain so response chaining delta calculation is accurate after thread hydration on restart - Make NearAiResponse.usage optional and preserve response_id in alt response path for chaining continuity - Persist session token to DB during onboarding wizard so runtime loads it without legacy-key fallback; suppress spurious warning on fresh installs - Fix dev tool double-registration when builder already registers them - Load dotenv/ironclaw env for doctor and status subcommands - Reduce startup log noise (demote info→debug for skills, remove redundant info lines) Co-Authored-By: Claude Opus 4.6 <[email protected]> * Nudge to not loop over tools continuesly * refactor: remove Responses API, consolidate NEAR AI to Chat Completions only The Responses API provider (nearai.rs, 1278 lines) added significant complexity (response chaining state machine, delta message calculation, previous_response_id persistence) for marginal benefit. This consolidates to the Chat Completions API only, upgrading NearAiChatProvider with dual auth (session token + API key) and 401 retry for session token renewal. - Delete src/llm/nearai.rs (Responses API provider) - Upgrade nearai_chat.rs with SessionManager, dual auth, flexible list_models - Remove response_id from CompletionResponse and ToolCompletionResponse - Remove seed_response_chain/get_response_chain_id from LlmProvider trait - Remove response chain persistence from agent (thread_ops, session) - Remove NearAiApiMode enum and NEARAI_API_MODE config - Clean up all wrapper providers (retry, circuit_breaker, failover, cache) - Update documentation (CLAUDE.md, .env.example) Co-Authored-By: Claude Opus 4.6 <[email protected]> * feat: runtime log level control via gateway UI and URL parameter Add server-side log level switching using tracing_subscriber::reload::Layer so the EnvFilter can be swapped at runtime without restarting. Expose via GET/PUT /api/logs/level endpoints, a "Server: LEVEL" dropdown in the logs toolbar, and a ?log_level=debug URL parameter for one-click activation. Also applies cargo fmt to pre-existing files (llm/, tests/). Co-Authored-By: Claude Opus 4.6 <[email protected]> --------- Co-authored-by: Claude Opus 4.6 <[email protected]>
355 lines
10 KiB
Rust
355 lines
10 KiB
Rust
//! Test harness for constructing `AgentDeps` with sensible defaults.
|
|
//!
|
|
//! Provides:
|
|
//! - [`StubLlm`]: A configurable LLM provider that returns a fixed response
|
|
//! - [`TestHarnessBuilder`]: Builder for wiring `AgentDeps` with defaults
|
|
//! - [`TestHarness`]: The assembled components ready for use in tests
|
|
//!
|
|
//! # Usage
|
|
//!
|
|
//! ```rust,no_run
|
|
//! use ironclaw::testing::TestHarnessBuilder;
|
|
//!
|
|
//! #[tokio::test]
|
|
//! async fn test_something() {
|
|
//! let harness = TestHarnessBuilder::new().build().await;
|
|
//! // use harness.deps, harness.db, etc.
|
|
//! }
|
|
//! ```
|
|
|
|
use std::sync::Arc;
|
|
use std::sync::atomic::{AtomicBool, AtomicU32, Ordering};
|
|
|
|
use async_trait::async_trait;
|
|
use rust_decimal::Decimal;
|
|
|
|
use crate::agent::AgentDeps;
|
|
use crate::db::Database;
|
|
use crate::error::LlmError;
|
|
use crate::llm::{
|
|
CompletionRequest, CompletionResponse, FinishReason, LlmProvider, ToolCompletionRequest,
|
|
ToolCompletionResponse,
|
|
};
|
|
use crate::tools::ToolRegistry;
|
|
|
|
/// Create a libSQL-backed test database in a temporary directory.
|
|
///
|
|
/// Returns the database and a `TempDir` guard — the database file is
|
|
/// deleted when the guard is dropped.
|
|
#[cfg(feature = "libsql")]
|
|
pub async fn test_db() -> (Arc<dyn Database>, tempfile::TempDir) {
|
|
use crate::db::libsql::LibSqlBackend;
|
|
|
|
let dir = tempfile::tempdir().expect("failed to create temp dir");
|
|
let path = dir.path().join("test.db");
|
|
let backend = LibSqlBackend::new_local(&path)
|
|
.await
|
|
.expect("failed to create test LibSqlBackend");
|
|
backend
|
|
.run_migrations()
|
|
.await
|
|
.expect("failed to run migrations");
|
|
(Arc::new(backend) as Arc<dyn Database>, dir)
|
|
}
|
|
|
|
/// What kind of error the stub should produce when failing.
|
|
#[derive(Clone, Copy, Debug)]
|
|
pub enum StubErrorKind {
|
|
/// Transient/retryable error (`LlmError::RequestFailed`).
|
|
Transient,
|
|
/// Non-transient error (`LlmError::ContextLengthExceeded`).
|
|
NonTransient,
|
|
}
|
|
|
|
/// A configurable LLM provider stub for tests.
|
|
///
|
|
/// Supports:
|
|
/// - Fixed response content
|
|
/// - Call counting via [`calls()`](Self::calls)
|
|
/// - Runtime failure toggling via [`set_failing()`](Self::set_failing)
|
|
/// - Configurable error kinds (transient vs non-transient)
|
|
///
|
|
/// Use this in tests instead of creating ad-hoc stub implementations.
|
|
pub struct StubLlm {
|
|
model_name: String,
|
|
response: String,
|
|
call_count: AtomicU32,
|
|
should_fail: AtomicBool,
|
|
error_kind: StubErrorKind,
|
|
}
|
|
|
|
impl StubLlm {
|
|
/// Create a new stub that returns the given response.
|
|
pub fn new(response: impl Into<String>) -> Self {
|
|
Self {
|
|
model_name: "stub-model".to_string(),
|
|
response: response.into(),
|
|
call_count: AtomicU32::new(0),
|
|
should_fail: AtomicBool::new(false),
|
|
error_kind: StubErrorKind::Transient,
|
|
}
|
|
}
|
|
|
|
/// Create a stub that always fails with a transient error.
|
|
pub fn failing(name: impl Into<String>) -> Self {
|
|
Self {
|
|
model_name: name.into(),
|
|
response: String::new(),
|
|
call_count: AtomicU32::new(0),
|
|
should_fail: AtomicBool::new(true),
|
|
error_kind: StubErrorKind::Transient,
|
|
}
|
|
}
|
|
|
|
/// Create a stub that always fails with a non-transient error.
|
|
pub fn failing_non_transient(name: impl Into<String>) -> Self {
|
|
Self {
|
|
model_name: name.into(),
|
|
response: String::new(),
|
|
call_count: AtomicU32::new(0),
|
|
should_fail: AtomicBool::new(true),
|
|
error_kind: StubErrorKind::NonTransient,
|
|
}
|
|
}
|
|
|
|
/// Set the model name.
|
|
pub fn with_model_name(mut self, name: impl Into<String>) -> Self {
|
|
self.model_name = name.into();
|
|
self
|
|
}
|
|
|
|
/// Get the number of times `complete` or `complete_with_tools` was called.
|
|
pub fn calls(&self) -> u32 {
|
|
self.call_count.load(Ordering::Relaxed)
|
|
}
|
|
|
|
/// Toggle whether calls should fail at runtime.
|
|
pub fn set_failing(&self, fail: bool) {
|
|
self.should_fail.store(fail, Ordering::Relaxed);
|
|
}
|
|
|
|
fn make_error(&self) -> LlmError {
|
|
match self.error_kind {
|
|
StubErrorKind::Transient => LlmError::RequestFailed {
|
|
provider: self.model_name.clone(),
|
|
reason: "server error".to_string(),
|
|
},
|
|
StubErrorKind::NonTransient => LlmError::ContextLengthExceeded {
|
|
used: 100_000,
|
|
limit: 50_000,
|
|
},
|
|
}
|
|
}
|
|
}
|
|
|
|
impl Default for StubLlm {
|
|
fn default() -> Self {
|
|
Self::new("OK")
|
|
}
|
|
}
|
|
|
|
#[async_trait]
|
|
impl LlmProvider for StubLlm {
|
|
fn model_name(&self) -> &str {
|
|
&self.model_name
|
|
}
|
|
|
|
fn cost_per_token(&self) -> (Decimal, Decimal) {
|
|
(Decimal::ZERO, Decimal::ZERO)
|
|
}
|
|
|
|
async fn complete(&self, _request: CompletionRequest) -> Result<CompletionResponse, LlmError> {
|
|
self.call_count.fetch_add(1, Ordering::Relaxed);
|
|
if self.should_fail.load(Ordering::Relaxed) {
|
|
return Err(self.make_error());
|
|
}
|
|
Ok(CompletionResponse {
|
|
content: self.response.clone(),
|
|
input_tokens: 10,
|
|
output_tokens: 5,
|
|
finish_reason: FinishReason::Stop,
|
|
})
|
|
}
|
|
|
|
async fn complete_with_tools(
|
|
&self,
|
|
_request: ToolCompletionRequest,
|
|
) -> Result<ToolCompletionResponse, LlmError> {
|
|
self.call_count.fetch_add(1, Ordering::Relaxed);
|
|
if self.should_fail.load(Ordering::Relaxed) {
|
|
return Err(self.make_error());
|
|
}
|
|
Ok(ToolCompletionResponse {
|
|
content: Some(self.response.clone()),
|
|
tool_calls: Vec::new(),
|
|
input_tokens: 10,
|
|
output_tokens: 5,
|
|
finish_reason: FinishReason::Stop,
|
|
})
|
|
}
|
|
}
|
|
|
|
/// Assembled test components.
|
|
pub struct TestHarness {
|
|
/// The agent dependencies, ready for use.
|
|
pub deps: AgentDeps,
|
|
/// Direct reference to the database (as `Arc<dyn Database>`).
|
|
pub db: Arc<dyn Database>,
|
|
/// Temp directory guard — keeps the test database alive. Dropped
|
|
/// automatically when the harness goes out of scope.
|
|
#[cfg(feature = "libsql")]
|
|
_temp_dir: tempfile::TempDir,
|
|
}
|
|
|
|
/// Builder for constructing a [`TestHarness`] with sensible defaults.
|
|
///
|
|
/// All defaults are designed to work without any external services:
|
|
/// - Database: libSQL in a temp directory (real SQL, FTS5, no network)
|
|
/// - LLM: `StubLlm` returning "OK"
|
|
/// - Safety: permissive config
|
|
/// - Tools: builtin tools registered
|
|
/// - Hooks: empty registry
|
|
/// - Cost guard: no limits
|
|
pub struct TestHarnessBuilder {
|
|
db: Option<Arc<dyn Database>>,
|
|
llm: Option<Arc<dyn LlmProvider>>,
|
|
tools: Option<Arc<ToolRegistry>>,
|
|
}
|
|
|
|
impl TestHarnessBuilder {
|
|
/// Create a new builder with all defaults.
|
|
pub fn new() -> Self {
|
|
Self {
|
|
db: None,
|
|
llm: None,
|
|
tools: None,
|
|
}
|
|
}
|
|
|
|
/// Override the database backend.
|
|
pub fn with_db(mut self, db: Arc<dyn Database>) -> Self {
|
|
self.db = Some(db);
|
|
self
|
|
}
|
|
|
|
/// Override the LLM provider.
|
|
pub fn with_llm(mut self, llm: Arc<dyn LlmProvider>) -> Self {
|
|
self.llm = Some(llm);
|
|
self
|
|
}
|
|
|
|
/// Override the tool registry.
|
|
pub fn with_tools(mut self, tools: Arc<ToolRegistry>) -> Self {
|
|
self.tools = Some(tools);
|
|
self
|
|
}
|
|
|
|
/// Build the harness with defaults applied.
|
|
#[cfg(feature = "libsql")]
|
|
pub async fn build(self) -> TestHarness {
|
|
use crate::agent::cost_guard::{CostGuard, CostGuardConfig};
|
|
use crate::config::{SafetyConfig, SkillsConfig};
|
|
use crate::hooks::HookRegistry;
|
|
use crate::safety::SafetyLayer;
|
|
|
|
let (db, temp_dir) = if let Some(db) = self.db {
|
|
// Caller provided a DB; create a dummy temp dir to satisfy the struct.
|
|
let dir = tempfile::tempdir().expect("failed to create temp dir");
|
|
(db, dir)
|
|
} else {
|
|
test_db().await
|
|
};
|
|
|
|
let llm: Arc<dyn LlmProvider> = self.llm.unwrap_or_else(|| Arc::new(StubLlm::default()));
|
|
|
|
let tools = self.tools.unwrap_or_else(|| {
|
|
let t = Arc::new(ToolRegistry::new());
|
|
t.register_builtin_tools();
|
|
t
|
|
});
|
|
|
|
let safety = Arc::new(SafetyLayer::new(&SafetyConfig {
|
|
max_output_length: 100_000,
|
|
injection_check_enabled: false,
|
|
}));
|
|
|
|
let hooks = Arc::new(HookRegistry::new());
|
|
|
|
let cost_guard = Arc::new(CostGuard::new(CostGuardConfig {
|
|
max_cost_per_day_cents: None,
|
|
max_actions_per_hour: None,
|
|
}));
|
|
|
|
let deps = AgentDeps {
|
|
store: Some(Arc::clone(&db)),
|
|
llm,
|
|
cheap_llm: None,
|
|
safety,
|
|
tools,
|
|
workspace: None,
|
|
extension_manager: None,
|
|
skill_registry: None,
|
|
skills_config: SkillsConfig::default(),
|
|
hooks,
|
|
cost_guard,
|
|
};
|
|
|
|
TestHarness {
|
|
deps,
|
|
db,
|
|
_temp_dir: temp_dir,
|
|
}
|
|
}
|
|
}
|
|
|
|
impl Default for TestHarnessBuilder {
|
|
fn default() -> Self {
|
|
Self::new()
|
|
}
|
|
}
|
|
|
|
#[cfg(test)]
|
|
mod tests {
|
|
use super::*;
|
|
|
|
#[cfg(feature = "libsql")]
|
|
#[tokio::test]
|
|
async fn test_harness_builds_with_defaults() {
|
|
let harness = TestHarnessBuilder::new().build().await;
|
|
assert!(harness.deps.store.is_some());
|
|
assert_eq!(harness.deps.llm.model_name(), "stub-model");
|
|
}
|
|
|
|
#[cfg(feature = "libsql")]
|
|
#[tokio::test]
|
|
async fn test_harness_custom_llm() {
|
|
let custom_llm = Arc::new(StubLlm::new("custom response").with_model_name("my-model"));
|
|
let harness = TestHarnessBuilder::new().with_llm(custom_llm).build().await;
|
|
assert_eq!(harness.deps.llm.model_name(), "my-model");
|
|
}
|
|
|
|
#[cfg(feature = "libsql")]
|
|
#[tokio::test]
|
|
async fn test_harness_db_works() {
|
|
let harness = TestHarnessBuilder::new().build().await;
|
|
|
|
let id = harness
|
|
.db
|
|
.create_conversation("test", "user1", None)
|
|
.await
|
|
.expect("create conversation");
|
|
assert!(!id.is_nil());
|
|
}
|
|
|
|
#[tokio::test]
|
|
async fn test_stub_llm_complete() {
|
|
let llm = StubLlm::new("hello world");
|
|
let response = llm
|
|
.complete(CompletionRequest::new(vec![]))
|
|
.await
|
|
.expect("complete");
|
|
assert_eq!(response.content, "hello world");
|
|
assert_eq!(response.finish_reason, FinishReason::Stop);
|
|
}
|
|
}
|