docs: observability page facts (issue #83) - #90
Open
SebConejo wants to merge 1 commit into
Open
Conversation
…ts, custom-provider pricing, alert delivery, request states, usage API shapes, Pro ranges, Limits card (issue #83)
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.
Processes the eight findings of #83, recommended options, each re-verified in the platform code, plus a live check of the usage endpoints.
Finding 1 (option 1) — the limit period is not rolling. It is a calendar window on the server's clock: day at midnight, week on Monday, month on the 1st, hour at the top of the previous clock hour, restart at the next boundary. The page now agrees with the gateway page's "the block clears at the start of the next period". Deliberately not UTC: the boundaries are computed in the server's local timezone.
Finding 2 (option 1) — a failed attempt does not burn tokens. The recorder writes cost and tokens on the success paths only; failure rows keep zero tokens and no cost. The paragraph now says what the record holds instead of the wrong justification.
Finding 3 (option 1) — the Tip claimed Manifest cannot price custom-provider models. The connect form asks for the two prices and Manifest computes cost from them. The Tip now states the conditional truth, and
providers/custom-providers.mdxcarried the same wrong claim, fixed in the same PR as the issue prescribed.Finding 4 (option 1) — the crossing is recorded before the email is sent and never retried inside the period, so a failed send costs the period its email while Triggered still counts up. One sentence on the evaluation paragraph, and the self-hosted email block now points at Send test email.
Finding 5 (option 1) — Pending and Cancelled join the request model, and one paragraph explains why the Overview's Failed requests card links to a longer list than the number on the card: the Overview counts finished requests, the Failed filter returns everything that did not succeed.
Finding 6 (option 1) — the usage API section now carries the
rangeparameter with its accepted values, the two different defaults (24hversus7d), the rolling-versus-calendar warning, and one sample response per endpoint. I checked the shapes live on a local instance:/usageanswers24hby default withagentName,/costsanswers7dwithout it, andrange=1dgets a 400. The sample numbers are illustrative; the field lists are the live ones.Finding 7 (option 1) — the Pro lock on long ranges applies on the Overview and on the Requests log, and the log's All time option stays free. The "covers up to 7 days of history" phrasing is gone, since it read as a retention limit.
Finding 8 (option 1) — the Limits card said "plus email provider setup" to every reader, while the picker only exists on self-hosted. The card now reads "Threshold rules per harness." and the self-hosted block keeps the setup instructions.
Coherence GREEN (48 pages, 57 vars).
Closes #83