Dev to main, 27th Aug, 2026 - #838
Merged
Merged
Conversation
Adds a cookbook quickstart that takes an app with no tracing to a verified integration, covering both the Future AGI SDK and plain OpenTelemetry routes, and ships fi_verify.py as a downloadable asset so the ten gates can be run against the spans an app really produced.
Install, Tutorial with Steps, What you built with Check, Next steps, matching the 34 sibling quickstarts rather than the one that deviates from them, and restoring the done-condition and the blocked-report format the first draft dropped.
…boundary Time and difficulty were out of family: 45 min was the longest value on any of the 35 quickstarts and this page is half the length of the one that says 20. The Python prerequisite read as a constraint on the reader's app; only fi_verify.attach() is Python-bound, so the not-Python section now says what to do without it.
Runs the six steps against openai/openai-agents-python's customer service agent and shows the trace list, the trace tree and the session from that run. Name the track inside every listing so the two tracks stay separable when the page is flattened to text, print the entry point the worked example adds, and link the Colab notebook.
Resolve navigation.ts: dev restructured the cookbook nav into collapsible groups while this branch added a single entry. Took dev's structure and placed the entry first in Tracing and Debugging, ahead of Manual Tracing, because the siblings in that group assume tracing already works.
/docs/tracing/auto moved to /docs/integrations/traceai. The three references on this page were written before the move and now fail the link audit.
The collector prices any span carrying a model and non-zero tokens, so requiring a user-sent root cost failed an integration that leans on platform pricing. G9 now asks that every LLM span is priceable or carries its own cost, and names the double count when a root total sits on top. Also: a real key ending 0000 made the wrong-key control the real key, so preflight false-failed. Python floor is 3.11.
Cost: the collector prices LLM spans from the model and tokens (LiteLLM table, then per-org rates) and a sent value wins. Names which surface reads the root and which sum every span, so the roll-up is now the unpriced-model path rather than the mainline. Shape: full frontmatter, Future AGI as two words, 3.11 floor, badges on blob/main, Troubleshooting table, one forward link instead of the card grid, image captions and borderRadius, hub card and count.
All three are now uniform 3024x1890 full-window dark, taken off the same run in the platform. The trace list is filtered to traces that carried tokens, so the failed first attempt and the preflight probe are out of frame, and the cost annotation now says what is actually true: our table keys this model as groq/llama-3.3-70b-versatile, so nothing prices it and the number on the root is the one the roll-up put there.
docs(cookbook): instrument an existing app and verify it end to end
khushalsonawat
approved these changes
Aug 27, 2026
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
Promotion PR. Everything in it is already reviewed and merged to
devin #786; this moves it tomainso it deploys to prod.The payload is one cookbook page,
instrument-and-verify, plus the checker script it tells the reader to download and the three captures it embeds.maincurrently serves the cookbook index but not this page, so the quickstart is live on dev and invisible in prod.Why this is needed
.github/workflows/deploy.ymlbuilds and deploys onpushtomainonly. Merging #786 intodevtherefore shipped nothing todocs.futureagi.com.Checked against prod just now:
The second 404 is the page itself. The third one matters more than it looks: Step 2 of the page instructs the reader to run
so until
maincarriespublic/fi_verify.py, anyone following the cookbook from the live site hits a 404 on the first command that fetches anything.What changed
Nothing was authored in this PR. The 11 commits and 8 files below are the full
main...devdelta.src/pages/docs/cookbook/quickstart/instrument-and-verify.mdxpublic/fi_verify.pypublic/images/docs/cookbook-instrument-and-verify/trace-list.pngpublic/images/docs/cookbook-instrument-and-verify/trace-tree.pngpublic/images/docs/cookbook-instrument-and-verify/session.pngsrc/lib/navigation.tssrc/pages/docs/cookbook/index.mdxsrc/pages/docs/cookbook/platform/index.mdxVerification
75ce8d9cb5bafa1cbeea42affe8b464433ab97e438a53410f27678855d484de7as the expectedshasum -a 256offi_verify.py. Ran it against the copy on this branch: identical, 11,925 bytes.Link audit & redirect enforcementwas green on docs(cookbook): instrument an existing app and verify it end to end #786 before Khushal merged it.Blast radius
main...devis 11 commits and 8 files, and every one of them belongs to this cookbook page. No unrelated work rides along.The reverse compare
dev...mainis 0 files, sodevalready contains everything onmain.maingains exactly the 8 files above and loses nothing.Rollback
Revert the merge commit. No schema, no migration, no config. The three prod URLs above return to their current state.
Note on the description template
The standard internal template asks for a tests table, a local suite-run screenshot and a test-run video. This is a docs promotion PR with no code and no test suite, and the content it carries was already reviewed and proved in #786, so those sections are deliberately omitted rather than filled with placeholders. The verification that does apply is above.