Skip to content

fix: bound DUUL MCP review latency - #23

Merged
Suprhimp merged 1 commit into
masterfrom
duul-errors-investigation
Jul 31, 2026
Merged

fix: bound DUUL MCP review latency#23
Suprhimp merged 1 commit into
masterfrom
duul-errors-investigation

Conversation

@Suprhimp

Copy link
Copy Markdown
Member

Summary

  • cap each MCP review at 75 seconds and return structured timeout fallbacks
  • reduce default reviewer exploration to three rounds, with bounded environment overrides
  • add opt-in file diagnostics via DUUL_LOG_FILE

Investigation

  • the current trenton session's last completed plan review was logged at 14:20; its subsequent submission never reached DUUL.

Validation

  • npm test (96 tests passed)

@Suprhimp
Suprhimp force-pushed the duul-errors-investigation branch from 1ae8a54 to 1199e64 Compare July 31, 2026 06:00
@Suprhimp
Suprhimp merged commit cb3767d into master Jul 31, 2026
1 check passed
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