Rename DatabaseDict to CloudDatabaseDict#2967
Conversation
DatabaseDict represents a cloud database in the target manager. This rename clarifies that there are three database types: Device, Cloud, and VuMark. Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
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.
|
|
||
| @dataclass(frozen=True) | ||
| class VuMarkVuforiaDatabase: | ||
| class VuMarkCloudDatabase: |
There was a problem hiding this comment.
VuMark types incorrectly renamed from Vuforia to Cloud
Low Severity
VuMarkVuforiaDatabase was renamed to VuMarkCloudDatabase, and _VuMarkVuforiaDatabaseSettings to _VuMarkCloudDatabaseSettings. The PR description explicitly states there are three separate database types: Device, Cloud, and VuMark. "Vuforia" in the original names referred to the platform, not the Cloud database type. Calling VuMark types "Cloud" is semantically misleading since VuMark is explicitly a distinct type from Cloud. The fixture name vumark_vuforia_database and env_prefix="VUMARK_VUFORIA_" were left unchanged, creating an inconsistency within the same file.


Summary
DatabaseDicttoCloudDatabaseDictto clarify it represents a cloud database typeVuforiaDatabasetoCloudDatabasefor the same reasonThere are three database types in the target manager: Device, Cloud, and VuMark.
🤖 Generated with Claude Code
Note
Medium Risk
Public API renames (
VuforiaDatabase/DatabaseDict->CloudDatabase/CloudDatabaseDict) can break downstream imports and type references. Behavior appears unchanged, but the surface-level rename touches many modules and tests.Overview
Renames the core database model from
VuforiaDatabasetoCloudDatabaseand the associated TypedDict fromDatabaseDicttoCloudDatabaseDictto clarify these represent cloud databases.Propagates the rename through request matchers, Flask server apps, validators,
MockVWS/TargetManagerAPIs, docs/examples, and tests, plus updates static-analysis ignore lists inpyproject.toml.Written by Cursor Bugbot for commit fef9aa3. This will update automatically on new commits. Configure here.