Read the work calendar as a notification signal, not a mailbox (#126) #57
Closed
claude
wants to merge 1 commits from
overnight/ambient-calendar into overnight/local-calendar
pull from: overnight/ambient-calendar
merge into: kami:overnight/local-calendar
kami:master
kami:task/725-capability-ledger-and-empirical-baseline
kami:task/692-heads-path-may-equal-model-path-and-noth
kami:task/694-staticcheck-and-deadcode-are-still-not-i
kami:task/682-go-1-25-5-and-x-text-0-14-0-carry-20-rea
kami:task/674-caveats
kami:task/673-mavgpud-serves-the-model-to-the-whole-la
kami:task/487-capture-device-doc
kami:task/487-capture-device
kami:task/487-wake-word-deploy
kami:task/487-wake-word-threshold
kami:task/487-wake-word-stage-two
kami:task/671-mavwaked-registers-as-a-voice-consumer-i
kami:task/670-cut-claude-md-to-200-lines
kami:task/515-deploy-mavwaked-workpc
kami:task/669-prune-claude-md
kami:task/668-e4b-phrasing
kami:task/668-title-capital
kami:task/668-kiwix-answers-a-question-it-cannot-answe
kami:task/666-only-a-stage-0-grammar-may-take-the-pers
kami:task/487-mavwaked-has-no-wake-word-only-an-energy
kami:task/486-deploy-the-workstation-transcriber
kami:task/486-move-stt-and-tts-to-the-workstation-wher
kami:task/665-crisperwhisper-2-russian
kami:task/664-routing-heads-in-go
kami:task/662-usage-harness-source-badge
kami:task/661-post-merge-usage-rerun
kami:task/661-routing-heads-step-3-train-the-multi-hea
kami:task/660-router-prompt-destination
kami:task/659-destination-fixture
kami:task/655-query-source-is-a-routing-decision-made
kami:task/654-a-pending-clarify-has-no-way-out-neither
kami:task/654-week-of-usage-eval-docs
kami:task/649-needs-kami-telegram-is-the-only-reach-an
kami:task/643-memorystore-search-decodes-and-unmarshal
kami:task/641-two-maps-grow-for-the-process-lifetime-w
kami:task/644-mavcaldav-is-built-documented-as-running
kami:task/642-the-store-caps-sqlite-at-one-connection
kami:task/647-factenrichmentworker-walks-the-pending-q
kami:task/646-v-637-follow-up-telegram-intake-has-no-d
kami:task/638-no-deadline-survives-the-turn-path-from
kami:task/637-inbound-telegram-turns-and-corrections-f
kami:task/636-correcting-a-turn-from-telegram-and-from
kami:task/634-an-act-alias-resolves-the-verb-but-not-t
kami:task/630-one-gesture-correction-on-chat-v-628
kami:task/629-persist-the-routing-trace-and-record-it
kami:task/631-mode-inventory-written-from-the-handlers
kami:task/586-defaultfactparser-uses-hand-written-russ
kami:task/633-reconcile-the-seed-labels-with-the-handl
kami:task/627-reminder-verbs-has-no-alarm-verb-so-an-a
kami:task/626-the-classifier-seeds-teach-an-older-inte
kami:task/546-route-with-a-fine-tuned-e5-small-instead
kami:task/586-measure-the-fact-parser
kami:fix/gofmt-ecosystem-acts
kami:task/584-media-store-a-failed-write-leaks-its-bud
kami:task/518-no-write-path-for-a-backdated-event-so-t
kami:task/287-qa-voice-session-quality-polish
kami:task/492-qa-plan-reconcile
kami:task/530-sweep-tail-four-files-the-russian-sweep
kami:task/405-score-how-often-a-real-utterance-reaches
kami:task/529-money-and-list-pick-a-mechanism
kami:task/528-sweep-tail-the-three-files-on-467
kami:task/527-embedder-open-set-phrasings-stop-being-r
kami:task/526-morphology-a-dictionary-answers-the-gram
kami:task/525-lexicons-the-finite-russian-sets-move-to
kami:task/524-entity-reference-ask-nexus-about-every-l
kami:task/523-risk-tiers-take-hexis-s-tier-for-a-hexis
kami:task/521-review-pr-111-query-strings-declension-h
kami:task/491-llama-server-core-dumps-on-every-sigterm
kami:task/479-bug-an-unconfigured-capability-does-not
kami:task/467-bug-spoken-task-capture-is-dead-the-rout
kami:task/463-deploy-mavwaked-and-mavenclient-run-nowh
kami:task/480-hearing-no-shipped-client-can-start-a-re
kami:task/432-ambient-calendar-intake-is-fragile-and-p
kami:task/431-board-surface-maven-holds-the-work-board
kami:task/433-reactivehandler-has-30-fields-and-is-pas
kami:task/371-swap-the-embedder-for-an-asymmetric-retr
kami:task/408-review-31-07-split-the-30-method-coreapi
kami:task/410-review-31-07-hand-rolled-string-enums-st
kami:task/423-review-pr50-split-internal-ipc-server-go
kami:task/422-review-pr50-split-cmd-mavend-tick-go-860
kami:task/409-review-31-07-finish-moving-mavweb-markup
kami:task/482-ambient-ingest-reads-a-notification-s-ti
kami:task/444-kuma-a-fact-per-monitor-so-she-can-name
kami:task/452-capability-model-homelab-docker-restart
kami:task/449-destructive-confirm-policy-risk-tiers-no
kami:task/453-grocery-list-items-table-fourth-append-o
kami:task/399-run-the-persona-checks-inside-the-daemon
kami:task/448-bounded-follow-up-state-pending-candidat
kami:task/455-conversation-repair-name-the-misroute-co
kami:task/454-go-mod-tidy
kami:task/458-pronunciation-dictionary-for-piper
kami:task/456-command-history-read-only-query-over-exi
kami:task/457-clarification-templates-for-the-router-s
kami:task/474-query-source-ordering-feeds-and-calendar
kami:task/469-reminders-spelled-out-times-fail-the-bod
kami:task/475-bug-the-praxis-attention-capability-is-u
kami:task/481-bug-a-transient-complaint-is-stored-as-a
kami:task/476-bug-the-router-transliterates-latin-enti
kami:task/385-decide-whether-a-parked-clarify-question
kami:task/377-backfill-routines
kami:task/421-weather-geocoder
kami:task/390-no-read-path-for-delivery-attempts
kami:task/386-recall-fixture-filler-note-ids
kami:task/473-bug-morning-item-has-no-required-flag
kami:task/465-bug-make-simulate-routes-with-an-empty
kami:task/467-bug-spoken-task-capture-is-dead
kami:task/466-bug-a-pending-clarify-is-global-so-one-u
kami:task/468-bug-pattern-detect-has-no-minimum-interv
kami:task/462-bug-checkfeminine-flags-second-person-ma
kami:task/443-safekey-drops-cyrillic-so-russian-calend
kami:task/471-bug-agendaquerygrammars-covers-today-but
kami:task/383-slottext-in-clarify-answer-would-clobber
kami:task/323-qa-phraser-coverage-is-65-3-but-the-llam
kami:task/498-bug-and-x-reach-the-model-with-no-determ
kami:task/506-strings-family-6-summaries-and-reports-i
kami:task/504-strings-family-4-act-and-smart-home-repl
kami:task/503-strings-family-3-query-answers-and-gaps
kami:task/502-strings-family-2-capture-acknowledgement
kami:task/501-strings-family-1-phrasing-fallbacks-into
kami:task/397-phrasechat-and-phrasequery-hide-model-fa
kami:task/396-the-reply-path-can-t-be-tested-llmreplie
kami:task/496-recall-a-cross-language-question-loses-i
kami:task/495-bug-x-escapes-the-personal-boundary-and
kami:task/499-llama-server-holds-7-9gb-rss-for-a-1-1gb
kami:task/470-bug-a-question-writes-invented-knowledge
kami:task/493-bug-the-memory-index-stores-the-raw-utte
kami:task/490-name-the-gap-world-questions-through-the
kami:task/485-run-the-big-model-on-the-workstation-wit
kami:task/489-workstation-deploy-mavgpud-on-workpc-and
kami:task/488-workstation-a-supervisor-that-keeps-llam
kami:task/483-docs-offload-design
kami:task/483-design-offload-ml-to-the-workstation-kee
kami:task/459-docs-refresh-the-qa-plan-against-the-liv
kami:task/446-doc-reorg-tier-the-tree-retire-the-three
kami:fix/367-voice-parks-routine-accept
kami:task/365-dialogue-slots-and-router-slots-are-hand
kami:task/364-snooze-does-nothing-at-runtime-the-gate
kami:task/447-retire-progress-md-the-backlog-and-the-f
kami:task/445-session-workflow
kami:overnight/eco-versioned-traces
kami:overnight/eco-entity-refs
kami:overnight/eco-degraded-suite
kami:overnight/netscan
kami:overnight/smarthome
kami:overnight/replay-simulator
kami:overnight/event-envelope
kami:overnight/coldstart-unlock
kami:overnight/voice-barge-in
kami:overnight/stt-golden-audio
kami:overnight/senses-speaker
kami:overnight/senses-hearing
kami:overnight/senses-media-vision
kami:overnight/mcp-tools
kami:overnight/mcp-client
kami:overnight/self-update
kami:overnight/model-swap
kami:overnight/web-crawler
kami:overnight/rss-feeds
kami:overnight/email-poller
kami:overnight/email-extract
kami:overnight/email-imap
kami:overnight/money-zenmoney
kami:overnight/task-priority
kami:overnight/task-capture
kami:overnight/behavior-profile
kami:overnight/day-plan
kami:overnight/local-calendar
kami:overnight/memory-eval
kami:overnight/proactive-proposals
kami:overnight/split-voice-quiet
kami:overnight/nginx-maven-block
kami:overnight/stepup-chat-surface
kami:integration/small-batch
kami:docs/fix-drift
kami:fix/ru-wording
kami:integration/jul31
kami:overnight/resident-1.7b
kami:overnight/nudge-templates
kami:overnight/kiwix-rewrite
kami:overnight/eval-writeup
kami:overnight/fix-truncation
kami:overnight/kiwix-client
kami:overnight/ru-prompts
kami:overnight/external-data
kami:overnight/phrasing-grammar
kami:overnight/talk-eval
kami:overnight/prompt-context
kami:overnight/prompt-address
kami:overnight/eval-label-kill
kami:overnight/delivery-boundary
kami:overnight/address-check
kami:overnight/system-replies-pr
kami:overnight/clock-intent-pr
kami:overnight/embedder-backfill-pr
kami:overnight/embedder-marker-pr
kami:overnight/note-recall-pr
kami:overnight/thinking-off-pr
kami:overnight/dialogue-persist-pr
kami:overnight/persona-2p-pr
kami:overnight/clarify-expiry-pr
kami:overnight/clarify-rework
kami:overnight/phrasing
kami:overnight/bakeoff
kami:overnight/recall-margin
kami:overnight/router-on
kami:overnight/slot-extract
kami:overnight/embedder-e5
kami:overnight/router-refusal
kami:overnight/eval-rerun
kami:overnight/eval-harnesses
kami:overnight/eval-rerun-base
kami:overnight/fmt-gate
kami:overnight/routines-fire
kami:overnight/router-prompt
kami:overnight/away-leak
kami:overnight/recall-eval
kami:overnight/snooze-works
kami:overnight/clarify-wiring
kami:overnight/delivery-tests
kami:overnight/routine-accept
kami:overnight/llm-router-flag
kami:overnight/clarify-data-layer
kami:overnight/loop-rule-tests
Reference in New Issue
Block a user
Blocking a user prevents them from interacting with repositories, such as opening or commenting on pull requests or issues. Learn more about blocking a user.
Delete Branch "overnight/ambient-calendar"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
What
The work calendar is read without a work credential. Per the task body: a corp mail or calendar session on the homelab ties the box's blast radius to the employer's data — don't. So maven reads the signal instead.
internal/calendar/ambient.go—NotificationandEventFromNotification. Pure parse, no HTTP, no store.cmd/mavweb/ambient.go—POST /api/ambient. An Android notification-listener relays meeting notifications over wg/LAN; the ones that clearly describe a meeting become facts atsource=ambient:notif,confidence=0.6.internal/store/facts.go—CalendarEventsnow reads every calendar source instead of onlypoll:caldav, and keeps each row's source and confidence so the caller can judge them. A non-calendar source still cannot forge a calendar event.internal/router/calendar.go—CalendarEntry+FormatEntries.Formatis unchanged in behaviour (it delegates with everything marked certain).cmd/mavend/actions_query.go—queryCalendarmarks events below full confidence as uncertain.Provenance is the feature
A notification is evidence about a meeting, not a reading of a calendar, so it is never indistinguishable from one. Stored below 1.0, and recited hedged:
The parse refuses to guess
It needs a real clock reading and a summary that is not just that clock reading. Otherwise it stores nothing:
Планёрка/10:00-10:30Встреча с подрядчиком в 14:00Sprint review/Today 16:00 – 17:00, MeetСозвон/с 11:30 до 12:153 новых письмаПланёрка в 14Входящие/12 непрочитанныхОтчёт/срок 2026.08.1510:00-10:30with no wordsA mailbox of noise rendered as invented meetings is worse than a gap.
Off unless configured
No
-ambient-token, no route registered — an unconfigured ingest does not exist rather than 503ing at probes. The token is a shared secret compared in constant time (Authorization: BearerorX-Maven-Token), because the poster is a background Android service and WebAuthn has no answer for one. The endpoint is write-only, accepts exactly one shape of write, and cannot read anything back out. Reposts dedupe against the latest fact for that key+source — a phone reposts the same notification many times.Not shipped
The Android notification-listener app itself: a separate artifact and a physical device, not Go in this repo. Everything on maven's side is done and tested; pairing it needs the app.
Verified
make buildandmake testboth exit 0. New tests: 15 parse cases ininternal/calendar(ranges, prepositions, en-dash,до, dotted clocks, midnight crossing, and 6 refusal cases), 10 incmd/mavweb(store, dedupe, ignore non-meetings, 401 on missing/wrong token,X-Maven-Token, 503 when off, 405 on GET, bad JSON, missingposted_at, read error), plus a store test that ambient events are returned with their source and a forged source is not, and a formatter test for hedging.Vikunja #126
@@ -0,0 +15,4 @@// POST /api/ambient — the work calendar read (Vikunja #126).//// Maven does not hold a work credential. A corp mail or calendar session on thewe'll have to think about it in the future, cause right now this is fragile and won't even work, probably.
Reading the work calendar as a relayed signal instead of holding a work credential is the right shape, and the code keeps that promise end to end.
kind=envrather than a self-fact.AmbientConfidencebelow 1.0 rather than pretending to be a CalDAV read. The query path hedging on that confidence. Refusing bare hours, so "в 14" cannot become a meeting. A notification with no clock reading storing nothing at all.subtle.ConstantTimeCompareon the token, and no route registered when the token is unset, so an unconfigured ingest is not a 503 to probe.Your comment on this one is tracked as Vikunja #432.
Three things.
1. The ambient signal never reaches the busy gate.
handleAmbientwritescalendar_event_*and nothing else.calendar_busyis written only by the CalDAV poller. So Maven now knows about a work meeting well enough to recite it, and not well enough to stop nudging him during it. That is backwards: suppressing a nudge is the lower-risk use of a low-confidence signal, and reciting it is the higher-risk one. The hard part is thatcalendar_busyis a level, not an event, so an ambient writer needs an expiry rather than a single write. Worth a task rather than a change here.2.
/api/ambientis missing from the route table PR 51 just added. That table is the mechanism for making each new route's gate a decision. This is the first route added after it, and it is not listed. The gate itself is fine and arguably better than step-up for a background Android service. Add the row, with "shared secret, no step-up, poster is not a browser" as the reason.3. Every ambient event is dated to the notification's own day.
EventFromNotificationtakesy, m, dfromn.Posted, justified by "a meeting notification is about today or it would not be firing now". Calendar apps break that. A 21:00 "Tomorrow at 09:00" reminder becomes an event at 09:00 today, which is in the past, andFactKeyfiles it under today's date. It stores a wrong meeting rather than nothing, which is the one outcome the rest of this file works hard to avoid. Recognising "завтра" and "tomorrow" would cover most of it. Refusing a start time more than a few hours in the past would cover the rest.Smaller:
strings.TrimPrefix(..., "Bearer")is case-sensitive, so a client sendingbearer <tok>falls through to theX-Maven-Tokenbranch and gets a 401. Not wrong, but the failure is silent from the phone's side.strings.EqualFoldon the scheme costs nothing.Landed on master. The stack was one linear chain, so #84 carried every commit from #50 up, and master now contains this branch in full. Merging this PR on its own is an empty diff, so it is closed rather than merged. The review findings for it were fixed in the 2026-08-01 pass and are on master as commits on the stack tip, not on this branch.
Pull request closed