Skip to content

[nightly-review] Prune old failed meeting audio#818

Merged
r3dbars merged 1 commit into
mainfrom
codex/nightly-review-failed-meeting-cleanup-20260520
May 20, 2026
Merged

[nightly-review] Prune old failed meeting audio#818
r3dbars merged 1 commit into
mainfrom
codex/nightly-review-failed-meeting-cleanup-20260520

Conversation

@r3dbars
Copy link
Copy Markdown
Owner

@r3dbars r3dbars commented May 20, 2026

Summary

  • Wires the existing FailedTranscriptionManager age cleanup into MeetingSessionController startup.
  • Adds a named 30-day failed-meeting audio retention constant.
  • Adds fast regression coverage so the cleanup hook does not disappear again.

Nightly evidence

Verification

  • bash build-deps.sh --force
  • bash build.sh --no-open
  • bash run-tests.sh
  • bash run-integration-smoke.sh

@r3dbars
Copy link
Copy Markdown
Owner Author

r3dbars commented May 20, 2026

verdict: no-findings-but-human

findings: none from this pass.
actions: human signoff recommended because this is draft behavior that deletes old failed-meeting audio after the 30-day retention window.
head_sha: 4323652
fix_commit: none
checks: gh pr view, gh pr diff, Meeting/Support/Test docs read, FailedTranscriptionManager cleanup path reviewed, git diff --check origin/main...origin/codex/nightly-review-failed-meeting-cleanup-20260520; GitHub checks reported none.

manual_merge_handoff:
head_sha: 4323652
verdict: no-findings-but-human
blockers: draft PR plus destructive failed-audio retention behavior needs human signoff
checks: gh metadata/diff, local docs/static review, cleanup path review, diff check

@r3dbars r3dbars marked this pull request as ready for review May 20, 2026 16:45
@r3dbars r3dbars merged commit 5ecd217 into main May 20, 2026
@r3dbars r3dbars deleted the codex/nightly-review-failed-meeting-cleanup-20260520 branch May 20, 2026 16:45
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