Review runner: propagate PII abstention (parse_error) into the PII lane
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 3/5
- Tempo stimato
- 1-2 giorni
- Idoneità per principianti
- 67/100
- Tipo di issue
- Bug
- Chiarezza
- Abbastanza chiara
- Stato di attività
- Tranquilla
- Stack tecnologico
- typescript
- Ambito
- backend, data-engineering, security
Direzione di ricerca
Individua ReviewRunner.detectPii, il runner dispatcher che gestisce checkQueryPii e il percorso PII dell’orchestrator. Traccia come pii.parse_error diventa attualmente un risultato di colonne vuoto, quindi verifica che i file con astensione vengano contrassegnati come non verificati, mentre i file verificati come puliti mantengano il comportamento esistente; aggiungi o esegui i test pertinenti di review-runner e dell’orchestrator, se presenti.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
Follow-up from PR #1090 review (Codex P1, comment 3768687908): when the composite check's checkQueryPii abstains (data.pii.parse_error), the review runner's detectPii returns { columns: [] }, indistinguishable from a verified-clean result, and the PII lane treats the model as clean.
Practical exposure is currently nil — an unparseable model already fails the review loudly via the validation/lint lanes, and the abstention is visible in the composite output and fails closed in the CLI/tool paths — but for the "degrade loudly" contract the lane should know the difference.
Scope: extend ReviewRunner.detectPii to return an optional abstained flag (optional field, non-breaking for mocks), set it from pii.parse_error in the dispatcher runner, and have the orchestrator PII lane mark PII checks as unverified (rather than clean) for that file.
- Lingua principale
- TypeScript
- Stelle
- 813
- Fork
- 134
- Merge medio
- 2g 2h
- PR unite (30g)
- 67
Guida per i contributori
Apri la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di AltimateAI/altimate-code
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 70/100
AltimateAI/altimate-code#1359 ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 84/100
AltimateAI/altimate-code#1323 ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 86/100
AltimateAI/altimate-code#1288 ·
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 92/100
AltimateAI/altimate-code#1285 ·
-
privacy: Altimate Base consent dialog no longer discloses persistent per-installation identifier Aperta
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 88/100
AltimateAI/altimate-code#1284 ·
Tutte le issue di AltimateAI/altimate-code
Issue simili
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 84/100
bcgov/bc-wallet-mobile#4761 · 1 commento ·
-
external-issue to-triage
Difficoltà 2/5 1-3 ore Idoneità per principianti 88/100
-
area-deployment area-integrations triage:bot-seen
Difficoltà 2/5 Mezza giornata Idoneità per principianti 86/100
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
-
refactor
Difficoltà 2/5 1-3 ore Idoneità per principianti 84/100