posthog-js icon indicating copy to clipboard operation
posthog-js copied to clipboard

feat: super properties for posthog-node

Open harshit078 opened this issue 4 months ago • 4 comments

Problem

  • This PR address issue https://github.com/PostHog/posthog-js/issues/2727
  • added super properties in posthog-node

Changes

  • added registeredProperties in prepareEventMessage
  • this allows prepareEventMessage to ensure that every event includes globally registered super properties.
  • Added tests to verify the correct behavior of property registration and overriding.

Release info Sub-libraries affected

Libraries affected

  • [ ] All of them
  • [ ] posthog-js (web)
  • [ ] posthog-js-lite (web lite)
  • [x] posthog-node
  • [ ] posthog-react-native
  • [ ] @posthog/react
  • [ ] @posthog/ai
  • [ ] @posthog/nextjs-config
  • [ ] @posthog/nuxt
  • [ ] @posthog/rollup-plugin
  • [ ] @posthog/webpack-plugin

Checklist

  • [x] Tests for new code
  • [ ] Accounted for the impact of any changes across different platforms
  • [x] Accounted for backwards compatibility of any changes (no breaking changes!)
  • [x] Took care not to unnecessarily increase the bundle size

If releasing new changes

  • [ ] Ran pnpm changeset to generate a changeset file
  • [ ] Added the "release" label to the PR to indicate we're publishing new versions for the affected packages

harshit078 avatar Dec 11 '25 11:12 harshit078

@harshit078 is attempting to deploy a commit to the PostHog Team on Vercel.

A member of the Team first needs to authorize it.

vercel[bot] avatar Dec 11 '25 11:12 vercel[bot]

Hey @pauldambra , can you review the PR ? Thanks !

harshit078 avatar Dec 15 '25 19:12 harshit078

hey Harshit! I'll review this one tomorrow - got stuck today fixing some other issues, but it's on my list

lucasheriques avatar Dec 17 '25 00:12 lucasheriques

Hey @lucasheriques , sure I'll do the 3 steps and update the PR. Thanks !

harshit078 avatar Dec 19 '25 19:12 harshit078