chore: cleanup unused props/methods in Vue components
-
Please check this box if this contribution uses AI-generated content (including content generated by GitLab Duo features) as outlined in the GitLab DCO & CLA. As a benefit of being a GitLab Community Contributor, you receive complimentary access to GitLab Duo.
Title: chore(vue): cleanup unused props in security reports components
What does this MR do and why? This MR removes unused or unnecessary props in several Vue components as part of the vue/no-unused-properties cleanup plan. It also documents and preserves a valid false positive with proper inline ESLint annotation.
Removed unused props and methods from the following components after verifying no usage across the codebase and ensuring test suites pass.
Summary of changes:
- code_flow_steps_section.vue
-
✅ Removed unused props: description, descriptionHtml -
🧠 Other props were validated and retained as needed.
-
- vulnerability_code_flow.vue
-
✅ Removed unused props: description, descriptionHtml -
✅ Updated component usage accordingly to avoid passing now-removed props.
-
- epic_token.vue
-
✅ Confirmed currentValue prop is unused across template, script, and all parent component contexts. -
✅ Removed from definition.
-
- card_security_discover_app.vue
-
✅ Removed group prop: confirmed unused and not injected/bound from parent.
-
- security_training_promo.vue
-
⚠️ False positive: projectFullPath is not directly used inside the component, but it is passed via INJECT from parent security_training_promo_banner.vue. -
✅ Retained and annotated with inline ESLint directive to avoid mis-flagging: // eslint-disable-next-line vue/no-unused-properties -- used in the parent component, false positive.
-
References
- Related to: eslint-plugin-vue/no-unused-properties
- Internal cleanup task ID: [#525827 (closed)]
Screenshots or screen recordings It is not applicable for this issue – no UI changes.
How to set up and validate locally
- Checkout this branch
- Run yarn jest – ensure all relevant tests pass
- Manually verify that all parent components and affected files render as expected
- Optionally run the linter to confirm vue/no-unused-properties no longer flags updated files
MR acceptance checklist
-
✅ Code cleanup matches linting rules -
✅ False positives properly annotated with inline comments -
✅ No regressions or behavioral changes -
✅ Tests and builds pass
Kindly let me know if there are any further corrections, perhaps items I missed so I can rectify them. Thanks again.
Edited by 🤖 GitLab Bot 🤖