Skip to content

Phase 4: Background Review at Session End - #139

Merged
yogthos merged 2 commits into
phase-3-skillsfrom
phase-4-background-review
May 26, 2026
Merged

Phase 4: Background Review at Session End#139
yogthos merged 2 commits into
phase-3-skillsfrom
phase-4-background-review

Conversation

@yogthos

@yogthos yogthos commented May 26, 2026

Copy link
Copy Markdown
Collaborator

Port of Hermes's agent/background_review.py. After every session completes and is truly idle, a background review fork evaluates the transcript and writes project learnings.

Features:

  • Fire-and-forget tokio::spawn — never blocks the user
  • Restricted agent with memory + skill tools only (spawn_review_runner filters loop_tools)
  • Coding-specific combined review prompt covering memory and skills
  • Same provider/credentials as parent session
  • 15-minute rate limiter via AtomicU64 (advance fix)
  • Full transcript builder includes tool calls and results (advance fix)
  • Errors logged via tracing, never surfaced to user

Integration: triggers from Done event handler when session is idle.

Yogthos added 2 commits May 25, 2026 22:51
Port of Hermes's agent/background_review.py. After every session
completes (and no plugin followup or loop iteration claims the
next turn), a background review fork evaluates the transcript
and writes project learnings.

Features:
- Fire-and-forget tokio::spawn — never blocks the user
- Forked agent with memory + skill tools only (spawn_review_runner
  filters loop_tools to just 'memory' and 'skill')
- Coding-specific review prompts covering memory (MEMORY.md,
  PITFALLS.md) and skills (create, edit, patch)
- Same provider/credentials as parent session
- Errors logged via tracing, never surfaced to user
- review_provider + review_model config fields for routing
  review work to a cheaper/faster model (optional)

Integration: triggers from the Done event handler when the
session is truly idle (is_running=false).

All 1216 tests pass.
@yogthos
yogthos merged commit 41ee64f into phase-3-skills May 26, 2026
1 check passed
@yogthos
yogthos deleted the phase-4-background-review branch May 26, 2026 04:10
allen-munsch pushed a commit to allen-munsch/dirge that referenced this pull request Jun 3, 2026
…review

Phase 4: Background Review at Session End
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