// Fix code style issues and ensure linting CI gates pass in S3Mock. Use when ktlint or Checkstyle violations are reported, when asked to fix formatting, or as a required final step after any code change. Runs ktlint for Kotlin source files and Checkstyle for Java and XML files.
| name | lint |
| description | Fix code style issues and ensure linting CI gates pass in S3Mock. Use when ktlint or Checkstyle violations are reported, when asked to fix formatting, or as a required final step after any code change. Runs ktlint for Kotlin source files and Checkstyle for Java and XML files. |
Use this skill when:
implement, test, or refactor skill| Tool | Target | Config | Auto-fix? |
|---|---|---|---|
| ktlint | Kotlin source files | .editorconfig | Yes — make format |
| Checkstyle | Java source + XML files | etc/checkstyle.xml | No — fix manually |
make format — auto-formats all Kotlin files with ktlint. Fixes the vast majority of Kotlin style issues../mvnw checkstyle:check — reports Checkstyle violations for Java and XML files. Fix violations manually.make sort — run this if any pom.xml was added or modified during this task.make install — confirm all linting CI gates pass before finishing.make format auto-fixes most of these:
To check without modifying files: ./mvnw ktlint:check
Fix these manually:
.editorconfig)make format run (Kotlin auto-fixed)./mvnw checkstyle:check passes with no violationsmake sort run if any pom.xml was modifiedmake install passes all CI gatesAGENTS.md — build command referencedocs/KOTLIN.md — Kotlin style conventionsdocs/JAVA.md — Java style conventionsetc/checkstyle.xml — Checkstyle configuration.editorconfig — indentation and line-length settingsGenerate and update S3Mock project documentation. Use when asked to document code, update CHANGELOG.md, update README.md, update AGENTS.md files, or fix formatting, wording, or style in any existing documentation file. Also invoked at the end of the implement skill workflow.
Implement features, fix bugs, and modify production source code in S3Mock. Use when asked to add functionality, change behavior, or modify source files in server/, integration-tests/, or testsupport/. Not for documentation-only or test-only changes. Orchestrates lint, test, and document skills as part of the complete workflow.
Improve code readability, enforce idiomatic Kotlin conventions, and enhance documentation quality in S3Mock without changing behavior. Use when asked to clean up code, improve naming, add KDoc, or align with Kotlin/project idioms. Also invoked by the implement and test skills before adding new code to messy areas.
Review code changes in pull requests or local diffs for S3Mock. Use when asked to review a PR, inspect changes, or provide structured feedback on correctness, conventions, test quality, and documentation. Outputs findings structured as Must fix / Should fix / Nit.
Write, update, and fix tests in S3Mock. Use when asked to add test coverage, create test cases, fix failing tests, or verify test correctness. Covers unit tests (*Test.kt in server/), controller slice tests (@WebMvcTest), and integration tests (*IT.kt in integration-tests/ against a live Docker container).