bone-agent 1.3.3 → 1.4.0

package/src/ui/commands.py

@@ -8,7 +8,7 @@ from typing import Optional
 from llm import config
 
 from core.config_manager import ConfigManager as ConfigManagerClass
-from ui.displays import show_help_table, show_cron_help_table
+from ui.displays import show_help_table, show_cron_help_table, show_skills_help_table
 from ui.banner import display_startup_banner
 from core.agentic import SubAgentPanel
 from ui.setting_selector import SettingSelector, SettingCategory, SettingOption
@@ -591,12 +591,15 @@ def _handle_clear(chat_manager, console, debug_mode_container, args, cron_schedu
     conv_cache_creation = chat_manager.token_tracker.conv_cache_creation_tokens
     if conv_cache_read > 0 or conv_cache_creation > 0:
         total_cached = conv_cache_read + conv_cache_creation
-        cache_hit_pct = (
+        cache_activity_read_pct = (
             conv_cache_read / total_cached * 100
         ) if total_cached > 0 else 0
+        cache_coverage_pct = (
+            conv_cache_read / conv_in * 100
+        ) if conv_in > 0 else 0
         console.print(f"  Cache read:   {conv_cache_read:,} tokens")
         console.print(f"  Cache write:  {conv_cache_creation:,} tokens")
-        console.print(f"  ({cache_hit_pct:.0f}% cache hit rate)")
+        console.print(f"  ({cache_coverage_pct:.0f}% input cached, {cache_activity_read_pct:.0f}% cache reads)")
 
     # Display cost — combined actual + estimated, with config-based fallback
     tracker_conv = chat_manager.token_tracker
@@ -665,11 +668,12 @@ def _open_provider_editor(chat_manager, console, provider):
             min_val=0.0, step=0.01,
         ))
 
-    category = SettingCategory(title=f"{provider.capitalize()} Settings", settings=settings)
+    provider_label = config.get_provider_display_name(provider)
+    category = SettingCategory(title=f"{provider_label} Settings", settings=settings)
 
     selector = SettingSelector(
         categories=[category],
-        title=f"Configure {provider.capitalize()}",
+        title=f"Configure {provider_label}",
     )
 
     changes = selector.run()
@@ -742,7 +746,8 @@ def _handle_provider(chat_manager, console, debug_mode_container, args, cron_sch
         # Validate provider name
         if provider not in config.get_providers():
             console.print(f"[red]Error: Unknown provider '{provider}'[/red]")
-            console.print(f"[dim]Available providers: {', '.join(config.get_providers())}[/dim]")
+            available = ', '.join(config.get_provider_display_name(prov) for prov in config.get_providers())
+            console.print(f"[dim]Available providers: {available}[/dim]")
             return CommandResult(status="handled")
 
         # Switch directly to the named provider
@@ -756,7 +761,7 @@ def _handle_provider(chat_manager, console, debug_mode_container, args, cron_sch
 
         cfg = config.get_provider_config(provider)
         model = cfg.get('model') or cfg.get('api_model') or ''
-        label = f"{provider.capitalize()}"
+        label = config.get_provider_display_name(provider)
         if model:
             label += f" ({model})"
         console.print(f"[green]Switched to {label}[/green]")
@@ -772,7 +777,7 @@ def _handle_provider(chat_manager, console, debug_mode_container, args, cron_sch
         for prov in config.get_providers():
             cfg = config.get_provider_config(prov)
             model = cfg.get('model') or cfg.get('api_model') or ''
-            entry = {"value": prov, "text": prov.capitalize()}
+            entry = {"value": prov, "text": config.get_provider_display_name(prov)}
             if model:
                 entry["description"] = model[:40]
             provider_options.append(entry)
@@ -1092,18 +1097,38 @@ def _handle_usage(chat_manager, console, debug_mode_container, args, cron_schedu
     console.print(f"  Output tokens: {tracker.total_completion_tokens:,}")
     console.print(f"  Total tokens:  {tracker.total_tokens:,}")
 
-    # Display cache token breakdown (if any cache tokens were recorded)
-    has_cache = tracker.total_cache_read_tokens > 0 or tracker.total_cache_creation_tokens > 0
-    if has_cache:
+    # Display cache token breakdown when cache tokens were recorded.
+    # Codex can report an explicit 0 cached_tokens value, which is still useful
+    # confirmation that prompt-cache usage data is flowing through.
+    show_cache = (
+        tracker.total_cache_read_tokens > 0
+        or tracker.total_cache_creation_tokens > 0
+        or current_provider == "codex"
+    )
+    if show_cache:
         total_cached = tracker.total_cache_read_tokens + tracker.total_cache_creation_tokens
-        cache_hit_pct = (
+        cache_activity_read_pct = (
             tracker.total_cache_read_tokens
             / total_cached * 100
         ) if total_cached > 0 else 0
+        cache_coverage_pct = (
+            tracker.total_cache_read_tokens
+            / tracker.total_prompt_tokens * 100
+        ) if tracker.total_prompt_tokens > 0 else 0
         console.print()
-        console.print(f"[#5F9EA0]Input Cache ({cache_hit_pct:.0f}% hit rate):[/#5F9EA0]")
+        console.print(
+            f"[#5F9EA0]Input Cache ({cache_coverage_pct:.0f}% input cached, "
+            f"{cache_activity_read_pct:.0f}% cache reads):[/#5F9EA0]"
+        )
         console.print(f"  Cache read:   {tracker.total_cache_read_tokens:,} tokens")
         console.print(f"  Cache write:  {tracker.total_cache_creation_tokens:,} tokens")
+        if current_provider == "codex" and total_cached == 0:
+            if tracker.last_cache_metrics_reported is False:
+                keys = ", ".join(tracker.last_usage_keys) if tracker.last_usage_keys else "none"
+                console.print("  [dim]Codex did not report any cache-token fields in the last usage payload.[/dim]")
+                console.print(f"  [dim]Last usage keys: {keys}[/dim]")
+            elif tracker.last_cache_metrics_reported is None:
+                console.print("  [dim]No usage payload has been recorded yet for this Codex session.[/dim]")
     console.print()
     
 
@@ -2184,8 +2209,8 @@ def _handle_obsidian(chat_manager, console, debug_mode_container, args, cron_sch
     return CommandResult(status="handled")
 
 
-def _persist_disabled_tools(console):
-    """Persist current disabled_tools to config file.
+def _persist_tool_visibility(console):
+    """Persist tool and skill visibility state to config file.
 
     Returns True on success, False on failure.
     """
@@ -2194,6 +2219,7 @@ def _persist_disabled_tools(console):
         if "TOOL_SETTINGS" not in cfg_data:
             cfg_data["TOOL_SETTINGS"] = {}
         cfg_data["TOOL_SETTINGS"]["disabled_tools"] = list(tool_settings.disabled_tools)
+        cfg_data["TOOL_SETTINGS"]["hidden_skills"] = list(tool_settings.hidden_skills)
         config_manager.save(cfg_data)
         return True
     except Exception as e:
@@ -2202,18 +2228,23 @@ def _persist_disabled_tools(console):
 
 
 def _handle_tools(chat_manager, console, debug_mode_container, args, cron_scheduler=None):
-    """Handle /tools command — toggle individual tools or groups on/off.
+    """Handle /tools command — manage tool availability and skill discovery visibility.
 
-    No args: Launch interactive SettingSelector with tools grouped by category.
+    No args: Launch interactive SettingSelector with tools/plugins grouped by category
+    and a skills section for discovery visibility.
     Subcommands:
-      list — show all tools with group labels and status
-      enable <name> — enable a single tool
-      disable <name> — disable a single tool
-      enable-group <key> — enable all tools in a group (e.g. file_ops, search, shell)
+      list — show all tools, plugins, and skills with status
+      enable <name> — enable a core tool or plugin
+      disable <name> — disable a core tool or plugin
+      show-skill <name> — make a skill visible in discovery surfaces
+      hide-skill <name> — hide a skill from discovery surfaces
+      enable-group <key> — enable all tools in a group (e.g. file_ops, task_mgmt)
       disable-group <key> — disable all tools in a group
     """
+    from core.skills import iter_skill_summaries, validate_skill_name
     from ui.setting_selector import SettingOption, SettingCategory, SettingSelector
     from tools.helpers.base import ToolRegistry, TOOL_GROUPS
+    from tools.helpers.plugin_manifest import plugin_manifest
 
     # Text subcommands
     if args:
@@ -2221,17 +2252,27 @@ def _handle_tools(chat_manager, console, debug_mode_container, args, cron_schedu
 
         if args_clean.lower() in ("list", "status"):
             all_tools = sorted(ToolRegistry._tools.values(), key=lambda t: t.name)
+            plugin_defs = sorted(plugin_manifest.get_all(), key=lambda t: t.name)
+            skills = sorted(iter_skill_summaries(), key=lambda s: s.name)
             disabled = ToolRegistry.get_disabled()
-            console.print(f"[bold #5F9EA0]Tools: {len(all_tools) - len(disabled)} enabled, {len(disabled)} disabled[/bold #5F9EA0]")
+            hidden_skills = set(tool_settings.hidden_skills)
+            disabled_tools = {t.name for t in all_tools if t.name in disabled}
+            console.print(
+                f"[bold #5F9EA0]Tools: {len(all_tools) - len(disabled_tools)} enabled, {len(disabled_tools)} disabled[/bold #5F9EA0]"
+            )
+            console.print(
+                f"[bold #5F9EA0]User plugins: {sum(1 for p in plugin_defs if p.name not in disabled)} enabled, {sum(1 for p in plugin_defs if p.name in disabled)} disabled[/bold #5F9EA0]"
+            )
+            console.print(
+                f"[bold #5F9EA0]Skills: {len(skills) - len(hidden_skills)} visible, {len(hidden_skills)} hidden[/bold #5F9EA0]"
+            )
             console.print()
 
-            # Build reverse lookup: tool_name -> group_label
             tool_to_group = {}
             for gkey, gdef in TOOL_GROUPS.items():
                 for tname in gdef["tools"]:
                     tool_to_group.setdefault(tname, []).append(gdef["label"])
 
-            # Group tools for display
             current_group = None
             for t in all_tools:
                 groups = tool_to_group.get(t.name, [])
@@ -2243,6 +2284,18 @@ def _handle_tools(chat_manager, console, debug_mode_container, args, cron_schedu
                 status = "[red]off[/red]" if is_off else "[green]on[/green] "
                 console.print(f"    {status} {t.name}")
 
+            console.print()
+            console.print("  [bold]User plugins[/bold]")
+            for plugin in plugin_defs:
+                status = "[red]off[/red]" if plugin.name in disabled else "[green]on[/green] "
+                console.print(f"    {status} {plugin.name}")
+
+            console.print()
+            console.print("  [bold]Skills[/bold]")
+            for skill in skills:
+                status = "[red]hidden[/red]" if skill.name in hidden_skills else "[green]visible[/green]"
+                console.print(f"    {status} {skill.name}")
+
             console.print()
             console.print("[dim]Groups:[/dim] " + ", ".join(
                 f"[bold]{k}[/bold] ({v['label']})" for k, v in TOOL_GROUPS.items()
@@ -2281,17 +2334,17 @@ def _handle_tools(chat_manager, console, debug_mode_container, args, cron_schedu
 
                 # Sync and persist
                 tool_settings.disabled_tools = sorted(ToolRegistry.get_disabled())
-                _persist_disabled_tools(console)
+                _persist_tool_visibility(console)
                 console.print()
                 return CommandResult(status="handled")
 
-            # Single tool operations
+            # Single tool/plugin operations
             if action in ("enable", "disable"):
-                # Match case-insensitively against registered tools
                 all_registered_lower = {t.name.lower(): t.name for t in ToolRegistry._tools.values()}
+                all_registered_lower.update({t.name.lower(): t.name for t in plugin_manifest.get_all()})
                 matched = all_registered_lower.get(target.lower())
                 if not matched:
-                    console.print(f"[red]Unknown tool: {target}[/red]")
+                    console.print(f"[red]Unknown tool or plugin: {target}[/red]")
                     console.print(f"[dim]Run [bold #5F9EA0]/tools list[/bold #5F9EA0] to see all tools.[/dim]")
                     return CommandResult(status="handled")
 
@@ -2305,17 +2358,44 @@ def _handle_tools(chat_manager, console, debug_mode_container, args, cron_schedu
                         tool_settings.disabled_tools.append(matched)
                     console.print(f"[yellow]Disabled: {matched}[/yellow]")
 
-                _persist_disabled_tools(console)
+                _persist_tool_visibility(console)
+                console.print()
+                return CommandResult(status="handled")
+
+            if action in ("show-skill", "hide-skill"):
+                try:
+                    skill_name = validate_skill_name(target)
+                except Exception as e:
+                    console.print(f"[red]{e}[/red]")
+                    return CommandResult(status="handled")
+
+                known_skills = {skill.name for skill in iter_skill_summaries()}
+                if skill_name not in known_skills:
+                    console.print(f"[red]Unknown skill: {skill_name}[/red]")
+                    return CommandResult(status="handled")
+
+                if action == "show-skill":
+                    tool_settings.hidden_skills = [n for n in tool_settings.hidden_skills if n != skill_name]
+                    console.print(f"[green]Skill visible in discovery: {skill_name}[/green]")
+                else:
+                    if skill_name not in tool_settings.hidden_skills:
+                        tool_settings.hidden_skills.append(skill_name)
+                    console.print(f"[yellow]Skill hidden from discovery: {skill_name}[/yellow]")
+
+                tool_settings.hidden_skills = sorted(set(tool_settings.hidden_skills))
+                _persist_tool_visibility(console)
                 console.print()
                 return CommandResult(status="handled")
 
         console.print(f"[red]Unknown subcommand: {args}[/red]")
-        console.print("Usage: [bold #5F9EA0]/tools[/bold #5F9EA0] [list | enable <name> | disable <name> | enable-group <key> | disable-group <key>]")
+        console.print("Usage: [bold #5F9EA0]/tools[/bold #5F9EA0] [list | enable <name> | disable <name> | show-skill <name> | hide-skill <name> | enable-group <key> | disable-group <key>]")
         return CommandResult(status="handled")
 
     # No args — interactive toggle UI, organized by groups
     all_tools_map = {t.name: t for t in ToolRegistry._tools.values()}
+    plugin_tools = {t.name: t for t in plugin_manifest.get_all()}
     disabled = ToolRegistry.get_disabled()
+    hidden_skills = set(tool_settings.hidden_skills)
 
     categories = []
     for gkey, gdef in TOOL_GROUPS.items():
@@ -2355,10 +2435,36 @@ def _handle_tools(chat_manager, console, debug_mode_container, args, cron_schedu
                 input_type="boolean",
                 on_text="ON",
                 off_text="OFF",
-                description=f"Modes: {modes}",
             ))
         categories.append(SettingCategory(title="Other", settings=other_options))
 
+    plugin_options = []
+    for name in sorted(plugin_tools):
+        is_off = name in disabled
+        plugin_options.append(SettingOption(
+            key=name,
+            text=name,
+            value=not is_off,
+            input_type="boolean",
+            on_text="ON",
+            off_text="OFF",
+        ))
+    if plugin_options:
+        categories.append(SettingCategory(title="User plugins", settings=plugin_options))
+
+    skill_options = []
+    for skill in sorted(iter_skill_summaries(), key=lambda s: s.name):
+        skill_options.append(SettingOption(
+            key=f"skill:{skill.name}",
+            text=skill.name,
+            value=skill.name not in hidden_skills,
+            input_type="boolean",
+            on_text="VISIBLE",
+            off_text="HIDDEN",
+        ))
+    if skill_options:
+        categories.append(SettingCategory(title="Skills", settings=skill_options))
+
     selector = SettingSelector(
         categories=categories,
         title="Tool Settings",
@@ -2377,7 +2483,17 @@ def _handle_tools(chat_manager, console, debug_mode_container, args, cron_schedu
     # Apply changes
     newly_disabled = []
     newly_enabled = []
+    newly_hidden_skills = []
+    newly_visible_skills = []
     for name, enabled in changes.items():
+        if name.startswith("skill:"):
+            skill_name = name.split(":", 1)[1]
+            if enabled and skill_name in hidden_skills:
+                newly_visible_skills.append(skill_name)
+            elif not enabled and skill_name not in hidden_skills:
+                newly_hidden_skills.append(skill_name)
+            continue
+
         if not enabled and name not in disabled:
             ToolRegistry.disable(name)
             newly_disabled.append(name)
@@ -2385,10 +2501,13 @@ def _handle_tools(chat_manager, console, debug_mode_container, args, cron_schedu
             ToolRegistry.enable(name)
             newly_enabled.append(name)
 
-    # Sync tool_settings.disabled_tools to be the full current disabled set
     tool_settings.disabled_tools = sorted(ToolRegistry.get_disabled())
+    next_hidden_skills = set(hidden_skills)
+    next_hidden_skills.update(newly_hidden_skills)
+    next_hidden_skills.difference_update(newly_visible_skills)
+    tool_settings.hidden_skills = sorted(next_hidden_skills)
 
-    _persist_disabled_tools(console)
+    _persist_tool_visibility(console)
 
     # Summary
     change_lines = []
@@ -2396,6 +2515,10 @@ def _handle_tools(chat_manager, console, debug_mode_container, args, cron_schedu
         change_lines.append(f"  [yellow]Disabled:[/yellow] {name}")
     for name in newly_enabled:
         change_lines.append(f"  [green]Enabled:[/green] {name}")
+    for name in newly_hidden_skills:
+        change_lines.append(f"  [yellow]Hidden skill:[/yellow] {name}")
+    for name in newly_visible_skills:
+        change_lines.append(f"  [green]Visible skill:[/green] {name}")
 
     if change_lines:
         total_enabled = len(ToolRegistry.get_all())
@@ -2461,6 +2584,12 @@ def _handle_cd(chat_manager, console, debug_mode_container, args, cron_scheduler
     except Exception:
         pass
 
+    # Rebuild system prompt so project root stays current
+    try:
+        chat_manager.update_system_prompt()
+    except Exception:
+        pass
+
     return CommandResult(status="handled")
 
 
@@ -2515,6 +2644,198 @@ def _handle_prompt(chat_manager, console, debug_mode_container, args, cron_sched
     return CommandResult(status="handled")
 
 
+def _print_skills_usage(console):
+    show_skills_help_table(console)
+
+
+def _skills_list(console, query=None):
+    from datetime import datetime
+    from core.skills import get_skills_dir, list_skills
+
+    skills = list_skills(query=query)
+    if not skills:
+        console.print("[dim]No skills found.[/dim]")
+        console.print(f"[dim]Directory: {get_skills_dir()}[/dim]")
+        console.print("[dim]Create one with: [bold #5F9EA0]/skills add frontend_design[/bold #5F9EA0][/dim]")
+        return
+
+    table = Table(show_header=True, box=box.SIMPLE_HEAD)
+    table.add_column("Skill", no_wrap=True)
+    table.add_column("Preview")
+    table.add_column("Modified", no_wrap=True)
+    for skill in skills:
+        modified = datetime.fromtimestamp(skill.modified).strftime("%Y-%m-%d %H:%M")
+        table.add_row(f"[bold]{skill.name}[/bold]", skill.preview, modified)
+    console.print(table)
+    console.print("[dim]Load with: [bold #5F9EA0]/skills load <name>[/bold #5F9EA0][/dim]")
+
+
+def _open_skill_editor(console, debug_mode_container, initial_content):
+    from utils.editor import open_editor_for_content
+
+    return open_editor_for_content(
+        console,
+        initial_content=initial_content,
+        debug_mode=debug_mode_container["debug"],
+    )
+
+
+def _write_skill_from_editor(console, debug_mode_container, name, initial_content, *, overwrite, verb):
+    from core.skills import write_skill
+
+    success, content = _open_skill_editor(console, debug_mode_container, initial_content)
+    if not success or not content or not content.strip():
+        console.print("[dim]Cancelled.[/dim]")
+        return None
+    path = write_skill(name, content, overwrite=overwrite)
+    console.print(f"[green]{verb} skill '{name}'.[/green] [dim]{path}[/dim]")
+    return path
+
+
+def _handle_skills(chat_manager, console, debug_mode_container, args, cron_sched=None):
+    """Handle /skills command — manage reusable prompt snippets."""
+    from core.skills import (
+        SkillError,
+        activate_skill,
+        get_skills_dir,
+        read_skill,
+        remove_skill,
+        validate_skill_name,
+        write_skill,
+    )
+
+    if not args or not args.strip():
+        _skills_list(console)
+        return CommandResult(status="handled")
+
+    args_clean = args.strip()
+    parts = args_clean.split(maxsplit=2)
+    subcmd = parts[0].lower()
+
+    try:
+        if subcmd in ("help", "-h", "--help"):
+            _print_skills_usage(console)
+            return CommandResult(status="handled")
+
+        if subcmd in ("list", "ls"):
+            list_parts = args_clean.split(maxsplit=1)
+            query = list_parts[1] if len(list_parts) > 1 else None
+            _skills_list(console, query=query)
+            return CommandResult(status="handled")
+
+        if subcmd == "dir":
+            console.print(str(get_skills_dir()))
+            return CommandResult(status="handled")
+
+        if subcmd == "show":
+            if len(parts) < 2:
+                console.print("[red]Usage: /skills show <name>[/red]")
+                return CommandResult(status="handled")
+            name = validate_skill_name(parts[1])
+            content = read_skill(name, strip_heading=False)
+            console.print(Markdown(left_align_headings(content), code_theme=MonokaiDarkBGStyle, justify="left"))
+            return CommandResult(status="handled")
+
+        if subcmd in ("load", "use"):
+            if len(parts) < 2:
+                console.print(f"[red]Usage: /skills {subcmd} <name>[/red]")
+                return CommandResult(status="handled")
+            name = validate_skill_name(parts[1])
+            tokens = activate_skill(chat_manager, name, read_skill(name))
+            console.print(f"[green]Activated skill '{name}' for this chat.[/green] [dim](~{tokens:,} tokens)[/dim]")
+            return CommandResult(status="handled")
+
+        if subcmd in ("remove", "rm", "delete"):
+            if len(parts) < 2:
+                console.print("[red]Usage: /skills remove <name>[/red]")
+                return CommandResult(status="handled")
+            name = validate_skill_name(parts[1])
+            from rich.prompt import Confirm
+            if not Confirm.ask(f"Remove skill '{name}'?", default=False):
+                console.print("[dim]Cancelled.[/dim]")
+                return CommandResult(status="handled")
+            remove_skill(name)
+            console.print(f"[green]Removed skill '{name}'.[/green]")
+            return CommandResult(status="handled")
+
+        if subcmd == "edit":
+            if len(parts) < 2:
+                console.print("[red]Usage: /skills edit <name>[/red]")
+                return CommandResult(status="handled")
+            name = validate_skill_name(parts[1])
+            try:
+                initial_content = read_skill(name, strip_heading=False)
+            except SkillError:
+                initial_content = f"# {name}\n\n"
+            _write_skill_from_editor(
+                console,
+                debug_mode_container,
+                name,
+                initial_content,
+                overwrite=True,
+                verb="Saved",
+            )
+            return CommandResult(status="handled")
+
+        if subcmd in ("add", "create", "new"):
+            if len(parts) < 2:
+                console.print(f"[red]Usage: /skills {subcmd} <name> [prompt][/red]")
+                return CommandResult(status="handled")
+            name = validate_skill_name(parts[1])
+            if len(parts) >= 3:
+                path = write_skill(name, parts[2], overwrite=False)
+                console.print(f"[green]Created skill '{name}'.[/green] [dim]{path}[/dim]")
+                return CommandResult(status="handled")
+            _write_skill_from_editor(
+                console,
+                debug_mode_container,
+                name,
+                f"# {name}\n\n",
+                overwrite=False,
+                verb="Created",
+            )
+            return CommandResult(status="handled")
+
+        if subcmd == "modify":
+            cmd_parts = args_clean.split(maxsplit=2)
+            if len(cmd_parts) < 2:
+                console.print("[red]Usage: /skills modify <name> [prompt][/red]")
+                return CommandResult(status="handled")
+            name = validate_skill_name(cmd_parts[1])
+            if len(cmd_parts) < 3:
+                _write_skill_from_editor(
+                    console,
+                    debug_mode_container,
+                    name,
+                    read_skill(name, strip_heading=False),
+                    overwrite=True,
+                    verb="Updated",
+                )
+                return CommandResult(status="handled")
+            read_skill(name)
+            path = write_skill(name, cmd_parts[2], overwrite=True)
+            console.print(f"[green]Updated skill '{name}'.[/green] [dim]{path}[/dim]")
+            return CommandResult(status="handled")
+
+        if len(parts) >= 2:
+            name = validate_skill_name(parts[0])
+            _, body = args_clean.split(maxsplit=1)
+            path = write_skill(name, body, overwrite=False)
+            console.print(f"[green]Created skill '{name}'.[/green] [dim]{path}[/dim]")
+            return CommandResult(status="handled")
+
+        console.print("[red]Usage: /skills add <name> [prompt][/red]")
+        console.print("[dim]Run [bold #5F9EA0]/skills help[/bold #5F9EA0] for all commands.[/dim]")
+        return CommandResult(status="handled")
+
+    except SkillError as e:
+        console.print(f"[red]{e}[/red]")
+        return CommandResult(status="handled")
+    except Exception as e:
+        console.print(f"[red]Skills command failed: {e}[/red]")
+        return CommandResult(status="handled")
+
+
 def _handle_obsidian_init(console, obsidian_settings):
     """Handle /obsidian init — scaffold project folder structure in vault."""
     if not obsidian_settings.is_active():
@@ -2741,6 +3062,7 @@ _COMMAND_HANDLERS = {
     "/setup": _handle_setup,
     "/cron": _handle_cron,
     "/prompt": _handle_prompt,
+    "/skills": _handle_skills,
 }
 
 

package/src/ui/displays.py

@@ -22,7 +22,7 @@ def show_provider_table(current_provider: str, console):
         else:
             status = "✅" if cfg.get("api_key") else "❌ (set API key)"
         active = " [green](active)[/green]" if provider == current_provider else ""
-        table.add_row(provider.capitalize(), status, f"{model[:40]}{active}")
+        table.add_row(config.get_provider_display_name(provider), status, f"{model[:40]}{active}")
 
     console.print(table)
 
@@ -66,6 +66,7 @@ def show_help_table(console):
     table.add_row("[bold #5F9EA0]/cd[/bold #5F9EA0] [path]", "Change working directory (no args to show current)")
     table.add_row("[bold #5F9EA0]/edit[/bold #5F9EA0], [bold #5F9EA0]/e[/bold #5F9EA0]", "Open editor for multi-line input")
     table.add_row("[bold #5F9EA0]/review[/bold #5F9EA0] [args], [bold #5F9EA0]/r[/bold #5F9EA0]", "Code review git changes (e.g. /review --staged, /review main..HEAD)")
+    table.add_row("[bold #5F9EA0]/skills[/bold #5F9EA0] [list|add|modify|remove|use]", "Manage reusable prompt skills")
     table.add_row("[bold #5F9EA0]/obsidian[/bold #5F9EA0] [set|enable|disable|status|init]", "Manage vault integration, scaffold project folders")
     table.add_row("[bold #5F9EA0]/tools[/bold #5F9EA0] [list|enable|disable|enable-group|disable-group]", "Toggle tools or groups (e.g. file_ops, task_mgmt)")
     table.add_row("[bold #5F9EA0]/setup[/bold #5F9EA0]", "Re-run the first-run setup wizard")
@@ -142,6 +143,30 @@ def show_cron_help_table(console):
     console.print("")
 
 
+def show_skills_help_table(console):
+    """Display skills command help table.
+
+    Args:
+        console: Rich Console instance for output.
+    """
+    console.print("")
+    table = Table(show_header=True, box=box.SIMPLE_HEAD)
+    table.add_column("Command", no_wrap=True)
+    table.add_column("Description")
+
+    table.add_row("[bold #5F9EA0]/skills list[/bold #5F9EA0]", "List skills")
+    table.add_row("[bold #5F9EA0]/skills add[/bold #5F9EA0] <name>", "Create a skill in your editor")
+    table.add_row("[bold #5F9EA0]/skills edit[/bold #5F9EA0] <name>", "Edit an existing skill")
+    table.add_row("[bold #5F9EA0]/skills modify[/bold #5F9EA0] <name> <prompt>", "Replace a skill")
+    table.add_row("[bold #5F9EA0]/skills show[/bold #5F9EA0] <name>", "Show a skill")
+    table.add_row("[bold #5F9EA0]/skills load[/bold #5F9EA0] <name>", "Load a skill into this chat")
+    table.add_row("[bold #5F9EA0]/skills remove[/bold #5F9EA0] <name>", "Delete a skill")
+    table.add_row("[bold #5F9EA0]/skills dir[/bold #5F9EA0]", "Show the skills directory")
+
+    console.print(Panel(table, title="[bold #5F9EA0]Skills[/bold #5F9EA0]", border_style="grey23", padding=(0, 2)))
+    console.print("")
+
+
 def show_config_overview(chat_manager, console, debug_mode_container, current_provider):
     """Display comprehensive configuration overview.
 

package/src/ui/main.py

@@ -35,7 +35,6 @@ from core.agentic import agentic_answer
 from utils.settings import MonokaiDarkBGStyle, left_align_headings
 from utils.paths import REPO_ROOT, RG_EXE_PATH
 from exceptions import BoneAgentError
-from tools.loader import load_all_tools
 
 # Console setup
 console = Console(theme=Theme({
@@ -382,9 +381,6 @@ def main():
     """Main interactive chat loop."""
 
     # Load all tools (built-in and user tools)
-    # This populates the ToolRegistry with all decorated tools
-    load_all_tools()
-
     # Check for config.yaml — run setup wizard on first run
     from ui.setup_wizard import is_first_run, run_wizard as _run_setup_wizard