davsclaus commented on code in PR #24251:
URL: https://github.com/apache/camel/pull/24251#discussion_r3474969654
##########
.oss-ai-helper-rules/project-standards.md:
##########
@@ -0,0 +1,16 @@
+# Project Standards
+
+This rule file contains build tools, commands, and code style constraints for
the project. Commands read this file to determine how to build, test, and
format code.
+
+- **Build tool:** Maven
+- **Build command:** `mvn -Dquickly install`
+- **Test command:** `mvn verify`
+- **Format command:** `cd <module> && mvn -DskipTests install`
+- **Module-specific build:** yes (always run `mvn` in the module directory
where changes occurred)
+- **Parallelized Maven:** no (resource intensive, do NOT parallelize Maven
jobs)
+- **Code style restrictions:**
+ - Do NOT use Lombok (unless already present in the file)
+ - Do NOT change public API signatures without justification
+ - Do NOT add new dependencies without justification
+ - Maintain backwards compatibility for public APIs
Review Comment:
Missing a code style rule that exists in `CLAUDE.md`:
```suggestion
- Do NOT add new dependencies without justification
- Records are allowed for internal/non-API classes; do NOT convert
existing public API classes to Records
```
##########
.oss-ai-helper-rules/project-standards.md:
##########
@@ -0,0 +1,16 @@
+# Project Standards
+
+This rule file contains build tools, commands, and code style constraints for
the project. Commands read this file to determine how to build, test, and
format code.
+
+- **Build tool:** Maven
+- **Build command:** `mvn -Dquickly install`
+- **Test command:** `mvn verify`
Review Comment:
This skips tests. The canonical build command per `CLAUDE.md` and the
existing project rules is `mvn verify`. An AI agent using `mvn -Dquickly
install` as its default build command would never run tests.
```suggestion
- **Build command:** `mvn verify`
```
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
