Conversation
Summary of ChangesHello @keithguerin, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! This pull request significantly refines the user interface for the CLI, particularly focusing on how 'thinking' states and tips are presented. The changes aim to improve clarity and user experience by standardizing the visual representation of ongoing processes and making informative messages more distinct. This overhaul ensures a more consistent and accessible interaction for users tracking the model's activity. Highlights
Changelog
Activity
Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here. You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension. Footnotes
|
There was a problem hiding this comment.
Code Review
This pull request provides a significant overhaul to the thinking UI, making it clearer and more modern. It also improves the visibility of status indicators and tips. The changes are well-implemented across the UI components, hooks, and tests. The new thinking UI logic in ThinkingMessage.tsx is sound, and the props isFirstThinking and isLastThinking are correctly determined in MainContent.tsx. The status bar indicator is now text-based, and tips are prefixed with "Tip: ", both of which are good UX improvements. The related tests have been updated accordingly. The change to how thinking items are added to history in useGeminiStream.ts is a good bug fix that prevents thoughts from being overwritten. Overall, this is a solid PR that improves the CLI's user experience. I found no high or critical issues.
|
Size Change: +1.9 kB (+0.01%) Total Size: 26 MB
ℹ️ View Unchanged
|
ad5ecbb to
9b14071
Compare
cd8480e to
f40be93
Compare
f40be93 to
83c24a5
Compare
83c24a5 to
1c7d527
Compare
8fc936d to
8822c19
Compare
chore(cli): remove Tip: prefix from informative tips Checkpoint fixing thinking display test: add sequentially rendering thinking messages test to MainContent.test.tsx Code review comments from Jacob written as a pr.
8822c19 to
5750a6c
Compare

Summary
This PR overhauls the inline thinking UI to match modern design standards and improves the visibility of status indicators. It replaces the thinking emoji in the status bar with a clear text-based indicator.
Details
Related Issues
Closes #18724
How to Validate
Pre-Merge Checklist