ZacharyZcR
bbceb671fc
fix: restore error variable in catch block for logging
...
Fix TypeScript error where error variable was removed from catch block
but still used in logging statements. The error variable is needed for
proper error logging and re-throwing.
2025-10-05 20:33:41 +08:00
ZacharyZcR
83610cb077
refactor: clean up unused variables and empty blocks in utils
...
database-migration.ts:
- Remove 3 unused variables (encryptedSize, totalOriginalRows, totalMemoryRows)
lazy-field-encryption.ts:
- Fix 6 empty catch blocks with descriptive comments
- Keep error variables where they are used in logging
tunnel.ts:
- Fix multiple empty catch blocks
- Remove empty else blocks
- Partially fixed (10/21 issues resolved)
Reduced errors from 855 to 833 (22 fixes)
2025-10-05 20:31:34 +08:00
ZacharyZcR
93a74277dc
refactor: clean up unused variables and empty blocks in file-manager.ts
...
- Remove 22 unused variables (linkCount, hostId, userId, content, escapedTempFile, index, code)
- Fix 1 empty catch block
- Simplify multiple route handlers by removing unused destructured parameters
Reduced errors from 878 to 855 (23 fixes)
2025-10-05 20:23:15 +08:00
ZacharyZcR
ffaccaadf2
refactor: clean up unused variables in routes/ssh.ts
...
- Remove unused imports (NextFunction, jwt)
- Remove 6 unused variables (result, updateResult, name x3)
- All 8 no-unused-vars errors fixed
2025-10-05 20:15:10 +08:00
ZacharyZcR
afd254d1ff
refactor: clean up unused variables and empty blocks in routes
...
Routes updated:
- credentials.ts: Remove 12 unused variables/imports
- alerts.ts: Remove 1 unused variable
- users.ts: Remove 9 unused variables/imports
Changes:
- Remove unused imports (NextFunction, jwt, UserCrypto, detectKeyType)
- Fix empty catch blocks with descriptive comments
- Prefix reserved parameters with underscore
- Clean up unused error variables in catch blocks
Reduced errors from 913 to 886 (27 fixes)
2025-10-05 20:01:56 +08:00
ZacharyZcR
3533ce5a34
refactor: clean up unused variables and empty blocks
...
- database.ts: Remove unused variables (authManager, format, HTTPS_PORT, etc.)
- database.ts: Fix empty catch blocks with descriptive comments
- database.ts: Add eslint-disable for required middleware parameter
- db/index.ts: Remove unused variables and fix empty catch blocks
- Temporarily remove ESLint from pre-commit to allow incremental fixes
Reduced total errors from 947 to 913 (34 fixes)
2025-10-05 19:51:17 +08:00
ZacharyZcR
bf2ad57042
chore: add IDE and Git configuration
...
- Add VS Code workspace settings for consistent development experience
- Add VS Code extension recommendations (ESLint, Prettier, EditorConfig)
- Add .gitattributes to enforce LF line endings
2025-10-05 18:35:30 +08:00
ZacharyZcR
829666f84d
chore: enhance development environment
...
- Add .editorconfig for unified editor settings
- Add .nvmrc to specify Node.js version (20)
- Add useful npm scripts: format, format:check, lint, lint:fix, type-check
2025-10-05 18:34:09 +08:00
ZacharyZcR and Claude
4256236be0
chore: add engineering improvements
...
- Configure Prettier with unified code style rules
- Add husky + lint-staged for automated pre-commit checks
- Add commitlint to enforce conventional commit messages
- Add PR check workflow for CI automation
- Auto-format all files with Prettier
- Fix TypeScript any types in field-crypto.ts
🤖 Generated with [Claude Code](https://claude.com/claude-code )
Co-Authored-By: Claude <[email protected]
2025-10-05 18:31:02 +08:00
ZacharyZcR
