refactor
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.
Source facts
- Repository
- adobe/S3Mock
- Last source activity
- July 27, 2026 at 17:34
- Detected SKILL.md language
- English
- Stars
- 1,174
- Forks
- 200
Install options
The review-first prompt is selected by default. You can switch to a direct command or download a local copy.
Review the source files
Read SKILL.md and any companion files shown by SkillsMP before deciding whether to install.
Showing SKILL.md
SKILL.md
Source instructions · Read-only preview- name
- refactor
- description
- 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.
# Refactor Skill
## Entry Criteria
Use to clean up code structure/naming, improve or add KDoc, remove anti-patterns, or prepare code before a feature addition (invoked by the `implement`/`test` skill).
**Hard constraint**: must not change observable behavior. If a change would alter behavior, it belongs in the `implement` skill instead.
## Before Starting
1. Read root `AGENTS.md` and the relevant module `AGENTS.md`.
2. Read callers and tests to understand the code's purpose before making any changes.
3. Confirm existing test coverage is sufficient to catch regressions.
## Execution Steps
1. Apply naming, comment, KDoc, and idiom fixes per **[docs/KOTLIN.md](../../../docs/KOTLIN.md)**.
2. Run `make test` and/or `make integration-tests` to verify no behavior changed.
3. Update the copyright year in every modified file (`INVARIANTS.md`).
4. Invoke the **`lint` skill**.
## Completion Criteria
- [ ] No behavior changes — tests still pass
- [ ] Follows `docs/KOTLIN.md` naming, comment, and idiom conventions
- [ ] Copyright year updated in every modified file
- [ ] Lint gates pass (via `lint` skill)
## Resources
- [`AGENTS.md`](../../../AGENTS.md), [`INVARIANTS.md`](../../../INVARIANTS.md)
- Relevant module `AGENTS.md`
- [`docs/KOTLIN.md`](../../../docs/KOTLIN.md)
View on GitHub