refactor: implement comprehensive naming convention standardization #160
Conversation
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
- Add UserMenu component with dropdown interface - Create Avatar UI component for user profile display - Refactor header to use proper Layout component instead of inline styles - Update Layout component to remove Sidebar dependency - Fix authentication state loading from IndexedDB - Remove unused Sidebar component - Update __root.tsx to use new Layout architecture - Add login button fallback when user is not authenticated - Implement proper logout flow with state clearing and redirect 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <[email protected]>
Add comprehensive development server management rules to prevent port infinite growth and resource waste: - Port checking and cleanup procedures - Recommended separate startup approach (services first, then client) - Clear guidelines for port occupation handling - Service verification commands - Advantages of separate startup over containerized scripts This ensures better controllability, clearer error localization, and prevents unexpected script complexities. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <[email protected]>
Fix the missing connection between keyboard shortcuts and the new note modal: - Add useShortcutAction hook to register 'openNewNoteModal' action handler - Import useShortcutAction from shortcut-provider - Add debug logging to verify shortcut activation - Connect Ctrl+N (Windows/Linux) and Cmd+N (Mac) shortcuts to actual modal opening The keyboard shortcuts were already defined in ShortcutProvider but the action handler was missing, preventing the modal from opening when shortcuts were pressed. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <[email protected]>
- Rename component files from kebab-case to PascalCase following React best practices - Rename hook files from kebab-case to camelCase following React official specifications - Update ESLint configurations across all packages to enforce naming conventions - Enhance tech-guide.md with detailed naming standards and examples - Fix all import references and ensure type checking passes 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <[email protected]>
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.
Summary
Changes
Button.tsx,NewNoteModal.tsx)useNewNoteModal.ts,useShortcuts.ts)docs/tech-guide.md中的命名规范说明Files Changed
apps/client,packages/ui等核心包Test Plan
🤖 Generated with Claude Code