Skip to content

[PM-39009] Do not do heuristic filtering on targeted fields#21245

Merged
jprusik merged 1 commit into
mainfrom
pm-39009
Jun 15, 2026
Merged

[PM-39009] Do not do heuristic filtering on targeted fields#21245
jprusik merged 1 commit into
mainfrom
pm-39009

Conversation

@jprusik

@jprusik jprusik commented Jun 12, 2026

Copy link
Copy Markdown
Contributor

🎟️ Tracking

PM-39009

📔 Objective

Note: split from #20970

Do not do heuristic filtering on targeted fields

@jprusik jprusik requested a review from audreyality June 12, 2026 22:01
@jprusik jprusik self-assigned this Jun 12, 2026
@jprusik jprusik added the ai-review-vnext Request a Claude code review using the vNext workflow label Jun 12, 2026
@github-actions

github-actions Bot commented Jun 12, 2026

Copy link
Copy Markdown
Contributor

🤖 Bitwarden Claude Code Review

Overall Assessment: APPROVE

Reviewed the change to fillGeneratedPassword in overlay.background.ts, which widens the new-password field filter so targeted fields qualified as newPassword by targeting rules bypass the heuristic isNewPasswordField check. The change is additive — non-targeted fields retain existing heuristic behavior — and the strict field.targeted === true guard plus string-constant comparison are sound. The accompanying spec covers the confirmPassword case that the heuristic misses. No security, correctness, or breaking-change concerns found.

Code Review Details

No findings.

@sonarqubecloud

Copy link
Copy Markdown

@jprusik jprusik marked this pull request as ready for review June 12, 2026 22:09
@jprusik jprusik requested a review from a team as a code owner June 12, 2026 22:09
@codecov

codecov Bot commented Jun 12, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 49.22%. Comparing base (3ca5564) to head (3e08225).
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@           Coverage Diff           @@
##             main   #21245   +/-   ##
=======================================
  Coverage   49.22%   49.22%           
=======================================
  Files        4054     4054           
  Lines      126968   126969    +1     
  Branches    19360    19361    +1     
=======================================
+ Hits        62496    62497    +1     
- Misses      59866    59867    +1     
+ Partials     4606     4605    -1     

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

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@bw-ghapp

bw-ghapp Bot commented Jun 12, 2026

Copy link
Copy Markdown
Contributor

Changes in this PR impact the Autofill experience of the browser client

BIT has tested the core experience with these changes and the feature flag configuration used by vault.bitwarden.com.

✅ Fortunately, these BIT tests have passed! 🎉

@bw-ghapp

bw-ghapp Bot commented Jun 12, 2026

Copy link
Copy Markdown
Contributor

Changes in this PR impact the Autofill experience of the browser client

BIT has tested the core experience with these changes and all feature flags disabled.

✅ Fortunately, these BIT tests have passed! 🎉

@jprusik jprusik enabled auto-merge (squash) June 12, 2026 22:35
@jprusik jprusik merged commit 4020f94 into main Jun 15, 2026
74 of 77 checks passed
@jprusik jprusik deleted the pm-39009 branch June 15, 2026 12:35
@djsmith85 djsmith85 added browser Browser Extension t:bugfix Change Type - Bugfix labels Jun 24, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ai-review-vnext Request a Claude code review using the vNext workflow browser Browser Extension t:bugfix Change Type - Bugfix

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants