Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Feat: Mutator Hooks #870

Merged
merged 26 commits into from
Jan 20, 2025
Merged

Feat: Mutator Hooks #870

merged 26 commits into from
Jan 20, 2025

Conversation

VisargD
Copy link
Collaborator

@VisargD VisargD commented Jan 17, 2025 •
edited
Loading

Title:

  • Mutator Hooks

Related Issues: (optional)

VisargD added 15 commits January 17, 2025 18:18
@VisargD
feat: add mutator hook type support in hooks middleware
afcb0e9
@VisargD
feat: add promptfoo redact pii plugin handler
cbaf20a
@VisargD
feat: add portkey redact pii plugin handler
6b3cd03
@VisargD
feat: add patronus redact pii plugin handler
702e18a
@VisargD
feat: add patronus redact phi plugin handler
45eea77
@VisargD
feat: add pange redact pii plugin handler
94bfd36
@VisargD
chore: update portkey detect pii function to allow multiple text parts
a5c4292
@VisargD
chore: add patronus entity masking functions
2e0318d
@VisargD
chore: add util functions for content part get and set
f92d676
@VisargD
chore: update plugins index with new handlers
8ad2574
@VisargD
feat: use plugin transformed response if available
17e50a4
@VisargD
feat: use plugin transformed request if available
8ceae85
@VisargD
merge main into feat/mutator-hooks
0848746
@VisargD
feat: add mutator shorthand handling in try targets method
55dc4c6
@VisargD
fix: try post variable uninitialized error
af8a0ca
@VisargD VisargD requested review from b4s36t4 and narengogi January 17, 2025 19:58
@VisargD VisargD marked this pull request as ready for review January 17, 2025 19:58
narengogi
narengogi previously approved these changes Jan 18, 2025
View reviewed changes
b4s36t4
b4s36t4 suggested changes Jan 20, 2025
View reviewed changes
plugins/index.ts Show resolved Hide resolved
plugins/patronus/redactPhi.ts Show resolved Hide resolved
plugins/patronus/redactPhi.ts Outdated Show resolved Hide resolved
plugins/patronus/redactPii.ts Show resolved Hide resolved
src/handlers/handlerUtils.ts Outdated Show resolved Hide resolved
src/handlers/handlerUtils.ts Outdated Show resolved Hide resolved
@VisargD VisargD requested a review from b4s36t4 January 20, 2025 10:31
b4s36t4
b4s36t4 approved these changes Jan 20, 2025
View reviewed changes
Copy link
Contributor

@b4s36t4 b4s36t4 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@VisargD VisargD merged commit 1ca2bed into main Jan 20, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@b4s36t4 b4s36t4 b4s36t4 approved these changes

@narengogi narengogi narengogi left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[Feature] Mutator Hooks
3 participants
@VisargD @narengogi @b4s36t4