Added additional SupportedEffects and SupportedTimedEffects values#39
Added additional SupportedEffects and SupportedTimedEffects values#39thibauult merged 1 commit intoopenhue:mainfrom
Conversation
…llowing testing with my local hue hub.
📝 WalkthroughWalkthroughTwo YAML schema files were updated to expand their enum value sets. The Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Poem
Pre-merge checks✅ Passed checks (3 passed)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
🧹 Nitpick comments (2)
src/common/SupportedEffects.yaml (1)
2-13: Consider alphabetical ordering for maintainability.The enum values could be ordered alphabetically to improve consistency and maintainability. This would make it easier to spot duplicates and find specific values.
🔎 Optional: Alphabetically ordered enum
type: string enum: - - prism - - opal - - glisten - - sparkle + - candle + - cosmos + - enchant - fire - - candle + - glisten - no_effect + - opal + - prism + - sparkle + - sunbeam - underwater - - cosmos - - sunbeam - - enchantsrc/light/schemas/SupportedTimedEffects.yaml (1)
3-6: Consider alphabetical ordering for consistency.Similar to the other schema file, alphabetical ordering of enum values could improve maintainability.
🔎 Optional: Alphabetically ordered enum
enum: + - no_effect - sunrise - sunset - - no_effect
📜 Review details
Configuration used: defaults
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (2)
src/common/SupportedEffects.yamlsrc/light/schemas/SupportedTimedEffects.yaml
🔇 Additional comments (1)
src/common/SupportedEffects.yaml (1)
10-13: LGTM! New effect values added correctly.The four new effect values (underwater, cosmos, sunbeam, enchant) are officially supported in the Philips Hue API v2 (announced February 2025 under the effects_v2 feature). They follow the existing snake_case naming convention matching the legacy effects pattern, and are syntactically correct. These additions prevent validation failures in client libraries when interacting with Hue hubs that expose these effects.
thibauult
left a comment
There was a problem hiding this comment.
LGTM 👍 Thanks for your contribution!
My local hue hub reports these SupportedEffects and SupportedTimedEffects values. Extending these enums with these additional values avoids validation failures when using a client library built from the OpenHue API.
Summary by CodeRabbit
New Features
✏️ Tip: You can customize this high-level summary in your review settings.