Skip to content

Conversation

@kpavlov
Copy link
Contributor

@kpavlov kpavlov commented Jan 11, 2026

Integrate Detekt for static code analysis

  • Add Detekt configuration and gradle plugin. ⚠️ Don't fail on errors yet
  • Update codeql.yml to include Detekt in the CI workflow
  • Define Detekt settings in a new detekt.yml file
  • Fix or suppress some current detekt warnings

Motivation and Context

To ensure code quality checks

How Has This Been Tested?

CI

Breaking Changes

No

Types of changes

  • CI update

Checklist

  • I have read the MCP Documentation
  • My code follows the repository's style guidelines
  • New and existing tests pass locally
  • I have added appropriate error handling
  • I have added or updated documentation as needed

Additional context

- Add Detekt configuration and plugin to `build.gradle.kts`
- Update `codeql.yml` to include Detekt in the CI workflow
- Define Detekt settings in a new `detekt.yml` file
@kpavlov kpavlov added the enhancement New feature or request label Jan 11, 2026
@kpavlov kpavlov force-pushed the kpavlov/detekt branch 2 times, most recently from c0757bb to 3bceafa Compare January 11, 2026 09:11
…tekt errors yet

- Suppressed deprecation warnings in multiple "Old" test files.
- Fixed some detekt issues.
- Renamed TypeScript server/client methods for clarity (`startSse` → `startSseServer`, etc.).
- Updated Detekt configuration.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants