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 Plus 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. 🔍 LintChecking the alignment of your contribution. 📏 ❌ ruff: issues found — see job log 📋 Repo HealthEnsuring the repository remains a happy place. 😊 ✅ All required files present. Latest Version: ✅ ⚖️ License CheckThe license check is now finished. 🏁 ✅ No license violations found. Policy: Apache 2.0 (universal donor). StrongCopyleft / NetworkCopyleft / WeakCopyleft / Other / Error categories fail. MPL allowed. 🔒 Security (pip-audit)Checking if we're following security best practices. 📏 ✅ No known vulnerabilities found (74 packages scanned). 🔨 Build TestsBuild test complete! Let's see if everything fits together. 🧩 ✅ All versions pass
The silent guardian of the dev branch. 🦇 |
7473e2d to
a359cc9
Compare
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>
a359cc9 to
3fa046f
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