Skip to content

fix(memory-harness): allow structured output completion - #104

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

  • raise the isolated reviewer's Claude CLI turn ceiling from 1 to 2 so structured-output completion is not terminated as max_turns
  • keep the existing one-call, no-tools, no-session-persistence, low-effort, 75-second isolation boundary
  • pin the two-turn ceiling in the argv regression and rebuild the immutable worker bundle

Evidence

  • production OAuth + full real 2.4k-token review snapshot canary: decision=no_op, terminal_reason=completed, 26.4s, stderr 0
  • before fix, the same snapshot returned terminal_reason=max_turns, num_turns=2, exit 1
  • targeted generator tests
  • bun run check
  • immutable worker bundle readback
  • privacy scan clean

Safety

  • still one isolated model process per attempt
  • no tools, MCP, channels, resume, or session persistence
  • retry remains capped at one
  • no prompt/provider output persisted

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