Fix memory leaks, improve error messages, and clean up technical debt#28394
Open
DavidLiedle wants to merge 4 commits intokeybase:masterfrom
Open
Fix memory leaks, improve error messages, and clean up technical debt#28394DavidLiedle wants to merge 4 commits intokeybase:masterfrom
DavidLiedle wants to merge 4 commits intokeybase:masterfrom
Conversation
…dd missing UI This commit includes several improvements to make the codebase cleaner and more maintainable: 1. Remove deprecated BTC command - Deleted unused CmdBTC that only returned an error message - Users should use 'keybase currency add' instead - Cleaned up related imports and command registration 2. Replace console.log with proper logging - Updated device-revoke.tsx and provision.tsx to use logger - Improves debugging and production error tracking - Follows existing logging patterns in the codebase 3. Modernize browser extension SPA monitoring - Replaced 1-second polling hack with MutationObserver API - Added proper history API interception for pushState/replaceState - More efficient and responsive to navigation changes - Addresses FIXME comment about the polling approach 4. Implement missing git repo rename UI - Added GitPushRename component for repository rename notifications - Service already supported RENAMEREPO_3 but UI was missing - Follows existing component patterns for consistency All changes tested and verified to maintain compatibility. 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
This commit addresses code quality and performance issues: 1. Fix syntax error in cmd_currency.go - Added missing closing quote in error message - Prevents potential string parsing issues 2. Optimize slice operations in teams/member_set.go - Replace append with direct index assignment for pre-allocated slices - Reduces memory allocations in recipientUids() and restrictedBotRecipientUids() - More efficient for large member sets - Added early return for empty collections These optimizations improve performance in frequently called code paths while maintaining the same functionality. 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
Added proper cleanup for the debug interval that logs outstanding RPCs: - Store interval ID in _debugIntervalID class property - Add _cleanupDebugging() method to clear the interval - Call cleanup in reset() method to prevent memory leaks This prevents the interval from continuing to run after the engine is reset, which could cause memory leaks in development environments. 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
This commit tackles the tedious but essential maintenance work that often gets pushed aside but genuinely improves code quality: 1. Standardized generic error messages - Replaced casual "Oops", "Ah Shoot\!", "Whoops\!" with professional messages - Changed vague "Something went wrong" to specific, actionable messages - Consistent capitalization and punctuation across error messages - Better user guidance in error states 2. Removed large blocks of commented-out dead code - Cleaned up 160+ lines of commented radar visualization code in runtime-stats.tsx - Code had been commented out for an extended period with no plans to restore - Reduces file size and improves readability 3. Replaced debug console.log/warn statements with proper logging - Updated drop-view error handling to fail silently - Added proper logger usage in teams and login components - Better production-ready error handling These changes improve code maintainability, reduce confusion, and provide a more professional user experience. The boring work nobody wants to do, but that makes the codebase genuinely better. 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
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.
This PR addresses several maintenance items I noticed while exploring the codebase. These are mostly small improvements that help with code quality and user experience. They were done with Claude Code.
Changes Made
Bug Fixes
Code Cleanup
User Experience Improvements
Performance
Testing
Notes
These are mostly small, unglamorous fixes, but I hope they help make the codebase a bit cleaner and more maintainable. Happy to make any adjustments based on your team's preferences or coding
standards.