Drive marker PII sanitization from schemas

[fatadel]

Main | Deploy preview

Markers requiring PII sanitization were identified through hardcoded data.type checks. Putting PII directives in marker schema fields removes type-specific branching and makes sanitization generic.

Closes #6290

---

Profile

Published using Prod | Published using Deploy Preview

[codecov]

Codecov Report

❌ Patch coverage is 93.33333% with 6 lines in your changes missing coverage. Please review.
✅ Project coverage is 83.87%. Comparing base (7abc8b8) to head (8942af7).
⚠️ Report is 42 commits behind head on main.

Files with missing lines	Patch %	Lines
src/profile-logic/marker-data.ts	91.04%	6 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #6291      +/-   ##
==========================================
+ Coverage   83.82%   83.87%   +0.05%     
==========================================
  Files         350      353       +3     
  Lines       37744    37888     +144     
  Branches    10646    10687      +41     
==========================================
+ Hits        31638    31779     +141     
- Misses       5677     5680       +3     
  Partials      429      429              

☔ View full report in Codecov by Harness.
📢 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.

[canova]

Thanks a lot! I have some high level schema related question, let me know what you think!

[mstange]

I haven't looked at the code yet but I'd like to point out #5301 which has a related proposal.

[canova]

@mstange ah thanks! I forgot that this was filed. Structurally it's actually very similar to what I suggested (minus the naming differences) but I forgot the sanitized-string one. It looks nice

[fatadel]

@mstange ah thanks! I forgot that this was filed. Structurally it's actually very similar to what I suggested (minus the naming differences) but I forgot the sanitized-string one. It looks nice

Let's handle sanitized-string in that issue then and keep this one's scope to the one we already defined?

[fatadel]

Thanks for the review, @canova! I've addressed the issues now.

[fatadel]

Thanks for the feedback, @mstange! Done now ✅

[mstange]

I don't see an upgrader; what happens if you re-upload an existing old uploaded profile and uncheck some checkboxes in the re-upload?

[mstange]

Also does there need to be code in profile processing which adds these annotations when converting from a Gecko profiler?