Skip to content

Ban any, unknown and casts through unknown in ui's lint (GRYT-1792) - #294

Merged
grytchat merged 1 commit into
mainfrom
claude/GRYT-1792-ui-lint
Oct 19, 2026
Merged

grytchat merged 1 commit into
mainfrom
claude/GRYT-1792-ui-lint

Conversation

@grytchat

Copy link
Copy Markdown
Contributor

The ui half of Gryt-chat/client#890 and Gryt-chat/server#299. One file, eslint.config.js, and no code changes.

New errors on **/*.{ts,tsx}: no-explicit-any, unknown (via no-restricted-types), x as unknown as Y, no-nested-ternary, no-non-null-assertion, no-param-reassign, eqeqeq (smart), no-else-return, prefer-const, consistent-type-imports.

Existing debt is listed per rule:

  • unknown: 14 files, grouped by why, 6 of them "not typed yet"
  • nested ternaries: 27
  • non-null assertions: 26
  • casts through unknown: 3
  • type-only imports: 2

The warning count stays at the 114 cap. No changeset, since nothing in the published package changes.

Vikunja: GRYT-1792

🤖 Generated with Claude Code

The same rules as the client and server: any, unknown, casts through
unknown, nested ternaries, non-null assertions, reassigned parameters,
loose equality, else after return, let that could be const, and unmarked
type-only imports are errors. Existing debt is listed per rule in the
config; no code file changes.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@grytchat
grytchat merged commit 701edf0 into main Oct 19, 2026
7 checks passed
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