docs(readme): retake the screenshots against the reworked pages - #93
Closed
Fl0p wants to merge 1 commit into
Closed
Conversation
The hero shot predated the Overview rework: it showed the user-search cloud that has since been deleted, Sessions at the top of the page, KPI labels hardcoded to (30d) and no range switcher. It is replaced by the current Overview plus a Screens gallery of Sessions, Costs, Users and Tools. The images come from a throwaway instance seeded by scripts/seed-demo.py rather than from production, so no real user name or real spend is published. The seeder goes in over the OTLP endpoint instead of writing to DuckDB, so the numbers on the page are what ingest, cost derivation and the roll-up actually produce, and it sends only attributes Claude Code sends — no command on Bash spans, so the Tools page keeps the same "no command detail" state a real install has. scripts/shoot-screenshots.mjs crops each page at the bottom edge of a named element, so a row-height or chart-size change cannot slice a card in half on the next retake. Also fixes two stale things the retake surfaced: the logo <img> pointed at assets/logo.svg, a path that does not exist in the repo and rendered broken on GitHub, and the current-release tag still read :0.2. Co-Authored-By: Daedalus <daedalus@agents.flopbut.local> Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
|
Important
This repository does not receive automatic reviews because it has fewer than 10 stars. ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Contributor
Author
|
Squash-merged to |
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.
The README hero shot was taken before the Overview rework. It showed the user-search cloud that has since been deleted, Sessions at the top of the page, KPI labels hardcoded to
(30d), and no range switcher — a picture of software that no longer exists.What is in here
## Screensgallery — Sessions, Costs, Users, Tools.scripts/seed-demo.py— fills a throwaway instance with a synthetic team (7 users, 3 models, 90 days, ~485 sessions / ~25 600 spans). It goes in over the OTLP endpoint rather than writing to DuckDB, so the numbers on the page are what ingest, cost derivation and the roll-up actually produce. Fixed RNG seed, so a re-run against a fresh volume reproduces the same figures.scripts/shoot-screenshots.mjs+docs/operations/screenshots.md— the retake recipe. Each page is cropped at the bottom edge of a named element, not at a pixel count, so a row-height or chart-size change cannot slice a card in half next time.Two stale things the retake surfaced, fixed here: the logo
<img src="assets/logo.svg">pointed at a path that does not exist in the repo (broken image on GitHub) and the "current release" tag still read:0.2.Why synthetic data
Real telemetry carries real user names and real spend, and a screenshot is forever. The seeder sends only the attributes Claude Code actually sends — in particular no
commandonBashspans — so the Tools page shows the same "no command detail in this data" state a real install sees rather than advertising a view nobody gets.Verification
docs/VitePress build:vitepress build→build complete in 5.95s, no dead links (the Deploy Docs job fails fatally on those).cotel:flo592built from this branch's base (2ff50d4), seeded, then shot with the committedscripts/shoot-screenshots.mjs— the committed script is what produced the committed images.🤖 Generated with Claude Code