Skip to content

fix: populate summary template from hook payload fallbacks#477

Merged
garysheng merged 1 commit intoPeonPing:mainfrom
pandego:fix/476-summary-template
Apr 19, 2026
Merged

fix: populate summary template from hook payload fallbacks#477
garysheng merged 1 commit intoPeonPing:mainfrom
pandego:fix/476-summary-template

Conversation

@pandego
Copy link
Copy Markdown
Contributor

@pandego pandego commented Apr 19, 2026

Fixes #476

Summary

  • populate {summary} from common hook payload fallback fields used by Claude, Codex, Gemini, and notification events
  • reuse the same fallback order in the Windows hook path so templates behave the same cross-platform
  • add regression coverage for the Claude, Codex, and Gemini stop payload variants

Testing

  • bats tests/peon.bats -f 'TTS:' (via temporary local bats-core install)

@vercel
Copy link
Copy Markdown

vercel Bot commented Apr 19, 2026

@pandego is attempting to deploy a commit to the Gary Sheng's projects Team on Vercel.

A member of the Team first needs to authorize it.

@garysheng garysheng merged commit 0693fa9 into PeonPing:main Apr 19, 2026
2 of 3 checks passed
garysheng added a commit that referenced this pull request Apr 19, 2026
Features:
- feat: IDE rules and exclude_dirs pack selection (#475, closes #298)

Fixes:
- fix: populate summary template from hook payload fallbacks (#477, closes #476)
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.

{summary} template variable empty for Claude / Codex / Gemini turn-complete hooks

2 participants