feat: add environment deployment websocket updates by mertilginoglu · Pull Request #1043 · ls1intum/Helios

mertilginoglu · 2026-05-14T22:56:37Z

Motivation

Environment deployment status and timer updates should refresh seamlessly without relying on workflow-run-detail websocket subscriptions. The previous websocket branch was reverted, and this PR restores only the environment deployment websocket path while leaving workflow run details on the regular query flow.

Description

Add repository-scoped environment deployment websocket infrastructure with token/repository handshake validation.
Add a client environment deployment websocket service that invalidates environment queries and disables polling while connected.
Publish environment invalidations after deployment, lock, scheduler, workflow run, and workflow job timing changes so deployment timers can transition from pre-deployment to deployment in the environment view.
Keep workflow-run detail websocket functionality out of this PR.

Testing Instructions

Prerequisites:

GitHub account with access to a Helios repository.
A repository with at least one configured environment and deployment workflow.

Flow:

Log in to Helios and open the environment page for the repository.
Trigger a deployment from Helios.
Keep the environment page open while the deployment workflow progresses.
Verify that the deployment status and timer update without manually refreshing the page.
Verify that the timer changes from pre-deployment to deployment when the configured deploy job starts.
Open a workflow run details page and verify it still loads run and job data without websocket-specific behavior.

Checklist

General

PR description explains the purpose and changes. (f.e. following the 'Conventional Commits')
Changes have been tested locally.

Server

Code is performant and follows best practices

Client

I translated all newly inserted strings into English and German.

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> Co-authored-by: meryemefe <meryemefe.cs@gmail.com>

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

…fication, and helios-status-spring-starter (#1021) chore(deps): upgrade to Spring Boot 4.0.6 in application-server, notification, and helios-status-spring-starter fix(aop): update pointcut expression for SessionBuilder in RepositoryFilterAspect chore(deps): update helios-status-spring-starter to version 1.1.1

… request table component (#982)

…eployment blocking logic during environment unlock (#1028)

…ment configuration

…rite/lock actions after sign-out (#1027)

- add filter bar support for author, assignee, label, review state, and requested reviewer, sort - extend backend pull request query filtering and filter-options endpoint support

… limits, and confirmation UX (#977) - implement AI-based test failure analysis flow - persist AI test failure analyses and add scheduled cleanup - enforce per-user rate limits for analysis requests - show AI usage limits and confirmation modal before running analysis

…n deployment configuration

…ation

…ith JWT authentication (#1012)

#1036)

…iosDeploymentWorkflowJobTimingMeta projection and improved caching logic (#1038)

codacy-production · 2026-05-14T22:57:32Z

Not up to standards ⛔

🟢 Issues 0 issues

Results:
0 new issues

View in Codacy

🔴 Metrics 104 complexity

Metric Results

Complexity ⚠️ 104 (≤ 20 complexity)

View in Codacy

🟢 Coverage 9.22% diff coverage · -0.38% coverage variation

Metric Results

Coverage variation ✅ -0.38% coverage variation (-1.00%)

Diff coverage ✅ 9.22% diff coverage

View coverage diff in Codacy

Coverage variation details

Coverable lines Covered lines Coverage

Common ancestor commit (aef5d83) 8429 3961 46.99%

Head commit (a916dc5) 8635 (+206) 4025 (+64) 46.61% (-0.38%)

Coverage variation is the difference between the coverage for the head and common ancestor commits of the pull request branch: <coverage of head commit> - <coverage of common ancestor commit>

Diff coverage details

Coverable lines Covered lines Diff coverage

Pull request (#1043) 206 19 9.22%

Diff coverage is the percentage of lines that are covered by tests out of the coverable lines that the pull request added or modified: <covered lines added or modified>/<coverable lines added or modified> * 100%

_{NEW Get contextual insights on your PRs based on Codacy's metrics, along with PR and Jira context, without leaving GitHub. Enable AI reviewer}
_{TIP This summary will be updated as you push new changes.}

github-actions · 2026-05-25T13:16:45Z

There hasn't been any activity on this pull request recently. Therefore, this pull request has been automatically marked as stale and will be closed if no further activity occurs within seven days. Thank you for your contributions.

…websocket-only # Conflicts: # .github/workflows/pullrequest_linting.yml # .github/workflows/test.yml # client/.nvmrc # client/Dockerfile # client/Dockerfile.dev # client/package.json # client/src/app/components/pipeline/test-results/pipeline-test-results.component.spec.ts # client/src/app/components/pipeline/test-results/pipeline-test-results.component.ts # client/src/app/core/modules/openapi/client/types.gen.ts # client/yarn.lock # server/application-server/build.gradle # server/application-server/src/main/java/de/tum/cit/aet/helios/workflow/github/GitHubWorkflowJobTimingService.java # server/application-server/src/test/java/de/tum/cit/aet/helios/workflow/github/GitHubWorkflowJobTimingServiceTest.java # server/helios-status-spring-starter/pom.xml # server/notification/build.gradle

krusche · 2026-05-28T07:24:42Z

Rebased on current staging. 15 conflicts: mostly the same mechanical churn (pnpm migration, CI workflow updates, Spring Boot 4 + spring-ai 2.0.0-M8 + dep bumps) plus two genuine logic conflicts that needed actual merging.

Mechanical (take staging): yarn.lock dropped; client/package.json, Dockerfiles, .nvmrc, both workflow YAMLs, pom.xml, notification/build.gradle, client/types.gen.ts. Regenerated openapi.yaml from the merged server and ran pnpm generate:openapi.

build.gradle — manual union: kept this PR's spring-boot-starter-websocket (the only dep staging doesn't already have for the websocket feature); took staging's values for everything else — spring-ai-bom 2.0.0-M8 (was M6), jnats 2.25.3, sentry 8.43, +spring-boot-starter-data-jpa-test.

GitHubWorkflowJobTimingService + its test — kept this PR's side: all three hunks were one-sided additive (this PR added the EnvironmentDeploymentWebSocketPublisher field/import and the publishAfterCommit(heliosDeployment) call; staging side empty). Preserved by union, not overwriting any staging change.

pipeline-test-results.component.ts + spec — kept staging's side: one-sided additive in the other direction (staging's test-failure-analysis PR added the aiAnalysisEnabled environment gate; this PR's side empty for those hunks). Both features now coexist.

Silent auto-merge bug caught + fixed in EnvironmentServiceTest.java: the file auto-merged "cleanly" but both sides had added the same 5-method block (testLockOwnerCanNotUnlockWithActiveDeployment and four neighbours) — git concatenated them, producing 5 duplicate-method definitions that broke compileTestJava. Deleted the second identical copy.

Local verification: server compiles + test compiles, EnvironmentServiceTest + GitHubWorkflowJobTimingServiceTest pass, client builds + lints clean. Mergeable; left as draft because this is a non-trivial websocket+token-handshake change that needs human review.

github-actions · 2026-05-28T11:03:31Z

🚨 Client Code Validation Failed 🚨

The client code in /client/src/app/core/modules/openapi is not up-to-date.
Please regenerate the client code by running:

cd ./client
pnpm generate:openapi

Commit and push the updated files.

github-actions · 2026-06-05T13:09:35Z

There hasn't been any activity on this pull request recently. Therefore, this pull request has been automatically marked as stale and will be closed if no further activity occurs within seven days. Thank you for your contributions.

renovate Bot and others added 19 commits May 5, 2026 12:30

fix(deps): update all non-major client dependencies (#851)

414ea0d

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> Co-authored-by: meryemefe <meryemefe.cs@gmail.com>

chore(deps): update actions/upload-artifact action to v7 (#1023)

40ead93

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

chore(deps): update actions/stale action to v10 (#1022)

f232003

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

refactor: simplify label rendering and improve color handling in pull…

20685b7

… request table component (#982)

fix: add database index for deployment workflow run ID (#1031)

b3cb8dd

refactor: introduce statusUpdatedAt to HeliosDeployment and enforce d…

a3df999

…eployment blocking logic during environment unlock (#1028)

fix: remove inactive developers from heliosDevelopers list in environ…

7861047

…ment configuration

fix(client): broadcast cross-tab logout to sync auth state and hide w…

5f412f8

…rite/lock actions after sign-out (#1027)

feat: add advanced pull request filtering options (#1033)

556e01e

- add filter bar support for author, assignee, label, review state, and requested reviewer, sort - extend backend pull request query filtering and filter-options endpoint support

fix(deploy): update OPENAI_BASE_URL and OPENAI_MODEL to use secrets i…

4fba51b

…n deployment configuration

fix(deploy): add AI-related environment variables for Helios configur…

53ea950

…ation

fix(deploy): add AI-related environment variables for compose.prod.yaml

f2e34a0

chore: add 30-second delays to deployment workflow steps (#1035)

573c89a

feat: implement WebSocket-based real-time updates for workflow runs w…

9afbc46

…ith JWT authentication (#1012)

Revert "feat: implement WebSocket-based real-time updates for workflo… (

7f9253e

#1036)

refactor: optimize workflow job timing persistence by introducing Hel…

db92475

…iosDeploymentWorkflowJobTimingMeta projection and improved caching logic (#1038)

feat: add environment deployment websocket updates

163b4d3

github-actions Bot added client application-server size:XXL labels May 14, 2026

github-actions Bot assigned mertilginoglu May 14, 2026

mertilginoglu force-pushed the staging branch from db92475 to 1f3daab Compare May 17, 2026 13:02

github-actions Bot added the stale label May 25, 2026

Merge branch 'staging' into codex/environment-websocket-only

a916dc5

github-actions Bot removed the stale label May 28, 2026

github-actions Bot added the stale label Jun 5, 2026

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

feat: add environment deployment websocket updates#1043

feat: add environment deployment websocket updates#1043
mertilginoglu wants to merge 21 commits into
stagingfrom
codex/environment-websocket-only

mertilginoglu commented May 14, 2026

Uh oh!

codacy-production Bot commented May 14, 2026 •

edited

Loading

Uh oh!

github-actions Bot commented May 25, 2026

Uh oh!

krusche commented May 28, 2026

Uh oh!

github-actions Bot commented May 28, 2026

Uh oh!

github-actions Bot commented Jun 5, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Conversation

mertilginoglu commented May 14, 2026

Motivation

Description

Testing Instructions

Prerequisites:

Flow:

Checklist

General

Server

Client

Uh oh!

codacy-production Bot commented May 14, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Not up to standards ⛔

Uh oh!

github-actions Bot commented May 25, 2026

Uh oh!

krusche commented May 28, 2026

Uh oh!

github-actions Bot commented May 28, 2026

Uh oh!

github-actions Bot commented Jun 5, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

codacy-production Bot commented May 14, 2026 •

edited

Loading