Skip to content

feat: add new rotki plugin rules and fix type errors#79

Merged
kelsos merged 4 commits intorotki:mainfrom
kelsos:feat/new-rules
Feb 23, 2026
Merged

feat: add new rotki plugin rules and fix type errors#79
kelsos merged 4 commits intorotki:mainfrom
kelsos:feat/new-rules

Conversation

@kelsos
Copy link
Member

@kelsos kelsos commented Feb 20, 2026

Summary

  • Bump @rotki/eslint-plugin peer dependency to >=1.3.0
  • Add all new composable rules from the plugin (strict rules as warn, stylistic rules gated by stylistic option)
  • Replace @intlify/vue-i18n/no-unused-keys with the 14x faster @rotki/no-unused-i18n-keys, always enabled by default
  • Fix pre-existing type errors in formatters.ts, jsonc.ts, and yaml.ts (generic mergePrettierOptions, normalize IndentRuleOptions array form)

Test plan

  • pnpm run build passes
  • pnpm run typecheck passes (0 errors, previously had 4)
  • pnpm run lint passes (0 errors, only pre-existing complexity warnings)
  • pnpm run test passes (16/16)

BREAKING CHANGE: `as` type assertions are now forbidden. Use type guards,
generics, or `satisfies` instead.
@kelsos kelsos requested a review from a team as a code owner February 20, 2026 12:56
BREAKING CHANGE: `else` and `else if` blocks are now disallowed.
Use early returns or ternary operators instead.
Bump @rotki/eslint-plugin peer dependency to >=1.3.0 and enable new
composable rules (strict as warn, stylistic gated by stylistic option).
Replace @intlify/vue-i18n/no-unused-keys with the faster
@rotki/no-unused-i18n-keys rule, always enabled by default.
Make mergePrettierOptions generic to preserve parser type from overrides.
Normalize StylisticConfig indent to handle array form of IndentRuleOptions.
@kelsos kelsos changed the title Disable type assertions and add early return for if feat: add new rotki plugin rules and fix type errors Feb 20, 2026
@kelsos kelsos merged commit 3aac6af into rotki:main Feb 23, 2026
3 checks passed
@kelsos kelsos deleted the feat/new-rules branch February 23, 2026 09:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant