Skip to content

Update CLAUDE.md #628

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
merged 2 commits into from
Oct 21, 2025
+43 −8
Merged

Update CLAUDE.md #628

Changes from 1 commit
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
2d711cf
Update CLAUDE.md
EhabY Oct 17, 2025
c4d1d51
Simplify CLAUDE.md
EhabY Oct 21, 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
Prev Previous commit
Simplify CLAUDE.md
  • Loading branch information
@EhabY
EhabY committed Oct 21, 2025
commit c4d1d51a94b02a5a9bfc31b165f343cc54aa155f
6 changes: 2 additions & 4 deletions CLAUDE.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -51,14 +51,12 @@ Comments explain what code does or why it exists:
## Code Style

- TypeScript with strict typing
- Use semicolons
- Trailing commas for all multi-line lists
- 120 character line width
- Use Prettier for code formatting and ESLint for code linting
- Use ES6 features (arrow functions, destructuring, etc.)
- Use `const` by default; `let` only when necessary
- Never use `any`, and use exact types when you can
- Prefix unused variables with underscore (e.g., `_unused`)
- Sort imports alphabetically in groups (see `import/order` in `.eslintrc.json`): external → parent → sibling
- Error handling: wrap and type errors appropriately
- Use async/await for promises, avoid explicit Promise construction where possible
- Unit test files must be named `*.test.ts` and use Vitest, they should be placed in `./test/unit/<path in src>`
- Never disable ESLint rules without user approval