From 460cd4456790092ea42a8848bb0ea517a1174fca Mon Sep 17 00:00:00 2001 From: "Danilo M." Date: Fri, 16 Oct 2026 13:31:42 +0200 Subject: Final review: startup probe, error messages, refusal exit, docs A startup probe that failed with anything but OSError (llama-server not in router mode, an empty model list) crashed the daemon at boot, against its own comment; it is caught like the per-request failures now. llama-server's reasons reach the client: an HTTP error carries its body and an error chunk mid-stream its message, where the client used to see 'choices'. A daemon that refuses to start, or fails before serving, exits 1 instead of 0. assistant.sh no longer mistakes a --test run for the daemon. AGENTS.md said every section from Hardware to Kokoro was voice-only, which included findings the daemon still applies; it now names the voice ones. It also claimed a stopped reply emits nothing after done, while a late ctx can. The spec lists every error message, the example config no longer shows a non-default temperature as the default, and the updates tool stops telling the model not to read lists aloud. The spec is marked implemented. Co-Authored-By: Claude Opus 5.5 --- examples/tools/updates.py | 4 ++-- examples/tools/updates.toml | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) (limited to 'examples/tools') diff --git a/examples/tools/updates.py b/examples/tools/updates.py index 57cc61a..6308bef 100755 --- a/examples/tools/updates.py +++ b/examples/tools/updates.py @@ -102,7 +102,7 @@ def by_action(todo): def summary(newest, applied, n, todo): - # Spoken: counts only, names just for the short lists. + # Counts only, names just for the short lists. if not todo: return f"Up to date with the ChangeLog of {newest}." by = by_action(todo) @@ -117,7 +117,7 @@ def summary(newest, applied, n, todo): for a, label in (("Added", "New"), ("Removed", "Removed")): if by[a]: out.append(f"{label}: " + ", ".join(by[a]) + ".") - out.append("Offer a written recap (report = true); do not read package lists aloud.") + out.append("Offer a written recap (report = true); do not list every package.") return "\n".join(out) diff --git a/examples/tools/updates.toml b/examples/tools/updates.toml index 86eac1d..3b05817 100644 --- a/examples/tools/updates.toml +++ b/examples/tools/updates.toml @@ -1,4 +1,4 @@ -description = "Pending Slackware -current upgrades: ChangeLog entries not yet installed. Summary gives counts (upgraded, rebuilt, new, removed, security fixes) and a pending kernel version bump; say the counts, never read package lists aloud. Can instead open a written markdown recap in the user's editor, or give one package's ChangeLog notes. Read-only." +description = "Pending Slackware -current upgrades: ChangeLog entries not yet installed. Summary gives counts (upgraded, rebuilt, new, removed, security fixes) and a pending kernel version bump; give the counts, not whole package lists. Can instead open a written markdown recap in the user's editor, or give one package's ChangeLog notes. Read-only." command = ["updates.py", "{package}", "{report}"] timeout = 10 -- cgit v1.2.3