Skip to content

PIR: Update completeRequestData to support scans and email extracted data#8583

Open
landomen wants to merge 2 commits into
feature/domen/pir-email-flexibility-pollingfrom
feature/domen/pir-email-flexibility-complete-request-data
Open

PIR: Update completeRequestData to support scans and email extracted data#8583
landomen wants to merge 2 commits into
feature/domen/pir-email-flexibility-pollingfrom
feature/domen/pir-email-flexibility-complete-request-data

Conversation

@landomen
Copy link
Copy Markdown
Contributor

@landomen landomen commented May 15, 2026

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

Description

Updates completeRequestData() to support scans and email extracted data

Steps to test this PR

Will be testable later when feature is complete.

UI changes

No UI changes


Note

Medium Risk
Touches PIR action execution request-building logic and retry behavior; mistakes could cause scripts to receive incomplete/wrong extractedProfile data during scans/opt-outs and change retry handling after email polling.

Overview
Updates PIR’s broker action execution to enrich UserProfile request data when an action’s dataSource is EXTRACTED_PROFILE.

completeRequestData is refactored to build ExtractedProfileParams from the current State (not separate args), now supporting scan steps (optionally injecting generated email) and attaching email-extracted fields (emailExtractedData) when present. ExtractedProfileParams gains a new emailExtractedData map field.

When EmailDataReceived is handled, the engine now resets actionRetryCount to 0 while storing the extracted email data. Tests are updated/expanded to cover scan-step behavior, email data propagation, and retry-count reset.

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

Copy link
Copy Markdown
Contributor Author

landomen commented May 15, 2026

@landomen landomen force-pushed the feature/domen/pir-email-flexibility-polling branch from f7a3f45 to de20666 Compare May 19, 2026 11:00
@landomen landomen force-pushed the feature/domen/pir-email-flexibility-complete-request-data branch from bc43e3d to 72e7b4b Compare May 19, 2026 11:00
@landomen landomen force-pushed the feature/domen/pir-email-flexibility-polling branch from de20666 to 8f49ae6 Compare May 21, 2026 17:48
@landomen landomen force-pushed the feature/domen/pir-email-flexibility-complete-request-data branch from 72e7b4b to 0fa4b73 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-polling branch from 8f49ae6 to 90eb151 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