Skip to content

chore: Remove deprecated KitIntegration.getAllUserAttributes()#682

Open
denischilik wants to merge 3 commits intoworkstation/6.0-Releasefrom
chore/remove-getAllUserAttributes
Open

chore: Remove deprecated KitIntegration.getAllUserAttributes()#682
denischilik wants to merge 3 commits intoworkstation/6.0-Releasefrom
chore/remove-getAllUserAttributes

Conversation

@denischilik
Copy link

@denischilik denischilik commented Mar 26, 2026

Background

KitIntegration.getAllUserAttributes() was deprecated in favor of filtered user APIs. This removes the method so custom kits rely on supported entry points.

What Has Changed

  • Removed KitIntegration.getAllUserAttributes() and its list-flattening behavior
  • Documented migration: use getCurrentUser().getUserAttributes() / FilteredMParticleUser and AttributeListener callbacks
  • Removed unit tests that only asserted the deleted API
  • Added CHANGELOG entry under Unreleased

Screenshots/Video

N/A

Checklist

  • I have performed a self-review of my own code.
  • I have made corresponding changes to the documentation.
  • I have added tests that prove my fix is effective or that my feature works. (Removed tests for deleted API; no new behavior to cover.)
  • I have tested this locally.

Additional Notes

Stacks on chore/remove-kitintegration-getUserIdentities (same line of kit API cleanup as getUserIdentities removal).

@denischilik denischilik requested a review from a team as a code owner March 26, 2026 18:12
@cursor
Copy link

cursor bot commented Mar 26, 2026

PR Summary

Medium Risk
Medium risk because this removes a previously-available (deprecated) kit API and will break compilation for any custom kits still calling it; runtime behavior for supported APIs is unchanged.

Overview
Removes the deprecated KitIntegration.getAllUserAttributes() API (including its list-flattening behavior), pushing kit authors to use getCurrentUser().getUserAttributes() / other FilteredMParticleUser APIs and AttributeListener callbacks.

Updates dataplan blocking instrumentation tests to assert against getCurrentUser()?.userAttributes and deletes the unit test suite that only validated the removed API; adds a corresponding Unreleased changelog entry.

Written by Cursor Bugbot for commit 05d62d5. This will update automatically on new commits. Configure here.

Base automatically changed from chore/remove-kitintegration-getUserIdentities to workstation/6.0-Release March 26, 2026 18:41
- Remove tests that only covered the removed API
- Add CHANGELOG entry for custom kit migration path

Made-with: Cursor
@denischilik denischilik force-pushed the chore/remove-getAllUserAttributes branch from 31eaa16 to 953ec49 Compare March 26, 2026 18:46
…tributes removal

Use getCurrentUser().userAttributes instead of removed allUserAttributes property.

Made-with: Cursor
@sonarqubecloud
Copy link

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