[cloudsmith-iduffy]

Description

Sentry CLOUDSMITH-CLI-1V is a TypeError from cloudsmith check rates on 1.30.1: float() argument must be a string or a real number, not 'NoneType'.

get_rate_limits() turns the SDK response into a dict with to_dict(). The SDK model puts every field in that dict, and sets a field that the API leaves out to None:

>>> ResourcesRateCheck(resources={"core": RateCheck(limit=600)}).to_dict()
{'resources': {'core': {'interval': None, 'limit': 600, 'remaining': None, 'reset': None, 'reset_iso_8601': None, 'throttled': None}}}

RateLimitsInfo.from_dict checked "interval" in data, which is always true for this dict, and then called float(None). Each check is now data.get(key) is not None, so a missing value stays None. The table shows None for it, as it already does when the rate limit headers are missing. throttled still falls back to remaining == 0.

Linear: ENG-15482. Fixes CLOUDSMITH-CLI-1V

Type of Change

• Bug fix

Additional Notes

The patch in ENG-15482 was correct. This PR keeps its fix and cuts the tests to the one case that fails before the fix.

🤖 Generated with Claude Code

[Copilot]

🟢 Approval recommended

No active review issues remain for the fix to null rate-limit handling.

No open issues 🎉

What changed in this PR

Fixes cloudsmith check rates crashing when the API omits rate-limit values that the SDK represents as None.

Changes:

• Skip conversions for missing or null values while preserving the remaining == 0 fallback for throttled status.
• Add a regression test covering null fields.
• Document the fix in the changelog.

File	Description
cloudsmith_cli/​core/​tests/​test_ratelimits.py	Tests null-field handling and the throttled fallback.
cloudsmith_cli/​core/​ratelimits.py	Guards conversions against None values.
CHANGELOG.md	Records the rate-limit display fix.

_{Review effort: Balanced}

[cloudsmith-iduffy]

@copilot resolve the merge conflicts in this pull request

[Copilot]

@copilot resolve the merge conflicts in this pull request

Resolved in 8a152c3 by merging master and retaining both sets of changelog entries. The rate-limit fix and regression test are unchanged. The regression test passed when invoked directly; pytest is unavailable in this environment.

[Copilot]

🟢 Approval recommended

No active review issues block the fix for crashes caused by missing or null rate-limit values.

No open issues 🎉

_{Review effort: Balanced}