fix(inbox): track signal_report_id on Create PR task_created event#2458
Merged
Conversation
The Discuss flow attaches signal_report_id to its task_created analytics event, but the Create PR flow omitted it, leaving created-PR tasks unattributable to their originating inbox report from that event. Add the property to bring Create PR to parity with Discuss.
Contributor
|
Reviews (1): Last reviewed commit: "fix(inbox): include signal_report_id in ..." | Re-trigger Greptile |
oliverb123
approved these changes
Jun 2, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Brings the Create PR inbox action to tracking parity with Discuss.
Both inbox actions fire a
task_createdanalytics event on success, but the Create PR flow (useCreatePrReport.ts) omittedsignal_report_id, while Discuss (useDiscussReport.ts) includes it. That left created-PR-from-inbox tasks unattributable to their originating report when analyzing thetask_createdevent (funnels, attribution), even though the linkage exists on the task record itself viasignalReportIdinTaskCreationInput.This adds
signal_report_id: reportIdto the Create PRtask_createdevent — a one-line change.signal_report_idis already a valid optional property onTaskCreateProperties.Before / after
cloud_run_source: "signal_report", cloud_pr_authorship_mode: "user", + signal_report_id: reportId, adapter,Notes
inbox_report_actionviafireDetailAction) already covers both buttons correctly; Discuss only differs there byhas_question/question_text, which is expected (PR has no free-text question).@posthog/agent(Claude Agent SDK type mismatches). This change only touches a renderer hook and does not affect those.Test plan
task_createdevent carriessignal_report_id.