From 5036ee8f41a8a105a9e30315614fc3490a2e1a25 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Manuel=20Kie=C3=9Fling?= <manuel@kiessling.net>
Date: Mon, 16 Feb 2026 21:44:38 +0100
Subject: [PATCH 1/9] =?UTF-8?q?e2e:=20full=20flow=20with=20simulated=20ext?=
 =?UTF-8?q?ernals=20(sign=20in=20=E2=86=92=20create=20project=20=E2=86=92?=
 =?UTF-8?q?=20editor)?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

- Add test doubles for APP_ENV=test: SimulatedRemoteManifestValidator,
  SimulatedS3AssetUploader, SimulatedGitAdapter (clone from fixture, no-op push),
  SimulatedGitHubAdapter (fake PR URL), E2eNoOpProjectSetupStepsProvider
- Wire doubles in config/services.yaml when@test; e2e fixture path parameter
- E2E fixture: tests/fixtures/e2e-workspace-template (minimal package.json + index.html)
- Set APP_ENV=test for messenger in docker-compose.e2e.yml so async setup uses doubles
- Add data-test-id / data-test-class to project list, project form, editor, workspace setup
- New e2e spec: full-flow.spec.ts (sign in, create project, open editor, send message)
- No external calls in e2e: LLM, manifest URL, S3, git clone, GitHub API all simulated
---
 config/services.yaml                          |  25 ++++
 docker-compose.e2e.yml                        |   6 +-
 .../templates/chat_based_content_editor.twig  |   3 +
 .../Resources/templates/workspace_setup.twig  |   2 +-
 .../Resources/templates/project_form.twig     |   6 +
 .../Resources/templates/project_list.twig     |   5 +-
 .../SimulatedRemoteManifestValidator.php      |  26 ++++
 .../TestHarness/SimulatedS3AssetUploader.php  |  42 ++++++
 .../E2eNoOpProjectSetupStepsProvider.php      |  32 +++++
 .../TestHarness/SimulatedGitAdapter.php       | 126 ++++++++++++++++++
 .../TestHarness/SimulatedGitHubAdapter.php    |  35 +++++
 tests/End2End/tests/full-flow.spec.ts         |  69 ++++++++++
 .../e2e-workspace-template/index.html         |  10 ++
 .../e2e-workspace-template/package.json       |   8 ++
 14 files changed, 392 insertions(+), 3 deletions(-)
 create mode 100644 src/RemoteContentAssets/TestHarness/SimulatedRemoteManifestValidator.php
 create mode 100644 src/RemoteContentAssets/TestHarness/SimulatedS3AssetUploader.php
 create mode 100644 src/WorkspaceMgmt/TestHarness/E2eNoOpProjectSetupStepsProvider.php
 create mode 100644 src/WorkspaceMgmt/TestHarness/SimulatedGitAdapter.php
 create mode 100644 src/WorkspaceMgmt/TestHarness/SimulatedGitHubAdapter.php
 create mode 100644 tests/End2End/tests/full-flow.spec.ts
 create mode 100644 tests/fixtures/e2e-workspace-template/index.html
 create mode 100644 tests/fixtures/e2e-workspace-template/package.json

diff --git a/config/services.yaml b/config/services.yaml
index d060cc6f..7bbc374c 100644
--- a/config/services.yaml
+++ b/config/services.yaml
@@ -25,6 +25,9 @@ parameters:
     photo_builder.openai_timeout_seconds: 120
     llm_content_editor.simulate: "%env(bool:LLM_CONTENT_EDITOR_SIMULATE)%"
 
+    # E2E workspace fixture (used by SimulatedGitAdapter when APP_ENV=test)
+    e2e.workspace_fixture_path: "%kernel.project_dir%/tests/fixtures/e2e-workspace-template"
+
 services:
     # default configuration for services in *this* file
     _defaults:
@@ -210,3 +213,25 @@ services:
 
     App\LlmContentEditor\Facade\LlmContentEditorFacadeInterface:
         alias: App\LlmContentEditor\Facade\SwitchableLlmContentEditorFacade
+
+when@test:
+    services:
+        # E2E/test doubles: no external HTTP or git/GitHub calls
+        App\RemoteContentAssets\Infrastructure\RemoteManifestValidatorInterface:
+            class: App\RemoteContentAssets\TestHarness\SimulatedRemoteManifestValidator
+
+        App\RemoteContentAssets\Infrastructure\S3AssetUploaderInterface:
+            class: App\RemoteContentAssets\TestHarness\SimulatedS3AssetUploader
+
+        App\WorkspaceMgmt\Infrastructure\Adapter\GitAdapterInterface:
+            class: App\WorkspaceMgmt\TestHarness\SimulatedGitAdapter
+            arguments:
+                $workspaceFixturePath: "%e2e.workspace_fixture_path%"
+                $realGitAdapter: "@App\\WorkspaceMgmt\\Infrastructure\\Adapter\\GitCliAdapter"
+
+        App\WorkspaceMgmt\Infrastructure\Adapter\GitHubAdapterInterface:
+            class: App\WorkspaceMgmt\TestHarness\SimulatedGitHubAdapter
+
+        # E2E: no-op setup steps (priority so it is used instead of DefaultProjectSetupStepsProvider)
+        App\WorkspaceMgmt\TestHarness\E2eNoOpProjectSetupStepsProvider:
+            tags: [{ name: workspace_mgmt.setup_steps_provider, priority: 100 }]
diff --git a/docker-compose.e2e.yml b/docker-compose.e2e.yml
index e7f3d2b0..293ffe45 100644
--- a/docker-compose.e2e.yml
+++ b/docker-compose.e2e.yml
@@ -1,10 +1,14 @@
-# E2E override: run app in test env, nginx with test env and fixed port for Playwright.
+# E2E override: run app and messenger in test env, nginx with test env and fixed port for Playwright.
 # Use: docker compose -f docker-compose.yml -f docker-compose.e2e.yml up -d
 services:
     app:
         environment:
             APP_ENV: test
 
+    messenger:
+        environment:
+            APP_ENV: test
+
     nginx:
         ports:
             - "127.0.0.1:8080:80"
diff --git a/src/ChatBasedContentEditor/Presentation/Resources/templates/chat_based_content_editor.twig b/src/ChatBasedContentEditor/Presentation/Resources/templates/chat_based_content_editor.twig
index daaba15b..dc4af916 100644
--- a/src/ChatBasedContentEditor/Presentation/Resources/templates/chat_based_content_editor.twig
+++ b/src/ChatBasedContentEditor/Presentation/Resources/templates/chat_based_content_editor.twig
@@ -12,6 +12,7 @@
     {% endif %}
 
     <div class="space-y-6"
+         data-test-id="editor-page"
          {{ stimulus_controller('chat-based-content-editor', {
              runUrl: runUrl,
              pollUrlTemplate: pollUrlTemplate,
@@ -226,6 +227,7 @@
                                 <label for="instruction" class="block text-sm font-medium text-dark-700 dark:text-dark-300 mb-1">{{ 'editor.input_label'|trans }}</label>
                                 <textarea {{ stimulus_target('chat-based-content-editor', 'instruction') }}
                                           {{ stimulus_action('chat-based-content-editor', 'handleKeydown', 'keydown') }}
+                                          data-test-id="editor-message-input"
                                           name="instruction"
                                           id="instruction"
                                           rows="3"
@@ -236,6 +238,7 @@
                             <div class="flex items-center gap-4">
                                 <button type="submit"
                                         {{ stimulus_target('chat-based-content-editor', 'submit') }}
+                                        data-test-id="editor-send-button"
                                         class="px-4 py-2 rounded-md bg-primary-600 text-white text-sm font-medium hover:bg-primary-700 focus:outline-none focus:ring-2 focus:ring-primary-500 focus:ring-offset-2 dark:focus:ring-offset-dark-900 disabled:opacity-50 disabled:cursor-not-allowed">
                                     {{ 'editor.make_changes'|trans }}
                                 </button>
diff --git a/src/ChatBasedContentEditor/Presentation/Resources/templates/workspace_setup.twig b/src/ChatBasedContentEditor/Presentation/Resources/templates/workspace_setup.twig
index 33a44547..92b939f1 100644
--- a/src/ChatBasedContentEditor/Presentation/Resources/templates/workspace_setup.twig
+++ b/src/ChatBasedContentEditor/Presentation/Resources/templates/workspace_setup.twig
@@ -3,7 +3,7 @@
 {% block title %}{{ 'workspace.setup.title'|trans }}{% endblock %}
 
 {% block content %}
-    <div class="max-w-2xl mx-auto space-y-6" {{ stimulus_controller('workspace-setup', {
+    <div class="max-w-2xl mx-auto space-y-6" data-test-id="workspace-setup-page" {{ stimulus_controller('workspace-setup', {
         pollUrl: pollUrl,
         redirectUrl: redirectUrl,
         translations: {
diff --git a/src/ProjectMgmt/Presentation/Resources/templates/project_form.twig b/src/ProjectMgmt/Presentation/Resources/templates/project_form.twig
index f9314abe..9fa082fe 100644
--- a/src/ProjectMgmt/Presentation/Resources/templates/project_form.twig
+++ b/src/ProjectMgmt/Presentation/Resources/templates/project_form.twig
@@ -16,6 +16,7 @@
         {% endfor %}
 
         <form method="POST"
+              data-test-id="project-form"
               action="{{ project ? path('project_mgmt.presentation.update', { id: project.id }) : path('project_mgmt.presentation.create') }}"
               class="space-y-6 bg-white dark:bg-dark-800 rounded-lg border border-dark-200 dark:border-dark-700 p-6"
               {{ stimulus_controller('manifest-urls', {
@@ -31,6 +32,7 @@
                 <input type="text"
                        name="name"
                        id="name"
+                       data-test-id="project-form-name"
                        value="{{ project ? project.name : '' }}"
                        required
                        placeholder="{{ 'project.form.placeholder_project_name'|trans }}"
@@ -42,6 +44,7 @@
                 <input type="url"
                        name="git_url"
                        id="git_url"
+                       data-test-id="project-form-git-url"
                        value="{{ project ? project.gitUrl : '' }}"
                        required
                        placeholder="{{ 'project.form.placeholder_project_link'|trans }}"
@@ -57,6 +60,7 @@
                 <input type="password"
                        name="github_token"
                        id="github_token"
+                       data-test-id="project-form-github-token"
                        value="{{ project ? (displayGithubToken is defined ? displayGithubToken : project.githubToken) : '' }}"
                        {{ (keysVisible is not defined or keysVisible) ? 'required' : 'disabled' }}
                        placeholder="{{ (keysVisible is defined and not keysVisible) ? 'project.form.keys_managed_by_deployment'|trans : 'project.form.placeholder_github_token'|trans }}"
@@ -93,6 +97,7 @@
                 <input type="password"
                        name="content_editing_api_key"
                        id="content_editing_api_key"
+                       data-test-id="project-form-content-editing-api-key"
                        value="{{ project ? (displayContentEditingApiKey is defined ? displayContentEditingApiKey : project.contentEditingLlmModelProviderApiKey) : '' }}"
                        {{ (keysVisible is not defined or keysVisible) ? 'required' : 'disabled' }}
                        placeholder="{{ (keysVisible is defined and not keysVisible) ? 'project.form.keys_managed_by_deployment'|trans : 'project.form.placeholder_llm_api_key'|trans }}"
@@ -593,6 +598,7 @@
                     {{ 'common.cancel'|trans }}
                 </a>
                 <button type="submit"
+                        data-test-id="project-form-submit"
                         class="px-4 py-2 rounded-md bg-primary-600 text-white text-sm font-medium hover:bg-primary-700 focus:outline-none focus:ring-2 focus:ring-primary-500 focus:ring-offset-2 dark:focus:ring-offset-dark-900">
                     {{ project ? 'project.form.update_project_button'|trans : 'project.form.add_project_button'|trans }}
                 </button>
diff --git a/src/ProjectMgmt/Presentation/Resources/templates/project_list.twig b/src/ProjectMgmt/Presentation/Resources/templates/project_list.twig
index 32fa0908..c122bf3e 100644
--- a/src/ProjectMgmt/Presentation/Resources/templates/project_list.twig
+++ b/src/ProjectMgmt/Presentation/Resources/templates/project_list.twig
@@ -11,6 +11,7 @@
             </div>
             {% if projectsWithStatus is not empty %}
                 <a href="{{ path('project_mgmt.presentation.new') }}"
+                   data-test-id="project-list-add-project"
                    class="inline-flex justify-center px-4 py-2 rounded-md bg-primary-600 text-white text-sm font-medium hover:bg-primary-700 focus:outline-none focus:ring-2 focus:ring-primary-500 focus:ring-offset-2 dark:focus:ring-offset-dark-900">
                     {{ 'project.list.add_project'|trans }}
                 </a>
@@ -26,6 +27,7 @@
                 <p class="mt-1 text-sm text-dark-500 dark:text-dark-400">{{ 'project.list.no_projects_description'|trans }}</p>
                 <div class="mt-6">
                     <a href="{{ path('project_mgmt.presentation.new') }}"
+                       data-test-id="project-list-add-project"
                        class="inline-flex items-center px-4 py-2 rounded-md bg-primary-600 text-white text-sm font-medium hover:bg-primary-700">
                         {{ 'project.list.add_project'|trans }}
                     </a>
@@ -43,7 +45,7 @@
                         'MERGED': 'bg-indigo-100 text-indigo-800 dark:bg-indigo-900 dark:text-indigo-200',
                         'PROBLEM': 'bg-red-100 text-red-800 dark:bg-red-900 dark:text-red-200'
                     } %}
-                    <div class="rounded-lg border border-dark-200 dark:border-dark-700 bg-white dark:bg-dark-900 p-4">
+                    <div class="rounded-lg border border-dark-200 dark:border-dark-700 bg-white dark:bg-dark-900 p-4" data-test-class="project-list-item">
                         <div class="flex flex-col gap-4 lg:flex-row lg:items-center lg:justify-between">
                             {# Project info section #}
                             <div class="flex-1 min-w-0 space-y-2">
@@ -103,6 +105,7 @@
                                     </a>
                                 {% else %}
                                     <a href="{{ path('chat_based_content_editor.presentation.start', { projectId: item.project.id }) }}"
+                                       data-test-class="project-list-edit-content-link"
                                        class="inline-flex items-center px-3 py-1.5 rounded-md bg-primary-600 text-white text-sm font-medium hover:bg-primary-700 focus:outline-none focus:ring-2 focus:ring-primary-500">
                                         {{ 'project.list.edit_content'|trans }}
                                     </a>
diff --git a/src/RemoteContentAssets/TestHarness/SimulatedRemoteManifestValidator.php b/src/RemoteContentAssets/TestHarness/SimulatedRemoteManifestValidator.php
new file mode 100644
index 00000000..a7f5b647
--- /dev/null
+++ b/src/RemoteContentAssets/TestHarness/SimulatedRemoteManifestValidator.php
@@ -0,0 +1,26 @@
+<?php
+
+declare(strict_types=1);
+
+namespace App\RemoteContentAssets\TestHarness;
+
+use App\RemoteContentAssets\Infrastructure\RemoteManifestValidatorInterface;
+
+/**
+ * E2E/test double: accepts any non-empty http(s) manifest URL without making HTTP requests.
+ */
+final class SimulatedRemoteManifestValidator implements RemoteManifestValidatorInterface
+{
+    public function isValidManifestUrl(string $url): bool
+    {
+        $url = trim($url);
+        if ($url === '') {
+            return false;
+        }
+
+        $parsed = parse_url($url);
+        $scheme = $parsed['scheme'] ?? null;
+
+        return $scheme === 'http' || $scheme === 'https';
+    }
+}
diff --git a/src/RemoteContentAssets/TestHarness/SimulatedS3AssetUploader.php b/src/RemoteContentAssets/TestHarness/SimulatedS3AssetUploader.php
new file mode 100644
index 00000000..60b26a04
--- /dev/null
+++ b/src/RemoteContentAssets/TestHarness/SimulatedS3AssetUploader.php
@@ -0,0 +1,42 @@
+<?php
+
+declare(strict_types=1);
+
+namespace App\RemoteContentAssets\TestHarness;
+
+use App\RemoteContentAssets\Infrastructure\S3AssetUploaderInterface;
+
+/**
+ * E2E/test double: no AWS calls; verifyCredentials returns true, upload returns a deterministic fake URL.
+ */
+final class SimulatedS3AssetUploader implements S3AssetUploaderInterface
+{
+    private const string FAKE_UPLOAD_BASE = 'https://e2e-simulated.s3.example.com';
+
+    public function verifyCredentials(
+        string  $bucketName,
+        string  $region,
+        string  $accessKeyId,
+        string  $secretAccessKey,
+        ?string $iamRoleArn
+    ): bool {
+        return true;
+    }
+
+    public function upload(
+        string  $bucketName,
+        string  $region,
+        string  $accessKeyId,
+        string  $secretAccessKey,
+        ?string $iamRoleArn,
+        ?string $keyPrefix,
+        string  $filename,
+        string  $contents,
+        string  $mimeType
+    ): string {
+        $prefix = $keyPrefix !== null && $keyPrefix !== '' ? trim($keyPrefix, '/') . '/' : '';
+        $key    = $prefix . $filename;
+
+        return self::FAKE_UPLOAD_BASE . '/' . $key;
+    }
+}
diff --git a/src/WorkspaceMgmt/TestHarness/E2eNoOpProjectSetupStepsProvider.php b/src/WorkspaceMgmt/TestHarness/E2eNoOpProjectSetupStepsProvider.php
new file mode 100644
index 00000000..272b6b56
--- /dev/null
+++ b/src/WorkspaceMgmt/TestHarness/E2eNoOpProjectSetupStepsProvider.php
@@ -0,0 +1,32 @@
+<?php
+
+declare(strict_types=1);
+
+namespace App\WorkspaceMgmt\TestHarness;
+
+use App\ProjectMgmt\Facade\Enum\ProjectType;
+use App\WorkspaceMgmt\Infrastructure\SetupSteps\ProjectSetupStepsProviderInterface;
+use App\WorkspaceMgmt\Infrastructure\SetupSteps\SetupStep;
+
+/**
+ * E2E/test: single no-op setup step so workspace setup completes quickly without npm/mise.
+ */
+final class E2eNoOpProjectSetupStepsProvider implements ProjectSetupStepsProviderInterface
+{
+    public function supports(ProjectType $projectType): bool
+    {
+        return match ($projectType) {
+            ProjectType::DEFAULT => true,
+        };
+    }
+
+    /**
+     * @return list<SetupStep>
+     */
+    public function getSetupSteps(): array
+    {
+        return [
+            new SetupStep('E2E no-op', 'true', [], 5),
+        ];
+    }
+}
diff --git a/src/WorkspaceMgmt/TestHarness/SimulatedGitAdapter.php b/src/WorkspaceMgmt/TestHarness/SimulatedGitAdapter.php
new file mode 100644
index 00000000..ecabef35
--- /dev/null
+++ b/src/WorkspaceMgmt/TestHarness/SimulatedGitAdapter.php
@@ -0,0 +1,126 @@
+<?php
+
+declare(strict_types=1);
+
+namespace App\WorkspaceMgmt\TestHarness;
+
+use App\WorkspaceMgmt\Infrastructure\Adapter\GitAdapterInterface;
+use App\WorkspaceMgmt\Infrastructure\Adapter\GitCliAdapter;
+use RecursiveDirectoryIterator;
+use RecursiveIteratorIterator;
+use RuntimeException;
+use SplFileInfo;
+use Symfony\Component\Process\Exception\ProcessFailedException;
+use Symfony\Component\Process\Process;
+
+use function dirname;
+use function is_dir;
+use function mkdir;
+
+use const DIRECTORY_SEPARATOR;
+
+/**
+ * E2E/test double: clone copies a local fixture and runs git init; push is a no-op.
+ * Other git operations delegate to the real CLI adapter.
+ */
+final class SimulatedGitAdapter implements GitAdapterInterface
+{
+    public function __construct(
+        private readonly string        $workspaceFixturePath,
+        private readonly GitCliAdapter $realGitAdapter,
+    ) {
+    }
+
+    public function clone(string $repoUrl, string $targetPath, string $token): void
+    {
+        $this->copyDirectory($this->workspaceFixturePath, $targetPath);
+
+        $this->runProcess(new Process(['git', 'init']), $targetPath, 'Failed to init repo');
+        $this->runProcess(new Process(['git', 'add', '-A']), $targetPath, 'Failed to stage');
+        $this->runProcess(
+            new Process(['git', '-c', 'user.name=E2E', '-c', 'user.email=e2e@example.com', 'commit', '-m', 'Initial']),
+            $targetPath,
+            'Failed to create initial commit'
+        );
+        $this->runProcess(new Process(['git', 'remote', 'add', 'origin', $repoUrl]), $targetPath, 'Failed to add remote');
+    }
+
+    public function checkoutNewBranch(string $workspacePath, string $branchName): void
+    {
+        $this->realGitAdapter->checkoutNewBranch($workspacePath, $branchName);
+    }
+
+    public function hasChanges(string $workspacePath): bool
+    {
+        return $this->realGitAdapter->hasChanges($workspacePath);
+    }
+
+    public function commitAll(
+        string $workspacePath,
+        string $message,
+        string $authorName,
+        string $authorEmail
+    ): void {
+        $this->realGitAdapter->commitAll($workspacePath, $message, $authorName, $authorEmail);
+    }
+
+    public function push(string $workspacePath, string $branchName, string $token): void
+    {
+        // No-op: no real push in e2e simulation
+    }
+
+    public function hasBranchDifferences(string $workspacePath, string $branchName, string $baseBranch = 'main'): bool
+    {
+        return $this->realGitAdapter->hasBranchDifferences($workspacePath, $branchName, $baseBranch);
+    }
+
+    private function copyDirectory(string $source, string $target): void
+    {
+        if (!is_dir($source)) {
+            throw new RuntimeException('Workspace fixture path is not a directory: ' . $source);
+        }
+
+        if (!is_dir($target)) {
+            mkdir($target, 0777, true);
+        }
+
+        $iterator = new RecursiveIteratorIterator(
+            new RecursiveDirectoryIterator($source, RecursiveDirectoryIterator::SKIP_DOTS),
+            RecursiveIteratorIterator::SELF_FIRST
+        );
+
+        foreach ($iterator as $item) {
+            if (!$item instanceof SplFileInfo) {
+                continue;
+            }
+            $subPath = $iterator->getSubPathname();
+            $dest    = $target . DIRECTORY_SEPARATOR . $subPath;
+
+            if ($item->isDir()) {
+                if (!is_dir($dest)) {
+                    mkdir($dest, 0777, true);
+                }
+            } else {
+                $dir = dirname($dest);
+                if (!is_dir($dir)) {
+                    mkdir($dir, 0777, true);
+                }
+                $realPath = $item->getRealPath();
+                if ($realPath !== false) {
+                    copy($realPath, $dest);
+                }
+            }
+        }
+    }
+
+    private function runProcess(Process $process, string $workingDir, string $errorMessage = 'Command failed'): void
+    {
+        $process->setWorkingDirectory($workingDir);
+        $process->setTimeout(60);
+        $process->run();
+
+        if (!$process->isSuccessful()) {
+            throw new ProcessFailedException($process);
+        }
+    }
+}
diff --git a/src/WorkspaceMgmt/TestHarness/SimulatedGitHubAdapter.php b/src/WorkspaceMgmt/TestHarness/SimulatedGitHubAdapter.php
new file mode 100644
index 00000000..4074ff46
--- /dev/null
+++ b/src/WorkspaceMgmt/TestHarness/SimulatedGitHubAdapter.php
@@ -0,0 +1,35 @@
+<?php
+
+declare(strict_types=1);
+
+namespace App\WorkspaceMgmt\TestHarness;
+
+use App\WorkspaceMgmt\Infrastructure\Adapter\GitHubAdapterInterface;
+
+/**
+ * E2E/test double: no GitHub API calls; findPullRequestForBranch returns null, createPullRequest returns a constant URL.
+ */
+final class SimulatedGitHubAdapter implements GitHubAdapterInterface
+{
+    private const string FAKE_PR_URL = 'https://github.com/e2e-simulated/repo/pull/1';
+
+    public function findPullRequestForBranch(
+        string $owner,
+        string $repo,
+        string $branchName,
+        string $token
+    ): ?string {
+        return null;
+    }
+
+    public function createPullRequest(
+        string $owner,
+        string $repo,
+        string $branchName,
+        string $title,
+        string $body,
+        string $token
+    ): string {
+        return self::FAKE_PR_URL;
+    }
+}
diff --git a/tests/End2End/tests/full-flow.spec.ts b/tests/End2End/tests/full-flow.spec.ts
new file mode 100644
index 00000000..3a2e3ca2
--- /dev/null
+++ b/tests/End2End/tests/full-flow.spec.ts
@@ -0,0 +1,69 @@
+import { test, expect } from "@playwright/test";
+
+/**
+ * Full e2e flow with all external dependencies simulated (APP_ENV=test):
+ * sign in → create project → open editor → send message.
+ * Uses test doubles for: LLM, manifest URL, S3, git clone (fixture), GitHub PR.
+ */
+test.describe("full flow (sign in, create project, use editor)", () => {
+    test("user can sign in, create a project, open editor, and send a message", async ({ page }) => {
+        test.setTimeout(180000);
+
+        // 1. Sign in
+        await page.goto("/en/account/sign-in");
+        await page.getByTestId("sign-in-email").fill("e2e@example.com");
+        await page.getByTestId("sign-in-password").fill("e2e-secret");
+        await page.getByTestId("sign-in-submit").click();
+
+        await expect(page).toHaveURL(/\/en\/projects/);
+        await expect(page.getByTestId("project-list-page")).toBeVisible();
+
+        // 2. Go to add project (one link when empty, one in header when not)
+        await page.getByTestId("project-list-add-project").first().click();
+        await expect(page.getByTestId("project-form")).toBeVisible();
+
+        // 3. Fill project form (simulated validators accept these in test env)
+        await page.getByTestId("project-form-name").fill("E2E Test Project");
+        await page.getByTestId("project-form-git-url").fill("https://github.com/e2e/fake-repo.git");
+        await page.getByTestId("project-form-github-token").fill("fake-token");
+        await page.getByTestId("project-form-content-editing-api-key").fill("fake-llm-key");
+        // Blur so optional client-side LLM key verification can complete (simulated in test env)
+        await page.getByTestId("project-form-name").click();
+        await page.waitForTimeout(500);
+
+        await page.getByTestId("project-form-submit").click();
+        await page.waitForURL(/\/en\/projects\/?$/, { timeout: 20000 });
+
+        // 4. Project list after create (may have prefab projects + our new one)
+        await expect(page.getByTestId("project-list-page")).toBeVisible({
+            timeout: 10000,
+        });
+        await expect(page.getByTestId("project-list-heading")).toBeVisible();
+        expect(await page.locator('[data-test-class="project-list-item"]').count()).toBeGreaterThanOrEqual(1);
+
+        // 5. Open editor (Edit content) for the first project
+        await page.locator('[data-test-class="project-list-edit-content-link"]').first().click();
+
+        // 6. Either workspace setup page (then redirect) or editor directly
+        const setupPage = page.getByTestId("workspace-setup-page");
+        const editorPage = page.getByTestId("editor-page");
+
+        if (await setupPage.isVisible()) {
+            await expect(editorPage).toBeVisible({ timeout: 120000 });
+        } else {
+            await expect(editorPage).toBeVisible({ timeout: 10000 });
+        }
+
+        // 7. Send a message in the editor
+        await expect(page.getByTestId("editor-message-input")).toBeVisible();
+        await page.getByTestId("editor-message-input").fill("Add a title to the page");
+        await page.getByTestId("editor-send-button").click();
+
+        // 8. Wait for simulated LLM to respond (stream completes)
+        await expect(page.getByTestId("editor-page")).toBeVisible({
+            timeout: 30000,
+        });
+        await page.waitForTimeout(3000);
+        await expect(page.getByTestId("editor-message-input")).toBeVisible();
+    });
+});
diff --git a/tests/fixtures/e2e-workspace-template/index.html b/tests/fixtures/e2e-workspace-template/index.html
new file mode 100644
index 00000000..cb986fce
--- /dev/null
+++ b/tests/fixtures/e2e-workspace-template/index.html
@@ -0,0 +1,10 @@
+<!DOCTYPE html>
+<html lang="en">
+<head>
+  <meta charset="UTF-8">
+  <title>E2E Fixture</title>
+</head>
+<body>
+  <h1>E2E Fixture Page</h1>
+</body>
+</html>
diff --git a/tests/fixtures/e2e-workspace-template/package.json b/tests/fixtures/e2e-workspace-template/package.json
new file mode 100644
index 00000000..ae6dad61
--- /dev/null
+++ b/tests/fixtures/e2e-workspace-template/package.json
@@ -0,0 +1,8 @@
+{
+    "name": "e2e-fixture",
+    "version": "1.0.0",
+    "private": true,
+    "scripts": {
+        "build": "echo ok"
+    }
+}

From 5b171de320960db1ce6d133bfa6b52218656fb22 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Manuel=20Kie=C3=9Fling?= <manuel@kiessling.net>
Date: Mon, 16 Feb 2026 21:54:05 +0100
Subject: [PATCH 2/9] e2e: clarify no prefabs in repo (prefabs.yaml missing,
 PrefabLoader returns [])

---
 tests/End2End/tests/full-flow.spec.ts | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/tests/End2End/tests/full-flow.spec.ts b/tests/End2End/tests/full-flow.spec.ts
index 3a2e3ca2..3f1ca207 100644
--- a/tests/End2End/tests/full-flow.spec.ts
+++ b/tests/End2End/tests/full-flow.spec.ts
@@ -34,14 +34,14 @@ test.describe("full flow (sign in, create project, use editor)", () => {
         await page.getByTestId("project-form-submit").click();
         await page.waitForURL(/\/en\/projects\/?$/, { timeout: 20000 });
 
-        // 4. Project list after create (may have prefab projects + our new one)
+        // 4. Project list after create: exactly one project (no prefabs.yaml in repo)
         await expect(page.getByTestId("project-list-page")).toBeVisible({
             timeout: 10000,
         });
         await expect(page.getByTestId("project-list-heading")).toBeVisible();
-        expect(await page.locator('[data-test-class="project-list-item"]').count()).toBeGreaterThanOrEqual(1);
+        await expect(page.locator('[data-test-class="project-list-item"]')).toHaveCount(1);
 
-        // 5. Open editor (Edit content) for the first project
+        // 5. Open editor (Edit content) for our project
         await page.locator('[data-test-class="project-list-edit-content-link"]').first().click();
 
         // 6. Either workspace setup page (then redirect) or editor directly

From d8e91eb59b787188d8b19b36e99e91334473e4bd Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Manuel=20Kie=C3=9Fling?= <manuel@kiessling.net>
Date: Mon, 16 Feb 2026 21:58:31 +0100
Subject: [PATCH 3/9] fix: add default S3AssetUploaderInterface binding for
 prod/CI (when@test overrides in test)

---
 config/services.yaml | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/config/services.yaml b/config/services.yaml
index 7bbc374c..956828f4 100644
--- a/config/services.yaml
+++ b/config/services.yaml
@@ -109,6 +109,9 @@ services:
     App\RemoteContentAssets\Infrastructure\RemoteManifestFetcherInterface:
         class: App\RemoteContentAssets\Infrastructure\RemoteManifestFetcher
 
+    App\RemoteContentAssets\Infrastructure\S3AssetUploaderInterface:
+        class: App\RemoteContentAssets\Infrastructure\S3AssetUploader
+
     # Remote content assets facade - metadata for remote images (dimensions, size, mime type),
     # manifest validation and fetching
     App\RemoteContentAssets\Facade\RemoteContentAssetsFacadeInterface:
@@ -117,6 +120,7 @@ services:
             - "@App\\RemoteContentAssets\\Infrastructure\\RemoteImageInfoFetcherInterface"
             - "@App\\RemoteContentAssets\\Infrastructure\\RemoteManifestValidatorInterface"
             - "@App\\RemoteContentAssets\\Infrastructure\\RemoteManifestFetcherInterface"
+            - "@App\\RemoteContentAssets\\Infrastructure\\S3AssetUploaderInterface"
 
     # Sitebuilder-specific workspace tooling facade
     App\WorkspaceTooling\Facade\WorkspaceToolingFacadeInterface:

From 91831c46597b0282fef3075cfbddc6cea3408fa3 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Manuel=20Kie=C3=9Fling?= <manuel@kiessling.net>
Date: Mon, 16 Feb 2026 22:08:43 +0100
Subject: [PATCH 4/9] e2e fix

---
 tests/End2End/tests/full-flow.spec.ts | 22 ++++++++++------------
 1 file changed, 10 insertions(+), 12 deletions(-)

diff --git a/tests/End2End/tests/full-flow.spec.ts b/tests/End2End/tests/full-flow.spec.ts
index 3f1ca207..d20c8c1f 100644
--- a/tests/End2End/tests/full-flow.spec.ts
+++ b/tests/End2End/tests/full-flow.spec.ts
@@ -34,25 +34,23 @@ test.describe("full flow (sign in, create project, use editor)", () => {
         await page.getByTestId("project-form-submit").click();
         await page.waitForURL(/\/en\/projects\/?$/, { timeout: 20000 });
 
-        // 4. Project list after create: exactly one project (no prefabs.yaml in repo)
+        // 4. Project list after create: at least one project (no prefabs in repo; retries may leave N from prior runs)
         await expect(page.getByTestId("project-list-page")).toBeVisible({
             timeout: 10000,
         });
         await expect(page.getByTestId("project-list-heading")).toBeVisible();
-        await expect(page.locator('[data-test-class="project-list-item"]')).toHaveCount(1);
+        await expect(page.locator('[data-test-class="project-list-item"]').first()).toBeVisible();
 
-        // 5. Open editor (Edit content) for our project
-        await page.locator('[data-test-class="project-list-edit-content-link"]').first().click();
+        // 5. Open editor (Edit content) for the project we created (by name; robust to retries)
+        await page
+            .locator('[data-test-class="project-list-item"]')
+            .filter({ hasText: "E2E Test Project" })
+            .locator('[data-test-class="project-list-edit-content-link"]')
+            .click();
 
-        // 6. Either workspace setup page (then redirect) or editor directly
-        const setupPage = page.getByTestId("workspace-setup-page");
+        // 6. Wait for editor (setup may show first then redirect; CI can be slow)
         const editorPage = page.getByTestId("editor-page");
-
-        if (await setupPage.isVisible()) {
-            await expect(editorPage).toBeVisible({ timeout: 120000 });
-        } else {
-            await expect(editorPage).toBeVisible({ timeout: 10000 });
-        }
+        await expect(editorPage).toBeVisible({ timeout: 120000 });
 
         // 7. Send a message in the editor
         await expect(page.getByTestId("editor-message-input")).toBeVisible();

From 34198146e5640116d50a87e3fb2d114855b0bb11 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Manuel=20Kie=C3=9Fling?= <manuel@kiessling.net>
Date: Tue, 17 Feb 2026 11:40:16 +0100
Subject: [PATCH 5/9] e2e: start messenger in CI and fix retry flakiness (click
 last project, 120s editor wait)

---
 .github/workflows/ci.yml              | 2 +-
 tests/End2End/tests/full-flow.spec.ts | 3 ++-
 2 files changed, 3 insertions(+), 2 deletions(-)

diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
index 18560f03..f388f177 100644
--- a/.github/workflows/ci.yml
+++ b/.github/workflows/ci.yml
@@ -210,7 +210,7 @@ jobs:
             - name: Pull app image
               run: docker pull ${{ env.APP_IMAGE }}
             - name: Start E2E stack
-              run: docker compose ${{ env.COMPOSE_FILES }} up -d app nginx mariadb
+              run: docker compose ${{ env.COMPOSE_FILES }} up -d app messenger nginx mariadb
             - name: Wait for MariaDB
               run: |
                   for i in $(seq 1 30); do
diff --git a/tests/End2End/tests/full-flow.spec.ts b/tests/End2End/tests/full-flow.spec.ts
index d20c8c1f..07978706 100644
--- a/tests/End2End/tests/full-flow.spec.ts
+++ b/tests/End2End/tests/full-flow.spec.ts
@@ -41,10 +41,11 @@ test.describe("full flow (sign in, create project, use editor)", () => {
         await expect(page.getByTestId("project-list-heading")).toBeVisible();
         await expect(page.locator('[data-test-class="project-list-item"]').first()).toBeVisible();
 
-        // 5. Open editor (Edit content) for the project we created (by name; robust to retries)
+        // 5. Open editor (Edit content) for the project we just created (last row with this name; retries add more)
         await page
             .locator('[data-test-class="project-list-item"]')
             .filter({ hasText: "E2E Test Project" })
+            .last()
             .locator('[data-test-class="project-list-edit-content-link"]')
             .click();
 

From b50290ab470b54edadf9c6464e048df57ba56779 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Manuel=20Kie=C3=9Fling?= <manuel@kiessling.net>
Date: Tue, 17 Feb 2026 12:08:59 +0100
Subject: [PATCH 6/9] ci: add dependency caches (Composer, npm, Playwright) to
 speed up all jobs

---
 .github/workflows/ci.yml | 72 ++++++++++++++++++++++++++++++++++++++++
 1 file changed, 72 insertions(+)

diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
index f388f177..0f6c455c 100644
--- a/.github/workflows/ci.yml
+++ b/.github/workflows/ci.yml
@@ -48,6 +48,18 @@ jobs:
             HOST_PROJECT_PATH: ${{ github.workspace }}
         steps:
             - uses: actions/checkout@v4
+            - name: Restore Composer cache
+              uses: actions/cache@v4
+              with:
+                path: vendor
+                key: composer-${{ hashFiles('composer.lock') }}
+                restore-keys: composer-
+            - name: Restore npm cache
+              uses: actions/cache@v4
+              with:
+                path: node_modules
+                key: npm-${{ hashFiles('package-lock.json') }}
+                restore-keys: npm-
             - name: Login to GitHub Container Registry
               uses: docker/login-action@v3
               with:
@@ -78,6 +90,18 @@ jobs:
             HOST_PROJECT_PATH: ${{ github.workspace }}
         steps:
             - uses: actions/checkout@v4
+            - name: Restore Composer cache
+              uses: actions/cache@v4
+              with:
+                path: vendor
+                key: composer-${{ hashFiles('composer.lock') }}
+                restore-keys: composer-
+            - name: Restore npm cache
+              uses: actions/cache@v4
+              with:
+                path: node_modules
+                key: npm-${{ hashFiles('package-lock.json') }}
+                restore-keys: npm-
             - name: Login to GitHub Container Registry
               uses: docker/login-action@v3
               with:
@@ -105,6 +129,18 @@ jobs:
             HOST_PROJECT_PATH: ${{ github.workspace }}
         steps:
             - uses: actions/checkout@v4
+            - name: Restore Composer cache
+              uses: actions/cache@v4
+              with:
+                path: vendor
+                key: composer-${{ hashFiles('composer.lock') }}
+                restore-keys: composer-
+            - name: Restore npm cache
+              uses: actions/cache@v4
+              with:
+                path: node_modules
+                key: npm-${{ hashFiles('package-lock.json') }}
+                restore-keys: npm-
             - name: Login to GitHub Container Registry
               uses: docker/login-action@v3
               with:
@@ -162,6 +198,18 @@ jobs:
             HOST_PROJECT_PATH: ${{ github.workspace }}
         steps:
             - uses: actions/checkout@v4
+            - name: Restore Composer cache
+              uses: actions/cache@v4
+              with:
+                path: vendor
+                key: composer-${{ hashFiles('composer.lock') }}
+                restore-keys: composer-
+            - name: Restore npm cache
+              uses: actions/cache@v4
+              with:
+                path: node_modules
+                key: npm-${{ hashFiles('package-lock.json') }}
+                restore-keys: npm-
             - name: Login to GitHub Container Registry
               uses: docker/login-action@v3
               with:
@@ -201,6 +249,18 @@ jobs:
             COMPOSE_FILES: -f docker-compose.yml -f docker-compose.ci.yml -f docker-compose.e2e.yml
         steps:
             - uses: actions/checkout@v4
+            - name: Restore Composer cache
+              uses: actions/cache@v4
+              with:
+                path: vendor
+                key: composer-${{ hashFiles('composer.lock') }}
+                restore-keys: composer-
+            - name: Restore npm cache
+              uses: actions/cache@v4
+              with:
+                path: node_modules
+                key: npm-${{ hashFiles('package-lock.json') }}
+                restore-keys: npm-
             - name: Login to GitHub Container Registry
               uses: docker/login-action@v3
               with:
@@ -244,6 +304,18 @@ jobs:
                   docker compose ${{ env.COMPOSE_FILES }} exec -T app php bin/console doctrine:database:create --env=test
                   docker compose ${{ env.COMPOSE_FILES }} exec -T app php bin/console doctrine:migrations:migrate --no-interaction --env=test
                   docker compose ${{ env.COMPOSE_FILES }} exec -T app php bin/console app:e2e:create-user e2e@example.com --password=e2e-secret
+            - name: Restore Playwright deps cache
+              uses: actions/cache@v4
+              with:
+                path: tests/End2End/node_modules
+                key: playwright-deps-${{ hashFiles('tests/End2End/package-lock.json') }}
+                restore-keys: playwright-deps-
+            - name: Restore Playwright browsers cache
+              uses: actions/cache@v4
+              with:
+                path: /home/runner/.cache/ms-playwright
+                key: playwright-browsers-${{ hashFiles('tests/End2End/package-lock.json') }}
+                restore-keys: playwright-browsers-
             - name: Install Playwright
               run: |
                   cd tests/End2End

From bf6867db9d2b476dff7136b07ce53cfcd2420e91 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Manuel=20Kie=C3=9Fling?= <manuel@kiessling.net>
Date: Tue, 17 Feb 2026 12:38:56 +0100
Subject: [PATCH 7/9] Code style fix

---
 .github/workflows/ci.yml | 72 ++++++++++++++++++++--------------------
 1 file changed, 36 insertions(+), 36 deletions(-)

diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
index 0f6c455c..9cd66471 100644
--- a/.github/workflows/ci.yml
+++ b/.github/workflows/ci.yml
@@ -51,15 +51,15 @@ jobs:
             - name: Restore Composer cache
               uses: actions/cache@v4
               with:
-                path: vendor
-                key: composer-${{ hashFiles('composer.lock') }}
-                restore-keys: composer-
+                  path: vendor
+                  key: composer-${{ hashFiles('composer.lock') }}
+                  restore-keys: composer-
             - name: Restore npm cache
               uses: actions/cache@v4
               with:
-                path: node_modules
-                key: npm-${{ hashFiles('package-lock.json') }}
-                restore-keys: npm-
+                  path: node_modules
+                  key: npm-${{ hashFiles('package-lock.json') }}
+                  restore-keys: npm-
             - name: Login to GitHub Container Registry
               uses: docker/login-action@v3
               with:
@@ -93,15 +93,15 @@ jobs:
             - name: Restore Composer cache
               uses: actions/cache@v4
               with:
-                path: vendor
-                key: composer-${{ hashFiles('composer.lock') }}
-                restore-keys: composer-
+                  path: vendor
+                  key: composer-${{ hashFiles('composer.lock') }}
+                  restore-keys: composer-
             - name: Restore npm cache
               uses: actions/cache@v4
               with:
-                path: node_modules
-                key: npm-${{ hashFiles('package-lock.json') }}
-                restore-keys: npm-
+                  path: node_modules
+                  key: npm-${{ hashFiles('package-lock.json') }}
+                  restore-keys: npm-
             - name: Login to GitHub Container Registry
               uses: docker/login-action@v3
               with:
@@ -132,15 +132,15 @@ jobs:
             - name: Restore Composer cache
               uses: actions/cache@v4
               with:
-                path: vendor
-                key: composer-${{ hashFiles('composer.lock') }}
-                restore-keys: composer-
+                  path: vendor
+                  key: composer-${{ hashFiles('composer.lock') }}
+                  restore-keys: composer-
             - name: Restore npm cache
               uses: actions/cache@v4
               with:
-                path: node_modules
-                key: npm-${{ hashFiles('package-lock.json') }}
-                restore-keys: npm-
+                  path: node_modules
+                  key: npm-${{ hashFiles('package-lock.json') }}
+                  restore-keys: npm-
             - name: Login to GitHub Container Registry
               uses: docker/login-action@v3
               with:
@@ -201,15 +201,15 @@ jobs:
             - name: Restore Composer cache
               uses: actions/cache@v4
               with:
-                path: vendor
-                key: composer-${{ hashFiles('composer.lock') }}
-                restore-keys: composer-
+                  path: vendor
+                  key: composer-${{ hashFiles('composer.lock') }}
+                  restore-keys: composer-
             - name: Restore npm cache
               uses: actions/cache@v4
               with:
-                path: node_modules
-                key: npm-${{ hashFiles('package-lock.json') }}
-                restore-keys: npm-
+                  path: node_modules
+                  key: npm-${{ hashFiles('package-lock.json') }}
+                  restore-keys: npm-
             - name: Login to GitHub Container Registry
               uses: docker/login-action@v3
               with:
@@ -252,15 +252,15 @@ jobs:
             - name: Restore Composer cache
               uses: actions/cache@v4
               with:
-                path: vendor
-                key: composer-${{ hashFiles('composer.lock') }}
-                restore-keys: composer-
+                  path: vendor
+                  key: composer-${{ hashFiles('composer.lock') }}
+                  restore-keys: composer-
             - name: Restore npm cache
               uses: actions/cache@v4
               with:
-                path: node_modules
-                key: npm-${{ hashFiles('package-lock.json') }}
-                restore-keys: npm-
+                  path: node_modules
+                  key: npm-${{ hashFiles('package-lock.json') }}
+                  restore-keys: npm-
             - name: Login to GitHub Container Registry
               uses: docker/login-action@v3
               with:
@@ -307,15 +307,15 @@ jobs:
             - name: Restore Playwright deps cache
               uses: actions/cache@v4
               with:
-                path: tests/End2End/node_modules
-                key: playwright-deps-${{ hashFiles('tests/End2End/package-lock.json') }}
-                restore-keys: playwright-deps-
+                  path: tests/End2End/node_modules
+                  key: playwright-deps-${{ hashFiles('tests/End2End/package-lock.json') }}
+                  restore-keys: playwright-deps-
             - name: Restore Playwright browsers cache
               uses: actions/cache@v4
               with:
-                path: /home/runner/.cache/ms-playwright
-                key: playwright-browsers-${{ hashFiles('tests/End2End/package-lock.json') }}
-                restore-keys: playwright-browsers-
+                  path: /home/runner/.cache/ms-playwright
+                  key: playwright-browsers-${{ hashFiles('tests/End2End/package-lock.json') }}
+                  restore-keys: playwright-browsers-
             - name: Install Playwright
               run: |
                   cd tests/End2End

From e19353e3e7bce7b6150f5f85a0780140d685ce62 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Manuel=20Kie=C3=9Fling?= <manuel@kiessling.net>
Date: Tue, 17 Feb 2026 12:51:53 +0100
Subject: [PATCH 8/9] fix: restore dev stack after e2e tests to prevent env
 contamination

The e2e script mutates running containers in-place with test env config
but never restores them. This leaves nginx on port 8080 with APP_ENV=test
after tests complete. Add an EXIT trap that runs `docker compose up -d`
(without the e2e override) to recreate containers with dev defaults.
---
 .mise/tasks/tests/e2e.sh | 18 +++++++++++++++++-
 1 file changed, 17 insertions(+), 1 deletion(-)

diff --git a/.mise/tasks/tests/e2e.sh b/.mise/tasks/tests/e2e.sh
index 61d99dd7..22cca48f 100755
--- a/.mise/tasks/tests/e2e.sh
+++ b/.mise/tasks/tests/e2e.sh
@@ -13,7 +13,14 @@ echo "End-to-end tests (Playwright)"
 echo "BASE_URL=${BASE_URL}"
 echo
 
+restore_dev_stack() {
+    echo ""
+    echo "Restoring dev stack (docker compose up -d)..."
+    docker compose up -d
+}
+
 if [ "${NO_START}" != "true" ]; then
+    trap restore_dev_stack EXIT
     echo "Starting stack with e2e override..."
     docker compose $COMPOSE_FILES up -d
     echo "Waiting for MariaDB..."
@@ -48,6 +55,15 @@ echo "Installing Playwright dependencies (if needed)..."
 (cd tests/End2End && npx playwright install chromium 2>/dev/null || true)
 
 echo "Running Playwright tests..."
+set +e
 (cd tests/End2End && BASE_URL="$BASE_URL" npx playwright test)
+PLAYWRIGHT_EXIT=$?
+set -e
+
+if [ $PLAYWRIGHT_EXIT -eq 0 ]; then
+    echo "E2E tests completed!"
+else
+    echo "E2E tests failed (exit code $PLAYWRIGHT_EXIT)."
+fi
 
-echo "E2E tests completed!"
+exit $PLAYWRIGHT_EXIT

From 691f4679449d6263aeefdb85a6406aa0ce8f8f86 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Manuel=20Kie=C3=9Fling?= <manuel@kiessling.net>
Date: Tue, 17 Feb 2026 12:54:04 +0100
Subject: [PATCH 9/9] ci: wait for MariaDB in quality job before doctrine
 commands

---
 .github/workflows/ci.yml | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
index 9cd66471..f180f51d 100644
--- a/.github/workflows/ci.yml
+++ b/.github/workflows/ci.yml
@@ -220,6 +220,13 @@ jobs:
               run: docker pull ${{ env.APP_IMAGE }}
             - name: Start Docker services
               run: docker compose -f docker-compose.yml -f docker-compose.ci.yml up -d app mariadb
+            - name: Wait for MariaDB
+              run: |
+                  for i in {1..30}; do
+                    docker compose exec -T mariadb mariadb -uroot -psecret -e "SELECT 1" && break
+                    echo "Waiting for MariaDB... ($i/30)"
+                    sleep 2
+                  done
             - name: Install dependencies
               run: |
                   docker compose exec -T app mise trust