Skip to content

PIR: Clear emailExtractedData on step completion#8612

Open
landomen wants to merge 1 commit into
feature/domen/pir-email-flexibility-complete-request-datafrom
feature/domen/pir-email-flexibility-clear-on-completion
Open

PIR: Clear emailExtractedData on step completion#8612
landomen wants to merge 1 commit into
feature/domen/pir-email-flexibility-complete-request-datafrom
feature/domen/pir-email-flexibility-clear-on-completion

Conversation

@landomen
Copy link
Copy Markdown
Contributor

@landomen landomen commented May 19, 2026

Task/Issue URL: https://app.asana.com/1/137249556945/task/1213993026941875?focus=true

Description

Clear emailExtractedData on step completion

Steps to test this PR

Will be testable on later PRs once the whole feature is complete

UI changes

No UI changes


Note

Low Risk
Low risk: small state-reset change limited to the PIR action runner state transition, with a matching unit test update.

Overview
Clears per-step emailExtractedData when a BrokerStepCompleted event advances to the next broker step, ensuring extracted email fields don’t leak into subsequent steps.

Updates the corresponding unit test to seed emailExtractedData and assert it is reset to an empty map on completion.

Reviewed by Cursor Bugbot for commit 733cabf. Bugbot is set up for automated code reviews on this repo. Configure here.

Copy link
Copy Markdown
Contributor Author

landomen commented May 19, 2026

@landomen landomen force-pushed the feature/domen/pir-email-flexibility-clear-on-completion branch from bc869a1 to f6b07d2 Compare May 19, 2026 11:00
@landomen landomen force-pushed the feature/domen/pir-email-flexibility-clear-on-completion branch from f6b07d2 to 73245d0 Compare May 21, 2026 17:48
@landomen landomen force-pushed the feature/domen/pir-email-flexibility-complete-request-data branch from 0fa4b73 to d736d56 Compare May 22, 2026 06:46
@landomen landomen force-pushed the feature/domen/pir-email-flexibility-clear-on-completion branch from 73245d0 to 733cabf Compare May 22, 2026 06:46
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