Clarify package id parsing and AutofillView creation

[david-livefront]

🎟️ Tracking

N/A

📔 Objective

This PR is intended to improve clarity of the Autofill ViewNode traversal logic without making any functional changes.

⏰ 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 @david-livefront's task —— View job

---

Review Summary

Overall Assessment: APPROVE

This refactoring successfully improves code clarity without functional changes.

---

[david-livefront]

I have lifted the blank and android check here to simplify the logic below. Additionally I have renamed the local idPackage to storedIdPackage so that it does not conflict with the property on the ViewNode.

[github-actions]

Checkmarx One – Scan Summary & Details – 1112f4d0-c1d7-49ca-9f2c-c4587061c96a

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

[david-livefront]

Thanks @SaintPatrck

[codecov]

Codecov Report

❌ Patch coverage is 86.36364% with 3 lines in your changes missing coverage. Please review.
✅ Project coverage is 84.97%. Comparing base (9baec6e) to head (cb80473).
⚠️ Report is 3 commits behind head on main.

Files with missing lines	Patch %	Lines
...bitwarden/data/autofill/util/ViewNodeExtensions.kt	88.88%	0 Missing and 2 partials ⚠️
...twarden/data/autofill/parser/AutofillParserImpl.kt	75.00%	0 Missing and 1 partial ⚠️

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #6138   +/-   ##
=======================================
  Coverage   84.97%   84.97%           
=======================================
  Files         723      723           
  Lines       52746    52740    -6     
  Branches     7649     7649           
=======================================
- Hits        44822    44817    -5     
+ Misses       5250     5249    -1     
  Partials     2674     2674           

☔ 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.