Skip to content

Preserve manual handoff notes from SessionEnd overwrite#581

Merged
claude-ERA merged 1 commit intomainfrom
fix/session-end-overwrite
Mar 1, 2026
Merged

Preserve manual handoff notes from SessionEnd overwrite#581
claude-ERA merged 1 commit intomainfrom
fix/session-end-overwrite

Conversation

@jonschull
Copy link
Owner

Summary

  • Fixes SessionEnd hook overwrites manual handoff notes #566
  • When the CTO writes a detailed handoff mid-session, the SessionEnd hook was overwriting it with Haiku's lossy summary
  • Now checks if the handoff file was modified within the last 30 minutes — if so, appends Haiku's summary as an addendum instead of replacing

Test plan

  • Write a manual handoff note, then trigger SessionEnd — verify manual content preserved with Haiku addendum appended
  • Wait >30 min (or touch with old mtime), trigger SessionEnd — verify normal overwrite behavior unchanged

🤖 Generated with Claude Code

When the CTO writes a detailed handoff mid-session, the SessionEnd hook
was overwriting it with Haiku's lossy summary. Now checks if the handoff
file was modified within the last 30 minutes — if so, appends Haiku's
summary as an addendum instead of replacing the manual note.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@jonschull jonschull added the auto-merge Non-strategy change: auto-merge after review PASS label Mar 1, 2026
@claude-ERA claude-ERA merged commit 24ce0ab into main Mar 1, 2026
1 check passed
@claude-ERA claude-ERA deleted the fix/session-end-overwrite branch March 1, 2026 23:35
@claude-ERA
Copy link
Collaborator

Auto-merged

PR #581 merged and deployed.

Smoke test: WARNING — see output

Verify: https://era-website.onrender.com

Test output
Traceback (most recent call last):
  File "/Users/admin/era2c/era2/website/tests/smoke_test.py", line 12, in <module>
    from playwright.sync_api import sync_playwright
ModuleNotFoundError: No module named 'playwright'

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto-merge Non-strategy change: auto-merge after review PASS

Projects

None yet

Development

Successfully merging this pull request may close these issues.

SessionEnd hook overwrites manual handoff notes

2 participants