Skip to content

fix(website): resolve all 27 lint warnings#96

Merged
blove merged 1 commit into
mainfrom
fix/lint-warnings
Apr 10, 2026
Merged

fix(website): resolve all 27 lint warnings#96
blove merged 1 commit into
mainfrom
fix/lint-warnings

Conversation

@blove
Copy link
Copy Markdown
Contributor

@blove blove commented Apr 10, 2026

Summary

  • Remove unused _request param in hello API route
  • Eliminate non-null assertions in ArchFlowDiagram by extracting chatBubble to local var
  • Add eslint-disable for unavoidable any cast in MdxRenderer rehype plugin config
  • Guard msgsRef.current null check and use optional chaining in ChatFeaturesSection
  • Add file-level eslint-disable for generate-api-docs.ts (TypeDoc reflection API is untyped)

Result: 27 warnings → 0 warnings

Test plan

  • nx lint website — 0 errors, 0 warnings
  • nx build website — success
  • CI green

🤖 Generated with Claude Code

- hello/route.ts: remove unused _request parameter
- ArchFlowDiagram.tsx: remove unused loop var, eliminate non-null assertions
- MdxRenderer.tsx: eslint-disable for unavoidable rehype plugin any cast
- ChatFeaturesSection.tsx: guard msgsRef.current, use optional chaining
- generate-api-docs.ts: eslint-disable file for untyped TypeDoc reflection API

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@vercel
Copy link
Copy Markdown

vercel Bot commented Apr 10, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
cacheplane Ready Ready Preview, Comment Apr 10, 2026 6:18am

Request Review

@blove blove merged commit 0b0745f into main Apr 10, 2026
14 checks passed
@blove blove deleted the fix/lint-warnings branch May 7, 2026 16:30
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