docs(readme): restructure around three integration paths - #613
Conversation
Signed-off-by: Itay Zitvar <izitvar@nvidia.com>
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Enterprise Run ID: 📒 Files selected for processing (1)
Included review availability: Your plan provides up to 12 included reviews per hour; 11 remain after this review. WalkthroughChangesREADME onboarding
Estimated code review effort: 2 (Simple) | ~10 minutes Merge Risk: 🔵 Low · up to The reorganized onboarding is broadly mergeable, but the introductory proxy instructions should confirm the default network exposure and authentication expectations to avoid unsafe deployment assumptions. Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
Full details: Docstring CoverageExplanation 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 files. (1 skipped: 1 unsupported.) Comment |
Signed-off-by: Itay Zitvar <izitvar@nvidia.com>
What
Restructures the README around how people actually adopt Switchyard, in the
order they should reach for it: use it inside a gateway you already run →
integrate it into a gateway you build → run the standalone proxy.
What is Switchyardopens with the benchmark chart, then three sections:Use Switchyard (NeMo Relay, LiteLLM), Integrate Switchyard into your
gateway or harness (
pip install nemo-switchyard, then the guide), and RunSwitchyard as a standalone proxy. Each has a diagram; all three share one
shape and differ only in what box Switchyard sits inside. The Switchyard node
is NVIDIA green in every diagram.
Maturity→Components. The warning callout is replaced by a table ofstability, intended use, and adoption guidance per crate — beta / alpha / demo.
stating where you start and what you finish with. Path 2 is now built around
a verified Python example: the
run_streamloop overCallModel/Done,executed against the real
switchyard.libsybindings before it went in.Capability + Stage, Escalation, Advisor Gate, Sub-Agent-Aware, Custom, Random.
docs/getting_started.md. Chart labels read "Switchyard:"rather than "SY:"; a footnote states the cost basis.
Why
The library is the primary way we want Switchyard adopted, and the README led
with the proxy. It also had no working example for the embed path in any
language, and every install path started mid-story.
Notes for reviewers
Read the rendered page, not the diff — it is a rewrite. The mermaid diagrams
and the chart only render on GitHub. Flip your theme once: the green node is
verified to survive mermaid's dark theme, but worth one look.
Judgement calls worth a second opinion:
grepped every crate README and
docs/page forStep.CallModel/Step.Done— zero hits. It should probably also live in
crates/libsy/README.md; that isa follow-up, not this PR.
docs/getting_started.mdis now unreferenced by the README but stillexists. Deleting it is an 11-file change that also removes
tests/getting_started/and a CI workflow, so it is left for its own PR. Notethat test asserts
## Server Pathbefore## Library Path— the opposite ofthe order adopted here.
inside something you already run; the proxy is its own process.
parse ISP as Internet Service Provider; spelling out "inference-provider" may
be worth it.
🤖 Generated with Claude Code