Repository navigation
Expose native budget capacity admission - #377
Conversation
Co-authored-by: Medulla <medulla@tinyhumans.ai>
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
Tiny Sweeper reviewTiny Sweeper reviewed this change across 5 lane(s) and found 0 active actionable finding(s). Detailed lane evidence and any incomplete work are listed below. State: Ready for maintainer review Review snapshot
Completeness: Complete FeaturesNone identified with supported citations. TestsNo supported feature-to-test mapping was produced. Test execution is not inferred. FindingsNo active actionable findings. Before mergeNone. Agent review detailscritique
security
tests
commits
description
Evidence and run details
|
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info
📝 Walkthrough
Merge Risk: ⚪ Minimal · up to No actionable merge-blocking risk is established for this pin; normal merge checks still apply. Architecture SummaryArchitecture risk: 🔵 Low · up to The changed surface does not map to a changed system, dependency edge, entrypoint, or external dependency. Changed systems: None identified. Architecture concerns Review details
Pre-merge checks |
 |
| Check name | Status | Explanation |
|---|---|---|
| Docstring Coverage |  |
No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0… |
| Linked Issues check | |
Check skipped because no linked issues were found for this pull request. |
| Out of Scope Changes check | |
Check skipped because no linked issues were found for this pull request. |
| Description Check | |
Check skipped - CodeRabbit’s high-level summary is enabled. |
| Title check | |
The title clearly identifies the main change: exposing native budget capacity admission through the pinned TinyInference commit. |
- Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts
A rabbit checks the vendor trail
One commit shifts along the way
Tinyinference hops ahead
No other paths have changed today
The rabbit nods, then bounds away
Comment @coderabbitai help to get the list of available commands.
![tinysweeper[bot]](https://avatars.githubusercontent.com/in/4526585?s=60&v=4){kind=small}
tinysweeper
Bot
added
the
priority: p3
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
Pin TinyInference to immutable
9b583f92336cebf0be57ed42976d0321095639effrom canonical tinyhumansai/tinyinference#75. This exposes opt-in native budget capacity admission for host policies: affordable calls stay concurrent, temporary reservation pressure waits, and permanently unaffordable calls remain refused. Default admission and internal transport retries remain fail-fast; canceled or unknown paid calls keep their conservative charges.The native change includes thirteen deterministic regressions, including temporary waits not poisoning terminal refusal state. No module versions, release artifacts, policies or pricing bounds change here; this PR only updates the recursive gitlink.
Validation on local Rust stable 1.99: TinyAgents harness 2476 tests passed against this exact native commit; native LLM617 tests and alltargets/allfeatures Clippy passed. Refreshed TinyAgents harness alltargets Clippy with
-D warningsalso passed. Canonical hosted workspace CI remains required before merge.Summary by CodeRabbit