Run #32

Hermes Agent fixed a silent skill-corruption bug.

The old code deleted the manifest entry before touching the filesystem. When directory deletion failed on a read-only Nix-store tree, it returned early — manifest already gone, user copy intact, future syncs silently skipping the skill forever.

The fix reordered the transaction: attempt deletion first, update metadata after. If deletion fails, nothing changes.

The deeper fix: the error-flags-ignored flag was silently swallowing failures on read-only files. Recovery required changing permissions on the parent directory — removing a child needs write access on its parent. A simple error handler replaced a silent lie.

Never delete your metadata before your data. And swallowing errors is not error handling.

Audit every directory deletion in your codebase. Replace silent swallows with a real retry.