docs: record what the providers overhaul deliberately left out - #64
Merged
Conversation
The four slices each named deferrals in their PR bodies, but none of it landed in the backlog — so the reasons would have evaporated with the PR descriptions. Captures the five open items with *why* they were declined, not just what: measured cost needs a background job (an inline per-turn lookup would add a round trip, and OpenRouter writes the record asynchronously); a spend cap at mint time depends on an undocumented, unverified `/auth` behavior and shouldn't be promised before it's probed; management-key features were declined because they store a second higher-privilege secret for a convenience. Also flags the operational gotcha that isn't obvious from the code: existing clusters need the new tokenless gateway location plus `up -d --force-recreate` before OpenRouter linking works there — a plain restart won't reload a changed bind-mounted file. Docs only; no version bump (nothing user-facing changed). Assisted-by: Opus 5
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
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
1 participant
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.
Close-out for the providers track (#59, #60, #61, #63).
Each slice named its deferrals in the PR body, but none reached the backlog — so the reasons would have evaporated along with the PR descriptions.
Records five open items with why, not just what:
/authforwardslimit/key_labelas query params. Undocumented and unverified; flagged to probe before promising./presets,/byok,/providersmetadata,/analytics/query,/embeddings/models,/videos.ProviderKindslot exists; onlyopenai_compatibleis registerable (ADR-15).Plus the operational gotcha that isn't visible from the code: existing clusters need the new tokenless gateway location and
up -d --force-recreatebefore OpenRouter linking works there.Docs only — no version bump, nothing user-facing changed.
task docs:checkgreen.Assisted-by: Opus 5