Skip to content

Pin Home meeting artifact row targets#1123

Merged
r3dbars merged 1 commit into
mainfrom
codex/meeting-artifact-integrity
Jun 15, 2026
Merged

Pin Home meeting artifact row targets#1123
r3dbars merged 1 commit into
mainfrom
codex/meeting-artifact-integrity

Conversation

@r3dbars

@r3dbars r3dbars commented Jun 14, 2026

Copy link
Copy Markdown
Owner

Summary

  • Add a small Home meeting row target helper so transcript/audio Finder actions resolve through one tested path.
  • Route saved and failed meeting reveal actions through that helper, while keeping retranscribe gated on retained audio that has a real Finder target.
  • Add deterministic fast coverage for exact transcript/audio reveal targets and failed-audio visibility vs retry readiness.

Validation

  • scripts/dev/agent-preflight.sh
  • bash -n run-tests.sh
  • bash -n scripts/entrypoints/run-tests.sh
  • bash build-deps.sh --force (needed because build guard reported missing/stale deps)
  • bash build.sh --no-open
  • bash run-tests.sh --filter RecentCaptureScanners
  • bash run-tests.sh --filter FailedMeetingPresentation
  • bash run-tests.sh --filter UIAutomationSurfaceContract
  • bash run-tests.sh (5304/5304 passed)

Review

  • codex review --base origin/main was attempted, but local Codex usage was exhausted before a verdict.
  • Fallback independent review: Claude Code 2.1.157 reviewed the branch diff. It found only P3/low notes; the retranscribe/audio-target guard note was addressed before final validation. Remaining low note: audio reveal intentionally selects the first retained audio URL, preserving existing behavior.

Notes

@r3dbars r3dbars marked this pull request as ready for review June 15, 2026 15:26
@r3dbars r3dbars merged commit 7c04249 into main Jun 15, 2026
3 checks passed
@r3dbars r3dbars deleted the codex/meeting-artifact-integrity branch June 15, 2026 15:26
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