You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
Least-privilege remediation can broaden allowed-tools preapproval #674
With a detected shell capability and allowed-tools: [Read], an LP1 probe produces a HIGH finding and recommends adding a tool that covers shell execution. With the optional declaration absent, LP3 reports no declared scope. Adding Bash removes the probe finding.
That remediation can broaden preapproval instead of reducing authority. Optional skill metadata is not, by itself, proof of the host's enforced permission ceiling.
Distinguish tool preapproval from observed capabilities and evidence of runtime enforcement. Remediation should not tell users to broaden approval grants merely to silence a finding. Continue reporting dangerous capabilities and broad grants when supported, and state when runtime permissions are unknown.
Related work
#300 and #313 concern the field name used in remediation. PR #447 corrects EA1 wildcard patterns. Those changes do not address the distinction between preapproval and enforced permissions.
Resolution verified on 2026-10-04: PR #685 has merged, and GitHub automatically closed this issue.
LP1/LP3 guidance now distinguishes tool preapproval from actual runtime enforcement and no longer recommends broadening allowed-tools merely to silence a finding. Any capability addition must be independently justified and approved.
Verification on merged main (05d7882): all 19 tests in tests/test_mcp_least_privilege.py passed, covering both analyzer remediation and fallback guidance.
Activity
rng1995 commentedon Oct 4, 2026
Resolution verified on 2026-10-04: PR #685 has merged, and GitHub automatically closed this issue.
LP1/LP3 guidance now distinguishes tool preapproval from actual runtime enforcement and no longer recommends broadening allowed-tools merely to silence a finding. Any capability addition must be independently justified and approved.
Verification on merged main (05d7882): all 19 tests in tests/test_mcp_least_privilege.py passed, covering both analyzer remediation and fallback guidance.