Handle errors when getting modifications by FranckLecuyer · Pull Request #61 · gridsuite/monitor-core

FranckLecuyer · 2026-03-04T14:00:57Z

PR Summary

Launch an security analysis process, using an existing config uuid

based on PR : #58
to be merged together with PR : gridsuite/network-modification-server#775

Signed-off-by: Franck LECUYER <franck.lecuyer@rte-france.com>

…onfig Signed-off-by: Franck LECUYER <franck.lecuyer@rte-france.com> # Conflicts: # monitor-commons/src/main/java/org/gridsuite/monitor/commons/SecurityAnalysisConfig.java # monitor-server/src/main/java/org/gridsuite/monitor/server/entities/SecurityAnalysisConfigEntity.java # monitor-server/src/main/java/org/gridsuite/monitor/server/mapper/SecurityAnalysisConfigMapper.java # monitor-server/src/main/resources/db/changelog/db.changelog-master.yaml # monitor-server/src/test/java/org/gridsuite/monitor/server/MonitorIntegrationTest.java # monitor-server/src/test/java/org/gridsuite/monitor/server/controllers/ProcessConfigControllerTest.java

Signed-off-by: Franck LECUYER <franck.lecuyer@rte-france.com>

…config_uuid Signed-off-by: Franck LECUYER <franck.lecuyer@rte-france.com> # Conflicts: # monitor-worker-server/src/main/java/org/gridsuite/monitor/worker/server/services/SecurityAnalysisRestService.java

Signed-off-by: Franck LECUYER <franck.lecuyer@rte-france.com>

Don't set startedAt and completedAt for skipped step Signed-off-by: Franck LECUYER <franck.lecuyer@rte-france.com>

Add process config id in the process execution entity Signed-off-by: Franck LECUYER <franck.lecuyer@rte-france.com>

…en_getting_modifications

coderabbitai · 2026-03-04T14:01:07Z

Note

Reviews paused

It looks like this branch is under active development. To avoid overwhelming you with review comments due to an influx of new commits, CodeRabbit has automatically paused this review. You can configure this behavior by changing the reviews.auto_review.auto_pause_after_reviewed_commits setting.

Use the following commands to manage reviews:

@coderabbitai resume to resume automatic reviews.
@coderabbitai review to trigger a single review.

Use the checkboxes below for quick actions:

▶️ Resume reviews
🔍 Trigger review

📝 Walkthrough

Walkthrough

Adds DTO NetworkModificationsWithMissingInfo and updates REST service, processing step, tests, and resources to return/consume this wrapper; processing now reports and fails when network composite modifications are missing before applying modifications.

Changes

Cohort / File(s)	Summary
New DTO `monitor-worker-server/src/main/java/org/gridsuite/monitor/worker/server/dto/NetworkModificationsWithMissingInfo.java`	Adds public record `NetworkModificationsWithMissingInfo(List<ModificationInfos> networkModifications, List<UUID> missingCompositeModifications)`.
Service API `monitor-worker-server/src/main/java/org/gridsuite/monitor/worker/server/services/NetworkModificationRestService.java`	`getModifications` return type changed to `NetworkModificationsWithMissingInfo`; endpoint path updated to `network-modifications-with-missing-info`; empty response now returns an empty wrapper.
Processing Step `monitor-worker-server/src/main/java/org/gridsuite/monitor/worker/server/processes/commons/steps/ApplyModificationsStep.java`	Consumes `NetworkModificationsWithMissingInfo`; if `missingCompositeModifications` non-empty, creates an error report node (`monitor.worker.server.modifications.error`) with `uuids`, sets severity ERROR, then throws `PowsyblException`; otherwise proceeds with `networkModifications()`.
Localization `monitor-worker-server/src/main/resources/org/gridsuite/monitor/worker/server/reports.properties`	Adds key `monitor.worker.server.modifications.error = Some network composite modifications are missing : ${uuids}`.
Tests `monitor-worker-server/src/test/java/org/gridsuite/monitor/worker/server/processes/commons/steps/ApplyModificationsStepTest.java`, `monitor-worker-server/src/test/java/org/gridsuite/monitor/worker/server/services/NetworkModificationRestServiceTest.java`	Tests updated to use `NetworkModificationsWithMissingInfo`, added cases for missing modifications and empty UUID lists, adjusted mocks and assertions to inspect wrapped fields.
Minor edits / Cleanup `monitor-worker-server/src/main/java/org/gridsuite/monitor/worker/server/services/SecurityAnalysisParametersService.java`, `monitor-worker-server/src/main/java/org/gridsuite/monitor/worker/server/services/StepExecutionService.java`, `.../StepExecutionServiceTest.java`	Whitespace/formatting and removal of `startedAt`/`completedAt` on skipped steps; small test verification additions.

Sequence Diagram

sequenceDiagram
    participant Client as Client
    participant Apply as ApplyModificationsStep
    participant REST as NetworkModificationRestService
    participant Report as ReportService

    Client->>Apply: execute(context)
    Apply->>REST: getModifications(modificationUuids)
    REST-->>Apply: NetworkModificationsWithMissingInfo

    alt missingCompositeModifications non-empty
        Apply->>Report: createReportNode("monitor.worker.server.modifications.error", uuids)
        Report-->>Apply: reportNode
        Apply->>Apply: throw PowsyblException
        Apply-->>Client: Exception
    else no missing composites
        Apply->>Apply: applyModifications(networkModifications)
        Apply-->>Client: Success
    end

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~25 minutes

Possibly related PRs

Homogenise rest services and replace restTemplate with restClient #67: Modifies the same NetworkModificationRestService; related change to return type/endpoint handling.
Get all modifications to apply with only one rest call to the network-modification-server #31: Alters how getModifications is fetched/handled; overlaps REST interaction changes.
Simplify code: do not extract step execution to commons #66: Touches step execution and reporting-related code paths referenced by ApplyModificationsStep updates.

Suggested reviewers

antoinebhs
TheMaskedTurtle
klesaulnier

Poem

🐇 I wrapped the mods in neat little rows,
A missing UUID made my whiskers go,
I hopped to report with a thump and a cheer,
An error arose, the message was clear,
Then I munched a carrot and disappeared.

🚥 Pre-merge checks | ✅ 1 | ❌ 2

❌ Failed checks (1 warning, 1 inconclusive)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.
Description check	❓ Inconclusive	The description mentions launching a security analysis process with existing config UUID and references related PRs, but does not clearly explain the actual changes in this PR regarding error handling for missing modifications.	Clarify the description to explicitly explain that this PR adds error handling for missing composite modifications when fetching network modifications, and how it integrates with the referenced PRs.

✅ Passed checks (1 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title 'Handle errors when getting modifications' directly reflects the main changes: adding error handling for missing composite modifications when retrieving network modifications, including null checks, exception throwing, and error reporting.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

📝 Coding Plan

Generate coding plan for human review comments

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

Tip

You can customize the high-level summary generated by CodeRabbit.

Configure the reviews.high_level_summary_instructions setting to provide custom instructions for generating the high-level summary.

…config_uuid Signed-off-by: Franck LECUYER <franck.lecuyer@rte-france.com> # Conflicts: # monitor-worker-server/src/main/java/org/gridsuite/monitor/worker/server/services/StepExecutionService.java # monitor-worker-server/src/test/java/org/gridsuite/monitor/worker/server/processes/securityanalysis/steps/SecurityAnalysisRunComputationStepTest.java

Signed-off-by: Franck LECUYER <franck.lecuyer@rte-france.com>

…' into handle_errors_when_getting_modifications Signed-off-by: Franck LECUYER <franck.lecuyer@rte-france.com> # Conflicts: # monitor-worker-server/src/main/java/org/gridsuite/monitor/worker/server/services/StepExecutionService.java # monitor-worker-server/src/test/java/org/gridsuite/monitor/worker/server/processes/commons/steps/ApplyModificationsStepTest.java

…tting_modifications Signed-off-by: Franck LECUYER <franck.lecuyer@rte-france.com> # Conflicts: # monitor-commons/src/main/java/org/gridsuite/monitor/commons/SecurityAnalysisConfig.java # monitor-commons/src/main/java/org/gridsuite/monitor/commons/steps/AbstractStepExecutor.java # monitor-commons/src/test/java/org/gridsuite/monitor/commons/steps/StepExecutorTest.java # monitor-server/src/main/java/org/gridsuite/monitor/server/controllers/MonitorController.java # monitor-server/src/main/java/org/gridsuite/monitor/server/mapper/ProcessExecutionMapper.java # monitor-server/src/main/java/org/gridsuite/monitor/server/mapper/SecurityAnalysisConfigMapper.java # monitor-server/src/main/java/org/gridsuite/monitor/server/services/MonitorService.java # monitor-server/src/main/resources/db/changelog/changesets/changelog_20260227T101113Z.xml # monitor-server/src/test/java/org/gridsuite/monitor/server/MonitorIntegrationTest.java # monitor-server/src/test/java/org/gridsuite/monitor/server/controllers/MonitorControllerTest.java # monitor-server/src/test/java/org/gridsuite/monitor/server/services/MonitorServiceTest.java # monitor-server/src/test/java/org/gridsuite/monitor/server/services/ProcessConfigServiceTest.java # monitor-worker-server/pom.xml # monitor-worker-server/src/main/java/org/gridsuite/monitor/worker/server/processes/commons/steps/ApplyModificationsStep.java # monitor-worker-server/src/main/java/org/gridsuite/monitor/worker/server/processes/securityanalysis/steps/SecurityAnalysisRunComputationStep.java # monitor-worker-server/src/main/java/org/gridsuite/monitor/worker/server/services/ActionsRestService.java # monitor-worker-server/src/main/java/org/gridsuite/monitor/worker/server/services/SecurityAnalysisParametersService.java # monitor-worker-server/src/main/java/org/gridsuite/monitor/worker/server/services/SecurityAnalysisRestService.java # monitor-worker-server/src/main/java/org/gridsuite/monitor/worker/server/services/StepExecutionService.java # monitor-worker-server/src/main/resources/org/gridsuite/monitor/worker/server/reports.properties # monitor-worker-server/src/test/java/org/gridsuite/monitor/worker/server/processes/commons/steps/ApplyModificationsStepTest.java # monitor-worker-server/src/test/java/org/gridsuite/monitor/worker/server/processes/securityanalysis/steps/SecurityAnalysisRunComputationStepTest.java # monitor-worker-server/src/test/java/org/gridsuite/monitor/worker/server/services/ActionsRestServiceTest.java # monitor-worker-server/src/test/java/org/gridsuite/monitor/worker/server/services/SecurityAnalysisRestServiceTest.java # monitor-worker-server/src/test/java/org/gridsuite/monitor/worker/server/services/StepExecutionServiceTest.java

Signed-off-by: Franck LECUYER <franck.lecuyer@rte-france.com>

coderabbitai

🧹 Nitpick comments (1)

monitor-worker-server/src/test/java/org/gridsuite/monitor/worker/server/processes/commons/steps/ApplyModificationsStepTest.java (1)
102-119: Consider asserting the specific exception type.

The mocks are now properly set up (addressing previous feedback). However, the production code throws PowsyblException (as seen in ApplyModificationsStep.java line 94), but the test asserts RuntimeException.class. While this works because PowsyblException extends RuntimeException, using the specific type provides better test precision and ensures other runtime exceptions aren't inadvertently caught.
♻️ Proposed fix for more precise assertion
-        assertThrows(RuntimeException.class, () -> applyModificationsStep.execute(stepContext));
+        assertThrows(PowsyblException.class, () -> applyModificationsStep.execute(stepContext));
Add the import:
import com.powsybl.commons.PowsyblException;
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In
`@monitor-worker-server/src/test/java/org/gridsuite/monitor/worker/server/processes/commons/steps/ApplyModificationsStepTest.java`
around lines 102 - 119, Change the test to assert the specific PowsyblException
instead of RuntimeException: in ApplyModificationsStepTest (method
executeApplyModificationsFailWhenModificationsAreMissing) replace
assertThrows(RuntimeException.class, ...) with
assertThrows(PowsyblException.class, ...) and add the import
com.powsybl.commons.PowsyblException; this aligns the test with the actual
exception thrown by ApplyModificationsStep (PowsyblException).

🤖 Prompt for all review comments with AI agents

Verify each finding against the current code and only fix it if needed.

Nitpick comments:
In
`@monitor-worker-server/src/test/java/org/gridsuite/monitor/worker/server/processes/commons/steps/ApplyModificationsStepTest.java`:
- Around line 102-119: Change the test to assert the specific PowsyblException
instead of RuntimeException: in ApplyModificationsStepTest (method
executeApplyModificationsFailWhenModificationsAreMissing) replace
assertThrows(RuntimeException.class, ...) with
assertThrows(PowsyblException.class, ...) and add the import
com.powsybl.commons.PowsyblException; this aligns the test with the actual
exception thrown by ApplyModificationsStep (PowsyblException).

ℹ️ Review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 9611a4fb-9959-4409-8826-b085c9b2002e

📥 Commits

Reviewing files that changed from the base of the PR and between ddc3801 and 60a59a3.

📒 Files selected for processing (1)

monitor-worker-server/src/test/java/org/gridsuite/monitor/worker/server/processes/commons/steps/ApplyModificationsStepTest.java

Signed-off-by: Franck LECUYER <franck.lecuyer@rte-france.com>

…tting_modifications

Signed-off-by: Franck LECUYER <franck.lecuyer@rte-france.com>

antoinebhs

I'm not sure to understand why we need this complexity?
On main, we know the ID we want:
networkModificationRestService.getModifications(modificationIds);
and the ID we get List modificationInfos.getUuid()
Why can't we deduce the missing modifications ?

antoinebhs · 2026-03-13T07:59:07Z

...ver/src/main/java/org/gridsuite/monitor/worker/server/orchestrator/StepExecutionService.java

@@ -32,8 +32,6 @@ public void skipStep(ProcessStepExecutionContext<?> context, ProcessStep<?> step
                .stepType(step.getType().getName())
                .stepOrder(context.getStepOrder())
                .status(StepStatus.SKIPPED)
-                .startedAt(context.getStartedAt())
-                .completedAt(Instant.now())


this helped to keep track of when it failed?

I think it would be better to keep it to avoid null values?

Signed-off-by: Franck LECUYER <franck.lecuyer@rte-france.com>

…_modifications # Conflicts: # monitor-worker-server/src/main/java/org/gridsuite/monitor/worker/server/process/commons/steps/ApplyModificationsStep.java # monitor-worker-server/src/main/java/org/gridsuite/monitor/worker/server/services/SecurityAnalysisParametersService.java # monitor-worker-server/src/test/java/org/gridsuite/monitor/worker/server/clients/NetworkModificationRestClientTest.java # monitor-worker-server/src/test/java/org/gridsuite/monitor/worker/server/orchestrator/StepExecutionServiceTest.java # monitor-worker-server/src/test/java/org/gridsuite/monitor/worker/server/process/commons/steps/ApplyModificationsStepTest.java

antoinebhs

Some maybe to discuss? let me know

antoinebhs · 2026-03-19T14:17:44Z

...uite/monitor/worker/server/dto/networkmodifications/NetworkModificationsWithMissingInfo.java

@@ -0,0 +1,18 @@
+/**


to put in the right subpackage

antoinebhs · 2026-03-19T14:18:09Z

...ver/src/main/java/org/gridsuite/monitor/worker/server/orchestrator/StepExecutionService.java

@@ -32,8 +32,6 @@ public void skipStep(ProcessStepExecutionContext<?> context, ProcessStep<?> step
                .stepType(step.getType().getName())
                .stepOrder(context.getStepOrder())
                .status(StepStatus.SKIPPED)
-                .startedAt(context.getStartedAt())
-                .completedAt(Instant.now())


I think it would be better to keep it to avoid null values?

antoinebhs · 2026-03-19T14:18:35Z

...n/java/org/gridsuite/monitor/worker/server/process/commons/steps/ApplyModificationsStep.java

-        networkModificationService.applyModifications(network, modificationInfos, reportNode, filterService);
+        NetworkModificationsWithMissingInfo networkModificationsWithMissingInfo = networkModificationRestClient.getModifications(modificationIds);
+        if (networkModificationsWithMissingInfo == null) {
+            throw new PowsyblException("Failed to retrieve network modifications");


Can't be null because it's an empty list in this case?

antoinebhs · 2026-03-19T14:18:55Z

...n/java/org/gridsuite/monitor/worker/server/process/commons/steps/ApplyModificationsStep.java

+                .withUntypedValue("uuids", missingUuids)
+                .withSeverity(TypedValue.ERROR_SEVERITY)
+                .add();
+            throw new PowsyblException("Some network composite modifications are missing !!");


!! to remove?

antoinebhs · 2026-03-19T14:19:31Z

...src/test/java/org/gridsuite/monitor/worker/server/orchestrator/StepExecutionServiceTest.java

        when(context.getProcessExecutionId()).thenReturn(executionId);
        when(context.getStepExecutionId()).thenReturn(UUID.randomUUID());
-        when(context.getStartedAt()).thenReturn(java.time.Instant.now());
        when(context.getStepOrder()).thenReturn(stepOrder);


...va/org/gridsuite/monitor/worker/server/process/commons/steps/ApplyModificationsStepTest.java

Signed-off-by: Franck LECUYER <franck.lecuyer@rte-france.com>

sonarqubecloud · 2026-03-20T12:53:16Z

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
100.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

antoinebhs · 2026-03-20T16:18:22Z

...va/org/gridsuite/monitor/worker/server/process/commons/steps/ApplyModificationsStepTest.java

+        when(stepContext.getReportInfos()).thenReturn(reportInfos);
+        when(networkModificationRestClient.getModifications(any(List.class))).thenReturn(networkModificationsWithMissingInfo);
+
+        assertThrows(RuntimeException.class, () -> applyModificationsStep.execute(stepContext));


Check that it's a powsybl exception with the correct message?

antoinebhs · 2026-03-20T16:19:16Z

...va/org/gridsuite/monitor/worker/server/process/commons/steps/ApplyModificationsStepTest.java

+        String stepType = applyModificationsStep.getType().getName();
+        assertEquals("APPLY_MODIFICATIONS", stepType);


Already tested?

FranckLecuyer added 12 commits February 4, 2026 15:31

Add loadflow parameters to security analysis config

cdbacad

Signed-off-by: Franck LECUYER <franck.lecuyer@rte-france.com>

Add loadflow parameters to security analysis config

137c20c

Signed-off-by: Franck LECUYER <franck.lecuyer@rte-france.com>

Launch process execution using config uuid : in progress ...

4bf70f8

Signed-off-by: Franck LECUYER <franck.lecuyer@rte-france.com>

Merge remote-tracking branch 'origin/main' into launch_process_using_…

fc79b8e

…config_uuid Signed-off-by: Franck LECUYER <franck.lecuyer@rte-france.com> # Conflicts: # monitor-worker-server/src/main/java/org/gridsuite/monitor/worker/server/services/SecurityAnalysisRestService.java

Launch process execution using config uuid

26832f2

Signed-off-by: Franck LECUYER <franck.lecuyer@rte-france.com>

Add process config id in the process execution entity

2ca2962

Signed-off-by: Franck LECUYER <franck.lecuyer@rte-france.com>

Chnge some things after coderabbit review

4f3cd3c

Signed-off-by: Franck LECUYER <franck.lecuyer@rte-france.com>

Fix return code explanation

7e19290

Signed-off-by: Franck LECUYER <franck.lecuyer@rte-france.com>

Handle errors when getting modifications

89d7610

Don't set startedAt and completedAt for skipped step Signed-off-by: Franck LECUYER <franck.lecuyer@rte-france.com>

Launch process execution using config uuid

a3f0d73

Add process config id in the process execution entity Signed-off-by: Franck LECUYER <franck.lecuyer@rte-france.com>

Merge branch 'launch_process_using_config_uuid' into handle_errors_wh…

5a7a55d

…en_getting_modifications

FranckLecuyer changed the base branch from launch_process_using_config_uuid to main March 4, 2026 14:03

FranckLecuyer changed the base branch from main to launch_process_using_config_uuid March 4, 2026 14:05

FranckLecuyer added 3 commits March 5, 2026 16:16

Changes after code review

9e55c27

Signed-off-by: Franck LECUYER <franck.lecuyer@rte-france.com>

Base automatically changed from launch_process_using_config_uuid to main March 12, 2026 08:55

FranckLecuyer added 2 commits March 12, 2026 13:01

Fix coverage pb

60a59a3

Signed-off-by: Franck LECUYER <franck.lecuyer@rte-france.com>

This comment was marked as outdated.

Sign in to view

Fix coderabbit issues

e7feedd

Signed-off-by: Franck LECUYER <franck.lecuyer@rte-france.com>

coderabbitai bot reviewed Mar 12, 2026

View reviewed changes

This comment was marked as outdated.

Sign in to view

FranckLecuyer added 2 commits March 12, 2026 14:01

Fix potential null pointer exception

d2a3812

Signed-off-by: Franck LECUYER <franck.lecuyer@rte-france.com>

Merge remote-tracking branch 'origin/main' into handle_errors_when_ge…

a5fdbb7

…tting_modifications

FranckLecuyer force-pushed the handle_errors_when_getting_modifications branch from f84ea7c to a5fdbb7 Compare March 12, 2026 13:04

This comment was marked as outdated.

Sign in to view

Fix potential null pointer exception

4df4730

Signed-off-by: Franck LECUYER <franck.lecuyer@rte-france.com>

antoinebhs reviewed Mar 13, 2026

View reviewed changes

FranckLecuyer and others added 2 commits March 18, 2026 10:16

Merge branch 'main' into handle_errors_when_getting_modifications

67c462b

Fix compilation pb

7e67f77

Signed-off-by: Franck LECUYER <franck.lecuyer@rte-france.com>

This comment was marked as outdated.

Sign in to view

FranckLecuyer and others added 3 commits March 19, 2026 10:35

Merge branch 'main' into handle_errors_when_getting_modifications

1afb07d

Fix merge

463edb7

antoinebhs reviewed Mar 19, 2026

View reviewed changes

FranckLecuyer and others added 3 commits March 20, 2026 11:42

Merge branch 'main' into handle_errors_when_getting_modifications

0ec2657

Merge branch 'main' into handle_errors_when_getting_modifications

1782d94

Changes after review

12f78fa

Signed-off-by: Franck LECUYER <franck.lecuyer@rte-france.com>

FranckLecuyer requested a review from antoinebhs March 20, 2026 12:51

antoinebhs reviewed Mar 20, 2026

View reviewed changes

		String stepType = applyModificationsStep.getType().getName();
		assertEquals("APPLY_MODIFICATIONS", stepType);

Conversation

FranckLecuyer commented Mar 4, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

PR Summary

Uh oh!

coderabbitai bot commented Mar 4, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Reviews paused

Walkthrough

Changes

Sequence Diagram

Estimated code review effort

Possibly related PRs

Suggested reviewers

Poem

❌ Failed checks (1 warning, 1 inconclusive)

Uh oh!

This comment was marked as outdated.

Uh oh!

coderabbitai bot left a comment

Choose a reason for hiding this comment

Uh oh!

This comment was marked as outdated.

Uh oh!

This comment was marked as outdated.

Uh oh!

antoinebhs left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

This comment was marked as outdated.

Uh oh!

antoinebhs left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

sonarqubecloud bot commented Mar 20, 2026

Quality Gate passed

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

FranckLecuyer commented Mar 4, 2026 •

edited

Loading

coderabbitai bot commented Mar 4, 2026 •

edited

Loading