deploy pages#2
Merged
Merged
Conversation
… installations of a fixed Zig version.
…x scanner `stat.mode` type casting.
…g build` command.
…s from the build matrix and build command.
… to skip GUI compilation.
…he resources directory
- Display status message in header after deletion operations complete - Color-coded messages: green for success, red for failure - Updated web binary sizes to actual values (~1MB GUI, ~633KB CLI) - Removed debug print statement
The issue was that `statFile()` only works for files, not directories. When trying to delete directories like node_modules, the stat call would fail immediately, causing the deletion to silently fail. Fixed by: - First trying to open path as directory to detect directories - Only using statFile() for actual files - Updated permanentDelete() with same fix - Readonly check now only applies to files (skipped for directories) 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- Add safer roundness calculation for button backgrounds (clamp 0-1) - Update release workflow to prefer zig-out/bin/resources - Add verification logging for package contents - Fix potential edge case in Windows packaging 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- Show proper status message with success/failure counts after deletion - Deselect items that failed to delete instead of leaving them selected - Remove ~/Library/Caches and ~/Library/Logs from scan (system protected) - Only scan specific app cache subfolders that users can safely delete - Add more cache paths: yarn, npm, node-gyp 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Each DeleteResult contains allocated strings (path, trash_path, error_message) that need to be freed. Added proper cleanup in defer block to call deinit() on each result before freeing the results array.
TrashResult.original_path was being allocated but never freed. Since we already have the path in DeleteResult, we don't need original_path and should free it in the defer block.
The home and temp paths from getHomeDir() and getTempDir() were allocated but never freed in Paths.deinit(). 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
…tructions, and fix a memory leak in trash handling.
…nd direct GitHub links for downloads and source.
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
1 participant
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.
No description provided.