Skip to content

fix: run remi-diary summary via codex exec with safe prompt piping#1

Merged
lsj5031 merged 1 commit intomasterfrom
fix/remi-diary-codex-exec
Feb 22, 2026
Merged

fix: run remi-diary summary via codex exec with safe prompt piping#1
lsj5031 merged 1 commit intomasterfrom
fix/remi-diary-codex-exec

Conversation

@lsj5031
Copy link
Owner

@lsj5031 lsj5031 commented Feb 22, 2026

Summary

  • switch scripts/remi-diary.sh summary generation from pi prompt argument passing to codex exec
  • feed prompt via stdin and capture final summary via --output-last-message temp file
  • add robust fallback when summary command is missing/fails so diary generation still succeeds
  • update README warning text to reflect configurable summary provider

Why

  • avoids argument length overflow from passing very large prompts as command args
  • removes hard dependency on pi API key for diary summary generation path
  • preserves daily diary output even when AI summary step fails

Verification

  • scripts/remi-diary.sh --date yesterday (in this environment codex is missing; script now degrades gracefully)
  • DIARY_SUMMARY_CMD='nonexistent-cmd' scripts/remi-diary.sh --date yesterday (confirmed fallback summary + explicit error in markdown)

@lsj5031 lsj5031 merged commit d102933 into master Feb 22, 2026
2 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