Skip to content

Fixes 30715: block saving a persona AI context rule with an unfinished condition - #30719

Open
pmbrull wants to merge 1 commit into
pmbrull/collate-backend-60-sentryfrom
pmbrull/persona-context-rule-save-validation
Open

Fixes 30715: block saving a persona AI context rule with an unfinished condition#30719
pmbrull wants to merge 1 commit into
pmbrull/collate-backend-60-sentryfrom
pmbrull/persona-context-rule-save-validation

fix(ui): block saving a persona rule with an unfinished condition

4b87fe8
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / [open-metadata-ui] SonarCloud Code Analysis succeeded Jul 30, 2026 in 8m 46s

Quality Gate passed

Issues
965 New issues
0 Accepted issues

Measures
13 Security Hotspots
0.0% Coverage on New Code
3.0% Duplication on New Code

See analysis details on SonarQube Cloud

Annotations

Check warning on line 186 in src/components/DataQuality/TestCases/useTestCaseFilters.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Consider Object.entries() instead of entries() from lodash; check that the behavior is equivalent because the library handles nullish values differently from the native API. 

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ-zxy59wjERoAOKcje1&open=AZ-zxy59wjERoAOKcje1&pullRequest=30719

Check failure on line 160 in src/utils/BlockEditorUtils.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Prefer `.dataset` over `setAttribute(…)`. 

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ-zx_o2wjERoAOKcjpV&open=AZ-zx_o2wjERoAOKcjpV&pullRequest=30719

Check failure on line 60 in src/components/BlockEditor/Extensions/File/FileNode.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Prefer `.dataset` over `getAttribute(…)`. 

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ-zxyKuwjERoAOKcjeP&open=AZ-zxyKuwjERoAOKcjeP&pullRequest=30719

Check warning on line 250 in src/utils/EntityLineageNodeUtils.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Default parameters should be last. 

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ-zyBUowjERoAOKcjrB&open=AZ-zyBUowjERoAOKcjrB&pullRequest=30719

Check warning on line 99 in src/components/ExploreV1/ExploreSearchCard/ExploreSearchCard.tsx

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Use the "RegExp.exec()" method instead. 

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ-zx817wjERoAOKcjmp&open=AZ-zx817wjERoAOKcjmp&pullRequest=30719

Check warning on line 410 in src/utils/CSV/CSVPureUtils.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

This pattern can be replaced with '\\"'. 

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ-zx-q5wjERoAOKcjok&open=AZ-zx-q5wjERoAOKcjok&pullRequest=30719

Check warning on line 738 in src/hooks/useGridEditController.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Prefer `Number.parseInt` over `parseInt`. 

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ-zyEQwwjERoAOKcjtP&open=AZ-zyEQwwjERoAOKcjtP&pullRequest=30719

Check failure on line 64 in src/components/BlockEditor/Extensions/File/FileNode.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Prefer `.dataset` over `getAttribute(…)`. 

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ-zxyKuwjERoAOKcjeT&open=AZ-zxyKuwjERoAOKcjeT&pullRequest=30719

Check warning on line 85 in src/utils/AdvancedSearchPureUtils.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Use Array.isArray() instead of isArray() from lodash. 

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ-zx_9lwjERoAOKcjpo&open=AZ-zx_9lwjERoAOKcjpo&pullRequest=30719

Check warning on line 55 in src/utils/ServiceConnectionDetailsUtils.tsx

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Use Array.isArray() instead of isArray() from lodash. 

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ-zyBq6wjERoAOKcjrQ&open=AZ-zyBq6wjERoAOKcjrQ&pullRequest=30719

Check warning on line 262 in src/utils/QueryBuilderElasticsearchFormatUtils.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Use the "RegExp.exec()" method instead. 

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ-zyAWLwjERoAOKcjp7&open=AZ-zyAWLwjERoAOKcjp7&pullRequest=30719

Check warning on line 724 in src/utils/QueryBuilderElasticsearchFormatUtils.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

This branch's code block is the same as the block for the branch on line 513. 

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ-zyAWLwjERoAOKcjqC&open=AZ-zyAWLwjERoAOKcjqC&pullRequest=30719

Check warning on line 358 in src/components/TestLibrary/TestDefinitionForm/TestDefinitionFormBody.tsx

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Avoid non-native interactive elements. If using native HTML is not possible, add an appropriate role and support for tabbing, mouse, keyboard, and touch inputs to an interactive content element. 

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ-zx3SNwjERoAOKcjiH&open=AZ-zx3SNwjERoAOKcjiH&pullRequest=30719

Check warning on line 47 in src/components/Entity/EntityLineage/LineageNodeLabelV1.tsx

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

'toggleOnlyShowColumnsWithLineageFilterActive' PropType is defined but prop is never used 

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ-zx6KHwjERoAOKcjkf&open=AZ-zx6KHwjERoAOKcjkf&pullRequest=30719

Check warning on line 185 in src/utils/KnowledgePagePureUtils.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Default parameters should be last. 

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ-zyAGywjERoAOKcjpw&open=AZ-zyAGywjERoAOKcjpw&pullRequest=30719

Check warning on line 171 in src/components/BlockEditor/BarMenu/BarMenu.tsx

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Add an explicit "type" attribute to this button. 

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ-zxyV9wjERoAOKcjee&open=AZ-zxyV9wjERoAOKcjee&pullRequest=30719

Check warning on line 391 in src/components/TestLibrary/TestDefinitionForm/TestDefinitionFormBody.tsx

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Visible, non-interactive elements with click handlers must have at least one keyboard listener. 

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ-zx3SNwjERoAOKcjiK&open=AZ-zx3SNwjERoAOKcjiK&pullRequest=30719

Check warning on line 96 in src/utils/DataInsightPureUtils.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Default parameters should be last. 

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ-zx_H6wjERoAOKcjo7&open=AZ-zx_H6wjERoAOKcjo7&pullRequest=30719

Check failure on line 430 in src/components/DataQuality/AddDataQualityTest/components/ParameterForm.tsx

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Refactor this code to not nest functions more than 5 levels deep. 

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ-zxyqZwjERoAOKcjen&open=AZ-zxyqZwjERoAOKcjen&pullRequest=30719

Check warning on line 423 in src/components/Settings/Users/UsersProfile/UserPermissions/UserPermissions.component.tsx

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Do not use Array index in keys 

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ-zx4UZwjERoAOKcjjM&open=AZ-zx4UZwjERoAOKcjjM&pullRequest=30719

Check warning on line 73 in src/components/BlockEditor/Extensions/File/FileNode.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Prefer `Number.parseInt` over `parseInt`. 

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ-zxyKuwjERoAOKcjeY&open=AZ-zxyKuwjERoAOKcjeY&pullRequest=30719

Check warning on line 191 in src/utils/FeedUtilsPure.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Use the "RegExp.exec()" method instead. 

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ-zx999wjERoAOKcjni&open=AZ-zx999wjERoAOKcjni&pullRequest=30719

Check failure on line 577 in src/pages/EntityVersionPage/EntityVersionPage.component.tsx

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Refactor this function to reduce its Cognitive Complexity from 45 to the 15 allowed. 

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ-zyDHowjERoAOKcjsM&open=AZ-zyDHowjERoAOKcjsM&pullRequest=30719

Check warning on line 303 in src/context/LineageProvider/LineageProvider.tsx

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

useState call is not destructured into value + setter pair 

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ-zyEjEwjERoAOKcjta&open=AZ-zyEjEwjERoAOKcjta&pullRequest=30719

Check warning on line 180 in src/utils/CustomProperty.utils.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

'value' will use Object's default stringification format ('[object Object]') when stringified. 

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ-zyATQwjERoAOKcjp5&open=AZ-zyATQwjERoAOKcjp5&pullRequest=30719
View more details on SonarQubeCloud