What does this PR do?

Fixes five silent exception blocks in agent/skill_commands.py where bare except Exception: was discarding errors without any logging. Users had no way to diagnose why skills failed to load, scan, or resolve paths.

Type of Change

• 🐛 Bug fix (non-breaking change that fixes an issue)

Changes Made

scan_skill_commands()

• Inner loop (except Exception: continue): Now logs logger.debug() with the skill path and error before continuing — helps identify broken individual skill files without flooding logs.
• Outer block (except Exception: pass): Now logs logger.warning() when the entire skill scanning system fails — critical for diagnosing import errors or missing SKILLS_DIR.

_load_skill_payload()

• Path resolution (except Exception: normalized = raw_identifier): Now logs logger.debug() when path normalization fails.
• Skill loading (except Exception: return None): Now logs logger.warning() when a skill fails to load — users can see which skill and why.
• Directory resolution (except Exception: skill_dir = None): Now logs logger.debug() when skill directory path computation fails.

No new imports needed — logger = logging.getLogger(__name__) was already defined at the top of the file.

How to Test

1. Introduce a syntax error in any ~/.hermes/skills/*/SKILL.md file
2. Run hermes and invoke a skill slash command
3. Check logs — you should now see warnings/debug messages identifying the failure
4. Run tests: python3 -m pytest tests/ -q

Checklist

Code

• I've read the Contributing Guide
• My commit messages follow Conventional Commits
• I searched for existing PRs to make sure this isn't a duplicate
• My PR contains only changes related to this fix
• I've tested on my platform: Ubuntu 22.04 (WSL2)

Documentation & Housekeeping

• No documentation changes needed (logging-only change)
• No config key changes
• No architecture changes
• Cross-platform safe (logging module is stdlib)

Screenshots / Logs

N/A — logging change, no UI impact.