Skip to content

docs(devbook): specify meeting recording on a phone note - #1710

Merged
github-actions[bot] merged 1 commit into
plan/meeting-recordingfrom
meeting-recording/90bc0d51-e8f6-4272-a0e1-a856fa33a047
Nov 10, 2026
Merged

github-actions[bot] merged 1 commit into
plan/meeting-recordingfrom
meeting-recording/90bc0d51-e8f6-4272-a0e1-a856fa33a047

Conversation

@JSdotNet

Copy link
Copy Markdown
Owner

What changed

The recording half of the meeting-recording design (artifact), written into the devbook through delivery:flow-spec. Plan +meeting-recording, entry spec-recording (90bc0d51-e8f6-4272-a0e1-a856fa33a047).

File Change
.devbook/domain/capture/features.md New ### Meeting recording sub-feature after Talk note, plus a pointer from Talk note
.devbook/domain/inbox/requirements.md New ## Recordings on a note: 6 SHALL requirements, 8 scenarios
.devbook/arc42/06-runtime-view.md New ## Meeting Recording sequence, reusing the staged note-file upload
.devbook/arc42/adr/0048-meeting-audio-is-recorded-as-aac-in-an-adts-stream.md New ADR (proposed) + index line in adr/README.md
.devbook/tech/mobile.md, technology-graph.md New ## Android MediaRecorder chapter (with the microphone foreground service) and its graph node

Acceptance criteria

Criterion How verified
Meeting recording sub-feature: Record beside Dictate and Photo; recording bar + ongoing notification with timer, Pause, Stop; typing and photos keep working; Stop attaches "Recording · " that plays in place; no consent prompt Read against the entry: capture/features.md#meeting-recording
SHALL requirements with scenarios: AAC types; part 2 at 24 MB with no gap; "Recording (interrupted)" on next start; a call pauses until Resume; upload over any connection; deleted note → new note "Recording · " One requirement each in inbox/requirements.md#recordings-on-a-note
Meeting Recording sequence reusing the staged note-file upload 06-runtime-view.md#meeting-recording points at Talk Note Upload and Mobile Note Sync
ADR, next free number: AAC ADTS 16 kHz mono 32 kbps; M4A, Opus, WAV rejected with the artifact's reasons ADR 0048; no other branch claims 0048 (checked every remote branch)
tech/mobile.md entry for MediaRecorder and a microphone foreground service mobile.md#android-mediarecorder
Devbook checks pass node .devbook/_tools/devbook-meta/build.mjs --check exit 0; node tools/devbook/check-metadata.mjs 0 blocking, 651 advisory (unchanged from base)

Build & Test

Devbook-only change: no build or test suite applies. The two devbook checks above pass. No application was started.

Review findings left open

None. An independent architecture:architect review found 1 blocker (Mermaid's reserved note as a participant id), 3 major (unbuilt sync types stated as present fact; talk note used for a meeting) and 5 minor; all nine are fixed in this commit.

What could not be proved

  • The Mermaid sequence renders. No renderer ran; the reserved-word blocker was fixed by reading, not by rendering.
  • The ADR's platform facts (Ogg/Opus from API 29, MP4's index written on stop, WebView2 and Android WebView playing ADTS AAC) come from the artifact and the reviewer's knowledge, not from a test.
  • Rating adopted on Android MediaRecorder rests on its use by cloud dictation (AndroidAudioRecorder); the meeting recorder and RecordingService are not built. Validate that one chapter with one rating fits, or split it.
  • ADR 0048 is proposed: accepting it is this gate's decision.

Assumptions

  • Meeting recording is a ### sub-feature placed right after Talk note, not a #### under it: no features file in the repo nests a sub-feature, and the entry's "under" is read as "beside, extending".
  • The requirements group is named ## Recordings on a note and lives in the Inbox context as the entry says, pointing at the Capture sub-feature through related.
  • The first part is named "Recording · ", the next "Recording · part 2 · ", and parts 3 and on follow the same rule.
  • audio/x-m4a is accepted beside audio/aac and audio/mp4, as SyncTranscriptionLimits already does.
  • Every new chapter says the behaviour is not built yet, so nothing reads as present fact.
  • Drafting ran inline instead of through the per-folder drafting agents, because the entry fixed every fact; the independent review stood in as the check.

Opened by an unattended execute-plan run; Personal Validation is pending on this draft.

🤖 Generated with Claude Code

Meeting recording sub-feature, the requirements for recordings on a note,
the Meeting Recording runtime sequence, ADR 0048 (AAC in an ADTS stream,
16 kHz mono, 32 kbps) and the Android MediaRecorder tech chapter.

Plan +meeting-recording, entry spec-recording (90bc0d51-e8f6-4272-a0e1-a856fa33a047).

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@JSdotNet
JSdotNet marked this pull request as ready for review November 10, 2026 01:09
@github-actions
github-actions Bot merged commit 8693179 into plan/meeting-recording Nov 10, 2026
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant