[PM-22372] - remove feature end user activation mvp flag

[jaasen-livefront]

🎟️ Tracking

https://bitwarden.atlassian.net/browse/PM-22372

📔 Objective

📸 Screenshots

⏰ Reminders before review

• Contributor guidelines followed
• All formatters and local linters executed and passed
• Written new unit and / or integration tests where applicable
• Protected functional changes with optionality (feature flags)
• Used internationalization (i18n) for all UI strings
• CI builds passed
• Communicated to DevOps any deployment requirements
• Updated any necessary documentation (Confluence, contributing docs) or informed the documentation team

🦮 Reviewer guidelines

• 👍 (:+1:) or similar for great changes
• 📝 (:memo:) or ℹ️ (:information_source:) for notes or general info
• ❓ (:question:) for questions
• 🤔 (:thinking:) or 💭 (:thought_balloon:) for more open inquiry that's not quite a confirmed issue and could potentially benefit from discussion
• 🎨 (:art:) for suggestions / improvements
• ❌ (:x:) or ⚠️ (:warning:) for more significant problems or concerns needing attention
• 🌱 (:seedling:) or ♻️ (:recycle:) for future improvements or indications of technical debt
• ⛏ (:pick:) for minor or nitpick changes

[claude]

Claude finished @jaasen-livefront's task —— View job

---

Code Review Complete

Tasks

• Read changed file and understand context
• Review the PR changes using bitwarden-code-reviewer
• Post review findings

---

Review Summary

Overall Assessment: ✅ APPROVE

This PR safely removes the unused feature flag PM19315EndUserActivationMvp from the Constants.cs file.

Findings

✅ Safe to Remove

The feature flag PM19315EndUserActivationMvp (value: "pm-19315-end-user-activation-mvp") has been verified to have no references in the codebase. The flag was never utilized, making this a safe cleanup operation.

✅ No Breaking Changes

• No code dependencies on this constant
• No configuration or runtime impact
• Clean removal from the FeatureFlagKeys class in src/Core/Constants.cs:250

✅ Security & Quality

• No security implications
• Change aligns with code hygiene best practices (removing unused code)
• Proper location within the Vault Team section of feature flags

Recommendation

This is a straightforward cleanup PR that removes technical debt. The change is safe to merge.

---

[github-actions]

Checkmarx One – Scan Summary & Details – 3a856144-c92a-46f4-aec2-994c1de9e126

Great job! No new security vulnerabilities introduced in this pull request

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 53.60%. Comparing base (b18506a) to head (73c2b59).
⚠️ Report is 4 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #6696      +/-   ##
==========================================
- Coverage   53.61%   53.60%   -0.02%     
==========================================
  Files        1921     1921              
  Lines       85650    85650              
  Branches     7687     7687              
==========================================
- Hits        45922    45912      -10     
- Misses      37956    37966      +10     
  Partials     1772     1772              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.