feat: add OpenRouter System One client support - #31
Conversation
Co-authored-by: Medulla <medulla@tinyhumans.ai>
Tiny Sweeper reviewTiny Sweeper reviewed this change across 6 lane(s) and found 0 active actionable findings. Detailed lane evidence and any incomplete work are listed below. State: Incomplete Review snapshot
Completeness: Incomplete What changedThe pull request adds a System One endpoint for OpenRouter, registers two public routes (primary and TypeSafe-compatible alias), increments path/operation counts in the OpenAPI manifest, and provides a client method and a matching integration test. The review found no critique, security, or test findings. Features
Tests
FindingsNo active actionable findings. Resolved this pass
Could not review: tinysweeper/description, tinysweeper/tests Before merge
How this fits togetherflowchart LR
n0["messages_speaks_the_anthropic_shape<br/>changed"]:::changed
n1["openrouter_chat_completion"]:::impacted
n2["...ng_is_rejected_before_it_reaches_the_wire"]:::impacted
n3["the_video_flow_submits_polls_and_downloads"]:::impacted
n4["get_openrouter_video"]:::impacted
n5["openrouter_message"]:::impacted
n6["openrouter_video_content"]:::impacted
n0 -->|calls| n5
n0 -->|tests| n5
n2 -->|calls| n1
n2 -->|tests| n1
n2 -->|calls| n5
n2 -->|tests| n5
n3 -->|calls| n4
n3 -->|tests| n4
n3 -->|calls| n6
n3 -->|tests| n6
classDef changed fill:#0d4429,stroke:#238636,color:#e6edf3
classDef impacted fill:#161b22,stroke:#6e7681,color:#c9d1d9
classDef flagged fill:#5a1e02,stroke:#d93f0b,color:#ffffff
classDef blocking fill:#67060c,stroke:#f85149,color:#ffffff
Agent review detailscritique
security
tests
commits
description
e2e
Evidence and run details
|
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
|
Warning Review limit reached
This review includes 3 billable files and costs up to $0.75. Or wait 51 minutes for your next included review. View limit detailsLimit details: You’ve used the included review currently available. Review configuration: ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Advanced Run ID: 📒 Files selected for processing (3)
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Advanced Run ID: 📒 Files selected for processing (5)
Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review. 📝 WalkthroughWalkthroughChangesOpenRouter System One
Priority: ⬇️ Low Estimated code review effort: 3 (Moderate) | ~20 minutes Change: Feature Suggested reviewers: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches 💡 1⚔️ Resolve merge conflicts 💡
📝 Generate docstrings
A rabbit sends typed questions bright Comment |
There was a problem hiding this comment.
Requesting changes: 2 lane(s) blocking, worst finding is critical.
Fix or reply to the findings below and push. The next review clears this automatically once they are gone — you should not need to dismiss anything by hand.
$0.0123 · 184,442 in / 6,527 out · 9,870 cached (5%) · flash, ladder/vectors, gpt-5.6-luna, deepseek/deepseek-v4-flash · 319 embedded
critique: $0.0068 · 99,483 in / 3,390 out · 6,231 cached (6%) · gpt-5.6-luna
security: $0.0052 · 79,336 in / 1,938 out · 3,639 cached (5%) · gpt-5.6-luna
There was a problem hiding this comment.
Requesting changes: 2 lane(s) blocking, worst finding is critical.
Fix or reply to the findings below and push. The next review clears this automatically once they are gone — you should not need to dismiss anything by hand.
$0.0108 · 171,782 in / 6,549 out · 19,713 cached (11%) · ladder/vectors, gpt-5.6-luna, deepseek/deepseek-v4-flash · 319 embedded
critique: $0.0054 · 82,401 in / 3,767 out · 10,361 cached (13%) · gpt-5.6-luna, deepseek/deepseek-v4-flash
security: $0.0049 · 78,125 in / 2,085 out · 9,352 cached (12%) · gpt-5.6-luna
description: $0.0003 · 7,540 in / 86 out · 0 cached (0%) · deepseek/deepseek-v4-flash
…mone # Conflicts: # api/tinyhumans.backend.json # tests/openapi_sync.rs
There was a problem hiding this comment.
tinysweeper found nothing blocking, but could not review everything, so this is not an approval: tinysweeper/description, tinysweeper/tests.
$0.0059 · 98,610 in / 2,117 out · 7,612 cached (8%) · flash, ladder/vectors, gpt-5.6-luna, deepseek/deepseek-v4-flash · 319 embedded
critique: $0.0028 · 47,030 in / 548 out · 4,050 cached (9%) · gpt-5.6-luna
security: $0.0028 · 45,816 in / 740 out · 3,562 cached (8%) · gpt-5.6-luna
Overview
Adds typed OpenRouter System One support to the Rust SDK for JEV requests.
Changes
openrouter_system_onefor serializable request payloadsTesting
cargo fmt --all -- --checkcargo clippy --all-targets -- -D warningscargo testcargo package --allow-dirtyNotes
Linked backend PR: https://github.com/tinyhumansai/backend/pull/1361