feat: triage workspace + PAT auth + upgrade infrastructure + seed dataset + OpenAPI hardening - #731
feat: triage workspace + PAT auth + upgrade infrastructure + seed dataset + OpenAPI hardening#731aaronlippold wants to merge 714 commits into
Quality Gate failed
Failed conditions
D Reliability Rating on New Code (required ≥ A)
E Security Rating on New Code (required ≥ A)
See analysis details on SonarQube Cloud
Catch issues before they fail your Quality Gate with our IDE extension
SonarQube for IDE
Annotations
Check failure on line 20 in spec/requests/users/initiate_link_spec.rb
sonarqubecloud / SonarCloud Code Analysis
Define a constant instead of duplicating this literal "/users/initiate_link" 5 times.
See more on https://sonarcloud.io/project/issues?id=mitre_vulcan&issues=AZ9IihfvVWwQadk59svs&open=AZ9IihfvVWwQadk59svs&pullRequest=731
Check failure on line 20 in spec/lib/oidc_provider_registry_spec.rb
sonarqubecloud / SonarCloud Code Analysis
Define a constant instead of duplicating this literal "https://org.okta.com/oauth2/default" 5 times.
See more on https://sonarcloud.io/project/issues?id=mitre_vulcan&issues=AZ9IihNOVWwQadk59ssI&open=AZ9IihNOVWwQadk59ssI&pullRequest=731
Check warning on line 168 in spec/javascript/constants/triageVocabulary.spec.js
sonarqubecloud / SonarCloud Code Analysis
Prefer "expect(opts).toHaveLength(2 + nonPending.length)" over this generic assertion for better reporting; it works on any object with a numeric length property.
See more on https://sonarcloud.io/project/issues?id=mitre_vulcan&issues=AZ9Iihc8VWwQadk59svX&open=AZ9Iihc8VWwQadk59svX&pullRequest=731
Check warning on line 109 in spec/javascript/composables/useReplyComposer.spec.js
sonarqubecloud / SonarCloud Code Analysis
Prefer "expect(composerState.value.parentRuleId).toBeNull()" over this generic assertion; dedicated matchers read better and report clearer failures.
See more on https://sonarcloud.io/project/issues?id=mitre_vulcan&issues=AZ9IihNZVWwQadk59ssR&open=AZ9IihNZVWwQadk59ssR&pullRequest=731
Check warning on line 76 in app/controllers/users/omniauth_callbacks_controller.rb
sonarqubecloud / SonarCloud Code Analysis
Move the private methods to the end of the class.
See more on https://sonarcloud.io/project/issues?id=mitre_vulcan&issues=AZ9IihBoVWwQadk59sqz&open=AZ9IihBoVWwQadk59sqz&pullRequest=731
Check warning on line 81 in app/javascript/api/baseApi.js
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=mitre_vulcan&issues=AZ_EHXlKUItUTMcRMMe4&open=AZ_EHXlKUItUTMcRMMe4&pullRequest=731
Check failure on line 205 in spec/requests/components_creation_sources_spec.rb
sonarqubecloud / SonarCloud Code Analysis
Define a constant instead of duplicating this literal "Slack Merge" 3 times.
See more on https://sonarcloud.io/project/issues?id=mitre_vulcan&issues=AZ_EHYRrUItUTMcRMMf0&open=AZ_EHYRrUItUTMcRMMf0&pullRequest=731
Check warning on line 78 in spec/javascript/components/shared/SatisfiedByIndicator.spec.js
sonarqubecloud / SonarCloud Code Analysis
Prefer "expect(buttons).toHaveLength(2)" over this generic assertion for better reporting; it works on any object with a numeric length property.
See more on https://sonarcloud.io/project/issues?id=mitre_vulcan&issues=AZ9dZJWIj-NSMWZNV6ut&open=AZ9dZJWIj-NSMWZNV6ut&pullRequest=731
Check warning on line 67 in eslint-rules/comment-tracker.js
sonarqubecloud / SonarCloud Code Analysis
`String.raw` should be used to avoid escaping `\`.
See more on https://sonarcloud.io/project/issues?id=mitre_vulcan&issues=AZ9IihqgVWwQadk59sw3&open=AZ9IihqgVWwQadk59sw3&pullRequest=731
Check failure on line 6 in app/javascript/stores/theme.js
sonarqubecloud / SonarCloud Code Analysis
Prefer `.dataset` over `getAttribute(…)`.
See more on https://sonarcloud.io/project/issues?id=mitre_vulcan&issues=AZ9Iig_DVWwQadk59sqc&open=AZ9Iig_DVWwQadk59sqc&pullRequest=731
Check warning on line 189 in app/javascript/components/rules/forms/CheckForm.vue
sonarqubecloud / SonarCloud Code Analysis
Extract this nested ternary operation into an independent statement.
See more on https://sonarcloud.io/project/issues?id=mitre_vulcan&issues=AZ9hzi1X4Zz5JRbUToe_&open=AZ9hzi1X4Zz5JRbUToe_&pullRequest=731
Check failure on line 14 in spec/models/concerns/version_sortable_spec.rb
sonarqubecloud / SonarCloud Code Analysis
Define a constant instead of duplicating this literal "<xml/>" 3 times.
See more on https://sonarcloud.io/project/issues?id=mitre_vulcan&issues=AZ9IihIgVWwQadk59srR&open=AZ9IihIgVWwQadk59srR&pullRequest=731
Check warning on line 50 in spec/models/serialized_collection_ordering_spec.rb
sonarqubecloud / SonarCloud Code Analysis
Replace 'pluck(:id)' with the more semantic 'ids' method.
See more on https://sonarcloud.io/project/issues?id=mitre_vulcan&issues=AZ9ZLrq2qB1O_tn3XkO9&open=AZ9ZLrq2qB1O_tn3XkO9&pullRequest=731
Check failure on line 10 in spec/requests/api/api_filterable_spec.rb
sonarqubecloud / SonarCloud Code Analysis
Define a constant instead of duplicating this literal "Alpha Project" 3 times.
See more on https://sonarcloud.io/project/issues?id=mitre_vulcan&issues=AZ_EHYN8UItUTMcRMMfq&open=AZ_EHYN8UItUTMcRMMfq&pullRequest=731
Check warning on line 10 in app/javascript/stores/ruleSelection.js
sonarqubecloud / SonarCloud Code Analysis
Move function '_readStorage' to the outer scope.
See more on https://sonarcloud.io/project/issues?id=mitre_vulcan&issues=AZ9Iig-6VWwQadk59sqa&open=AZ9Iig-6VWwQadk59sqa&pullRequest=731
Check warning on line 28 in app/javascript/composables/useFormFeedback.js
sonarqubecloud / SonarCloud Code Analysis
Use 'Object.hasOwn()' instead of 'Object.prototype.hasOwnProperty.call()'.
See more on https://sonarcloud.io/project/issues?id=mitre_vulcan&issues=AZ9Iig9TVWwQadk59sqH&open=AZ9Iig9TVWwQadk59sqH&pullRequest=731
Check warning on line 423 in spec/javascript/composables/useRuleNavigation.spec.js
sonarqubecloud / SonarCloud Code Analysis
Prefer "expect(filteredRules.value).toHaveLength(1)" over this generic assertion for better reporting; it works on any object with a numeric length property.
See more on https://sonarcloud.io/project/issues?id=mitre_vulcan&issues=AZ9hziWV4Zz5JRbUToe8&open=AZ9hziWV4Zz5JRbUToe8&pullRequest=731
Check failure on line 24 in spec/helpers/vue_props_helper_spec.rb
sonarqubecloud / SonarCloud Code Analysis
Define a constant instead of duplicating this literal "v-bind:statuses" 4 times.
See more on https://sonarcloud.io/project/issues?id=mitre_vulcan&issues=AZ9hzisW4Zz5JRbUToe-&open=AZ9hzisW4Zz5JRbUToe-&pullRequest=731
Check failure on line 44 in spec/requests/api/components_spec.rb
sonarqubecloud / SonarCloud Code Analysis
Define a constant instead of duplicating this literal "/api/components/latest" 7 times.
See more on https://sonarcloud.io/project/issues?id=mitre_vulcan&issues=AZ9SA8zZOsIJlDz0ajEW&open=AZ9SA8zZOsIJlDz0ajEW&pullRequest=731
Check warning on line 54 in app/models/security_requirements_guide.rb
sonarqubecloud / SonarCloud Code Analysis
Replace 'pluck(:id)' with the more semantic 'ids' method.
See more on https://sonarcloud.io/project/issues?id=mitre_vulcan&issues=AZ9IihD1VWwQadk59sq5&open=AZ9IihD1VWwQadk59sq5&pullRequest=731
Check warning on line 95 in spec/javascript/components/triage/CommentProgressBar.spec.js
sonarqubecloud / SonarCloud Code Analysis
Prefer "expect(segments).toHaveLength(5)" over this generic assertion for better reporting; it works on any object with a numeric length property.
See more on https://sonarcloud.io/project/issues?id=mitre_vulcan&issues=AZ9IihVrVWwQadk59suC&open=AZ9IihVrVWwQadk59suC&pullRequest=731
Check failure on line 18 in spec/contracts/devise_auth_flows_contract_spec.rb
sonarqubecloud / SonarCloud Code Analysis
Define a constant instead of duplicating this literal "/users/password" 4 times.
See more on https://sonarcloud.io/project/issues?id=mitre_vulcan&issues=AZ9IihlhVWwQadk59swl&open=AZ9IihlhVWwQadk59swl&pullRequest=731
Check failure on line 220 in spec/requests/components_creation_sources_spec.rb
sonarqubecloud / SonarCloud Code Analysis
Define a constant instead of duplicating this literal "Slack Only" 3 times.
See more on https://sonarcloud.io/project/issues?id=mitre_vulcan&issues=AZ_EHYRrUItUTMcRMMf7&open=AZ_EHYRrUItUTMcRMMf7&pullRequest=731
Check warning on line 157 in spec/javascript/components/benchmarks/RuleDetails.spec.js
sonarqubecloud / SonarCloud Code Analysis
Prefer "expect(textareas).toHaveLength(4)" over this generic assertion for better reporting; it works on any object with a numeric length property.
See more on https://sonarcloud.io/project/issues?id=mitre_vulcan&issues=AZ9IihbXVWwQadk59svI&open=AZ9IihbXVWwQadk59svI&pullRequest=731
Check warning on line 13 in app/javascript/utilities/markdownPreprocessor.js
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=mitre_vulcan&issues=AZ9Iig_YVWwQadk59sqg&open=AZ9Iig_YVWwQadk59sqg&pullRequest=731