Skip to content

[Infra] Stricter tsconfig flags (noUnusedLocals/Parameters/ImplicitReturns/Fallthrough/ImplicitOverride) are commented out #1214

Description

@grantfox-oss

Telegram (ask questions / claim the issue here first): https://t.me/+DOylgFv1jyJlNzM0

Why this matters

backend/tsconfig.json:25-30 leaves noImplicitReturns, noImplicitOverride, noUnusedLocals, noUnusedParameters, and noFallthroughCasesInSwitch disabled. Enabling them surfaces dead code, missing returns, and switch fallthrough at build/typecheck time - the class of defects already being filed manually (e.g. dead route registrations).

Acceptance criteria

  • Enable noUnusedLocals, noUnusedParameters, noImplicitReturns, and noFallthroughCasesInSwitch (at minimum)
  • Fix or annotate resulting errors so npm run typecheck stays green
  • Keep the eslint no-unused-vars argsIgnorePattern aligned

Files to touch

  • backend/tsconfig.json

Out of scope

  • exactOptionalPropertyTypes / noUncheckedIndexedAccess (already enabled)

Metadata

Metadata

Labels

Stellar WaveIssues in the Stellar wave programbackendIssues related to backend developmentenhancementNew feature or requestinfraCI/CD and infrastructure

Type

No type
No fields configured for issues without a type.

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions