Skip to content

fix(snapshots): Use snake_case for sidecar metadata JSON keys#1157

Merged
runningcode merged 1 commit intomainfrom
no/snapshots-sidecar-snake-case
Apr 21, 2026
Merged

fix(snapshots): Use snake_case for sidecar metadata JSON keys#1157
runningcode merged 1 commit intomainfrom
no/snapshots-sidecar-snake-case

Conversation

@runningcode
Copy link
Copy Markdown
Contributor

@runningcode runningcode commented Apr 21, 2026

Summary

Normalize every key in the per-snapshot sidecar metadata JSON to snake_case. Previously only display_name and image_file_name were snake_case while the other ten keys (className, methodName, previewName, nightMode, fontScale, apiLevel, widthDp, heightDp, showSystemUi, showBackground) used camelCase, forcing downstream consumers to handle both conventions.

#skip-changelog (None of the snapshot features are mentioned in the changelog)

Normalize all keys in the per-snapshot sidecar metadata JSON to
snake_case. Previously only `display_name` and `image_file_name` were
snake_case while the remaining ten keys used camelCase, forcing
downstream consumers to handle both conventions.
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 21, 2026

Semver Impact of This PR

🟢 Patch (bug fixes)

📋 Changelog Preview

This is how your changes will appear in the changelog.
Entries from this PR are highlighted with a left border (blockquote style).


This PR will not appear in the changelog.


🤖 This preview updates automatically when you update the PR.

@runningcode runningcode merged commit df80572 into main Apr 21, 2026
21 of 23 checks passed
@runningcode runningcode deleted the no/snapshots-sidecar-snake-case branch April 21, 2026 09:58
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.

2 participants