Skip to content

feat: language-specific analyzers (#233) - #241

Merged
ajianaz merged 1 commit into
developfrom
feat/233-language-analyzers
Jun 10, 2026
Merged

ajianaz merged 1 commit into
developfrom
feat/233-language-analyzers

Conversation

@ajianaz

@ajianaz ajianaz commented Jun 10, 2026

Copy link
Copy Markdown
Collaborator

Enriches LLM review prompt with language-specific guidance based on detected file extensions.

Languages

Language Focus Areas
Dart/Flutter Widget lifecycle, state management, null safety
Svelte/SvelteKit Reactive declarations, store patterns, SSR
TypeScript/JS Type safety, null handling, async
Go Error handling, goroutine safety, context
Rust Unsafe usage, ownership, lifetimes
Python Type hints, exceptions, security

Changes

  • New src/engine/language_analyzer.rs
  • Added .svelte to diff_parser language detection
  • Integrated into review pipeline (injected into LLM prompt)
  • 8 unit tests, 437 total tests pass

…hon (#233)

Enriches LLM review prompt with language-specific guidance:
- Dart/Flutter: widget lifecycle, state management, null safety
- Svelte/SvelteKit: reactive declarations, store patterns, SSR
- TypeScript: type safety, null handling, async patterns
- Go: error handling, goroutine safety, context propagation
- Rust: unsafe usage, ownership, lifetime, error handling
- Python: type hints, exception handling, security

Auto-detects languages from diff file extensions.
Added .svelte to diff_parser language detection.
8 unit tests.
@ajianaz
ajianaz merged commit 393b09b into develop Jun 10, 2026
10 checks passed
@ajianaz
ajianaz deleted the feat/233-language-analyzers branch June 11, 2026 07:27
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