Skip to content

feat: PostHog analytics improvements#4209

Open
gaga1307 wants to merge 4 commits intomainfrom
feat/posthog-enhancements
Open

feat: PostHog analytics improvements#4209
gaga1307 wants to merge 4 commits intomainfrom
feat/posthog-enhancements

Conversation

@gaga1307
Copy link
Copy Markdown
Collaborator

Internal User Identification

  • Added is_internal flag in posthog.identify
  • Users with @superplane.com email are marked as internal for easier filtering in analytics

Event Naming Cleanup

  • Standardized event names
  • Applied consistent snake_case naming

Session Replay — Canvas Recording

  • Enabled canvas capture for React Flow
  • Limited to 1 FPS and 10% quality to reduce overhead

Session Replay — Privacy & Performance

  • Marked sensitive elements with ph-no-capture:

    • Password fields (OwnerSetup, login, SMTP)
    • API tokens and service account tokens
    • Secret values and secret key references
    • OpenAI API key field
    • Webhook secrets/tokens
    • Integration credentials
  • Marked sidebar components (component config, AI builder, canvas logs) with ph-no-capture to improve performance by reducing DOM observation overhead

Signed-off-by: Dragica Draskic <dragica.draskic@gmail.com>
Signed-off-by: Dragica Draskic <dragica.draskic@gmail.com>
…r to improve performance

Signed-off-by: Dragica Draskic <dragica.draskic@gmail.com>
Signed-off-by: Dragica Draskic <dragica.draskic@gmail.com>
@superplanehq-integration
Copy link
Copy Markdown

👋 Commands for maintainers:

  • /sp start - Start an ephemeral machine (takes ~30s)
  • /sp stop - Stop a running machine (auto-executed on pr close)

Copy link
Copy Markdown
Contributor

@cursor cursor bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit da3e3e6. Configure here.

organizationId={organizationId}
/>
))}
</div>
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wrapper div loses spacing between configuration fields

Low Severity

The new ph-no-capture wrapper div at line 324 doesn't include space-y-4, so when multiple ConfigurationFieldRenderer components render (e.g. both signingSecret and webhookSigningSecret, or several fields via initialStepFieldNames), they stack with no vertical gap. The non-webhook branch at line 368 correctly uses "space-y-4 ph-no-capture", making this inconsistent. Previously, each field was a direct child of the parent space-y-4 container and received proper spacing.

Additional Locations (1)
Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit da3e3e6. Configure here.

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