feat: session-aware game-skill + OCP playback#428
Conversation
|
Important Review skippedDraft detected. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
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 |
64c35d7 to
f776228
Compare
f776228 to
7473e2d
Compare
Hey! I've got some fresh data on your pull request. 📈I've aggregated the results of the automated checks for this PR below. 🔍 LintI've gathered the facts for your review. 📖 ❌ ruff: issues found — see job log 📋 Repo HealthI've checked the repo's posture (aka architectural alignment). 🧘 ✅ All required files present. Latest Version: ✅ ⚖️ License CheckEnsuring no copyleft violations in this PR. ⬅️ ✅ No license violations found. Policy: Apache 2.0 (universal donor). StrongCopyleft / NetworkCopyleft / WeakCopyleft / Other / Error categories fail. MPL allowed. 🔒 Security (pip-audit)Ensuring no malicious actors are hitching a ride. 🎭 ✅ No known vulnerabilities found (73 packages scanned). 🔨 Build TestsEnsuring no loose screws in the assembly. 🔩
❌ 3.10: Install OK, tests failed The silent guardian of the dev branch. 🦇 |
Per-session playback state in OVOSCommonPlaybackSkill (state events emit via message.reply so they route to the originating session) and game_skill on the context-gated layers (probing skill_will_match with its session). OCPCommonPlayback deprecation is handled in #423. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
7473e2d to
a359cc9
Compare
Split out from #427. Stacked on #427 (
feat/intent-layers-context) — merge that first.OVOSCommonPlaybackSkill— state events emit viamessage.replyso they route back to the originating session (HiveMind-friendly).game_skill.pyadopts the context-gated IntentLayers.OVOSCommonPlaybackSkilldeprecation is handled separately in feat: deprecate OVOSCommonPlaybackSkill in favor of MediaProvider plugins #423 (not duplicated here).488 tests pass locally.
🤖 Generated with Claude Code