docs: resync skills with smg HEAD (v1.6.0) - #20
Merged
Conversation
Skills were last verified against smg on 2026-06-17; ~103 commits have landed since (HEAD 72ec2bfa). Audited every recipe against the current code (not commit messages) via parallel per-subsystem audits and fixed the drift. Routing policies (9 -> 10, `passthrough` added): - map: "9 routing policies" / "9 load-balancing policies" -> 10. - routing-policy: add `passthrough` to the impl list. - bindings-update: PolicyType/PolicyConfig list -> 10 (add Passthrough). gRPC router refactor (#1923, unified regular/PD/EPD): - grpc-backend: document the Mode-parameterized GrpcRouter + RequestPipeline/PipelineStage architecture (pd_router.rs removed, epd_encode -> common/stages/encode.rs); add the TokenSpeedEncoderClient EPD-encode exception (not a GrpcClient enum variant). Factual fixes: - multimodal: drop non-existent `::new_dynamic` (`::new` is generic over dimensionality); add AudioUrl/AudioData to MediaContentPart; scope the registry-ordering rule to the spec Vec (processor registry is a HashMap); fetch_video returns Arc<VideoClip>. - mcp: rmcp 0.8.x -> 1.7; SSE client transport dropped, model new transports on the Streamable arm; drop SseClientTransport from the anti-pattern list. - rate-limit: `rate_limit_tokens_per_second = 0` does NOT yield semaphore mode; it is coerced to max_concurrent_requests via .filter(|&t| t > 0). - discovery: add the Encode pod type / --encode-selector; note Router is an is_router overlay, not a PodType. - provider-api: ProviderType gains Custom(String). - config-plumbing: to_server_config does take a RouterConfig and uses .or_else() for discovery-derived fields. - implement: realtime router path -> routers/common/realtime/. - wasm: OnError attach point is defined but not implemented. - auth: JTI replay check is opt-in / off by default. - storage-backend: data-connector v2.3.0 -> v2.3.2. - review-pr: correct the e2e pytest marker set (model/workers/gateway/ e2e/slow). - map: add the mm_rdma (smg-mm-rdma) crate row; bindings param anchor. Verified in sync (no changes): scheduler, tenancy, tool-parser, reasoning-parser, observability, kv-index, mesh, contribute gate, review anti-patterns. Signed-off-by: Simo Lin <25425177+slin1237@users.noreply.github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Skills were last verified against smg on 2026-06-17; ~103 commits have landed since (HEAD
72ec2bfa, 2026-07-15). Audited every recipe against the current code (not commit messages) via parallel per-subsystem audits and fixed the drift. 15 files changed, 23 confirmed drifts plus one architecture-coverage addition.Highlights
Routing policies (9 → 10,
passthroughadded) — rippled throughmap/SKILL.md(×2),routing-policy.md,bindings-update.md(PolicyType/PolicyConfig).gRPC router refactor (#1923) — documented the unified
Mode-parameterizedGrpcRouter+RequestPipeline/PipelineStagearchitecture ingrpc-backend.md(pd_router.rsremoved,epd_encode→common/stages/encode.rs), and added theTokenSpeedEncoderClientEPD-encode exception. This was a coverage gap, not a wrong claim — the recipes never described the router internals.Factual fixes
::new_dynamic(::newis generic over dimensionality); addAudioUrl/AudioDatatoMediaContentPart; scope the registry-ordering rule to the specVec(processor registry is aHashMap);fetch_video→Arc<VideoClip>.0.8.x→1.7; SSE client transport dropped — model new transports on theStreamablearm; dropSseClientTransportfrom the anti-pattern.rate_limit_tokens_per_second = 0does not yield semaphore mode — coerced tomax_concurrent_requestsvia.filter(|&t| t > 0).Encodepod type /--encode-selector; Router is anis_routeroverlay, not aPodType.ProviderTypegainsCustom(String).to_server_configdoes take aRouterConfigand uses.or_else()for discovery-derived fields.routers/common/realtime/.OnErrorattach point is defined but not implemented.data-connectorv2.3.0 → v2.3.2.model/workers/gateway/e2e/slow).mm_rdma(smg-mm-rdma) crate row; fix the bindings param anchor.Verified in sync (no changes)
scheduler, tenancy, tool-parser, reasoning-parser, observability, kv-index, mesh, contribute gate, review anti-patterns. Counts re-confirmed: 14 tool parsers, 8 reasoning families, Python
Routernow 117 params.Known non-changes (flagged, not drift)
contribute/SKILL.mdusesclippy --workspace;review-pr/CONTRIBUTING.mddon't — both valid per their respective sources, just internally inconsistent.contribute/SKILL.mddocuments no branch-name regex — a coverage gap; the enforced patterns live in the pre-commitbranch-name-checkhook andpr-naming-check.yml.