Conversation
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
| consumer_id=data.get('consumer_id'), | ||
| created_at=datetime.fromisoformat(data.get('created_at').replace('Z', '+00:00')) if data.get('created_at') else None, | ||
| updated_at=datetime.fromisoformat(data.get('updated_at').replace('Z', '+00:00')) if data.get('updated_at') else None, | ||
| surface=mapConsumersProfilesGetOutputSurface.from_dict(data.get('surface')) if data.get('surface') else None |
There was a problem hiding this comment.
Missing mapper classes cause runtime NameError on deserialization
High Severity
mapConsumersProfilesGetOutput.from_dict references mapConsumersProfilesGetOutputGroups and mapConsumersProfilesGetOutputSurface, but neither mapper class is defined in the file or imported. The same issue exists for the DashboardInstance variant. The dataclass definitions for the nested types exist, but the corresponding mapper classes needed for deserialization were not generated. This will cause a NameError at runtime whenever consumer profile data containing groups or surface is deserialized.
Additional Locations (1)
1 participant
Note
Medium Risk
Medium risk because it expands and reshapes the generated client API surface (new endpoints, new request params, and removal/migration of several
api_keysmethods), which may be breaking for SDK consumers even though changes are largely mechanical and generated.Overview
Adds generated SDK support for consumers and consumer surfaces across top-level,
dashboard instance, andmanagement instancenamespaces, including list/get/create/update flows plus consumer profile listing.Extends several existing list/create endpoints with new consumer-related filters/fields (e.g.,
consumer_id,consumer_profile_id,actor_id, identity filters) and updates identity actor models to include aconsumerfield; also simplifies identity-actor list item typing to rawDictitems.Restructures API key support by trimming
MetorialApiKeysEndpointdown torotate/revealand introducing dedicateddashboard_organizations_api_keysandmanagement_organization_api_keysendpoints for list/get/create/update/revoke, alongside updated exports/initializers.Written by Cursor Bugbot for commit 6c544b1. This will update automatically on new commits. Configure here.