Skip to content

[nightly-review] Keep Apple summaries tied to transcript participants#1069

Draft
r3dbars wants to merge 1 commit into
mainfrom
codex/nightly-review-local-summary-participants
Draft

[nightly-review] Keep Apple summaries tied to transcript participants#1069
r3dbars wants to merge 1 commit into
mainfrom
codex/nightly-review-local-summary-participants

Conversation

@r3dbars

@r3dbars r3dbars commented Jun 11, 2026

Copy link
Copy Markdown
Owner

Summary

  • keep local summary participant metadata sourced from transcript speaker labels for every provider
  • apply the same transcript-derived participant override to the Apple on-device summary path that Gemma already used
  • add a fast regression covering provider output that omits participants

Nightly evidence

  • recent health signal still shows meeting degradation/speaker-finalization events, but those are pre- or not-yet-shipped relative to yesterday's instrumentation patch
  • the highest-confidence current-main bug found tonight was in the new local summary provider split: Apple summaries could write None found. participants even when the transcript had canonical speaker labels

Verification

  • bash scripts/dev/agent-preflight.sh Sources/Meeting/LocalMeetingSummarizer.swift Tests/LocalMeetingSummarizerTests.swift
  • bash build-deps.sh --force
  • bash build.sh --no-open
  • bash run-tests.sh (4,656 passed)
  • bash run-integration-smoke.sh
  • /Users/redbars/.codex/skills/codex-review/scripts/codex-review --mode local --output /tmp/transcripted-nightly-codex-review-20260611.txt (clean)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant