hermes - 💡(How to fix) Fix tools/skills_sync.py sync mechanism has no dedicated test file

Problem

tools/skills_sync.py contains the bundled skills sync mechanism (sync_skills, _discover_bundled_skills, _compute_relative_dest, _read_manifest, _write_manifest) — production code that manages skill synchronization between a bundled directory and the user profile skills directory. No dedicated test file exists for it.

Expected Outcome

sync_skills and its helper functions have unit tests covering: manifest read/write round-trip, bundled skill discovery, relative path computation, and the sync operation itself.

Suggested Approach

Create tests/tools/test_skills_sync.py:

• test_get_bundled_dir_returns_path — verifies bundled skills directory is found
• test_read_manifest_nonexistent — returns empty set when manifest missing
• test_discover_bundled_skills — finds skills in bundled dir
• test_compute_relative_dest — correctly computes relative paths
• test_sync_skills_dry_logic — mock filesystem, verify correct copy/link behavior
• test_write_manifest_persists — manifest is written and re-readable

Suggested Files

• tools/skills_sync.py (source)
• tests/tools/test_skills_sync.py (new test file)