Skip to content

Commit ba1bcfc

Browse files
committed
Merge branch 'main' into refactor/ingestion-safety-hardening
2 parents 4f7c9b8 + d6f8b0e commit ba1bcfc

4,165 files changed

Lines changed: 410738 additions & 107169 deletions

File tree

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.
Lines changed: 58 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,58 @@
1+
---
2+
name: java-checkstyle
3+
description: Run `mvn spotless:apply` to fix Java checkstyle / formatting failures and verify the result. Run after authoring or modifying any `.java` files, or when CI reports a "Java checkstyle failed" / "Fix Java checkstyle" issue on a PR.
4+
---
5+
6+
# Java Checkstyle / Spotless (Codex agent)
7+
8+
OpenMetadata enforces Java formatting via the Spotless Maven plugin. Every CI
9+
build runs `mvn spotless:check` and fails the PR if any file is not formatted.
10+
11+
## When to activate
12+
13+
- The user asks to "fix checkstyle", "fix Java formatting", "apply spotless",
14+
"run spotless", "format Java", or similar.
15+
- CI posts a `Java checkstyle failed` / `Fix Java checkstyle` comment on a PR
16+
(the bot's exact phrasing is "Please run `mvn spotless:apply` in the root of
17+
your repository and commit the changes to this PR").
18+
- After you have finished authoring or editing any `.java` files — before
19+
opening a PR or pushing a commit that touches Java.
20+
21+
## Procedure
22+
23+
1. From the repo root run Spotless:
24+
25+
```bash
26+
mvn spotless:apply # formats everything
27+
# or
28+
mvn -pl <module> spotless:apply # scope to a single Maven module for speed
29+
# or
30+
mvn spotless:check # verify only, without rewriting files
31+
```
32+
33+
Spotless is fast (seconds, no compilation). If it fails with a plugin error
34+
rather than a formatting diff, surface the error and stop — do not try to
35+
hand-edit formatting around the failure.
36+
37+
2. Inspect the diff:
38+
39+
```bash
40+
git status --short
41+
git diff --stat
42+
```
43+
44+
Expect changes only in `.java` (and possibly `pom.xml`) files. If Spotless
45+
keeps rewriting a change you just made, re-read the root `pom.xml`'s
46+
`spotless-maven-plugin` config — Spotless is the source of truth, not the
47+
IDE.
48+
49+
3. Only commit if the user asked to. Report the changed-file list first so the
50+
user can decide whether to fold the reformat into the in-progress commit or
51+
make a separate "Fix Java checkstyle" commit (matches the repo's existing
52+
history for bot-triggered formatting-only commits).
53+
54+
## Out of scope
55+
56+
- UI / TypeScript formatting — use `yarn pretty` / ESLint flow (see AGENTS.md
57+
UI section).
58+
- Python formatting — use `make py_format` (black + isort + pycln).

.agents/skills/playwright-validation/SKILL.md

Lines changed: 0 additions & 175 deletions
This file was deleted.

.agents/skills/playwright/SKILL.md

Lines changed: 0 additions & 123 deletions
This file was deleted.

0 commit comments

Comments
 (0)