From 56aec13ff1b5832bd2870612dfca103a2f004ac2 Mon Sep 17 00:00:00 2001
From: Violin <violin@local>
Date: Sun, 12 Jul 2026 16:47:32 +0100
Subject: [PATCH] Remove stale history tool references

---
 plugins/violin_guard/schemas.py | 4 ++--
 skills/pentest/SKILL.md         | 2 +-
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/plugins/violin_guard/schemas.py b/plugins/violin_guard/schemas.py
index 7f1da9a..3851164 100644
--- a/plugins/violin_guard/schemas.py
+++ b/plugins/violin_guard/schemas.py
@@ -277,7 +277,7 @@ TARGET_SCHEMA = {
 
 STATUS_SCHEMA = {
     "name": "violin_status",
-    "description": "One-shot engagement health read: bootstrap completeness, skill-load freshness, pending doc-sync, heartbeat-pending, sync credit remaining, and command/message counts. Consolidates violin_check_bootstrap + violin_check_skill_loaded + violin_sync_done + violin_message_tick into a single read-only call so the agent can poll engagement state without burning four tool calls. Mutates no state.",
+    "description": "One-shot engagement health read: bootstrap completeness, skill-load freshness, pending doc-sync, heartbeat-pending, sync credit remaining, and command/message counts. Mutates no state.",
     "parameters": {
         "type": "object",
         "properties": {
@@ -293,4 +293,4 @@ STATUS_SCHEMA = {
         "required": [],
         "additionalProperties": False,
     },
-}
\ No newline at end of file
+}
diff --git a/skills/pentest/SKILL.md b/skills/pentest/SKILL.md
index 1e76a41..2334f2c 100644
--- a/skills/pentest/SKILL.md
+++ b/skills/pentest/SKILL.md
@@ -121,7 +121,7 @@ The phase workflow is mandatory for the entire session, including long, compress
    python $HOME/.hermes/profiles/violin/scripts/violin_guard.py record-ptt --eng-dir "$ENG_DIR" --id PT-XXX --status "[~]" --note "starting task"
    ```
    The guard hard-blocks target execution when there is no unambiguous active task. The executor never changes this row or its `*Last updated*` timestamp. After each bounded batch, review the results and explicitly call `violin_record_ptt` with `[~]`, `[x]`, `[!]`, or `[-]` plus a truthful result summary, then call `violin_sync_done`.
-4. **Command history is executor-owned** — target commands executed through `violin_exec` are appended automatically after the process exits. The model must not call `violin_record_history` for normal execution. Use it only to repair/import a command run outside the enforced executor:
+4. **Command history is executor-owned** — target commands executed through `violin_exec` are appended automatically after the process exits. The model must not call a history tool for normal execution. Use the standalone CLI `record-history` only to repair/import a command run outside the enforced executor:
    ```bash
    python $HOME/.hermes/profiles/violin/scripts/violin_guard.py record-history --eng-dir "$ENG_DIR" --command "<cmd>" --exit-code <N> --phase <PHASE>
    ```