perf(demo): reach the first tool call sooner - #658
Merged
Conversation
The prompt is preamble; the run is what the reader came for. Type it at 18ms/char instead of 42, and stream the opening thought on a 1.1s budget instead of a per-token cadence that stretched it past 2s. Measured on the built demo: prompt typed at 1.67s, the opening thought streams in 1.09s, and engine::workers::list is on screen at 5.18s. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Contributor
skill-check — worker0 verified, 51 skipped (no docs/).
Four for four. Nicely done. |
📝 WalkthroughWalkthroughThe demo now calculates opening-thought streaming pace for a 1.1-second duration and types prompts at a shorter per-character interval. ChangesDemo timing
Estimated code review effort: 2 (Simple) | ~10 minutes Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches 💡 1🛠️ Fix failing CI checks 💡
📝 Generate docstrings
🧪 Generate unit tests (beta)
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 |
The close button was ink on cream, indistinguishable from pause and replay beside it. Alert red now, with black rather than cream text on it: at 10px the label needs the contrast (7.8:1 against the red, versus 4.0:1 for white). Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
anthonyiscoding
added a commit
to iii-hq/iii
that referenced
this pull request
Jul 31, 2026
Picks up iii-hq/workers#658: the close control in the embedded console's window chrome is alert red on black, so the way out is the loudest thing in the frame. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Raw #ff0026 read as an error rather than as the way out. Mixed 72% with a near-black in oklab it lands on #b8071f, which stays the loudest thing in the chrome bar while letting white hold 6.8:1 at the 10px the bar uses. Derived from the alert token and identical in both themes, since the control means the same thing in each. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
anthonyiscoding
added a commit
to iii-hq/iii
that referenced
this pull request
Jul 31, 2026
Picks up iii-hq/workers#658: the close control sits at #b8071f instead of raw alert red, still the loudest thing in the window chrome. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
andersonleal
approved these changes
Aug 14, 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.
The landing demo spent over two seconds typing the prompt and another two on
the opening thought before anything happened. Both are preamble; the run is
what the reader came for.
paceOver(body, totalMs)helper, instead of a per-token cadence whoselength decided the duration.
Measured on the built demo, from page load: prompt fully typed at 1.6s, the
opening thought streams in 1.09s,
engine::workers::liston screen at 5.1s.The website vendors this build, so it ships alongside
iii-hq/iii#2029 there.
MOT-4440
🤖 Generated with Claude Code
Summary by CodeRabbit