Update PII categories for AI Security for Apps model migration - #32825
Conversation
Review
👉 Fix in your agent 👈Fix the following review findings in PR #32825 (https://github.com/cloudflare/cloudflare-docs/pull/32825).
Before making changes, review each finding and present a brief summary table:
- For each finding, state whether you agree, disagree, or need clarification
- If you disagree (e.g. the fix requires disproportionate effort for minimal benefit,
or the finding is factually incorrect), explain why
- If you need clarification before deciding, ask those questions
- Then share your plan for which issues to tackle and in what order
After triaging, follow this order:
1. Post a comment on this PR for any findings you are skipping, with the finding ID and your reasoning.
2. Then commit the fixes for the legitimate findings.
The comment must come before the commit — the bot reads PR comments when a new
push triggers a review, so skip comments posted after the push will be missed.
---
## Code Review
### Warnings (1)
#### CR-118ecf525262 · Category list out of sync with field reference
- **File:** `src/content/docs/waf/detections/ai-security-for-apps/pii-detection.mdx` line 30
- **Issue:** Line 30 points readers to the `cf.llm.prompt.pii_categories` field reference for the full list of categories. The current `src/content/fields/index.yaml` still enumerates the old set (`CRYPTO`, `US_BANK_NUMBER`, `IBAN_CODE`, `US_DRIVER_LICENSE`, `US_ITIN`, `US_PASSPORT`, `NRP`, `UK_NHS`, `UK_NINO`, `ES_NIF`), which does not match the migrated categories shown in the new table. Unless the field reference is also updated, the two pages will contradict each other.
- **Fix:** Update the `cf.llm.prompt.pii_categories` field reference (in `src/content/fields/index.yaml` or the generated page) to the same migrated category set so the linked reference and this table stay consistent.
Code ReviewThis code review is in beta and may not always be helpful — use your judgment. Warnings (1)
ConventionsNo convention issues found. Style Guide ReviewNo style-guide issues found. CommandsOnly codeowners can run commands. Post a comment with the command to trigger it.
|
5febddf to
c263eec
Compare
Replace the 39 Presidio-based PII categories with 13 categories supported by the new NER model. Remove Presidio attribution from docs; update example rules to use BANK_ACCOUNT instead of dropped IBAN_CODE/US_BANK_NUMBER.
c263eec to
2491b4e
Compare
We are updating |
Summary
Updates public documentation to reflect the PII detection model migration from Presidio to a new AI-based Named Entity Recognition (NER) model.
Replace the 39 Presidio-based PII categories with 13 categories supported by the new NER model. Remove Presidio attribution from docs; update example rules to use BANK_ACCOUNT instead of dropped IBAN_CODE/US_BANK_NUMBER.
Screenshots (optional)
Documentation checklist