Skip to content

Replace recursive evaluator traversal with continuation stack - #554

Draft
prozacchiwawa wants to merge 11 commits into
mainfrom
cursor/iterative-evaluator-2af2
Draft

prozacchiwawa wants to merge 11 commits into
mainfrom
cursor/iterative-evaluator-2af2

Conversation

@prozacchiwawa

@prozacchiwawa prozacchiwawa commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

Summary

  • model evaluator work as owned requests, typed results, and explicit boxed continuations
  • run evaluator operations through one trampoline loop instead of recursive evaluator calls
  • preserve the prior logical depth-limit semantics and shared mash-condition state
  • remove the obsolete recursive stack-visitor module
  • cover stack-limit boundaries and 10,000 nested evaluations

Testing

  • cargo clippy --workspace -- -D warnings
  • cargo fmt --all -- --check
  • cargo check --all-targets
  • cargo test tests::compiler::evaluate — 17 passed
  • cargo test -- --skip compile_performance_suite — 697 passed, 0 failed, 1 ignored, 1 filtered

The repository's compile-performance test also passed in the prior CI run. The earlier Linux wheel failure was only its coverage gate reporting the now-removed unused stackvisit.rs; the earlier M1 wheel failure was toolchain setup (cargo missing), unrelated to this code.

Open in Web Open in Cursor 

prozacchiwawa and others added 7 commits August 5, 2026 21:23
Co-authored-by: arty <prozacchiwawa@users.noreply.github.com>
Co-authored-by: arty <prozacchiwawa@users.noreply.github.com>
Co-authored-by: arty <prozacchiwawa@users.noreply.github.com>
Co-authored-by: arty <prozacchiwawa@users.noreply.github.com>
Co-authored-by: arty <prozacchiwawa@users.noreply.github.com>
Co-authored-by: arty <prozacchiwawa@users.noreply.github.com>
@cursor cursor Bot changed the title Replace recursive evaluator traversal with explicit stack Replace recursive evaluator traversal with continuation stack Aug 6, 2026
cursoragent and others added 4 commits August 6, 2026 02:36
Co-authored-by: arty <prozacchiwawa@users.noreply.github.com>
Co-authored-by: arty <prozacchiwawa@users.noreply.github.com>
Co-authored-by: arty <prozacchiwawa@users.noreply.github.com>
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