Skip to content

[codex] Remove smol runtime#1549

Open
svarlamov wants to merge 1 commit into
mainfrom
chore/remove-smol
Open

[codex] Remove smol runtime#1549
svarlamov wants to merge 1 commit into
mainfrom
chore/remove-smol

Conversation

@svarlamov

Copy link
Copy Markdown
Member

Summary

  • Remove the smol dependency and stale lockfile entries.
  • Route synchronous async wait points through a shared Tokio runtime helper.
  • Replace bounded smol task/unblock usage with Tokio semaphores and spawn_blocking.
  • Update tests and stale docs so repo-wide runtime references are gone.

Validation

  • cargo check --all-targets
  • task fmt
  • task lint
  • task test TEST_FILTER=rebase_realworld::test_human_conflict_python_pipeline_mixed_baseline_c3_conflict after one transient full-suite daemon timeout
  • task test clean rerun
  • rg -n "\\bsmol\\b|smol::" -S Cargo.toml Cargo.lock src tests docs returns no matches

Notes

The initial full task test run had one daemon completion timeout in rebase_realworld::test_human_conflict_python_pipeline_mixed_baseline_c3_conflict; the exact test passed on rerun and the subsequent full suite passed cleanly.

@svarlamov svarlamov marked this pull request as ready for review June 12, 2026 15:17

@devin-ai-integration devin-ai-integration Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no bugs or issues to report.

Open in Devin Review

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