Skip to content

feat(codehints): language-agnostic "doc comment" generator#2999

Open
abose wants to merge 5 commits into
mainfrom
ai
Open

feat(codehints): language-agnostic "doc comment" generator#2999
abose wants to merge 5 commits into
mainfrom
ai

fix(codehints): TypeScript JSDoc carries names only, not {type}

2eec977
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / SonarCloud Code Analysis failed Jun 30, 2026 in 50s

Quality Gate failed

Annotations

Check warning on line 268 in src/extensions/default/DocCommentHints/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=AZ8ZX4nCLu1dXKlfvxLC&open=AZ8ZX4nCLu1dXKlfvxLC&pullRequest=2999

Check warning on line 434 in src/extensions/default/DocCommentHints/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=AZ8ZOlBFLxoge3EN151a&open=AZ8ZOlBFLxoge3EN151a&pullRequest=2999

Check warning on line 436 in src/extensions/default/DocCommentHints/main.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Do not call `Array#push()` multiple times.

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

Check warning on line 264 in src/extensions/default/DocCommentHints/main.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Replace this character class by the character itself.

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

Check failure on line 530 in src/extensions/default/DocCommentHints/main.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Refactor this function to not always return the same value.

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

Check warning on line 503 in src/extensions/default/DocCommentHints/main.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Move function 'DocCommentHintProvider' to the outer scope.

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

Check warning on line 230 in src/extensions/default/DocCommentHints/main.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Move function '_isDefaultEq' to the outer scope.

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

Check warning on line 444 in src/extensions/default/DocCommentHints/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=AZ8ZOlBFLxoge3EN151c&open=AZ8ZOlBFLxoge3EN151c&pullRequest=2999

Check warning on line 118 in src/extensions/default/DocCommentHints/main.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Move function '_eachTopLevel' to the outer scope.

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

Check warning on line 497 in src/extensions/default/DocCommentHints/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=AZ8ZOlBFLxoge3EN151i&open=AZ8ZOlBFLxoge3EN151i&pullRequest=2999

Check warning on line 164 in src/extensions/default/DocCommentHints/unittests.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Simplify this regular expression to reduce its runtime, as it has super-linear performance due to backtracking.

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

Check warning on line 211 in src/extensions/default/DocCommentHints/main.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Move function '_parenDepth' to the outer scope.

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

Check failure on line 118 in src/extensions/default/DocCommentHints/main.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=AZ8ZX4nCLu1dXKlfvxK9&open=AZ8ZX4nCLu1dXKlfvxK9&pullRequest=2999

Check warning on line 161 in src/extensions/default/DocCommentHints/main.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Move function '_validType' to the outer scope.

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

Check warning on line 455 in src/extensions/default/DocCommentHints/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=AZ8ZOlBFLxoge3EN151g&open=AZ8ZOlBFLxoge3EN151g&pullRequest=2999

Check warning on line 454 in src/extensions/default/DocCommentHints/main.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Move function '_escDesc' to the outer scope.

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

Check warning on line 288 in src/extensions/default/DocCommentHints/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=AZ8ZOk-FLxoge3EN151Q&open=AZ8ZOk-FLxoge3EN151Q&pullRequest=2999

Check warning on line 480 in src/extensions/default/DocCommentHints/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=AZ8ZOlBFLxoge3EN151h&open=AZ8ZOlBFLxoge3EN151h&pullRequest=2999

Check failure on line 503 in src/extensions/default/DocCommentHints/main.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Unexpected empty function 'DocCommentHintProvider'.

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

Check warning on line 215 in src/extensions/default/DocCommentHints/main.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Expected a `for-of` loop instead of a `for` loop with this simple iteration.

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

Check warning on line 226 in src/extensions/default/DocCommentHints/main.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Replace this character class by the character itself.

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

Check warning on line 447 in src/extensions/default/DocCommentHints/main.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Do not call `Array#push()` multiple times.

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

Check warning on line 367 in src/extensions/default/DocCommentHints/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=AZ8ZOlBFLxoge3EN151Z&open=AZ8ZOlBFLxoge3EN151Z&pullRequest=2999

Check warning on line 508 in src/extensions/default/DocCommentHints/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=AZ8ZOlBFLxoge3EN151l&open=AZ8ZOlBFLxoge3EN151l&pullRequest=2999

Check warning on line 207 in src/extensions/default/DocCommentHints/unittests.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=AZ8ZOk-FLxoge3EN151O&open=AZ8ZOk-FLxoge3EN151O&pullRequest=2999