[PM-36] - [Tech Debt] Move SearchService to libs/common/vault
đī¸ Tracking
https://bitwarden.atlassian.net/browse/PM-36
đ Objective
This PR moves the search service to libs/common/vault which ensures vault code ownership.
đ¸ 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
Checkmarx One â Scan Summary & Details â 5be1b936-f017-425e-8957-b2a789cf6328
Great job, no security vulnerabilities found in this Pull Request
@djsmith85 @BTreston @mzieniukbw @nick-livefront Had to resolve a merge conflict so in GH's infinite wisdom all your reviews have been marked stale.
Quality Gate passed
Issues
0 New issues
0 Accepted issues
Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code
Codecov Report
Attention: Patch coverage is 20.00000% with 16 lines in your changes missing coverage. Please review.
Project coverage is 36.86%. Comparing base (
95841eb) to head (0a51221). Report is 36 commits behind head on main.
:white_check_mark: All tests successful. No failed tests found.
Additional details and impacted files
@@ Coverage Diff @@
## main #15251 +/- ##
==========================================
- Coverage 36.90% 36.86% -0.05%
==========================================
Files 3226 3228 +2
Lines 93225 93388 +163
Branches 14029 14056 +27
==========================================
+ Hits 34404 34426 +22
- Misses 57397 57536 +139
- Partials 1424 1426 +2
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
:rocket: New features to boost your workflow:
- :package: JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.