Repository navigation
fix(vision): stop background captures in Kid Mode and at the source - #215
Merged
Merged
Conversation
Every idle face_detected without a cached description asked the robot for a photo. The 120 s cooldown in dotty-behaviour only ran after the upload, and its "no one in view" reply left the cache empty, so the next detection captured again: about 160 captures in an hour, while lowest free SRAM fell to 1.2 KB. Check the cooldown before the MCP call, per device so a reconnect doesn't reset it. Kid Mode says the camera is off, but room_view and the idle photographer still captured. Gate both with the fail-closed camera policy. Security mode's photos are unchanged. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
BrettKinny
added a commit
that referenced
this pull request
Nov 9, 2026
…216) Adds the joke hint, the background-capture and Kid Mode camera fixes, the mcp-call admin route, the MCP/ambient checks and --rotate, and records that the noise-close defaults were confirmed on 2026-11-02. Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What
_capture_room_description_asyncchecks the camera policy and a per-device 130 s cooldown before asking the robot for a photo.take-photoroute are unchanged.Why
The robot ran about 160 background captures in an hour, because
dotty-behaviour's 120 s cooldown only runs after the upload, and its "no one in view" reply leaves the cache empty. Lowest free SRAM fell to 1.2 KB during that session; the link isn't proven yet. Kid Mode also told users the camera was off while these captures ran.Effect on behaviour
In Kid Mode, room-view person matching and the
[Room view]context stop, as does the idle perception log.Tests
tests/test_voice_camera_policy.pyandtests/test_reconnect_state_restore.py: 23 passed.dotty-behaviour/tests: 317 passed. The new idle photographer test fails with the gate removed.Deployed to the tower ahead of merge at the maintainer's request.
AI-assisted: written by Claude Opus 5.5; needs human review before merge.
🤖 Generated with Claude Code