Skip to content

feat: run declared JavaScript without compiling it in the browser - #25749

Merged
Artur- merged 59 commits into
mainfrom
feat/js-invoker-seam-for-focusable
Sep 21, 2026
Merged

feat: run declared JavaScript without compiling it in the browser#25749
Artur- merged 59 commits into
mainfrom
feat/js-invoker-seam-for-focusable

docs: say that the guard keeps the lists apart, not that it changes a…

39953e1
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / SonarCloud Code Analysis succeeded Sep 21, 2026 in 1m 45s

Quality Gate passed

Issues
15 New issues
0 Accepted issues

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

See analysis details on SonarQube Cloud

Annotations

Check warning on line 87 in flow-server/src/test/java/com/vaadin/flow/js/JsCallTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Refactor the code of the lambda to have only one invocation possibly throwing a runtime exception. 

See more on https://sonarcloud.io/project/issues?id=vaadin_flow&issues=AaCpMJcB2JmT49p53bpi&open=AaCpMJcB2JmT49p53bpi&pullRequest=25749

Check warning on line 98 in flow-server/src/main/java/com/vaadin/flow/js/JsDefinitionProxy.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Reduce the total number of break and continue statements in this loop to use at most one. 

See more on https://sonarcloud.io/project/issues?id=vaadin_flow&issues=AaDFH8Sw3poOSLI6vaV9&open=AaDFH8Sw3poOSLI6vaV9&pullRequest=25749

Check warning on line 129 in vaadin-dev-server/src/main/java/com/vaadin/base/devserver/hotswap/impl/JsDefinitionHotswapper.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Invoke method(s) only conditionally. 

See more on https://sonarcloud.io/project/issues?id=vaadin_flow&issues=AaC-_Q5Qy0vepUx49bgi&open=AaC-_Q5Qy0vepUx49bgi&pullRequest=25749

Check warning on line 359 in flow-server/src/main/java/com/vaadin/flow/server/communication/UidlRequestHandler.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use "==" to perform this enum comparison instead of using "equals" 

See more on https://sonarcloud.io/project/issues?id=vaadin_flow&issues=AaDCli0wgUv0fnrTKyAX&open=AaDCli0wgUv0fnrTKyAX&pullRequest=25749

Check warning on line 92 in flow-server/src/main/java/com/vaadin/flow/server/communication/UidlRequestHandler.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Move this static variable to comply with Java Code Conventions. 

See more on https://sonarcloud.io/project/issues?id=vaadin_flow&issues=AaDCli0wgUv0fnrTKyAV&open=AaDCli0wgUv0fnrTKyAV&pullRequest=25749

Check warning on line 307 in flow-server/src/main/java/com/vaadin/flow/server/communication/UidlRequestHandler.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Reduce the total number of break and continue statements in this loop to use at most one. 

See more on https://sonarcloud.io/project/issues?id=vaadin_flow&issues=AaDCli0wgUv0fnrTKyAW&open=AaDCli0wgUv0fnrTKyAW&pullRequest=25749

Check failure on line 59 in flow-server/src/test/java/com/vaadin/flow/js/JsCallTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Add a nested comment explaining why this method is empty, throw an UnsupportedOperationException or complete the implementation. 

See more on https://sonarcloud.io/project/issues?id=vaadin_flow&issues=AaCpMJcB2JmT49p53bpf&open=AaCpMJcB2JmT49p53bpf&pullRequest=25749

Check failure on line 647 in flow-server/src/test/java/com/vaadin/flow/internal/ReflectToolsTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Add a nested comment explaining why this method is empty, throw an UnsupportedOperationException or complete the implementation. 

See more on https://sonarcloud.io/project/issues?id=vaadin_flow&issues=AaC_rWkzRffk6HRCpfh3&open=AaC_rWkzRffk6HRCpfh3&pullRequest=25749

Check warning on line 364 in flow-server/src/main/java/com/vaadin/flow/server/communication/UidlRequestHandler.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use "==" to perform this enum comparison instead of using "equals" 

See more on https://sonarcloud.io/project/issues?id=vaadin_flow&issues=AaDCli0wgUv0fnrTKyAY&open=AaDCli0wgUv0fnrTKyAY&pullRequest=25749

Check warning on line 117 in flow-server/src/test/java/com/vaadin/flow/js/JsCallTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Refactor the code of the lambda to have only one invocation possibly throwing a runtime exception. 

See more on https://sonarcloud.io/project/issues?id=vaadin_flow&issues=AaCpMJcB2JmT49p53bpj&open=AaCpMJcB2JmT49p53bpj&pullRequest=25749

Check failure on line 650 in flow-server/src/test/java/com/vaadin/flow/internal/ReflectToolsTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Add a nested comment explaining why this method is empty, throw an UnsupportedOperationException or complete the implementation. 

See more on https://sonarcloud.io/project/issues?id=vaadin_flow&issues=AaC_rWkzRffk6HRCpfh4&open=AaC_rWkzRffk6HRCpfh4&pullRequest=25749

Check warning on line 74 in flow-server/src/test/java/com/vaadin/flow/js/JsCallTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Refactor the code of the lambda to have only one invocation possibly throwing a runtime exception. 

See more on https://sonarcloud.io/project/issues?id=vaadin_flow&issues=AaCpMJcB2JmT49p53bph&open=AaCpMJcB2JmT49p53bph&pullRequest=25749

Check failure on line 63 in flow-server/src/test/java/com/vaadin/flow/js/JsCallTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Add a nested comment explaining why this method is empty, throw an UnsupportedOperationException or complete the implementation. 

See more on https://sonarcloud.io/project/issues?id=vaadin_flow&issues=AaCpMJcB2JmT49p53bpg&open=AaCpMJcB2JmT49p53bpg&pullRequest=25749

Check warning on line 128 in flow-server/src/test/java/com/vaadin/flow/js/JsCallTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Refactor the code of the lambda to have only one invocation possibly throwing a runtime exception. 

See more on https://sonarcloud.io/project/issues?id=vaadin_flow&issues=AaCpMJcB2JmT49p53bpk&open=AaCpMJcB2JmT49p53bpk&pullRequest=25749

Check failure on line 653 in flow-server/src/test/java/com/vaadin/flow/internal/ReflectToolsTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Add a nested comment explaining why this method is empty, throw an UnsupportedOperationException or complete the implementation. 

See more on https://sonarcloud.io/project/issues?id=vaadin_flow&issues=AaC_rWkzRffk6HRCpfh5&open=AaC_rWkzRffk6HRCpfh5&pullRequest=25749
View more details on SonarQubeCloud