Skip to content

feat(python): python code intelligence via Pyrefly with Ruff formatting - #3016

Merged
abose merged 6 commits into
mainfrom
ai
Jul 24, 2026
Merged

feat(python): python code intelligence via Pyrefly with Ruff formatting#3016
abose merged 6 commits into
mainfrom
ai

refactor(lsp): shouldShowParameterHints config hook replaces provider…

0d36113
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / SonarCloud Code Analysis failed Jul 23, 2026 in 51s

Quality Gate failed

Failed conditions
15.6% Duplication on New Code (required ≤ 3%)

See analysis details on SonarQube Cloud

Annotations

Check warning on line 176 in src/extensions/default/PHPSupport/unittests.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Prefer using an optional chain expression instead, as it's more concise and easier to read. 

See more on https://sonarcloud.io/project/issues?id=phcode-dev_phoenix&issues=AZ-JuhLmlM3PEnpizNGz&open=AZ-JuhLmlM3PEnpizNGz&pullRequest=3016

Check warning on line 185 in src/extensions/default/PythonSupport/unittests.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Prefer using an optional chain expression instead, as it's more concise and easier to read. 

See more on https://sonarcloud.io/project/issues?id=phcode-dev_phoenix&issues=AZ8-VH4_NMmptJydKTxs&open=AZ8-VH4_NMmptJydKTxs&pullRequest=3016

Check warning on line 110 in src/extensions/default/PythonSupport/main.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

`String.raw` should be used to avoid escaping `\`. 

See more on https://sonarcloud.io/project/issues?id=phcode-dev_phoenix&issues=AZ-Icg9v13EHziOTs_Pf&open=AZ-Icg9v13EHziOTs_Pf&pullRequest=3016

Check failure on line 278 in src/extensions/default/TypeScriptSupport/main.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

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

See more on https://sonarcloud.io/project/issues?id=phcode-dev_phoenix&issues=AZ-O5W0YQhrEfyBr0An-&open=AZ-O5W0YQhrEfyBr0An-&pullRequest=3016

Check warning on line 132 in src/extensions/default/PythonSupport/unittests.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use `.includes()`, rather than `.indexOf()`, when checking for existence. 

See more on https://sonarcloud.io/project/issues?id=phcode-dev_phoenix&issues=AZ8-VH4_NMmptJydKTxm&open=AZ8-VH4_NMmptJydKTxm&pullRequest=3016

Check warning on line 229 in src/utils/NodeUtils.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Prefer using an optional chain expression instead, as it's more concise and easier to read. 

See more on https://sonarcloud.io/project/issues?id=phcode-dev_phoenix&issues=AZ8-VHzsNMmptJydKTxT&open=AZ8-VHzsNMmptJydKTxT&pullRequest=3016

Check warning on line 213 in src/extensions/default/PythonSupport/ServerInstaller.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Prefer using an optional chain expression instead, as it's more concise and easier to read. 

See more on https://sonarcloud.io/project/issues?id=phcode-dev_phoenix&issues=AZ8-VH43NMmptJydKTxa&open=AZ8-VH43NMmptJydKTxa&pullRequest=3016

Check warning on line 410 in src/extensions/default/PythonSupport/ServerInstaller.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Prefer using an optional chain expression instead, as it's more concise and easier to read. 

See more on https://sonarcloud.io/project/issues?id=phcode-dev_phoenix&issues=AZ8-VH43NMmptJydKTxe&open=AZ8-VH43NMmptJydKTxe&pullRequest=3016

Check warning on line 131 in src/extensions/default/PythonSupport/unittests.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use `.includes()`, rather than `.indexOf()`, when checking for existence. 

See more on https://sonarcloud.io/project/issues?id=phcode-dev_phoenix&issues=AZ8-VH4_NMmptJydKTxl&open=AZ8-VH4_NMmptJydKTxl&pullRequest=3016

Check warning on line 478 in src/extensions/default/PythonSupport/ServerInstaller.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Move function 'hidePanelRow' to the outer scope. 

See more on https://sonarcloud.io/project/issues?id=phcode-dev_phoenix&issues=AZ8-VH43NMmptJydKTxg&open=AZ8-VH43NMmptJydKTxg&pullRequest=3016

Check warning on line 140 in src/extensions/default/PythonSupport/ServerInstaller.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Handle this exception or don't catch it at all. 

See more on https://sonarcloud.io/project/issues?id=phcode-dev_phoenix&issues=AZ8-VH43NMmptJydKTxY&open=AZ8-VH43NMmptJydKTxY&pullRequest=3016

Check warning on line 175 in src/extensions/default/PythonSupport/main.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Prefer using an optional chain expression instead, as it's more concise and easier to read. 

See more on https://sonarcloud.io/project/issues?id=phcode-dev_phoenix&issues=AZ8-VH5INMmptJydKTxv&open=AZ8-VH5INMmptJydKTxv&pullRequest=3016

Check warning on line 157 in src/utils/NodeUtils.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Prefer using an optional chain expression instead, as it's more concise and easier to read. 

See more on https://sonarcloud.io/project/issues?id=phcode-dev_phoenix&issues=AZ8-VHzsNMmptJydKTxR&open=AZ8-VHzsNMmptJydKTxR&pullRequest=3016

Check warning on line 152 in src/utils/NodeUtils.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Prefer using an optional chain expression instead, as it's more concise and easier to read. 

See more on https://sonarcloud.io/project/issues?id=phcode-dev_phoenix&issues=AZ8-VHzsNMmptJydKTxQ&open=AZ8-VHzsNMmptJydKTxQ&pullRequest=3016

Check warning on line 408 in src/languageTools/LSPClient.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Prefer using an optional chain expression instead, as it's more concise and easier to read. 

See more on https://sonarcloud.io/project/issues?id=phcode-dev_phoenix&issues=AZ-JuhTblM3PEnpizNG0&open=AZ-JuhTblM3PEnpizNG0&pullRequest=3016

Check warning on line 205 in src/extensions/default/PythonSupport/ServerInstaller.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Move async function '_resolveWheel' to the outer scope. 

See more on https://sonarcloud.io/project/issues?id=phcode-dev_phoenix&issues=AZ8-VH43NMmptJydKTxZ&open=AZ8-VH43NMmptJydKTxZ&pullRequest=3016

Check warning on line 166 in src/utils/NodeUtils.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Prefer using an optional chain expression instead, as it's more concise and easier to read. 

See more on https://sonarcloud.io/project/issues?id=phcode-dev_phoenix&issues=AZ8-VHzsNMmptJydKTxS&open=AZ8-VHzsNMmptJydKTxS&pullRequest=3016

Check warning on line 144 in src/extensions/default/PythonSupport/unittests.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use `.includes()`, rather than `.indexOf()`, when checking for existence. 

See more on https://sonarcloud.io/project/issues?id=phcode-dev_phoenix&issues=AZ8-VH4_NMmptJydKTxn&open=AZ8-VH4_NMmptJydKTxn&pullRequest=3016

Check warning on line 162 in src/extensions/default/TypeScriptSupport/unittests.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Move function '_parseCursorMarker' to the outer scope. 

See more on https://sonarcloud.io/project/issues?id=phcode-dev_phoenix&issues=AZ-O5WuUQhrEfyBr0An8&open=AZ-O5WuUQhrEfyBr0An8&pullRequest=3016

Check failure on line 159 in src/extensions/default/PHPSupport/unittests.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

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

See more on https://sonarcloud.io/project/issues?id=phcode-dev_phoenix&issues=AZ-JuhLmlM3PEnpizNGy&open=AZ-JuhLmlM3PEnpizNGy&pullRequest=3016

Check warning on line 230 in src/utils/NodeUtils.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Prefer using an optional chain expression instead, as it's more concise and easier to read. 

See more on https://sonarcloud.io/project/issues?id=phcode-dev_phoenix&issues=AZ8-VHzsNMmptJydKTxU&open=AZ8-VHzsNMmptJydKTxU&pullRequest=3016

Check warning on line 317 in src/extensions/default/TypeScriptSupport/main.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use `.includes()`, rather than `.indexOf()`, when checking for existence. 

See more on https://sonarcloud.io/project/issues?id=phcode-dev_phoenix&issues=AZ-O5W0YQhrEfyBr0AoA&open=AZ-O5W0YQhrEfyBr0AoA&pullRequest=3016

Check warning on line 203 in src/extensions/default/PythonSupport/main.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Prefer using an optional chain expression instead, as it's more concise and easier to read. 

See more on https://sonarcloud.io/project/issues?id=phcode-dev_phoenix&issues=AZ8-VH5INMmptJydKTxw&open=AZ8-VH5INMmptJydKTxw&pullRequest=3016

Check warning on line 117 in src/extensions/default/PythonSupport/unittests.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Prefer using an optional chain expression instead, as it's more concise and easier to read. 

See more on https://sonarcloud.io/project/issues?id=phcode-dev_phoenix&issues=AZ8-VH4_NMmptJydKTxj&open=AZ8-VH4_NMmptJydKTxj&pullRequest=3016

Check warning on line 213 in src/extensions/default/PythonSupport/ServerInstaller.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use `.includes()`, rather than `.indexOf()`, when checking for existence. 

See more on https://sonarcloud.io/project/issues?id=phcode-dev_phoenix&issues=AZ8-VH43NMmptJydKTxb&open=AZ8-VH43NMmptJydKTxb&pullRequest=3016
View more details on SonarQubeCloud