feat(sandbox): mvnw-fallback in Maven MCP-agent + README split#26
Merged
Conversation
- maven_agent.py: gebruik ./mvnw als die executable bestaat in PROJECT_DIR; anders fallback naar `mvn` op PATH. Maven hoeft dan niet meer op de host geïnstalleerd te zijn voor projecten met een wrapper. - maven_agent.py: logging-filter dat de SDK-warning "Received request before initialization was complete" aanvult met een reconnect-hint (en de waarschuwing om in /mcp NIET 'Authenticate' te kiezen — deze server heeft geen auth-laag, dat eindigt in een 404). - docs/maven-mcp-agent.md (nieuw): Maven MCP-sectie verplaatst uit README; troubleshooting (UFW/firewalld op host, OAuth-404) en JAVA_HOME-noot toegevoegd. - docs/opstarten-en-afsluiten.md (nieuw): Opstarten / Na installatie / Afsluiten verplaatst uit README. - README: ingekort van 366 naar 149 regels; Quick start verwijst via een bullet-lijst naar de docs en de Firewall-sectie. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
ericwout-overheid
requested review from
loek-rijksoverheid and
mreuvekamp
as code owners
mreuvekamp
approved these changes
May 13, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
mvnw-fallback in de Maven MCP-host-agent: gebruikt./mvnwals die executable bestaat inPROJECT_DIR, andersmvnop PATH. Geen Maven-host-installatie meer nodig voor projecten met een wrapper.Received request before initialization was completeaan met een actionable tip (én een waarschuwing om in/mcpgéén Authenticate te kiezen — deze server heeft geen auth-laag, wat eindigt in een verwarrende404 / JSON parse error).claude-sandbox/docs/:maven-mcp-agent.md— incl. troubleshooting voor host-firewall (UFW/firewalld blokkeert poort 7777 van Docker-bridge), de OAuth-404 bij Authenticate, en JAVA_HOME-noot bij host-agent start.opstarten-en-afsluiten.md— Opstarten, Na installatie en Afsluiten secties.Test plan
mvnwen gebruikt die (zichtbaar in proces-tree / Maven-output).mvnwvalt terug opmvnop PATH.docs/...werken in GitHub-rendering.[Optionele componenten](docs/opstarten-en-afsluiten.md#optionele-componenten)werkt vanuit de README.compose.override.yml(host-gateway) + UFW-regel op compose-bridge-subnet →host.docker.internal:7777bereikbaar.🤖 Generated with Claude Code