Skip to content

feat(safety): adding safety hooks - #3

Merged
manas-fortifyroot merged 1 commit into
fr-v0.52.6.xfrom
feature/phase-1h-safety-hooks
Mar 11, 2026
Merged

feat(safety): adding safety hooks#3
manas-fortifyroot merged 1 commit into
fr-v0.52.6.xfrom
feature/phase-1h-safety-hooks

Conversation

@manas-fortifyroot

Copy link
Copy Markdown

Summary

  • add a minimal FortifyRoot safety hook registry in a new opentelemetry-instrumentation-fortifyroot package
  • wire prompt/completion safety into the first provider wave:
    • OpenAI
    • Anthropic
    • Cohere
    • Google Generative AI
  • keep upstream-facing diffs in existing provider files narrow by moving most new logic into provider-local safety.py helper modules
  • emit event-only fortifyroot.safety.violation telemetry with finding-local fields only
  • keep current scope to prompt enforcement plus non-stream completion enforcement
  • add safety-focused tests for the new hook paths and helper logic
  • add a PR GitHub Actions workflow that runs ./scripts/run-tests.sh --safety for any pull request target branch
  • add a repo-level scripts/run-tests.sh helper with shared .venv support and safety-only mode

Scope Notes

  • current provider slice covers OpenAI, Anthropic, Cohere, and Google Generative AI
  • streaming completion enforcement is still deferred
  • remaining provider parity and LiteLLM work are not part of this PR

Testing

  • ./scripts/run-tests.sh --safety

@manas-fortifyroot
manas-fortifyroot merged commit 384d54c into fr-v0.52.6.x Mar 11, 2026
1 check passed
@manas-fortifyroot
manas-fortifyroot deleted the feature/phase-1h-safety-hooks branch March 11, 2026 12:47
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