Spaces:
Running
Running
solidprivacy-nl commited on
Commit ·
c5e7765
1
Parent(s): 031c8c6
Record AI-output reinsert workflow review
Browse files- CHANGELOG.md +94 -1
CHANGELOG.md
CHANGED
|
@@ -26,6 +26,98 @@ For UI/UX-only work, prefer pure helper modules and tests before touching Stream
|
|
| 26 |
|
| 27 |
---
|
| 28 |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| 29 |
## v13.3 — Deterministic reinsert UI app verification closeout
|
| 30 |
|
| 31 |
Status: completed and app-verified after Actions/sync verification.
|
|
@@ -245,5 +337,6 @@ Outcome:
|
|
| 245 |
|
| 246 |
Possible directions:
|
| 247 |
|
| 248 |
-
-
|
|
|
|
| 249 |
- Further recognizer expansion by legal domain.
|
|
|
|
| 26 |
|
| 27 |
---
|
| 28 |
|
| 29 |
+
## WP9 — AI-output / document reinsert workflow UX and architecture review
|
| 30 |
+
|
| 31 |
+
Status: completed; review-only workpackage.
|
| 32 |
+
|
| 33 |
+
Purpose:
|
| 34 |
+
|
| 35 |
+
- Decide what Scrub should do next for AI-output and document-level reinsert before implementation starts.
|
| 36 |
+
- Challenge whether pasted-text reinsert is enough.
|
| 37 |
+
- Challenge whether direct DOCX/PDF reinsert should be added immediately.
|
| 38 |
+
- Challenge whether anonymization and de-anonymization should remain in one combined long screen.
|
| 39 |
+
- Recommend a model architecture, product direction, tactical sequence, operational safety position and visual/UX direction.
|
| 40 |
+
|
| 41 |
+
Files added or changed:
|
| 42 |
+
|
| 43 |
+
- Added `AI_OUTPUT_REINSERT_WORKFLOW_REVIEW.md`.
|
| 44 |
+
- Changed `WORKPACKAGES.md`.
|
| 45 |
+
- Changed `CHANGELOG.md`.
|
| 46 |
+
- Added `handover/workpackages/20260608_0000_ai_output_reinsert_workflow_review.md`.
|
| 47 |
+
|
| 48 |
+
Main recommendation:
|
| 49 |
+
|
| 50 |
+
- Keep pasted-text reinsert as a safe baseline and fallback, but do not treat it as the final legal-document workflow.
|
| 51 |
+
- Move toward a two-mode interface:
|
| 52 |
+
- `Anonimiseren`;
|
| 53 |
+
- `Originele waarden terugzetten`.
|
| 54 |
+
- Add document-level reinsert in phases.
|
| 55 |
+
- Prioritize TXT and DOCX before PDF.
|
| 56 |
+
- Keep PDF as investigation/reliability-review work only for now.
|
| 57 |
+
- Keep all reinsert behavior local-only, deterministic and helper-first.
|
| 58 |
+
- Do not add AI calls.
|
| 59 |
+
- Do not add cloud processing.
|
| 60 |
+
|
| 61 |
+
First three obvious ideas challenged:
|
| 62 |
+
|
| 63 |
+
1. `Keep only pasted-text reinsert` was limited: useful fallback, not enough for final legal-document workflow because of usability burden, formatting loss, legal workflow friction, incomplete-copy risk and poor DOCX/PDF fit.
|
| 64 |
+
2. `Add direct PDF/DOCX reinsert immediately` was rejected as too broad: DOCX is feasible but needs careful replacement; PDF is high-risk because of extraction, layout reconstruction, scanned documents, OCR and metadata hygiene.
|
| 65 |
+
3. `Add one combined screen for everything` was rejected as main UX direction: it creates cognitive overload and mixes anonymization with reversible de-anonymization risk.
|
| 66 |
+
|
| 67 |
+
Recommended implementation sequence:
|
| 68 |
+
|
| 69 |
+
1. Keep current text-paste reinsert and improve guidance when the UI is next touched.
|
| 70 |
+
2. Add TXT upload/download reinsert.
|
| 71 |
+
3. Add DOCX reinsert helper, pure helper + tests.
|
| 72 |
+
4. Add DOCX reinsert UI only after helper validation.
|
| 73 |
+
5. Investigate PDF text extraction only.
|
| 74 |
+
6. Consider PDF output only after a reliability review.
|
| 75 |
+
|
| 76 |
+
Next recommended implementation workpackage:
|
| 77 |
+
|
| 78 |
+
```text
|
| 79 |
+
WP10 — v13.4 TXT/DOCX reinsert foundation helper and tests
|
| 80 |
+
```
|
| 81 |
+
|
| 82 |
+
Recommended WP10 boundaries:
|
| 83 |
+
|
| 84 |
+
- helper/test-first;
|
| 85 |
+
- no UI changes;
|
| 86 |
+
- no edit to `fix_streamlit_nested_expanders.py`;
|
| 87 |
+
- no edit to `presidio_streamlit.py`;
|
| 88 |
+
- no existing export/download behavior changes;
|
| 89 |
+
- no Scrub Key export/import behavior changes;
|
| 90 |
+
- no PDF reinsert implementation;
|
| 91 |
+
- no AI calls;
|
| 92 |
+
- no cloud processing;
|
| 93 |
+
- synthetic data only.
|
| 94 |
+
|
| 95 |
+
Testing and validation:
|
| 96 |
+
|
| 97 |
+
- Tests: not applicable; planning/review-only workpackage.
|
| 98 |
+
- App verification: not applicable; no UI behavior changed.
|
| 99 |
+
- GitHub Actions: not checked for this docs-only change at handover time.
|
| 100 |
+
- Hugging Face sync: not checked for this docs-only change at handover time.
|
| 101 |
+
|
| 102 |
+
Intentionally not changed:
|
| 103 |
+
|
| 104 |
+
- No code files changed.
|
| 105 |
+
- No tests changed.
|
| 106 |
+
- No UI changed.
|
| 107 |
+
- No export/download behavior changed.
|
| 108 |
+
- No Scrub Key export/import behavior changed.
|
| 109 |
+
- No TXT/DOCX/PDF reinsert implementation added.
|
| 110 |
+
- No AI calls added.
|
| 111 |
+
- No cloud processing added.
|
| 112 |
+
- No secrets, tokens or real personal data stored.
|
| 113 |
+
|
| 114 |
+
Outcome:
|
| 115 |
+
|
| 116 |
+
- WP9 is complete.
|
| 117 |
+
- Product direction for reinsert is now documented before implementation.
|
| 118 |
+
|
| 119 |
+
---
|
| 120 |
+
|
| 121 |
## v13.3 — Deterministic reinsert UI app verification closeout
|
| 122 |
|
| 123 |
Status: completed and app-verified after Actions/sync verification.
|
|
|
|
| 337 |
|
| 338 |
Possible directions:
|
| 339 |
|
| 340 |
+
- TXT/DOCX document-level reinsert foundation.
|
| 341 |
+
- Two-mode UI planning for `Anonimiseren` and `Originele waarden terugzetten`.
|
| 342 |
- Further recognizer expansion by legal domain.
|