Skip to content

fix: reduce Windows Defender false positive on fileless malware skill - #112

Open
kevglynn wants to merge 2 commits into
mukul975:mainfrom
kevglynn:contrib/100-defender-false-positive
Open

fix: reduce Windows Defender false positive on fileless malware skill#112
kevglynn wants to merge 2 commits into
mukul975:mainfrom
kevglynn:contrib/100-defender-false-positive

Conversation

@kevglynn

@kevglynn kevglynn commented Jul 16, 2026

Copy link
Copy Markdown

Summary

Addresses #100 — Windows Defender quarantines detecting-fileless-malware-techniques/SKILL.md as Trojan:script/Wacatac.H!ml. The skill content is defensive/educational; the detection is a false positive caused by literal malware-pattern strings embedded in documentation.

Scope and limitations: This PR reduces the triggering surface by removing the highest-signal tokens (inline YARA rules with literal malware strings, a Base64-encoded PowerShell payload sample). However, SKILL.md intentionally retains educational references to LOLBin abuse patterns, Invoke-Mimikatz, CobaltStrike, and AMSI bypass — these are core to the skill's purpose and cannot be sanitized without gutting the content. This PR may reduce but not eliminate false positives. The primary deliverable is the discoverability section so users who hit this know it's a known issue and how to resolve it.

Changes:

Test plan

  • python3 tools/validate-skill.py skills/detecting-fileless-malware-techniques → PASS
  • python3 tools/validate-skill.py --all → 817/817 PASS
  • Windows Defender re-scan on Windows (cannot verify in Linux CI — would appreciate confirmation from a Windows user)

Mitigates #100

kevglynn added 2 commits July 16, 2026 17:08
Add AV false-positive guidance for issue mukul975#100, move inline YARA rule to
references/yara-fileless-powershell.yar, and replace literal encoded
PowerShell payload with a placeholder in the sample report.
The --yara-file reference should not assume a specific working directory.
Use a placeholder path that analysts will substitute for their setup.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant