Skip to content

Commit dbe8c1f

Browse files
committed
Fixed linter errors
1 parent c890fdf commit dbe8c1f

2 files changed

Lines changed: 10 additions & 1 deletion

File tree

.claude/agents/reviewer.md

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -14,11 +14,17 @@ Review code changes and provide constructive, actionable feedback. You cannot ed
1414

1515
1. Run `git diff` or `git diff --cached` to see recent changes
1616
2. Read the modified files for full context
17-
3. Analyze the changes against the checklist below
17+
3. **Run linting and formatting checks** — always run these regardless of what changed:
18+
- **Backend (C++)**: `clang-format --dry-run --Werror $(git diff --cached --name-only | grep -E '\.(cpp|h|c)$' | tr '\n' ' ')` — if any files match
19+
- **Frontend (JS/CSS)**: from the `frontend/` directory, run `npx eslint src/ test/` and `npx prettier --check src/ *.css` — if any frontend files changed
20+
- All linting errors (`error` severity) are blocking; warnings are informational
21+
4. Analyze the changes against the checklist below
1822

1923
## Review Checklist
2024

2125
### Critical (must fix)
26+
- Linting errors (`error` severity) in any changed file — block approval until fixed
27+
- Clang-format violations in C++ files — run `clang-format -i` to fix, then re-stage
2228
- Memory leaks or buffer overflows
2329
- Null pointer dereferences
2430
- Race conditions or deadlocks

frontend/eslint.config.js

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -37,6 +37,9 @@ export default [
3737
afterAll: 'readonly',
3838
},
3939
},
40+
rules: {
41+
'no-unused-vars': ['warn', { argsIgnorePattern: '^_' }],
42+
},
4043
},
4144

4245
// Config and build scripts — Node.js environment

0 commit comments

Comments
 (0)