Skip to content

fix llm guard ci checks#17

Open
poojakira wants to merge 1 commit into
mainfrom
codex/fix-llm-guard-ci
Open

fix llm guard ci checks#17
poojakira wants to merge 1 commit into
mainfrom
codex/fix-llm-guard-ci

Conversation

@poojakira

Copy link
Copy Markdown
Owner

what changed

  • fixes CI paths after package layout moved from old src imports to llm_guard_scanner package imports
  • updates red-team corpus gate to use the package CLI module instead of missing scan.py
  • corrects coverage target to the installed package and adds deterministic CLI tests
  • removes overstated demo wording and keeps mappings scoped to implemented checks

why

  • current main CI was failing on missing files, stale imports, and a broken coverage target
  • the repo is a flagship ML-security portfolio project, so green reproducible checks matter more than broad claims

risks/edge cases

  • optional embedding and broader OWASP detector modules still have low direct test coverage; this PR does not claim full coverage
  • local Python environment showed unrelated broken package warnings for existing site-packages, but repo tests pass offline

validation performed

  • py -3.12 -m ruff check src tests demo.py verify.py
  • py -3.12 -m ruff format --check src tests demo.py verify.py
  • py -3.12 -m pytest -q: 84 passed, coverage 68.62% over 60% gate
  • py -3.12 demo.py
  • py -3.12 verify.py

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