fix(devcontainer): use Maven wrapper for dependency caching#330
Merged
MohsinHashmi-DataInn merged 6 commits intodevelopfrom Dec 22, 2025
Merged
fix(devcontainer): use Maven wrapper for dependency caching#330MohsinHashmi-DataInn merged 6 commits intodevelopfrom
MohsinHashmi-DataInn merged 6 commits intodevelopfrom
Conversation
- add explicit workspace mount ../:/workspaces/SimpleAccounts-UAE - use fixed workspace path instead of variable - fixes 'lstat /workspaces/content: no such file or directory' error 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- Bake Playwright system dependencies into Dockerfile - Combine apt-get layers for efficiency - Pre-cache Maven and NPM dependencies during build - Update post-create.sh to utilize pre-seeded caches
The base image mcr.microsoft.com/devcontainers/java:21-bookworm does not have Maven pre-installed, causing the prebuild to fail with exit code 127 (mvn: command not found). Changes: - Use Maven wrapper (./mvnw) instead of mvn command - Copy mvnw and .mvn/ directory for wrapper to work - Use dependency:resolve instead of go-offline to avoid HTTP repo blocking - Add graceful fallbacks (|| true) for partial dependency caching - Add npm install fallback when lockfiles are out of sync 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
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
3 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
mvn: command not found)mcr.microsoft.com/devcontainers/java:21-bookwormdoes not have Maven pre-installed./mvnw) which is already present in the projectChanges
mvnwand.mvn/directory alongsidepom.xml./mvnw dependency:resolveinstead ofmvn dependency:go-offline|| truefallbacks so image builds even if some dependencies failTest plan
docker buildx build --platform linux/amd64 -f .devcontainer/Dockerfile .Fixes: https://github.com/SimpleAccounts/SimpleAccounts-UAE/actions/runs/20424643047
🤖 Generated with Claude Code