Skip to content

fix: CPU hogging GUI #2877

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
michaelneale merged 2 commits into from
Jun 12, 2025
Merged

fix: CPU hogging GUI #2877

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
1b1bd11
type comment
michaelneale Jun 12, 2025
e239ae9
trying to trim storage
michaelneale Jun 12, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
25 changes: 19 additions & 6 deletions .github/workflows/ci.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -45,8 +45,19 @@ jobs:
/usr/local/share/chromium \
/usr/local/share/powershell \
/usr/share/dotnet \
/usr/share/swift

/usr/share/swift \
/opt/ghc \
/opt/hostedtoolcache \
/usr/local/graalvm \
/usr/local/sqlpackage

# Clean package manager caches
sudo apt-get clean
sudo apt-get autoremove -y

# Clean docker if present
docker system prune -af 2>/dev/null || true

df -h

- name: Checkout Code
Expand Down Expand Up @@ -98,16 +109,18 @@ jobs:
rm -rf target/debug/deps
rm -rf target/debug/build
rm -rf target/debug/incremental
# Clean cargo cache more aggressively
cargo clean || true
# Clean npm cache if it exists
npm cache clean --force || true
npm cache clean --force 2>/dev/null || true
# Clean apt cache
sudo apt-get clean
sudo apt-get autoremove -y
# Remove unnecessary large directories
rm -rf ~/.cargo/registry/index || true
rm -rf ~/.cargo/registry/cache || true
# Remove docker images if any
docker system prune -af || true
# Remove unused packages
sudo apt-get autoremove -y || true
docker system prune -af 2>/dev/null || true

- name: Check disk space after cleanup
run: df -h
Expand Down
5 changes: 3 additions & 2 deletions ui/desktop/src/components/ChatView.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -280,8 +280,9 @@ function ChatContent({

// Update chat messages when they change and save to sessionStorage
useEffect(() => {
setChat({ ...chat, messages });
}, [messages, setChat, chat]);
// @ts-expect-error - TypeScript being overly strict about the return type
Copy link
Collaborator

@zanesq zanesq Jun 12, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

you can use // eslint-disable-next-line react-hooks/exhaustive-deps we use it in other places.. seems like we should just remove this linting requirement as it seems to causes issues more often than not

setChat((prevChat: ChatType) => ({ ...prevChat, messages }));
}, [messages, setChat]);

useEffect(() => {
if (messages.length > 0) {
Expand Down
Loading