Skip to content

fix(hermes): validate briefing state and event overlap - #76

Merged
furedea merged 1 commit into
mainfrom
fix/secretary-briefing-state
Aug 31, 2026
Merged

fix(hermes): validate briefing state and event overlap#76
furedea merged 1 commit into
mainfrom
fix/secretary-briefing-state

Conversation

@furedea

@furedea furedea commented Aug 31, 2026

Copy link
Copy Markdown
Owner

Summary

  • treat calendar events as interval overlaps so ongoing multi-day events remain visible
  • require newly surfaced research and technology identifiers to match the final report
  • preserve prior digest state when atomic writes or read-back validation fail
  • prohibit direct non-atomic state-write fallbacks

Test plan

  • bats --print-output-on-failure tests/nix/hermes_secretary.bats
  • lefthook pre-commit checks
  • verify the secretary profile uses its profile root as terminal.cwd without the deprecated TERMINAL_CWD setting

Copilot AI lite review requested due to automatic review settings August 31, 2026 13:06

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@furedea
furedea merged commit 7dab857 into main Aug 31, 2026
11 checks passed
@furedea
furedea deleted the fix/secretary-briefing-state branch August 31, 2026 13:12
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.

2 participants