Skip to content

Commit 84c3d0f

Browse files
domenkozarclaude
andcommitted
fix: demote eval cache hit log back to debug level
The "Cache hit" message was promoted to info in 81d76bf to fix a test, but this caused it to print to the terminal during normal usage. Revert to debug level and use --verbose in the cli test instead. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
1 parent 99c4fdf commit 84c3d0f

File tree

2 files changed

+3
-3
lines changed

2 files changed

+3
-3
lines changed

devenv-eval-cache/src/caching_eval.rs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,7 @@ use std::future::Future;
2222
use std::io;
2323
use std::sync::Arc;
2424
use std::time::SystemTime;
25-
use tracing::{debug, info, trace, warn};
25+
use tracing::{debug, trace, warn};
2626

2727
use crate::db::{self, EnvInputRow, EvalRow, FileInputRow};
2828
use crate::eval_inputs::{
@@ -140,7 +140,7 @@ impl CachingEvalService {
140140
// Update timestamp
141141
db::update_eval_updated_at(&self.pool, eval_row.id).await?;
142142

143-
info!(
143+
debug!(
144144
key_hash = %key.key_hash,
145145
attr_name = %key.attr_name,
146146
"Cache hit"

tests/cli/.test.sh

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@ devenv shell ls ../ | grep "cli"
88
devenv info | grep "python3-"
99
devenv show | grep "python3-"
1010
devenv search ncdu 2>&1 | grep -E "Found [0-9]+ packages and [0-9]+ options for 'ncdu'"
11-
devenv --trace-output file:search-trace.log search '^ncdu$' 2>&1 | grep -E "Found [0-9]+ packages and [0-9]+ options"
11+
devenv --verbose --trace-output file:search-trace.log search '^ncdu$' 2>&1 | grep -E "Found [0-9]+ packages and [0-9]+ options"
1212
grep "Cache hit" search-trace.log | grep "optionsJSON"
1313
devenv search xyznonexistentpackagexyz 2>&1 | grep -E "Found 0 packages and 0 options"
1414

0 commit comments

Comments
 (0)