Skip to content

feat: --ci flag, 3 new security checks, security-scan workflow - #2

Open
xlyoung wants to merge 1 commit into
masterfrom
feat/ci-mode-and-new-checks
Open

feat: --ci flag, 3 new security checks, security-scan workflow#2
xlyoung wants to merge 1 commit into
masterfrom
feat/ci-mode-and-new-checks

Conversation

@xlyoung

@xlyoung xlyoung commented Jun 8, 2026

Copy link
Copy Markdown
Owner

Summary

  • Add --ci flag to scan command for CI/CD pipelines (JSON output + exit codes)
  • Add 3 new security detection engines: unsafe deserialization, dynamic import, log injection
  • Create reusable security-scan.yml GitHub Action workflow
  • Improve mcp-scan-action.yml with --ci flag and fail_on_high option
  • Add 17 new unit tests (78 total, all passing)

New Security Checks (12 engines, up from 9)

  1. Unsafe Deserialization — pickle, yaml.load, marshal, shelve, jsonpickle
  2. Dynamic Importimport(), importlib.import_module(), require() with user input
  3. Log Injection — user input in log statements

- Add --ci flag to 'scan' command: JSON output + exit code for CI/CD
- Add unsafe deserialization detection (pickle, yaml.load, marshal, shelve, jsonpickle)
- Add dynamic import detection (__import__, importlib, require with user input)
- Add log injection detection (user input in log statements)
- Create .github/workflows/security-scan.yml for easy CI integration
- Improve mcp-scan-action.yml with --ci flag and fail_on_high option
- Add 17 new unit tests for scanner checks
- Update README: CI/CD integration section, 12 engines
- Update CHANGELOG
- All 78 tests passing
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