ci: e2e job overlaying fresh server build on nightly image

[juliusknorr]

Adds an e2e workflow that builds the server binaries, overlays them onto ghcr.io/euro-office/documentserver:nightly, and runs the DocumentServer Playwright e2e suite against it. Full-stack feedback on a server change without rebuilding core/sdkjs/web-apps.

How it works

• Pulls the nightly image and reads the PRODUCT_VERSION it was built with.
• Stamps that version into commondefines.js (mirroring the bake), then pkg-builds the service binaries (docservice, fileconverter, metrics).
• Replaces the matching binaries in the image (server/DocService/docservice, server/FileConverter/converter, server/Metrics/metrics); install path discovered via the sdkjs dir. Runs the e2e suite from Euro-Office/DocumentServer (matching branch, fallback main).

Notes

• docservice rejects an editor whose buildVersion differs from its own, so the fresh docservice must carry the nightly image version — hence the commondefines.js stamp.
• The AdminPanel webpack client is intentionally not overlaid; the e2e suite never exercises the admin UI.
• All involved repos and the documentserver package are public (no PAT).

Verified green against the current nightly.

[chrip]

The e2e workflow is well designed and addresses a real need (server-only CI feedback). But the PR contains a lot of redundant changes that are already on main — the branch was created before those were merged. Please rebase on main to drop them, leaving only e2e.yml.

Redundant changes (already on main):

• eslint.config.js: react/react-hooks config is already present
• .prettierignore: AiIntegration entries are already there
• package.json: eslint-plugin-react and eslint-plugin-react-hooks already in devDeps
• npm-shrinkwrap.json: auto-generated changes from above
• Color changes (#0082C9 → #0082c9): already normalized on main
• webpack.config.js: formatting changes already on main
• All the .module.scss files: color changes already merged

e2e workflow — verified correct:

• sed replacement of buildVersion in commondefines.js matches current format (const buildVersion = '4.1.2';) ✓
• buildVersion stamp is necessary: docservice rejects editors with differing versions (DocsCoServer.js:2776) ✓
• pkg builds for DocService, FileConverter, Metrics ✓
• Overlay Dockerfile correctly discovers install path via sdkjs dir ✓
• Nightly version extraction from api/documents/api.js ✓
• E2E checkout fallback logic (branch → main) ✓
• AdminPanel webpack client intentionally not overlaid ✓

Suggestion: Consider npm ci instead of npm install in the build step for reproducible builds.

[chrip]

The e2e workflow is well designed and addresses a real need (server-only CI feedback). But the PR contains a lot of redundant changes that are already on main — the branch was created before those were merged. Please rebase on main to drop them, leaving only e2e.yml.

Redundant changes (already on main):

• eslint.config.js: react/react-hooks config is already present
• .prettierignore: AiIntegration entries are already there
• package.json: eslint-plugin-react and eslint-plugin-react-hooks already in devDeps
• npm-shrinkwrap.json: auto-generated changes from above
• Color changes (#0082C9 → #0082c9): already normalized on main
• webpack.config.js: formatting changes already on main
• All the .module.scss files: color changes already merged

e2e workflow — verified correct:

• sed replacement of buildVersion in commondefines.js matches current format (const buildVersion = '4.1.2';) ✓
• buildVersion stamp is necessary: docservice rejects editors with differing versions (DocsCoServer.js:2776) ✓
• pkg builds for DocService, FileConverter, Metrics ✓
• Overlay Dockerfile correctly discovers install path via sdkjs dir ✓
• Nightly version extraction from api/documents/api.js ✓
• E2E checkout fallback logic (branch → main) ✓
• AdminPanel webpack client intentionally not overlaid ✓

Suggestion: Consider npm ci instead of npm install in the build step for reproducible builds.

[juliusknorr]

That is a lot of text for just requiring a rebase 😁

Did that, just relevant pieces left as it was based on #28