Skip to content

fix(ci): scan Vue/Svelte/Astro/Go files as code, not as disguised payloads - #220

Merged
afeezorobsco-cyber merged 1 commit into
mainfrom
fix/scanner-component-extensions
Oct 21, 2026
Merged

afeezorobsco-cyber merged 1 commit into
mainfrom
fix/scanner-component-extensions

Conversation

@afeezorobsco-cyber

Copy link
Copy Markdown
Contributor

The malware scan's disguised-code rule fires when a file whose extension isn't a known code type contains JavaScript. That catches payloads hidden as fonts or images, like the one in escrow #163. But .vue single-file components legitimately contain <script> blocks, so docforum-core #207 (docs/site/api/ApiReference.vue) was flagged HIGH, and because Malware scan is a required check, it was blocked.

Change: vue, svelte, astro, less, sass, go, mod and sum are added to the scanner's code/text extension list. These files are still scanned by every line rule (pipe-to-shell, exfiltration endpoints, decode-and-run and so on); they're just no longer treated as "this shouldn't contain code". Real media files (fonts, images, archives, .wasm) are unchanged.

Tested with the scanner harness against live PRs:

pull_request_target runs the scanner from main, so this takes effect once merged. I'll then re-scan core #207 with the backfill.

🤖 Generated with Claude Code

…loads

The disguised-code rule flags JavaScript inside a file whose extension
isn't a known code type. .vue single-file components legitimately
contain <script> blocks, so docforum-core #207 was blocked by a false
positive. Add vue, svelte, astro, less, sass, go, mod and sum to the
text/code extension list; every line rule still runs on them.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@afeezorobsco-cyber
afeezorobsco-cyber merged commit 5e33389 into main Oct 21, 2026
9 checks passed
@afeezorobsco-cyber
afeezorobsco-cyber deleted the fix/scanner-component-extensions branch October 21, 2026 06:07
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