Skip to content

add claude config#346

Open
s-yonkov-yonkov wants to merge 3 commits into
masterfrom
add-claude-config
Open

add claude config#346
s-yonkov-yonkov wants to merge 3 commits into
masterfrom
add-claude-config

Conversation

@s-yonkov-yonkov
Copy link
Copy Markdown
Contributor

No description provided.

@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented May 4, 2026

Comment thread pom.xml
<version>3.4.0</version>
<configuration>
<!-- THE MAGIC LINE: Spotless will only format files changed relative to this branch -->
<ratchetFrom>origin/master</ratchetFrom>
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why only this branch? won't this break the local development because there will be no formatter?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It won't break formatting, it just limits which files get formatted. With this setup when spotless:apply is executed it will be triggered only for the files that are changed compared to master and the legacy files that were never formatted or updated will remain untouched.

Comment thread CLAUDE.md
Application (MTA) specification. It serves as a foundational dependency for the
Cloud Foundry MultiApps Controller.

## Security Boundary
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

does this security boundary note need to be doubled here in addition to in the rooot md file? Can it not only have a single source of truth in the root?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I presume that we can keep both the root instructions along with the local instructions as well. For example if we have a multi agent development and when an AI agent is invoked inside multiapps/, there is no guarantee the root CLAUDE.md is in context — especially in sub-agent invocations, IDE integrations, or any session where the working directory is the sub-repo directly. If the root table isn't loaded, and the sub-repo has no boundary note, the AI has zero guardrails against leaking proprietary content.

Comment thread CLAUDE.md
Before completing any task or committing code, you **MUST** run:

```bash
mvn spotless:apply
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this and the above mvn commands would probably be the same for all repos using maven - would it not shrink the context to have them in the root md file and keep the local files only for project specific things?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants