Skip to content

JSON code intelligence and LSP support#3010

Merged
abose merged 3 commits into
mainfrom
ai
Jul 3, 2026
Merged

JSON code intelligence and LSP support#3010
abose merged 3 commits into
mainfrom
ai

fix(build): releaseProd died on nullish coalescing - terser 3 can't p…

aadee81
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / SonarCloud Code Analysis succeeded Jul 2, 2026 in 42s

Annotations

Check warning on line 215 in src/extensionsIntegrated/JSONSupport/VulnerabilityInspection.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=AZ8kAvO0RoBB8p1B2vhv&open=AZ8kAvO0RoBB8p1B2vhv&pullRequest=3010

Check warning on line 194 in src/extensionsIntegrated/JSONSupport/VulnerabilityInspection.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Move function '_depHash' to the outer scope.

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

Check warning on line 176 in src/extensionsIntegrated/JSONSupport/NpmHints.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use 'String#startsWith' method instead.

See more on https://sonarcloud.io/project/issues?id=phcode-dev_phoenix&issues=AZ8kAvO-RoBB8p1B2vh8&open=AZ8kAvO-RoBB8p1B2vh8&pullRequest=3010

Check warning on line 193 in src/extensionsIntegrated/JSONSupport/JsonLsp.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Prefer `globalThis` over `window`.

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

Check warning on line 60 in src/extensionsIntegrated/JSONSupport/VulnerabilityInspection.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Move function '_matchBrace' to the outer scope.

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

Check failure on line 60 in src/extensionsIntegrated/JSONSupport/VulnerabilityInspection.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

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

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

Check warning on line 56 in src/extensionsIntegrated/JSONSupport/NpmRegistry.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Prefer `globalThis` over `window`.

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

Check warning on line 226 in src/extensionsIntegrated/JSONSupport/VulnerabilityInspection.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=AZ8kAvO0RoBB8p1B2vhw&open=AZ8kAvO0RoBB8p1B2vhw&pullRequest=3010

Check warning on line 64 in src/extensionsIntegrated/JSONSupport/NpmHints.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=AZ8kAvO-RoBB8p1B2vh3&open=AZ8kAvO-RoBB8p1B2vh3&pullRequest=3010

Check warning on line 237 in src/extensionsIntegrated/JSONSupport/NpmHints.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=AZ8kAvO-RoBB8p1B2vh_&open=AZ8kAvO-RoBB8p1B2vh_&pullRequest=3010

Check warning on line 92 in src/extensionsIntegrated/JSONSupport/NpmHover.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Expected the Promise rejection reason to be an Error.

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

Check failure on line 128 in src/extensionsIntegrated/JSONSupport/NpmHints.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Do not assign `this` to `self`.

See more on https://sonarcloud.io/project/issues?id=phcode-dev_phoenix&issues=AZ8kAvO-RoBB8p1B2vh5&open=AZ8kAvO-RoBB8p1B2vh5&pullRequest=3010

Check warning on line 649 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=AZ8kAvRURoBB8p1B2viB&open=AZ8kAvRURoBB8p1B2viB&pullRequest=3010

Check warning on line 75 in src/extensionsIntegrated/JSONSupport/NpmHints.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Move function '_highlightMatches' to the outer scope.

See more on https://sonarcloud.io/project/issues?id=phcode-dev_phoenix&issues=AZ8kAvO-RoBB8p1B2vh4&open=AZ8kAvO-RoBB8p1B2vh4&pullRequest=3010

Check warning on line 36 in src/extensionsIntegrated/JSONSupport/NpmHover.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove the declaration of the unused '_' variable.

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

Check warning on line 131 in src/extensionsIntegrated/JSONSupport/NpmRegistry.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=AZ8kAvOhRoBB8p1B2vhg&open=AZ8kAvOhRoBB8p1B2vhg&pullRequest=3010

Check warning on line 50 in src/extensionsIntegrated/JSONSupport/NpmHints.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Move function 'NpmHints' to the outer scope.

See more on https://sonarcloud.io/project/issues?id=phcode-dev_phoenix&issues=AZ8kAvO-RoBB8p1B2vh0&open=AZ8kAvO-RoBB8p1B2vh0&pullRequest=3010

Check warning on line 106 in src/extensionsIntegrated/JSONSupport/NpmHover.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Expected the Promise rejection reason to be an Error.

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

Check warning on line 97 in src/extensionsIntegrated/JSONSupport/NpmHover.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Expected the Promise rejection reason to be an Error.

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

Check warning on line 156 in src/extensionsIntegrated/JSONSupport/NpmRegistry.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=AZ8kAvOhRoBB8p1B2vhj&open=AZ8kAvOhRoBB8p1B2vhj&pullRequest=3010

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

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Move function '_clientCapabilities' to the outer scope.

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

Check warning on line 193 in src/extensionsIntegrated/JSONSupport/JsonLsp.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=AZ8kAvOqRoBB8p1B2vhq&open=AZ8kAvOqRoBB8p1B2vhq&pullRequest=3010

Check warning on line 169 in src/extensionsIntegrated/JSONSupport/NpmHints.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use 'String#startsWith' method instead.

See more on https://sonarcloud.io/project/issues?id=phcode-dev_phoenix&issues=AZ8kAvO-RoBB8p1B2vh7&open=AZ8kAvO-RoBB8p1B2vh7&pullRequest=3010

Check warning on line 172 in src/extensionsIntegrated/JSONSupport/JsonLsp.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=AZ8kAvOqRoBB8p1B2vho&open=AZ8kAvOqRoBB8p1B2vho&pullRequest=3010

Check warning on line 126 in src/extensionsIntegrated/JSONSupport/NpmHover.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Expected the Promise rejection reason to be an Error.

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