Skip to content

refactor: remove legacy PII scrubber#185

Open
huazhuang80-star wants to merge 1 commit into
ChainForgee:mainfrom
huazhuang80-star:remove-root-pii-scrubber-112
Open

refactor: remove legacy PII scrubber#185
huazhuang80-star wants to merge 1 commit into
ChainForgee:mainfrom
huazhuang80-star:remove-root-pii-scrubber-112

Conversation

@huazhuang80-star

Copy link
Copy Markdown

Summary

  • remove the root-level legacy scrubber.py implementation
  • point the existing PII regression test at �pp/ai-service/services/pii_scrubber.py
  • update legacy fixtures to match the canonical service token names

Fixes #112

Testing

  • git diff --check
  • Not run: python -m pytest tests/test_pii_scrubber.py because the local Python launcher exits without a usable interpreter in this shell
  • Not run: bundled Python lacks project test dependencies (pytest, then prometheus_client when checking the service import directly)

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.

[LOW] Deprecate and remove root scrubber.py in favor of AI service implementation

1 participant