Skip to content

fix(memory-harness): bound reviewer effort - #103

Merged
wang2 merged 1 commit into
mainfrom
fix/preserve-runtime-activation-config
Aug 28, 2026
Merged

wang2 merged 1 commit into
mainfrom
fix/preserve-runtime-activation-config

Conversation

@wang2

@wang2 wang2 commented Aug 28, 2026

Copy link
Copy Markdown
Member

Summary

  • run the isolated Haiku memory reviewer with --effort low for this bounded classification task
  • extend its internal model timeout from 45s to 75s while keeping it below the root worker's 90s process timeout
  • pin both argv and timeout in regression tests

Evidence

  • production OAuth + full reviewer schema synthetic canary completed in 7.1s with low effort
  • bun test packages/session-control-mcp/tests/memory-review-generator.test.ts
  • bun run check
  • immutable worker bundle readback
  • privacy scan clean

Safety

  • retry remains capped at one
  • root process boundary remains 90 seconds per worker invocation
  • no prompt/provider output is persisted

@wang2
wang2 merged commit bd2f930 into main Aug 28, 2026
2 checks passed
@wang2
wang2 deleted the fix/preserve-runtime-activation-config branch August 28, 2026 03:53
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