Code Audit Finding §6.4 — Low
Problem: Comment "executeCommand removed for security reasons" in HelperProtocol.swift lacks context for future maintainers.
Locations:
- `Sources/KeyPathHelper/HelperProtocol.swift:89`
- `Sources/KeyPathAppKit/Core/HelperProtocol.swift:89`
Action: Add a brief security note explaining the command injection risk that motivated removal, so a future developer doesn't re-introduce generic command execution.
From comprehensive code audit 2026-05-21
Code Audit Finding §6.4 — Low
Problem: Comment "executeCommand removed for security reasons" in HelperProtocol.swift lacks context for future maintainers.
Locations:
Action: Add a brief security note explaining the command injection risk that motivated removal, so a future developer doesn't re-introduce generic command execution.
From comprehensive code audit 2026-05-21