mirror of
https://github.com/BerriAI/litellm.git
synced 2026-10-13 03:59:12 +00:00
* docs(agents): ban semicolons and colons in human-facing text Tighten the writing rule so ";" and ":" are only allowed where the sentence would be nonsensical without them, rewrite the existing semicolons in AGENTS.md to follow it, and note that docs (litellm-docs) keep their trailing "." * docs(agents): rewrite semicolons in nested AGENTS.md files * docs(agents): hand-maintain the dashboard Next.js warning and sync e2e CONTRIBUTING lines * docs(agents): point at the PR template path that exists * docs(agents): restore the generated Next.js block in the dashboard AGENTS.md
3.2 KiB
3.2 KiB
Requirements
Core must pause mid-call to ask the host for things it cannot do itself (Python callbacks, secret and token reads, before_send rewrites, stream demand), then continue where it stopped. Any change to this crate must keep every requirement below. The alternatives section says which one each rejected design breaks
- R1 Core never calls the host: it names an op and waits for the answer, so it stays free of PyO3 and of any other host runtime
- R2 Async host work is awaited by the host's own driver in the caller's asyncio task (
litellm/rust_bridge/lifecycle.py), socontextvarswrites reach the caller. A Rust-sideinto_futurewould run it in a copied context - R3 The body awaits real I/O (HTTP,
spawn_blocking, timers) between yields, soresumeis itself a future driven by the caller's runtime - R4 Route code stays straight-line async (
host.route(OcrOp::ReadDocument).await?) instead of hand-written states - R5 Each op fixes its answer type at compile time: a host cannot answer
ReadDocumentwith a token, and core never matches a result variant it did not ask for - R6 A yield the body makes while being resumed is returned by that same poll, so the host driver's inline first poll needs no extra event-loop turn per op
- R7 No task is spawned:
cancel, or dropping the coroutine, drops the body, and nothing waits forever on an answer that cannot come - R8 Several yields can be pending at once, since route code hands clones of its
Coto token providers and hooks - R9 Stable Rust
Other implementations and why they do not fit
- Nightly
std::ops::Coroutine: breaks R9, and its body cannot await futures between yields (R3) genawaiter: resumes async bodies only with a noop waker, so the body cannot await real I/O (R3)simple_coro: typestateCoromakes answering before resuming a compile-time rule, but its body cannot await arbitrary futures (R3) and its reply typeRis fixed per coroutine (R5)corosenseiand other stackful coroutines: sync bodies on their own stack, no async I/O inside (R3)- A hand-written phase enum with an
advancematch (the oldHostPhase): every await point becomes a state (R4) - An injected host trait with
async fns: core would call the host itself (R1, R2) - Sans-IO, where core does no I/O and HTTP becomes one more host op: keeps every requirement and makes
resumea pure step function, but HTTP, streaming, retries and timeouts would move out of core into every bridge. The one real alternative, not taken - Temporal's Rust workflow SDK (
WorkflowFuture,WfContext) is the closest precedent: anasync fnpolled in place, commands sent over a channel with a oneshot to unblock them. Roles are inverted there (the language SDK owns the program, core answers), and its workflow body may not do real I/O
Tradeoffs accepted
- A tokio
mpscchannel plus aoneshotper yield instead of compiler-generated states - Protocol mistakes (resuming before answering, resuming after the end) are runtime
ResumeErrors, not compile errors - Pending yields come out one per
resume, in the order they were made, and each reply goes back to the yield that made it (R8) - An answer sent after its yield stopped waiting (for example the body timed out on it) is discarded, since the body already moved on