refactor: updates to quality QA mechanics to improve utility#171
refactor: updates to quality QA mechanics to improve utility#171asteier2026 wants to merge 4 commits into
Conversation
Greptile SummaryThis PR refactors the QA mechanics in the rewrite pipeline to improve utility by adding
Confidence Score: 5/5Safe to merge — changes are well-scoped, all existing validation contracts are preserved, and new behavior is gated correctly behind protection method checks. All three modified pipeline stages handle the new generalization_suggestion field consistently: the schema provides a safe default, pre-validation correction prevents spurious validation failures, and the field is conditionally forwarded only when semantically relevant. Test coverage is comprehensive. No files require special attention. Important Files Changed
Flowchart%%{init: {'theme': 'neutral'}}%%
flowchart TD
A[LLM: Sensitivity Disposition] --> B[parse_sensitivity_disposition]
B --> C{_correct_disposition_consistency}
C -->|low + non-leave_as_is| D[Force leave_as_is]
C -->|Valid| E[model_validate]
D --> E
E --> F[_format_disposition_block]
G[COL_LATENT_ENTITIES] --> F
F -->|generalize| H[Add generalization_suggestion]
F -->|suppress_inference| I[Add evidence]
F -->|other| J[Base entry only]
H --> K[Meaning Unit LLM]
I --> K
J --> K
E --> L[_format_rewrite_disposition_block]
L -->|generalize| M[Add generalization_suggestion]
L -->|other protected| N[Base fields only]
M --> O[Rewrite LLM]
N --> O
O --> P[Rewritten Text]
P --> Q[Quality Re-answer with abstraction tolerance]
Q --> R[Quality Score]
Reviews (4): Last reviewed commit: "fix: misc greptile suggestions" | Re-trigger Greptile |
Co-authored-by: greptile-apps[bot] <165735046+greptile-apps[bot]@users.noreply.github.com>
Changes include: