Skip to content

Propagate publish failures and redact fetch errors - #61

Merged
walker1211 merged 1 commit into
mainfrom
codex/fix-publish-hook-errors
Aug 10, 2026
Merged

Propagate publish failures and redact fetch errors#61
walker1211 merged 1 commit into
mainfrom
codex/fix-publish-hook-errors

Conversation

@walker1211

Copy link
Copy Markdown
Owner

Summary

  • Return publish-hook failures to the caller and update seen state only after post-Markdown actions succeed.
  • Redact credentials from failed-source console output, source stats, and email content.
  • Restrict history secret scanning to publishable refs so tool-internal refs do not cause false positives.
  • Ensure the scheduler heartbeat test stops its background watcher before temporary state cleanup.

Test plan

  • scripts/ci-local.sh clean
  • go test ./cmds/news-briefing -run TestScheduledBriefingCronWaitsWhileXHeartbeatFresh -count=50

@walker1211
walker1211 merged commit 9283e88 into main Aug 10, 2026
3 checks passed
@walker1211
walker1211 deleted the codex/fix-publish-hook-errors branch August 10, 2026 13:31
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