Turn the docker Makefile target into an e2e full target

[jviotti]

Signed-off-by: Juan Cruz Viotti jv@jviotti.com

[augmentcode]

🤖 Augment PR Summary

Summary: Repurposes the docker Makefile target to run a full Docker-based E2E flow (build image, build+start sandbox, run E2E/UI tests, then teardown).
CI: Updates GitHub Actions to call make docker-build so CI keeps only building the image instead of running the full suite.

_{🤖 Was this summary useful? React with 👍 or 👎}

[augmentcode]

Review completed. 2 suggestions posted.

Comment augment review to trigger a new review at any time.

[cubic-dev-ai]

1 issue found across 2 files

Prompt for AI agents (unresolved issues)

Check if these issues are valid — if so, understand the root cause of each and fix them. If appropriate, use sub-agents to investigate and fix each issue separately.


<file name="Makefile">

<violation number="1" location="Makefile:37">
P1: If `test-e2e` or `test-ui` fails, Make will abort before reaching `docker-sandbox-down`, leaving Docker sandbox containers running and ports occupied. The teardown step should be guaranteed to run regardless of test outcome (similar to a `trap` or `try/finally` pattern). Consider using a recipe that captures the test exit code and always runs teardown.</violation>
</file>

_{Reply with feedback, questions, or to request a fix. Tag @cubic-dev-ai to re-run a review.}

[jviotti]

augment review

[augmentcode]

Review completed. 1 suggestions posted.

Comment augment review to trigger a new review at any time.