Lot 42 : outils web de l'agent exécutés par la façade

Les outils web des tours de la façade (web_search, web_images, web_open,
web_read, web_grep) ne passent plus par /tool/exec du Go : src/webtools.rs
porte chat_internet_tools.go, chat_internet.go et chat_web_images.go. Le
cache des pages (10 min, clé MD5 des options comme le Go) est dans la
façade ; client Crawl4AI au corps près, réponses lues comme json.Unmarshal ;
résultats DuckDuckGo tirés du markdown (uddg, entités, extraits) ;
recherche d'images avec le jeton vqd et les cookies gardés par hôte ;
web_grep sur les regex du Go, erreurs au texte de regexp.

Le moteur intégré lit le HTML avec go-readability et html-to-markdown, que
rien en Rust ne reproduit au caractère près : la conversion d'une page et la
lecture des résultats restent au Go, derrière /internal/v1/web/fetch et
/internal/v1/web/search (sans état). web_screenshot reste au Go.

Le traducteur go_regex refuse la répétition d'une répétition et les comptes
au-delà de 1000, et rend littéral un « { » qui n'ouvre pas de répétition,
comme le Go (motifs retry_patterns compris).

Parité : journal de 74 appels tiré du Go (parity_web_tools_test.go →
tests/parity_web/outils.json : résultats, requêtes reçues par les faux
Crawl4AI et DuckDuckGo, pages demandées au moteur intégré), rejoué par
tests/webtools.rs ; un tour de la façade qui ouvre une page (tests/lot6.rs) ;
TestInternalMoteurWebIntegre côté Go.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_016tcGfJWefW5kUGoC2wL8cv
This commit is contained in:
Claude committed 2026-10-16 05:29:35 +00:00
1 parent bed6d07739
commit f66d1b7621
19 files changed
+4561 -7

No files matched your search

+1
View File
@@ -1322,6 +1322,7 @@ impl ChatShared {
mcp: None,
tasks: None,
memory: None,
web: Default::default(),
go_version: Default::default(),
}
}
+9
View File
@@ -347,6 +347,8 @@ pub struct ChatShared {
/// La mémoire, quand la façade la tient (LOKI_MEM=rust) : les outils
/// `mem_*` et `tracker` du tour s'exécutent ici (lot 40).
pub memory: Option<Arc<crate::memory::Memory>>,
/// Les outils web (lot 42) : cache des pages, Crawl4AI, DuckDuckGo.
pub web: Arc<crate::webtools::Web>,
/// La version du Go, annoncée aux serveurs MCP à une reconnexion ; lue
/// une fois.
pub go_version: tokio::sync::OnceCell<String>,
@@ -698,6 +700,13 @@ impl TurnHost for GoTurnHost {
};
}
}
// Outils web (lot 42) : le cache des pages est ici.
if let Some(result) = self.shared.web.exec(&self.shared.store, name, args).await {
return ToolOutcome {
result,
..Default::default()
};
}
// Outils task_*, quand la façade tient les tâches (lot 37).
if crate::tasks::tools::is_tool(name)
&& let Some(tasks) = &self.shared.tasks
+84
View File
@@ -968,8 +968,67 @@ pub fn go_regex(p: &str) -> Option<String> {
let cs: Vec<char> = p.chars().collect();
let mut out = String::with_capacity(p.len() + 16);
let (mut i, mut class, mut class_start) = (0, false, false);
// Répétition qui précède : 0 aucune, 1 une répétition (un « ? » la
// rend paresseuse), 2 déjà paresseuse. Le Go refuse d'en répéter une.
let mut rep = 0u8;
while i < cs.len() {
let c = cs[i];
if !class {
match c {
'*' | '+' | '?' => {
if rep == 1 && c == '?' {
rep = 2;
} else if rep != 0 {
return None;
} else {
rep = 1;
}
out.push(c);
i += 1;
continue;
}
'{' => {
// `{n}`, `{n,}`, `{n,m}` (1000 au plus) ; sinon un « { »
// littéral pour le Go.
let rest: String = cs[i + 1..].iter().collect();
let body = rest.split_once('}').map(|(b, _)| b);
let parsed = body.and_then(|b| {
let (lo, hi) = match b.split_once(',') {
Some((lo, hi)) => (lo, Some(hi)),
None => (b, None),
};
let num = |t: &str| {
(!t.is_empty() && t.bytes().all(|d| d.is_ascii_digit()))
.then(|| t.parse::<u64>().unwrap_or(u64::MAX))
};
let lo = num(lo)?;
let hi = match hi {
None => Some(lo),
Some("") => None,
Some(h) => Some(num(h)?),
};
Some((b.chars().count(), lo, hi))
});
match parsed {
Some((len, lo, hi)) => {
if lo > 1000 || hi.is_some_and(|h| h > 1000 || h < lo) || rep != 0 {
return None;
}
rep = 1;
out.extend(&cs[i..i + len + 2]);
i += len + 2;
}
None => {
rep = 0;
out.push_str(r"\{");
i += 1;
}
}
continue;
}
_ => rep = 0,
}
}
if c == '\\' && i + 1 < cs.len() {
let n = cs[i + 1];
i += 2;
@@ -990,6 +1049,18 @@ pub fn go_regex(p: &str) -> Option<String> {
class_start = false;
continue;
}
// `\p{…}`, `\P{…}`, `\x{…}` : les accolades sont à l'échappement.
if matches!(n, 'p' | 'P' | 'x')
&& cs.get(i) == Some(&'{')
&& let Some(k) = cs[i..].iter().position(|&c| c == '}')
{
out.push('\\');
out.push(n);
out.extend(&cs[i..=i + k]);
i += k + 1;
class_start = false;
continue;
}
// Un chiffre seul est une référence arrière pour le Go, qui la
// refuse ; avec l'octal permis, `regex` l'accepterait.
if ('1'..='7').contains(&n) && !cs.get(i).is_some_and(|d| ('0'..='7').contains(d)) {
@@ -2872,6 +2943,19 @@ mod tests {
assert_eq!(m(r"\Q(x", "a(x"), Some("(x".into()));
// Le Go refuse \Q dans une classe.
assert_eq!(go_regex(r"[\Q]-\E]+"), None);
// Répétition d'une répétition : refusée par le Go ; « ? » paresseux
// permis une fois ; « { » hors répétition littéral ; bornes ≤ 1000.
for p in ["a**", "a+*", "a*??", "a{2}*", "a*{2}", "a{1001}", "a{3,2}"] {
assert_eq!(go_regex(p), None, "{p}");
}
assert_eq!(go_regex("a*?b+?c??").as_deref(), Some("a*?b+?c??"));
assert_eq!(go_regex("a{2,}{").as_deref(), Some(r"a{2,}\{"));
assert_eq!(go_regex("x{a}").as_deref(), Some(r"x\{a}"));
assert!(compile_go_regex("x{,3}").unwrap().is_match("x{,3}"));
assert_eq!(
go_regex(r"\p{Greek}+\x{41}{2}").as_deref(),
Some(r"\p{Greek}+\x{41}{2}")
);
// Dans une classe, « [ », « && », « -- » et « ~~ » sont littéraux.
assert_eq!(m(r"[a[]+", "x[a["), Some("[a[".into()));
assert_eq!(m(r"[a&&b]+", "x&ab"), Some("&ab".into()));
+1 -1
View File
@@ -304,7 +304,7 @@ fn kind_of(v: &Value) -> &'static str {
}
/// `quoteChar` du scanner Go.
fn quote_char(c: u8) -> String {
pub(crate) fn quote_char(c: u8) -> String {
match c {
b'\'' => r"'\''".into(),
b'"' => "'\"'".into(),
+1
View File
@@ -52,3 +52,4 @@ pub mod tune;
pub mod tz;
pub mod ui;
pub mod update;
pub mod webtools;
File diff suppressed because it is too large. Load diff
+192
View File
@@ -0,0 +1,192 @@
use super::*;
#[test]
fn md5_rfc1321() {
for (input, want) in [
("", "d41d8cd98f00b204e9800998ecf8427e"),
("a", "0cc175b9c0f1b6a831c399e269772661"),
("abc", "900150983cd24fb0d6963f7d28e17f72"),
("message digest", "f96b697d7cb7938d525a2f31aaf161d0"),
(
"12345678901234567890123456789012345678901234567890123456789012345678901234567890",
"57edf4a22be3c955ac49da2e2107b67a",
),
] {
assert_eq!(hex::encode(md5(input.as_bytes())), want, "{input}");
}
}
#[test]
fn plafond_de_sortie() {
let s = "é".repeat(MAX_OUTPUT);
assert_eq!(cap_output(&s), s);
let long = format!("{s}x");
let out = cap_output(&long);
assert!(out.starts_with(&s));
assert!(out.ends_with("ou cible avec web_grep.]"));
assert!(!out.contains('x'));
}
#[test]
fn cles_du_cache() {
let base = FetchOpts {
dismiss_popups: true,
..Default::default()
};
assert_eq!(cache_key_for("u", &base), "u");
let o = FetchOpts {
dismiss_popups: false,
..Default::default()
};
// Valeurs tirées de cacheKeyFor du Go.
assert_eq!(cache_key_for("u", &o), "u#77fc370a");
let o = FetchOpts {
dismiss_popups: true,
actions: vec!["a < b && c".into()],
..Default::default()
};
assert_eq!(cache_key_for("u", &o), "u#d1533b44");
}
#[test]
fn urls() {
for (raw, want) in [
(
"https://github.com/a/b",
"https://raw.githubusercontent.com/a/b/HEAD/README.md",
),
(
"//github.com/a/b/",
"https://raw.githubusercontent.com/a/b/HEAD/README.md",
),
(
"https://github.com/a%20x/b?tab=1#f",
"https://raw.githubusercontent.com/a x/b/HEAD/README.md",
),
("https://GitHub.com/a/b", "https://GitHub.com/a/b"),
("https://github.com:443/a/b", "https://github.com:443/a/b"),
("https://github.com/a", "https://github.com/a"),
("https://github.com/a/b/c", "https://github.com/a/b/c"),
("https://github.com/%zz/b", "https://github.com/%zz/b"),
("github.com/a/b", "github.com/a/b"),
(":github.com/a/b", ":github.com/a/b"),
] {
assert_eq!(normalize_crawl_url(raw), want, "{raw}");
}
for (raw, want) in [
(
"//duckduckgo.com/l/?uddg=https%3A%2F%2Fa.example%2F%3Fx%3D1%26y%3D2&rut=z",
"https://a.example/?x=1&y=2",
),
("https://b.example/", "https://b.example/"),
(
"//duckduckgo.com/l/?uddg=%ZZ",
"https://duckduckgo.com/l/?uddg=%ZZ",
),
("//duckduckgo.com/l/?x;y=1&uddg=a%2Bb+c", "a b c"),
("/l/?uddg=", "/l/?uddg="),
] {
assert_eq!(decode_uddg(raw), want, "{raw}");
}
}
#[test]
fn entites() {
assert_eq!(
decode_html_entities(
"a &amp;lt; &#233;t&#233; &#0; &#4294967361; &#99999999999999999999; &quot;"
),
"a &lt; été &#0; A &#99999999999999999999; \""
);
}
#[test]
fn nombres_du_go() {
assert_eq!(go_int(3.9), 3);
assert_eq!(go_int(-3.9), -3);
if cfg!(target_arch = "x86_64") {
assert_eq!(go_int(1e300), i64::MIN);
assert_eq!(go_int(f64::NAN), i64::MIN);
}
assert_eq!(format_bytes(1023), "1023 B");
assert_eq!(format_bytes(1536), "1.5 KB");
assert_eq!(format_bytes(3 * 1024 * 1024 + 1), "3.00 MB");
assert_eq!(tail_runes("abcdé", 2), "dé");
assert_eq!(tail_runes("ab", 5), "ab");
}
#[test]
fn erreurs_de_regex() {
for (expr, want) in [
("(?i)(", "missing closing ): `(?i)(`"),
("(?i)a)", "unexpected ): `(?i)a)`"),
("(?i)[abc", "missing closing ]: `[abc`"),
("(?i)[[:alpha:]", "missing closing ]: `[[:alpha:]`"),
("(?i)*x", "missing argument to repetition operator: `*`"),
("(?i)a|+", "missing argument to repetition operator: `+`"),
("(?i)a**", "invalid nested repetition operator: `**`"),
("(?i)a*??", "invalid nested repetition operator: `*??`"),
("(?i)\\q", "invalid escape sequence: `\\q`"),
("(?i)\\1", "invalid escape sequence: `\\1`"),
("(?i)a\\", "trailing backslash at end of expression: ``"),
] {
assert_eq!(
go_regexp_error(expr),
format!("error parsing regexp: {want}"),
"{expr}"
);
}
}
#[test]
fn reponses_de_crawl4ai() {
let md = |raw: &str, pr: bool| crawl_markdown(raw.as_bytes(), pr);
assert_eq!(
md(
r#"{"results":[{"success":true,"markdown":{"raw_markdown":"r","fit_markdown":"f"}}]}"#,
true
),
Ok("r".into())
);
assert_eq!(
md(
r#"{"results":[{"success":true,"markdown":{"raw_markdown":"","fit_markdown":"f"}}]}"#,
true
),
Ok("f".into())
);
assert_eq!(
md(
r#"{"results":[{"success":true,"markdown":{"raw_markdown":"r"}}]}"#,
false
),
Ok("r".into())
);
assert_eq!(
md(r#"{"Results":[{"Success":true,"Markdown":"x"}]}"#, false),
Ok("x".into())
);
assert_eq!(md("null", false), Err("Crawl4AI : échec du crawl".into()));
assert_eq!(md("5", false), Err("Crawl4AI : réponse vide".into()));
assert_eq!(
md(r#"{"results":[{"success":"x"}],"success":"y"}"#, false),
Err("Crawl4AI : échec du crawl".into())
);
assert_eq!(
md(r#"{"results":[null,{"success":true}]}"#, false),
Err("Crawl4AI : échec du crawl".into())
);
assert_eq!(
md(r#"{"results":[{"success":true,"markdown":null}]}"#, false),
Ok(String::new())
);
}
#[test]
fn lignes_normalisees() {
assert_eq!(
normalize_lines("\n\n a \r\nb \n\n\n\nb\nb\nc\t\n\n"),
[" a", "b", "", "b", "c"]
);
assert!(normalize_lines(" \n\t\n").is_empty());
}