Skip to content

RUM-12199: Device Capabilities Attributes Refactoring#337

Merged
marco-saia-datadog merged 3 commits into
masterfrom
marcosaia/RUM-12199/device-capabilities-refactoring
Jan 13, 2026
Merged

RUM-12199: Device Capabilities Attributes Refactoring#337
marco-saia-datadog merged 3 commits into
masterfrom
marcosaia/RUM-12199/device-capabilities-refactoring

Conversation

@marco-saia-datadog

Copy link
Copy Markdown
Member

What does this PR do?

This PR refines device capability attribute names to improve clarity and reduce ambiguity.

processor_countlogical_cpu_count

The attribute was renamed to explicitly reflect what is actually reported by mobile platform APIs.

  • On Android, Runtime.availableProcessors() returns the number of logical CPU cores available to the runtime.
  • On iOS, ProcessInfo.processInfo.processorCount likewise reports the number of logical (not physical) cores.

Because both platforms expose logical core counts, the new name and updated description make this behavior explicit and avoid confusion with physical CPU cores.

is_low_ram_deviceis_low_ram

The _device suffix was removed because the attribute already lives under the device parent, making the original name redundant.

@marco-saia-datadog
marco-saia-datadog requested review from a team as code owners January 8, 2026 09:10
@marco-saia-datadog
marco-saia-datadog force-pushed the marcosaia/RUM-12199/device-capabilities-refactoring branch from fa44516 to 11d1885 Compare January 8, 2026 15:08

@samcolmanetti samcolmanetti left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Looks great!

@marco-saia-datadog
marco-saia-datadog merged commit 32918d9 into master Jan 13, 2026
12 checks passed
@marco-saia-datadog
marco-saia-datadog deleted the marcosaia/RUM-12199/device-capabilities-refactoring branch January 13, 2026 15:22
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.

3 participants