Skip to content

Expand Semgrep coverage and integration tests#16

Merged
haasonsaas merged 1 commit into
mainfrom
feat/expand-semgrep-coverage
Jul 17, 2026
Merged

Expand Semgrep coverage and integration tests#16
haasonsaas merged 1 commit into
mainfrom
feat/expand-semgrep-coverage

Conversation

@haasonsaas

Copy link
Copy Markdown
Collaborator

Summary

  • add defensible Go/Rust rules for secret literals, untrusted outbound destinations, and Rust command arguments
  • broaden standard-library input and SQL sink coverage while documenting the exact supported families
  • run real semgrep-core 1.139.0 integration tests in CI, including positive, negative, diff-filtering, invalid-rule, and fail-closed cases

Test plan

  • go test -count=1 ./...
  • ROOM_SEMGREP_CORE=/home/developer/.local/bin/semgrep-core go test -count=1 -tags=semgrep_integration ./cmd/room-semgrep
  • semgrep scan --oss-only --metrics off --disable-version-check --strict --validate --config analyzers/semgrep/room.yml
  • actionlint .github/workflows/ci.yml

@haasonsaas
haasonsaas merged commit e71d1be into main Jul 17, 2026
4 checks passed
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