Deep scan with vulture, pyflakes, and manual cross-referencing identified: - 41 dead functions/methods (zero callers in production) - 7 production-dead functions (only test callers, tests deleted) - 5 dead constants/variables - ~35 unused imports across agent/, hermes_cli/, tools/, gateway/ Categories of dead code removed: - Refactoring leftovers: _set_default_model, _setup_copilot_reasoning_selection, rebuild_lookups, clear_session_context, get_logs_dir, clear_session - Unused API surface: search_models_dev, get_pricing, skills_categories, get_read_files_summary, clear_read_tracker, menu_labels, get_spinner_list - Dead compatibility wrappers: schedule_cronjob, list_cronjobs, remove_cronjob - Stale debug helpers: get_debug_session_info copies in 4 tool files (centralized version in debug_helpers.py already exists) - Dead gateway methods: send_emote, send_notice (matrix), send_reaction (bluebubbles), _normalize_inbound_text (feishu), fetch_room_history (matrix), _start_typing_indicator (signal), parse_feishu_post_content - Dead constants: NOUS_API_BASE_URL, SKILLS_TOOL_DESCRIPTION, FILE_TOOLS, VALID_ASPECT_RATIOS, MEMORY_DIR - Unused UI code: _interactive_provider_selection, _interactive_model_selection (superseded by prompt_toolkit picker) Test suite verified: 609 tests covering affected files all pass. Tests for removed functions deleted. Tests using removed utilities (clear_read_tracker, MEMORY_DIR) updated to use internal APIs directly.
178 lines
7.0 KiB
Python
178 lines
7.0 KiB
Python
"""
|
|
Skills configuration for Hermes Agent.
|
|
`hermes skills` enters this module.
|
|
|
|
Toggle individual skills or categories on/off, globally or per-platform.
|
|
Config stored in ~/.hermes/config.yaml under:
|
|
|
|
skills:
|
|
disabled: [skill-a, skill-b] # global disabled list
|
|
platform_disabled: # per-platform overrides
|
|
telegram: [skill-c]
|
|
cli: []
|
|
"""
|
|
from typing import List, Optional, Set
|
|
|
|
from hermes_cli.config import load_config, save_config
|
|
from hermes_cli.colors import Colors, color
|
|
from hermes_cli.platforms import PLATFORMS as _PLATFORMS
|
|
|
|
# Backward-compatible view: {key: label_string} so existing code that
|
|
# iterates ``PLATFORMS.items()`` or calls ``PLATFORMS.get(key)`` keeps
|
|
# working without changes to every call site.
|
|
PLATFORMS = {k: info.label for k, info in _PLATFORMS.items() if k != "api_server"}
|
|
|
|
# ─── Config Helpers ───────────────────────────────────────────────────────────
|
|
|
|
def get_disabled_skills(config: dict, platform: Optional[str] = None) -> Set[str]:
|
|
"""Return disabled skill names. Platform-specific list falls back to global."""
|
|
skills_cfg = config.get("skills", {})
|
|
global_disabled = set(skills_cfg.get("disabled", []))
|
|
if platform is None:
|
|
return global_disabled
|
|
platform_disabled = skills_cfg.get("platform_disabled", {}).get(platform)
|
|
if platform_disabled is None:
|
|
return global_disabled
|
|
return set(platform_disabled)
|
|
|
|
|
|
def save_disabled_skills(config: dict, disabled: Set[str], platform: Optional[str] = None):
|
|
"""Persist disabled skill names to config."""
|
|
config.setdefault("skills", {})
|
|
if platform is None:
|
|
config["skills"]["disabled"] = sorted(disabled)
|
|
else:
|
|
config["skills"].setdefault("platform_disabled", {})
|
|
config["skills"]["platform_disabled"][platform] = sorted(disabled)
|
|
save_config(config)
|
|
|
|
|
|
# ─── Skill Discovery ─────────────────────────────────────────────────────────
|
|
|
|
def _list_all_skills() -> List[dict]:
|
|
"""Return all installed skills (ignoring disabled state)."""
|
|
try:
|
|
from tools.skills_tool import _find_all_skills
|
|
return _find_all_skills(skip_disabled=True)
|
|
except Exception:
|
|
return []
|
|
|
|
|
|
def _get_categories(skills: List[dict]) -> List[str]:
|
|
"""Return sorted unique category names (None -> 'uncategorized')."""
|
|
return sorted({s["category"] or "uncategorized" for s in skills})
|
|
|
|
|
|
# ─── Platform Selection ──────────────────────────────────────────────────────
|
|
|
|
def _select_platform() -> Optional[str]:
|
|
"""Ask user which platform to configure, or global."""
|
|
options = [("global", "All platforms (global default)")] + list(PLATFORMS.items())
|
|
print()
|
|
print(color(" Configure skills for:", Colors.BOLD))
|
|
for i, (key, label) in enumerate(options, 1):
|
|
print(f" {i}. {label}")
|
|
print()
|
|
try:
|
|
raw = input(color(" Select [1]: ", Colors.YELLOW)).strip()
|
|
except (KeyboardInterrupt, EOFError):
|
|
return None
|
|
if not raw:
|
|
return None # global
|
|
try:
|
|
idx = int(raw) - 1
|
|
if 0 <= idx < len(options):
|
|
key = options[idx][0]
|
|
return None if key == "global" else key
|
|
except ValueError:
|
|
pass
|
|
return None
|
|
|
|
|
|
# ─── Category Toggle ─────────────────────────────────────────────────────────
|
|
|
|
def _toggle_by_category(skills: List[dict], disabled: Set[str]) -> Set[str]:
|
|
"""Toggle all skills in a category at once."""
|
|
from hermes_cli.curses_ui import curses_checklist
|
|
|
|
categories = _get_categories(skills)
|
|
cat_labels = []
|
|
# A category is "enabled" (checked) when NOT all its skills are disabled
|
|
pre_selected = set()
|
|
for i, cat in enumerate(categories):
|
|
cat_skills = [s["name"] for s in skills if (s["category"] or "uncategorized") == cat]
|
|
cat_labels.append(f"{cat} ({len(cat_skills)} skills)")
|
|
if not all(s in disabled for s in cat_skills):
|
|
pre_selected.add(i)
|
|
|
|
chosen = curses_checklist(
|
|
"Categories — toggle entire categories",
|
|
cat_labels, pre_selected, cancel_returns=pre_selected,
|
|
)
|
|
|
|
new_disabled = set(disabled)
|
|
for i, cat in enumerate(categories):
|
|
cat_skills = {s["name"] for s in skills if (s["category"] or "uncategorized") == cat}
|
|
if i in chosen:
|
|
new_disabled -= cat_skills # category enabled → remove from disabled
|
|
else:
|
|
new_disabled |= cat_skills # category disabled → add to disabled
|
|
return new_disabled
|
|
|
|
|
|
# ─── Entry Point ──────────────────────────────────────────────────────────────
|
|
|
|
def skills_command(args=None):
|
|
"""Entry point for `hermes skills`."""
|
|
from hermes_cli.curses_ui import curses_checklist
|
|
|
|
config = load_config()
|
|
skills = _list_all_skills()
|
|
|
|
if not skills:
|
|
print(color(" No skills installed.", Colors.DIM))
|
|
return
|
|
|
|
# Step 1: Select platform
|
|
platform = _select_platform()
|
|
platform_label = PLATFORMS.get(platform, "All platforms") if platform else "All platforms"
|
|
|
|
# Step 2: Select mode — individual or by category
|
|
print()
|
|
print(color(f" Configure for: {platform_label}", Colors.DIM))
|
|
print()
|
|
print(" 1. Toggle individual skills")
|
|
print(" 2. Toggle by category")
|
|
print()
|
|
try:
|
|
mode = input(color(" Select [1]: ", Colors.YELLOW)).strip() or "1"
|
|
except (KeyboardInterrupt, EOFError):
|
|
return
|
|
|
|
disabled = get_disabled_skills(config, platform)
|
|
|
|
if mode == "2":
|
|
new_disabled = _toggle_by_category(skills, disabled)
|
|
else:
|
|
# Build labels and map indices → skill names
|
|
labels = [
|
|
f"{s['name']} ({s['category'] or 'uncategorized'}) — {s['description'][:55]}"
|
|
for s in skills
|
|
]
|
|
# "selected" = enabled (not disabled) — matches the [✓] convention
|
|
pre_selected = {i for i, s in enumerate(skills) if s["name"] not in disabled}
|
|
chosen = curses_checklist(
|
|
f"Skills for {platform_label}",
|
|
labels, pre_selected, cancel_returns=pre_selected,
|
|
)
|
|
# Anything NOT chosen is disabled
|
|
new_disabled = {skills[i]["name"] for i in range(len(skills)) if i not in chosen}
|
|
|
|
if new_disabled == disabled:
|
|
print(color(" No changes.", Colors.DIM))
|
|
return
|
|
|
|
save_disabled_skills(config, new_disabled, platform)
|
|
enabled_count = len(skills) - len(new_disabled)
|
|
print(color(f"✓ Saved: {enabled_count} enabled, {len(new_disabled)} disabled ({platform_label}).", Colors.GREEN))
|