From 4a357b335f74ae64714800527ac3e95b7057a3ce Mon Sep 17 00:00:00 2001 From: "Danilo M." Date: Sat, 26 Sep 2026 08:34:55 +0200 Subject: Fix mail counts and finance ranges found testing live - mail printed threads and the model called them emails; it now reports messages and threads. - finance spending takes a since date (default the first of the month) instead of a day count, and an optional asset account. The per-account listing needs an end date or the API rejects it; an unknown account name gets the user's accounts back. - Tool errors are capped at 2000 characters like normal output. Co-Authored-By: Claude Opus 5.5 --- AGENTS.md | 31 +++++++++++++++++++++++++++++++ 1 file changed, 31 insertions(+) (limited to 'AGENTS.md') diff --git a/AGENTS.md b/AGENTS.md index 51a46d3..bfc29f4 100644 --- a/AGENTS.md +++ b/AGENTS.md @@ -331,6 +331,37 @@ distance, a person, next appointment twice). - `agenda.py` matched khal's listing over 7 and 60 days on the real vdir, runs in ~60 ms. +### More tools, live test (2026-09-26) + +`calc` (qalc), `mail` (mailctl search), `playing` (playerctl/mpc), +`finance` (firefly list commands). Every command is fixed in its tool +file: the model fills parameters but cannot reach `mailctl tag` or +`firefly tx add`. + +- qalc reads "17.5% of 240" as a modulo. With the hint "percent as a + factor: 240 * 17.5%" the model sent exactly that and got 42. +- "Days until Christmas": the model built `days(; 2026-12-25)` + from the date in the system prompt, right in one call (1.9 s). +- mail said "47 unread emails" for 47 *threads* (58 messages). The tool + now prints "N messages in M threads". +- finance "this month" used the 30-day default and the model called it + the month's total. Spending now takes a `since` date, empty = first of + the month. +- "Spent on account X" had no account filter: the model fetched balances + plus total spending and honestly said it could not split them. Added + `account`; firefly's per-account listing needs `--until` or the API + answers 422. An unknown name gets the user's asset accounts back (not + firefly's own error, which lists every expense account); tool errors + are now capped at 2000 characters like normal output. +- The model passed a lowercase account name; matching is + case-insensitive, firefly accepted it. +- First sentence 1.9-3.6 s with local tools, ~5-6.6 s with firefly + (two API calls when filtering by account). +- STT misheard "mail" as "snail"; the model answered about a snail, + harmless for read-only tools. Phase 2 (modifying tools) needs + confirmation before acting for exactly this reason. +- These tools speak personal data (mail subjects, amounts) out loud. + ### TTS (2026-09-25) `piper-tts` on CPU (onnxruntime), ~5 s sentence per voice: -- cgit v1.2.3