docs: document -Ppre-commit as auto-fixing, not a pass/fail check

[cuioss-oliver]

Aligns this repo's -Ppre-commit framing to what the command actually does.

-Ppre-commit is a formatter, not a pass/fail check. The profile inherited from cui-java-parent binds two mutating executions ahead of verify:

• license:format (process-sources) — adds or updates license headers
• rewrite:run (no declared phase → descriptor default process-test-classes) — applies the OpenRewrite recipes

Both run before verify, so a run that repaired the tree and a run that changed nothing both exit 0. Documenting it as a "check", "gate", or "quality verification" implies a green run means the tree was already clean — it does not. Empirically: strip a license header, run -Ppre-commit verify, and it exits 0 with the header silently restored and git status clean.

The corpus carried two contradictory framings. This aligns everything to: -Ppre-commit auto-fixes; review what it changed and commit it.

The reference wording is cuioss-parent-pom's own docs, which already say it correctly ("Adds or updates Apache 2.0 license headers…", "Applies all configured OpenRewrite recipes…") and are unchanged.

Note this is a documentation change only — no build behaviour changes. Auto-fixing is the intended org-wide norm, in every language.

🤖 Generated with Claude Code

[coderabbitai]

Important

Review skipped

Auto reviews are limited based on label configuration.

🚫 Excluded labels (none allowed) (1)

• skip-bot-review

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository: cuioss/coderabbit/.coderabbit.yaml

Review profile: CHILL

Plan: Advanced

Run ID: 1586507e-e60e-479d-9959-461807d353ce

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[sourcery-ai]

Hey - I've found 1 issue

Prompt for AI Agents

Please address the comments from this code review:

## Individual Comments

### Comment 1
<location path="CLAUDE.md" line_range="20" />
<code_context>
 ./mvnw test -Dtest=ClassName#methodName

-# Pre-commit checks
+# Pre-commit auto-fix (license headers + formatting) - review and commit what it changed
 ./mvnw -Ppre-commit clean verify -DskipTests
 ```
</code_context>
<issue_to_address>
**nitpick:** The new descriptions reduce the profile's mutations to “license headers + formatting,” but `rewrite:run` applies all configured OpenRewrite recipes, which can make non-formatting source changes. Users following this wording can review the profile as a formatter only and overlook semantic recipe changes.

**Triggers:** When a configured OpenRewrite recipe performs a transformation that is not merely formatting.

**Suggested fix:** Describe the profile as applying license formatting and all configured OpenRewrite recipes, and require reviewing every resulting diff.

```suggestion
# Pre-commit auto-fix (license formatting + all configured OpenRewrite recipes) - review every resulting diff before committing
```
</issue_to_address>

Sourcery assessment

Approved.

---

Sourcery is free for open source - if you like our reviews please consider sharing them ✨

• X
• Mastodon
• LinkedIn
• Facebook

[sourcery-ai]

Sourcery assessment

Approved.

[sourcery-ai]

Sourcery assessment

Approved.

Sourcery withdrew this approval because the latest commits introduced blocking findings.