fix: resolve lint issues in analytics route (#174)#246
Conversation
|
Quick status: local lint proof is green for this scope ( |
Could you please attach terminals proof in description. |
|
added the terminal proof to the PR description |
Sorry but I cannot see changes made in PR description. |
|
reran locally, terminal output: also re-saved the PR description with the same proof block. |
|
reran from my local clone (DevCard):
there is no eslint output for this file when it passes. |
|
reran on current head from local clone: eslint prints no line output when this file passes, so exit code is the only signal here. |

Summary Fixes ESLint issues in
apps/backend/src/routes/analytics.tsfor issue #174 by removing unused handler args, avoiding unbound method usage for auth preHandler, and adding an explicit return type. Closes #174 --- ## Type of Change - [x] Bug fix - [ ] New feature - [ ] Refactor (no functional change) - [ ] UI / Design change - [ ] Tests only - [ ] Documentation - [ ] Infrastructure / DevOps - [ ] Security --- ## What Changed - AddedPromise<void>return type toanalyticsRoutes. - Replaced per-routepreHandler: [app.authenticate]with a hook wrapper call to avoid@typescript-eslint/unbound-method. - Removed unusedFastifyReplyimport and unusedreplyargs from analytics handlers. --- ## How to Test 1.pnpm install2.cd apps/backend3.pnpm exec eslint src/routes/analytics.ts--- ## Checklist - [ ] My code follows the project's coding style (pnpm -r run lintpasses). - [ ] TypeScript compiles without errors (pnpm -r run typecheck). - [ ] I have added or updated tests for the changes I made. - [ ] All tests pass locally (pnpm -r run test). - [ ] I have updated documentation where necessary. - [x] No newconsole.logor debug statements left in the code. - [ ] Breaking changes are documented in this PR description. --- ## Additional Context Scope is intentionally limited toapps/backend/src/routes/analytics.tsas requested in #174.Terminal Proof
apps/backend/node_modules/.bin/eslint src/routes/analytics.ts