refactor(cdp): rename emailAssets to messageAssets - #75801
Merged
dmarchuk merged 1 commit intoJul 31, 2026
Merged
Conversation
dmarchuk
force-pushed
the
dmarchuk/push-test-sends-skip-metrics
branch
from
July 31, 2026 08:17
96d5bd2 to
ffffc62
Compare
dmarchuk
force-pushed
the
dmarchuk/rename-email-assets-to-message-assets
branch
2 times, most recently
from
July 31, 2026 08:53
33c6a3f to
dacb1d7
Compare
dmarchuk
force-pushed
the
dmarchuk/push-test-sends-skip-metrics
branch
from
July 31, 2026 09:45
7835edf to
3f667e1
Compare
dmarchuk
force-pushed
the
dmarchuk/rename-email-assets-to-message-assets
branch
from
July 31, 2026 09:47
dacb1d7 to
e64da88
Compare
dmarchuk
marked this pull request as ready for review
July 31, 2026 09:49
Contributor
|
Reviews (1): Last reviewed commit: "refactor(cdp): rename emailAssets to mes..." | Re-trigger Greptile |
mayteio
approved these changes
Jul 31, 2026
dmarchuk
merged commit Jul 31, 2026
4c15a55
into
dmarchuk/push-test-sends-skip-metrics
274 checks passed
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.
Problem
CyclotronJobInvocationResult.emailAssetsnow carries push rows as well as email ones. #74555 called this out as the one place the reuse leaks and left the rename out to keep that diff reviewable:This is that follow-up. The underlying table, service and row type are already channel-agnostic (
message_assets,MessageAssetsService,MessageAssetRow), so the field name was the last thing still claiming email.Changes
Pure rename,
emailAssetstomessageAssets, across 17 files. No behavior change: the field is renamed at its declaration, at every producer and consumer, and in the tests.messageAssetsService(the service handle onPushNotificationServiceandEmailService) is a different identifier and is untouched.How did you test this code?
Mechanical, so the check is that nothing else moved:
emailAssetsremain innodejs/src; 56 ofmessageAssets.tsc --noEmitadds no new errors versus the base branch. Both report the same 7 pre-existing@posthog/replay-anonymizermodule-resolution errors from an unbuilt local package.jest src/cdp/services/messaging src/cdp/services/hogflows src/cdp/utils- 658 passing. The 2 failures are the maildev-backed email tests, which fail identically on the base branch without a maildev container and pass in CI.No test was added: a rename with no behavior change has no regression to catch that the type checker doesn't already catch.
Automatic notifications
Docs update
None needed. The field is internal to the invocation result and not part of any documented surface.
🤖 Agent context
Autonomy: Human-driven (agent-assisted). Directed by @dmarchuk, assigned as DRI.
Stacked on #75800, which is stacked on #74555. Kept last in the stack deliberately: a mechanical sweep should come after the semantic changes so it picks them up, rather than forcing them to rebase across a rename.