Skip to content

Split PR #522 [4/9]: Admin scripts package #560

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Draft
seanspeaks wants to merge 1 commit into
base: next
Choose a base branch
from

feat(admin-scripts): extract admin-scripts package from PR #522

a3764cb
Select commit
Loading
Failed to load commit list.
Draft

Split PR #522 [4/9]: Admin scripts package #560

feat(admin-scripts): extract admin-scripts package from PR #522
a3764cb
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / SonarCloud Code Analysis failed Mar 30, 2026 in 1m 8s

Quality Gate failed

Failed conditions
1 Security Hotspot

See analysis details on SonarQube Cloud

Annotations

Check warning on line 122 in packages/admin-scripts/src/application/dry-run-repository-wrapper.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

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

See more on https://sonarcloud.io/project/issues?id=friggframework_frigg&issues=AZ09DtvzBfEcQZ3_bVY3&open=AZ09DtvzBfEcQZ3_bVY3&pullRequest=560

Check failure on line 71 in packages/admin-scripts/src/builtins/integration-health-check.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

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

See more on https://sonarcloud.io/project/issues?id=friggframework_frigg&issues=AZ09DtwJBfEcQZ3_bVY5&open=AZ09DtwJBfEcQZ3_bVY5&pullRequest=560

Check failure on line 49 in packages/admin-scripts/src/application/validate-script-input.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

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

See more on https://sonarcloud.io/project/issues?id=friggframework_frigg&issues=AZ09DtvgBfEcQZ3_bVYx&open=AZ09DtvgBfEcQZ3_bVYx&pullRequest=560

Check failure on line 50 in packages/admin-scripts/src/application/dry-run-http-interceptor.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

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

See more on https://sonarcloud.io/project/issues?id=friggframework_frigg&issues=AZ09DtvqBfEcQZ3_bVYy&open=AZ09DtvqBfEcQZ3_bVYy&pullRequest=560

Check warning on line 67 in packages/admin-scripts/src/application/dry-run-repository-wrapper.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=friggframework_frigg&issues=AZ09DtvzBfEcQZ3_bVY0&open=AZ09DtvzBfEcQZ3_bVY0&pullRequest=560

Check warning on line 118 in packages/admin-scripts/src/application/dry-run-repository-wrapper.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

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

See more on https://sonarcloud.io/project/issues?id=friggframework_frigg&issues=AZ09DtvzBfEcQZ3_bVY2&open=AZ09DtvzBfEcQZ3_bVY2&pullRequest=560

Check warning on line 16 in packages/admin-scripts/src/application/__tests__/script-runner.test.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove the declaration of the unused 'testScript' variable. 

See more on https://sonarcloud.io/project/issues?id=friggframework_frigg&issues=AZ09DtsTBfEcQZ3_bVYt&open=AZ09DtsTBfEcQZ3_bVYt&pullRequest=560

Check warning on line 28 in packages/admin-scripts/src/application/script-factory.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=friggframework_frigg&issues=AZ09Dtv9BfEcQZ3_bVY4&open=AZ09Dtv9BfEcQZ3_bVY4&pullRequest=560

Check warning on line 222 in packages/admin-scripts/src/application/__tests__/dry-run-repository-wrapper.test.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove the declaration of the unused 'list' variable. 

See more on https://sonarcloud.io/project/issues?id=friggframework_frigg&issues=AZ09DtvXBfEcQZ3_bVYv&open=AZ09DtvXBfEcQZ3_bVYv&pullRequest=560

Check warning on line 32 in packages/admin-scripts/src/application/__tests__/script-factory.test.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Unexpected empty class. 

See more on https://sonarcloud.io/project/issues?id=friggframework_frigg&issues=AZ09DtvNBfEcQZ3_bVYu&open=AZ09DtvNBfEcQZ3_bVYu&pullRequest=560

Check failure on line 39 in packages/admin-scripts/src/application/dry-run-repository-wrapper.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

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

See more on https://sonarcloud.io/project/issues?id=friggframework_frigg&issues=AZ09DtvzBfEcQZ3_bVYz&open=AZ09DtvzBfEcQZ3_bVYz&pullRequest=560

Check warning on line 222 in packages/admin-scripts/src/application/__tests__/dry-run-repository-wrapper.test.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this useless assignment to variable "list". 

See more on https://sonarcloud.io/project/issues?id=friggframework_frigg&issues=AZ09DtvXBfEcQZ3_bVYw&open=AZ09DtvXBfEcQZ3_bVYw&pullRequest=560
View more details on SonarQubeCloud